Re: [PATCH v1] tools: ipxe: update for fixing build with GCC11

[Olaf Hering <olaf@xxxxxxxxx>]

Please revert bf4ccd4265ac614fbfa38bf168b6eeaf4c17d51e in ipxe.git, CentOS 7 
apparently fails to handle '-D'.

It worked in my testing with SLE12SP5 and SLE15SP3 as a base system.

See below.


I guess for xen.git, updating to just bf4ccd4265ac614fbfa38bf168b6eeaf4c17d51e^ 
will be good enough.

Olaf

Am Wed, 16 Jun 2021 13:33:52 +0100
schrieb Andrew Cooper <andrew.cooper3@xxxxxxxxxx>:

> On 15/06/2021 22:26, Olaf Hering wrote:
> > Use a snapshot which includes commit
> > f3f568e382a5f19824b3bfc6081cde39eee661e8 ("[crypto] Add
> > memory output constraints for big-integer inline assembly"),
> > which fixes build with gcc11.
> >
> > Signed-off-by: Olaf Hering <olaf@xxxxxxxxx>
> > ---
> >  tools/firmware/etherboot/Makefile | 3 ++-
> >  1 file changed, 2 insertions(+), 1 deletion(-)
> >
> > diff --git a/tools/firmware/etherboot/Makefile 
> > b/tools/firmware/etherboot/Makefile
> > index ed9e11305f..23b3f6ca9d 100644
> > --- a/tools/firmware/etherboot/Makefile
> > +++ b/tools/firmware/etherboot/Makefile
> > @@ -10,7 +10,8 @@ else
> >  IPXE_GIT_URL ?= git://git.ipxe.org/ipxe.git
> >  endif
> >  
> > -IPXE_GIT_TAG := 988d2c13cdf0f0b4140685af35ced70ac5b3283c
> > +# put an updated tar.gz on xenbits after changes to this variable
> > +IPXE_GIT_TAG := bf4ccd4265ac614fbfa38bf168b6eeaf4c17d51e
> 
> CI says no.
> 
> Gitlab CI is currently fairly red because of a clang build fix which
> hasn't made its way into master yet, but this job:
> 
>   https://gitlab.com/xen-project/patchew/xen/-/jobs/1349871230
> 
> shows a real failure on CentOS 7.
> 
> ...
>   [VERSION] bin/version.rtl8139.rom.o
>   [AR] bin/blib.a
> ar: creating bin/blib.a
> objcopy: invalid option -- 'D'
> Usage: objcopy [option(s)] in-file [out-file]
> ...
> 
> ~Andrew

Attachment: pgppIDsyNpuWp.pgp
Description: Digitale Signatur von OpenPGP