September 2024 - gerrit-log - lists.osmocom.org

[XS] Change in osmo-ttcn3-hacks[master]: gen_links.sh.inc: 'set -e' to abort on error

by fixeria

fixeria has submitted this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/38245?usp=email ) Change subject: gen_links.sh.inc: 'set -e' to abort on error ...................................................................... gen_links.sh.inc: 'set -e' to abort on error We should not tolerate errors in '*/gen_links.sh' files. Change-Id: Id9367e4c45c84dcbf1c308581360be73242c0d38 --- M gen_links.sh.inc 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: laforge: Looks good to me, but someone else must approve pespin: Looks good to me, approved Jenkins Builder: Verified diff --git a/gen_links.sh.inc b/gen_links.sh.inc index c21bcd6..f134427 100644 --- a/gen_links.sh.inc +++ b/gen_links.sh.inc @@ -16,6 +16,8 @@ # See the License for the specific language governing permissions and # limitations under the License. +set -e + rm -f .gitignore gen_link() { -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/38245?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: osmo-ttcn3-hacks Gerrit-Branch: master Gerrit-Change-Id: Id9367e4c45c84dcbf1c308581360be73242c0d38 Gerrit-Change-Number: 38245 Gerrit-PatchSet: 1 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>

10 months, 4 weeks

1
0
0 0

[XS] Change in osmo-ttcn3-hacks[master]: gen_links.sh.inc: 'set -e' to abort on error

by pespin

Attention is currently required from: fixeria. pespin has posted comments on this change by fixeria. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/38245?usp=email ) Change subject: gen_links.sh.inc: 'set -e' to abort on error ...................................................................... Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/38245?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-ttcn3-hacks Gerrit-Branch: master Gerrit-Change-Id: Id9367e4c45c84dcbf1c308581360be73242c0d38 Gerrit-Change-Number: 38245 Gerrit-PatchSet: 1 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Comment-Date: Mon, 23 Sep 2024 09:04:01 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

10 months, 4 weeks

1
0
0 0

[XS] Change in osmo-ttcn3-hacks[master]: {bsc-nat,selftest,sysinfo}/gen_links.sh: use bash, not sh

by pespin

Attention is currently required from: fixeria. pespin has posted comments on this change by fixeria. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/38244?usp=email ) Change subject: {bsc-nat,selftest,sysinfo}/gen_links.sh: use bash, not sh ...................................................................... Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/38244?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-ttcn3-hacks Gerrit-Branch: master Gerrit-Change-Id: I994e241efb73f6f8b78803cf4ecdfb057b210a3b Gerrit-Change-Number: 38244 Gerrit-PatchSet: 1 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Comment-Date: Mon, 23 Sep 2024 09:03:41 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes

10 months, 4 weeks

1
0
0 0

[S] Change in osmo-msc[master]: Remove speech codec list from bearer_cap for phase 1 mobile station

by jolly

