unity-jar-resolver icon indicating copy to clipboard operation
unity-jar-resolver copied to clipboard

[FR] Abort build if pod install failed

Open sp-jordi-vilalta opened this issue 4 years ago • 3 comments

Feature proposal

  • EDM4U Component: iOS Resolver

When pod install fails, the iOS Resolver only shows a log message but the build continues: https://github.com/googlesamples/unity-jar-resolver/blob/822d341110f9ecf770f872fdad5f98cbb0531d8b/source/IOSResolver/src/IOSResolver.cs#L2567 Ideally I would like the build to abort because, as the log message says, This will will likely result in an non-functional Xcode project (double "will" BTW), and continuing the process is most likely just a waste of time.

Would it be possible to convert the log into an exception (BuildFailedException?) that aborts the build?

In case you think there's some use case where people may want to continue the build, could we have it as an option? Alternatively, adding a way to query the result of this step would allow us to abort in a later custom step.

sp-jordi-vilalta avatar Apr 07 '21 17:04 sp-jordi-vilalta

This issue does not seem to follow the issue template. Make sure you provide all the required information.

google-oss-bot avatar Apr 07 '21 17:04 google-oss-bot

Hi @sp-jordi-vilalta,

Thanks for reporting this. I'll be filing a request for this internally so that the team can take a look at this possible feature. I can't give you any timelines for updates regarding this, so I'll leave this open.

paulinon avatar Apr 08 '21 09:04 paulinon

Thank you @paulinon! If you can confirm that the exception would be an acceptable solution I could probably try the change and open a PR.

sp-jordi-vilalta avatar Apr 08 '21 14:04 sp-jordi-vilalta