sysroot_wrapper: avoid check for SYSROOT outside chroot. |
|||||||
Issue descriptionThe sysroot wrapper is checking the value of SYSROOT and, if it is not set, it logs a message in a log file. Noone seems to be paying attention to this. And when the check fails it consumes a lot of time as it is show in issue 773138. from e-mail discussion Vapier said: "i don't believe anyone looks at the log until something goes really wrong, and they even know this log exists. so rarely. it has no real value in the standalone case. we could disable it via cros_setup_toolchains.py:_ProcessSysrootWrappers like we do with ccache support. that said, i think fundamentally the check makes sense in our cross-compiling scenario as it usually means we'd be using the wrong sysroot to compile/link. in fact, as i look my log now, i see packages doing it wrong like media-libs/dlm. making the check require an env var be set wouldn't help ... in the case of packages like dlm, they'd remove that env var too. the code walks back through its parents, so we could have it trigger when emerge shows up. if it does, we can have it dump to stderr and then fail, otherwise just be silent."
,
Oct 16 2017
it is also important to fix this for the standalone sdk case since the check will always fail there.
,
Dec 26 2017
,
Jan 2 2018
[It appears that a bunch of old cros issues bulk-added the "Infra" component recently, but they should probably be "Infra>Client>ChromeOS".]
,
Jan 4 2018
,
Feb 5 2018
,
Aug 3
This bug has an owner, thus, it's been triaged. Changing status to "assigned".
,
Aug 3
I removed the logging code in https://chromium-review.googlesource.com/c/chromiumos/overlays/chromiumos-overlay/+/1142301 |
|||||||
►
Sign in to add a comment |
|||||||
Comment 1 by llozano@chromium.org
, Oct 16 2017