diff --git a/.idea/libraries/bld.xml b/.idea/libraries/bld.xml index 48e8fd5..59ffe59 100644 --- a/.idea/libraries/bld.xml +++ b/.idea/libraries/bld.xml @@ -2,12 +2,12 @@ - + - + diff --git a/.vscode/settings.json b/.vscode/settings.json index d136e4d..9e8368e 100644 --- a/.vscode/settings.json +++ b/.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-1.9.0.jar", "lib/**/*.jar" ] } diff --git a/README.md b/README.md index 9b2a9a1..c707871 100755 --- a/README.md +++ b/README.md @@ -2,7 +2,7 @@ [![License](https://img.shields.io/badge/license-Apache%20License%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) [![Java](https://img.shields.io/badge/java-17%2B-blue)](https://www.oracle.com/java/technologies/javase/jdk17-archive-downloads.html) -[![bld](https://img.shields.io/badge/1.9.1-FA9052?label=bld&labelColor=2392FF)](https://rife2.com/bld) +[![bld](https://img.shields.io/badge/1.9.0-FA9052?label=bld&labelColor=2392FF)](https://rife2.com/bld) [![Release](https://flat.badgen.net/maven/v/metadata-url/repo.rife2.com/releases/com/uwyn/rife2/bld-exec/maven-metadata.xml?color=blue)](https://repo.rife2.com/#/releases/com/uwyn/rife2/bld-exec) [![Snapshot](https://flat.badgen.net/maven/v/metadata-url/repo.rife2.com/snapshots/com/uwyn/rife2/bld-exec/maven-metadata.xml?label=snapshot)](https://repo.rife2.com/#/snapshots/com/uwyn/rife2/bld-exec) [![GitHub CI](https://github.com/rife2/bld-exec/actions/workflows/bld.yml/badge.svg)](https://github.com/rife2/bld-exec/actions/workflows/bld.yml) @@ -21,9 +21,9 @@ public void startServer() throws Exception { } ``` -## Exit Value +## Exit Status -Use the `failOnExit` function to specify whether a command non-zero exit value (status) constitutes a failure. +Use the `failOnExit` function to specify whether a command non-zero exit status constitutes a failure. ```java @BuildCommand diff --git a/lib/bld/bld-wrapper.jar b/lib/bld/bld-wrapper.jar index ffdaf71..82762f1 100644 Binary files a/lib/bld/bld-wrapper.jar and b/lib/bld/bld-wrapper.jar differ diff --git a/lib/bld/bld-wrapper.properties b/lib/bld/bld-wrapper.properties index cab667f..89f7a72 100644 --- a/lib/bld/bld-wrapper.properties +++ b/lib/bld/bld-wrapper.properties @@ -1,8 +1,8 @@ bld.downloadExtensionJavadoc=false bld.downloadExtensionSources=true -bld.extension-pmd=com.uwyn.rife2:bld-pmd:0.9.9 +bld.extension-pmd=com.uwyn.rife2:bld-pmd:0.9.8 bld.extension-jacoco=com.uwyn.rife2:bld-jacoco-report:0.9.5 bld.repositories=MAVEN_CENTRAL,MAVEN_LOCAL,RIFE2_SNAPSHOTS,RIFE2_RELEASES bld.downloadLocation= bld.sourceDirectories= -bld.version=1.9.1 +bld.version=1.9.0 diff --git a/src/bld/java/rife/bld/extension/ExecOperationBuild.java b/src/bld/java/rife/bld/extension/ExecOperationBuild.java index 46e4875..37db549 100644 --- a/src/bld/java/rife/bld/extension/ExecOperationBuild.java +++ b/src/bld/java/rife/bld/extension/ExecOperationBuild.java @@ -35,7 +35,7 @@ public class ExecOperationBuild extends Project { public ExecOperationBuild() { pkg = "rife.bld.extension"; name = "ExecOperation"; - version = version(1, 0, 1, "SNAPSHOT"); + version = version(1, 0, 0); javaRelease = 17; @@ -44,7 +44,7 @@ public class ExecOperationBuild extends Project { repositories = List.of(MAVEN_CENTRAL, RIFE2_RELEASES); scope(compile) - .include(dependency("com.uwyn.rife2", "bld", version(1, 9, 1))); + .include(dependency("com.uwyn.rife2", "bld", version(1, 9, 0))); 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))) diff --git a/src/main/java/rife/bld/extension/ExecOperation.java b/src/main/java/rife/bld/extension/ExecOperation.java index af192d0..f07d740 100644 --- a/src/main/java/rife/bld/extension/ExecOperation.java +++ b/src/main/java/rife/bld/extension/ExecOperation.java @@ -24,7 +24,6 @@ import java.io.IOException; import java.util.ArrayList; import java.util.Collection; import java.util.List; -import java.util.Objects; import java.util.concurrent.TimeUnit; import java.util.logging.Level; import java.util.logging.Logger; @@ -40,8 +39,8 @@ public class ExecOperation extends AbstractOperation { private final List args_ = new ArrayList<>(); private boolean failOnExit_ = true; private BaseProject project_; - private int timeout_ = 30; - private File workDir_; + private int timeout = 30; + private String workDir_; /** * Configures the command and arguments to be executed. @@ -82,11 +81,11 @@ public class ExecOperation extends AbstractOperation { 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()); + final File workDir; + if (workDir_ == null || workDir_.isBlank()) { + workDir = new File(project_.workDirectory().getAbsolutePath()); + } else { + workDir = new File(workDir_); } if (workDir.isDirectory()) { @@ -100,21 +99,21 @@ public class ExecOperation extends AbstractOperation { } var proc = pb.start(); - var err = proc.waitFor(timeout_, TimeUnit.SECONDS); + var err = proc.waitFor(timeout, TimeUnit.SECONDS); 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()); + throw new IOException("The command exit status is: " + proc.exitValue()); } } else { - throw new IOException("Invalid working directory: " + workDir); + throw new IOException("Invalid work directory: " + workDir); } } /** - * Configures whether the operation should fail if the command exit value/status is not 0. + * Configures whether the operation should fail if the command exit status is not 0. *

