laforge has submitted this change. ( https://gerrit.osmocom.org/c/pysim/+/37966?usp=email )
Change subject: contrib/jenkins.sh: Install dependencies before calling pylint
......................................................................
contrib/jenkins.sh: Install dependencies before calling pylint
This is the only way we can make sure pylint has all required
information about imports from packages we depend upon.
Change-Id: I29582aa3d7f9ace9ce832d5b907420aaf14881fb
---
M contrib/jenkins.sh
1 file changed, 6 insertions(+), 0 deletions(-)
Approvals:
fixeria: Looks good to me, but someone else must approve
laforge: Looks good to me, approved
Jenkins Builder: Verified
diff --git a/contrib/jenkins.sh b/contrib/jenkins.sh
index b3d83dd..5356fcb 100755
--- a/contrib/jenkins.sh
+++ b/contrib/jenkins.sh
@@ -51,6 +51,12 @@
"pylint")
# Print pylint version
pip3 freeze | grep pylint
+
+ virtualenv -p python3 venv --system-site-packages
+ . venv/bin/activate
+
+ pip install .
+
# Run pylint to find potential errors
# Ignore E1102: not-callable
# pySim/filesystem.py: E1102: method is not callable (not-callable)
--
To view, visit https://gerrit.osmocom.org/c/pysim/+/37966?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: merged
Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: I29582aa3d7f9ace9ce832d5b907420aaf14881fb
Gerrit-Change-Number: 37966
Gerrit-PatchSet: 3
Gerrit-Owner: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: dexter <pmaier(a)sysmocom.de>
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
laforge has submitted this change. ( https://gerrit.osmocom.org/c/pysim/+/38009?usp=email )
Change subject: pySim.esim.saip: Fix weird DF names
......................................................................
pySim.esim.saip: Fix weird DF names
Sometimes the struct member is called like df-telecom, but in other
cases it's called df-df-saip with a double 'df' in front. That makes
no sense, but we have to deal with it from our constructors...
Change-Id: If5e670441f03a47fa34e97a326909b24927c12f7
---
M pySim/esim/saip/__init__.py
1 file changed, 3 insertions(+), 3 deletions(-)
Approvals:
fixeria: Looks good to me, but someone else must approve
laforge: Looks good to me, approved
Jenkins Builder: Verified
diff --git a/pySim/esim/saip/__init__.py b/pySim/esim/saip/__init__.py
index 47e67e8..bf39025 100644
--- a/pySim/esim/saip/__init__.py
+++ b/pySim/esim/saip/__init__.py
@@ -817,7 +817,7 @@
return
# provide some reasonable defaults
self.decoded['templateID'] = str(oid.DF_5GS_v3)
- for fname in ['df-5gs']:
+ for fname in ['df-df-5gs']:
self.decoded[fname] = []
class ProfileElementEAP(FsProfileElement):
@@ -841,7 +841,7 @@
return
# provide some reasonable defaults
self.decoded['templateID'] = str(oid.DF_SAIP)
- for fname in ['df-saip']:
+ for fname in ['df-df-saip']:
self.decoded[fname] = []
class ProfileElementDfSNPN(FsProfileElement):
@@ -853,7 +853,7 @@
return
# provide some reasonable defaults
self.decoded['templateID'] = str(oid.DF_SNPN)
- for fname in ['df-snpn']:
+ for fname in ['df-df-snpn']:
self.decoded[fname] = []
class ProfileElementDf5GProSe(FsProfileElement):
--
To view, visit https://gerrit.osmocom.org/c/pysim/+/38009?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: merged
Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: If5e670441f03a47fa34e97a326909b24927c12f7
Gerrit-Change-Number: 38009
Gerrit-PatchSet: 3
Gerrit-Owner: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>