Page MenuHomeFreeBSD

bmake: make opt-debug-x-trace broken on Linux
ClosedPublic

Authored by emaste on Feb 9 2022, 1:20 AM.
Tags
None
Referenced Files
F102758253: D34219.diff
Sat, Nov 16, 6:28 PM
Unknown Object (File)
Tue, Nov 12, 4:08 PM
Unknown Object (File)
Wed, Oct 30, 4:00 PM
Unknown Object (File)
Oct 3 2024, 10:06 PM
Unknown Object (File)
Oct 2 2024, 7:15 PM
Unknown Object (File)
Oct 1 2024, 1:46 PM
Unknown Object (File)
Sep 24 2024, 4:29 PM
Unknown Object (File)
Sep 20 2024, 2:04 AM
Subscribers

Details

Summary

It fails during GitHub cross-build actions from Ubuntu 20.04. @sjg reports it is likely an issue with dash.

Failed tests: opt-debug-x-trace
-+ echo 'Counting 1 2 3 4 5 6 7'
++ echo Counting 1 2 3 4 5 6 7

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

emaste requested review of this revision.Feb 9 2022, 1:20 AM
emaste created this revision.
contrib/bmake/unit-tests/Makefile
444

The change I'm about to commit to NetBSD is a bit more involved.
Will take care of this...

Correction, not NetBSD, most BROKEN_TEST stuff goes in bmake, fwiw next version will have:

--- a/bmake/unit-tests/Makefile Tue Feb 08 14:45:13 2022 -0800
+++ b/bmake/unit-tests/Makefile Tue Feb 08 18:45:22 2022 -0800
@@ -446,6 +446,17 @@
 .if ${.MAKE.OS:NDarwin} == ""
 BROKEN_TESTS+= shell-ksh
 .endif
+.if ${.MAKE.OS} == "Linux" && ${.SHELL:tA:T} != "bash"
+.if exists(/etc/os-release)
+distro!= . /etc/os-release && echo $$NAME
+.endif
+# dash fails -x output
+# .SHELL is not bash so may be dash
+# if distro is Ubuntu or we cannot tell, assume the worst
+.if ${distro:U:NUbuntu} == ""
+BROKEN_TESTS+= opt-debug-x-trace
+.endif
+.endif
 .if ${.MAKE.OS} == "SCO_SV"
 BROKEN_TESTS+= \
        opt-debug-graph[23] \

will cleanup at next import

This revision is now accepted and ready to land.Feb 9 2022, 3:16 AM
This revision was automatically updated to reflect the committed changes.