Code review comment for lp:~arky/ubuntu/karmic/envyng-core/fix-411915

Revision history for this message
James Westby (james-w) wrote :

> assert DistUtilsExtra.auto.__version__ >= '2.10', 'needs DistUtilsExtra.auto
> >= 2.10'
>
> or, if you want to compare floating point numbers:
> assert float(DistUtilsExtra.auto.__version__) >= 2.10, 'needs
> DistUtilsExtra.auto >= 2.10'

Well, that's changing the requirement isn't it? Plus it will make
2.2 pass the check, when it isn't supported.

Thanks,

James

« Back to merge proposal