On Tue, Aug 25, 2009 at 02:54:16PM +0200, Michael Hanselmann wrote: > 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.
Oh, we temp-temp-dired stuff? LGTM then :) iustin
