db-derby-dev mailing list archives

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

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

Eli Barzilay commented on DERBY-6925:
-------------------------------------

1. Yes, I haven't seen that file distributed, but I have seen its
   contents in other places (IIIRC, it was in the start script version
   for linux).

2. To see the failure, try

       expr $foo : '.*sh.exe$'

   and you'll get a syntax error since $foo doesn't exist; but if you
   quote it:

       expr "$foo" : '.*sh.exe$'

   it works fine whether it exists or not.  It's true that the chances
   of $SHELL not being set are small, but in a docker environment that's
   exactly what I ran into...


> 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
(v6.3.15#6346)

Mime
View raw message