Ensured exit status is set on failure

.github/workflows/bld.yml

@@ -22,11 +22,9 @@ jobs:
           distribution: "zulu"
           java-version: ${{ matrix.java-version }}
 
-      - name: Grant execute permission for bld
-        run: chmod +x bld
-
-      - name: Download the dependencies
+      - name: Download dependencies
         run: ./bld download
 
-      - name: Run tests with bld
+
+      - name: Run tests
         run: ./bld compile test

.idea/libraries/bld.xml

@@ -2,12 +2,12 @@
   <library name="bld">
     <CLASSES>
       <root url="file://$PROJECT_DIR$/lib/bld" />
-      <root url="jar://$USER_HOME$/.bld/dist/bld-1.9.1.jar!/" />
+      <root url="jar://$USER_HOME$/.bld/dist/bld-2.0.0-SNAPSHOT.jar!/" />
     </CLASSES>
     <JAVADOC />
     <SOURCES>
       <root url="file://$PROJECT_DIR$/lib/bld" />
-      <root url="jar://$USER_HOME$/.bld/dist/bld-1.9.1-sources.jar!/" />
+      <root url="jar://$USER_HOME$/.bld/dist/bld-2.0.0-SNAPSHOT-sources.jar!/" />
     </SOURCES>
     <excluded>
       <root url="jar://$PROJECT_DIR$/lib/bld/bld-wrapper.jar!/" />

.vscode/settings.json

@@ -9,7 +9,7 @@
     ],
     "java.configuration.updateBuildConfiguration": "automatic",
     "java.project.referencedLibraries": [
-        "${HOME}/.bld/dist/bld-1.9.1.jar",
+        "${HOME}/.bld/dist/bld-2.0.0-SNAPSHOT.jar",
         "lib/**/*.jar"
     ]
 }

lib/bld/bld-wrapper.properties

@@ -1,8 +1,8 @@
 bld.downloadExtensionJavadoc=false
 bld.downloadExtensionSources=true
 bld.downloadLocation=
-bld.extension-jacoco=com.uwyn.rife2:bld-jacoco-report:0.9.5
-bld.extension-pmd=com.uwyn.rife2:bld-pmd:1.0.1
+bld.extension-jacoco=com.uwyn.rife2:bld-jacoco-report:0.9.6
+bld.extension-pmd=com.uwyn.rife2:bld-pmd:1.1.2
 bld.repositories=MAVEN_CENTRAL,MAVEN_LOCAL,RIFE2_SNAPSHOTS,RIFE2_RELEASES
 bld.sourceDirectories=
-bld.version=1.9.1
+bld.version=2.0.0-SNAPSHOT

src/bld/java/rife/bld/extension/ExecOperationBuild.java

@@ -25,8 +25,7 @@ import rife.bld.publish.PublishScm;
 import java.io.IOException;
 import java.util.List;
 
-import static rife.bld.dependencies.Repository.MAVEN_CENTRAL;
-import static rife.bld.dependencies.Repository.RIFE2_RELEASES;
+import static rife.bld.dependencies.Repository.*;
 import static rife.bld.dependencies.Scope.compile;
 import static rife.bld.dependencies.Scope.test;
 import static rife.bld.operations.JavadocOptions.DocLinkOption.NO_MISSING;
