pysim[master]: pySim-*.py: Set shebang to use python v2
Patch Set 2: Verified+1 -- To view, visit https://gerrit.osmocom.org/5621 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I6ab4e9edc44a8045915d4828c6de2fa98027fb7e Gerrit-PatchSet: 2 Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Owner: Pau Espin PedrolGerrit-Reviewer: Harald Welte Gerrit-Reviewer: Max Gerrit-Reviewer: Pau Espin Pedrol Gerrit-HasComments: No
[MERGED] pysim[master]: pySim-*.py: Set shebang to use python v2
Pau Espin Pedrol has submitted this change and it was merged. Change subject: pySim-*.py: Set shebang to use python v2 .. pySim-*.py: Set shebang to use python v2 Nowadays bin/python usually points to python3, and this script is written in python2, which means if run directly from terminal it will fail with some print syntax errors. Change-Id: I6ab4e9edc44a8045915d4828c6de2fa98027fb7e --- M pySim-prog.py M pySim-read.py 2 files changed, 2 insertions(+), 2 deletions(-) Approvals: Pau Espin Pedrol: Verified Harald Welte: Looks good to me, approved diff --git a/pySim-prog.py b/pySim-prog.py index 14874cd..d3a29c7 100755 --- a/pySim-prog.py +++ b/pySim-prog.py @@ -1,4 +1,4 @@ -#!/usr/bin/env python +#!/usr/bin/env python2 # # Utility to deal with sim cards and program the 'magic' ones easily diff --git a/pySim-read.py b/pySim-read.py index 2c10866..5ca39ee 100755 --- a/pySim-read.py +++ b/pySim-read.py @@ -1,4 +1,4 @@ -#!/usr/bin/env python +#!/usr/bin/env python2 # # Utility to display some informations about a SIM card -- To view, visit https://gerrit.osmocom.org/5621 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6ab4e9edc44a8045915d4828c6de2fa98027fb7e Gerrit-PatchSet: 2 Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Owner: Pau Espin PedrolGerrit-Reviewer: Harald Welte Gerrit-Reviewer: Max Gerrit-Reviewer: Pau Espin Pedrol
[MERGED] pysim[master]: pySim-prog.py: Fix trailing whitespace
Pau Espin Pedrol has submitted this change and it was merged. Change subject: pySim-prog.py: Fix trailing whitespace .. pySim-prog.py: Fix trailing whitespace Change-Id: I735dc7bb774d77d3b60b1712b0f0afcbb81dc726 --- M pySim-prog.py 1 file changed, 0 insertions(+), 1 deletion(-) Approvals: Pau Espin Pedrol: Verified Max: Looks good to me, but someone else must approve Harald Welte: Looks good to me, approved diff --git a/pySim-prog.py b/pySim-prog.py index d3a29c7..3df18ba 100755 --- a/pySim-prog.py +++ b/pySim-prog.py @@ -650,4 +650,3 @@ if not opts.batch_mode: done = True - -- To view, visit https://gerrit.osmocom.org/5622 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I735dc7bb774d77d3b60b1712b0f0afcbb81dc726 Gerrit-PatchSet: 2 Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Owner: Pau Espin PedrolGerrit-Reviewer: Harald Welte Gerrit-Reviewer: Max Gerrit-Reviewer: Pau Espin Pedrol
pysim[master]: pySim-prog.py: Fix trailing whitespace
Patch Set 2: Verified+1 -- To view, visit https://gerrit.osmocom.org/5622 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I735dc7bb774d77d3b60b1712b0f0afcbb81dc726 Gerrit-PatchSet: 2 Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Owner: Pau Espin PedrolGerrit-Reviewer: Harald Welte Gerrit-Reviewer: Max Gerrit-Reviewer: Pau Espin Pedrol Gerrit-HasComments: No
Build failure of network:osmocom:nightly/osmo-bsc in Debian_9.0/armv7l
Visit https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/osmo-bsc/Debian_9.0/armv7l Package network:osmocom:nightly/osmo-bsc failed to build in Debian_9.0/armv7l Check out the package for editing: osc checkout network:osmocom:nightly osmo-bsc Last lines of build log: [ 177s]printf(line); [ 177s]^~ [ 177s] cc1: some warnings being treated as errors [ 177s] Makefile:517: recipe for target 'abisip-find.o' failed [ 177s] make[4]: *** [abisip-find.o] Error 1 [ 177s] make[4]: Leaving directory '/usr/src/packages/BUILD/src/ipaccess' [ 177s] Makefile:408: recipe for target 'all-recursive' failed [ 177s] make[3]: *** [all-recursive] Error 1 [ 177s] make[3]: Leaving directory '/usr/src/packages/BUILD/src' [ 177s] Makefile:443: recipe for target 'all-recursive' failed [ 177s] make[2]: *** [all-recursive] Error 1 [ 177s] make[2]: Leaving directory '/usr/src/packages/BUILD' [ 177s] Makefile:375: recipe for target 'all' failed [ 177s] make[1]: *** [all] Error 2 [ 177s] make[1]: Leaving directory '/usr/src/packages/BUILD' [ 177s] dh_auto_build: make -j1 returned exit code 2 [ 177s] debian/rules:45: recipe for target 'build' failed [ 177s] make: *** [build] Error 2 [ 177s] dpkg-buildpackage: error: debian/rules build gave error exit status 2 [ 177s] [ 177s] armbuild02 failed "build osmo-bsc_1.1.2.20171230.dsc" at Sat Dec 30 21:40:49 UTC 2017. [ 177s] [ 177s] ### VM INTERACTION START ### [ 180s] [ 163.908256] SysRq : Power Off [ 180s] [ 163.927807] reboot: Power down [ 180s] ### VM INTERACTION END ### [ 180s] [ 180s] armbuild02 failed "build osmo-bsc_1.1.2.20171230.dsc" at Sat Dec 30 21:40:52 UTC 2017. [ 180s] -- Configure notifications at https://build.opensuse.org/user/notifications openSUSE Build Service (https://build.opensuse.org/)
Build failure of network:osmocom:nightly/osmo-bsc in Debian_8.0/x86_64
Visit https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/osmo-bsc/Debian_8.0/x86_64 Package network:osmocom:nightly/osmo-bsc failed to build in Debian_8.0/x86_64 Check out the package for editing: osc checkout network:osmocom:nightly osmo-bsc Last lines of build log: [ 108s]printf(line); [ 108s]^ [ 108s] cc1: some warnings being treated as errors [ 108s] Makefile:506: recipe for target 'abisip-find.o' failed [ 108s] make[4]: *** [abisip-find.o] Error 1 [ 108s] make[4]: Leaving directory '/usr/src/packages/BUILD/src/ipaccess' [ 108s] Makefile:396: recipe for target 'all-recursive' failed [ 108s] make[3]: *** [all-recursive] Error 1 [ 108s] make[3]: Leaving directory '/usr/src/packages/BUILD/src' [ 108s] Makefile:431: recipe for target 'all-recursive' failed [ 108s] make[2]: *** [all-recursive] Error 1 [ 108s] make[2]: Leaving directory '/usr/src/packages/BUILD' [ 108s] Makefile:362: recipe for target 'all' failed [ 108s] make[1]: *** [all] Error 2 [ 108s] make[1]: Leaving directory '/usr/src/packages/BUILD' [ 108s] dh_auto_build: make -j1 returned exit code 2 [ 108s] debian/rules:45: recipe for target 'build' failed [ 108s] make: *** [build] Error 2 [ 108s] dpkg-buildpackage: error: debian/rules build gave error exit status 2 [ 108s] [ 108s] build79 failed "build osmo-bsc_1.1.2.20171230.dsc" at Sat Dec 30 20:07:20 UTC 2017. [ 108s] [ 108s] ### VM INTERACTION START ### [ 109s] Powering off. [ 109s] [ 101.848645] reboot: Power down [ 109s] ### VM INTERACTION END ### [ 109s] [ 109s] build79 failed "build osmo-bsc_1.1.2.20171230.dsc" at Sat Dec 30 20:07:21 UTC 2017. [ 109s] -- Configure notifications at https://build.opensuse.org/user/notifications openSUSE Build Service (https://build.opensuse.org/)
Build failure of network:osmocom:nightly/osmo-bsc in Debian_9.0/i586
Visit https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/osmo-bsc/Debian_9.0/i586 Package network:osmocom:nightly/osmo-bsc failed to build in Debian_9.0/i586 Check out the package for editing: osc checkout network:osmocom:nightly osmo-bsc Last lines of build log: [ 131s] abisip-find.c:317:3: error: format not a string literal and no format arguments [-Werror=format-security] [ 131s]printf(line); [ 131s]^~ [ 131s] cc1: some warnings being treated as errors [ 131s] Makefile:517: recipe for target 'abisip-find.o' failed [ 131s] make[4]: *** [abisip-find.o] Error 1 [ 131s] make[4]: Leaving directory '/usr/src/packages/BUILD/src/ipaccess' [ 131s] Makefile:408: recipe for target 'all-recursive' failed [ 131s] make[3]: *** [all-recursive] Error 1 [ 131s] make[3]: Leaving directory '/usr/src/packages/BUILD/src' [ 131s] Makefile:443: recipe for target 'all-recursive' failed [ 131s] make[2]: *** [all-recursive] Error 1 [ 131s] make[2]: Leaving directory '/usr/src/packages/BUILD' [ 131s] Makefile:375: recipe for target 'all' failed [ 131s] make[1]: *** [all] Error 2 [ 131s] make[1]: Leaving directory '/usr/src/packages/BUILD' [ 131s] dh_auto_build: make -j1 returned exit code 2 [ 131s] debian/rules:45: recipe for target 'build' failed [ 131s] make: *** [build] Error 2 [ 131s] dpkg-buildpackage: error: debian/rules build gave error exit status 2 [ 131s] [ 131s] lamb05 failed "build osmo-bsc_1.1.2.20171230.dsc" at Sat Dec 30 20:05:43 UTC 2017. [ 131s] [ 131s] ### VM INTERACTION START ### [ 134s] [ 125.820549] reboot: Power down [ 134s] ### VM INTERACTION END ### [ 134s] [ 134s] lamb05 failed "build osmo-bsc_1.1.2.20171230.dsc" at Sat Dec 30 20:05:47 UTC 2017. [ 134s] -- Configure notifications at https://build.opensuse.org/user/notifications openSUSE Build Service (https://build.opensuse.org/)
Build failure of network:osmocom:nightly/osmo-bsc in Debian_8.0/i586
Visit https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/osmo-bsc/Debian_8.0/i586 Package network:osmocom:nightly/osmo-bsc failed to build in Debian_8.0/i586 Check out the package for editing: osc checkout network:osmocom:nightly osmo-bsc Last lines of build log: [ 105s]printf(line); [ 105s]^ [ 106s] cc1: some warnings being treated as errors [ 106s] Makefile:506: recipe for target 'abisip-find.o' failed [ 106s] make[4]: *** [abisip-find.o] Error 1 [ 106s] make[4]: Leaving directory '/usr/src/packages/BUILD/src/ipaccess' [ 106s] Makefile:396: recipe for target 'all-recursive' failed [ 106s] make[3]: *** [all-recursive] Error 1 [ 106s] make[3]: Leaving directory '/usr/src/packages/BUILD/src' [ 106s] Makefile:431: recipe for target 'all-recursive' failed [ 106s] make[2]: *** [all-recursive] Error 1 [ 106s] make[2]: Leaving directory '/usr/src/packages/BUILD' [ 106s] Makefile:362: recipe for target 'all' failed [ 106s] make[1]: *** [all] Error 2 [ 106s] make[1]: Leaving directory '/usr/src/packages/BUILD' [ 106s] dh_auto_build: make -j1 returned exit code 2 [ 106s] debian/rules:45: recipe for target 'build' failed [ 106s] make: *** [build] Error 2 [ 106s] dpkg-buildpackage: error: debian/rules build gave error exit status 2 [ 106s] [ 106s] lamb74 failed "build osmo-bsc_1.1.2.20171230.dsc" at Sat Dec 30 20:05:10 UTC 2017. [ 106s] [ 106s] ### VM INTERACTION START ### [ 107s] Powering off. [ 107s] [ 100.130792] reboot: Power down [ 107s] ### VM INTERACTION END ### [ 107s] [ 107s] lamb74 failed "build osmo-bsc_1.1.2.20171230.dsc" at Sat Dec 30 20:05:12 UTC 2017. [ 107s] -- Configure notifications at https://build.opensuse.org/user/notifications openSUSE Build Service (https://build.opensuse.org/)
Build failure of network:osmocom:nightly/osmo-bsc in xUbuntu_17.10/x86_64
Visit https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/osmo-bsc/xUbuntu_17.10/x86_64 Package network:osmocom:nightly/osmo-bsc failed to build in xUbuntu_17.10/x86_64 Check out the package for editing: osc checkout network:osmocom:nightly osmo-bsc Last lines of build log: [ 149s] abisip-find.c:317:3: error: format not a string literal and no format arguments [-Werror=format-security] [ 149s]printf(line); [ 149s]^~ [ 149s] cc1: some warnings being treated as errors [ 149s] Makefile:517: recipe for target 'abisip-find.o' failed [ 149s] make[4]: *** [abisip-find.o] Error 1 [ 149s] make[4]: Leaving directory '/usr/src/packages/BUILD/src/ipaccess' [ 149s] Makefile:408: recipe for target 'all-recursive' failed [ 149s] make[3]: *** [all-recursive] Error 1 [ 149s] make[3]: Leaving directory '/usr/src/packages/BUILD/src' [ 149s] Makefile:443: recipe for target 'all-recursive' failed [ 149s] make[2]: *** [all-recursive] Error 1 [ 149s] make[2]: Leaving directory '/usr/src/packages/BUILD' [ 149s] Makefile:375: recipe for target 'all' failed [ 149s] make[1]: *** [all] Error 2 [ 149s] make[1]: Leaving directory '/usr/src/packages/BUILD' [ 149s] dh_auto_build: make -j1 returned exit code 2 [ 149s] debian/rules:45: recipe for target 'build' failed [ 149s] make: *** [build] Error 2 [ 149s] dpkg-buildpackage: error: debian/rules build gave error exit status 2 [ 150s] [ 150s] cloud115 failed "build osmo-bsc_1.1.2.20171230.dsc" at Sat Dec 30 20:05:11 UTC 2017. [ 150s] [ 150s] ### VM INTERACTION START ### [ 153s] [ 120.671954] reboot: Power down [ 155s] ### VM INTERACTION END ### [ 156s] [ 156s] cloud115 failed "build osmo-bsc_1.1.2.20171230.dsc" at Sat Dec 30 20:05:18 UTC 2017. [ 156s] -- Configure notifications at https://build.opensuse.org/user/notifications openSUSE Build Service (https://build.opensuse.org/)
Build failure of network:osmocom:nightly/osmo-bsc in xUbuntu_16.04/i586
Visit https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/osmo-bsc/xUbuntu_16.04/i586 Package network:osmocom:nightly/osmo-bsc failed to build in xUbuntu_16.04/i586 Check out the package for editing: osc checkout network:osmocom:nightly osmo-bsc Last lines of build log: [ 106s] abisip-find.c:317:3: error: format not a string literal and no format arguments [-Werror=format-security] [ 106s]printf(line); [ 106s]^ [ 106s] cc1: some warnings being treated as errors [ 106s] Makefile:517: recipe for target 'abisip-find.o' failed [ 106s] make[4]: *** [abisip-find.o] Error 1 [ 106s] make[4]: Leaving directory '/usr/src/packages/BUILD/src/ipaccess' [ 106s] Makefile:408: recipe for target 'all-recursive' failed [ 106s] make[3]: *** [all-recursive] Error 1 [ 106s] make[3]: Leaving directory '/usr/src/packages/BUILD/src' [ 106s] Makefile:443: recipe for target 'all-recursive' failed [ 106s] make[2]: *** [all-recursive] Error 1 [ 106s] make[2]: Leaving directory '/usr/src/packages/BUILD' [ 106s] Makefile:375: recipe for target 'all' failed [ 106s] make[1]: *** [all] Error 2 [ 106s] make[1]: Leaving directory '/usr/src/packages/BUILD' [ 106s] dh_auto_build: make -j1 returned exit code 2 [ 106s] debian/rules:45: recipe for target 'build' failed [ 106s] make: *** [build] Error 2 [ 106s] dpkg-buildpackage: error: debian/rules build gave error exit status 2 [ 106s] [ 106s] build75 failed "build osmo-bsc_1.1.2.20171230.dsc" at Sat Dec 30 20:04:27 UTC 2017. [ 106s] [ 106s] ### VM INTERACTION START ### [ 109s] [ 103.073332] reboot: Power down [ 109s] ### VM INTERACTION END ### [ 109s] [ 109s] build75 failed "build osmo-bsc_1.1.2.20171230.dsc" at Sat Dec 30 20:04:31 UTC 2017. [ 109s] -- Configure notifications at https://build.opensuse.org/user/notifications openSUSE Build Service (https://build.opensuse.org/)
Build failure of network:osmocom:nightly/osmo-bsc in Debian_9.0/x86_64
Visit https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/osmo-bsc/Debian_9.0/x86_64 Package network:osmocom:nightly/osmo-bsc failed to build in Debian_9.0/x86_64 Check out the package for editing: osc checkout network:osmocom:nightly osmo-bsc Last lines of build log: [ 168s] abisip-find.c:317:3: error: format not a string literal and no format arguments [-Werror=format-security] [ 168s]printf(line); [ 168s]^~ [ 168s] cc1: some warnings being treated as errors [ 168s] Makefile:517: recipe for target 'abisip-find.o' failed [ 168s] make[4]: *** [abisip-find.o] Error 1 [ 168s] make[4]: Leaving directory '/usr/src/packages/BUILD/src/ipaccess' [ 168s] Makefile:408: recipe for target 'all-recursive' failed [ 168s] make[3]: *** [all-recursive] Error 1 [ 168s] make[3]: Leaving directory '/usr/src/packages/BUILD/src' [ 168s] Makefile:443: recipe for target 'all-recursive' failed [ 168s] make[2]: *** [all-recursive] Error 1 [ 168s] make[2]: Leaving directory '/usr/src/packages/BUILD' [ 168s] Makefile:375: recipe for target 'all' failed [ 168s] make[1]: *** [all] Error 2 [ 168s] make[1]: Leaving directory '/usr/src/packages/BUILD' [ 168s] dh_auto_build: make -j1 returned exit code 2 [ 168s] debian/rules:45: recipe for target 'build' failed [ 168s] make: *** [build] Error 2 [ 168s] dpkg-buildpackage: error: debian/rules build gave error exit status 2 [ 168s] [ 168s] lamb22 failed "build osmo-bsc_1.1.2.20171230.dsc" at Sat Dec 30 20:05:07 UTC 2017. [ 168s] [ 168s] ### VM INTERACTION START ### [ 171s] [ 160.632510] reboot: Power down [ 171s] ### VM INTERACTION END ### [ 171s] [ 171s] lamb22 failed "build osmo-bsc_1.1.2.20171230.dsc" at Sat Dec 30 20:05:10 UTC 2017. [ 171s] -- Configure notifications at https://build.opensuse.org/user/notifications openSUSE Build Service (https://build.opensuse.org/)
Build failure of network:osmocom:nightly/osmo-bsc in xUbuntu_16.04/x86_64
Visit https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/osmo-bsc/xUbuntu_16.04/x86_64 Package network:osmocom:nightly/osmo-bsc failed to build in xUbuntu_16.04/x86_64 Check out the package for editing: osc checkout network:osmocom:nightly osmo-bsc Last lines of build log: [ 108s] abisip-find.c:317:3: error: format not a string literal and no format arguments [-Werror=format-security] [ 108s]printf(line); [ 108s]^ [ 109s] cc1: some warnings being treated as errors [ 109s] Makefile:517: recipe for target 'abisip-find.o' failed [ 109s] make[4]: *** [abisip-find.o] Error 1 [ 109s] make[4]: Leaving directory '/usr/src/packages/BUILD/src/ipaccess' [ 109s] Makefile:408: recipe for target 'all-recursive' failed [ 109s] make[3]: *** [all-recursive] Error 1 [ 109s] make[3]: Leaving directory '/usr/src/packages/BUILD/src' [ 109s] Makefile:443: recipe for target 'all-recursive' failed [ 109s] make[2]: *** [all-recursive] Error 1 [ 109s] make[2]: Leaving directory '/usr/src/packages/BUILD' [ 109s] Makefile:375: recipe for target 'all' failed [ 109s] make[1]: *** [all] Error 2 [ 109s] make[1]: Leaving directory '/usr/src/packages/BUILD' [ 109s] dh_auto_build: make -j1 returned exit code 2 [ 109s] debian/rules:45: recipe for target 'build' failed [ 109s] make: *** [build] Error 2 [ 109s] dpkg-buildpackage: error: debian/rules build gave error exit status 2 [ 109s] [ 109s] lamb67 failed "build osmo-bsc_1.1.2.20171230.dsc" at Sat Dec 30 20:05:13 UTC 2017. [ 109s] [ 109s] ### VM INTERACTION START ### [ 112s] [ 105.154204] reboot: Power down [ 112s] ### VM INTERACTION END ### [ 112s] [ 112s] lamb67 failed "build osmo-bsc_1.1.2.20171230.dsc" at Sat Dec 30 20:05:17 UTC 2017. [ 112s] -- Configure notifications at https://build.opensuse.org/user/notifications openSUSE Build Service (https://build.opensuse.org/)
Build failure of network:osmocom:nightly/osmo-bsc in xUbuntu_16.10/x86_64
Visit https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/osmo-bsc/xUbuntu_16.10/x86_64 Package network:osmocom:nightly/osmo-bsc failed to build in xUbuntu_16.10/x86_64 Check out the package for editing: osc checkout network:osmocom:nightly osmo-bsc Last lines of build log: [ 97s] abisip-find.c:317:3: error: format not a string literal and no format arguments [-Werror=format-security] [ 97s]printf(line); [ 97s]^~ [ 97s] cc1: some warnings being treated as errors [ 97s] Makefile:517: recipe for target 'abisip-find.o' failed [ 97s] make[4]: *** [abisip-find.o] Error 1 [ 97s] make[4]: Leaving directory '/usr/src/packages/BUILD/src/ipaccess' [ 97s] Makefile:408: recipe for target 'all-recursive' failed [ 97s] make[3]: *** [all-recursive] Error 1 [ 97s] make[3]: Leaving directory '/usr/src/packages/BUILD/src' [ 97s] Makefile:443: recipe for target 'all-recursive' failed [ 97s] make[2]: *** [all-recursive] Error 1 [ 97s] make[2]: Leaving directory '/usr/src/packages/BUILD' [ 97s] Makefile:375: recipe for target 'all' failed [ 97s] make[1]: *** [all] Error 2 [ 97s] make[1]: Leaving directory '/usr/src/packages/BUILD' [ 97s] dh_auto_build: make -j1 returned exit code 2 [ 97s] debian/rules:45: recipe for target 'build' failed [ 97s] make: *** [build] Error 2 [ 97s] dpkg-buildpackage: error: debian/rules build gave error exit status 2 [ 97s] [ 97s] lamb71 failed "build osmo-bsc_1.1.2.20171230.dsc" at Sat Dec 30 20:04:25 UTC 2017. [ 97s] [ 97s] ### VM INTERACTION START ### [ 100s] [ 92.350446] reboot: Power down [ 100s] ### VM INTERACTION END ### [ 100s] [ 100s] lamb71 failed "build osmo-bsc_1.1.2.20171230.dsc" at Sat Dec 30 20:04:28 UTC 2017. [ 100s] -- Configure notifications at https://build.opensuse.org/user/notifications openSUSE Build Service (https://build.opensuse.org/)
Build failure of network:osmocom:nightly/osmo-bsc in xUbuntu_17.04/x86_64
Visit https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/osmo-bsc/xUbuntu_17.04/x86_64 Package network:osmocom:nightly/osmo-bsc failed to build in xUbuntu_17.04/x86_64 Check out the package for editing: osc checkout network:osmocom:nightly osmo-bsc Last lines of build log: [ 115s] abisip-find.c:317:3: error: format not a string literal and no format arguments [-Werror=format-security] [ 115s]printf(line); [ 115s]^~ [ 115s] cc1: some warnings being treated as errors [ 115s] Makefile:517: recipe for target 'abisip-find.o' failed [ 115s] make[4]: *** [abisip-find.o] Error 1 [ 115s] make[4]: Leaving directory '/usr/src/packages/BUILD/src/ipaccess' [ 115s] Makefile:408: recipe for target 'all-recursive' failed [ 115s] make[3]: *** [all-recursive] Error 1 [ 115s] make[3]: Leaving directory '/usr/src/packages/BUILD/src' [ 115s] Makefile:443: recipe for target 'all-recursive' failed [ 115s] make[2]: *** [all-recursive] Error 1 [ 115s] make[2]: Leaving directory '/usr/src/packages/BUILD' [ 115s] Makefile:375: recipe for target 'all' failed [ 115s] make[1]: *** [all] Error 2 [ 115s] make[1]: Leaving directory '/usr/src/packages/BUILD' [ 115s] dh_auto_build: make -j1 returned exit code 2 [ 115s] debian/rules:45: recipe for target 'build' failed [ 115s] make: *** [build] Error 2 [ 115s] dpkg-buildpackage: error: debian/rules build gave error exit status 2 [ 115s] [ 115s] build31 failed "build osmo-bsc_1.1.2.20171230.dsc" at Sat Dec 30 20:04:00 UTC 2017. [ 115s] [ 116s] ### VM INTERACTION START ### [ 119s] [ 110.545900] reboot: Power down [ 119s] ### VM INTERACTION END ### [ 119s] [ 119s] build31 failed "build osmo-bsc_1.1.2.20171230.dsc" at Sat Dec 30 20:04:03 UTC 2017. [ 119s] -- Configure notifications at https://build.opensuse.org/user/notifications openSUSE Build Service (https://build.opensuse.org/)
Build failure of network:osmocom:nightly/osmo-bsc in xUbuntu_17.04/i586
Visit https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/osmo-bsc/xUbuntu_17.04/i586 Package network:osmocom:nightly/osmo-bsc failed to build in xUbuntu_17.04/i586 Check out the package for editing: osc checkout network:osmocom:nightly osmo-bsc Last lines of build log: [ 212s] abisip-find.c:317:3: error: format not a string literal and no format arguments [-Werror=format-security] [ 212s]printf(line); [ 212s]^~ [ 213s] cc1: some warnings being treated as errors [ 213s] Makefile:517: recipe for target 'abisip-find.o' failed [ 213s] make[4]: *** [abisip-find.o] Error 1 [ 213s] make[4]: Leaving directory '/usr/src/packages/BUILD/src/ipaccess' [ 213s] Makefile:408: recipe for target 'all-recursive' failed [ 213s] make[3]: *** [all-recursive] Error 1 [ 213s] make[3]: Leaving directory '/usr/src/packages/BUILD/src' [ 213s] Makefile:443: recipe for target 'all-recursive' failed [ 213s] make[2]: *** [all-recursive] Error 1 [ 213s] make[2]: Leaving directory '/usr/src/packages/BUILD' [ 213s] Makefile:375: recipe for target 'all' failed [ 213s] make[1]: *** [all] Error 2 [ 213s] make[1]: Leaving directory '/usr/src/packages/BUILD' [ 213s] dh_auto_build: make -j1 returned exit code 2 [ 213s] debian/rules:45: recipe for target 'build' failed [ 213s] make: *** [build] Error 2 [ 213s] dpkg-buildpackage: error: debian/rules build gave error exit status 2 [ 213s] [ 213s] cloud123 failed "build osmo-bsc_1.1.2.20171230.dsc" at Sat Dec 30 20:03:54 UTC 2017. [ 213s] [ 213s] ### VM INTERACTION START ### [ 216s] [ 180.058221] reboot: Power down [ 217s] ### VM INTERACTION END ### [ 217s] [ 217s] cloud123 failed "build osmo-bsc_1.1.2.20171230.dsc" at Sat Dec 30 20:03:59 UTC 2017. [ 217s] -- Configure notifications at https://build.opensuse.org/user/notifications openSUSE Build Service (https://build.opensuse.org/)
Build failure of network:osmocom:nightly/osmo-bsc in xUbuntu_16.10/i586
Visit https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/osmo-bsc/xUbuntu_16.10/i586 Package network:osmocom:nightly/osmo-bsc failed to build in xUbuntu_16.10/i586 Check out the package for editing: osc checkout network:osmocom:nightly osmo-bsc Last lines of build log: [ 100s] abisip-find.c:317:3: error: format not a string literal and no format arguments [-Werror=format-security] [ 100s]printf(line); [ 100s]^~ [ 100s] cc1: some warnings being treated as errors [ 100s] Makefile:517: recipe for target 'abisip-find.o' failed [ 100s] make[4]: *** [abisip-find.o] Error 1 [ 100s] make[4]: Leaving directory '/usr/src/packages/BUILD/src/ipaccess' [ 100s] Makefile:408: recipe for target 'all-recursive' failed [ 100s] make[3]: *** [all-recursive] Error 1 [ 100s] make[3]: Leaving directory '/usr/src/packages/BUILD/src' [ 100s] Makefile:443: recipe for target 'all-recursive' failed [ 100s] make[2]: *** [all-recursive] Error 1 [ 100s] make[2]: Leaving directory '/usr/src/packages/BUILD' [ 100s] Makefile:375: recipe for target 'all' failed [ 100s] make[1]: *** [all] Error 2 [ 100s] make[1]: Leaving directory '/usr/src/packages/BUILD' [ 100s] dh_auto_build: make -j1 returned exit code 2 [ 100s] debian/rules:45: recipe for target 'build' failed [ 100s] make: *** [build] Error 2 [ 100s] dpkg-buildpackage: error: debian/rules build gave error exit status 2 [ 100s] [ 100s] lamb61 failed "build osmo-bsc_1.1.2.20171230.dsc" at Sat Dec 30 20:02:13 UTC 2017. [ 100s] [ 100s] ### VM INTERACTION START ### [ 103s] [ 95.867208] reboot: Power down [ 103s] ### VM INTERACTION END ### [ 103s] [ 103s] lamb61 failed "build osmo-bsc_1.1.2.20171230.dsc" at Sat Dec 30 20:02:16 UTC 2017. [ 103s] -- Configure notifications at https://build.opensuse.org/user/notifications openSUSE Build Service (https://build.opensuse.org/)
[MERGED] osmocom-bb[master]: mobile: Add lua examples for basic functions that are available
Harald Welte has submitted this change and it was merged. Change subject: mobile: Add lua examples for basic functions that are available .. mobile: Add lua examples for basic functions that are available Link to Lua docs, link to our manual. Demo logging, timer, MS on/off, sms sending and sms receiving. For the MM state I will need to export constants/names to make this usable. Currently it is a bit of a hack (first time to MM_IDLE) to send a SMS. We would know when we attached though. Change-Id: I10ac656330b65e3905d6cbbb7865aa0f969cd9ff --- A doc/examples/mobile/lua_helloworld.lua A doc/examples/mobile/lua_ms_on_off.lua A doc/examples/mobile/lua_sms_on_attach.lua A doc/examples/mobile/lua_sms_receive.lua A doc/examples/mobile/lua_timer.lua 5 files changed, 91 insertions(+), 0 deletions(-) Approvals: Harald Welte: Looks good to me, approved Jenkins Builder: Verified diff --git a/doc/examples/mobile/lua_helloworld.lua b/doc/examples/mobile/lua_helloworld.lua new file mode 100644 index 000..fd5abcc --- /dev/null +++ b/doc/examples/mobile/lua_helloworld.lua @@ -0,0 +1,9 @@ +-- See https://www.lua.org/manual/5.3/ for Lua +-- See http://ftp.osmocom.org/docs/latest/osmocombb-usermanual.pdf -- Scripting with Lua + +-- Standard print and log_* are forwarded to the Osmocom logging framework +print("Hellp from Lua"); +log_notice("Notice from lua"); +log_debug("Debug from Lua"); +log_error("Error from Lua"); +log_fatal("Fatal from Lua"); diff --git a/doc/examples/mobile/lua_ms_on_off.lua b/doc/examples/mobile/lua_ms_on_off.lua new file mode 100644 index 000..57e492d --- /dev/null +++ b/doc/examples/mobile/lua_ms_on_off.lua @@ -0,0 +1,23 @@ +-- See https://www.lua.org/manual/5.3/ for Lua +-- See http://ftp.osmocom.org/docs/latest/osmocombb-usermanual.pdf -- Scripting with Lua + + +-- Switch the MS on/off but this can only be done if the MS +-- is in the right state. This assumes that the MS is fully +-- connected and doesn't stall. + +local start = false +osmo.ms().start() +function toggle_ms_state() + timer = osmo.timeout(20, function() + if start then + print("STARTING", osmo.ms().start()) + start = false + else + print("STOPPING", osmo.ms().stop(true)) + start = true + end + toggle_ms_state() + end) +end +toggle_ms_state() diff --git a/doc/examples/mobile/lua_sms_on_attach.lua b/doc/examples/mobile/lua_sms_on_attach.lua new file mode 100644 index 000..8d73f2f --- /dev/null +++ b/doc/examples/mobile/lua_sms_on_attach.lua @@ -0,0 +1,32 @@ +-- See https://www.lua.org/manual/5.3/ for Lua +-- See http://ftp.osmocom.org/docs/latest/osmocombb-usermanual.pdf -- Scripting with Lua + + +-- State change +local sent_sms = false +function mm_cb(new_state, new_substate, old_substate) + -- The system has attached and returned to idle. Send a SMS the first time + -- it happens. + if new_state == 19 and new_substate == 1 then + if not sent_sms then + sent_sms = true + osmo.ms():sms_send_simple("1234", "21321324", "fooo", 23) + end + end +end + +-- Called when a new SMS arrives or status for delivery +-- is updated. Check the msg_ref field. +function sms_cb(sms, cause, valid) + print("SMS data cb", sms, cause, valid) + for i, v in pairs(sms) do + print(i, v) + end +end + +-- We need to register a callback +local cbs = { + Sms=sms_cb, + Mm=mm_cb +} +osmo.ms():register(cbs) diff --git a/doc/examples/mobile/lua_sms_receive.lua b/doc/examples/mobile/lua_sms_receive.lua new file mode 100644 index 000..4be4e0a --- /dev/null +++ b/doc/examples/mobile/lua_sms_receive.lua @@ -0,0 +1,15 @@ +-- See https://www.lua.org/manual/5.3/ for Lua +-- See http://ftp.osmocom.org/docs/latest/osmocombb-usermanual.pdf -- Scripting with Lua + +function sms_cb(sms, cause, valid) + print("SMS data cb", sms, cause, valid) + for i, v in pairs(sms) do + print(i, v) + end +end + +local cbs = { + Sms=sms_cb, +} + +osmo.ms():register(cbs) diff --git a/doc/examples/mobile/lua_timer.lua b/doc/examples/mobile/lua_timer.lua new file mode 100644 index 000..1119af8 --- /dev/null +++ b/doc/examples/mobile/lua_timer.lua @@ -0,0 +1,12 @@ +-- See https://www.lua.org/manual/5.3/ for Lua +-- See http://ftp.osmocom.org/docs/latest/osmocombb-usermanual.pdf -- Scripting with Lua + +-- Start and stop timer with callback. Schedule a timeout and +-- resume execution then. + +-- Timeout in 10 seconds +local timer = osmo.timeout(10, function() + print("Timeout occurred"); +end) +-- We can cancel it. The callback will not be called +timer:cancel() -- To view, visit https://gerrit.osmocom.org/5596 To unsubscribe, visit
[MERGED] osmocom-bb[master]: mobile: Send SMS through the primitive interface
Harald Welte has submitted this change and it was merged. Change subject: mobile: Send SMS through the primitive interface .. mobile: Send SMS through the primitive interface Make this symmetric and send the SMS through the primitive interface. Construct and copy the sms into the prim, store the SCA in the prim as well. In 04.11 we see we can store 2*10 digits in the destination address and a NUL. Change-Id: I91d7537f4f6ce5ba00218c58f3456947ec7bc662 --- M src/host/layer23/include/osmocom/bb/mobile/gsm411_sms.h M src/host/layer23/include/osmocom/bb/mobile/primitives.h M src/host/layer23/src/mobile/gsm411_sms.c M src/host/layer23/src/mobile/primitives.c M src/host/layer23/src/mobile/script_lua.c 5 files changed, 39 insertions(+), 4 deletions(-) Approvals: Harald Welte: Looks good to me, approved Jenkins Builder: Verified diff --git a/src/host/layer23/include/osmocom/bb/mobile/gsm411_sms.h b/src/host/layer23/include/osmocom/bb/mobile/gsm411_sms.h index d3074fb..3ed6710 100644 --- a/src/host/layer23/include/osmocom/bb/mobile/gsm411_sms.h +++ b/src/host/layer23/include/osmocom/bb/mobile/gsm411_sms.h @@ -35,5 +35,7 @@ int gsm411_rcv_sms(struct osmocom_ms *ms, struct msgb *msg); int sms_send(struct osmocom_ms *ms, const char *sms_sca, const char *number, const char *text, uint8_t msg_ref); +int gsm411_tx_sms_submit(struct osmocom_ms *ms, const char *sms_sca, + struct gsm_sms *sms); #endif /* _GSM411_SMS_H */ diff --git a/src/host/layer23/include/osmocom/bb/mobile/primitives.h b/src/host/layer23/include/osmocom/bb/mobile/primitives.h index 39b4945..034b202 100644 --- a/src/host/layer23/include/osmocom/bb/mobile/primitives.h +++ b/src/host/layer23/include/osmocom/bb/mobile/primitives.h @@ -62,6 +62,8 @@ struct mobile_sms_param { struct gsm_sms sms; + char sca[20+1]; /*sms; + + /* Force a NUL at the end of the string */ + param->sca[sizeof(param->sca) - 1] = '\0'; + + return gsm411_tx_sms_submit(intf->ms, param->sca, sms); +} + int mobile_prim_intf_req(struct mobile_prim_intf *intf, struct mobile_prim *prim) { int rc = 0; @@ -184,6 +197,9 @@ case OSMO_PRIM(PRIM_MOB_TIMER_CANCEL, PRIM_OP_REQUEST): rc = cancel_timer(intf, >u.timer); break; + case OSMO_PRIM(PRIM_MOB_SMS, PRIM_OP_REQUEST): + rc = send_sms(intf, >u.sms); + break; default: LOGP(DPRIM, LOGL_ERROR, "Unknown primitive: %d\n", OSMO_PRIM_HDR(>hdr)); break; diff --git a/src/host/layer23/src/mobile/script_lua.c b/src/host/layer23/src/mobile/script_lua.c index 1b80c48..14ab58f 100644 --- a/src/host/layer23/src/mobile/script_lua.c +++ b/src/host/layer23/src/mobile/script_lua.c @@ -372,7 +372,9 @@ static int lua_ms_sms_send_simple(lua_State *L) { const char *sms_sca, *number, *text; - int msg_ref, rc; + struct mobile_prim *prim; + struct gsm_sms *sms; + int msg_ref; luaL_argcheck(L, lua_isnumber(L, -1), 4, "msg_ref needs to be a number"); luaL_argcheck(L, lua_isstring(L, -2), 3, "text must be a string"); @@ -384,8 +386,21 @@ number = lua_tostring(L, -3); sms_sca = lua_tostring(L, -4); - rc = sms_send(get_primitive(L)->ms, sms_sca, number, text, msg_ref); - lua_pushinteger(L, rc); + prim = mobile_prim_alloc(PRIM_MOB_SMS, PRIM_OP_REQUEST); + + sms = sms_from_text(number, 0, text); + if (!sms) { + lua_pushinteger(L, -ENOMEM); + msgb_free(prim->hdr.msg); + return 1; + } + + prim->u.sms.sms = *sms; + prim->u.sms.sms.msg_ref = msg_ref; + osmo_strlcpy(prim->u.sms.sca, sms_sca, sizeof(prim->u.sms.sca)); + mobile_prim_intf_req(get_primitive(L), prim); + + lua_pushinteger(L, 0); return 1; } -- To view, visit https://gerrit.osmocom.org/5594 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I91d7537f4f6ce5ba00218c58f3456947ec7bc662 Gerrit-PatchSet: 1 Gerrit-Project: osmocom-bb Gerrit-Branch: master Gerrit-Owner: Holger FreytherGerrit-Reviewer: Harald Welte Gerrit-Reviewer: Jenkins Builder
[MERGED] osmocom-bb[master]: mobile: Properly close the primitive interface on reload
Harald Welte has submitted this change and it was merged. Change subject: mobile: Properly close the primitive interface on reload .. mobile: Properly close the primitive interface on reload When reloading a script go through script_lua_close. Get the primitive first. Then destruct the lua environment which will lead to GC (e.g. cancellation of timers) and then delete the primitive code. Change-Id: I5bb4fa9e7c5010f3ad50b258dcb14956eea8822a --- M src/host/layer23/src/mobile/script_lua.c 1 file changed, 6 insertions(+), 3 deletions(-) Approvals: Harald Welte: Looks good to me, approved Jenkins Builder: Verified diff --git a/src/host/layer23/src/mobile/script_lua.c b/src/host/layer23/src/mobile/script_lua.c index 14ab58f..9b256d3 100644 --- a/src/host/layer23/src/mobile/script_lua.c +++ b/src/host/layer23/src/mobile/script_lua.c @@ -510,10 +510,14 @@ int script_lua_close(struct osmocom_ms *ms) { + struct mobile_prim_intf *intf; + if (!ms->lua_state) return 0; + intf = get_primitive(ms->lua_state); lua_close(ms->lua_state); + mobile_prim_intf_free(intf); ms->lua_state = NULL; return 0; } @@ -523,8 +527,7 @@ struct mobile_prim_intf *intf; int err; - if (ms->lua_state) - lua_close(ms->lua_state); + script_lua_close(ms); ms->lua_state = lua_newstate(talloc_lua_alloc, ms); if (!ms->lua_state) return -1; @@ -550,7 +553,7 @@ vty_out(vty, "%% LUA execute error: %s%s", lua_tostring(ms->lua_state, -1), VTY_NEWLINE); lua_pop(ms->lua_state, 1); - return 3; + return -3; } return 0; -- To view, visit https://gerrit.osmocom.org/5595 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5bb4fa9e7c5010f3ad50b258dcb14956eea8822a Gerrit-PatchSet: 1 Gerrit-Project: osmocom-bb Gerrit-Branch: master Gerrit-Owner: Holger FreytherGerrit-Reviewer: Harald Welte Gerrit-Reviewer: Jenkins Builder
osmocom-bb[master]: mobile: Send SMS through the primitive interface
Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/5594 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I91d7537f4f6ce5ba00218c58f3456947ec7bc662 Gerrit-PatchSet: 1 Gerrit-Project: osmocom-bb Gerrit-Branch: master Gerrit-Owner: Holger FreytherGerrit-Reviewer: Harald Welte Gerrit-Reviewer: Jenkins Builder Gerrit-HasComments: No
osmocom-bb[master]: mobile: Properly close the primitive interface on reload
Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/5595 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I5bb4fa9e7c5010f3ad50b258dcb14956eea8822a Gerrit-PatchSet: 1 Gerrit-Project: osmocom-bb Gerrit-Branch: master Gerrit-Owner: Holger FreytherGerrit-Reviewer: Harald Welte Gerrit-Reviewer: Jenkins Builder Gerrit-HasComments: No
osmocom-bb[master]: mobile: Add lua examples for basic functions that are available
Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/5596 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I10ac656330b65e3905d6cbbb7865aa0f969cd9ff Gerrit-PatchSet: 1 Gerrit-Project: osmocom-bb Gerrit-Branch: master Gerrit-Owner: Holger FreytherGerrit-Reviewer: Harald Welte Gerrit-Reviewer: Jenkins Builder Gerrit-HasComments: No
osmo-iuh[master]: Add control interface
Patch Set 2: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/5591 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I4637e88da00bac1ab0237c29ac73806d024863ba Gerrit-PatchSet: 2 Gerrit-Project: osmo-iuh Gerrit-Branch: master Gerrit-Owner: MaxGerrit-Reviewer: Harald Welte Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: Neels Hofmeyr Gerrit-Reviewer: lynxis lazus Gerrit-HasComments: No
osmo-iuh[master]: Expand ctrl interface
Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/5599 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I3a2d6fa3d6d0829ccee4ecc0998d9299c97820e9 Gerrit-PatchSet: 1 Gerrit-Project: osmo-iuh Gerrit-Branch: master Gerrit-Owner: MaxGerrit-Reviewer: Harald Welte Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: Neels Hofmeyr Gerrit-Reviewer: lynxis lazus Gerrit-HasComments: No
osmo-sgsn[master]: Remove obsolete ./configure option
Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/5606 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I6230cfcf10247b601e16e7b872305499815520a9 Gerrit-PatchSet: 1 Gerrit-Project: osmo-sgsn Gerrit-Branch: master Gerrit-Owner: MaxGerrit-Reviewer: Harald Welte Gerrit-Reviewer: Jenkins Builder Gerrit-HasComments: No
pysim[master]: utils.py: dec_imsi: Fix ValueError
Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/5623 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I7d3ecbf9edd190d1941816796cee60e3957d5943 Gerrit-PatchSet: 1 Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Owner: Pau Espin PedrolGerrit-Reviewer: Harald Welte Gerrit-HasComments: No
[MERGED] osmo-bsc[master]: Fix .deb builds
Harald Welte has submitted this change and it was merged. Change subject: Fix .deb builds .. Fix .deb builds The code triggers following error: abisip-find.c:317:3: error: format not a string literal and no format arguments [-Werror=format-security] The error was introduced in 5bf1e15c55340f236d84f70d3d04c871403d3503. Change-Id: I613781495edbc53916ca70ff7b78d28ffabd3f5d --- M src/ipaccess/abisip-find.c 1 file changed, 2 insertions(+), 6 deletions(-) Approvals: Harald Welte: Looks good to me, approved Jenkins Builder: Verified diff --git a/src/ipaccess/abisip-find.c b/src/ipaccess/abisip-find.c index d94e18a..4e1d192 100644 --- a/src/ipaccess/abisip-find.c +++ b/src/ipaccess/abisip-find.c @@ -312,12 +312,8 @@ base_stations_bump(changed); printf("RX: %u \r", responses); fflush(stdout); - } else { - char *line = parse_response(ctx, buf, len); - printf(line); - printf("\n"); - talloc_free(line); - } + } else + printf("%s\n", parse_response(ctx, buf, len)); } static int read_response(int fd) -- To view, visit https://gerrit.osmocom.org/5619 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I613781495edbc53916ca70ff7b78d28ffabd3f5d Gerrit-PatchSet: 1 Gerrit-Project: osmo-bsc Gerrit-Branch: master Gerrit-Owner: MaxGerrit-Reviewer: Harald Welte Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: Neels Hofmeyr
osmo-bsc[master]: Fix .deb builds
Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/5619 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I613781495edbc53916ca70ff7b78d28ffabd3f5d Gerrit-PatchSet: 1 Gerrit-Project: osmo-bsc Gerrit-Branch: master Gerrit-Owner: MaxGerrit-Reviewer: Harald Welte Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: Neels Hofmeyr Gerrit-HasComments: No
osmo-bsc[master]: Check and handle SMS encoding failure
Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/5615 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I9fc16e24f7df5ebad6f4f1b389b2c5e861be95d7 Gerrit-PatchSet: 1 Gerrit-Project: osmo-bsc Gerrit-Branch: master Gerrit-Owner: MaxGerrit-Reviewer: Harald Welte Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: daniel Gerrit-Reviewer: lynxis lazus Gerrit-HasComments: No
[MERGED] osmo-msc[master]: VLR: remove unused parameter
Harald Welte has submitted this change and it was merged. Change subject: VLR: remove unused parameter .. VLR: remove unused parameter The expire_lu is never used but is printed for every subscriber. Let's remove it to avoid confusion. Change-Id: I6f7ad1670836384d1e6a58f47a13464fdbbf8509 --- M include/osmocom/msc/vlr.h M src/libmsc/vty_interface_layer3.c 2 files changed, 0 insertions(+), 8 deletions(-) Approvals: Harald Welte: Looks good to me, approved Jenkins Builder: Verified diff --git a/include/osmocom/msc/vlr.h b/include/osmocom/msc/vlr.h index 1b365a9..d68b7be 100644 --- a/include/osmocom/msc/vlr.h +++ b/include/osmocom/msc/vlr.h @@ -136,7 +136,6 @@ bool la_allowed; int use_count; - time_t expire_lu; /* FIXME: overlap with periodic_lu_timer/age_indicator */ struct osmo_fsm_inst *lu_fsm; struct osmo_fsm_inst *auth_fsm; diff --git a/src/libmsc/vty_interface_layer3.c b/src/libmsc/vty_interface_layer3.c index 02a3600..65ba178 100644 --- a/src/libmsc/vty_interface_layer3.c +++ b/src/libmsc/vty_interface_layer3.c @@ -64,7 +64,6 @@ { int reqs; struct llist_head *entry; - char expire_time[200]; if (strlen(vsub->name)) vty_out(vty, "Name: '%s'%s", vsub->name, VTY_NEWLINE); @@ -109,12 +108,6 @@ osmo_hexdump(t->vec.kc, sizeof(t->vec.kc)), VTY_NEWLINE); } - - /* print the expiration time of a subscriber */ - strftime(expire_time, sizeof(expire_time), - "%a, %d %b %Y %T %z", localtime(>expire_lu)); - expire_time[sizeof(expire_time) - 1] = '\0'; - vty_out(vty, "Expiration Time: %s%s", expire_time, VTY_NEWLINE); reqs = 0; llist_for_each(entry, >cs.requests) -- To view, visit https://gerrit.osmocom.org/5620 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6f7ad1670836384d1e6a58f47a13464fdbbf8509 Gerrit-PatchSet: 1 Gerrit-Project: osmo-msc Gerrit-Branch: master Gerrit-Owner: MaxGerrit-Reviewer: Harald Welte Gerrit-Reviewer: Jenkins Builder
osmo-msc[master]: VLR: remove unused parameter
Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/5620 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I6f7ad1670836384d1e6a58f47a13464fdbbf8509 Gerrit-PatchSet: 1 Gerrit-Project: osmo-msc Gerrit-Branch: master Gerrit-Owner: MaxGerrit-Reviewer: Harald Welte Gerrit-Reviewer: Jenkins Builder Gerrit-HasComments: No
gapk[master]: Add git-review config
Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/5624 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Id4304dc8248087c23aaf3a5d205cd341ed6a758a Gerrit-PatchSet: 1 Gerrit-Project: gapk Gerrit-Branch: master Gerrit-Owner: MaxGerrit-Reviewer: Harald Welte Gerrit-Reviewer: fixeria Gerrit-Reviewer: lynxis lazus Gerrit-Reviewer: tnt Gerrit-HasComments: No
pysim[master]: pySim-*.py: Set shebang to use python v2
Patch Set 2: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/5621 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I6ab4e9edc44a8045915d4828c6de2fa98027fb7e Gerrit-PatchSet: 2 Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Owner: Pau Espin PedrolGerrit-Reviewer: Harald Welte Gerrit-Reviewer: Max Gerrit-HasComments: No
pysim[master]: pySim-prog.py: Fix trailing whitespace
Patch Set 2: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/5622 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I735dc7bb774d77d3b60b1712b0f0afcbb81dc726 Gerrit-PatchSet: 2 Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Owner: Pau Espin PedrolGerrit-Reviewer: Harald Welte Gerrit-Reviewer: Max Gerrit-HasComments: No