diff --git a/HASH b/HASH index 4ae336f2..02caa49c 100644 --- a/HASH +++ b/HASH @@ -1,4 +1,4 @@ -suse-tumbleweed/x86_64: 0ac23bf5e3b8c21775911bc8d7afc9c9 +suse-tumbleweed/x86_64: 6d39e6eb74af4b013e5b98175aea29ae suse-tumbleweed/i586: 86ea923ca3861900dd03d4b1da44f8b6 debian-10/amd64: 47167eb2d7fe2d845a1b12e0702a6e9b suse-42.3/x86_64: bd0e29fda82e38c89d2ed6f11797ce3a @@ -7,4 +7,4 @@ debian-8/amd64: b16e55fee0967a75aa67a4d859f0455d ubuntu-18.04/amd64: 190b5c45953399fd93354db0621d6f5f debian-10/armhf: 5d92c234d483deaf44c80cad168a8d5d debian-2025.4/amd64: 977965179baa6fd24d0975a05d583eb6 -kali-rolling/amd64: e76880f8ee0608d4220eb77e0abddc62 +kali-rolling/amd64: 6d39e6eb74af4b013e5b98175aea29ae diff --git a/RELEASES b/RELEASES index d83a069c..4c9d0084 100644 --- a/RELEASES +++ b/RELEASES @@ -1,4 +1,4 @@ -suse-tumbleweed/x86_64: 1.0.0-260 +suse-tumbleweed/x86_64: 1.0.0-262 suse-tumbleweed/i586: 1.0.0-44 debian-10/amd64: 1.0.0-47 suse-42.3/x86_64: 1.0.0-48 @@ -7,4 +7,4 @@ debian-8/amd64: 1.0.0-84 ubuntu-18.04/amd64: 1.0.0-99 debian-10/armhf: 1.0.0-102 debian-2025.4/amd64: 1.0.0-172 -kali-rolling/amd64: 1.0.0-261 +kali-rolling/amd64: 1.0.0-262 diff --git a/VERSION b/VERSION index 3f5d3bf1..e77026b9 100644 --- a/VERSION +++ b/VERSION @@ -1 +1 @@ -1.0.0-261-dev +1.0.0-262-dev diff --git a/src/python/jw/pkg/App.py b/src/python/jw/pkg/App.py index af9b9311..32a4db97 100644 --- a/src/python/jw/pkg/App.py +++ b/src/python/jw/pkg/App.py @@ -209,8 +209,9 @@ class App(Base): scope: Scope, names_only: bool, ) -> None: - name = self.strip_module_from_spec(spec) - mod = re.split('([=><]+)', spec)[0].strip() + dep = Dependency(spec) + name = dep.base_name + mod = dep.full_name if names_only: spec = name if spec in buf: diff --git a/src/python/jw/pkg/lib/version/Dependency.py b/src/python/jw/pkg/lib/version/Dependency.py index 8ed8ee3b..01c710c8 100644 --- a/src/python/jw/pkg/lib/version/Dependency.py +++ b/src/python/jw/pkg/lib/version/Dependency.py @@ -22,7 +22,7 @@ class Dependency: # export class Error(ValueError): pass - __SPLIT_RE: ClassVar[re.Pattern[str]] = re.compile('([=><]+)') + __SPLIT_RE: ClassVar[re.Pattern[str]] = re.compile('([~=> str: @@ -45,6 +45,12 @@ class Dependency: # export case 3: if not parts[0] or not parts[2]: self.__raise(f'Invalid dependency spec "{self.__spec}"') + if parts[1] not in ('=', '<', '<=', '>', '>='): + self.__raise( + f'Spec "{self.__spec}": unsupported operator ' + f'"{parts[1]}", supported operators are =, <, ' + '<=, > and >=' + ) if parts[2] == 'REVISION': self.__raise( f'Spec "{self.__spec}": a bare REVISION renders as a ' diff --git a/test/unit/python/jw/pkg/lib/version/test.py b/test/unit/python/jw/pkg/lib/version/test.py index 91783ae3..07817f87 100644 --- a/test/unit/python/jw/pkg/lib/version/test.py +++ b/test/unit/python/jw/pkg/lib/version/test.py @@ -40,7 +40,7 @@ assert d.base_name == 'dev' assert d.full_name == 'dev' # Operators with and without whitespace -for op in ['=', '==', '<', '<=', '>', '>=']: +for op in ['=', '<', '<=', '>', '>=']: d = Dependency(f'foo {op} 1.0') assert d.base_name == 'foo' assert d.full_name == 'foo' @@ -135,14 +135,6 @@ d = Dependency('foo = 1.2.3', app.get_version) assert d.constraint_str(as_range = True) == 'foo = 1.2.3' d = Dependency('foo = 1.0-rc1', app.get_version) assert d.constraint_str(as_range = True) == 'foo = 1.0-rc1' -# unimplemented operators raise -d = Dependency('foo == 1.2.3-45', app.get_version) -try: - d.constraint_str(as_range = True) - assert False, 'Should have raised' -except NotImplementedError: - pass - # Default syntax is SEM_VER d = Dependency('foo < 2.0') assert d.constraint_str(untemplated = False) == 'foo < 2.0' @@ -238,6 +230,26 @@ for bad in ['', 'foo =', ' = 1.0']: except Dependency.Error: pass +# Operators outside the supported set are rejected when parsed, +# not folded into the package name or deferred to expansion +for bad in [ + 'foo ~= 1.0', + 'foo~=1.0', + 'foo ~ 1.0', + 'foo != 1.0', + 'foo!=1.0', + 'foo == 1.0', + 'foo==1.0', + 'foo === 1.0', + 'foo << 1.0', + 'foo >> 1.0', +]: + try: + Dependency(bad).full_name + assert False, f'Should have raised for {bad!r}' + except Dependency.Error as e: + assert 'unsupported operator' in str(e) + # str assert str(Dependency('foo-devel >= 1.0')) == 'foo-devel >= 1.0' assert str(Dependency('foo')) == 'foo'