* Default is {@code TRUE} * @@ -144,28 +143,18 @@ public class ExecOperation extends AbstractOperation { * @return this operation instance */ public ExecOperation timeout(int timeout) { - timeout_ = timeout; + this.timeout = timeout; return this; } /** - * Configures the working directory. - * - * @param dir the directory - * @return this operation instance - */ - public ExecOperation workDir(File dir) { - workDir_ = dir; - return this; - } - - /** - * Configures the working directory. + * Configures the work directory. * * @param dir the directory path * @return this operation instance */ public ExecOperation workDir(String dir) { - return workDir(new File(dir)); + workDir_ = dir; + return this; } } diff --git a/src/test/java/rife/bld/extension/ExecOperationTest.java b/src/test/java/rife/bld/extension/ExecOperationTest.java index d070e15..e0b584c 100644 --- a/src/test/java/rife/bld/extension/ExecOperationTest.java +++ b/src/test/java/rife/bld/extension/ExecOperationTest.java @@ -53,12 +53,12 @@ class ExecOperationTest { } @Test - void testExitValue() { + void testExitStatus() { assertThatCode(() -> new ExecOperation() .fromProject(new BaseProject()) .command(List.of("cat", FOO)) - .execute()).message().contains("exit value/status"); + .execute()).message().contains("exit status"); } @Test @@ -83,21 +83,11 @@ class ExecOperationTest { @Test void testWorkDir() { - assertThatCode(() -> - new ExecOperation() - .fromProject(new BaseProject()) - .command("echo", FOO) - .workDir(new File(System.getProperty("java.io.tmpdir"))) - .execute()).doesNotThrowAnyException(); - } - - @Test - void testWorkDirInvalid() { assertThatCode(() -> new ExecOperation() .fromProject(new BaseProject()) .command("echo") .workDir(FOO) - .execute()).message().startsWith("Invalid working directory: ").endsWith(FOO); + .execute()).message().startsWith("Invalid work directory: ").endsWith(FOO); } }