[jhalfs] 02/05: Only replace $(nproc) in scriptlets

"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 3c43655ec646c94ab3da44b72e856bc890468f25
Author: Pierre Labastie <[email protected]>
AuthorDate: Wed Nov 15 18:10:07 2023 +0100

    Only replace $(nproc) in scriptlets
    
    There is no more $(nproc || echo 1). Note that we may replace
    it with $(nproc) if using all cores, so don't replace it in
    the comment-test template (otherwise we have infinite recursion).
---
 LFS/lfs.xsl                  | 16 ++--------------
 common/libs/func_book_parser | 14 ++++++--------
 2 files changed, 8 insertions(+), 22 deletions(-)

diff --git a/LFS/lfs.xsl b/LFS/lfs.xsl
index 3b575ec..49fbdf2 100644
--- a/LFS/lfs.xsl
+++ b/LFS/lfs.xsl
@@ -29,8 +29,7 @@
   <xsl:param name="testsuite" select="1"/>
 
   <!-- Parallelism (LFS >= 12.1) -->
-  <xsl:param name="jobs_1" select="1"/>
-  <xsl:param name="jobs_2" select="1"/>
+  <xsl:param name="jobs" select="1"/>
 
   <!-- Install non wide character ncurses 5? -->
   <xsl:param name="ncurses5" select="'n'"/>
@@ -621,11 +620,8 @@ unset OLD_PKGDIR
       <xsl:when test="contains(string(.),'&lt;lfs&gt;')">
         <xsl:value-of select="$hostname"/>
       </xsl:when>
-      <xsl:when test="contains(string(.),'$(nproc || echo 1)')">
-        <xsl:value-of select="$jobs_1"/>
-      </xsl:when>
       <xsl:when test="contains(string(.),'$(nproc)')">
-        <xsl:value-of select="$jobs_2"/>
+        <xsl:value-of select="$jobs"/>
       </xsl:when>
       <xsl:otherwise>
         <xsl:text>**EDITME</xsl:text>
@@ -1092,14 +1088,6 @@ LOGLEVEL="</xsl:text>
     <xsl:param name="instructions" select="''"/>
     <xsl:param name="eof-seen" select="false()"/>
     <xsl:choose>
-      <xsl:when test="contains($instructions, '$(nproc)')">
-        <xsl:call-template name="comment-test">
-          <xsl:with-param
-            name="instructions"
-            select="concat(substring-before($instructions, '$(nproc)'), $jobs_2, substring-after($instructions, '$(nproc)'))"/>
-          <xsl:with-param name="eof-seen" select="$eof-seen"/>
-        </xsl:call-template>
-      </xsl:when>
       <xsl:when test="contains($instructions,'&#xA;')">
         <xsl:call-template name="comment-test">
           <xsl:with-param name="instructions"
diff --git a/common/libs/func_book_parser b/common/libs/func_book_parser
index 5dbd502..6b7435b 100644
--- a/common/libs/func_book_parser
+++ b/common/libs/func_book_parser
@@ -63,13 +63,12 @@ extract_commands() {         #
   fi
   popd > /dev/null
 
-  JOBS_1=1
-  JOBS_2=1
-  case "$OPTIMIZE" in
-    "1") JOBS_2="$N_PARALLEL";;
-    "2") JOBS_1="$N_PARALLEL"; JOBS_2="$N_PARALLEL";;
-  esac
+  if [ "$ALL_CORES" = "y" ]; then
+	  JOBS="\$(nproc)"
+  else
+	  JOBS="$N_PARALLEL"
 
+  fi
   # First profile the book, for revision and arch. Note that
   # MULTIBLIB is set to "default" if pure 64 bit book. In this case,
   # profiling on arch is useless, but does not hurt either.
@@ -109,8 +108,7 @@ extract_commands() {         #
            --stringparam local          "$LOCAL"        \
            --stringparam log-level      "$LOG_LEVEL"    \
            --stringparam script-root    "$SCRIPT_ROOT"  \
-           --stringparam jobs_1         "$JOBS_1"       \
-           --stringparam jobs_2         "$JOBS_2"       \
+           --stringparam jobs           "$JOBS"         \
            --output "./${COMMANDS}/"                    \
            $XSL                                         \
            prbook.xml >> $LOGDIR/$LOG 2>&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.