[jhalfs] 03/03: Fix blfs tools Makefile now that MAKEFLAGS is set

"Git Owner" ([email protected] via alfs-discuss Mailing List) <[email protected]>
Newsgroups gmane.linux.lfs.automated
Message-ID <[email protected]>
This is an automated email from the git hooks/post-receive script.

git pushed a commit to branch xry111/parallelism
in repository jhalfs.

commit da4bce358df104cc3e0ed59503eafb8c92c0db1e
Author: Pierre Labastie <[email protected]>
AuthorDate: Fri Nov 17 13:59:06 2023 +0100

    Fix blfs tools Makefile now that MAKEFLAGS is set
    
    Previously, the "update" target was just added to the end of
    the "all" target prerequisites, so that it was run at the end.
    But now, we enter make with some -j value in MAKEFLAGS, so that
    the "update" target may well be run before the other targets, which
    is wrong: for example if it is run before the "xxx-z-libxslt" target,
    xsltproc is not found and the update target fails.
    To fix, define "update" as the default target, and have it depend
    on "all".
---
 common/libs/func_install_blfs | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/common/libs/func_install_blfs b/common/libs/func_install_blfs
index 4261932..32a7851 100644
--- a/common/libs/func_install_blfs
+++ b/common/libs/func_install_blfs
@@ -218,10 +218,10 @@ pushd $BUILDDIR$BLFS_ROOT/work
 # installation, using libxslt, which is not installed yet. So move
 # updating to the end of the process, adding an 'update' target
 sed -i -e '/xsltproc/,+6d' \
-       -e '/^all/s@$@ update@' \
+       -e '/^all/i update:' \
        -e 's/touch/@touch/' Makefile
 cat >> Makefile << EOF
-update:
+update: all
 	@echo Updating the tracking file
 	@for pack in \$\$(grep '<productname' ../$LFS_XML/tmp/lfs-full.xml | \\
 		sed 's/.*>\([^<]*\)<.*/\1/' | \\

-- 
To stop receiving notification emails like this one, please contact
the administrator of this repository.

-- 
http://lists.linuxfromscratch.org/sympa/info/alfs-discuss
Unsubscribe: See the above information page
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.