Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 62365 - kernel-cc/util-cc don't do anything & return values lost leading to false successes
Summary: kernel-cc/util-cc don't do anything & return values lost leading to false suc...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Hosted Projects
Classification: Unclassified
Component: genkernel (show other bugs)
Hardware: All All
: High critical (vote)
Assignee: Gentoo Genkernel Maintainers
URL:
Whiteboard:
Keywords: InVCS
Depends on:
Blocks:
 
Reported: 2004-08-31 02:40 UTC by Robin Johnson
Modified: 2004-09-19 05:28 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments
genkernel-3.0.2f-fixes.patch (genkernel-3.0.2f-fixes.patch,2.76 KB, patch)
2004-08-31 02:40 UTC, Robin Johnson
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Robin Johnson archtester Gentoo Infrastructure gentoo-dev Security 2004-08-31 02:40:03 UTC
"genkernel --kernel-cc=/bin/false all"
this works fine, as it totally ignores the specified cc and uses the default of gcc instead.

from gen_compile.sh:
${MAKE} ${MAKEOPTS} ${1} 2>&1 | tee -a ${DEBUGFILE}
RET=$?
$? gives the exit value of tee, NOT make. So after the make fails, genkernel continues!

You need to use ${PIPESTATUS[0]} instead to get the return value of the first item in the pipe. $? is in ${PIPESTATUS[1]}.

I've attached a patch that fixes both of these issues, could you please get it into genkernel ASAP? If you don't have time, I'll commit it myself to the current gentoo/src/genkernel_bradmssw.
Comment 1 Robin Johnson archtester Gentoo Infrastructure gentoo-dev Security 2004-08-31 02:40:58 UTC
Created attachment 38567 [details, diff]
genkernel-3.0.2f-fixes.patch

patch for listed issues.
Comment 2 Robin Johnson archtester Gentoo Infrastructure gentoo-dev Security 2004-09-08 19:22:03 UTC
*bump*
Comment 3 Tim Yamin (RETIRED) gentoo-dev 2004-09-19 02:13:59 UTC
Thank you for the patch, it's now in CVS. Please accept my apology for this taking so long - if you have any more problems then please reopen this bug or open a new one. Thanks!
Comment 4 Tim Yamin (RETIRED) gentoo-dev 2004-09-19 05:28:27 UTC
Patch in genkernel-3.0.2g; please reopen this bug if you have any problems. Thanks!