http://codereview.appspot.com/3381041/diff/1/hgnested/__init__.py File hgnested/__init__.py (right): http://codereview.appspot.com/3381041/diff/1/hgnested/__init__.py#newcode24 hgnested/__init__.py:24: if mercurial_version >= '1.7': On 2010/12/02 20:18:02, Ian Wilson ...
14 years, 4 months ago
(2010-12-02 20:49:15 UTC)
#3
http://codereview.appspot.com/3381041/diff/1/hgnested/__init__.py File hgnested/__init__.py (right): http://codereview.appspot.com/3381041/diff/1/hgnested/__init__.py#newcode24 hgnested/__init__.py:24: if mercurial_version >= '1.7': On 2010/12/02 20:49:15, ced wrote: ...
14 years, 4 months ago
(2010-12-02 21:27:38 UTC)
#4
http://codereview.appspot.com/3381041/diff/1/hgnested/__init__.py
File hgnested/__init__.py (right):
http://codereview.appspot.com/3381041/diff/1/hgnested/__init__.py#newcode24
hgnested/__init__.py:24: if mercurial_version >= '1.7':
On 2010/12/02 20:49:15, ced wrote:
> On 2010/12/02 20:18:02, Ian Wilson wrote:
> > Comparing strings seems like it could be problematic eventually. Ie.
('1.10'
> <
> > '1.7') == True. Doesn't setuptools or distutils provide a proper version
> wrapper
> > for comparison?
>
> I did not find better solution.
Perhaps I could use something like:
map(int, mercurial_version.split('.')) >= [1, 7]
But it will be for an other patch.
Issue 3381041: hgnested patch to work with mercurial 1.7
(Closed)
Created 14 years, 4 months ago by ced
Modified 14 years, 4 months ago
Reviewers: yangoon1, Ian Wilson
Base URL:
Comments: 3