Skip to content

Commit 565ac8d

Browse files
committed
Avoid illegal filenames when building Documentation on NTFS
A '+' is not a valid part of a filename with Windows file systems (it is reserved because the '+' operator meant file concatenation back in the DOS days). Let's just not use it. Signed-off-by: Johannes Schindelin <[email protected]>
1 parent f36187a commit 565ac8d

File tree

1 file changed

+42
-42
lines changed

1 file changed

+42
-42
lines changed

Documentation/Makefile

Lines changed: 42 additions & 42 deletions
Original file line numberDiff line numberDiff line change
@@ -265,9 +265,9 @@ docdep_prereqs = \
265265
cmd-list.made $(cmds_txt)
266266

267267
doc.dep : $(docdep_prereqs) $(wildcard *.txt) build-docdep.perl
268-
$(QUIET_GEN)$(RM) $@+ $@ && \
269-
$(PERL_PATH) ./build-docdep.perl >$@+ $(QUIET_STDERR) && \
270-
mv $@+ $@
268+
$(QUIET_GEN)$(RM) $@.new $@ && \
269+
$(PERL_PATH) ./build-docdep.perl >$@.new $(QUIET_STDERR) && \
270+
mv $@.new $@
271271

272272
-include doc.dep
273273

@@ -303,23 +303,23 @@ mergetools-list.made: ../git-mergetool--lib.sh $(wildcard ../mergetools/*)
303303
date >$@
304304

305305
clean:
306-
$(RM) *.xml *.xml+ *.html *.html+ *.1 *.5 *.7
307-
$(RM) *.texi *.texi+ *.texi++ git.info gitman.info
306+
$(RM) *.xml *.xml.new *.html *.html.new *.1 *.5 *.7
307+
$(RM) *.texi *.texi.new *.texi.new.new git.info gitman.info
308308
$(RM) *.pdf
309309
$(RM) howto-index.txt howto/*.html doc.dep
310310
$(RM) technical/*.html technical/api-index.txt
311311
$(RM) $(cmds_txt) $(mergetools_txt) *.made
312312
$(RM) manpage-base-url.xsl
313313

314314
$(MAN_HTML): %.html : %.txt asciidoc.conf
315-
$(QUIET_ASCIIDOC)$(RM) $@+ $@ && \
316-
$(TXT_TO_HTML) -d manpage -o $@+ $< && \
317-
mv $@+ $@
315+
$(QUIET_ASCIIDOC)$(RM) $@.new $@ && \
316+
$(TXT_TO_HTML) -d manpage -o $@.new $< && \
317+
mv $@.new $@
318318

319319
$(OBSOLETE_HTML): %.html : %.txto asciidoc.conf
320-
$(QUIET_ASCIIDOC)$(RM) $@+ $@ && \
321-
$(TXT_TO_HTML) -o $@+ $< && \
322-
mv $@+ $@
320+
$(QUIET_ASCIIDOC)$(RM) $@.new $@ && \
321+
$(TXT_TO_HTML) -o $@.new $< && \
322+
mv $@.new $@
323323

324324
manpage-base-url.xsl: manpage-base-url.xsl.in
325325
sed "s|@@MAN_BASE_URL@@|$(MAN_BASE_URL)|" $< > $@
@@ -329,14 +329,14 @@ manpage-base-url.xsl: manpage-base-url.xsl.in
329329
$(XMLTO) -m $(MANPAGE_XSL) $(XMLTO_EXTRA) man $<
330330

331331
%.xml : %.txt asciidoc.conf
332-
$(QUIET_ASCIIDOC)$(RM) $@+ $@ && \
333-
$(TXT_TO_XML) -d manpage -o $@+ $< && \
334-
mv $@+ $@
332+
$(QUIET_ASCIIDOC)$(RM) $@.new $@ && \
333+
$(TXT_TO_XML) -d manpage -o $@.new $< && \
334+
mv $@.new $@
335335

336336
user-manual.xml: user-manual.txt user-manual.conf
337-
$(QUIET_ASCIIDOC)$(RM) $@+ $@ && \
338-
$(TXT_TO_XML) -d book -o $@+ $< && \
339-
mv $@+ $@
337+
$(QUIET_ASCIIDOC)$(RM) $@.new $@ && \
338+
$(TXT_TO_XML) -d book -o $@.new $< && \
339+
mv $@.new $@
340340

341341
technical/api-index.txt: technical/api-index-skel.txt \
342342
technical/api-index.sh $(patsubst %,%.txt,$(API_DOCS))
@@ -350,45 +350,45 @@ XSLT = docbook.xsl
350350
XSLTOPTS = --xinclude --stringparam html.stylesheet docbook-xsl.css
351351

352352
user-manual.html: user-manual.xml $(XSLT)
353-
$(QUIET_XSLTPROC)$(RM) $@+ $@ && \
354-
xsltproc $(XSLTOPTS) -o $@+ $(XSLT) $< && \
355-
mv $@+ $@
353+
$(QUIET_XSLTPROC)$(RM) $@.new $@ && \
354+
xsltproc $(XSLTOPTS) -o $@.new $(XSLT) $< && \
355+
mv $@.new $@
356356

357357
git.info: user-manual.texi
358358
$(QUIET_MAKEINFO)$(MAKEINFO) --no-split -o $@ user-manual.texi
359359

360360
user-manual.texi: user-manual.xml
361-
$(QUIET_DB2TEXI)$(RM) $@+ $@ && \
362-
$(DOCBOOK2X_TEXI) user-manual.xml --encoding=UTF-8 --to-stdout >$@++ && \
363-
$(PERL_PATH) fix-texi.perl <$@++ >$@+ && \
364-
rm $@++ && \
365-
mv $@+ $@
361+
$(QUIET_DB2TEXI)$(RM) $@.new $@ && \
362+
$(DOCBOOK2X_TEXI) user-manual.xml --encoding=UTF-8 --to-stdout >$@.new.new && \
363+
$(PERL_PATH) fix-texi.perl <$@.new.new >$@.new && \
364+
rm $@.new.new && \
365+
mv $@.new $@
366366

367367
user-manual.pdf: user-manual.xml
368-
$(QUIET_DBLATEX)$(RM) $@+ $@ && \
369-
$(DBLATEX) -o $@+ -p $(ASCIIDOC_DBLATEX_DIR)/asciidoc-dblatex.xsl -s $(ASCIIDOC_DBLATEX_DIR)/asciidoc-dblatex.sty $< && \
370-
mv $@+ $@
368+
$(QUIET_DBLATEX)$(RM) $@.new $@ && \
369+
$(DBLATEX) -o $@.new -p $(ASCIIDOC_DBLATEX_DIR)/asciidoc-dblatex.xsl -s $(ASCIIDOC_DBLATEX_DIR)/asciidoc-dblatex.sty $< && \
370+
mv $@.new $@
371371

372372
gitman.texi: $(MAN_XML) cat-texi.perl
373-
$(QUIET_DB2TEXI)$(RM) $@+ $@ && \
373+
$(QUIET_DB2TEXI)$(RM) $@.new $@ && \
374374
($(foreach xml,$(MAN_XML),$(DOCBOOK2X_TEXI) --encoding=UTF-8 \
375-
--to-stdout $(xml) &&) true) > $@++ && \
376-
$(PERL_PATH) cat-texi.perl $@ <$@++ >$@+ && \
377-
rm $@++ && \
378-
mv $@+ $@
375+
--to-stdout $(xml) &&) true) > $@.new.new && \
376+
$(PERL_PATH) cat-texi.perl $@ <$@.new.new >$@.new && \
377+
rm $@.new.new && \
378+
mv $@.new $@
379379

380380
gitman.info: gitman.texi
381381
$(QUIET_MAKEINFO)$(MAKEINFO) --no-split --no-validate $*.texi
382382

383383
$(patsubst %.txt,%.texi,$(MAN_TXT)): %.texi : %.xml
384-
$(QUIET_DB2TEXI)$(RM) $@+ $@ && \
385-
$(DOCBOOK2X_TEXI) --to-stdout $*.xml >$@+ && \
386-
mv $@+ $@
384+
$(QUIET_DB2TEXI)$(RM) $@.new $@ && \
385+
$(DOCBOOK2X_TEXI) --to-stdout $*.xml >$@.new && \
386+
mv $@.new $@
387387

388388
howto-index.txt: howto-index.sh $(wildcard howto/*.txt)
389-
$(QUIET_GEN)$(RM) $@+ $@ && \
390-
'$(SHELL_PATH_SQ)' ./howto-index.sh $(sort $(wildcard howto/*.txt)) >$@+ && \
391-
mv $@+ $@
389+
$(QUIET_GEN)$(RM) $@.new $@ && \
390+
'$(SHELL_PATH_SQ)' ./howto-index.sh $(sort $(wildcard howto/*.txt)) >$@.new && \
391+
mv $@.new $@
392392

393393
$(patsubst %,%.html,$(ARTICLES)) : %.html : %.txt
394394
$(QUIET_ASCIIDOC)$(TXT_TO_HTML) $*.txt
@@ -397,10 +397,10 @@ WEBDOC_DEST = /pub/software/scm/git/docs
397397

398398
howto/%.html: ASCIIDOC_EXTRA += -a git-relative-html-prefix=../
399399
$(patsubst %.txt,%.html,$(wildcard howto/*.txt)): %.html : %.txt
400-
$(QUIET_ASCIIDOC)$(RM) $@+ $@ && \
400+
$(QUIET_ASCIIDOC)$(RM) $@.new $@ && \
401401
sed -e '1,/^$$/d' $< | \
402-
$(TXT_TO_HTML) - >$@+ && \
403-
mv $@+ $@
402+
$(TXT_TO_HTML) - >$@.new && \
403+
mv $@.new $@
404404

405405
install-webdoc : html
406406
'$(SHELL_PATH_SQ)' ./install-webdoc.sh $(WEBDOC_DEST)

0 commit comments

Comments
 (0)