Fix to isolate_tests seems to be crashing Mac bots.
Reported by
beaudoin@chromium.org,
Apr 8 2016
|
|||||||
Issue descriptionSuspecting this fix: https://codereview.chromium.org/1875743002 Of crashing the Mac bots. Working on a fix that moves the target app_list_shower.gyp:app_list_shower_unittests to use_aura=1. Please use this bug to make sure I did the right thing or fix it if that wasn't correct.
,
Apr 8 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/736ad8abed54d7393cf672f4dbc2eea26e08bdcf commit 736ad8abed54d7393cf672f4dbc2eea26e08bdcf Author: cwallez <cwallez@chromium.org> Date: Fri Apr 08 21:54:37 2016 Build app_list_shower only if use_aura=1 This was being build on non-Android platforms and broke the Mac build. Guard the build with use_aura=1 in Gyp, like it is in GN. BUG= 601932 TBR=thakis@chromium.org NOTRY=true Review URL: https://codereview.chromium.org/1875833002 Cr-Commit-Position: refs/heads/master@{#386214} [modify] https://crrev.com/736ad8abed54d7393cf672f4dbc2eea26e08bdcf/build/gn_migration.gypi
,
Apr 9 2016
This seems to have fixed it. We should still figure out why all the CQ bots were green. Maybe some target is not being built?
,
Apr 11 2016
Removing Sheriff-Chromium label. Leaving to you guys to decide if you want to spend more time on investigating the root cause.
,
Apr 11 2016
+Infra-CommitQueue I am also curious if it is expected behavior that https://codereview.chromium.org/1875743002 passed through the commit queue and then broke the Mac builder: https://build.chromium.org/p/chromium.fyi/builders/ClangToTLinux/builds/4852 Shouldn't this have been caught by the trybots?
,
Apr 11 2016
+phajdan = CQ has allowed bad change to build rules (see fix https://codereview.chromium.org/1875833002). I wonder is this sad WAI or bug in analyze. FYI: Please, remove owner before sending to CQ team, as we typically ignore owned bugs.
,
Apr 11 2016
I think this is WAI: * original breakage: 'all' doesn't build everything with gyp (there's an open bug for that somewhere), and the original change didn't add the _run target to gn_migration.gypi. So bots building 'all' didn't build that _run target, and then later the isolate step failed. * follow-up breakage: The fix for that added the _run target to 'all', which caused it to be built on the clobber bots, but the new target didn't build on OS X. The fix made the _run target not part of 'all', but I'm guessing that explicitly building "app_list_shower" on OS X still causes errors -- the target should be made conditional too. Anyhoo, this bug is done.
,
Apr 11 2016
> The fix for that added the _run target to 'all', which caused it to be built on the clobber bots, but the new target didn't build on OS X. Shouldn't the fact that it didn't build on OSX been detected by CQ? (app_list_shower target was not supposed to be built on OSX since it uses Aura.)
,
Apr 26 2016
,
Apr 27 2016
|
|||||||
►
Sign in to add a comment |
|||||||
Comment 1 by h...@chromium.org
, Apr 8 2016