New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 813268 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Mar 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Task
Build-Toolchain



Sign in to add a comment

Tracking target packages built by gcc

Project Member Reported by laszio@chromium.org, Feb 17 2018

Issue description

Now that the transition from gcc to llvm is done, building package with gcc is deemed as a regression in chrome os.
 

Comment 1 by cmt...@chromium.org, Mar 16 2018

Cc: laszio@chromium.org
Status: Fixed (was: Untriaged)
I believe this is done now.

Comment 2 by laszio@chromium.org, Mar 16 2018

There seems to be some builders failing the test:
https://uberchromegw.corp.google.com/i/chromeos/builders/beaglebone-release/builds/1917
https://uberchromegw.corp.google.com/i/chromeos/builders/gale-release/builds/2140
https://uberchromegw.corp.google.com/i/chromeos/builders/lakitu-release/builds/3195
https://uberchromegw.corp.google.com/i/chromeos/builders/rainier-release/builds/353

I clicked through ~50 builds to check their status before giving up. Do you know any way to query the status of a given stage for all builders?

BTW, it looks like we should just check packages that are built by emerge. Namely, if a comp_path does not contain /portage/ then just ignore it. Otherwise, a newly added binary by gcc will fail the test.
Project Member

Comment 3 by bugdroid1@chromium.org, Mar 18 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/toolchain-utils/+/4bc22ee60558f57dce27ee2d35eba5aa40ec9a30

commit 4bc22ee60558f57dce27ee2d35eba5aa40ec9a30
Author: Caroline Tice <cmtice@google.com>
Date: Sun Mar 18 21:35:38 2018

[toolchain-utils] More whitelist updates.

We found a few more packages/files that need to be built with
GCC and are adding them to our debug_info_test whitelists.

BUG= chromium:813268 
TEST=Tested with remote tryjobs: lakitu-release, gale-release,
rainier-release, beaglebone-relase, fizz-release, cave-release.
All passed the DebugInfoTest stage.

Change-Id: I012d1620a85180f946c2ad2c8bbb175f5ef23e49
Reviewed-on: https://chromium-review.googlesource.com/967455
Commit-Ready: Caroline Tice <cmtice@chromium.org>
Tested-by: Caroline Tice <cmtice@chromium.org>
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>

[modify] https://crrev.com/4bc22ee60558f57dce27ee2d35eba5aa40ec9a30/debug_info_test/ngcc_comp_path.whitelist
[modify] https://crrev.com/4bc22ee60558f57dce27ee2d35eba5aa40ec9a30/debug_info_test/ngcc_dso_path.whitelist

Comment 4 by cmt...@chromium.org, May 15 2018

Status: Verified (was: Fixed)

Sign in to add a comment