Re: [PATCH] build: Create needed folders without dependency tracking

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Nov 30, 2015 at 15:30:21 +0100, Martin Kletzander wrote:

This is actually a v2 AFAIK.

> The parameter --disable-dependency-tracking is supposed to speed up
> one-time build due to the fact that it disables some dependency
> extractors that, apparently, take longer time to execute.  That is a
> problem for code that is generated into builddir (especially some
> specific subdirectory) because the directory it should be installed to
> does not exists in VPATH and without the dependency tracking is not
> created.  Generating such file hence fails with -ENOENT.  In order to
> keep generating files into builddir instead of srcdir, we must create
> the directory ourselves.  This should finally fix the problem that is
> being fixed multiple times since its introduction in commit a9fe62037214
> and let us continue with cleaning those parts of Makefiles that depend
> on generating files into the srcdir rather than builddir as it should
> be.

Not sure whether the last sentence is worth to be in the commit message.

> 
> Signed-off-by: Martin Kletzander <mkletzan@xxxxxxxxxx>
> ---
>  src/Makefile.am | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/src/Makefile.am b/src/Makefile.am
> index bcbee9de11d6..93b9c66662a1 100644
> --- a/src/Makefile.am
> +++ b/src/Makefile.am
> @@ -181,6 +181,7 @@ MAINTAINERCLEANFILES += util/virkeymaps.h
> 
>  util/virkeymaps.h: $(srcdir)/util/keymaps.csv	\
>  		$(srcdir)/util/virkeycode-mapgen.py
> +	$(MKDIR_P) util/
>  	$(AM_V_GEN)$(PYTHON) $(srcdir)/util/virkeycode-mapgen.py \
>  	  <$(srcdir)/util/keymaps.csv >util/virkeymaps.h

ACK, although I doubt that this is universal enough to allow doing the
cleanup you are mentioning in the last sentence. This will fix only this
one particular case, and for other serializations, other generated files
may fail.

Peter

Attachment: signature.asc
Description: Digital signature

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]