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

Bill Farner commented on AURORA-1273:
-------------------------------------

The change does sound reasonable, patch will be accepted!

> ./gradlew distZip fails when $CDPATH is set
> -------------------------------------------
>
>                 Key: AURORA-1273
>                 URL: https://issues.apache.org/jira/browse/AURORA-1273
>             Project: Aurora
>          Issue Type: Story
>         Environment: OSX 10.10.3
> Aurora 0.7.0-rc3
>            Reporter: Andrew Jorgensen
>            Priority: Minor
>
> When running `./gradlew distZip` after pulling down a fresh copy of Aurora I 
> got the following message:
> {quote}
> (cd thrift-0.9.1 && ./configure --disable-dependency-tracking 
> --disable-shared --without-c_glib --without-cpp --without-csharp --without-d 
> --without-erlang --without-go --without-java --without-haskell --without-perl 
> --without-php --without-php_extension --without-pic --without-python 
> --without-qt4 --without-ruby --without-tests && make -j4)
> /bin/sh: line 0: cd: thrift-0.9.1: No such file or directory
> {quote}
> The offending line was in ./build-support/thrift/Makefile:43 :
> {quote}
> $(THRIFT):
>       curl -s $(THRIFT_URL) | tar zxv
>       (cd $(THRIFT_DISTNAME) && ./configure $(CONFIGURE_FLAGS) && make -j4)
> {quote}
> I was able to fix the issue by changing this to the following (note the './' 
> in front of $(THRIFT_DISTNAME)):
> {quote}
> $(THRIFT):
>       curl -s $(THRIFT_URL) | tar zxv
>       (cd ./$(THRIFT_DISTNAME) && ./configure $(CONFIGURE_FLAGS) && make -j4)
> {quote}
> This problem will happen to anyone with the environmental variable CDPATH set 
> to anything. You can also get around the issue by doing:
> {quote}
> CDPATH=""; ./gradlew distZip
> {quote}
> I think adding the './' before the folder is better because it makes it clear 
> that it is specifying that cd should look for the folder in the current path 
> and not somewhere in the CDPATH.
> I can go ahead and make a patch and submit a PR for this if this change 
> sounds reasonable.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to