dexter has uploaded this change for review. ( https://gerrit.osmocom.org/c/libosmo-netif/+/27210 )
Change subject: amr: cosmetic: fix grammer in comment
......................................................................
amr: cosmetic: fix grammer in comment
Change-Id: I01ad26986d925acdcdb3760f89b8a85dccdc3d5b
---
M src/amr.c
1 file changed, 2 insertions(+), 2 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/libosmo-netif refs/changes/10/27210/1
diff --git a/src/amr.c b/src/amr.c
index 7bc24f5..add4285 100644
--- a/src/amr.c
+++ b/src/amr.c
@@ -103,8 +103,8 @@
* mode normally relys on out of band methods that explicitly select
* one of the two modes. (See also RFC 3267, chapter 3.8). However the
* A interface in GSM does not provide ways to communicate which mode
- * is used exactly used. The following functions uses some heuristics
- * to check if an AMR payload is octet aligned or not. */
+ * is exactly used. The following functions uses some heuristics to
+ * check if an AMR payload is octet aligned or not. */
struct amr_hdr *oa_hdr = (struct amr_hdr *)payload;
unsigned int frame_len;
--
To view, visit https://gerrit.osmocom.org/c/libosmo-netif/+/27210
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: libosmo-netif
Gerrit-Branch: master
Gerrit-Change-Id: I01ad26986d925acdcdb3760f89b8a85dccdc3d5b
Gerrit-Change-Number: 27210
Gerrit-PatchSet: 1
Gerrit-Owner: dexter <pmaier(a)sysmocom.de>
Gerrit-MessageType: newchange
laforge has uploaded this change for review. ( https://gerrit.osmocom.org/c/pysim/+/27209 )
Change subject: pySim-shell: export: Add FCP template to export
......................................................................
pySim-shell: export: Add FCP template to export
The FCP template provides us a lot of context, like the permissions of
a given file. Let's make it part of the 'export' output, both in raw
and in decoded form.
Change-Id: I05f17bbebd7a9b3535204b821900851a5f66e88f
Closes: OS#5457
---
M pySim-shell.py
M pySim/filesystem.py
2 files changed, 6 insertions(+), 1 deletion(-)
git pull ssh://gerrit.osmocom.org:29418/pysim refs/changes/09/27209/1
diff --git a/pySim-shell.py b/pySim-shell.py
index 12f35fd..ad76ba0 100755
--- a/pySim-shell.py
+++ b/pySim-shell.py
@@ -532,6 +532,8 @@
structure = self._cmd.rs.selected_file_structure()
self._cmd.poutput("# structure: %s" % str(structure))
+ self._cmd.poutput("# RAW FCP Template: %s" % str(self._cmd.rs.selected_file_fcp_hex))
+ self._cmd.poutput("# Decoded FCP Template: %s" % str(self._cmd.rs.selected_file_fcp))
for f in df_path_list:
self._cmd.poutput("select " + str(f))
diff --git a/pySim/filesystem.py b/pySim/filesystem.py
index 8303a4b..f8b86c5 100644
--- a/pySim/filesystem.py
+++ b/pySim/filesystem.py
@@ -1221,6 +1221,8 @@
self.card = card
self.selected_file = self.mf # type: CardDF
self.profile = profile
+ self.selected_file_fcp = None
+ self.selected_file_fcp_hex = None
# make sure the class and selection control bytes, which are specified
# by the card profile are used
@@ -1464,7 +1466,8 @@
select_resp = f.decode_select_response(data)
else:
select_resp = self.probe_file(name, cmd_app)
- # store the decoded FCP for later reference
+ # store the raw + decoded FCP for later reference
+ self.selected_file_fcp_hex = data
self.selected_file_fcp = select_resp
self._select_post(cmd_app)
--
To view, visit https://gerrit.osmocom.org/c/pysim/+/27209
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: I05f17bbebd7a9b3535204b821900851a5f66e88f
Gerrit-Change-Number: 27209
Gerrit-PatchSet: 1
Gerrit-Owner: laforge <laforge(a)osmocom.org>
Gerrit-MessageType: newchange
laforge has uploaded this change for review. ( https://gerrit.osmocom.org/c/pysim/+/27208 )
Change subject: docs/legcay.txt: Point to pySim-shell as replacement
......................................................................
docs/legcay.txt: Point to pySim-shell as replacement
Change-Id: I9ca6b9d8c35e23be2ec8752107bb7d1e4f6f9bc1
---
M docs/legacy.rst
1 file changed, 8 insertions(+), 0 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/pysim refs/changes/08/27208/1
diff --git a/docs/legacy.rst b/docs/legacy.rst
index ec3ddfc..d93358f 100644
--- a/docs/legacy.rst
+++ b/docs/legacy.rst
@@ -4,6 +4,9 @@
*legacy tools* are the classic ``pySim-prog`` and ``pySim-read`` programs that
existed long before ``pySim-shell``.
+These days, you should primarily use ``pySim-shell`` instead of these
+legacy tools.
+
pySim-prog
----------
@@ -45,6 +48,11 @@
``pySim-read`` allows you to read some data from a SIM card. It will only some files
of the card, and will only read files accessible to a normal user (without any special authentication)
+These days, you should use the ``export`` command of ``pySim-shell``
+instead. It performs a much more comprehensive export of all of the
+[standard] files that can be found on the card. To get a human-readable
+decode instead of the raw hex export, you can use ``export --json``.
+
Specifically, pySim-read will dump the following:
* MF
--
To view, visit https://gerrit.osmocom.org/c/pysim/+/27208
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: I9ca6b9d8c35e23be2ec8752107bb7d1e4f6f9bc1
Gerrit-Change-Number: 27208
Gerrit-PatchSet: 1
Gerrit-Owner: laforge <laforge(a)osmocom.org>
Gerrit-MessageType: newchange
laforge has submitted this change. ( https://gerrit.osmocom.org/c/pysim/+/27207 )
Change subject: README.md: Remove old usage examples, refer to user manual instead
......................................................................
README.md: Remove old usage examples, refer to user manual instead
We want people to use pySim-shell and should not mislead them by
having usage examples of old tools in README.md. Also, all
documentation should be in the manuals, let's try to have bits
and pieces in various places.
Change-Id: I8c07a2e0778ab95fb42be6074acb80874e681d20
---
M README.md
1 file changed, 20 insertions(+), 36 deletions(-)
Approvals:
Jenkins Builder: Verified
laforge: Looks good to me, approved
diff --git a/README.md b/README.md
index 7f46085..5e1937e 100644
--- a/README.md
+++ b/README.md
@@ -93,46 +93,30 @@
<https://osmocom.org/projects/cellular-infrastructure/wiki/Gerrit>
-Usage Examples
---------------
+Documentation
+-------------
- * Program customizable SIMs. Two modes are possible:
+The pySim user manual can be built from this very source code by means
+of sphinx (with sphinxcontrib-napoleon and sphinx-argparse). See the
+Makefile in the 'docs' directory.
- - one where you specify every parameter manually:
-```
-./pySim-prog.py -n 26C3 -c 49 -x 262 -y 42 -i <IMSI> -s <ICCID>
-```
+A pre-rendered HTML user manual of the current pySim 'git master' is
+available from <https://downloads.osmocom.org/docs/latest/pysim/> and
+a downloadable PDF version is published at
+<https://downloads.osmocom.org/docs/latest/osmopysim-usermanual.pdf>.
- - one where they are generated from some minimal set:
-```
-./pySim-prog.py -n 26C3 -c 49 -x 262 -y 42 -z <random_string_of_choice> -j <card_num>
-```
+A slightly dated video presentation about pySim-shell can be found at
+<https://media.ccc.de/v/osmodevcall-20210409-laforge-pysim-shell>.
-With ``<random_string_of_choice>`` and ``<card_num>``, the soft will generate
-'predictable' IMSI and ICCID, so make sure you choose them so as not to
-conflict with anyone. (for e.g. your name as ``<random_string_of_choice>`` and
-0 1 2 ... for ``<card num>``).
-You also need to enter some parameters to select the device:
+pySim-shell vs. legacy tools
+----------------------------
- -t TYPE : type of card (``supersim``, ``magicsim``, ``fakemagicsim`` or try ``auto``)
- -d DEV : Serial port device (default ``/dev/ttyUSB0``)
- -b BAUD : Baudrate (default 9600)
+While you will find a lot of online resources still describing the use of
+pySim-prog.py and pySim-read.py, those tools are considered legacy by
+now and have by far been superseded by the much more capable
+pySim-shell. We strongly encourage users to adopt pySim-shell, unless
+they have very specific requirements like batch programming of large
+quantities of cards, which is about the only remaining use case for the
+legacy tools.
- * Interact with SIMs from a python interactive shell (e.g. ipython):
-
-```
-from pySim.transport.serial import SerialSimLink
-from pySim.commands import SimCardCommands
-
-sl = SerialSimLink(device='/dev/ttyUSB0', baudrate=9600)
-sc = SimCardCommands(sl)
-
-sl.wait_for_card()
-
- # Print IMSI
-print(sc.read_binary(['3f00', '7f20', '6f07']))
-
- # Run A3/A8
-print(sc.run_gsm('00112233445566778899aabbccddeeff'))
-```
--
To view, visit https://gerrit.osmocom.org/c/pysim/+/27207
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: I8c07a2e0778ab95fb42be6074acb80874e681d20
Gerrit-Change-Number: 27207
Gerrit-PatchSet: 1
Gerrit-Owner: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-MessageType: merged
laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/pysim/+/27207 )
Change subject: README.md: Remove old usage examples, refer to user manual instead
......................................................................
Patch Set 1: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/c/pysim/+/27207
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: I8c07a2e0778ab95fb42be6074acb80874e681d20
Gerrit-Change-Number: 27207
Gerrit-PatchSet: 1
Gerrit-Owner: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Comment-Date: Tue, 15 Feb 2022 14:46:02 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
laforge has uploaded this change for review. ( https://gerrit.osmocom.org/c/pysim/+/27207 )
Change subject: README.md: Remove old usage examples, refer to user manual instead
......................................................................
README.md: Remove old usage examples, refer to user manual instead
We want people to use pySim-shell and should not mislead them by
having usage examples of old tools in README.md. Also, all
documentation should be in the manuals, let's try to have bits
and pieces in various places.
Change-Id: I8c07a2e0778ab95fb42be6074acb80874e681d20
---
M README.md
1 file changed, 20 insertions(+), 36 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/pysim refs/changes/07/27207/1
diff --git a/README.md b/README.md
index 7f46085..5e1937e 100644
--- a/README.md
+++ b/README.md
@@ -93,46 +93,30 @@
<https://osmocom.org/projects/cellular-infrastructure/wiki/Gerrit>
-Usage Examples
---------------
+Documentation
+-------------
- * Program customizable SIMs. Two modes are possible:
+The pySim user manual can be built from this very source code by means
+of sphinx (with sphinxcontrib-napoleon and sphinx-argparse). See the
+Makefile in the 'docs' directory.
- - one where you specify every parameter manually:
-```
-./pySim-prog.py -n 26C3 -c 49 -x 262 -y 42 -i <IMSI> -s <ICCID>
-```
+A pre-rendered HTML user manual of the current pySim 'git master' is
+available from <https://downloads.osmocom.org/docs/latest/pysim/> and
+a downloadable PDF version is published at
+<https://downloads.osmocom.org/docs/latest/osmopysim-usermanual.pdf>.
- - one where they are generated from some minimal set:
-```
-./pySim-prog.py -n 26C3 -c 49 -x 262 -y 42 -z <random_string_of_choice> -j <card_num>
-```
+A slightly dated video presentation about pySim-shell can be found at
+<https://media.ccc.de/v/osmodevcall-20210409-laforge-pysim-shell>.
-With ``<random_string_of_choice>`` and ``<card_num>``, the soft will generate
-'predictable' IMSI and ICCID, so make sure you choose them so as not to
-conflict with anyone. (for e.g. your name as ``<random_string_of_choice>`` and
-0 1 2 ... for ``<card num>``).
-You also need to enter some parameters to select the device:
+pySim-shell vs. legacy tools
+----------------------------
- -t TYPE : type of card (``supersim``, ``magicsim``, ``fakemagicsim`` or try ``auto``)
- -d DEV : Serial port device (default ``/dev/ttyUSB0``)
- -b BAUD : Baudrate (default 9600)
+While you will find a lot of online resources still describing the use of
+pySim-prog.py and pySim-read.py, those tools are considered legacy by
+now and have by far been superseded by the much more capable
+pySim-shell. We strongly encourage users to adopt pySim-shell, unless
+they have very specific requirements like batch programming of large
+quantities of cards, which is about the only remaining use case for the
+legacy tools.
- * Interact with SIMs from a python interactive shell (e.g. ipython):
-
-```
-from pySim.transport.serial import SerialSimLink
-from pySim.commands import SimCardCommands
-
-sl = SerialSimLink(device='/dev/ttyUSB0', baudrate=9600)
-sc = SimCardCommands(sl)
-
-sl.wait_for_card()
-
- # Print IMSI
-print(sc.read_binary(['3f00', '7f20', '6f07']))
-
- # Run A3/A8
-print(sc.run_gsm('00112233445566778899aabbccddeeff'))
-```
--
To view, visit https://gerrit.osmocom.org/c/pysim/+/27207
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: I8c07a2e0778ab95fb42be6074acb80874e681d20
Gerrit-Change-Number: 27207
Gerrit-PatchSet: 1
Gerrit-Owner: laforge <laforge(a)osmocom.org>
Gerrit-MessageType: newchange
osmith has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-ci/+/27206 )
Change subject: ansible/ttcn3: don't install titan/libfftranscode
......................................................................
ansible/ttcn3: don't install titan/libfftranscode
With the last patch, the jenkins infrastructure will always run
eclipse-titan in docker. Therefore do not install eclipse-titan and
libfftranscode on jenkins nodes anymore.
This avoids the problem that we couldn't upgrade libfftranscode with the
existing ansible rules.
libosmocore-utils is still being installed, as osmo-config-merge must be
installed on the jenkins nodes.
Closes: OS#5452
Change-Id: Ie46fe3dcb286a6258021fad9c35be10442544761
---
M ansible/roles/osmocom-jenkins-slave/tasks/ttcn3-slave.yml
1 file changed, 1 insertion(+), 14 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-ci refs/changes/06/27206/1
diff --git a/ansible/roles/osmocom-jenkins-slave/tasks/ttcn3-slave.yml b/ansible/roles/osmocom-jenkins-slave/tasks/ttcn3-slave.yml
index 5e15254..fc4fcc3 100644
--- a/ansible/roles/osmocom-jenkins-slave/tasks/ttcn3-slave.yml
+++ b/ansible/roles/osmocom-jenkins-slave/tasks/ttcn3-slave.yml
@@ -24,22 +24,9 @@
filename: obs_osmocom
update_cache: yes
-- name: install titan and libosmocore-utils
+- name: install libosmocore-utils
apt:
name:
- - eclipse-titan
- libosmocore-utils
update_cache: yes
cache_valid_time: 3600
-
-- name: Get DEB architecture
- shell: dpkg --print-architecture
- register: deb_architecture
-
-- name: Install libfftranscode0
- apt:
- deb: https://ftp.osmocom.org/binaries/libfftranscode/libfftranscode0_0.3_{{deb_a…
-
-- name: Install libfftranscode-dev
- apt:
- deb: https://ftp.osmocom.org/binaries/libfftranscode/libfftranscode-dev_0.3_{{de…
--
To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/27206
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-Change-Id: Ie46fe3dcb286a6258021fad9c35be10442544761
Gerrit-Change-Number: 27206
Gerrit-PatchSet: 1
Gerrit-Owner: osmith <osmith(a)sysmocom.de>
Gerrit-MessageType: newchange
osmith has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-ci/+/27205 )
Change subject: jobs/gerrit-verifications: build ttcn3 in docker
......................................................................
jobs/gerrit-verifications: build ttcn3 in docker
Build the testsuites in the same docker container during gerrit
verification, as it will be the case when building and running the
testsuites in the various ttcn3-* jenkins jobs.
This avoids the need to install eclipse-titan and libfftranscode on the
jenkins nodes.
Related: OS#5452
Change-Id: I6a667255bb55a945b4c6279d711142aa252c6996
---
M jobs/gerrit-verifications.yml
1 file changed, 12 insertions(+), 4 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-ci refs/changes/05/27205/1
diff --git a/jobs/gerrit-verifications.yml b/jobs/gerrit-verifications.yml
index 84fe805..9481047 100644
--- a/jobs/gerrit-verifications.yml
+++ b/jobs/gerrit-verifications.yml
@@ -261,12 +261,20 @@
cmd: '{timeout_cmd} ./contrib/jenkins.sh'
- osmo-ttcn3-hacks:
- repos_url: 'https://gerrit.osmocom.org/{repos}'
slave_axis: !!python/tuple [ttcn3]
- concurrent: false
cmd: |
- set -e
- make deps; make clean; make compile
+ docker run \
+ --rm \
+ -v "$PWD:/build" \
+ "registry.osmocom.org/osmocom-build/debian-bullseye-titan" \
+ {timeout_cmd} \
+ sh -e -x -c "
+ useradd --uid=1000 build;
+ cd /build;
+ su build -c 'make deps';
+ su build -c 'make clean';
+ su build -c 'make compile';
+ "
jobs:
- 'gerrit-{repos}'
--
To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/27205
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-Change-Id: I6a667255bb55a945b4c6279d711142aa252c6996
Gerrit-Change-Number: 27205
Gerrit-PatchSet: 1
Gerrit-Owner: osmith <osmith(a)sysmocom.de>
Gerrit-MessageType: newchange