db-derby-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Bryan Pendleton (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (DERBY-6925) Bug in derby_common.sh
Date Thu, 09 Mar 2017 04:22:38 GMT

    [ https://issues.apache.org/jira/browse/DERBY-6925?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15902454#comment-15902454

Bryan Pendleton commented on DERBY-6925:

As of DERBY-3207, derby_common.sh is not directly part of the release.

However, the contents of that file are present in the other scripts, e.g., bin/startNetworkServer
if [ \( "`expr $SHELL : '.*sh.exe$'`" -gt 0 \) -a \( "$cygwin" = "false" \) ]; then
  LOCALCLASSPATH=`echo $LOCALCLASSPATH | sed -E 's/([\d\w]*):([\d\w]*)/\1;\2/g'`

The Derby build tools construct the release version of the Unix scripts in the
bin directory by including the contents of the derby_common.sh script inside the
containing scripts, that is what was done by DERBY-3207.

I still don't understand what the particular problem is with quotes on your system,
nor do I understand what you think would be an improvement to these scripts.

Please clarify, if you can.

> Bug in derby_common.sh
> ----------------------
>                 Key: DERBY-6925
>                 URL: https://issues.apache.org/jira/browse/DERBY-6925
>             Project: Derby
>          Issue Type: Bug
>            Reporter: Eli Barzilay
>            Priority: Minor
>   Original Estimate: 1h
>  Remaining Estimate: 1h
> At the end of the `derby_common.sh` file, as I currently see in the GH
> mirror:
>     https://github.com/apache/derby/blob/trunk/bin/templates/derby_common.sh#L166
> there is an unquoted use of $SHELL which should really be quoted.  We
> have run into this bug trying to start the server in a docker image,
> with a setup that has no SHELL variable, which results in a syntax
> error.

This message was sent by Atlassian JIRA

View raw message