Fixed for GRAILS-9778 - Cygwin shows empty path #316

Merged
merged 1 commit into from Dec 2, 2013

Conversation

Projects
None yet
3 participants
@chanwit
Contributor

chanwit commented Jan 21, 2013

Here's a pull request for Issue: GRAILS-9778.

I took the solution from this discussion:
http://grails.1312388.n4.nabble.com/Grails-bug-with-cygwin-terminal-td4639236.html

@bond-

This comment has been minimized.

Show comment Hide comment
@bond-

bond- Jan 22, 2013

You should have used:
if [ ! -z $CP ] ; then or if [ -n $CP ]; then or if [ $CP ]; then

http://tldp.org/LDP/Bash-Beginners-Guide/html/sect_07_01.html

You should have used:
if [ ! -z $CP ] ; then or if [ -n $CP ]; then or if [ $CP ]; then

http://tldp.org/LDP/Bash-Beginners-Guide/html/sect_07_01.html

@chanwit

This comment has been minimized.

Show comment Hide comment
@chanwit

chanwit Jan 22, 2013

Owner

Thank you, Ravi. I am not good at scripting bash. So I've just used similar codes from startGrails script.

Owner

chanwit commented on df8367e Jan 22, 2013

Thank you, Ravi. I am not good at scripting bash. So I've just used similar codes from startGrails script.

This comment has been minimized.

Show comment Hide comment
@bond-

bond- Jan 22, 2013

Please change your commit to accommodate that change 👍

Please change your commit to accommodate that change 👍

graemerocher added a commit that referenced this pull request Dec 2, 2013

Merge pull request #316 from chanwit/patch-1
Fixed for GRAILS-9778 - Cygwin shows empty path

@graemerocher graemerocher merged commit 2700458 into grails:master Dec 2, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment