Jhalfs with package management
pierre.labastie at neuf.fr
Fri Jan 27 13:39:37 PST 2012
Le 27/01/2012 17:40, Matt Burgess a écrit :
> Hmm, that change may have caused a regression in chapter06 Glibc, but
> I'll wait for my latest build to get to that point before confirming.
> The last build I tried, with your patch applied,jhalfs bailed out after
> hitting an 'Error 2' output from Glibc's tests. The config I run with
> is 'Run all chapter 6 testsuites' with 'Abort the build at the first
> test suite failure'; that has usually continued on past the packages
> with known failures such as GCC and Glibc.
You are right. I forgot to mention _that_ change: I could not understand
why the scriptlets where generated with 'make -k check || true' when
'Abort the build at the first test suite failure' was selected. So I
the generation of '|| true'. Forgot about known testsuite failures. Sorry.
Here is a patch:
--- LFS/lfs.xsl (revision 3)
+++ LFS/lfs.xsl (working copy)
@@ -306,8 +306,12 @@
<xsl:when test="contains(string(), 'glibc-check-log')">
- <xsl:text>>> $TEST_LOG 2>&1
+ <xsl:text>>> $TEST_LOG 2>&1 || true
+ <xsl:when test="contains(string(), 'make -k')">
+ <xsl:text> >> $TEST_LOG 2>&1 || true
<xsl:if test="not(contains(string(), '>>'))">
With this patch applied, the logic is as follows for any <userinput
-if the input contains 'make -k' : generates
'make -k >> $TEST_LOG 2>&1 || true'
-the special input case (in glibc):
make -k check 2>&1 | tee glibc-check-log
grep Error glibc-check-log
is changed to:
'make -k check >> $TEST_LOG 2>&1 || true'
(no more 'grep Error')
- any line with 'make' (and not make -k)
is changed from 'make <blahblah>' to:
'make -k <blahblah> >> $TEST_LOG 2>&1 || true'
if 'do not stop on testsuite failure'
'make <blahblah> >> $TEST_LOG 2>&1
if 'Abort the build at the first test suite failure'
- any line without 'make' and without '>>' is changed
to the same line + '>> $TEST_LOG 2>&1
Hope it is what was intended, but the original lfs.xsl file
was so complicated that I am not sure I catched the whole
More information about the alfs-discuss