Attention is currently required from: fixeria, neels, nt2mku, pespin. jolly has posted comments on this change by jolly. ( https://gerrit.osmocom.org/c/osmo-msc/+/38197?usp=email ) Change subject: Remove speech codec list from bearer_cap for phase 1 mobile station ...................................................................... Patch Set 3: (2 comments) Commit Message: https://gerrit.osmocom.org/c/osmo-msc/+/38197/comment/73c8c066_a7afbe67?usp… : PS1, Line 9: ect. > I mean octet 3a and all the possibly following codecs. Done https://gerrit.osmocom.org/c/osmo-msc/+/38197/comment/e31d60fb_9a5ec779?usp… : PS1, Line 16: ect. > I mean octet 3a and all the possibly following codecs. Done -- To view, visit https://gerrit.osmocom.org/c/osmo-msc/+/38197?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-msc Gerrit-Branch: master Gerrit-Change-Id: Idd267dad0ade18cee7d5be813a57e1ee3168e2db Gerrit-Change-Number: 38197 Gerrit-PatchSet: 3 Gerrit-Owner: jolly <andreas(a)eversberg.eu> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: neels <nhofmeyr(a)sysmocom.de> Gerrit-Reviewer: nt2mku <degrunert.web(a)googlemail.com> Gerrit-CC: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-CC: pespin <pespin(a)sysmocom.de> Gerrit-Attention: neels <nhofmeyr(a)sysmocom.de> Gerrit-Attention: nt2mku <degrunert.web(a)googlemail.com> Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Comment-Date: Mon, 23 Sep 2024 08:11:42 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: jolly <andreas(a)eversberg.eu> Comment-In-Reply-To: fixeria <vyanitskiy(a)sysmocom.de>

10 months, 4 weeks

1
0
0 0

[S] Change in osmo-msc[master]: Remove speech codec list from bearer_cap for phase 1 mobile station

by jolly

Attention is currently required from: fixeria, neels, nt2mku, pespin. Hello Jenkins Builder, neels, nt2mku, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/c/osmo-msc/+/38197?usp=email to look at the new patch set (#3). Change subject: Remove speech codec list from bearer_cap for phase 1 mobile station ...................................................................... Remove speech codec list from bearer_cap for phase 1 mobile station According to TS 04.08 Clause 10.5.4.5.1, the octet 3a etc. shall not be included, if only GSM full/half rate speech version 1 is supported. As phase 1 mobile stations only support FR and HR speech, the speech list in the bearer capability must be reduced to the codecs listed above. If there is none of these codecs left, the call is rejected. The octet 3a etc. must be omitted when encoding bearer capability for phase 1 mobile stations. This is done by removing the speech list. I do not use a "codec filter", because it is not required. The phase 1 mobile station can only respond with codecs it supports, so no filtering is required. Related: OS#6461 Change-Id: Idd267dad0ade18cee7d5be813a57e1ee3168e2db --- M include/osmocom/msc/codec_mapping.h M src/libmsc/codec_mapping.c M src/libmsc/gsm_04_08_cc.c 3 files changed, 43 insertions(+), 0 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-msc refs/changes/97/38197/3 -- To view, visit https://gerrit.osmocom.org/c/osmo-msc/+/38197?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newpatchset Gerrit-Project: osmo-msc Gerrit-Branch: master Gerrit-Change-Id: Idd267dad0ade18cee7d5be813a57e1ee3168e2db Gerrit-Change-Number: 38197 Gerrit-PatchSet: 3 Gerrit-Owner: jolly <andreas(a)eversberg.eu> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: neels <nhofmeyr(a)sysmocom.de> Gerrit-Reviewer: nt2mku <degrunert.web(a)googlemail.com> Gerrit-CC: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-CC: pespin <pespin(a)sysmocom.de> Gerrit-Attention: neels <nhofmeyr(a)sysmocom.de> Gerrit-Attention: nt2mku <degrunert.web(a)googlemail.com> Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>

10 months, 4 weeks

1
0
0 0

[S] Change in osmo-msc[master]: Remove speech codec list from bearer_cap for phase 1 mobile station

by jolly

Attention is currently required from: fixeria, neels, nt2mku, pespin. jolly has posted comments on this change by jolly. ( https://gerrit.osmocom.org/c/osmo-msc/+/38197?usp=email ) Change subject: Remove speech codec list from bearer_cap for phase 1 mobile station ...................................................................... Patch Set 2: (10 comments) Commit Message: https://gerrit.osmocom.org/c/osmo-msc/+/38197/comment/ca993d3b_0386790f?usp… : PS1, Line 9: ect. > did you mean ext[ended]? I mean octet 3a and all the possibly following codecs. https://gerrit.osmocom.org/c/osmo-msc/+/38197/comment/38085e48_b6838253?usp… : PS1, Line 16: ect. > did you mean ext[ended]? I mean octet 3a and all the possibly following codecs. File src/libmsc/gsm_04_08_cc.c: https://gerrit.osmocom.org/c/osmo-msc/+/38197/comment/0896b3ea_61c80a4f?usp… : PS1, Line 929: /* Remove codecs in bearer_cap that are unsupported by phase 1 mobile stations. > It probably makes sense to move this to a helper function, eg […] Done https://gerrit.osmocom.org/c/osmo-msc/+/38197/comment/69657754_189e2994?usp… : PS1, Line 990: /* Speech codec list (oictets 3a ect.) must not be sent to phase 1 mobile stations. > octets Done https://gerrit.osmocom.org/c/osmo-msc/+/38197/comment/3d072922_cd5121e1?usp… : PS1, Line 992: if (trans->vsub->classmark.classmark1.rev_lev == 0) > I guess you could do this in the GSM48_BCAP_ITCAP_SPEECH code path above? Done https://gerrit.osmocom.org/c/osmo-msc/+/38197/comment/9cf72a70_fd7cca58?usp… : PS1, Line 1158: /* Speech codec list (oictets 3a ect.) must not be sent to phase 1 mobile stations. > octets Done https://gerrit.osmocom.org/c/osmo-msc/+/38197/comment/78e0e9dd_5f7d82f2?usp… : PS1, Line 1878: /* Speech codec list (oictets 3a ect.) must not be sent to phase 1 mobile stations. > octets Done https://gerrit.osmocom.org/c/osmo-msc/+/38197/comment/6e7a2dd8_4c4ef68c?usp… : PS1, Line 1930: /* Speech codec list (oictets 3a ect.) must not be sent to phase 1 mobile stations. > octets Done https://gerrit.osmocom.org/c/osmo-msc/+/38197/comment/aab4fb6e_a108045d?usp… : PS1, Line 1988: /* Speech codec list (oictets 3a ect.) must not be sent to phase 1 mobile stations. > octets Done https://gerrit.osmocom.org/c/osmo-msc/+/38197/comment/f1d55bd2_dce0c7b1?usp… : PS1, Line 1991: modify->bearer_cap.speech_ver[0] = -1; > I see this being done in lots of places, it may make sense to move the logic to a helper function to […] Done -- To view, visit https://gerrit.osmocom.org/c/osmo-msc/+/38197?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-msc Gerrit-Branch: master Gerrit-Change-Id: Idd267dad0ade18cee7d5be813a57e1ee3168e2db Gerrit-Change-Number: 38197 Gerrit-PatchSet: 2 Gerrit-Owner: jolly <andreas(a)eversberg.eu> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: neels <nhofmeyr(a)sysmocom.de> Gerrit-Reviewer: nt2mku <degrunert.web(a)googlemail.com> Gerrit-CC: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-CC: pespin <pespin(a)sysmocom.de> Gerrit-Attention: neels <nhofmeyr(a)sysmocom.de> Gerrit-Attention: nt2mku <degrunert.web(a)googlemail.com> Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Comment-Date: Mon, 23 Sep 2024 08:06:16 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: fixeria <vyanitskiy(a)sysmocom.de> Comment-In-Reply-To: pespin <pespin(a)sysmocom.de>

10 months, 4 weeks

1
0
0 0

[S] Change in osmo-msc[master]: Remove speech codec list from bearer_cap for phase 1 mobile station

by jolly

Attention is currently required from: jolly, neels, nt2mku. Hello Jenkins Builder, neels, nt2mku, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/c/osmo-msc/+/38197?usp=email to look at the new patch set (#2). The following approvals got outdated and were removed: Verified+1 by Jenkins Builder Change subject: Remove speech codec list from bearer_cap for phase 1 mobile station ...................................................................... Remove speech codec list from bearer_cap for phase 1 mobile station According to TS 04.08 Clause 10.5.4.5.1, the octet 3a ect. shall not be included, if only GSM full/half rate speech version 1 is supported. As phase 1 mobile stations only support FR and HR speech, the speech list in the bearer capability must be reduced to the codecs listed above. If there is none of these codecs left, the call is rejected. The octet 3a ect. must be omitted when encoding bearer capability for phase 1 mobile stations. This is done by removing the speech list. I do not use a "codec filter", because it is not required. The phase 1 mobile station can only respond with codecs it supports, so no filtering is required. Related: OS#6461 Change-Id: Idd267dad0ade18cee7d5be813a57e1ee3168e2db --- M include/osmocom/msc/codec_mapping.h M src/libmsc/codec_mapping.c M src/libmsc/gsm_04_08_cc.c 3 files changed, 43 insertions(+), 0 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-msc refs/changes/97/38197/2 -- To view, visit https://gerrit.osmocom.org/c/osmo-msc/+/38197?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newpatchset Gerrit-Project: osmo-msc Gerrit-Branch: master Gerrit-Change-Id: Idd267dad0ade18cee7d5be813a57e1ee3168e2db Gerrit-Change-Number: 38197 Gerrit-PatchSet: 2 Gerrit-Owner: jolly <andreas(a)eversberg.eu> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: neels <nhofmeyr(a)sysmocom.de> Gerrit-Reviewer: nt2mku <degrunert.web(a)googlemail.com> Gerrit-CC: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-CC: pespin <pespin(a)sysmocom.de> Gerrit-Attention: jolly <andreas(a)eversberg.eu> Gerrit-Attention: neels <nhofmeyr(a)sysmocom.de> Gerrit-Attention: nt2mku <degrunert.web(a)googlemail.com>

10 months, 4 weeks

1
0
0 0

[M] Change in ...pyosmocom[master]: Add debian packaging for pyosmocom

by osmith

osmith has submitted this change. ( https://gerrit.osmocom.org/c/python/pyosmocom/+/38246?usp=email ) Change subject: Add debian packaging for pyosmocom ...................................................................... Add debian packaging for pyosmocom Change-Id: If862c14c7802d1967d99c3939038b8eb7fe301f3 --- A debian/changelog A debian/compat A debian/control A debian/copyright A debian/python3-pyosmocom-doc.doc-base A debian/rules A debian/source/format 7 files changed, 73 insertions(+), 0 deletions(-) Approvals: osmith: Looks good to me, approved Jenkins Builder: Verified diff --git a/debian/changelog b/debian/changelog new file mode 100644 index 0000000..eaa9715 --- /dev/null +++ b/debian/changelog @@ -0,0 +1,5 @@ +pyosmocom (0.0.4) unstable; urgency=medium + + * initial release + + -- Harald Welte <laforge(a)osmocom.org> Fri, 19 Sep 2024 20:48:02 +0200 diff --git a/debian/compat b/debian/compat new file mode 100644 index 0000000..f599e28 --- /dev/null +++ b/debian/compat @@ -0,0 +1 @@ +10 diff --git a/debian/control b/debian/control new file mode 100644 index 0000000..c8d3275 --- /dev/null +++ b/debian/control @@ -0,0 +1,31 @@ +Source: pyosmocom +Section: python +Priority: optional +Maintainer: Osmocom team <openbsc(a)lists.osmocom.org> +Build-Depends: debhelper (>= 10), + dh-python, + python3, + python3-setuptools, + pybuild-plugin-pyproject, + python3-sphinx, + python3-sphinx-argparse +Standards-Version: 3.9.8 +Homepage: https://osmocom.org/projects/pyosmocom/wiki +Vcs-Git: https://gitea.osmocom.org/osmocom/pyosmocom +Vcs-Browser: https://gitea.osmocom.org/osmocom/pyosmocom + +Package: python3-pyosmocom-doc +Section: doc +Architecture: all +Depends: ${misc:Depends}, ${sphinxdoc:Depends} +Description: Python library code related to Open Source Mobile Communications + . + This package contains the documentation. + +Package: python3-pyosmocom +Architecture: all +Depends: ${python3:Depends}, ${misc:Depends}, + python3-construct +Description: Python library code related to Open Source Mobile Communications + . + This package contains the Python 3 version of the pyosmocom libraries. diff --git a/debian/copyright b/debian/copyright new file mode 100644 index 0000000..0122624 --- /dev/null +++ b/debian/copyright @@ -0,0 +1,16 @@ +Format: http://www.debian.org/doc/packaging-manuals/copyright-format/1.0/ +Upstream-Name: pyosmocom +Source: https://gitea.osmocom.org/osmocom/pyosmocom + +Files: src/osmocom/construct.py src/osmocom/gsmtap.py src/osmocom/tlv.py +Copyright: (C) 2021-2022 by Harald Welte <laforge(a)osmocom.org> +License: GPL-2+ + +Files: src/osmocom/utils.py +Copyright: Copyright (C) 2009-2010 Sylvain Munaut <tnt(a)246tNt.com> + Copyright (C) 2021 Harald Welte <laforge(a)osmocom.org> +License: GPL-2+ + +Files: src/osmocom/gsup/*.py +Copyright: (C) 2021-2024 by Harald Welte <laforge(a)osmocom.org> +License: GPL-2+ diff --git a/debian/python3-pyosmocom-doc.doc-base b/debian/python3-pyosmocom-doc.doc-base new file mode 100644 index 0000000..576a244 --- /dev/null +++ b/debian/python3-pyosmocom-doc.doc-base @@ -0,0 +1,9 @@ +Document: python3-pyosmocom-doc +Title: python3-pyosmocom Documentation +Author: Osmocom +Abstract: Sphinx documentation for python3-pyosmocom +Section: Programming/Python + +Format: HTML +Index: /usr/share/doc/python3-pyosmocom-doc/html/index.html +Files: /usr/share/doc/python3-pyosmocom-doc/html/* diff --git a/debian/rules b/debian/rules new file mode 100755 index 0000000..7ba83c9 --- /dev/null +++ b/debian/rules @@ -0,0 +1,10 @@ +#!/usr/bin/make -f + +export PYBUILD_NAME=pyosmocom + +%: + dh $@ --with python3,sphinxdoc --buildsystem=pybuild + +override_dh_sphinxdoc: + http_proxy=127.0.0.1:9 https_proxy=127.0.0.1:9 HTTP_PROXY=127.0.0.1:9 HTTPS_PROXY=127.0.0.1:9 PYTHONPATH=$(CURDIR)/.pybuild/cpython3_$(shell py3versions -vd)_pysmocom/build PYTHON=python3 python3 -m sphinx $(SPHINXOPTS) -b html docs $(CURDIR)/debian/python3-pyosmocom-doc/usr/share/doc/python3-pyosmocom-doc/html + dh_sphinxdoc diff --git a/debian/source/format b/debian/source/format new file mode 100644 index 0000000..89ae9db --- /dev/null +++ b/debian/source/format @@ -0,0 +1 @@ +3.0 (native) -- To view, visit https://gerrit.osmocom.org/c/python/pyosmocom/+/38246?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: python/pyosmocom Gerrit-Branch: master Gerrit-Change-Id: If862c14c7802d1967d99c3939038b8eb7fe301f3 Gerrit-Change-Number: 38246 Gerrit-PatchSet: 3 Gerrit-Owner: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>

10 months, 4 weeks

1
0
0 0

[M] Change in ...pyosmocom[master]: Add debian packaging for pyosmocom

by osmith

Attention is currently required from: dexter, laforge, pespin. osmith has posted comments on this change by laforge. ( https://gerrit.osmocom.org/c/python/pyosmocom/+/38246?usp=email ) Change subject: Add debian packaging for pyosmocom ...................................................................... Patch Set 2: (1 comment) Patchset: PS2: merging, so Osmocom_OBS_master doesn't fail anymore -- To view, visit https://gerrit.osmocom.org/c/python/pyosmocom/+/38246?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: python/pyosmocom Gerrit-Branch: master Gerrit-Change-Id: If862c14c7802d1967d99c3939038b8eb7fe301f3 Gerrit-Change-Number: 38246 Gerrit-PatchSet: 2 Gerrit-Owner: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: laforge <laforge(a)osmocom.org> Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Attention: dexter <pmaier(a)sysmocom.de> Gerrit-Comment-Date: Mon, 23 Sep 2024 07:44:27 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No

10 months, 4 weeks

1
0
0 0

[XS] Change in pysim[master]: contrib/jenkins: add SKIP_CLEAN_WORKSPACE

by osmith

osmith has uploaded this change for review. ( https://gerrit.osmocom.org/c/pysim/+/38253?usp=email ) Change subject: contrib/jenkins: add SKIP_CLEAN_WORKSPACE ...................................................................... contrib/jenkins: add SKIP_CLEAN_WORKSPACE In order to run this script from pyosmocom's contrib/jenkins.sh script, we want to skip the clean workspace step. Add an environment variable to do that. Related: OS#6570 Change-Id: Ic8dc9b85da17719195f7374d37eccb4dedba6ce8 --- M contrib/jenkins.sh 1 file changed, 4 insertions(+), 1 deletion(-) git pull ssh://gerrit.osmocom.org:29418/pysim refs/changes/53/38253/1 diff --git a/contrib/jenkins.sh b/contrib/jenkins.sh index 42bf5af..5fa5226 100755 --- a/contrib/jenkins.sh +++ b/contrib/jenkins.sh @@ -5,6 +5,7 @@ # * WITH_MANUALS: build manual PDFs if set to "1" # * PUBLISH: upload manuals after building if set to "1" (ignored without WITH_MANUALS = "1") # * JOB_TYPE: one of 'test', 'distcheck', 'pylint', 'docs' +# * SKIP_CLEAN_WORKSPACE: don't run osmo-clean-workspace.sh (for pyosmocom CI) # export PYTHONUNBUFFERED=1 @@ -16,7 +17,9 @@ exit 1 fi -osmo-clean-workspace.sh +if [ -z "$SKIP_CLEAN_WORKSPACE" ]; then + osmo-clean-workspace.sh +fi case "$JOB_TYPE" in "test") -- To view, visit https://gerrit.osmocom.org/c/pysim/+/38253?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newchange Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: Ic8dc9b85da17719195f7374d37eccb4dedba6ce8 Gerrit-Change-Number: 38253 Gerrit-PatchSet: 1 Gerrit-Owner: osmith <osmith(a)sysmocom.de>

10 months, 4 weeks

1
0
0 0

2025

2024

2023

2022

gerrit-log September 2024