[native_assets_builder] Stop throwing from BuildRunner

[dcharkes]

Closes #106.

• Streams errors to the logger.
• Returns a success boolean.

Closes #111.

• Deletes build_output.yaml to prevent caching on build failures. (See issue for alternative solutions.)

[coveralls]

coverage: 98.04% (-0.6%) from 98.649%
when pulling 6bced48 on build-result-errors
into 3f26f20 on main.

[mkustermann]

@dcharkes given you want to rely on logging as side-channel for any build related info (what is being build, whether it was successful or not, reproduction instructions, timings, ...) it seems this PR isn't needed, is it?

[dcharkes]

@dcharkes given you want to rely on logging as side-channel for any build related info (what is being build, whether it was successful or not, reproduction instructions, timings, ...) it seems this PR isn't needed, is it?

I still think it's a good idea to return a success boolean rather than throwing and having call sites catch or write onError on the futures. (That's way it's less likely that launchers will forget to handle errors and throw a stack trace at the user.)

I'll rework the PR to do that instead.

[dcharkes]

@mkustermann I've reworked the PR so that it doesn't throw any errors anymore and streams all errors to the logger.

[dcharkes]

Thanks @mkustermann !

[auto-submit]

auto label is removed for dart-lang/native/108, due to - The status or check suite build (windows, stable, native_toolchain_c) has failed. Please fix the issues identified (or deflake) before re-applying this label.

• The status or check suite build (windows, stable, native_assets_builder) has failed. Please fix the issues identified (or deflake) before re-applying this label.
• The status or check suite build (macos, stable, native_toolchain_c) has failed. Please fix the issues identified (or deflake) before re-applying this label.
• The status or check suite build (macos, stable, native_assets_builder) has failed. Please fix the issues identified (or deflake) before re-applying this label.
• The status or check suite build (ubuntu, stable, native_assets_builder) has failed. Please fix the issues identified (or deflake) before re-applying this label.