FS#55181 - [ovmf] build fail and break load VMs

Attached to Project: Arch Linux
Opened by Gustavo Alvarez (sl1pkn07) - Thursday, 17 August 2017, 15:27 GMT
Last edited by Doug Newgard (Scimmia) - Wednesday, 06 June 2018, 14:56 GMT
Task Type Bug Report
Category Packages: Extra
Status Closed
Assigned To Thomas Bächler (brain0)
Architecture All
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

that commit use in the pkgbuild fails to build with GCC 7.1.1

please update to commit ca56256d5e0b, wich fix this, and fix this other[0][1]

[0] https://bugzilla.tianocore.org/show_bug.cgi?id=657#c10
[1]edk2-devel@lists.01.org/msg28111.html"> https://www.mail-archive.com/edk2-devel@lists.01.org/msg28111.html


the patch is merged in upstream [2]

[2] https://github.com/tianocore/edk2/commit/ca56256d5e0b7e63325b049e90a6bd03f90e3598


also. if is possible, use the real name of the files instead of 'ovmf_foo_bar.bin' (in each platform directory, for example: /usr/share/ovmf/ovmf-"TARGET_ARCH"/OVMF_{CODE,VARS}.fd")

greetings
This task depends upon

Closed by  Doug Newgard (Scimmia)
Wednesday, 06 June 2018, 14:56 GMT
Reason for closing:  Fixed
Comment by Thomas Bächler (brain0) - Sunday, 17 September 2017, 13:47 GMT
Updated package is in [testing]. I was unable to test it myself, will do so tomorrow.

* I disabled the ia32 build, since the build is broken
* The images now have their upstream file names: /usr/share/ovmf/x64/OVMF_{CODE,VARS}.fd

I will move this to [extra] as soon as I can test it.
Comment by Gustavo Alvarez (sl1pkn07) - Sunday, 17 September 2017, 14:46 GMT
seems https://git.archlinux.org/svntogit/packages.git/tree/trunk/PKGBUILD?h=packages/ovmf#n15 need set to GCC49 for build with gcc 7.x

https://www.kraxel.org/cgit/jenkins/edk2/tree/edk2.git.spec#n120
Comment by Gustavo Alvarez (sl1pkn07) - Sunday, 17 September 2017, 15:45 GMT
my changes in the PKGBUILD

- Remove all sed (all can doing with build command options)
- Use GCC49 instead of GCC5 (see https://bugzilla.tianocore.org/show_bug.cgi?id=671 and https://www.kraxel.org/cgit/jenkins/edk2/tree/edk2.git.spec (from Gerd Hoffmann repo (one of vfio dev maintainers))

https://sl1pkn07.wtf/paste/view/raw/cd002b53
Comment by Thomas Bächler (brain0) - Wednesday, 20 September 2017, 22:31 GMT
Using GCC49 is wrong, see https://github.com/tianocore/edk2/commit/656ac0c7d8ea9912f23355d0e907bfe4173256fc

Loading...