Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Corrected some long-form getopt option-specifiers #3

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mikehaben69
Copy link

The set-imsi, set-mnclen and set-milenage option specifier strings each need a trailing equals sign, otherwise the getopt.getopt() function does not parse the subsequent argument correctly. Symptom is that
python3 sysmo-isim-tool.sja2.py --adm1 12345678 -J 001010000056789
works, but
python3 sysmo-isim-tool.sja2.py --adm1 12345678 --set-imsi 001010000056789
does not (and similarly -N works but --set-mnclen does not).

…ls sign, otherwise the following argument is not parsed correctly
@laf0rge
Copy link
Member

laf0rge commented May 17, 2022

Thanks! Please note that github is just a read-only mirror of the tool, which is maintained at https://gitea.sysmocom.de/sysmocom/sysmo-usim-tool

It would be great if you could raise your PR there, so we can review + merge there before it will be mirrored back to github as a read-only mirror. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants