| Issue 203807: | initramfs ebuild doesn't express dependencies correctly. | |
| 1 person starred this issue and may be notified of changes. | Back to list |
Sign in to add a comment
|
http://git.chromium.org/gitweb/?p=chromiumos/overlays/chromiumos-overlay.git;a=blob;f=chromeos-base/chromeos-initramfs/chromeos-initramfs-9999.ebuild Mandeep pointed out the ebuild has hardcoded versions and at least for libpng, it should not. When libpng rev'd: http://gerrit.chromium.org/gerrit/7880 initramfs build broke: http://chromeos-botmaster.mtv.corp.google.com:8026/builders/x86-alex-binary/builds/1471/steps/Archive/logs/stdio chromeos-initramfs-0.0.1-r46: cp: cannot stat `/build/x86-alex//usr/lib/libpng12.so.0.44.0': No such file or directory Nearly every build broke on internal builders. kliegs rev'd by hand but this will break again badly. msb reccomends a rewrite of the ebuild file to properly express the package dependencies.
Mar 16, 2013
#1
bugdroid1@chromium.org
Mergedinto:
chromium-os:17390
Sep 22, 2011
http://gerrit.chromium.org/gerrit/8155 should fix immediate build fails but not underlying issue. Thinking about this more...in general, can we test ebuilds for hard coded package versions? Some might be valid reasons for the hard coding (e.g. contain binary blobs) but shouldn't be that many.
Sep 22, 2011
In response to grundler's question, we could try writing a script for that. My gut instinct says it will be more trouble than it's worth to get right. But we could stage some kind of ebuild scrub event where people look for hardcoded packages, if this is a recurring problem.
Sep 22, 2011
My comment is just idle speculation. I suspect too it might not be worth it. But "static analysis" is working pretty well to find coding issues in the linux kernel that weren't obvious. Gentoo would probably find such a tool quite useful too...perhaps fund someone in that community to do the work? Summer of Code project? Anyway, don't forget the real issue here: initramfs ebuild dependencies need to be rewritten properly.
Sep 22, 2011
All this really says is things I already knew, and which are meant to be addressed in bug 17390 .
Status:
Duplicate
Mergedinto: 17390
Mar 6, 2013
(No comment was entered for this change.)
Labels:
OS-Chrome
Mar 9, 2013
(No comment was entered for this change.)
Labels:
-Area-LinuxUserspace -TreeCloser Cr-OS-Packages Hotlist-TreeCloser
|
||||||||||||||
| ► Sign in to add a comment | |||||||||||||||