Request for review (S): 7099454 /bin/sh does not support syntax used in the src/os/posix/launcher/launcher.script shell script
robert.ottenhag at oracle.com
Tue Oct 11 02:27:57 PDT 2011
The removal of bashisms looks fine. I also not you remove some multiple definitions of MYDIR and and the usused RUNDIR.
The following changes
+ REL_MYDIR=`dirname $0`
+ MYDIR=`cd $REL_MYDIR && pwd`
should however be written shorter as
+ MYDIR=`cd \`dirname $0\` && pwd`
> -----Original Message-----
> From: Bengt Rutisson
> Sent: den 11 oktober 2011 11:06
> To: hotspot-gc-dev at openjdk.java.net; hotspot-runtime-dev at openjdk.java.net
> Subject: Request for review (S): 7099454 /bin/sh does not support syntax
> used in the src/os/posix/launcher/launcher.script shell script
> Hi all,
> Sending this review request to both Runtime and GC. I think the change
> is Runtime responsibility, but I would like to submit it to hotspot-gc
> since it is getting in the way of testing that I am doing at the moment.
> The problem is that the hotspot script that the build process creates
> has been updated to use /bin/sh instead of /bin/bash. This is more
> correct since bash is optional. But the /bin/sh shell on Solaris does
> not accept some of the syntax in the script.
> In particular the $() notation is not supported by Solaris /bin/sh. The
> script also contains the bash specific "source" command.
> The fix is to change this to use `` and . respectively.
> 7099454 /bin/sh does not support syntax used in the
> src/os/posix/launcher/launcher.script shell script
> Tested the new script on Linux (Ubuntu) and Solaris. Ran JPRT.
More information about the hotspot-gc-dev