Re: [PATCH] docs: Drop unused rule for internals/%.html.tmp target

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

 



On Mon, Aug 31, 2015 at 05:22:23PM +0200, Martin Kletzander wrote:
> On Thu, Aug 20, 2015 at 10:50:54AM +0200, Guido Günther wrote:
> >On Thu, Aug 20, 2015 at 03:12:36AM +0200, Michal Privoznik wrote:
> >>On 19.08.2015 14:40, Guido Günther wrote:
> >>> We're using the %.html.tmp for all html files now so drop the unused one
> >>> and rather make sure the needed directory exists.
> >>>
> >>> This fixes build failures as described in
> >>>
> >>>     https://www.redhat.com/archives/libvir-list/2015-August/msg00603.html
> >>> ---
> 
> I'm glad it fixed your problem, but now it doesn't build on old
> autoconf/automake.  And it also generates the pages using different
> schema.  What was the issue for you (why you originally needed this
> patch)?  Couldn't that be fixed differently and this one reverted?

I had some off list discussion about Centos 5 and it seems the MKDIR_P
in configure.ac just isn't expanded correctly, that could be worked
around, I guess.

The reason I proposed this change was that from 1.2.17 on the build
failed with:

make[4]: Entering directory '/tmp/buildd/libvirt-1.2.17~rc1/debian/build/docs'
missing XHTML1 DTD
cat: internals/locking.html.tmp: No such file or directory
Makefile:2385: recipe for target 'internals/locking.html' failed
make[4]: *** [internals/locking.html] Error 1

that was caused by locking.html.tmp using the %.html.tmp: instead of the
internals/%.html.tmp: rule.

So the rule removal was more of a cleanup. Creating the directory via
configure.ac was already sufficient to "fix" the problem.

See:
    https://www.redhat.com/archives/libvir-list/2015-June/msg01603.html

for some more details.
Cheers,
 -- Guido

--
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]