Changeset 16d0182 for Makefile

Timestamp:

01/27/2024 09:42:07 PM (8 months ago)

Author:

Pierre Labastie <pierre.labastie@…>

Branches:

12.1, 12.1-rc1, 12.2, 12.2-rc1, multilib, trunk, xry111/arm64, xry111/clfs-ng, xry111/loongarch, xry111/loongarch-12.1, xry111/loongarch-12.2, xry111/mips64el, xry111/multilib, xry111/update-glibc

Children:

b6dd23c7

Parents:

4ac089b5

git-author:

Pierre Labastie <pierre.labastie@…> (01/27/2024 03:58:34 PM)

git-committer:

Pierre Labastie <pierre.labastie@…> (01/27/2024 09:42:07 PM)

Message:

Makefile: remove "true" after tidy

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

File:

• Makefile (modified) (2 diffs)

Makefile

@@ -54 +54 @@
         $(Q)for filename in `find $(BASEDIR) -name "*.html"`; do \
          tidy -config tidy.conf $$filename;           \
-         true;                                        \
          /bin/bash obfuscate.sh $$filename;           \
          sed -e "s|text/html|application/xhtml+xml|g" \
@@ -99 +98 @@
 
         @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..."