share/containers/, .github/workflows/: Don't make(1) twice

It was being done so that the second one prints errors without races.
However, the same thing can be achieved by passing -Orecurse to make(1).

And this makes the logs even more readable, since there's no racy output
at all.

Fixes: 97f79e3b2715 ("CI: Make build logs more readable")
Link: <https://github.com/shadow-maint/shadow/pull/702>
Link: <https://github.com/nginx/unit/pull/1123>
Acked-by: Iker Pedrosa <ipedrosa@redhat.com>
Cc: Andrew Clayton <a.clayton@nginx.com>
Cc: Konstantin Pavlov <thresh@nginx.com>
Cc: Dylan Arbour <https://github.com/arbourd>
Signed-off-by: Alejandro Colomar <alx@kernel.org>

diff --git a/.github/workflows/static-code-analysis.yml b/.github/workflows/static-code-analysis.yml
index b9cfb4e9da4389cd03afaa89f82d27b95385ad0d..59e4a16d98843da94569469db6e5a5b13883d8dc 100644 (file)
--- a/.github/workflows/static-code-analysis.yml
+++ b/.github/workflows/static-code-analysis.yml
@@ -32,10 +32,7 @@ jobs:
     - name: Build shadow-utils
       run: |
         PROCESSORS=$(/usr/bin/getconf _NPROCESSORS_ONLN)
-        make -kj$PROCESSORS || true
-
-    - name: Check build errors
-      run: make
+        make -Orecurse -j$PROCESSORS
 
     - name: Perform CodeQL Analysis
       uses: github/codeql-action/analyze@v2

diff --git a/share/containers/alpine.dockerfile b/share/containers/alpine.dockerfile
index e1886e40c6864a42e9bc528459c1eecb809d107b..f6d9f11c1641be28335e2530e2e2290ddfa8f417 100644 (file)
--- a/share/containers/alpine.dockerfile
+++ b/share/containers/alpine.dockerfile
@@ -26,8 +26,7 @@ RUN ./autogen.sh \
        --disable-man \
        --disable-nls \
        --with-yescrypt
-RUN make -kj4 || true
-RUN make
+RUN make -Orecurse -j4
 RUN bash -c "trap 'cat <tests/unit/test-suite.log >&2' ERR; make check;"
 RUN make install
 

diff --git a/share/containers/debian.dockerfile b/share/containers/debian.dockerfile
index 9a30296728fd0d4514b583befb2649b28f693f34..408c6fa10345d1c2fc86d46ebe23042a32d576cc 100644 (file)
--- a/share/containers/debian.dockerfile
+++ b/share/containers/debian.dockerfile
@@ -23,8 +23,7 @@ RUN ./autogen.sh \
        --without-selinux \
        --enable-man \
        --with-yescrypt
-RUN make -kj4 || true
-RUN make
+RUN make -Orecurse -j4
 RUN bash -c "trap 'cat <tests/unit/test-suite.log >&2' ERR; make check;"
 RUN make install
 

diff --git a/share/containers/fedora.dockerfile b/share/containers/fedora.dockerfile
index 5ad9d418f6330e02f6e0ed0414dd887929e6f89f..93c6b22015b16f01e6796aeec6d5705e44e50564 100644 (file)
--- a/share/containers/fedora.dockerfile
+++ b/share/containers/fedora.dockerfile
@@ -25,8 +25,7 @@ RUN ./autogen.sh \
         --with-group-name-max-length=32 \
        --enable-lastlog \
        --enable-logind=no
-RUN make -kj4 || true
-RUN make
+RUN make -Orecurse -j4
 RUN bash -c "trap 'cat <tests/unit/test-suite.log >&2' ERR; make check;"
 RUN make install