ansible / mazer

Experimental Ansible Galaxy Content Manager
GNU General Public License v3.0
114 stars 18 forks source link

Requirement spec cmp error 255 #260

Closed alikins closed 5 years ago

alikins commented 5 years ago
SUMMARY

Make version_spec not part of ReqSpec attr cmp

semantic_version.Spec() does not have comparison methods (ie, < or __lt__) so it should not be used for when comparing two RequirementSpec objects.

So set the version_spec attr to cmp=False so RequirementSpec doesn't use it for compare (and so cmps of Requirement that then compare RequirementSpec dont use version_spec for cmp)

Add unit test for sorting/cmp on Requirement

Fixes: #255

ISSUE TYPE
MAZER VERSION
name = mazer
version = 0.5.0
config_file = /home/adrian/.ansible/mazer.yml
uname = Linux, newswoop, 5.0.5-200.fc29.x86_64, #1 SMP Wed Mar 27 20:58:04 UTC 2019, x86_64
executable_location = /home/adrian/venvs/mazer_0.4.0_py36/bin/mazer
python_version = 3.6.8 (default, Jan 27 2019, 09:00:23) [GCC 8.2.1 20181215 (Red Hat 8.2.1-6)]
python_executable = /home/adrian/venvs/mazer_0.4.0_py36/bin/python