Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

frog build relies on release build being present #668

Closed
iposva-google opened this issue Dec 2, 2011 · 2 comments
Closed

frog build relies on release build being present #668

iposva-google opened this issue Dec 2, 2011 · 2 comments
Labels
closed-duplicate Closed in favor of an existing report P0 A serious issue requiring immediate resolution

Comments

@iposva-google
Copy link
Contributor

The build scripts rely on unbuilt stuff. Just building --mode=debug will fail with this message:

PhaseScriptExecution "Action "generate_frogsh"" /Sources/dart-all/dart/xcodebuild/dart-frog.build/Debug_ia32/frogsh.build/Script-095A7D91DE56B603C848FB82.sh
    cd /Sources/dart-all/dart/frog
    /bin/sh -c /Sources/dart-all/dart/xcodebuild/dart-frog.build/Debug_ia32/frogsh.build/Script-095A7D91DE56B603C848FB82.sh

note: Generating frogsh file
Dart VM not built. Please run the following command:
 ../tools/build.py -m release
Command /bin/sh failed with exit code 1
Command /bin/sh failed with exit code 1

This is broken on many levels, as you rebuild debug in will keep using an out-of-date Dart VM binary. This build needs to honor the current(!) build mode.

@iposva-google
Copy link
Contributor Author

cc @dgrove.

@dgrove
Copy link
Contributor

dgrove commented Dec 2, 2011

Added Duplicate label.
Marked as being merged into #669.

@iposva-google iposva-google added Type-Defect P0 A serious issue requiring immediate resolution closed-duplicate Closed in favor of an existing report labels Dec 2, 2011
copybara-service bot pushed a commit that referenced this issue Jun 8, 2022
…ions)

https://dart.googlesource.com/protobuf/+log/b149f801cf7a..408f69b98b2d

2022-06-08 omersa@google.com Update documentation according to Effective Dart documentation guide (#664)
2022-06-02 omersa@google.com Move closures in proto3 deserializer to top-level (#668)
2022-06-02 omersa@google.com Refactor protobuf bench runners, update README: (#665)
2022-06-02 omersa@google.com Benchmarks: add proto3 ser/deser benchs, move common code to common.dart (#667)
2022-06-02 omersa@google.com Remove makeLongInt helper (#666)
2022-05-31 lorenvs@outlook.com Use `-O4` when compiling benchmarks. (#552)
2022-05-25 omersa@google.com Remove invalid check in `_FieldSet.writeFieldValue` (#660)
2022-05-20 devoncarew@gmail.com Refactor file locations, update READMEs (#657)
2022-05-19 omersa@google.com Remove trailing whitespace in protoc_plugin/README

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/dart-protobuf-dart-sdk
Please CC dart-ecosystem-gardener@grotations.appspotmail.com on the revert to ensure that a human
is aware of the problem.

To file a bug in Dart Protobuf Support: https://github.com/dart-lang/protobuf/issues
To file a bug in Dart SDK: https://github.com/dart-lang/sdk/issues

To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Tbr: dart-ecosystem-gardener@grotations.appspotmail.com
Change-Id: I0f1aa66ee6fe167f37443957234f4d0140828416
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/247467
Reviewed-by: Devon Carew <devoncarew@google.com>
Commit-Queue: Devon Carew <devoncarew@google.com>
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
closed-duplicate Closed in favor of an existing report P0 A serious issue requiring immediate resolution
Projects
None yet
Development

No branches or pull requests

2 participants