Merge gnu-efi 3.0v-5 (main) from Debian unstable (main)

Bug #1396381 reported by Artur Rona
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
gnu-efi (Ubuntu)
Fix Released
Wishlist
Unassigned

Bug Description

gnu-efi (3.0v-5) unstable; urgency=medium

  * inc/efistdarg.h: Use gcc builtins instead of stubs
    - The last upload of gnu-efi caused things to compile, but severely
      broke them.

 -- Julian Andres Klode <email address hidden> Mon, 21 Jul 2014 13:20:23 +0200

gnu-efi (3.0v-4) unstable; urgency=medium

  * Take over maintenance
  * Add git information and git-buildpackage configuration
  * Use gbp pq for maintaining the patch queue
  * inc/efistdarg.h: Stub out va_copy (Closes: #753682)

 -- Julian Andres Klode <email address hidden> Sat, 19 Jul 2014 14:02:06 +0200

gnu-efi (3.0v-3) unstable; urgency=low

  * Stepping down from maintainers, no interest anymore since syslinux uses
    its own copy of gnu-efi that is bundled and sort-of impossible to patch
    out.

 -- Daniel Baumann <email address hidden> Wed, 16 Jul 2014 21:49:17 +0200

gnu-efi (3.0v-2) unstable; urgency=low

  * Temporarily build without stdarg.h in efistdarg.h (Closes: #747158).

 -- Daniel Baumann <email address hidden> Tue, 06 May 2014 06:15:54 +0200

gnu-efi (3.0v-1) unstable; urgency=low

  * Merging upstream version 3.0v.
  * Updating year in copyright file.
  * Updating make calls in rules for new upstream release.

 -- Daniel Baumann <email address hidden> Sat, 05 Apr 2014 13:03:14 +0200

Related branches

Revision history for this message
Artur Rona (ari-tczew) wrote :
Revision history for this message
Artur Rona (ari-tczew) wrote :
Changed in gnu-efi (Ubuntu):
importance: Undecided → Wishlist
status: New → Confirmed
Revision history for this message
Steve Langasek (vorlon) wrote :

I've built and installed gnu-efi locally. Rebuilding shim with this version of gnu-efi fails with:

In file included from Cryptlib/Include/string.h:15:0,
                 from netboot.c:38:
Cryptlib/Include/OpenSslSupport.h:39:0: error: "va_arg" redefined [-Werror]
 #define va_arg VA_ARG
 ^
In file included from /usr/include/efi/efilib.h:27:0,
                 from netboot.c:37:

This is probably a bug in the shim code; nevertheless this merge shouldn't happen before we have this resolved and have verified that shim is buildable (and works) with the new gnu-efi.

Revision history for this message
Sebastien Bacher (seb128) wrote :

unsubscribing the sponsors, it seems there are changes needed before that's ready for upload

Revision history for this message
Julian Andres Klode (juliank) wrote :

Yes, it seems to be a bug in the Shim. You should not define va_arg, as gnu-efi defines that (and it's a standard library thing, so it really should be in here ). It did not use to be defined all the time in gnu-efi, but this had to be changed because the resulting code was broken (it compiled fine, but did not work).

So, simply removing the va_arg definition from shim should make it work with newer gnu-efi.

Newer versions have a GNU_EFI_USE_EXTERNAL_STDARG to use the compiler's stdarg.h, but that's really relevant here, as we're building without it.

Revision history for this message
Steve Langasek (vorlon) wrote :

shim 0.8 is compatible with gnu-efi 3.0v-5. Packages have been published to https://launchpad.net/~canonical-foundations/+archive/ubuntu/shim for testing.

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package gnu-efi - 3.0v-5ubuntu1

---------------
gnu-efi (3.0v-5ubuntu1) wily; urgency=medium

  * Merge from Debian unstable (LP: #1396381), remaining changes:
    - debian/patches/gcc46-compatibility: don't break with old compilers
      and -DGNU_EFI_USE_MS_ABI.
  * Dropped changes, included in Debian:
    - pass architecture arguments when building the x86_64 code on i386

gnu-efi (3.0v-5) unstable; urgency=medium

  * inc/efistdarg.h: Use gcc builtins instead of stubs
    - The last upload of gnu-efi caused things to compile, but severely
      broke them.

gnu-efi (3.0v-4) unstable; urgency=medium

  * Take over maintenance
  * Add git information and git-buildpackage configuration
  * Use gbp pq for maintaining the patch queue
  * inc/efistdarg.h: Stub out va_copy (Closes: #753682)

gnu-efi (3.0v-3) unstable; urgency=low

  * Stepping down from maintainers, no interest anymore since syslinux uses
    its own copy of gnu-efi that is bundled and sort-of impossible to patch
    out.

gnu-efi (3.0v-2) unstable; urgency=low

  * Temporarily build without stdarg.h in efistdarg.h (Closes: #747158).

gnu-efi (3.0v-1) unstable; urgency=low

  * Merging upstream version 3.0v.
  * Updating year in copyright file.
  * Updating make calls in rules for new upstream release.

 -- Steve Langasek <email address hidden> Tue, 12 May 2015 17:45:11 +0000

Changed in gnu-efi (Ubuntu):
status: Confirmed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.