You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2020/06/02 14:06:37 UTC

[GitHub] [arrow] bkietz commented on a change in pull request #7292: ARROW-8471: [C++][Integration] Represent 64 bit integers as strings

bkietz commented on a change in pull request #7292:
URL: https://github.com/apache/arrow/pull/7292#discussion_r433904059



##########
File path: dev/archery/archery/integration/tester_go.py
##########
@@ -26,7 +26,8 @@ class GoTester(Tester):
     CONSUMER = True
 
     # FIXME(sbinet): revisit for Go modules
-    GOPATH = os.getenv('GOPATH', '~/go')
+    HOME = os.getenv('HOME', '~')

Review comment:
       I don't know why `~` failed resolve to `$HOME` locally but this seemed like a way to save someone the next person from the frustration of reading the script to figure out that `GOPATH` needs set.




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to 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