Attention is currently required from: JPM, fixeria, laforge.

dexter has posted comments on this change by JPM. ( https://gerrit.osmocom.org/c/python/pyosmocom/+/38687?usp=email )

Change subject: Fixing 3-digit MNC PlmnAdapter encoding & decoding, and related tests. ......................................................................

Patch Set 7:

(1 comment)

Patchset:

PS6: Unfortunately this fix has a chicken-egg problem. The code looks correct to me. I even verified one of the testcases with pen and paper. Everything should be correct.

However, the the build failure we see is in the pysim repository and I have fixed the testcases there now (see https://gerrit.osmocom.org/c/pysim/+/38941). The problem is that this of course only works with the fixed version of pyosmocom and the fixed version of pyosmocom only works with the fixed pysim.

My suggestion would be to remove the pysim test from jenkins.sh as this creates some kind of a circular dependency. I can take care of this, if the reviewers are ok with that.