Merge pull request orphan-oss#28 from iheinrich/27_execute_chmod_usin…

…g_process_builder Use ProcessBuilder to launch 'chmod'
thomas3 · Sep 3, 2015 · 4fd440c · 4fd440c
2 parents 88948eb + 5a41ac9
commit 4fd440c
Showing 1 changed file with 5 additions and 6 deletions.
diff --git a/src/main/java/com/akathist/maven/plugins/launch4j/Launch4jMojo.java b/src/main/java/com/akathist/maven/plugins/launch4j/Launch4jMojo.java
@@ -289,7 +289,7 @@ public void execute() throws MojoExecutionException {
  }
 
  File workdir = setupBuildEnvironment();
- 
+
  Config c = new Config();
 
  c.setHeaderType(headerType);
@@ -451,10 +451,9 @@ private File unpackWorkDir(Artifact a) throws MojoExecutionException {
  */
  private void setPermissions(File workdir) {
  if (!System.getProperty("os.name").startsWith("Windows")) {
- Runtime r = Runtime.getRuntime();
  try {
- r.exec("chmod 755 " + workdir + "/bin/ld").waitFor();
- r.exec("chmod 755 " + workdir + "/bin/windres").waitFor();
+ new ProcessBuilder("chmod", "755", workdir + "/bin/ld").start().waitFor();
+ new ProcessBuilder("chmod", "755", workdir + "/bin/windres").start().waitFor();
  } catch (InterruptedException e) {
  getLog().warn("Interrupted while chmodding platform-specific binaries", e);
  } catch (IOException e) {
@@ -468,7 +467,7 @@ private void setPermissions(File workdir) {
  */
  private List<String> relativizeAndCopy(File workdir, List<String> paths) throws MojoExecutionException {
  if(paths == null) return null;
- 
+
  List<String> result = new ArrayList<>();
  for(String path : paths) {
  Path source = basedir.toPath().resolve(path);
@@ -492,7 +491,7 @@ private List<String> relativizeAndCopy(File workdir, List<String> paths) throws
 
  return result;
  }
- 
+
  /**
  * Downloads the platform-specific parts, if necessary.
  */