Re: [Xen-devel] Build problems with latest xen-unstable.hg

[Roger Pau Monnà <roger.pau@xxxxxxxxxxxxx>]

2012/2/3 Ian Campbell <Ian.Campbell@xxxxxxxxxx>:
> On Fri, 2012-02-03 at 00:24 +0000, Dan Magenheimer wrote:
>> I'm building xen-devel from scratch for the first time in a long
>> time on a clean EL6u1 machine and running into a couple of problems.
>>
>> 1) The build of tools/firmware/etherboot does a wget of a weird
>> Â Âpxe tar file that looks like it has a git hash in the filename.
>> Â ÂI hacked around that (to wget the 1.0.0 version of the file).
>
> This one is known. ixpe hasn't done a release for ages but we needed a
> newer version for some bug fixes, hence we depend on a git revision now.
>
> It is expected that the wget fails and then it falls back to generating
> the tarball locally from a git checkout. I think this is pretty lame but
> it is apparently a consequence of how the Makefile is structured. We
> previously discussed putting a copy of that tarball on xenbits -- not
> sure what happened to that plan but I have done it now and the wget
> works for me (TM).
>
>> 2) Building blktap2 complains about a missing "uuid/uuid.h". ÂI
>> Â Âdid install the uuid and uuid-devel packages and there IS
>> Â Âa /usr/include/uuid.h but no /usr/include/uuid/uuid.h. ÂI found
>> Â ÂI also needed to install libuuid-devel (which didn't get
>> Â Âchecked in advance apparently, just uuid-devel and uuid I think).
>
> tools/check/check_uuid_devel looks for both "uuid.h" and "uuid/uuid.h",
> in that order but at least some headers (e.g. libxl_uuid.h, blktap's
> ones etc) use:
> Â Â Â Â#if __linux__
> Â Â Â Â#include <uuid/uuid.h>
> Â Â Â Â#elif __BSD__
> Â Â Â Â#include <uuid.h>
>
> It seems that on EL you can end up with uuid.h but not uuid/uuid.h which
> confuses the check into succeeding where it shouldn't.
>
> Please can you confirm that on EL6 uuid-devel
> includes /usr/include/uuid.h and libuuid-devel
> includes /usr/include/uuid/uuid.h
>
> Roger, can you handle this (Linux vs. BSD?) header distinction in your
> autoconf patch?

I've send an updated version that contains this check. The patch looks
ok in my repository, and this time I've sent it as an attachment,
let's see if that solves the whitespace problem.

>> 3) Missing texinfo package stops the build before it successfully
>> Â Âcompletes. ÂCan this be check'ed in advance?
>
> Please can you post this log so we can find where the texinfo
> requirement comes from?
>
> Normally for these things we would patch them to only build the docs if
> the required tool is present.
>
>> Since I haven't built xen-devel in a long time, I don't know if
>> these are recent problems with tip or old problems that don't
>> show up on [Debian and whatever other envs other developers are
>> using] but do show up on EL6. ÂSo I thought I'd report them.
>
> I think it's a mixture of old and new. Thanks for reporting.
>
> Ian.
>
>

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel