Re: [Xen-devel] [PATCH v7] build: add autoconf to replace custom checks in tools/check

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

2012/2/21 Ian Campbell <Ian.Campbell@xxxxxxxxxx>:
> On Tue, 2012-02-21 at 13:48 +0000, Ian Campbell wrote:
>> On Tue, 2012-02-21 at 12:59 +0000, Roger Pau Monnà wrote:
>>
>> arm: move check for CONFIG_DTB_FILE to xen/arch/arm/Makefile
>>
>> CONFIG_DTB_FILE only needs to be set when building Xen itself.
>>
>> Signed-off-by: David Vrabel <david.vrabel@xxxxxxxxxx>
>
> Oops, pasted something here unintentionally!
>
>> I'm doing a fresh
>> checkout and I will try to reproduce the error,
>> > following the same steps.
>>
>> @@ -40,11 +40,9 @@ dist: DESTDIR=$(DISTDIR)/install
>> Âdist: dist-xen dist-kernels dist-tools dist-stubdom dist-docs dist-misc
>>
>> Âdist-misc:
>> - Â Â Â $(INSTALL_DIR) $(DISTDIR)/check
>>
>> Previously this would have implicitly also created $(DISTDIR) I think
>> you want:
>> - Â Â Â $(INSTALL_DIR) $(DISTDIR)/check
>> + Â Â Â $(INSTALL_DIR) $(DISTDIR)/

Yes, it's probably this that's causing the error, what I don't
understand is why it has never happened to me, I've done a fresh test
of this patch more than once. Probably dist-misc target was executed
after some other target already created the dist folder.

Thanks, I will post an updated version (again).

>>
>> Ian.
>>
>>
>> _______________________________________________
>> Xen-devel mailing list
>> Xen-devel@xxxxxxxxxxxxx
>> http://lists.xen.org/xen-devel
>
>

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel