Hi, configure fails on NetBSD that there is no brctl. NetBSD has no brctl, NetBSD has brconfig. Christoph -- ---to satisfy European Law for business letters: Advanced Micro Devices GmbH Einsteinring 24, 85689 Dornach b. Muenchen Geschaeftsfuehrer: Alberto Bozzo, Andrew Bowd Sitz: Dornach, Gemeinde Aschheim, Landkreis Muenchen Registergericht Muenchen, HRB Nr. 43632
2012/2/22 Christoph Egger <Christoph.Egger@amd.com>:> > Hi, > > configure fails on NetBSD that there is no brctl. > NetBSD has no brctl, NetBSD has brconfig.Sorry Christoph, the brctl test is going away (from configure at least).> > Christoph > > -- > ---to satisfy European Law for business letters: > Advanced Micro Devices GmbH > Einsteinring 24, 85689 Dornach b. Muenchen > Geschaeftsfuehrer: Alberto Bozzo, Andrew Bowd > Sitz: Dornach, Gemeinde Aschheim, Landkreis Muenchen > Registergericht Muenchen, HRB Nr. 43632 > > > _______________________________________________ > Xen-devel mailing list > Xen-devel@lists.xen.org > http://lists.xen.org/xen-devel
On 02/22/12 10:59, Roger Pau Monné wrote:> 2012/2/22 Christoph Egger<Christoph.Egger@amd.com>: >> >> Hi, >> >> configure fails on NetBSD that there is no brctl. >> NetBSD has no brctl, NetBSD has brconfig. > > Sorry Christoph, the brctl test is going away (from configure at least).There are some more Linux specific tests: - ip - uuid_clear in libuuid Also the check for yajl_alloc in libyajl fails although I have libyajl version 1 installed. The check may be correct but configure shouldn't fail if version 1 is still supported. Christoph -- ---to satisfy European Law for business letters: Advanced Micro Devices GmbH Einsteinring 24, 85689 Dornach b. Muenchen Geschaeftsfuehrer: Alberto Bozzo, Andrew Bowd Sitz: Dornach, Gemeinde Aschheim, Landkreis Muenchen Registergericht Muenchen, HRB Nr. 43632 _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel
2012/2/22 Christoph Egger <Christoph.Egger@amd.com>:> On 02/22/12 10:59, Roger Pau Monné wrote: >> >> 2012/2/22 Christoph Egger<Christoph.Egger@amd.com>: >>> >>> >>> Hi, >>> >>> configure fails on NetBSD that there is no brctl. >>> NetBSD has no brctl, NetBSD has brconfig. >> >> >> Sorry Christoph, the brctl test is going away (from configure at least). > > > There are some more Linux specific tests: > - ip > - uuid_clear in libuuidThanks for reporting, I will take care of those once the patch has passed the test system (sorry for that, but I cannot test on so many systems).> Also the check for yajl_alloc in libyajl fails although I have libyajl > version 1 installed. The check may be correct but configure shouldn't fail > if version 1 is still supported.Since I guess you have it in /usr/pkg/, have you set PREPEND_LIB or APPEND_LIB env variables before executing configure?> > > Christoph > > -- > ---to satisfy European Law for business letters: > Advanced Micro Devices GmbH > Einsteinring 24, 85689 Dornach b. Muenchen > Geschaeftsfuehrer: Alberto Bozzo, Andrew Bowd > Sitz: Dornach, Gemeinde Aschheim, Landkreis Muenchen > Registergericht Muenchen, HRB Nr. 43632 > > > _______________________________________________ > Xen-devel mailing list > Xen-devel@lists.xen.org > http://lists.xen.org/xen-devel_______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel
On 02/22/12 11:30, Roger Pau Monné wrote:> 2012/2/22 Christoph Egger<Christoph.Egger@amd.com>: >> On 02/22/12 10:59, Roger Pau Monné wrote: >>> >>> 2012/2/22 Christoph Egger<Christoph.Egger@amd.com>: >>>> >>>> >>>> Hi, >>>> >>>> configure fails on NetBSD that there is no brctl. >>>> NetBSD has no brctl, NetBSD has brconfig. >>> >>> >>> Sorry Christoph, the brctl test is going away (from configure at least). >> >> >> There are some more Linux specific tests: >> - ip >> - uuid_clear in libuuid > > Thanks for reporting, I will take care of those once the patch has > passed the test system (sorry for that, but I cannot test on so many > systems). > >> Also the check for yajl_alloc in libyajl fails although I have libyajl >> version 1 installed. The check may be correct but configure shouldn't fail >> if version 1 is still supported. > > Since I guess you have it in /usr/pkg/, have you set PREPEND_LIB or > APPEND_LIB env variables before executing configure?No, I have set EXTRA_LIB and EXTRA_INCLUDES. I tried both PREPEND_LIB and APPEND_LIB and that works. Oh and the integer type checks fail. Snippet from config.log: configure:7600: checking for int64_t configure:7600: gcc -c -I/usr/pkg/include -g -O2 conftest.c >&5 conftest.c: In function 'main': conftest.c:90: error: expected ')' before ';' token conftest.c:91: error: expected expression before ']' token configure: 7600: $? = 1 The unsigned integer type checks succeed. Christoph -- ---to satisfy European Law for business letters: Advanced Micro Devices GmbH Einsteinring 24, 85689 Dornach b. Muenchen Geschaeftsfuehrer: Alberto Bozzo, Andrew Bowd Sitz: Dornach, Gemeinde Aschheim, Landkreis Muenchen Registergericht Muenchen, HRB Nr. 43632 _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel
2012/2/22 Christoph Egger <Christoph.Egger@amd.com>:> On 02/22/12 11:30, Roger Pau Monné wrote: >> >> 2012/2/22 Christoph Egger<Christoph.Egger@amd.com>: >>> >>> On 02/22/12 10:59, Roger Pau Monné wrote: >>>> >>>> >>>> 2012/2/22 Christoph Egger<Christoph.Egger@amd.com>: >>>>> >>>>> >>>>> >>>>> Hi, >>>>> >>>>> configure fails on NetBSD that there is no brctl. >>>>> NetBSD has no brctl, NetBSD has brconfig. >>>> >>>> >>>> >>>> Sorry Christoph, the brctl test is going away (from configure at least). >>> >>> >>> >>> There are some more Linux specific tests: >>> - ip >>> - uuid_clear in libuuid >> >> >> Thanks for reporting, I will take care of those once the patch has >> passed the test system (sorry for that, but I cannot test on so many >> systems). >> >>> Also the check for yajl_alloc in libyajl fails although I have libyajl >>> version 1 installed. The check may be correct but configure shouldn't >>> fail >>> if version 1 is still supported. >> >> >> Since I guess you have it in /usr/pkg/, have you set PREPEND_LIB or >> APPEND_LIB env variables before executing configure? > > > No, I have set EXTRA_LIB and EXTRA_INCLUDES. > I tried both PREPEND_LIB and APPEND_LIB and that works.EXTRA_LIB and EXTRA_INCLUDES is deprecated in favor of PREPEND_ and APPEND_, which it's basically the same but allows a more fine grained control.> > Oh and the integer type checks fail. Snippet from config.log: > > configure:7600: checking for int64_t > configure:7600: gcc -c -I/usr/pkg/include -g -O2 conftest.c >&5 > conftest.c: In function 'main': > conftest.c:90: error: expected ')' before ';' token > conftest.c:91: error: expected expression before ']' token > configure: 7600: $? = 1This was added by default when running autoscan, I guess we can disable those checks.> > The unsigned integer type checks succeed. > > > Christoph > > -- > ---to satisfy European Law for business letters: > Advanced Micro Devices GmbH > Einsteinring 24, 85689 Dornach b. Muenchen > Geschaeftsfuehrer: Alberto Bozzo, Andrew Bowd > Sitz: Dornach, Gemeinde Aschheim, Landkreis Muenchen > Registergericht Muenchen, HRB Nr. 43632 >_______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel
2012/2/22 Christoph Egger <Christoph.Egger@amd.com>:> On 02/22/12 10:59, Roger Pau Monné wrote: >> >> 2012/2/22 Christoph Egger<Christoph.Egger@amd.com>: >>> >>> >>> Hi, >>> >>> configure fails on NetBSD that there is no brctl. >>> NetBSD has no brctl, NetBSD has brconfig. >> >> >> Sorry Christoph, the brctl test is going away (from configure at least). > > > There are some more Linux specific tests: > - ip > - uuid_clear in libuuidI'm going to fix this right now. Should I commit the output from autoconf also with my patches Ian? I'm asking this because we use different versions and the output is slightly different.> Also the check for yajl_alloc in libyajl fails although I have libyajl > version 1 installed. The check may be correct but configure shouldn't fail > if version 1 is still supported. > > > Christoph > > -- > ---to satisfy European Law for business letters: > Advanced Micro Devices GmbH > Einsteinring 24, 85689 Dornach b. Muenchen > Geschaeftsfuehrer: Alberto Bozzo, Andrew Bowd > Sitz: Dornach, Gemeinde Aschheim, Landkreis Muenchen > Registergericht Muenchen, HRB Nr. 43632 > > > _______________________________________________ > Xen-devel mailing list > Xen-devel@lists.xen.org > http://lists.xen.org/xen-devel_______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel
On Thu, Feb 23, Roger Pau Monné wrote:> 2012/2/22 Christoph Egger <Christoph.Egger@amd.com>: > > On 02/22/12 10:59, Roger Pau Monné wrote: > >> > >> 2012/2/22 Christoph Egger<Christoph.Egger@amd.com>: > >>> > >>> > >>> Hi, > >>> > >>> configure fails on NetBSD that there is no brctl. > >>> NetBSD has no brctl, NetBSD has brconfig. > >> > >> > >> Sorry Christoph, the brctl test is going away (from configure at least). > > > > > > There are some more Linux specific tests: > > - ip > > - uuid_clear in libuuid > > I'm going to fix this right now. Should I commit the output from > autoconf also with my patches Ian? I'm asking this because we use > different versions and the output is slightly different.I also think configure should just check compile time requirements. ip(1) is most likely not needed to compile and link xen+tools. Olaf _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel
Roger Pau Monné writes ("Re: [Xen-devel] configure
failure"):> I''m going to fix this right now. Should I commit the output from
> autoconf also with my patches Ian? I''m asking this because we use
> different versions and the output is slightly different.
I don''t mind whether you commit the output from autoconf and include
it in your patch. It''s probably easier for other people not to have
to read the autoconf output diff if your version is different to mine.
BUT please include a note instructing me to rerun autogen.sh. Or I
might just possibly forget...
Ian.
Olaf Hering writes ("Re: [Xen-devel] configure
failure"):> I also think configure should just check compile time requirements.
> ip(1) is most likely not needed to compile and link xen+tools.
Indeed.
Ian.
2012/2/29 Ian Jackson <Ian.Jackson@eu.citrix.com>:> Roger Pau Monné writes ("Re: [Xen-devel] configure failure"): >> I'm going to fix this right now. Should I commit the output from >> autoconf also with my patches Ian? I'm asking this because we use >> different versions and the output is slightly different. > > I don't mind whether you commit the output from autoconf and include > it in your patch. It's probably easier for other people not to have > to read the autoconf output diff if your version is different to mine. > > BUT please include a note instructing me to rerun autogen.sh. Or I > might just possibly forget...Nevermind, I've installed the same version that you used, so changes to configure script are minimal and related to the actual changes being made in configure.ac.> Ian._______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel