gronod/lfs
1
0
Fork 0
mirror of https://git.linuxfromscratch.org/lfs.git synced 2025-06-19 03:39:20 +01:00
Code Issues Packages Projects Releases Wiki Activity

Makefile: remove "true" after tidy

Browse Source 
If in a series of commands, and not the last, true has no effect
If in the last command, it is better to exit if there is a real
error in tidy, so use "|| test $$? -le 1", but only when tidy is
the last in a series of commands

Part of a patch by Boian Berberov
This commit is contained in:
Pierre Labastie 2024-01-27 16:58:34 +01:00
parent 4ac089b5b2
commit 16d01822df
1 changed files with 1 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
Makefile
View File

@ -53,7 +53,6 @@ book: validate profile-html
@echo "Running Tidy and obfuscate.sh..." @echo "Running Tidy and obfuscate.sh..."
$(Q)for filename in `find $(BASEDIR) -name "*.html"`; do \ $(Q)for filename in `find $(BASEDIR) -name "*.html"`; do \
tidy -config tidy.conf $$filename; \ tidy -config tidy.conf $$filename; \
true; \
/bin/bash obfuscate.sh $$filename; \ /bin/bash obfuscate.sh $$filename; \
sed -e "s|text/html|application/xhtml+xml|g" \ sed -e "s|text/html|application/xhtml+xml|g" \
-i $$filename; \ -i $$filename; \
@ -98,7 +97,7 @@ nochunks: validate profile-html
$(RENDERTMP)/lfs-html.xml $(RENDERTMP)/lfs-html.xml
@echo "Running Tidy..." @echo "Running Tidy..."
$(Q)tidy -config tidy.conf $(BASEDIR)/$(NOCHUNKS_OUTPUT) || true $(Q)tidy -config tidy.conf $(BASEDIR)/$(NOCHUNKS_OUTPUT) || test $$? -le 1
@echo "Running obfuscate.sh..." @echo "Running obfuscate.sh..."
$(Q)bash obfuscate.sh $(BASEDIR)/$(NOCHUNKS_OUTPUT) $(Q)bash obfuscate.sh $(BASEDIR)/$(NOCHUNKS_OUTPUT)