@@ -35,20 +34,20 @@ public class ExecOperationBuild extends Project {
     public ExecOperationBuild() {
         pkg = "rife.bld.extension";
         name = "ExecOperation";
-        version = version(1, 0, 1);
+        version = version(1, 0, 2, "SNAPSHOT");
 
         javaRelease = 17;
 
         downloadSources = true;
         autoDownloadPurge = true;
 
-        repositories = List.of(MAVEN_CENTRAL, RIFE2_RELEASES);
+        repositories = List.of(MAVEN_LOCAL, MAVEN_CENTRAL, RIFE2_RELEASES, RIFE2_SNAPSHOTS);
         scope(compile)
-                .include(dependency("com.uwyn.rife2", "bld", version(1, 9, 1)));
+                .include(dependency("com.uwyn.rife2", "bld", version(2, 0, 0, "SNAPSHOT")));
         scope(test)
-                .include(dependency("org.junit.jupiter", "junit-jupiter", version(5, 10, 2)))
-                .include(dependency("org.junit.platform", "junit-platform-console-standalone", version(1, 10, 2)))
-                .include(dependency("org.assertj", "assertj-core", version(3, 26, 0)));
+                .include(dependency("org.junit.jupiter", "junit-jupiter", version(5, 10, 3)))
+                .include(dependency("org.junit.platform", "junit-platform-console-standalone", version(1, 10, 3)))
+                .include(dependency("org.assertj", "assertj-core", version(3, 26, 3)));
 
         javadocOperation()
                 .javadocOptions()
@@ -91,14 +90,14 @@ public class ExecOperationBuild extends Project {
     }
 
     @BuildCommand(summary = "Generates JaCoCo Reports")
-    public void jacoco() throws IOException {
+    public void jacoco() throws Exception {
         new JacocoReportOperation()
                 .fromProject(this)
                 .execute();
     }
 
     @BuildCommand(summary = "Runs PMD analysis")
-    public void pmd() {
+    public void pmd() throws Exception {
         new PmdOperation()
                 .fromProject(this)
                 .failOnViolation(true)

src/main/java/rife/bld/extension/ExecOperation.java

@@ -18,9 +18,9 @@ package rife.bld.extension;
 
 import rife.bld.BaseProject;
 import rife.bld.operations.AbstractOperation;
+import rife.bld.operations.exceptions.ExitStatusException;
 
 import java.io.File;
-import java.io.IOException;
 import java.util.ArrayList;
 import java.util.Collection;
 import java.util.List;
@@ -89,37 +89,49 @@ public class ExecOperation extends AbstractOperation<ExecOperation> {
     @Override
     public void execute() throws Exception {
         if (project_ == null) {
-            LOGGER.severe("A project must be specified.");
-        }
-
-        final File workDir = Objects.requireNonNullElseGet(workDir_,
-                () -> new File(project_.workDirectory().getAbsolutePath()));
-
-        if (LOGGER.isLoggable(Level.INFO)) {
-            LOGGER.info("Working directory: " + workDir.getAbsolutePath());
-        }
-
-        if (workDir.isDirectory()) {
-            var pb = new ProcessBuilder();
-            pb.inheritIO();
-            pb.command(args_.stream().toList());
-            pb.directory(workDir);
+            if (LOGGER.isLoggable(Level.SEVERE) && !silent()) {
+                LOGGER.severe("A project must be specified.");
+            }
+            throw new ExitStatusException(ExitStatusException.EXIT_FAILURE);
+        } else {
+            final File workDir = Objects.requireNonNullElseGet(workDir_,
+                    () -> new File(project_.workDirectory().getAbsolutePath()));
 
             if (LOGGER.isLoggable(Level.INFO)) {
-                LOGGER.info(String.join(" ", args_));
+                LOGGER.info("Working directory: " + workDir.getAbsolutePath());
             }
 
-            var proc = pb.start();
-            var err = proc.waitFor(timeout_, TimeUnit.SECONDS);
+            if (workDir.isDirectory()) {
+                var pb = new ProcessBuilder();
+                pb.inheritIO();
+                pb.command(args_.stream().toList());
+                pb.directory(workDir);
 
-            if (!err) {
-                proc.destroy();
-                throw new IOException("The command timed out.");
-            } else if (proc.exitValue() != 0 && failOnExit_) {
-                throw new IOException("The command exit value/status is: " + proc.exitValue());
+                if (LOGGER.isLoggable(Level.INFO) && !silent()) {
+                    LOGGER.info(String.join(" ", args_));
+                }
+
+                var proc = pb.start();
+                var err = proc.waitFor(timeout_, TimeUnit.SECONDS);
+
+                if (!err) {
+                    proc.destroy();
+                    if (LOGGER.isLoggable(Level.SEVERE) && !silent()) {
+                        LOGGER.severe("The command timed out.");
+                    }
+                    throw new ExitStatusException(ExitStatusException.EXIT_FAILURE);
+                } else if (proc.exitValue() != 0 && failOnExit_) {
+                    if (LOGGER.isLoggable(Level.SEVERE) && !silent()) {
+                        LOGGER.severe("The command exit value/status is: " + proc.exitValue());
+                    }
+                    ExitStatusException.throwOnFailure(proc.exitValue());
+                }
+            } else {
+                if (LOGGER.isLoggable(Level.SEVERE) && !silent()) {
+                    LOGGER.severe("Invalid working directory: " + workDir);
+                }
+                throw new ExitStatusException(ExitStatusException.EXIT_FAILURE);
             }
-        } else {
-            throw new IOException("Invalid working directory: " + workDir);
         }
     }