Attention is currently required from: dexter, laforge, neels.

fixeria has posted comments on this change by laforge. ( 
https://gerrit.osmocom.org/c/pysim/+/41642?usp=email )

Change subject: setup.py: Align cmd2 minimum version with requirements.txt
......................................................................


Patch Set 1:

(1 comment)

Patchset:

PS1:
I believe it's worth mentioning that `debian:trixie` provides `2.5.11`, and 
`pySim-shell` still works fine with that older version. The only problem was 
artifacts in auto-generated documentation (https://osmocom.org/issues/6776). 
The version was bumped here: https://gerrit.osmocom.org/c/pysim/+/40952 and 
honestly I am not sure if this alone is a strong reason to require `2.6.2`.

Another thing I wanted to bring up with backwards compatibility quirks in 
`pySim-shell.py` (grep for `cmd2.__version__`). If we go for bumping the 
minimum version to `2.6.2`, it no longer makes sense to have the quirks for 
`<2.3.0` and `<2.0.0`.



--
To view, visit https://gerrit.osmocom.org/c/pysim/+/41642?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings?usp=email

Gerrit-MessageType: comment
Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: I71cee0ec3ed2abec68ec567beaab13c868721dad
Gerrit-Change-Number: 41642
Gerrit-PatchSet: 1
Gerrit-Owner: laforge <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: dexter <[email protected]>
Gerrit-Reviewer: fixeria <[email protected]>
Gerrit-Reviewer: neels <[email protected]>
Gerrit-Attention: neels <[email protected]>
Gerrit-Attention: laforge <[email protected]>
Gerrit-Attention: dexter <[email protected]>
Gerrit-Comment-Date: Wed, 10 Dec 2025 14:20:05 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No

Reply via email to