The file permissions aren't preserved when unzipping and copying the JRE about packr HOT 7 CLOSED

Actually, it should not affect Packr, it works even though java doesn't have the executable bit.

from packr.

Actually, it causes ProcessBuilder to throw:

java.io.IOException: Cannot run program "java": error=0, spawn failed
	at java.lang.UNIXProcess.forkAndExec(Native Method)
	at java.lang.UNIXProcess.<init>(UNIXProcess.java:248)
	at java.lang.ProcessImpl.start(ProcessImpl.java:134)
	at java.lang.ProcessBuilder.start(ProcessBuilder.java:1029)

@code-disaster Can we please re-open this issue?

It shouldn't matter, but just in case:

OS: OS X 10.12.1 (Sierra)
JRE: 1.8.0_72-b15 (Oracle)

P.S. Maybe related to #70?

from packr.

Just in case there was any doubt, the issue is resolved by chmod +x jre/bin/java.

from packr.

@code-disaster This could be remedied in Packr.java at L208, L220, & L230 by using Java 7 nio: Files.copy (source, target, StandardCopyOption.COPY_ATTRIBUTES) in conjunction with nio Path Operations instead of Apache Commons IO.

from packr.

Ah. Looks like you're still trying to support Java 6. Any good reason for this? Can we require Java 7 and get access to nio?

from packr.

• Java 6: historical reasons. Just didn't bother yet.
• Attributes: good point, but I don't know if this works when e.g. packing a Mac or Linux bundle on a Windows host, because of the different file system attributes.
• ProcessBuilder: just curious, why do you want to fork java?

from packr.

ProcessBuilder: just curious, why do you want to fork java?

To launch a host-and-play java game server from a java client.

Attributes: good point, but I don't know if this works when e.g. packing a Mac or Linux bundle on a Windows host, because of the different file system attributes.

Only one way to find out! But think about it - packr is already failing to preserve any permissions. Why not make a best attempt with nio, and at least log a warning if permissions won't be preserved (i.e., nio throws on unsupported CopyOption(s)). That would be 100x better than the current behavior imo.

from packr.