Re: Xen 4.8

[John Nemeth <jnemeth%cue.bc.ca@localhost>]

On Apr 17,  5:19pm, Manuel Bouyer wrote:
} 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.

     Ugh!  This needs to be fixed (possibly upstream).

} >      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 does mean that there is no desginated MAINTAINER.  I wonder
if we can have some kind of group, i.e. xen-maintainers%NetBSD.org@localhost?
On the imports that I did, I set it to pkgsrc-users@ to explicitly
say not everything had to go through me since I didn't want to have
to deal with everything.

} It doens't means nobody else can touch the packages.
} 
}-- End of excerpt from Manuel Bouyer