Re: Xen 4.8

[Manuel Bouyer <bouyer%antioche.eu.org@localhost>]

On Mon, Apr 17, 2017 at 08:03:18AM -0700, John Nemeth wrote:
> On Apr 17, 10:18am, Roy Marples wrote:
> } On 30/03/2017 10:27, Manuel Bouyer wrote:
> } > Hello,
> } > I've just commited packages for xenkernel and xentools version 4.8.0.
> } > I've not done extensive testing yet, so for production I'd still
> } > recommend the 4.6 or 4.5 packages.
> } 
> } gcc -I/usr/include -I/usr/pkg/include  -m32 -march=i686 -DBUILD_ID 
> } -fno-strict-aliasing -std=gnu99 -Wall -Wstrict-prototypes 
> } -Wno-error=ignored-attributes -no-integrated-as 
> } -Wno-error=tautological-compare -Wno-error=null-dereference 
> } -Wno-error=empty-body
> }   -Wno-error=self-assign -Wno-error=sometimes-uninitialized 
> } -Wno-error=unused-function -Wno-error=format 
> } -Wno-error=unused-const-variable -Wno-error=gnu-designator 
> } -Wno-error=uninitialized -Wno-error=parentheses-equality 
> } -Wno-error=enum-conversion -Wdecla
> } ration-after-statement -Wno-unused-local-typedefs   -O2 
> } -fomit-frame-pointer 
> } -D__XEN_INTERFACE_VERSION__=__XEN_LATEST_INTERFACE_VERSION__ -MMD -MF 
> } .tpm_drivers.o.d   -DNDEBUG -Werror -nopie -fno-stack-protector 
> } -fno-exceptions -fno-builtin -msoft-float -
> } I/usr/obj/pkgsrc/sysutils/xentools48/work/xen-4.8.0/tools/firmware/rombios/32bit/tcgbios/../../../../../tools/include 
> } -I.. -I../.. -I../../../../libacpi  -c -o tpm_drivers.o tpm_drivers.c
> } tcgbios.c:1519:34: error: taking address of packed member 'u' of class 
> } or structure 'pushad_regs_t' may result in an unaligned pointer value 
> } [-Werror,-Waddress-of-packed-member]
> }                                                &regs->u.r32.edx);
> }                                                 ^~~~~~~~~~~~~~~
> } 1 error generated.
> } 
> } This is building on a very recent NetBSD-current w clang, but pkgsrc 
> } pulls in gcc for some reason when building xen.
> 
>      Yes, why does it force the use of GCC 4.8.  This means when
> building on -current, it pulls in the GCC package.  It really should
> not be doing this.

It uses gcc to cross-build some stuff (e.g. some firmwares). Because of
this, it wants gcc's stddef.h (and maybe others gcc-supplied headers).
It doens't build with the NetBSD ones.

> 
>      I'm also curious why you set MAINTAINER when there are a bunch
> of us that semi-regularly work on the Xen packages (I imported two
> versions of Xen).

I though setting MAINTAINER to pkgsrc-users%NetBSD.org@localhost means that there
is no MAINTAINER, which is not accurate for this package.
It doens't means nobody else can touch the packages.

-- 
Manuel Bouyer <bouyer%antioche.eu.org@localhost>
     NetBSD: 26 ans d'experience feront toujours la difference
--