[GitHub] mdeuser commented on a change in pull request #404: corrected description of go build steps

[GitBox <gi...@apache.org>]

mdeuser commented on a change in pull request #404: corrected description of go build steps
URL: https://github.com/apache/incubator-openwhisk-cli/pull/404#discussion_r252788645
 
 

 ##########
 File path: README.md
 ##########
 @@ -114,8 +114,9 @@ Then download the source code of the OpenWhisk CLI and the dependencies by
 typing:
 
 ```
-$ cd $GOPATH
-$ go get github.com/apache/incubator-openwhisk-cli
+$ mkdir -p $GOPATH/github.com/apache
+$ cd $GOPATH/github.com/apache
+$ git clone https://github.com/apache/incubator-openwhisk-cli
 
 Review comment:
   i recommend retaining the "go" flavor of this installation, so instead of the `git` command, how about just using the `-d` (download only; no compile/install) option.
   ```
   go get -d github.com/apache/incubator-openwhisk-cli
   ```
   this should avoid the
   ```
   src\github.com\apache\incubator-openwhisk-cli\wski18n\i18n.go:98:16: undefined: Asset
   ```
   error seen without the `-d` option
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services