2009/8/25 Iustin Pop <[email protected]>:
> On Tue, Aug 25, 2009 at 02:18:12PM +0200, Michael Hanselmann wrote:
>> --- a/Makefile.am
>> +++ b/Makefile.am
>>  man/%.7.in man/%.8.in: man/%.sgml man/footer.sgml $(DOCBOOK_WRAPPER)
>>       @test -n "$(DOCBOOK2MAN)" || { echo 'docbook2man' not found during 
>> configure; exit 1; }
>> -     TMPDIR=`mktemp -d` && { \
>> -     $(DOCBOOK_WRAPPER) "$(DOCBOOK2MAN)" $< $$TMPDIR/$(patsubst 
>> man/%.in,%,$@) ; \
>> -     mv $$TMPDIR/$(patsubst man/%.in,%,$@) $@ ; \
>> -     rm -rf "$$TMPDIR" ; \
>> -     }
>> +     $(DOCBOOK_WRAPPER) "$(DOCBOOK2MAN)" $< $(notdir $(@:.in=)) $@
>
> The reason  we use a temp dir here is that docbook2man leaves stuff
> around, e.g. manpage.links and manpage.refs.
>
> Are you sure you don't pollute the src dir with this?

Yes, that's why docbook-wrapper already used a temporary directory.
Just checked again and manpage.{links,refs} end up in the temporary
directory.

Regards,
Michael

Reply via email to