Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 462898 - sys-firmware/ipxe-1.0.0_p20120905: Fails to build with GCC 4.8.0
Summary: sys-firmware/ipxe-1.0.0_p20120905: Fails to build with GCC 4.8.0
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Core system (show other bugs)
Hardware: All All
: Normal normal (vote)
Assignee: Doug Goldstein (RETIRED)
URL: https://git.ipxe.org/ipxe.git/commitd...
Whiteboard:
Keywords: Bug, Inclusion, PATCH, REVIEWED
Depends on:
Blocks: werror 438780 gcc-4.8
  Show dependency tree
 
Reported: 2013-03-24 03:33 UTC by Richard Yao (RETIRED)
Modified: 2013-12-23 13:10 UTC (History)
5 users (show)

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


Attachments
sys-firmware/ipxe-1.0.0_p20120905 build failure (ipxe-1.0.0_p20120905:20130324-012002.log,61.35 KB, text/plain)
2013-03-24 03:36 UTC, Richard Yao (RETIRED)
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Richard Yao (RETIRED) gentoo-dev 2013-03-24 03:33:51 UTC
Incorrect code in iPXE to causes a build failure when using GCC 4.8.0. The URL field links to an upstream patch by Daniel P. Berrange of Redhat that resolves this issue.
Comment 1 Richard Yao (RETIRED) gentoo-dev 2013-03-24 03:36:46 UTC
Created attachment 343062 [details]
sys-firmware/ipxe-1.0.0_p20120905 build failure

Attaching the failure log failed due to permissions.
Comment 2 Alex Xu (Hello71) 2013-04-11 13:35:10 UTC
Not sure how _p versioning works in Gentoo, but can't we just bump it to sys-firmware/ipxe-1.0.0_p20130305 on eb5a2ba5962579e514b377f5fdab7292be0fb2a7?
Comment 3 Francesco Riosa 2013-05-13 13:09:46 UTC
and it should not compile with -Werror anyway. Build system allow to set NO_WERROR=1
i.e.

diff -u   /g/portage/sys-firmware/ipxe/ipxe-1.0.0_p20120905.ebuild  ipxe-1.0.0_p20120905-r1.ebuild
--- /g/portage/sys-firmware/ipxe/ipxe-1.0.0_p20120905.ebuild    2013-01-26 20:31:29.000000000 +0000
+++ ipxe-1.0.0_p20120905-r1.ebuild      2013-05-13 13:05:20.373976364 +0000
@@ -37,6 +37,7 @@
 }

 src_compile() {
+       export NO_WERROR=1
        if use qemu; then
                emake bin/808610de.rom # pxe-e1000.rom
                emake bin/80861209.rom # pxe-eepro100.rom
Comment 4 Doug Goldstein (RETIRED) gentoo-dev 2013-05-13 14:57:39 UTC
(In reply to comment #2)
> Not sure how _p versioning works in Gentoo, but can't we just bump it to
> sys-firmware/ipxe-1.0.0_p20130305 on
> eb5a2ba5962579e514b377f5fdab7292be0fb2a7?

Indeed. The _p just means "post" or "+ patch" this version. I just use the date I grabbed the git commit from. iPXE upstream isn't doing too frequent releases and makes sure their master stays fairly stable.
Comment 5 Doug Goldstein (RETIRED) gentoo-dev 2013-06-27 22:07:58 UTC
I added latest ipxe to the tree and the current version qemu uses as well. This should be resolved there.