Grant Henke has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/11735 )

Change subject: [build] Centralize project version to version.txt
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/11735/1/python/setup.py
File python/setup.py:

http://gerrit.cloudera.org:8080/#/c/11735/1/python/setup.py@44
PS1, Line 44:     version_file = open(os.path.join(setup_dir, 
'../version.txt')).read()
> Would the packaging stuff be smart enough to resolve symlinks? If not, hard
Gradle is unaffected because we publish compiled jars. Python is an issue 
because we do source distributions.



--
To view, visit http://gerrit.cloudera.org:8080/11735
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ib654be8657ae3dc2f193056484b5d0743aa1ce95
Gerrit-Change-Number: 11735
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Attila Bukor <[email protected]>
Gerrit-Reviewer: Grant Henke <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mike Percy <[email protected]>
Gerrit-Reviewer: Todd Lipcon <[email protected]>
Gerrit-Comment-Date: Fri, 19 Oct 2018 19:03:38 +0000
Gerrit-HasComments: Yes

Reply via email to