diff --git a/packaging/specifiers.py b/packaging/specifiers.py index 507cb991..6fe4809e 100644 --- a/packaging/specifiers.py +++ b/packaging/specifiers.py @@ -393,18 +393,16 @@ def _compare_equal(self, prospective: Version, spec: str) -> bool: # segment. split_prospective = _version_split(str(prospective)) + # 0-pad the prospective version before shortening it to get the correct + # shortened version. + padded_prospective, _ = _pad_version(split_prospective, split_spec) + # Shorten the prospective version to be the same length as the spec # so that we can determine if the specifier is a prefix of the # prospective version or not. - shortened_prospective = split_prospective[: len(split_spec)] - - # Pad out our two sides with zeros so that they both equal the same - # length. - padded_spec, padded_prospective = _pad_version( - split_spec, shortened_prospective - ) + shortened_prospective = padded_prospective[: len(split_spec)] - return padded_prospective == padded_spec + return shortened_prospective == split_spec else: # Convert our spec string into a Version spec_version = Version(spec) diff --git a/tests/test_specifiers.py b/tests/test_specifiers.py index ce946689..47f2e6e5 100644 --- a/tests/test_specifiers.py +++ b/tests/test_specifiers.py @@ -307,8 +307,11 @@ def test_comparison_non_specifier(self): ("2b1.dev1", "==2.*"), ("2c1", "==2.*"), ("2c1.post1.dev1", "==2.*"), + ("2c1.post1.dev1", "==2.0.*"), ("2rc1", "==2.*"), + ("2rc1", "==2.0.*"), ("2", "==2.*"), + ("2", "==2.0.*"), ("2", "==0!2.*"), ("0!2", "==2.*"), ("2.0", "==2.*"), @@ -404,8 +407,11 @@ def test_comparison_non_specifier(self): ("2b1.dev1", "!=2.*"), ("2c1", "!=2.*"), ("2c1.post1.dev1", "!=2.*"), + ("2c1.post1.dev1", "!=2.0.*"), ("2rc1", "!=2.*"), + ("2rc1", "!=2.0.*"), ("2", "!=2.*"), + ("2", "!=2.0.*"), ("2.0", "!=2.*"), ("2.0.0", "!=2.*"), # Test the greater than equal operation