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

Issue 859680 link

Starred by 2 users
Status: WontFix
Owner: ----
Closed: Jul 2
Cc:
bleung@chromium.org
█ vmreyes@google.com
vapier@chromium.org
█ saludlemus@google.com
tbroch@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug



Sign in to add a comment

cros_portage_upgrade succeeds while emerge-eve fails

Project Member Reported by saludlemus@google.com, Jul 2 
Chrome Version: <69.0.3464.0>
Chrome OS Version: <10799.0.2018_06_19_1616>
Chrome OS Platform: <GOOGLE-EVE>
Network info: <N/A, N/A, N/A>

Please specify Cr-* of the system to which this bug/feature applies (add
the label below).

Cr-Internals

Steps To Reproduce:
(1) N/A
(2) N/A
(3) N/A

Expected Result: cros_portage_upgrade should have alerted on emerge-eve failure.

Actual Result: cros_portage_upgrade succeeds while emerge-eve fails to build the package (net-libs/libproxy). We think that cros_portage_upgrade is using different flags than emerge-eve, specifically "-fno-exceptions".

How frequently does this problem reproduce? (Always, sometimes, hard to
reproduce?) N/A

What is the impact to the user, and is there a workaround? If so, what is
it? N/A

Please provide any additional information below. Attach a screen shot or
log if possible.

For graphics-related bugs, please copy/paste the contents of the about:gpu
page at the end of this report.

Comment 1 by bleung@chromium.org, Jul 2

Cc: saludlemus@google.com vmreyes@google.com bleung@chromium.org tbroch@chromium.org
Components: Infra>Client>ChromeOS>Build

Comment 2 by bleung@chromium.org, Jul 2

Cc: vapier@chromium.org
I think this was due to the fact that cros_portage_upgrade runs emerge-eve --pretend.

Thanks!

Comment 3 by bleung@google.com, Jul 2 

FWIW, the issue with net-libs/libproxy was solved by https://chromium-review.googlesource.com/c/chromiumos/overlays/chromiumos-overlay/+/1123323, but it required the extra step of running emerge-eve net-libs/libproxy.

The cros_portage_upgrade doesn't actually makes sure that the packages will build in the environment (doesn't compile) so that's why this wasn't caught then.

Comment 4 by vapier@google.com, Jul 2

Status: WontFix (was: Unconfirmed)
yeah, this is WAI. cros_portage_upgrade merely walks the depgraph, it doesn't build anything. I don't see this changing.
Project Member

Comment 5 by bugdroid1@chromium.org, Jul 25 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/727a1465f8eab7c7617d1a327fed437f30109993

commit 727a1465f8eab7c7617d1a327fed437f30109993
Author: Victor Reyes <vmreyes@google.com>
Date: Wed Jul 25 23:35:07 2018

netlibs: libproxy: Added CXX flags needed to emerge libproxy on eve

BUG= chromium:859680 
TEST=emerge-eve net-libs/libproxy

Change-Id: I89248849e40dc775bf1f3f015a0f997d2e17348e
Signed-off-by: Victor Reyes <vmreyes@google.com>
Signed-off-by: Salud Lemus <saludlemus@google.com>
Reviewed-on: https://chromium-review.googlesource.com/1123323
Commit-Ready: Victor Reyes Espinoza <vmreyes@google.com>
Tested-by: Victor Reyes Espinoza <vmreyes@google.com>
Reviewed-by: Benson Leung <bleung@chromium.org>

[add] https://crrev.com/727a1465f8eab7c7617d1a327fed437f30109993/chromeos/config/env/net-libs/libproxy
►

Sign in to add a comment