Build failure of network:osmocom:nightly/libosmocore in Raspbian_9.0/armv7l

2019-07-17 Thread OBS Notification
Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/libosmocore/Raspbian_9.0/armv7l

Package network:osmocom:nightly/libosmocore failed to build in 
Raspbian_9.0/armv7l

Check out the package for editing:
  osc checkout network:osmocom:nightly libosmocore

Last lines of build log:
[  771s] | This file was extended by libosmocore config.status 1.1.0.80-8f67, 
which was
[  771s] | generated by GNU Autoconf 2.69.  Invocation command line was
[  771s] | 
[  771s] |   CONFIG_FILES= 
[  771s] |   CONFIG_HEADERS  = 
[  771s] |   CONFIG_LINKS= 
[  771s] |   CONFIG_COMMANDS = 
[  771s] |   $ ./config.status Doxyfile.core
[  771s] | 
[  771s] | on obs-arm-5
[  771s] | 
[  771s] | config.status:1200: creating Doxyfile.core
[  771s] 
[  771s] debian/rules:26: recipe for target 'override_dh_auto_test' failed
[  771s] make[1]: *** [override_dh_auto_test] Error 1
[  771s] make[1]: Leaving directory '/usr/src/packages/BUILD'
[  771s] debian/rules:15: recipe for target 'build' failed
[  771s] make: *** [build] Error 2
[  771s] dpkg-buildpackage: error: debian/rules build gave error exit status 2
[  771s] 
[  771s] obs-arm-5 failed "build libosmocore_1.1.0.80.8f67.dsc" at Thu Jul 18 
04:44:53 UTC 2019.
[  771s] 
[  771s] ### VM INTERACTION START ###
[  773s] [  739.650671] sysrq: SysRq : Power Off
[  773s] [  739.652896] reboot: Power down
[  774s] ### VM INTERACTION END ###
[  774s] 
[  774s] obs-arm-5 failed "build libosmocore_1.1.0.80.8f67.dsc" at Thu Jul 18 
04:44:58 UTC 2019.
[  774s] 

-- 
Configure notifications at https://build.opensuse.org/user/notifications
openSUSE Build Service (https://build.opensuse.org/)


Build failure of network:osmocom:nightly/osmo-python-tests in Debian_Unstable/x86_64

2019-07-17 Thread OBS Notification
Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/osmo-python-tests/Debian_Unstable/x86_64

Package network:osmocom:nightly/osmo-python-tests failed to build in 
Debian_Unstable/x86_64

Check out the package for editing:
  osc checkout network:osmocom:nightly osmo-python-tests

Last lines of build log:
[  112s] rm -rf /usr/src/packages/BUILD/debian/python3-osmopy-utils/usr/lib
[  112s] make[1]: Leaving directory '/usr/src/packages/BUILD'
[  112s]dh_installdocs
[  112s]dh_installchangelogs
[  113s]dh_python2
[  113s] I: dh_python2 tools:114: replacing shebang in 
debian/python2-osmopy-utils/usr/bin/osmotestvty.py
[  113s] I: dh_python2 tools:114: replacing shebang in 
debian/python2-osmopy-utils/usr/bin/osmotestconfig.py
[  113s] I: dh_python2 tools:114: replacing shebang in 
debian/python2-osmopy-utils/usr/bin/osmodumpdoc.py
[  113s]dh_python3
[  113s]debian/rules override_dh_installinit
[  113s] make[1]: Entering directory '/usr/src/packages/BUILD'
[  113s] # Install service file with different name than package name:
[  113s] # 
https://unix.stackexchange.com/questions/306234/is-it-possible-to-install-two-services-for-one-package-using-dh-installinit-how
[  113s] dh_installinit --name=osmo-ctrl2cgi
[  113s] dh_installinit: --name=osmo-ctrl2cgi option specified, but init script 
nor default file not found
[  113s] make[1]: *** [debian/rules:19: override_dh_installinit] Error 255
[  113s] make[1]: Leaving directory '/usr/src/packages/BUILD'
[  113s] make: *** [debian/rules:4: binary] Error 2
[  113s] dpkg-buildpackage: error: fakeroot debian/rules binary subprocess 
returned exit status 2
[  113s] 
[  113s] cloud109 failed "build osmo-python-tests_0.0.9.20190718.dsc" at Thu 
Jul 18 04:23:02 UTC 2019.
[  113s] 
[  113s] ### VM INTERACTION START ###
[  116s] [   99.422146] sysrq: SysRq : Power Off
[  116s] [   99.426478] reboot: Power down
[  116s] ### VM INTERACTION END ###
[  116s] 
[  116s] cloud109 failed "build osmo-python-tests_0.0.9.20190718.dsc" at Thu 
Jul 18 04:23:06 UTC 2019.
[  116s] 

-- 
Configure notifications at https://build.opensuse.org/user/notifications
openSUSE Build Service (https://build.opensuse.org/)


Change in ...libosmocore[master]: bitvec/bitval2mask(): cosmetic: get rid of temporary int variable

2019-07-17 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/libosmocore/+/14825 )

Change subject: bitvec/bitval2mask(): cosmetic: get rid of temporary int 
variable
..


Patch Set 1: Code-Review+2


--
To view, visit https://gerrit.osmocom.org/c/libosmocore/+/14825
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I9d6f6b66c99c43107d1ad3e80af332e967bb19e8
Gerrit-Change-Number: 14825
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Wed, 17 Jul 2019 22:22:33 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in ...libosmocore[master]: osmo_gsm48_rest_octets_si3_decode(): fix copy-paste error in comment

2019-07-17 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/libosmocore/+/14824 )

Change subject: osmo_gsm48_rest_octets_si3_decode(): fix copy-paste error in 
comment
..


Patch Set 1: Code-Review+2


--
To view, visit https://gerrit.osmocom.org/c/libosmocore/+/14824
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I57a330e16cc2910597672e1b27cf971499cea5bc
Gerrit-Change-Number: 14824
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Wed, 17 Jul 2019 22:22:03 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in ...osmo-tetra[master]: Fix typos

2019-07-17 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-tetra/+/14823 )

Change subject: Fix typos
..


Patch Set 1: Code-Review+1


--
To view, visit https://gerrit.osmocom.org/c/osmo-tetra/+/14823
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-tetra
Gerrit-Branch: master
Gerrit-Change-Id: I1aae0a52fcf978900fa847a3f813beb2a5ad4a84
Gerrit-Change-Number: 14823
Gerrit-PatchSet: 1
Gerrit-Owner: roox 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Wed, 17 Jul 2019 22:21:35 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in ...gapk[master]: Fix typos

2019-07-17 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/gapk/+/14822 )

Change subject: Fix typos
..


Patch Set 1: Code-Review+1


--
To view, visit https://gerrit.osmocom.org/c/gapk/+/14822
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: gapk
Gerrit-Branch: master
Gerrit-Change-Id: Ie8054f3d484fe5d82cc5d91f159906a470fc4511
Gerrit-Change-Number: 14822
Gerrit-PatchSet: 1
Gerrit-Owner: roox 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Wed, 17 Jul 2019 22:21:02 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in ...osmo-bts[master]: Fix common misspellings and typos

2019-07-17 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-bts/+/14821 )

Change subject: Fix common misspellings and typos
..


Patch Set 1:

(2 comments)

https://gerrit.osmocom.org/#/c/14821/1/debian/changelog
File debian/changelog:

https://gerrit.osmocom.org/#/c/14821/1/debian/changelog@136
PS1, Line 136:   * rtp-amr: Describe requirement to receive all PH-DATA.ind 
events
Same, not sure we want to fix stuff in changelog comming frm commit logs.


https://gerrit.osmocom.org/#/c/14821/1/src/osmo-bts-sysmo/misc/sysmobts_mgr_calib.c
File src/osmo-bts-sysmo/misc/sysmobts_mgr_calib.c:

https://gerrit.osmocom.org/#/c/14821/1/src/osmo-bts-sysmo/misc/sysmobts_mgr_calib.c@57
PS1, Line 57:   CALIB_SUCCESS,
Probably this change and related modifications deserve a separate patch (not a 
simple comment typo).



--
To view, visit https://gerrit.osmocom.org/c/osmo-bts/+/14821
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: I00ebf013104b55decbdd7f758756a19b220dc7b3
Gerrit-Change-Number: 14821
Gerrit-PatchSet: 1
Gerrit-Owner: roox 
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: pespin 
Gerrit-Comment-Date: Wed, 17 Jul 2019 22:20:24 +
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment


Change in ...osmo-trx[master]: Fix common misspellings and typos

2019-07-17 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-trx/+/14819 )

Change subject: Fix common misspellings and typos
..


Patch Set 1:

(1 comment)

https://gerrit.osmocom.org/#/c/14819/1/debian/changelog 
File debian/changelog:

https://gerrit.osmocom.org/#/c/14819/1/debian/changelog@229
PS1, Line 229:   * Move arch specific files to arch subdir
I wouldn't change stuff from changelog, since that stuff comes from commit 
messages.



--
To view, visit https://gerrit.osmocom.org/c/osmo-trx/+/14819
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-trx
Gerrit-Branch: master
Gerrit-Change-Id: I8e0fe9d67d1e1e8dfce319610de63cde170694c5
Gerrit-Change-Number: 14819
Gerrit-PatchSet: 1
Gerrit-Owner: roox 
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: pespin 
Gerrit-Comment-Date: Wed, 17 Jul 2019 22:14:15 +
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment


Change in ...osmocom-bb[master]: Fix common misspellings and typos

2019-07-17 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmocom-bb/+/14818 )

Change subject: Fix common misspellings and typos
..


Patch Set 1:

(1 comment)

I'd remove the variable name from this change, looks not the same kind of fix.

https://gerrit.osmocom.org/#/c/14818/1/src/target/firmware/apps/simtest/main.c
File src/target/firmware/apps/simtest/main.c:

https://gerrit.osmocom.org/#/c/14818/1/src/target/firmware/apps/simtest/main.c@217
PS1, Line 217:  int receivedChars;
this one looks suspicious. Variable name changed but no issue?



--
To view, visit https://gerrit.osmocom.org/c/osmocom-bb/+/14818
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Change-Id: I75df25d3bc2ecb4985bf6fefdde6e44b21bffed9
Gerrit-Change-Number: 14818
Gerrit-PatchSet: 1
Gerrit-Owner: roox 
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: pespin 
Gerrit-Comment-Date: Wed, 17 Jul 2019 22:12:17 +
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment


Change in ...osmo-bts[master]: common/rsl.c: fix: properly handle SI3 Rest Octets

2019-07-17 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-bts/+/14817 )

Change subject: common/rsl.c: fix: properly handle SI3 Rest Octets
..


Patch Set 3: Code-Review+1


--
To view, visit https://gerrit.osmocom.org/c/osmo-bts/+/14817
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: Ia962cf21903ba674057cf52746996dd3254bc1c6
Gerrit-Change-Number: 14817
Gerrit-PatchSet: 3
Gerrit-Owner: fixeria 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria 
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: pespin 
Gerrit-CC: tnt 
Gerrit-Comment-Date: Wed, 17 Jul 2019 22:03:22 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in ...osmo-iuh[master]: osmo-hnbgw.cfg: set local-ip to 0.0.0.0

2019-07-17 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-iuh/+/14782 )

Change subject: osmo-hnbgw.cfg: set local-ip to 0.0.0.0
..


Patch Set 2: Code-Review+2


--
To view, visit https://gerrit.osmocom.org/c/osmo-iuh/+/14782
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-iuh
Gerrit-Branch: master
Gerrit-Change-Id: I354140f014854f1755b649e40a65e5d88b99c0ec
Gerrit-Change-Number: 14782
Gerrit-PatchSet: 2
Gerrit-Owner: osmith 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria 
Gerrit-Reviewer: osmith 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Wed, 17 Jul 2019 21:44:51 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in ...docker-playground[master]: debian-repo-install-test: kill already running

2019-07-17 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/docker-playground/+/14780 )

Change subject: debian-repo-install-test: kill already running
..


Patch Set 4: Code-Review+2


--
To view, visit https://gerrit.osmocom.org/c/docker-playground/+/14780
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: docker-playground
Gerrit-Branch: master
Gerrit-Change-Id: Ide795092b656c9f0eb92a075d8f662944089019f
Gerrit-Change-Number: 14780
Gerrit-PatchSet: 4
Gerrit-Owner: osmith 
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: osmith 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Wed, 17 Jul 2019 21:43:41 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in ...libosmocore[master]: bitvec/bitval2mask(): cosmetic: get rid of temporary int variable

2019-07-17 Thread fixeria
fixeria has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/libosmocore/+/14825


Change subject: bitvec/bitval2mask(): cosmetic: get rid of temporary int 
variable
..

bitvec/bitval2mask(): cosmetic: get rid of temporary int variable

Change-Id: I9d6f6b66c99c43107d1ad3e80af332e967bb19e8
---
M src/bitvec.c
1 file changed, 4 insertions(+), 11 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/libosmocore refs/changes/25/14825/1

diff --git a/src/bitvec.c b/src/bitvec.c
index dac0401..0c263ad 100644
--- a/src/bitvec.c
+++ b/src/bitvec.c
@@ -58,25 +58,18 @@
 /* convert ZERO/ONE/L/H to a bitmask at given pos in a byte */
 static uint8_t bitval2mask(enum bit_value bit, uint8_t bitnum)
 {
-   int bitval;
-
switch (bit) {
case ZERO:
-   bitval = (0 << bitnum);
-   break;
+   return (0 << bitnum);
case ONE:
-   bitval = (1 << bitnum);
-   break;
+   return (1 << bitnum);
case L:
-   bitval = ((0x2b ^ (0 << bitnum)) & (1 << bitnum));
-   break;
+   return ((0x2b ^ (0 << bitnum)) & (1 << bitnum));
case H:
-   bitval = ((0x2b ^ (1 << bitnum)) & (1 << bitnum));
-   break;
+   return ((0x2b ^ (1 << bitnum)) & (1 << bitnum));
default:
return 0;
}
-   return bitval;
 }

 /*! check if the bit is 0 or 1 for a given position inside a bitvec

--
To view, visit https://gerrit.osmocom.org/c/libosmocore/+/14825
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I9d6f6b66c99c43107d1ad3e80af332e967bb19e8
Gerrit-Change-Number: 14825
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria 
Gerrit-MessageType: newchange


Change in ...libosmocore[master]: osmo_gsm48_rest_octets_si3_decode(): fix copy-paste error in comment

2019-07-17 Thread fixeria
fixeria has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/libosmocore/+/14824


Change subject: osmo_gsm48_rest_octets_si3_decode(): fix copy-paste error in 
comment
..

osmo_gsm48_rest_octets_si3_decode(): fix copy-paste error in comment

Change-Id: I57a330e16cc2910597672e1b27cf971499cea5bc
---
M src/gsm/gsm48_rest_octets.c
1 file changed, 1 insertion(+), 1 deletion(-)



  git pull ssh://gerrit.osmocom.org:29418/libosmocore refs/changes/24/14824/1

diff --git a/src/gsm/gsm48_rest_octets.c b/src/gsm/gsm48_rest_octets.c
index 84136c6..518572e 100644
--- a/src/gsm/gsm48_rest_octets.c
+++ b/src/gsm/gsm48_rest_octets.c
@@ -966,7 +966,7 @@

memset(si3, 0, sizeof(*si3));

-   /* Optional Selection Paraosmo_gsm48_si_ro_infometers */
+   /* Optional Selection Parameters */
if (bitvec_get_bit_high() == H) {
sp->present = 1;
sp->cbq = bitvec_get_uint(, 1);

--
To view, visit https://gerrit.osmocom.org/c/libosmocore/+/14824
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I57a330e16cc2910597672e1b27cf971499cea5bc
Gerrit-Change-Number: 14824
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria 
Gerrit-MessageType: newchange


Change in ...osmo-tetra[master]: Fix typos

2019-07-17 Thread roox
roox has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmo-tetra/+/14823


Change subject: Fix typos
..

Fix typos

Change-Id: I1aae0a52fcf978900fa847a3f813beb2a5ad4a84
---
M etsi_codec-patches/README
M src/demod/cqpsk.py
2 files changed, 3 insertions(+), 3 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-tetra refs/changes/23/14823/1

diff --git a/etsi_codec-patches/README b/etsi_codec-patches/README
index 1c95c83..9b59b74 100644
--- a/etsi_codec-patches/README
+++ b/etsi_codec-patches/README
@@ -33,5 +33,5 @@
 Due to the use of uppercase file names in the zip file, you should use the "-L"
 flag to the unzip program, which ensures all files are completely lower-case.

-Please apply all of the patches foudn in this directory to the
+Please apply all of the patches found in this directory to the
 resulting source code.
diff --git a/src/demod/cqpsk.py b/src/demod/cqpsk.py
index 6be6ee7..c5a8f3d 100644
--- a/src/demod/cqpsk.py
+++ b/src/demod/cqpsk.py
@@ -151,7 +151,7 @@
 Adds QPSK modulation-specific options to the standard parser
 """
 parser.add_option("", "--excess-bw", type="float", 
default=_def_excess_bw,
-  help="set RRC excess bandwith factor 
[default=%default] (PSK)")
+  help="set RRC excess bandwidth factor 
[default=%default] (PSK)")
 parser.add_option("", "--no-gray-code", dest="gray_code",
   action="store_false", default=_def_gray_code,
   help="disable gray coding on modulated bits (PSK)")
@@ -325,7 +325,7 @@
 Adds modulation-specific options to the standard parser
 """
 parser.add_option("", "--excess-bw", type="float", 
default=_def_excess_bw,
-  help="set RRC excess bandwith factor 
[default=%default] (PSK)")
+  help="set RRC excess bandwidth factor 
[default=%default] (PSK)")
 parser.add_option("", "--no-gray-code", dest="gray_code",
   action="store_false", default=_def_gray_code,
   help="disable gray coding on modulated bits (PSK)")

--
To view, visit https://gerrit.osmocom.org/c/osmo-tetra/+/14823
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-tetra
Gerrit-Branch: master
Gerrit-Change-Id: I1aae0a52fcf978900fa847a3f813beb2a5ad4a84
Gerrit-Change-Number: 14823
Gerrit-PatchSet: 1
Gerrit-Owner: roox 
Gerrit-MessageType: newchange


Change in ...gapk[master]: Fix typos

2019-07-17 Thread roox
roox has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/gapk/+/14822


Change subject: Fix typos
..

Fix typos

Change-Id: Ie8054f3d484fe5d82cc5d91f159906a470fc4511
---
M src/pq_alsa.c
M src/pq_file.c
M src/pq_rtp.c
3 files changed, 5 insertions(+), 5 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/gapk refs/changes/22/14822/1

diff --git a/src/pq_alsa.c b/src/pq_alsa.c
index 3550221..50479e6 100644
--- a/src/pq_alsa.c
+++ b/src/pq_alsa.c
@@ -176,7 +176,7 @@
  *  \param pq Processing Queue to add the ALSA input to
  *  \param[in] hwdev ALSA hardware device to use
  *  \param[in] blk_len block length to be read from device
- *  \returns 0 on sucess; negative on error */
+ *  \returns 0 on success; negative on error */
 int
 osmo_gapk_pq_queue_alsa_input(struct osmo_gapk_pq *pq, const char *hwdev, 
unsigned int blk_len)
 {
@@ -190,7 +190,7 @@
  *  \param pq Processing Queue to add the ALSA output to
  *  \param[in] hwdev ALSA hardware device to use
  *  \param[in] blk_len block length to be written to device
- *  \returns 0 on sucess; negative on error */
+ *  \returns 0 on success; negative on error */
 int
 osmo_gapk_pq_queue_alsa_output(struct osmo_gapk_pq *pq, const char *hwdev, 
unsigned int blk_len)
 {
diff --git a/src/pq_file.c b/src/pq_file.c
index 22dc9cd..d496eec 100644
--- a/src/pq_file.c
+++ b/src/pq_file.c
@@ -103,7 +103,7 @@
  *  \param pq Processing Queue to add the input file to
  *  \param[in] src caller-fopen()ed input file
  *  \param[in] blk_len block length to be read from file
- *  \returns 0 on sucess; negative on error */
+ *  \returns 0 on success; negative on error */
 int
 osmo_gapk_pq_queue_file_input(struct osmo_gapk_pq *pq, FILE *src, unsigned int 
blk_len)
 {
@@ -117,7 +117,7 @@
  *  \param pq Processing Queue to add the output file to
  *  \param[in] dst caller-fopen()ed output file
  *  \param[in] blk_len block length to be written to file
- *  \returns 0 on sucess; negative on error */
+ *  \returns 0 on success; negative on error */
 int
 osmo_gapk_pq_queue_file_output(struct osmo_gapk_pq *pq, FILE *dst, unsigned 
int blk_len)
 {
diff --git a/src/pq_rtp.c b/src/pq_rtp.c
index 81eeb19..3a09ba7 100644
--- a/src/pq_rtp.c
+++ b/src/pq_rtp.c
@@ -116,7 +116,7 @@
payload = buf + sizeof(struct rtp_hdr) + (rtph->csrc_count << 2);
payload_len = rv - sizeof(struct rtp_hdr) - (rtph->csrc_count << 2);
if (payload_len < 0) {
-   rtp_err("non-existant RTP payload length %d\n", payload_len);
+   rtp_err("non-existent RTP payload length %d\n", payload_len);
return -1;
}


--
To view, visit https://gerrit.osmocom.org/c/gapk/+/14822
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: gapk
Gerrit-Branch: master
Gerrit-Change-Id: Ie8054f3d484fe5d82cc5d91f159906a470fc4511
Gerrit-Change-Number: 14822
Gerrit-PatchSet: 1
Gerrit-Owner: roox 
Gerrit-MessageType: newchange


Change in ...osmo-bts[master]: Fix common misspellings and typos

2019-07-17 Thread roox
roox has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmo-bts/+/14821


Change subject: Fix common misspellings and typos
..

Fix common misspellings and typos

Change-Id: I00ebf013104b55decbdd7f758756a19b220dc7b3
---
M README.md
M debian/changelog
M doc/manuals/rtp-amr.adoc
M doc/manuals/vty/bts_vty_reference.xml
M src/common/l1sap.c
M src/osmo-bts-litecell15/misc/lc15bts_mgr_calib.c
M src/osmo-bts-litecell15/misc/lc15bts_mgr_vty.c
M src/osmo-bts-oc2g/misc/oc2gbts_mgr_calib.c
M src/osmo-bts-oc2g/misc/oc2gbts_mgr_vty.c
M src/osmo-bts-octphy/l1_oml.c
M src/osmo-bts-sysmo/misc/sysmobts-layer1.c
M src/osmo-bts-sysmo/misc/sysmobts_mgr_calib.c
M src/osmo-bts-sysmo/misc/sysmobts_mgr_vty.c
M tests/meas/meas_test.c
14 files changed, 118 insertions(+), 118 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-bts refs/changes/21/14821/1

diff --git a/README.md b/README.md
index 7e26d82..1fca596 100644
--- a/README.md
+++ b/README.md
@@ -45,7 +45,7 @@
 as well as a
 [VTY Reference 
Manual](http://ftp.osmocom.org/docs/latest/osmobsc-vty-reference.pdf)
 and a
-[Abis refrence MAnual](http://ftp.osmocom.org/docs/latest/osmobts-abis.pdf)
+[Abis reference MAnual](http://ftp.osmocom.org/docs/latest/osmobts-abis.pdf)
 describing the OsmoBTS specific A-bis dialect.
 
 Mailing List
diff --git a/debian/changelog b/debian/changelog
index a33d21f..0dce96c 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -133,7 +133,7 @@
   * bts_model: Allow TS connect to be processed asynchronously
   * OsmoBTS/chapters/configuration.adoc: Add bts index in example
   * OsmoBTS: Update osmotrx ip to diff between local and remote
-  * rtp-amr: Describe requirement to recieve all PH-DATA.ind events
+  * rtp-amr: Describe requirement to receive all PH-DATA.ind events
   * rtp-amr: Fix typo and trailing whitespace
   * Allow easily disabling GFDL references
   * OsmoBTS: bts-models: Add Lime Microsystems devices as supported osmo-trx
@@ -341,7 +341,7 @@
   * RSL: Implement DELETE INDICATION on AGCH overflow
   * RSL: Send ERROR REPORT on too short/truncated messages + wrong 
discriminator
   * BTS: add rate_ctr about CCCH (paging, agch, pch)
-  * paging: Drop + Log paging requests for non-existant paging groups
+  * paging: Drop + Log paging requests for non-existent paging groups
   * paging.c: Fix encoding of optional Mobile ID RR PAGING TYPE 1 / 2
   * rsl: Improve ERROR REPORTing
   * paging: Fix encoding of PAGING TYPE 3 Rest Octets
@@ -601,7 +601,7 @@
   * Add Abis OML failure event reporting
   * Alarm on various errors
   * Remove obsolete define TLVP_PRES_LEN
-  * scheduler: log lchan on which prim error occured
+  * scheduler: log lchan on which prim error occurred
   * deb: use gsm_data_shared.* from openbsc-dev
   * OML: internalize failure reporting
   * Add ctrl command to send OML alert
diff --git a/doc/manuals/rtp-amr.adoc b/doc/manuals/rtp-amr.adoc
index 5a36aea..0c733d1 100644
--- a/doc/manuals/rtp-amr.adoc
+++ b/doc/manuals/rtp-amr.adoc
@@ -95,10 +95,10 @@
phy => bts  [label="PH-DATA.ind GsmL1_TchPlType_Amr (SID_FIRST)"];
bts => mgw  [label="RTP (AMR FT=SID,Q=1)"];

-   ms -x phy   [label="Supressed L1 burst"];
-   ms -x phy   [label="Supressed L1 burst"];
-   ms -x phy   [label="Supressed L1 burst"];
-   ms -x phy   [label="Supressed L1 burst"];
+   ms -x phy   [label="Suppressed L1 burst"];
+   ms -x phy   [label="Suppressed L1 burst"];
+   ms -x phy   [label="Suppressed L1 burst"];
+   ms -x phy   [label="Suppressed L1 burst"];
phy => bts  [label="PH-DATA.ind with empty payload"];
bts -x mgw  [label="Suppressed RTP frame"];

@@ -110,10 +110,10 @@
bts -x mgw  [label="Suppressed RTP frame"];


-   ms -x phy   [label="Supressed L1 burst"];
-   ms -x phy   [label="Supressed L1 burst"];
-   ms -x phy   [label="Supressed L1 burst"];
-   ms -x phy   [label="Supressed L1 burst"];
+   ms -x phy   [label="Suppressed L1 burst"];
+   ms -x phy   [label="Suppressed L1 burst"];
+   ms -x phy   [label="Suppressed L1 burst"];
+   ms -x phy   [label="Suppressed L1 burst"];
phy => bts  [label="PH-DATA.ind GsmL1_TchPlType_Amr (SID_UPDATE)"];
bts => mgw  [label="RTP (AMR FT=SID,Q=1)"];
 }
@@ -123,7 +123,7 @@
 the SID_FIRST frame is immediately followed by a speech frame. It has been 
observed that some phone
 does not transmit the last 4 bursts even if it is not followed by a speech 
frame.

-ULSU2:: There must be exactly two supressed voice frames between the
+ULSU2:: There must be exactly two suppressed voice frames between the
 SID_FIRST and the SID_UPDATE, i.e. there's 60ms between SID_FIRST and
 SID_UPDATE.

@@ -152,52 +152,52 @@
phy => bts  [label="PH-DATA.ind with empty payload"];
bts -x mgw  

Change in ...osmo-remsim[master]: Fix common misspellings and typos

2019-07-17 Thread roox
roox has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmo-remsim/+/14820


Change subject: Fix common misspellings and typos
..

Fix common misspellings and typos

Change-Id: I8152acaccd33a4b646d56cba0748aeefa0c16b35
---
M doc/manuals/chapters/remsim-bankd.adoc
M include/osmocom/rspro/INTEGER.h
M include/osmocom/rspro/constr_TYPE.h
M include/osmocom/rspro/per_support.h
M src/rspro/NativeInteger.c
M src/rspro/OCTET_STRING.c
M src/rspro/ber_tlv_tag.c
M src/rspro/constr_CHOICE.c
M src/rspro/constr_SEQUENCE.c
M src/rspro/converter-sample.c
M src/rspro/per_opentype.c
M src/simtrace2/apdu_dispatch.c
12 files changed, 14 insertions(+), 14 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-remsim refs/changes/20/14820/1

diff --git a/doc/manuals/chapters/remsim-bankd.adoc 
b/doc/manuals/chapters/remsim-bankd.adoc
index d35056b..a3a1921 100644
--- a/doc/manuals/chapters/remsim-bankd.adoc
+++ b/doc/manuals/chapters/remsim-bankd.adoc
@@ -127,4 +127,4 @@
 

 In this example, there's only a single PC/SC reader available, and it has a 
string of
-"Alcor Micro AU9560 00 00" whcih needs to be copy-pasted into the CSV file.
+"Alcor Micro AU9560 00 00" which needs to be copy-pasted into the CSV file.
diff --git a/include/osmocom/rspro/INTEGER.h b/include/osmocom/rspro/INTEGER.h
index e8b36c7..c8edf12 100644
--- a/include/osmocom/rspro/INTEGER.h
+++ b/include/osmocom/rspro/INTEGER.h
@@ -68,8 +68,8 @@
 ASN_STRTOL_ERROR_RANGE = -3,  /* Input outside of numeric range for long 
type */
 ASN_STRTOL_ERROR_INVAL = -2,  /* Invalid data encountered (e.g., "+-") */
 ASN_STRTOL_EXPECT_MORE = -1,  /* More data expected (e.g. "+") */
-ASN_STRTOL_OK  =  0,  /* Conversion succeded, number ends at 
(*end) */
-ASN_STRTOL_EXTRA_DATA  =  1,  /* Conversion succeded, but the string has 
extra stuff */
+ASN_STRTOL_OK  =  0,  /* Conversion succeeded, number ends at 
(*end) */
+ASN_STRTOL_EXTRA_DATA  =  1,  /* Conversion succeeded, but the string has 
extra stuff */
 };
 enum asn_strtol_result_e asn_strtol_lim(const char *str, const char **end, 
long *l);

diff --git a/include/osmocom/rspro/constr_TYPE.h 
b/include/osmocom/rspro/constr_TYPE.h
index 13c60f3..e244d20 100644
--- a/include/osmocom/rspro/constr_TYPE.h
+++ b/include/osmocom/rspro/constr_TYPE.h
@@ -158,8 +158,8 @@
 typedef struct asn_TYPE_tag2member_s {
ber_tlv_tag_t el_tag;   /* Outmost tag of the member */
int el_no;  /* Index of the associated member, base 0 */
-   int toff_first; /* First occurence of the el_tag, relative */
-   int toff_last;  /* Last occurence of the el_tag, relatvie */
+   int toff_first; /* First occurrence of the el_tag, relative */
+   int toff_last;  /* Last occurrence of the el_tag, relatvie */
 } asn_TYPE_tag2member_t;

 /*
diff --git a/include/osmocom/rspro/per_support.h 
b/include/osmocom/rspro/per_support.h
index 181fe24..ed945ce 100644
--- a/include/osmocom/rspro/per_support.h
+++ b/include/osmocom/rspro/per_support.h
@@ -53,7 +53,7 @@
  */
 int32_t per_get_few_bits(asn_per_data_t *per_data, int get_nbits);

-/* Undo the immediately preceeding "get_few_bits" operation */
+/* Undo the immediately preceding "get_few_bits" operation */
 void per_get_undo(asn_per_data_t *per_data, int get_nbits);

 /*
diff --git a/src/rspro/NativeInteger.c b/src/rspro/NativeInteger.c
index ef17bee..17a58ea 100644
--- a/src/rspro/NativeInteger.c
+++ b/src/rspro/NativeInteger.c
@@ -94,7 +94,7 @@
/*
 * ASN.1 encoded INTEGER: buf_ptr, length
 * Fill the native, at the same time checking for overflow.
-* If overflow occured, return with RC_FAIL.
+* If overflow occurred, return with RC_FAIL.
 */
{
INTEGER_t tmp;
diff --git a/src/rspro/OCTET_STRING.c b/src/rspro/OCTET_STRING.c
index 3e424e7..6df9a0a 100644
--- a/src/rspro/OCTET_STRING.c
+++ b/src/rspro/OCTET_STRING.c
@@ -109,7 +109,7 @@
 /*
  * The main reason why ASN.1 is still alive is that too much time and effort
  * is necessary for learning it more or less adequately, thus creating a gut
- * necessity to demonstrate that aquired skill everywhere afterwards.
+ * necessity to demonstrate that acquired skill everywhere afterwards.
  * No, I am not going to explain what the following stuff is.
  */
 struct _stack_el {
diff --git a/src/rspro/ber_tlv_tag.c b/src/rspro/ber_tlv_tag.c
index 4270876..adf3a25 100644
--- a/src/rspro/ber_tlv_tag.c
+++ b/src/rspro/ber_tlv_tag.c
@@ -42,7 +42,7 @@
 */
if(val >> ((8 * sizeof(val)) - 9)) {
/*
-* We would not be able to accomodate
+* We would not be able to accommodate
 * any more tag bits.
 */
 

Change in ...osmo-trx[master]: Fix common misspellings and typos

2019-07-17 Thread roox
roox has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmo-trx/+/14819


Change subject: Fix common misspellings and typos
..

Fix common misspellings and typos

Change-Id: I8e0fe9d67d1e1e8dfce319610de63cde170694c5
---
M CommonLibs/Interthread.h
M CommonLibs/Threads.h
M CommonLibs/trx_vty.c
M README
M Transceiver52M/Resampler.h
M Transceiver52M/Transceiver.h
M Transceiver52M/arch/x86/convert.c
M Transceiver52M/arch/x86/convolve.c
M Transceiver52M/device/common/smpl_buf.h
M Transceiver52M/inband-signaling-usb
M autogen.sh
M debian/changelog
M doc/manuals/vty/trx_vty_reference.xml
13 files changed, 17 insertions(+), 17 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-trx refs/changes/19/14819/1

diff --git a/CommonLibs/Interthread.h b/CommonLibs/Interthread.h
index 42e6f7f..abb11c6 100644
--- a/CommonLibs/Interthread.h
+++ b/CommonLibs/Interthread.h
@@ -45,7 +45,7 @@
 // (pat) The elements in the queue are type T*, and
 // the Fifo class implements the underlying queue.
 // The default is class PointerFIFO, which does not place any restrictions on 
the type of T,
-// and is implemented by allocating auxilliary structures for the queue,
+// and is implemented by allocating auxiliary structures for the queue,
 // or SingleLinkedList, which implements the queue using an internal pointer 
in type T,
 // which must implement the functional interface of class SingleLinkListNode,
 // namely: functions T*next() and void setNext(T*).
diff --git a/CommonLibs/Threads.h b/CommonLibs/Threads.h
index 8a6ba47..a914d51 100644
--- a/CommonLibs/Threads.h
+++ b/CommonLibs/Threads.h
@@ -182,7 +182,7 @@
}
}

-   /** Send cancelation to thread */
+   /** Send cancellation to thread */
void cancel() { pthread_cancel(mThread); }
 };

diff --git a/CommonLibs/trx_vty.c b/CommonLibs/trx_vty.c
index e184f49..3f1047e 100644
--- a/CommonLibs/trx_vty.c
+++ b/CommonLibs/trx_vty.c
@@ -222,7 +222,7 @@
 DEFUN(cfg_clock_ref, cfg_clock_ref_cmd,
"clock-ref (internal|external|gpsdo)",
"Set the Reference Clock\n"
-   "Enable internal referece (default)\n"
+   "Enable internal reference (default)\n"
"Enable external 10 MHz reference\n"
"Enable GPSDO reference\n")
 {
diff --git a/README b/README
index 502d661..0d8cdcd 100644
--- a/README
+++ b/README
@@ -22,7 +22,7 @@

 Commands on the Per-ARFCN Control Interface

-The per-ARFCN control interface uses a command-reponse protocol.
+The per-ARFCN control interface uses a command-response protocol.
 Commands are NULL-terminated ASCII strings, one per UDP socket.
 Each command has a corresponding response.
 Every command is of the form:
diff --git a/Transceiver52M/Resampler.h b/Transceiver52M/Resampler.h
index caffc08..14870fe 100644
--- a/Transceiver52M/Resampler.h
+++ b/Transceiver52M/Resampler.h
@@ -38,7 +38,7 @@
 *   @return false on error, zero otherwise
 *
 * Automatic setting is to compute the filter to prevent aliasing with
-* a Blackman-Harris window. Adjustment is made through a bandwith
+* a Blackman-Harris window. Adjustment is made through a bandwidth
 * factor to shift the cutoff and/or the constituent filter lengths.
 * Calculation of specific rolloff factors or 3-dB cutoff points is
 * left as an excersize for the reader.
diff --git a/Transceiver52M/Transceiver.h b/Transceiver52M/Transceiver.h
index a66c962..1d192d8 100644
--- a/Transceiver52M/Transceiver.h
+++ b/Transceiver52M/Transceiver.h
@@ -236,7 +236,7 @@
   bool start();
   void stop();

-  /** Protect destructor accessable stop call */
+  /** Protect destructor accessible stop call */
   Mutex mLock;

 protected:
diff --git a/Transceiver52M/arch/x86/convert.c 
b/Transceiver52M/arch/x86/convert.c
index 07cdf59..bbcfd67 100644
--- a/Transceiver52M/arch/x86/convert.c
+++ b/Transceiver52M/arch/x86/convert.c
@@ -27,7 +27,7 @@
 #include "config.h"
 #endif

-/* Architecture dependant function pointers */
+/* Architecture dependent function pointers */
 struct convert_cpu_context {
void (*convert_si16_ps_16n) (float *, const short *, int);
void (*convert_si16_ps) (float *, const short *, int);
diff --git a/Transceiver52M/arch/x86/convolve.c 
b/Transceiver52M/arch/x86/convolve.c
index 209d377..f25e2a7 100644
--- a/Transceiver52M/arch/x86/convolve.c
+++ b/Transceiver52M/arch/x86/convolve.c
@@ -27,7 +27,7 @@
 #include "config.h"
 #endif

-/* Architecture dependant function pointers */
+/* Architecture dependent function pointers */
 struct convolve_cpu_context {
void (*conv_cmplx_4n) (const float *, int, const float *, int, float *,
   int, int, int);
diff --git a/Transceiver52M/device/common/smpl_buf.h 
b/Transceiver52M/device/common/smpl_buf.h
index 24d3ce9..36e06b5 100644
--- a/Transceiver52M/device/common/smpl_buf.h
+++ 

Change in ...osmocom-bb[master]: Fix common misspellings and typos

2019-07-17 Thread roox
roox has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmocom-bb/+/14818


Change subject: Fix common misspellings and typos
..

Fix common misspellings and typos

Change-Id: I75df25d3bc2ecb4985bf6fefdde6e44b21bffed9
---
M src/host/layer23/README
M src/host/layer23/include/osmocom/bb/misc/layer3.h
M src/host/layer23/src/common/sap_proto.c
M src/host/layer23/src/common/sim.c
M src/host/layer23/src/misc/app_bcch_scan.c
M src/host/layer23/src/misc/app_cbch_sniff.c
M src/host/layer23/src/misc/cell_log.c
M src/host/layer23/src/mobile/gsm322.c
M src/host/layer23/src/mobile/gsm48_mm.c
M src/host/layer23/src/mobile/gsm48_rr.c
M src/host/layer23/src/mobile/main.c
M src/host/layer23/src/mobile/mnccms.c
M src/host/virt_phy/src/gsmtapl1_if.c
M src/shared/libosmocore/Doxyfile.codec.in
M src/shared/libosmocore/Doxyfile.core.in
M src/shared/libosmocore/Doxyfile.gsm.in
M src/shared/libosmocore/Doxyfile.vty.in
M src/shared/libosmocore/doc/vty/example.xml
M src/shared/libosmocore/include/osmocom/vty/command.h
M src/shared/libosmocore/include/osmocom/vty/vector.h
M src/shared/libosmocore/src/gb/gprs_ns.c
M src/shared/libosmocore/src/gsm/auth_core.c
M src/shared/libosmocore/src/gsm/gsm0411_smr.c
M src/shared/libosmocore/src/gsm/gsm48.c
M src/shared/libosmocore/src/gsm/lapd_core.c
M src/shared/libosmocore/src/gsm/lapdm.c
M src/shared/libosmocore/src/logging.c
M src/shared/libosmocore/src/talloc.c
M src/shared/libosmocore/src/utils.c
M src/shared/libosmocore/src/vty/command.c
M src/target/firmware/apps/rssi/main.c
M src/target/firmware/apps/simtest/main.c
M src/target/firmware/calypso/sim.c
M src/target/firmware/comm/sercomm.c
M src/target/firmware/fb/fb_s6b33b1x.c
M src/target/firmware/include/fb/fb_bw8.h
M src/target/firmware/layer1/prim_fbsb.c
M src/target/firmware/layer1/prim_tch.c
M src/target/firmware/layer1/sync.c
M src/target/firmware/rf/trf6151.c
M src/target/trx_toolkit/data_dump.py
41 files changed, 62 insertions(+), 62 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmocom-bb refs/changes/18/14818/1

diff --git a/src/host/layer23/README b/src/host/layer23/README
index dd59823..c52e7bc 100644
--- a/src/host/layer23/README
+++ b/src/host/layer23/README
@@ -30,7 +30,7 @@
 RSL header (struct abis_rsl_common_hdr).

 There are utility functions like rslms_tx_rll_req() and rslms_tx_rsll_req_l3()
-for creating msgb's with the apropriate RSL/RLL headers.
+for creating msgb's with the appropriate RSL/RLL headers.


 === LAPDm ===
diff --git a/src/host/layer23/include/osmocom/bb/misc/layer3.h 
b/src/host/layer23/include/osmocom/bb/misc/layer3.h
index bbf242d..ffafe76 100644
--- a/src/host/layer23/include/osmocom/bb/misc/layer3.h
+++ b/src/host/layer23/include/osmocom/bb/misc/layer3.h
@@ -11,7 +11,7 @@
 /* Initialize layer3 for the MS, hook it to L2 */
 int layer3_init(struct osmocom_ms *ms);

-/* Reset the 'aplication' state */
+/* Reset the 'application' state */
 void layer3_app_reset(void);

 #endif
diff --git a/src/host/layer23/src/common/sap_proto.c 
b/src/host/layer23/src/common/sap_proto.c
index c3d202f..ecacfe0 100644
--- a/src/host/layer23/src/common/sap_proto.c
+++ b/src/host/layer23/src/common/sap_proto.c
@@ -177,7 +177,7 @@
  * \param[in] sap_msg pointer to SAP message header
  * \param[in] param_type parameter type (see sap_param_type enum)
  * \param[out] param_len parameter length (if found)
- * \returns pointer to a given parameter withing the message, NULL otherwise
+ * \returns pointer to a given parameter within the message, NULL otherwise
  */
 struct sap_param *sap_get_param(const struct sap_message *sap_msg,
enum sap_param_type param_type, uint16_t *param_len)
diff --git a/src/host/layer23/src/common/sim.c 
b/src/host/layer23/src/common/sim.c
index ed7f54a..1e2bc51 100644
--- a/src/host/layer23/src/common/sim.c
+++ b/src/host/layer23/src/common/sim.c
@@ -1132,7 +1132,7 @@
case SIM_JOB_INCREASE:
if (length != 4) {
LOGP(DSIM, LOGL_ERROR, "expecting uint32_t as "
-   "value lenght, but got %d bytes\n",
+   "value length, but got %d bytes\n",
length);
goto request_error;
}
diff --git a/src/host/layer23/src/misc/app_bcch_scan.c 
b/src/host/layer23/src/misc/app_bcch_scan.c
index 22e6f1b..5818530 100644
--- a/src/host/layer23/src/misc/app_bcch_scan.c
+++ b/src/host/layer23/src/misc/app_bcch_scan.c
@@ -53,7 +53,7 @@

 int l23_app_init(struct osmocom_ms *ms)
 {
-   /* don't do layer3_init() as we don't want an actualy L3 */
+   /* don't do layer3_init() as we don't want an actually L3 */
fps_init();
l1ctl_tx_reset_req(ms, L1CTL_RES_T_FULL);
return osmo_signal_register_handler(SS_L1CTL, _cb, NULL);
diff --git 

Build failure of network:osmocom:nightly/osmo-sgsn in Raspbian_9.0/armv7l

2019-07-17 Thread OBS Notification
Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/osmo-sgsn/Raspbian_9.0/armv7l

Package network:osmocom:nightly/osmo-sgsn failed to build in Raspbian_9.0/armv7l

Check out the package for editing:
  osc checkout network:osmocom:nightly osmo-sgsn

Last lines of build log:
[  501s] bss_ptmsi = bss_ptmsi | GSM23003_TMSI_SGSN_MASK;
[  501s] ^~~
[  501s] gb_proxy.c:246:28: note: each undeclared identifier is reported only 
once for each function it appears in
[  501s] Makefile:615: recipe for target 'gb_proxy.o' failed
[  501s] make[4]: *** [gb_proxy.o] Error 1
[  501s] make[4]: Leaving directory '/usr/src/packages/BUILD/src/gprs'
[  501s] Makefile:391: recipe for target 'all-recursive' failed
[  501s] make[3]: *** [all-recursive] Error 1
[  501s] make[3]: Leaving directory '/usr/src/packages/BUILD/src'
[  501s] Makefile:453: recipe for target 'all-recursive' failed
[  501s] make[2]: *** [all-recursive] Error 1
[  501s] make[2]: Leaving directory '/usr/src/packages/BUILD'
[  501s] Makefile:385: recipe for target 'all' failed
[  501s] make[1]: *** [all] Error 2
[  501s] make[1]: Leaving directory '/usr/src/packages/BUILD'
[  501s] dh_auto_build: make -j1 returned exit code 2
[  501s] debian/rules:45: recipe for target 'build' failed
[  501s] make: *** [build] Error 2
[  501s] dpkg-buildpackage: error: debian/rules build gave error exit status 2
[  501s] 
[  501s] armbuild25 failed "build osmo-sgsn_1.4.0.28.43f16.dsc" at Wed Jul 17 
18:07:27 UTC 2019.
[  501s] 
[  501s] ### VM INTERACTION START ###
[  504s] [  458.751946] sysrq: SysRq : Power Off
[  504s] [  458.778271] reboot: Power down
[  504s] ### VM INTERACTION END ###
[  504s] 
[  504s] armbuild25 failed "build osmo-sgsn_1.4.0.28.43f16.dsc" at Wed Jul 17 
18:07:30 UTC 2019.
[  504s] 

-- 
Configure notifications at https://build.opensuse.org/user/notifications
openSUSE Build Service (https://build.opensuse.org/)


Build failure of network:osmocom:nightly/osmo-msc in Raspbian_9.0/armv7l

2019-07-17 Thread OBS Notification
Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/osmo-msc/Raspbian_9.0/armv7l

Package network:osmocom:nightly/osmo-msc failed to build in Raspbian_9.0/armv7l

Check out the package for editing:
  osc checkout network:osmocom:nightly osmo-msc

Last lines of build log:
[  669s] tmsi |= GSM23003_TMSI_SGSN_MASK;
[  669s] ^~~
[  669s] vlr.c:339:12: note: each undeclared identifier is reported only once 
for each function it appears in
[  669s] Makefile:442: recipe for target 'vlr.o' failed
[  669s] make[4]: *** [vlr.o] Error 1
[  669s] make[4]: Leaving directory '/usr/src/packages/BUILD/src/libvlr'
[  669s] Makefile:422: recipe for target 'all-recursive' failed
[  669s] make[3]: *** [all-recursive] Error 1
[  669s] make[3]: Leaving directory '/usr/src/packages/BUILD/src'
[  669s] Makefile:462: recipe for target 'all-recursive' failed
[  669s] make[2]: *** [all-recursive] Error 1
[  669s] make[2]: Leaving directory '/usr/src/packages/BUILD'
[  669s] Makefile:394: recipe for target 'all' failed
[  669s] make[1]: *** [all] Error 2
[  669s] make[1]: Leaving directory '/usr/src/packages/BUILD'
[  669s] dh_auto_build: make -j1 returned exit code 2
[  669s] debian/rules:45: recipe for target 'build' failed
[  669s] make: *** [build] Error 2
[  669s] dpkg-buildpackage: error: debian/rules build gave error exit status 2
[  669s] 
[  669s] armbuild16 failed "build osmo-msc_1.3.1.178.aa0a.dsc" at Wed Jul 17 
17:43:15 UTC 2019.
[  669s] 
[  669s] ### VM INTERACTION START ###
[  672s] [  617.737477] sysrq: SysRq : Power Off
[  672s] [  617.739848] reboot: Power down
[  672s] ### VM INTERACTION END ###
[  672s] 
[  672s] armbuild16 failed "build osmo-msc_1.3.1.178.aa0a.dsc" at Wed Jul 17 
17:43:19 UTC 2019.
[  672s] 

-- 
Configure notifications at https://build.opensuse.org/user/notifications
openSUSE Build Service (https://build.opensuse.org/)


Build failure of network:osmocom:nightly/osmo-python-tests in Debian_Unstable/x86_64

2019-07-17 Thread OBS Notification
Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/osmo-python-tests/Debian_Unstable/x86_64

Package network:osmocom:nightly/osmo-python-tests failed to build in 
Debian_Unstable/x86_64

Check out the package for editing:
  osc checkout network:osmocom:nightly osmo-python-tests

Last lines of build log:
[  209s] rm -rf /usr/src/packages/BUILD/debian/python3-osmopy-utils/usr/lib
[  209s] make[1]: Leaving directory '/usr/src/packages/BUILD'
[  209s]dh_installdocs
[  209s]dh_installchangelogs
[  209s]dh_python2
[  209s] I: dh_python2 tools:114: replacing shebang in 
debian/python2-osmopy-utils/usr/bin/osmotestvty.py
[  209s] I: dh_python2 tools:114: replacing shebang in 
debian/python2-osmopy-utils/usr/bin/osmotestconfig.py
[  209s] I: dh_python2 tools:114: replacing shebang in 
debian/python2-osmopy-utils/usr/bin/osmodumpdoc.py
[  209s]dh_python3
[  209s]debian/rules override_dh_installinit
[  209s] make[1]: Entering directory '/usr/src/packages/BUILD'
[  209s] # Install service file with different name than package name:
[  209s] # 
https://unix.stackexchange.com/questions/306234/is-it-possible-to-install-two-services-for-one-package-using-dh-installinit-how
[  209s] dh_installinit --name=osmo-ctrl2cgi
[  209s] dh_installinit: --name=osmo-ctrl2cgi option specified, but init script 
nor default file not found
[  209s] make[1]: *** [debian/rules:19: override_dh_installinit] Error 255
[  209s] make[1]: Leaving directory '/usr/src/packages/BUILD'
[  209s] make: *** [debian/rules:4: binary] Error 2
[  209s] dpkg-buildpackage: error: fakeroot debian/rules binary subprocess 
returned exit status 2
[  209s] 
[  209s] lamb56 failed "build osmo-python-tests_0.0.9.20190717.dsc" at Wed Jul 
17 14:54:33 UTC 2019.
[  209s] 
[  209s] ### VM INTERACTION START ###
[  213s] [  194.914009] sysrq: SysRq : Power Off
[  213s] [  194.919603] reboot: Power down
[  213s] ### VM INTERACTION END ###
[  213s] 
[  213s] lamb56 failed "build osmo-python-tests_0.0.9.20190717.dsc" at Wed Jul 
17 14:54:38 UTC 2019.
[  213s] 

-- 
Configure notifications at https://build.opensuse.org/user/notifications
openSUSE Build Service (https://build.opensuse.org/)


Change in ...osmo-bts[master]: common/rsl.c: fix: properly handle SI3 Rest Octets

2019-07-17 Thread fixeria
Hello laforge, laforge, Jenkins Builder,

I'd like you to reexamine a change. Please visit

https://gerrit.osmocom.org/c/osmo-bts/+/14817

to look at the new patch set (#3).

Change subject: common/rsl.c: fix: properly handle SI3 Rest Octets
..

common/rsl.c: fix: properly handle SI3 Rest Octets

It was noticed with old Sony Ericsson phones (like W595 and K510i)
that the service provided by Osmocom network becomes unreliable
from time to time. The RSSI indicator on those phones shows that
the signal is lost, so neither CS nor PS services are working.

As it then turned out, System Information 3 broadcasted on the
Um interface is different than the one received from the BSC.
In particular, the content of SI3 Rest Octets IE is different.

Among with the 'GPRS Indicator', which is actually expected to
indicate whether the PCU is connected or not, SI3 Rest Octets
on the Um interface contain both 'Optional Power Offset' and
'Scheduling if and where' IEs, which are not present in the
original messages from the BSC.

Moreover, as soon as the PCU is connected, 'GPRS Indicator' IE
contains different 'GPRS RA Colour' value, and informs the MS
that System Information 13 is sent on extended BCCH, which
is not even supported by OsmoBTS!

The culprit is in rsl_rx_bcch_info(), where we pass a pointer
to osmo_gsm48_rest_octets_si3_decode(). Instead of passing a
pointer to the beginning of SI3 buffer, we actually need to
shift it to the beginning of the SI3 Rest Octets IE.

This change makes my Sony Ericsson phones happy ;)

Change-Id: Ia962cf21903ba674057cf52746996dd3254bc1c6
---
M src/common/rsl.c
1 file changed, 3 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.osmocom.org:29418/osmo-bts refs/changes/17/14817/3
--
To view, visit https://gerrit.osmocom.org/c/osmo-bts/+/14817
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: Ia962cf21903ba674057cf52746996dd3254bc1c6
Gerrit-Change-Number: 14817
Gerrit-PatchSet: 3
Gerrit-Owner: fixeria 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria 
Gerrit-Reviewer: laforge 
Gerrit-CC: tnt 
Gerrit-MessageType: newpatchset


Change in ...osmo-iuh[master]: osmo-hnbgw.cfg: set local-ip to 0.0.0.0

2019-07-17 Thread fixeria
fixeria has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-iuh/+/14782 )

Change subject: osmo-hnbgw.cfg: set local-ip to 0.0.0.0
..


Patch Set 2: Code-Review+1


--
To view, visit https://gerrit.osmocom.org/c/osmo-iuh/+/14782
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-iuh
Gerrit-Branch: master
Gerrit-Change-Id: I354140f014854f1755b649e40a65e5d88b99c0ec
Gerrit-Change-Number: 14782
Gerrit-PatchSet: 2
Gerrit-Owner: osmith 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria 
Gerrit-Reviewer: osmith 
Gerrit-CC: pespin 
Gerrit-Comment-Date: Wed, 17 Jul 2019 12:21:02 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in ...osmo-bts[master]: common/rsl.c: fix: properly handle SI3 Rest Octets

2019-07-17 Thread fixeria
Hello laforge, laforge, Jenkins Builder,

I'd like you to reexamine a change. Please visit

https://gerrit.osmocom.org/c/osmo-bts/+/14817

to look at the new patch set (#2).

Change subject: common/rsl.c: fix: properly handle SI3 Rest Octets
..

common/rsl.c: fix: properly handle SI3 Rest Octets

It was noticed with old Sony Ericsson phones (like W595 and K510i)
that the service provided by Osmocom network becomes unreliable
from time to time. The RSSI indicator on those phones shows that
the signal is lost, so neither CS nor PS services are working.

As it then turned out, System Information 3 broadcasted on the
Um interface is different than the one received from the BSC.
In particular, the content SI3 Rest Octets IE is different.

Among with the 'GPRS Indicator', which is actually expected to
indicate whether the PCU is connected or now, SI3 Rest Octets
on the Um interface contain both 'Optional Power Offset' and
'Scheduling if and where' IEs, which are not present in the
original messages from the BSC.

Moreover, as soon as the PCU is connected, 'GPRS Indicator' IE
contains different 'GPRS RA Colour' value, and informs the MS
that System Information 13 is sent on extended BCCH, which
is not even supported by OsmoBTS!

The culprit is in rsl_rx_bcch_info(), where we pass a pointer
to osmo_gsm48_rest_octets_si3_decode(). Instead of passing a
pointer to the beginning of SI3 buffer, we actually need to
shift it to the beginning of the SI3 Rest Octets IE.

This change makes my Sony Ericsson phones happy ;)

Change-Id: Ia962cf21903ba674057cf52746996dd3254bc1c6
---
M src/common/rsl.c
1 file changed, 3 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.osmocom.org:29418/osmo-bts refs/changes/17/14817/2
--
To view, visit https://gerrit.osmocom.org/c/osmo-bts/+/14817
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: Ia962cf21903ba674057cf52746996dd3254bc1c6
Gerrit-Change-Number: 14817
Gerrit-PatchSet: 2
Gerrit-Owner: fixeria 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria 
Gerrit-Reviewer: laforge 
Gerrit-CC: tnt 
Gerrit-MessageType: newpatchset


Change in ...osmo-iuh[master]: osmo-hnbgw.cfg: set local-ip to 0.0.0.0

2019-07-17 Thread osmith
osmith has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-iuh/+/14782 )

Change subject: osmo-hnbgw.cfg: set local-ip to 0.0.0.0
..


Patch Set 2:

(1 comment)

https://gerrit.osmocom.org/#/c/14782/1/doc/examples/osmo-hnbgw.cfg
File doc/examples/osmo-hnbgw.cfg:

https://gerrit.osmocom.org/#/c/14782/1/doc/examples/osmo-hnbgw.cfg@24
PS1, Line 24:   local-ip 127.0.0.1
> ACK. How about 0.0.0. […]
Fine with me, changed to 0.0.0.0 (and checked that it works).



--
To view, visit https://gerrit.osmocom.org/c/osmo-iuh/+/14782
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-iuh
Gerrit-Branch: master
Gerrit-Change-Id: I354140f014854f1755b649e40a65e5d88b99c0ec
Gerrit-Change-Number: 14782
Gerrit-PatchSet: 2
Gerrit-Owner: osmith 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria 
Gerrit-Reviewer: osmith 
Gerrit-CC: pespin 
Gerrit-Comment-Date: Wed, 17 Jul 2019 12:06:27 +
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: fixeria 
Comment-In-Reply-To: pespin 
Gerrit-MessageType: comment


Change in ...osmo-iuh[master]: osmo-hnbgw.cfg: set local-ip to 0.0.0.0

2019-07-17 Thread osmith
Hello fixeria, Jenkins Builder,

I'd like you to reexamine a change. Please visit

https://gerrit.osmocom.org/c/osmo-iuh/+/14782

to look at the new patch set (#2).

Change subject: osmo-hnbgw.cfg: set local-ip to 0.0.0.0
..

osmo-hnbgw.cfg: set local-ip to 0.0.0.0

Make the config work out of the box for anyone.

Related: OS#3369
Change-Id: I354140f014854f1755b649e40a65e5d88b99c0ec
---
M doc/examples/osmo-hnbgw.cfg
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.osmocom.org:29418/osmo-iuh refs/changes/82/14782/2
--
To view, visit https://gerrit.osmocom.org/c/osmo-iuh/+/14782
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-iuh
Gerrit-Branch: master
Gerrit-Change-Id: I354140f014854f1755b649e40a65e5d88b99c0ec
Gerrit-Change-Number: 14782
Gerrit-PatchSet: 2
Gerrit-Owner: osmith 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria 
Gerrit-CC: pespin 
Gerrit-MessageType: newpatchset


Change in ...osmo-bts[master]: common/rsl.c: fix: properly handle SI3 Rest Octets

2019-07-17 Thread laforge
laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-bts/+/14817 )

Change subject: common/rsl.c: fix: properly handle SI3 Rest Octets
..


Patch Set 1: Code-Review+1

rsl.c:371:32: error: initialization from incompatible pointer type 
[-Werror=incompatible-pointer-types]
const uint8_t *si3_ro_buf = bts->si_buf[osmo_si];


--
To view, visit https://gerrit.osmocom.org/c/osmo-bts/+/14817
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: Ia962cf21903ba674057cf52746996dd3254bc1c6
Gerrit-Change-Number: 14817
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-CC: tnt 
Gerrit-Comment-Date: Wed, 17 Jul 2019 12:01:29 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in ...osmo-bts[master]: common/rsl.c: fix: properly handle SI3 Rest Octets

2019-07-17 Thread fixeria
fixeria has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmo-bts/+/14817


Change subject: common/rsl.c: fix: properly handle SI3 Rest Octets
..

common/rsl.c: fix: properly handle SI3 Rest Octets

It was noticed with old Sony Ericsson phones (like W595 and K510i)
that the service provided by Osmocom network becomes unreliable
from time to time. The RSSI indicator on those phones shows that
the signal is lost, so neither CS nor PS services are working.

As it then turned out, System Information 3 broadcasted on the
Um interface is different than the one received from the BSC.
In particular, the content SI3 Rest Octets IE is different.

Among with the 'GPRS Indicator', which is actually expected to
indicate whether the PCU is connected or now, SI3 Rest Octets
on the Um interface contain both 'Optional Power Offset' and
'Scheduling if and where' IEs, which are not present in the
original messages from the BSC.

Moreover, as soon as the PCU is connected, 'GPRS Indicator' IE
contains different 'GPRS RA Colour' value, and informs the MS
that System Information 13 is sent on extended BCCH, which
is not even supported by OsmoBTS!

The culprit is in rsl_rx_bcch_info(), where we pass a pointer
to osmo_gsm48_rest_octets_si3_decode(). Instead of passing a
pointer to the beginning of SI3 buffer, we actually need to
shift it to the beginning of the SI3 Rest Octets IE.

This change makes my Sony Ericsson phones happy ;)

Change-Id: Ia962cf21903ba674057cf52746996dd3254bc1c6
---
M src/common/rsl.c
1 file changed, 3 insertions(+), 1 deletion(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-bts refs/changes/17/14817/1

diff --git a/src/common/rsl.c b/src/common/rsl.c
index 0a613bb..d66fd4e 100644
--- a/src/common/rsl.c
+++ b/src/common/rsl.c
@@ -368,7 +368,9 @@
LCHAN_REL_ACT_REACT;
}
/* decode original SI3 Rest Octets as sent by BSC */
-   osmo_gsm48_rest_octets_si3_decode(>si3_ro_decoded, 
GSM_BTS_SI(bts, osmo_si));
+   const uint8_t *si3_ro_buf = bts->si_buf[osmo_si];
+   si3_ro_buf += offsetof(struct 
gsm48_system_information_type_3, rest_octets);
+   osmo_gsm48_rest_octets_si3_decode(>si3_ro_decoded, 
si3_ro_buf);
/* patch out GPRS indicator from binary if PCU is not 
connected; will be enabled
 * after PCU connects */
regenerate_si3_restoctets(bts);

--
To view, visit https://gerrit.osmocom.org/c/osmo-bts/+/14817
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: Ia962cf21903ba674057cf52746996dd3254bc1c6
Gerrit-Change-Number: 14817
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria 
Gerrit-MessageType: newchange


Change in ...docker-playground[master]: debian-repo-install-test: kill already running

2019-07-17 Thread osmith
Hello pespin, laforge,

I'd like you to reexamine a change. Please visit

https://gerrit.osmocom.org/c/docker-playground/+/14780

to look at the new patch set (#4).

Change subject: debian-repo-install-test: kill already running
..

debian-repo-install-test: kill already running

The container grows heavily in size as the test runs, so make sure to
always kill existing ones (from stopped jobs) before starting a new
one. In order to do that, do not use $BUILD_TAG as container name,
which changes with every new jenkins run.

Related: OS#3369
Change-Id: Ide795092b656c9f0eb92a075d8f662944089019f
---
M debian-repo-install-test/jenkins.sh
1 file changed, 13 insertions(+), 6 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/docker-playground 
refs/changes/80/14780/4
--
To view, visit https://gerrit.osmocom.org/c/docker-playground/+/14780
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: docker-playground
Gerrit-Branch: master
Gerrit-Change-Id: Ide795092b656c9f0eb92a075d8f662944089019f
Gerrit-Change-Number: 14780
Gerrit-PatchSet: 4
Gerrit-Owner: osmith 
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: osmith 
Gerrit-Reviewer: pespin 
Gerrit-MessageType: newpatchset


Change in ...docker-playground[master]: debian-repo-install-test: add docker container

2019-07-17 Thread osmith
osmith has posted comments on this change. ( 
https://gerrit.osmocom.org/c/docker-playground/+/14779 )

Change subject: debian-repo-install-test: add docker container
..


Patch Set 3:

(1 comment)

https://gerrit.osmocom.org/#/c/14779/2/debian-repo-install-test/jenkins.sh
File debian-repo-install-test/jenkins.sh:

https://gerrit.osmocom.org/#/c/14779/2/debian-repo-install-test/jenkins.sh@24
PS2, Line 24: if ! docker exec "$BUILD_TAG" systemctl status; then
> In order to avoid race conditions on busy/slow systems, it would make more 
> sense to have a 10 time l […]
Done



--
To view, visit https://gerrit.osmocom.org/c/docker-playground/+/14779
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: docker-playground
Gerrit-Branch: master
Gerrit-Change-Id: I0348f65a2ac184ba6b001e5130dfc8124e657367
Gerrit-Change-Number: 14779
Gerrit-PatchSet: 3
Gerrit-Owner: osmith 
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: osmith 
Gerrit-CC: pespin 
Gerrit-Comment-Date: Wed, 17 Jul 2019 11:43:51 +
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: pespin 
Gerrit-MessageType: comment


Change in ...docker-playground[master]: debian-repo-install-test: add docker container

2019-07-17 Thread osmith
Hello laforge,

I'd like you to reexamine a change. Please visit

https://gerrit.osmocom.org/c/docker-playground/+/14779

to look at the new patch set (#3).

Change subject: debian-repo-install-test: add docker container
..

debian-repo-install-test: add docker container

Add own container with systemd, so we can (in a follow-up commit) run
the Osmocom systemd services in this test job. Rewrite the
"interactive shell" logic to support the new systemd docker container,
and enable it with an INTERACTIVE environment variable instead of
hardcoding 'interactive="true"' in the script. While at it, move the
Repository.key install to the Dockerfile so it works more like the other
docker containers we have.

Related: OS#3369
Change-Id: I0348f65a2ac184ba6b001e5130dfc8124e657367
---
A debian-repo-install-test/Dockerfile
A debian-repo-install-test/Makefile
R debian-repo-install-test/Release.key
M debian-repo-install-test/jenkins.sh
M debian-repo-install-test/testdata/repo-install-test.sh
5 files changed, 50 insertions(+), 28 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/docker-playground 
refs/changes/79/14779/3
--
To view, visit https://gerrit.osmocom.org/c/docker-playground/+/14779
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: docker-playground
Gerrit-Branch: master
Gerrit-Change-Id: I0348f65a2ac184ba6b001e5130dfc8124e657367
Gerrit-Change-Number: 14779
Gerrit-PatchSet: 3
Gerrit-Owner: osmith 
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: osmith 
Gerrit-CC: pespin 
Gerrit-MessageType: newpatchset


Change in ...osmocom-bb[master]: target/*.py: shebang: use /usr/bin/env python

2019-07-17 Thread lynxis lazus
lynxis lazus has submitted this change and it was merged. ( 
https://gerrit.osmocom.org/c/osmocom-bb/+/14759 )

Change subject: target/*.py: shebang: use /usr/bin/env python
..

target/*.py: shebang: use /usr/bin/env python

Use the system default python instead of a hardcoded python2
Allow to use python2 and python3.

Change-Id: Iab185759b574eff1ca1b189dcbb4e1a3eec52132
---
M src/target/firmware/solve_envs.py
M src/target/trx_toolkit/app_common.py
M src/target/trx_toolkit/burst_fwd.py
M src/target/trx_toolkit/burst_gen.py
M src/target/trx_toolkit/burst_send.py
M src/target/trx_toolkit/clck_gen.py
M src/target/trx_toolkit/ctrl_cmd.py
M src/target/trx_toolkit/ctrl_if.py
M src/target/trx_toolkit/ctrl_if_trx.py
M src/target/trx_toolkit/data_dump.py
M src/target/trx_toolkit/data_if.py
M src/target/trx_toolkit/data_msg.py
M src/target/trx_toolkit/fake_pm.py
M src/target/trx_toolkit/fake_trx.py
M src/target/trx_toolkit/gsm_shared.py
M src/target/trx_toolkit/rand_burst_gen.py
M src/target/trx_toolkit/transceiver.py
M src/target/trx_toolkit/trx_list.py
M src/target/trx_toolkit/trx_sniff.py
M src/target/trx_toolkit/udp_link.py
20 files changed, 20 insertions(+), 20 deletions(-)

Approvals:
  Jenkins Builder: Verified
  fixeria: Looks good to me, approved
  laforge: Looks good to me, but someone else must approve



diff --git a/src/target/firmware/solve_envs.py 
b/src/target/firmware/solve_envs.py
index d4cee80..b010897 100755
--- a/src/target/firmware/solve_envs.py
+++ b/src/target/firmware/solve_envs.py
@@ -1,4 +1,4 @@
-#!/usr/bin/env python2
+#!/usr/bin/env python

 import sys

diff --git a/src/target/trx_toolkit/app_common.py 
b/src/target/trx_toolkit/app_common.py
index 9bcd593..f2a47c5 100644
--- a/src/target/trx_toolkit/app_common.py
+++ b/src/target/trx_toolkit/app_common.py
@@ -1,4 +1,4 @@
-#!/usr/bin/env python2
+#!/usr/bin/env python
 # -*- coding: utf-8 -*-

 # TRX Toolkit
diff --git a/src/target/trx_toolkit/burst_fwd.py 
b/src/target/trx_toolkit/burst_fwd.py
index 1d5dd7b..b418aef 100644
--- a/src/target/trx_toolkit/burst_fwd.py
+++ b/src/target/trx_toolkit/burst_fwd.py
@@ -1,4 +1,4 @@
-#!/usr/bin/env python2
+#!/usr/bin/env python
 # -*- coding: utf-8 -*-

 # TRX Toolkit
diff --git a/src/target/trx_toolkit/burst_gen.py 
b/src/target/trx_toolkit/burst_gen.py
index b62f48c..6db8219 100755
--- a/src/target/trx_toolkit/burst_gen.py
+++ b/src/target/trx_toolkit/burst_gen.py
@@ -1,4 +1,4 @@
-#!/usr/bin/env python2
+#!/usr/bin/env python
 # -*- coding: utf-8 -*-

 # TRX Toolkit
diff --git a/src/target/trx_toolkit/burst_send.py 
b/src/target/trx_toolkit/burst_send.py
index 6c19e6f..2565f4a 100755
--- a/src/target/trx_toolkit/burst_send.py
+++ b/src/target/trx_toolkit/burst_send.py
@@ -1,4 +1,4 @@
-#!/usr/bin/env python2
+#!/usr/bin/env python
 # -*- coding: utf-8 -*-

 # TRX Toolkit
diff --git a/src/target/trx_toolkit/clck_gen.py 
b/src/target/trx_toolkit/clck_gen.py
index a4b83b5..3e479e3 100755
--- a/src/target/trx_toolkit/clck_gen.py
+++ b/src/target/trx_toolkit/clck_gen.py
@@ -1,4 +1,4 @@
-#!/usr/bin/env python2
+#!/usr/bin/env python
 # -*- coding: utf-8 -*-

 # TRX Toolkit
diff --git a/src/target/trx_toolkit/ctrl_cmd.py 
b/src/target/trx_toolkit/ctrl_cmd.py
index bdacb64..7deeeac 100755
--- a/src/target/trx_toolkit/ctrl_cmd.py
+++ b/src/target/trx_toolkit/ctrl_cmd.py
@@ -1,4 +1,4 @@
-#!/usr/bin/env python2
+#!/usr/bin/env python
 # -*- coding: utf-8 -*-

 # TRX Toolkit
diff --git a/src/target/trx_toolkit/ctrl_if.py 
b/src/target/trx_toolkit/ctrl_if.py
index 902af18..89dfe46 100644
--- a/src/target/trx_toolkit/ctrl_if.py
+++ b/src/target/trx_toolkit/ctrl_if.py
@@ -1,4 +1,4 @@
-#!/usr/bin/env python2
+#!/usr/bin/env python
 # -*- coding: utf-8 -*-

 # TRX Toolkit
diff --git a/src/target/trx_toolkit/ctrl_if_trx.py 
b/src/target/trx_toolkit/ctrl_if_trx.py
index 8cb5c1a..e44f8a9 100644
--- a/src/target/trx_toolkit/ctrl_if_trx.py
+++ b/src/target/trx_toolkit/ctrl_if_trx.py
@@ -1,4 +1,4 @@
-#!/usr/bin/env python2
+#!/usr/bin/env python
 # -*- coding: utf-8 -*-

 # TRX Toolkit
diff --git a/src/target/trx_toolkit/data_dump.py 
b/src/target/trx_toolkit/data_dump.py
index 71e1261..97518bd 100644
--- a/src/target/trx_toolkit/data_dump.py
+++ b/src/target/trx_toolkit/data_dump.py
@@ -1,4 +1,4 @@
-#!/usr/bin/env python2
+#!/usr/bin/env python
 # -*- coding: utf-8 -*-

 # TRX Toolkit
diff --git a/src/target/trx_toolkit/data_if.py 
b/src/target/trx_toolkit/data_if.py
index 11df756..e43b388 100644
--- a/src/target/trx_toolkit/data_if.py
+++ b/src/target/trx_toolkit/data_if.py
@@ -1,4 +1,4 @@
-#!/usr/bin/env python2
+#!/usr/bin/env python
 # -*- coding: utf-8 -*-

 # TRX Toolkit
diff --git a/src/target/trx_toolkit/data_msg.py 
b/src/target/trx_toolkit/data_msg.py
index e946229..ebb7a0f 100644
--- a/src/target/trx_toolkit/data_msg.py
+++ b/src/target/trx_toolkit/data_msg.py
@@ -1,4 +1,4 @@
-#!/usr/bin/env python2
+#!/usr/bin/env python
 # -*- coding: 

Build failure of network:osmocom:nightly/osmo-bts in Raspbian_9.0/armv7l

2019-07-17 Thread OBS Notification
Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/osmo-bts/Raspbian_9.0/armv7l

Package network:osmocom:nightly/osmo-bts failed to build in Raspbian_9.0/armv7l

Check out the package for editing:
  osc checkout network:osmocom:nightly osmo-bts

Last lines of build log:
[  542s] ../../src/common/libbts.a(bts.o): In function `lchan_init_lapdm':
[  542s] ./src/common/bts.c:424: undefined reference to `lapdm_channel_init2'
[  542s] collect2: error: ld returned 1 exit status
[  542s] Makefile:421: recipe for target 'osmo-bts-virtual' failed
[  542s] make[4]: *** [osmo-bts-virtual] Error 1
[  542s] make[4]: Leaving directory 
'/usr/src/packages/BUILD/src/osmo-bts-virtual'
[  542s] Makefile:385: recipe for target 'all-recursive' failed
[  542s] make[3]: *** [all-recursive] Error 1
[  542s] make[3]: Leaving directory '/usr/src/packages/BUILD/src'
[  542s] Makefile:434: recipe for target 'all-recursive' failed
[  542s] make[2]: *** [all-recursive] Error 1
[  542s] make[2]: Leaving directory '/usr/src/packages/BUILD'
[  542s] Makefile:366: recipe for target 'all' failed
[  542s] make[1]: *** [all] Error 2
[  542s] make[1]: Leaving directory '/usr/src/packages/BUILD'
[  542s] dh_auto_build: make -j1 returned exit code 2
[  542s] debian/rules:12: recipe for target 'build' failed
[  542s] make: *** [build] Error 2
[  542s] dpkg-buildpackage: error: debian/rules build gave error exit status 2
[  542s] 
[  542s] armbuild17 failed "build osmo-bts_0.8.1.344.af6f.dsc" at Wed Jul 17 
11:07:49 UTC 2019.
[  542s] 
[  542s] ### VM INTERACTION START ###
[  545s] [  492.243361] sysrq: SysRq : Power Off
[  545s] [  492.246871] reboot: Power down
[  546s] ### VM INTERACTION END ###
[  546s] 
[  546s] armbuild17 failed "build osmo-bts_0.8.1.344.af6f.dsc" at Wed Jul 17 
11:07:53 UTC 2019.
[  546s] 

-- 
Configure notifications at https://build.opensuse.org/user/notifications
openSUSE Build Service (https://build.opensuse.org/)


Change in ...osmo-hlr[master]: fix spelling errors detected by lintian

2019-07-17 Thread laforge
laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-hlr/+/14806 )

Change subject: fix spelling errors detected by lintian
..


Patch Set 2: Code-Review+2


--
To view, visit https://gerrit.osmocom.org/c/osmo-hlr/+/14806
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-hlr
Gerrit-Branch: master
Gerrit-Change-Id: I2a1a21aceabc20fadc5dd28985a94689d2b873a2
Gerrit-Change-Number: 14806
Gerrit-PatchSet: 2
Gerrit-Owner: Hoernchen 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Wed, 17 Jul 2019 11:03:35 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in ...libosmocore[master]: Set HTML_TIMESTAMP to NO to make build reproducible

2019-07-17 Thread laforge
laforge has submitted this change and it was merged. ( 
https://gerrit.osmocom.org/c/libosmocore/+/14795 )

Change subject: Set HTML_TIMESTAMP to NO to make build reproducible
..

Set HTML_TIMESTAMP to NO to make build reproducible

Change-Id: I56cce5cda0621f5e9f1c28aaad4646e9e267d17d
---
M Doxyfile.codec.in
M Doxyfile.core.in
M Doxyfile.gsm.in
M Doxyfile.vty.in
4 files changed, 4 insertions(+), 4 deletions(-)

Approvals:
  laforge: Looks good to me, approved
  Jenkins Builder: Verified



diff --git a/Doxyfile.codec.in b/Doxyfile.codec.in
index 307e3e1..88f0d27 100644
--- a/Doxyfile.codec.in
+++ b/Doxyfile.codec.in
@@ -891,7 +891,7 @@
 # page will contain the date and time when the page was generated. Setting
 # this to NO can help when comparing the output of multiple runs.

-HTML_TIMESTAMP = YES
+HTML_TIMESTAMP = NO

 # If the HTML_ALIGN_MEMBERS tag is set to YES, the members of classes,
 # files or namespaces will be aligned in HTML using tables. If set to
diff --git a/Doxyfile.core.in b/Doxyfile.core.in
index f83205a..c2bfcd5 100644
--- a/Doxyfile.core.in
+++ b/Doxyfile.core.in
@@ -891,7 +891,7 @@
 # page will contain the date and time when the page was generated. Setting
 # this to NO can help when comparing the output of multiple runs.

-HTML_TIMESTAMP = YES
+HTML_TIMESTAMP = NO

 # If the HTML_ALIGN_MEMBERS tag is set to YES, the members of classes,
 # files or namespaces will be aligned in HTML using tables. If set to
diff --git a/Doxyfile.gsm.in b/Doxyfile.gsm.in
index aae0e89..fd89390 100644
--- a/Doxyfile.gsm.in
+++ b/Doxyfile.gsm.in
@@ -891,7 +891,7 @@
 # page will contain the date and time when the page was generated. Setting
 # this to NO can help when comparing the output of multiple runs.

-HTML_TIMESTAMP = YES
+HTML_TIMESTAMP = NO

 # If the HTML_ALIGN_MEMBERS tag is set to YES, the members of classes,
 # files or namespaces will be aligned in HTML using tables. If set to
diff --git a/Doxyfile.vty.in b/Doxyfile.vty.in
index fb1cabf..9f56fd7 100644
--- a/Doxyfile.vty.in
+++ b/Doxyfile.vty.in
@@ -891,7 +891,7 @@
 # page will contain the date and time when the page was generated. Setting
 # this to NO can help when comparing the output of multiple runs.

-HTML_TIMESTAMP = YES
+HTML_TIMESTAMP = NO

 # If the HTML_ALIGN_MEMBERS tag is set to YES, the members of classes,
 # files or namespaces will be aligned in HTML using tables. If set to

--
To view, visit https://gerrit.osmocom.org/c/libosmocore/+/14795
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I56cce5cda0621f5e9f1c28aaad4646e9e267d17d
Gerrit-Change-Number: 14795
Gerrit-PatchSet: 2
Gerrit-Owner: Hoernchen 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Ruben Undheim 
Gerrit-Reviewer: laforge 
Gerrit-CC: pespin 
Gerrit-MessageType: merged


Change in ...osmo-msc[master]: fix spelling detected by lintian

2019-07-17 Thread laforge
laforge has submitted this change and it was merged. ( 
https://gerrit.osmocom.org/c/osmo-msc/+/14812 )

Change subject: fix spelling detected by lintian
..

fix spelling detected by lintian

Change-Id: I01e54b5cf111677079a8ad57645d3ceb7834702a
---
M doc/manuals/chapters/counters_generated.adoc
M include/osmocom/msc/gsm_data.h
M include/osmocom/msc/signal.h
M src/libmsc/gsm_04_08.c
4 files changed, 6 insertions(+), 6 deletions(-)

Approvals:
  Jenkins Builder: Verified
  fixeria: Looks good to me, approved
  laforge: Looks good to me, approved



diff --git a/doc/manuals/chapters/counters_generated.adoc 
b/doc/manuals/chapters/counters_generated.adoc
index 6a3a3d2..1b40d8b 100644
--- a/doc/manuals/chapters/counters_generated.adoc
+++ b/doc/manuals/chapters/counters_generated.adoc
@@ -25,7 +25,7 @@
 | sms:delivered | <> | Global SMS Deliver attempts.
 | sms:rp_err_mem | <> | CAUSE_MT_MEM_EXCEEDED errors of MS 
responses on a sms deliver attempt.
 | sms:rp_err_other | <> | Other error of MS responses on 
a sms delive attempt.
-| sms:deliver_unknown_error | <> | Unknown 
error occured during sms delivery.
+| sms:deliver_unknown_error | <> | Unknown 
error occurred during sms delivery.
 | call:mo_setup | <> | Received setup requests from a MS to 
init a MO call.
 | call:mo_connect_ack | <> | Received a connect ack 
from MS of a MO call. Call is now succesful connected up.
 | call:mt_setup | <> | Sent setup requests to the MS (MT).
diff --git a/include/osmocom/msc/gsm_data.h b/include/osmocom/msc/gsm_data.h
index f6e3ed9..6050113 100644
--- a/include/osmocom/msc/gsm_data.h
+++ b/include/osmocom/msc/gsm_data.h
@@ -79,7 +79,7 @@
[MSC_CTR_SMS_DELIVERED] =   {"sms:delivered", "Global SMS 
Deliver attempts."},
[MSC_CTR_SMS_RP_ERR_MEM] =  {"sms:rp_err_mem", 
"CAUSE_MT_MEM_EXCEEDED errors of MS responses on a sms deliver attempt."},
[MSC_CTR_SMS_RP_ERR_OTHER] ={"sms:rp_err_other", "Other 
error of MS responses on a sms delive attempt."},
-   [MSC_CTR_SMS_DELIVER_UNKNOWN_ERROR] =   {"sms:deliver_unknown_error", 
"Unknown error occured during sms delivery."},
+   [MSC_CTR_SMS_DELIVER_UNKNOWN_ERROR] =   {"sms:deliver_unknown_error", 
"Unknown error occurred during sms delivery."},
/* FIXME: count also sms delivered */
[MSC_CTR_CALL_MO_SETUP] =   {"call:mo_setup", "Received 
setup requests from a MS to init a MO call."},
[MSC_CTR_CALL_MO_CONNECT_ACK] = {"call:mo_connect_ack", 
"Received a connect ack from MS of a MO call. Call is now succesful connected 
up."},
diff --git a/include/osmocom/msc/signal.h b/include/osmocom/msc/signal.h
index 16b5678..c7f8991 100644
--- a/include/osmocom/msc/signal.h
+++ b/include/osmocom/msc/signal.h
@@ -89,7 +89,7 @@
struct vty *vty;
 };
 struct sms_signal_data {
-   /* The transaction where this occured */
+   /* The transaction where this occurred */
struct gsm_trans *trans;
/* Can be NULL for SMMA */
struct gsm_sms *sms;
diff --git a/src/libmsc/gsm_04_08.c b/src/libmsc/gsm_04_08.c
index 57dff41..ee31747 100644
--- a/src/libmsc/gsm_04_08.c
+++ b/src/libmsc/gsm_04_08.c
@@ -389,7 +389,7 @@

/* From vlr_loc_update() we expect an implicit dispatch of
 * VLR_ULA_E_UPDATE_LA, and thus we expect msc_vlr_subscr_assoc() to
-* already have been called and completed. Has an error occured? */
+* already have been called and completed. Has an error occurred? */

vsub = msc_a_vsub(msc_a);
if (!vsub) {
@@ -786,7 +786,7 @@
 is_utran);

/* From vlr_proc_acc_req() we expect an implicit dispatch of 
PR_ARQ_E_START we expect
-* msc_vlr_subscr_assoc() to already have been called and completed. 
Has an error occured? */
+* msc_vlr_subscr_assoc() to already have been called and completed. 
Has an error occurred? */
vsub = msc_a_vsub(msc_a);
if (!vsub) {
LOG_MSC_A(msc_a, LOGL_ERROR, "subscriber not allowed to do a CM 
Service Request\n");
@@ -1158,7 +1158,7 @@
 is_utran);

/* From vlr_proc_acc_req() we expect an implicit dispatch of 
PR_ARQ_E_START we expect
-* msc_vlr_subscr_assoc() to already have been called and completed. 
Has an error occured? */
+* msc_vlr_subscr_assoc() to already have been called and completed. 
Has an error occurred? */
vsub = msc_a_vsub(msc_a);
if (!vsub) {
LOG_MSC_A(msc_a, LOGL_ERROR, "subscriber not allowed to do a 
Paging Response\n");

--
To view, visit https://gerrit.osmocom.org/c/osmo-msc/+/14812
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-msc
Gerrit-Branch: master
Gerrit-Change-Id: I01e54b5cf111677079a8ad57645d3ceb7834702a
Gerrit-Change-Number: 14812
Gerrit-PatchSet: 2

Change in ...osmo-msc[master]: fix spelling detected by lintian

2019-07-17 Thread laforge
laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-msc/+/14812 )

Change subject: fix spelling detected by lintian
..


Patch Set 2: Code-Review+2


--
To view, visit https://gerrit.osmocom.org/c/osmo-msc/+/14812
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-msc
Gerrit-Branch: master
Gerrit-Change-Id: I01e54b5cf111677079a8ad57645d3ceb7834702a
Gerrit-Change-Number: 14812
Gerrit-PatchSet: 2
Gerrit-Owner: Hoernchen 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria 
Gerrit-Reviewer: laforge 
Gerrit-Comment-Date: Wed, 17 Jul 2019 11:02:40 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in ...osmo-bsc[master]: Fix some spelling errors found by lintian

2019-07-17 Thread laforge
laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-bsc/+/14810 )

Change subject: Fix some spelling errors found by lintian
..


Patch Set 2: Code-Review+2


--
To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/14810
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: I63a733f8bea69f355a6686d99c3aa194c8ac9012
Gerrit-Change-Number: 14810
Gerrit-PatchSet: 2
Gerrit-Owner: Hoernchen 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Ruben Undheim 
Gerrit-Reviewer: fixeria 
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Wed, 17 Jul 2019 11:01:50 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in ...osmo-bsc[master]: Fix one struct for big-endian archs

2019-07-17 Thread laforge
laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-bsc/+/14811 )

Change subject: Fix one struct for big-endian archs
..


Patch Set 1:

osmo-ci/scripts/libosmocore/contrib/struct_endianess.py preferred


--
To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/14811
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: Ic6e654e0d95d662942c6485097f0b96e1fa46ead
Gerrit-Change-Number: 14811
Gerrit-PatchSet: 1
Gerrit-Owner: Hoernchen 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Ruben Undheim 
Gerrit-Reviewer: pespin 
Gerrit-CC: fixeria 
Gerrit-CC: laforge 
Gerrit-Comment-Date: Wed, 17 Jul 2019 11:02:12 +
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment


Change in ...osmo-bsc[master]: Fix some spelling errors found by lintian

2019-07-17 Thread laforge
laforge has submitted this change and it was merged. ( 
https://gerrit.osmocom.org/c/osmo-bsc/+/14810 )

Change subject: Fix some spelling errors found by lintian
..

Fix some spelling errors found by lintian

Change-Id: I63a733f8bea69f355a6686d99c3aa194c8ac9012
---
M doc/manuals/vty/bsc_vty_reference.xml
M src/osmo-bsc/a_reset.c
M src/osmo-bsc/abis_om2000_vty.c
M src/osmo-bsc/bsc_vty.c
M src/osmo-bsc/osmo_bsc_main.c
M src/osmo-bsc/osmo_bsc_sigtran.c
M src/osmo-bsc/osmo_bsc_vty.c
M src/osmo-bsc/pcu_sock.c
8 files changed, 10 insertions(+), 10 deletions(-)

Approvals:
  Jenkins Builder: Verified
  fixeria: Looks good to me, but someone else must approve
  laforge: Looks good to me, approved
  pespin: Looks good to me, approved



diff --git a/doc/manuals/vty/bsc_vty_reference.xml 
b/doc/manuals/vty/bsc_vty_reference.xml
index 4d25e69..db4f2c1 100644
--- a/doc/manuals/vty/bsc_vty_reference.xml
+++ b/doc/manuals/vty/bsc_vty_reference.xml
@@ -2958,7 +2958,7 @@
 
 
   
-
+
 
 
 
@@ -4830,7 +4830,7 @@
 
 
 
-
+
 
 
 
@@ -4841,7 +4841,7 @@
 
 
 
-
+
 
 
 
diff --git a/src/osmo-bsc/a_reset.c b/src/osmo-bsc/a_reset.c
index 3c21142..fc59ff4 100644
--- a/src/osmo-bsc/a_reset.c
+++ b/src/osmo-bsc/a_reset.c
@@ -71,7 +71,7 @@
 {
struct reset_ctx *reset_ctx = (struct reset_ctx *)fi->priv;
OSMO_ASSERT(reset_ctx);
-   LOGPFSML(fi, LOGL_NOTICE, "SIGTRAN connection succeded.\n");
+   LOGPFSML(fi, LOGL_NOTICE, "SIGTRAN connection succeeded.\n");

reset_ctx->conn_loss_counter = 0;
osmo_fsm_inst_state_chg(fi, ST_CONN, 0, 0);
diff --git a/src/osmo-bsc/abis_om2000_vty.c b/src/osmo-bsc/abis_om2000_vty.c
index faf39c1..3ae870e 100644
--- a/src/osmo-bsc/abis_om2000_vty.c
+++ b/src/osmo-bsc/abis_om2000_vty.c
@@ -368,7 +368,7 @@
 #define CON_PATH_HELP  "CON Path (In/Out)\n"   \
"Add CON Path to Concentration Group\n" \
"Delete CON Path from Concentration Group\n"\
-   "CON Conection Point\n" \
+   "CON Connection Point\n"
\
"Contiguity Index\n"\

 DEFUN(cfg_om2k_con_path_dec, cfg_om2k_con_path_dec_cmd,
diff --git a/src/osmo-bsc/bsc_vty.c b/src/osmo-bsc/bsc_vty.c
index c4e091c..24f8313 100644
--- a/src/osmo-bsc/bsc_vty.c
+++ b/src/osmo-bsc/bsc_vty.c
@@ -2371,7 +2371,7 @@

 DEFUN(cfg_bts_challoc, cfg_bts_challoc_cmd,
   "channel allocator (ascending|descending)",
-   "Channnel Allocator\n" "Channel Allocator\n"
+   "Channel Allocator\n" "Channel Allocator\n"
"Allocate Timeslots and Transceivers in ascending order\n"
"Allocate Timeslots and Transceivers in descending order\n")
 {
diff --git a/src/osmo-bsc/osmo_bsc_main.c b/src/osmo-bsc/osmo_bsc_main.c
index 0e0e8ca..aba8c40 100644
--- a/src/osmo-bsc/osmo_bsc_main.c
+++ b/src/osmo-bsc/osmo_bsc_main.c
@@ -906,7 +906,7 @@
}

if (osmo_bsc_sigtran_init(_gsmnet->bsc_data->mscs) != 0) {
-   LOGP(DNM, LOGL_ERROR, "Failed to initalize sigtran 
backhaul.\n");
+   LOGP(DNM, LOGL_ERROR, "Failed to initialize sigtran 
backhaul.\n");
exit(1);
}

diff --git a/src/osmo-bsc/osmo_bsc_sigtran.c b/src/osmo-bsc/osmo_bsc_sigtran.c
index 763e1e1..0bf48a7 100644
--- a/src/osmo-bsc/osmo_bsc_sigtran.c
+++ b/src/osmo-bsc/osmo_bsc_sigtran.c
@@ -564,7 +564,7 @@
/* Start MSC-Reset procedure */
a_reset_alloc(msc, msc_name, osmo_bsc_sigtran_reset_cb);

-   /* If we have detected that the SS7 configuration of the MSC we 
have just initalized
+   /* If we have detected that the SS7 configuration of the MSC we 
have just initialized
 * was incomplete or completely missing, we can not tolerate 
another incomplete
 * configuration. The reson for this is that we do only specify 
exactly one default
 * pointcode pair. We also specify localhost as default 
IP-Address. If we have wanted
diff --git a/src/osmo-bsc/osmo_bsc_vty.c b/src/osmo-bsc/osmo_bsc_vty.c
index 4591ad1..e656d70 100644
--- a/src/osmo-bsc/osmo_bsc_vty.c
+++ b/src/osmo-bsc/osmo_bsc_vty.c
@@ -569,7 +569,7 @@
if (addr->presence & OSMO_SCCP_ADDR_T_SSN) {
if (addr->ssn != OSMO_SCCP_SSN_BSSAP)
vty_out(vty,
-   "setting an SSN (%u) different from the 
standard (%u) is not allowd, will use standard SSN for address: %s%s",
+   "setting an SSN (%u) different from the 
standard (%u) is not allowed, will use standard SSN for address: %s%s",
 

Change in ...osmo-pcu[master]: Add structs for big-endian

2019-07-17 Thread laforge
laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-pcu/+/14802 )

Change subject: Add structs for big-endian
..


Patch Set 1:

we do have osmo-ci/scripts/libosmocore/contrib/struct_endianess.py to do this 
in an automatic/programmatic way, which I'd prefer over manual patches.


--
To view, visit https://gerrit.osmocom.org/c/osmo-pcu/+/14802
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-pcu
Gerrit-Branch: master
Gerrit-Change-Id: I6af6d5ef05a8b240acfd9da747d9657bb6aa0ef8
Gerrit-Change-Number: 14802
Gerrit-PatchSet: 1
Gerrit-Owner: Hoernchen 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Ruben Undheim 
Gerrit-Reviewer: pespin 
Gerrit-CC: laforge 
Gerrit-Comment-Date: Wed, 17 Jul 2019 11:01:24 +
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment


Change in ...osmo-sgsn[master]: Compare with -ENOTSUP to fix build on mipsel/mips64el/alpha

2019-07-17 Thread laforge
laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-sgsn/+/14816 )

Change subject: Compare with -ENOTSUP to fix build on mipsel/mips64el/alpha
..


Patch Set 1: Code-Review-1

same here, I'm really not convinced at all this is a valid fix.


--
To view, visit https://gerrit.osmocom.org/c/osmo-sgsn/+/14816
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-sgsn
Gerrit-Branch: master
Gerrit-Change-Id: I591d37377495829d2a2bbb350e2c78fe4bddc745
Gerrit-Change-Number: 14816
Gerrit-PatchSet: 1
Gerrit-Owner: Hoernchen 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Ruben Undheim 
Gerrit-Reviewer: laforge 
Gerrit-CC: pespin 
Gerrit-Comment-Date: Wed, 17 Jul 2019 11:00:05 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in ...osmo-sgsn[master]: Spelling fixes

2019-07-17 Thread laforge
laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-sgsn/+/14815 )

Change subject: Spelling fixes
..


Patch Set 1: Code-Review+2


--
To view, visit https://gerrit.osmocom.org/c/osmo-sgsn/+/14815
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-sgsn
Gerrit-Branch: master
Gerrit-Change-Id: Id1511c5022a239db5d0b44ec7adf048cca307751
Gerrit-Change-Number: 14815
Gerrit-PatchSet: 1
Gerrit-Owner: Hoernchen 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Ruben Undheim 
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Wed, 17 Jul 2019 10:59:22 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in ...osmo-sgsn[master]: Spelling fixes

2019-07-17 Thread laforge
laforge has submitted this change and it was merged. ( 
https://gerrit.osmocom.org/c/osmo-sgsn/+/14815 )

Change subject: Spelling fixes
..

Spelling fixes

Change-Id: Id1511c5022a239db5d0b44ec7adf048cca307751
---
M src/gprs/gprs_gmm.c
M src/gprs/gprs_sgsn.c
M src/gprs/gprs_sndcp_dcomp.c
M src/gprs/gprs_sndcp_pcomp.c
M src/gprs/gtphub_main.c
M src/gprs/sgsn_vty.c
6 files changed, 11 insertions(+), 11 deletions(-)

Approvals:
  laforge: Looks good to me, approved
  pespin: Looks good to me, approved
  Jenkins Builder: Verified



diff --git a/src/gprs/gprs_gmm.c b/src/gprs/gprs_gmm.c
index e7f0bfe..3465a52 100644
--- a/src/gprs/gprs_gmm.c
+++ b/src/gprs/gprs_gmm.c
@@ -2213,7 +2213,7 @@

rc = gsm48_tx_gmm_auth_ciph_req(mm, >vec, at->key_seq, 
false);
if (rc < 0)
-   LOGMMCTXP(LOGL_ERROR, mm, "failed sending Auth. & Ciph. 
Reuqest: %s \n", strerror(-rc));
+   LOGMMCTXP(LOGL_ERROR, mm, "failed sending Auth. & Ciph. 
Request: %s \n", strerror(-rc));
else
osmo_timer_schedule(>timer, sgsn->cfg.timers.T3360, 
0);
break;
diff --git a/src/gprs/gprs_sgsn.c b/src/gprs/gprs_sgsn.c
index 8d3de7b..fae09ae 100644
--- a/src/gprs/gprs_sgsn.c
+++ b/src/gprs/gprs_sgsn.c
@@ -103,9 +103,9 @@

 static const struct rate_ctr_desc sgsn_ctr_description[] = {
{ "llc:dl_bytes", "Count sent LLC bytes before giving it to the bssgp 
layer" },
-   { "llc:ul_bytes", "Count sucessful received LLC bytes (encrypt & fcs 
correct)" },
-   { "llc:dl_packets", "Count sucessful sent LLC packets before giving it 
to the bssgp layer" },
-   { "llc:ul_packets", "Count sucessful received LLC packets (encrypt & 
fcs correct)" },
+   { "llc:ul_bytes", "Count successful received LLC bytes (encrypt & fcs 
correct)" },
+   { "llc:dl_packets", "Count successful sent LLC packets before giving it 
to the bssgp layer" },
+   { "llc:ul_packets", "Count successful received LLC packets (encrypt & 
fcs correct)" },
{ "gprs:attach_requested", "Received attach requests" },
{ "gprs:attach_accepted", "Sent attach accepts" },
{ "gprs:attach_rejected", "Sent attach rejects" },
diff --git a/src/gprs/gprs_sndcp_dcomp.c b/src/gprs/gprs_sndcp_dcomp.c
index 00e40a7..c0da84d 100644
--- a/src/gprs/gprs_sndcp_dcomp.c
+++ b/src/gprs/gprs_sndcp_dcomp.c
@@ -94,7 +94,7 @@
_v42bis_data_handler, NULL,
V42BIS_MAX_OUTPUT_LENGTH);
LOGP(DSNDCP, LOGL_INFO,
-"V.42bis data compression initalized.\n");
+"V.42bis data compression initialized.\n");
return 0;
}

diff --git a/src/gprs/gprs_sndcp_pcomp.c b/src/gprs/gprs_sndcp_pcomp.c
index 5f7f22a..8c2fc97 100644
--- a/src/gprs/gprs_sndcp_pcomp.c
+++ b/src/gprs/gprs_sndcp_pcomp.c
@@ -59,7 +59,7 @@
slhc_init(ctx, comp_field->rfc1144_params->s01 + 1,
  comp_field->rfc1144_params->s01 + 1);
LOGP(DSNDCP, LOGL_INFO,
-"RFC1144 header compression initalized.\n");
+"RFC1144 header compression initialized.\n");
return 0;
}

diff --git a/src/gprs/gtphub_main.c b/src/gprs/gtphub_main.c
index 0369ed7..585a3cb 100644
--- a/src/gprs/gtphub_main.c
+++ b/src/gprs/gtphub_main.c
@@ -243,7 +243,7 @@

 static void list_categories(void)
 {
-   printf("Avaliable debug categories:\n");
+   printf("Available debug categories:\n");
int i;
for (i = 0; i < gtphub_log_info.num_cat; ++i) {
if (!gtphub_log_info.cat[i].name)
diff --git a/src/gprs/sgsn_vty.c b/src/gprs/sgsn_vty.c
index edc5b09..d1d53e6 100644
--- a/src/gprs/sgsn_vty.c
+++ b/src/gprs/sgsn_vty.c
@@ -1309,7 +1309,7 @@
 DEFUN(cfg_comp_rfc1144, cfg_comp_rfc1144_cmd,
   "compression rfc1144 active slots <1-256>",
   COMPRESSION_STR
-  "RFC1144 Header compresion scheme\n"
+  "RFC1144 Header compression scheme\n"
   "Compression is actively proposed\n"
   "Number of compression state slots\n"
   "Number of compression state slots\n")
@@ -1323,7 +1323,7 @@
 DEFUN(cfg_comp_rfc1144p, cfg_comp_rfc1144p_cmd,
   "compression rfc1144 passive",
   COMPRESSION_STR
-  "RFC1144 Header compresion scheme\n"
+  "RFC1144 Header compression scheme\n"
   "Compression is available on request\n")
 {
g_cfg->pcomp_rfc1144.active = 0;
@@ -1343,7 +1343,7 @@
 DEFUN(cfg_comp_v42bis, cfg_comp_v42bis_cmd,
   "compression v42bis active direction (ms|sgsn|both) codewords 
<512-65535> strlen <6-250>",
   COMPRESSION_STR
-  "V.42bis data compresion scheme\n"
+  "V.42bis data compression scheme\n"
   "Compression is actively proposed\n"
   "Direction in which the compression shall be active (p0)\n"
   "Compress 

Change in ...osmo-iuh[master]: spelling stuff found by lintian

2019-07-17 Thread laforge
laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-iuh/+/14808 )

Change subject: spelling stuff found by lintian
..


Patch Set 1: Code-Review+2


--
To view, visit https://gerrit.osmocom.org/c/osmo-iuh/+/14808
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-iuh
Gerrit-Branch: master
Gerrit-Change-Id: I8434640701746d5db8707472d1d26c313e0814e9
Gerrit-Change-Number: 14808
Gerrit-PatchSet: 1
Gerrit-Owner: Hoernchen 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Wed, 17 Jul 2019 10:58:37 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in ...osmo-bts[master]: fix spelling stuff mentioned by lintian

2019-07-17 Thread laforge
laforge has submitted this change and it was merged. ( 
https://gerrit.osmocom.org/c/osmo-bts/+/14813 )

Change subject: fix spelling stuff mentioned by lintian
..

fix spelling stuff mentioned by lintian

Change-Id: I3d6cb6fc1b182d8520ba60e431ab9b74e71d5e3c
---
M src/common/pcu_sock.c
M src/common/rsl.c
M src/osmo-bts-octphy/l1_if.c
M src/osmo-bts-trx/trx_vty.c
4 files changed, 4 insertions(+), 4 deletions(-)

Approvals:
  laforge: Looks good to me, approved
  pespin: Looks good to me, approved
  Jenkins Builder: Verified



diff --git a/src/common/pcu_sock.c b/src/common/pcu_sock.c
index 39b4568..1d9fa72 100644
--- a/src/common/pcu_sock.c
+++ b/src/common/pcu_sock.c
@@ -677,7 +677,7 @@
rc = pcu_rx_txt_ind(bts, _prim->u.txt_ind);
break;
default:
-   LOGP(DPCU, LOGL_ERROR, "Received unknwon PCU msg type %d\n",
+   LOGP(DPCU, LOGL_ERROR, "Received unknown PCU msg type %d\n",
msg_type);
rc = -EINVAL;
}
diff --git a/src/common/rsl.c b/src/common/rsl.c
index 0a613bb..0a6a4e5 100644
--- a/src/common/rsl.c
+++ b/src/common/rsl.c
@@ -3090,7 +3090,7 @@

msg->lchan = lchan_lookup(trx, dch->chan_nr, "RSL rx IPACC: ");
if (!msg->lchan) {
-   LOGP(DRSL, LOGL_ERROR, "Rx RSL %s for unknow lchan\n",
+   LOGP(DRSL, LOGL_ERROR, "Rx RSL %s for unknown lchan\n",
rsl_msg_name(dch->c.msg_type));
return rsl_reject_unknown_lchan(msg);
}
diff --git a/src/osmo-bts-octphy/l1_if.c b/src/osmo-bts-octphy/l1_if.c
index 32e738f..612c29a 100644
--- a/src/osmo-bts-octphy/l1_if.c
+++ b/src/osmo-bts-octphy/l1_if.c
@@ -1175,7 +1175,7 @@

if (ra_ind->ulMsgLength != 1) {
LOGPFN(DL1C, LOGL_ERROR, ra_ind->ulFrameNumber,
-   "Rx PH-RACH.ind has lenghth %d > 1\n", 
ra_ind->ulMsgLength);
+   "Rx PH-RACH.ind has length %d > 1\n", 
ra_ind->ulMsgLength);
msgb_free(l1p_msg);
return 0;
}
diff --git a/src/osmo-bts-trx/trx_vty.c b/src/osmo-bts-trx/trx_vty.c
index e9710ac..b9af445 100644
--- a/src/osmo-bts-trx/trx_vty.c
+++ b/src/osmo-bts-trx/trx_vty.c
@@ -237,7 +237,7 @@
OSMOTRX_STR
"Set the maximum acceptable delay of a Normal Burst (in GSM symbols)."
" USE FOR TESTING ONLY, DON'T CHANGE IN PRODUCTION USE!"
-   " During normal operation, Normal Bursts delay are controled by a 
Timing"
+   " During normal operation, Normal Bursts delay are controlled by a 
Timing"
" Advance control loop and thus Normal Bursts arrive to a BTS with no 
more"
" than a couple GSM symbols, which is already taken into account in 
osmo-trx."
" So changing this setting will have no effect in production 
installations"

--
To view, visit https://gerrit.osmocom.org/c/osmo-bts/+/14813
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: I3d6cb6fc1b182d8520ba60e431ab9b74e71d5e3c
Gerrit-Change-Number: 14813
Gerrit-PatchSet: 2
Gerrit-Owner: Hoernchen 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: pespin 
Gerrit-MessageType: merged


Change in ...osmo-iuh[master]: spelling stuff found by lintian

2019-07-17 Thread laforge
laforge has submitted this change and it was merged. ( 
https://gerrit.osmocom.org/c/osmo-iuh/+/14808 )

Change subject: spelling stuff found by lintian
..

spelling stuff found by lintian

Change-Id: I8434640701746d5db8707472d1d26c313e0814e9
---
M src/hnbgw_rua.c
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Jenkins Builder: Verified
  pespin: Looks good to me, approved
  laforge: Looks good to me, approved



diff --git a/src/hnbgw_rua.c b/src/hnbgw_rua.c
index 40d1d94..71c3f8e 100644
--- a/src/hnbgw_rua.c
+++ b/src/hnbgw_rua.c
@@ -505,7 +505,7 @@
 static int rua_rx_successful_outcome_msg(struct msgb *msg, 
RUA_SuccessfulOutcome_t *in)
 {
/* FIXME */
-   LOGP(DRUA, LOGL_NOTICE, "Unexpected RUA Sucessful Outcome\n");
+   LOGP(DRUA, LOGL_NOTICE, "Unexpected RUA Successful Outcome\n");
return -1;
 }


--
To view, visit https://gerrit.osmocom.org/c/osmo-iuh/+/14808
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-iuh
Gerrit-Branch: master
Gerrit-Change-Id: I8434640701746d5db8707472d1d26c313e0814e9
Gerrit-Change-Number: 14808
Gerrit-PatchSet: 1
Gerrit-Owner: Hoernchen 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: pespin 
Gerrit-MessageType: merged


Change in ...osmo-bts[master]: fix spelling stuff mentioned by lintian

2019-07-17 Thread laforge
laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-bts/+/14813 )

Change subject: fix spelling stuff mentioned by lintian
..


Patch Set 1: Code-Review+2


--
To view, visit https://gerrit.osmocom.org/c/osmo-bts/+/14813
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: I3d6cb6fc1b182d8520ba60e431ab9b74e71d5e3c
Gerrit-Change-Number: 14813
Gerrit-PatchSet: 1
Gerrit-Owner: Hoernchen 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Wed, 17 Jul 2019 10:58:56 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in ...osmo-trx[master]: Fix spelling discovered by lintian

2019-07-17 Thread laforge
laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-trx/+/14807 )

Change subject: Fix spelling discovered by lintian
..


Patch Set 1: Code-Review+2


--
To view, visit https://gerrit.osmocom.org/c/osmo-trx/+/14807
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-trx
Gerrit-Branch: master
Gerrit-Change-Id: I5ab9b9c7c47d0d6e674c1f5242e2b3a05006293e
Gerrit-Change-Number: 14807
Gerrit-PatchSet: 1
Gerrit-Owner: Hoernchen 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Ruben Undheim 
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Wed, 17 Jul 2019 10:58:21 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in ...osmo-trx[master]: Fix spelling discovered by lintian

2019-07-17 Thread laforge
laforge has submitted this change and it was merged. ( 
https://gerrit.osmocom.org/c/osmo-trx/+/14807 )

Change subject: Fix spelling discovered by lintian
..

Fix spelling discovered by lintian

Change-Id: I5ab9b9c7c47d0d6e674c1f5242e2b3a05006293e
---
M Transceiver52M/osmo-trx.cpp
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  laforge: Looks good to me, approved
  pespin: Looks good to me, approved
  Jenkins Builder: Verified



diff --git a/Transceiver52M/osmo-trx.cpp b/Transceiver52M/osmo-trx.cpp
index 06b1a18..b8cf968 100644
--- a/Transceiver52M/osmo-trx.cpp
+++ b/Transceiver52M/osmo-trx.cpp
@@ -255,7 +255,7 @@
 {
LOG(WARNING) << "Cmd line option '" << opt << "' is deprecated and will 
be soon removed."
<< " Please use VTY cfg option instead."
-   << " All cmd line options are already being overriden by VTY 
options if set.";
+   << " All cmd line options are already being overridden by VTY 
options if set.";
 }

 static void handle_options(int argc, char **argv, struct trx_ctx* trx)

--
To view, visit https://gerrit.osmocom.org/c/osmo-trx/+/14807
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-trx
Gerrit-Branch: master
Gerrit-Change-Id: I5ab9b9c7c47d0d6e674c1f5242e2b3a05006293e
Gerrit-Change-Number: 14807
Gerrit-PatchSet: 2
Gerrit-Owner: Hoernchen 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Ruben Undheim 
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: pespin 
Gerrit-MessageType: merged


Change in ...libosmo-sccp[master]: This patch fixes typos on which Lintian complains

2019-07-17 Thread laforge
laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/libosmo-sccp/+/14803 )

Change subject: This patch fixes typos on which Lintian complains
..


Patch Set 1: Code-Review+2


--
To view, visit https://gerrit.osmocom.org/c/libosmo-sccp/+/14803
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: libosmo-sccp
Gerrit-Branch: master
Gerrit-Change-Id: I657c694a9a7474182a97f90aa8424403ca9c58db
Gerrit-Change-Number: 14803
Gerrit-PatchSet: 1
Gerrit-Owner: Hoernchen 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Wed, 17 Jul 2019 10:56:36 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in ...libosmo-sccp[master]: This patch fixes typos on which Lintian complains

2019-07-17 Thread laforge
laforge has submitted this change and it was merged. ( 
https://gerrit.osmocom.org/c/libosmo-sccp/+/14803 )

Change subject: This patch fixes typos on which Lintian complains
..

This patch fixes typos on which Lintian complains

Change-Id: I657c694a9a7474182a97f90aa8424403ca9c58db
---
M src/m3ua.c
M stp/stp_main.c
2 files changed, 2 insertions(+), 2 deletions(-)

Approvals:
  laforge: Looks good to me, approved
  pespin: Looks good to me, approved
  Jenkins Builder: Verified



diff --git a/src/m3ua.c b/src/m3ua.c
index d7ede40..d733b51 100644
--- a/src/m3ua.c
+++ b/src/m3ua.c
@@ -91,7 +91,7 @@
 };

 const struct value_string m3ua_ntfy_other_names[] = {
-   { M3UA_NOTIFY_I_OT_INS_RES, "Insufficient ASP Resouces active in 
AS" },
+   { M3UA_NOTIFY_I_OT_INS_RES, "Insufficient ASP Resources active in 
AS" },
{ M3UA_NOTIFY_I_OT_ALT_ASP_ACT, "Alternative ASP Active" },
{ M3UA_NOTIFY_I_OT_ASP_FAILURE, "ASP Failure" },
{ 0, NULL }
diff --git a/stp/stp_main.c b/stp/stp_main.c
index 6a47403..ff33959 100644
--- a/stp/stp_main.c
+++ b/stp/stp_main.c
@@ -85,7 +85,7 @@
 static void print_help(void)
 {
printf("  -h --help This text.\n");
-   printf("  -D --daemonizeFork teh process into a 
background daemon\n");
+   printf("  -D --daemonizeFork the process into a 
background daemon\n");
printf("  -c --config-file filename The config file to use. 
Default: ./osmo-stp.cfg\n");
printf("  -V --version  Print the version of 
OsmoSTP\n");
 }

--
To view, visit https://gerrit.osmocom.org/c/libosmo-sccp/+/14803
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: libosmo-sccp
Gerrit-Branch: master
Gerrit-Change-Id: I657c694a9a7474182a97f90aa8424403ca9c58db
Gerrit-Change-Number: 14803
Gerrit-PatchSet: 2
Gerrit-Owner: Hoernchen 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: pespin 
Gerrit-MessageType: merged


Change in ...osmo-pcu[master]: fix spelling errors detected by lintian

2019-07-17 Thread laforge
laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-pcu/+/14801 )

Change subject: fix spelling errors detected by lintian
..


Patch Set 1: Code-Review+2


--
To view, visit https://gerrit.osmocom.org/c/osmo-pcu/+/14801
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-pcu
Gerrit-Branch: master
Gerrit-Change-Id: I381618eb55cc513cfa9c2e384c27cead0935c8bf
Gerrit-Change-Number: 14801
Gerrit-PatchSet: 1
Gerrit-Owner: Hoernchen 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Wed, 17 Jul 2019 10:56:18 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in ...osmo-pcu[master]: fix spelling errors detected by lintian

2019-07-17 Thread laforge
laforge has submitted this change and it was merged. ( 
https://gerrit.osmocom.org/c/osmo-pcu/+/14801 )

Change subject: fix spelling errors detected by lintian
..

fix spelling errors detected by lintian

Change-Id: I381618eb55cc513cfa9c2e384c27cead0935c8bf
---
M src/bts.cpp
M src/decoding.cpp
M src/encoding.cpp
M src/gsm_rlcmac.cpp
M src/gsm_rlcmac.h
M src/pcu_l1_if.cpp
M src/pdch.cpp
M src/tbf.cpp
M src/tbf.h
9 files changed, 14 insertions(+), 14 deletions(-)

Approvals:
  laforge: Looks good to me, approved
  pespin: Looks good to me, approved
  Jenkins Builder: Verified



diff --git a/src/bts.cpp b/src/bts.cpp
index 8799c3d..395fc73 100644
--- a/src/bts.cpp
+++ b/src/bts.cpp
@@ -611,7 +611,7 @@
 * we roll back the rounded frame number by one RFN_MODULUS */
if (abs(rfn - m_cur_rfn) > RFN_THRESHOLD) {
LOGP(DRLCMAC, LOGL_DEBUG,
-"Race condition between rfn (%u) and m_cur_fn (%u) 
detected: rfn belongs to the previos modulus %u cycle, wrappng...\n",
+"Race condition between rfn (%u) and m_cur_fn (%u) 
detected: rfn belongs to the previous modulus %u cycle, wrapping...\n",
 rfn, m_cur_fn, RFN_MODULUS);
if (fn_rounded < RFN_MODULUS) {
LOGP(DRLCMAC, LOGL_DEBUG,
diff --git a/src/decoding.cpp b/src/decoding.cpp
index d439e30..99a17dc 100644
--- a/src/decoding.cpp
+++ b/src/decoding.cpp
@@ -77,7 +77,7 @@
/* TS 44.060, table 10.4.14a.1, row 3 & 5 */
/* only filling bytes left */
LOGP(DRLCMACUL, LOGL_DEBUG, "UL DATA LI contains "
-   "only filling bytes with extention octet: 
LI=%d, E=%d, count=%d\n",
+   "only filling bytes with extension octet: 
LI=%d, E=%d, count=%d\n",
li->li, li->e, num_chunks);
break;
} else if (li->li > 0) {
@@ -92,7 +92,7 @@
}
 
LOGP(DRLCMACUL, LOGL_DEBUG, "UL DATA LI contains "
-   "extention octet: LI=%d, E=%d, count=%d\n",
+   "extension octet: LI=%d, E=%d, count=%d\n",
li->li, li->e, num_chunks);
num_chunks += 1;

@@ -168,7 +168,7 @@
chunks[num_chunks].is_complete = li->li || is_last_block;

LOGP(DRLCMACUL, LOGL_DEBUG, "UL DATA LI contains "
-   "extention octet: LI=%d, M=%d, E=%d, count=%d\n",
+   "extension octet: LI=%d, M=%d, E=%d, count=%d\n",
li->li, li->m, li->e, num_chunks);
num_chunks += 1;

diff --git a/src/encoding.cpp b/src/encoding.cpp
index ca9e906..3fac75e 100644
--- a/src/encoding.cpp
+++ b/src/encoding.cpp
@@ -532,7 +532,7 @@

if (rc < 0) {
LOGP(DRLCMAC, LOGL_ERROR,
-   "Failed to create IMMEDIATE ASSIGMENT (%s) for %s\n",
+   "Failed to create IMMEDIATE ASSIGNMENT (%s) for %s\n",
downlink ? "downlink" : "uplink",
tbf ? tbf->name() : "single block allocation");
return rc;
diff --git a/src/gsm_rlcmac.cpp b/src/gsm_rlcmac.cpp
index 6323b09..ef7abcd 100644
--- a/src/gsm_rlcmac.cpp
+++ b/src/gsm_rlcmac.cpp
@@ -1363,8 +1363,8 @@

 static const
 CSN_DESCR_BEGIN(PCCF_AdditionsR5_t)
-  M_NEXT_EXIST (PCCF_AdditionsR5_t, Exist_G_RNTI_extention, 1),
-  M_UINT   (PCCF_AdditionsR5_t,  G_RNTI_extention,  4),
+  M_NEXT_EXIST (PCCF_AdditionsR5_t, Exist_G_RNTI_extension, 1),
+  M_UINT   (PCCF_AdditionsR5_t,  G_RNTI_extension,  4),
   M_NEXT_EXIST_OR_NULL(PCCF_AdditionsR5_t, Exist_AdditionsR8, 1),
   M_TYPE   (PCCF_AdditionsR5_t, AdditionsR8, PCCF_AdditionsR8_t),
 CSN_DESCR_END  (PCCF_AdditionsR5_t)
diff --git a/src/gsm_rlcmac.h b/src/gsm_rlcmac.h
index 2ad1937..a4750e8 100644
--- a/src/gsm_rlcmac.h
+++ b/src/gsm_rlcmac.h
@@ -1602,8 +1602,8 @@

 typedef struct
 {
-  guint8 Exist_G_RNTI_extention;
-  guint8  G_RNTI_extention;
+  guint8 Exist_G_RNTI_extension;
+  guint8  G_RNTI_extension;
   guint8 Exist_AdditionsR8;
   PCCF_AdditionsR8_t AdditionsR8;
 } PCCF_AdditionsR5_t;
diff --git a/src/pcu_l1_if.cpp b/src/pcu_l1_if.cpp
index bc712fd..da85df4 100644
--- a/src/pcu_l1_if.cpp
+++ b/src/pcu_l1_if.cpp
@@ -650,7 +650,7 @@
rc = pcu_rx_susp_req(_prim->u.susp_req);
break;
default:
-   LOGP(DL1IF, LOGL_ERROR, "Received unknwon PCU msg type %d\n",
+   LOGP(DL1IF, LOGL_ERROR, "Received unknown PCU msg type %d\n",
msg_type);
rc = -EINVAL;
}
diff --git a/src/pdch.cpp b/src/pdch.cpp
index 8003105..7488b96 100644
--- a/src/pdch.cpp
+++ b/src/pdch.cpp
@@ -632,7 +632,7 @@
int8_t tfi = request->ID.u.Global_TFI.u.DOWNLINK_TFI;
 

Change in ...libosmo-abis[master]: spelling

2019-07-17 Thread laforge
laforge has submitted this change and it was merged. ( 
https://gerrit.osmocom.org/c/libosmo-abis/+/14800 )

Change subject: spelling
..

spelling

===

Change-Id: I22c98915648760a83abd5f0004af26d426a5e518
---
M src/input/ipa.c
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  laforge: Looks good to me, approved
  pespin: Looks good to me, approved
  Jenkins Builder: Verified



diff --git a/src/input/ipa.c b/src/input/ipa.c
index 25eeb4a..8aed7b3 100644
--- a/src/input/ipa.c
+++ b/src/input/ipa.c
@@ -272,7 +272,7 @@
ret = link->accept_cb(link, fd);
if (ret < 0) {
LOGP(DLINP, LOGL_ERROR,
-"failed to processs accept()ed new link, "
+"failed to process accept()ed new link, "
 "reason=`%s'\n", strerror(-ret));
close(fd);
return ret;

--
To view, visit https://gerrit.osmocom.org/c/libosmo-abis/+/14800
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: libosmo-abis
Gerrit-Branch: master
Gerrit-Change-Id: I22c98915648760a83abd5f0004af26d426a5e518
Gerrit-Change-Number: 14800
Gerrit-PatchSet: 2
Gerrit-Owner: Hoernchen 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: pespin 
Gerrit-MessageType: merged


Change in ...libosmo-abis[master]: spelling

2019-07-17 Thread laforge
laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/libosmo-abis/+/14800 )

Change subject: spelling
..


Patch Set 1: Code-Review+2


--
To view, visit https://gerrit.osmocom.org/c/libosmo-abis/+/14800
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: libosmo-abis
Gerrit-Branch: master
Gerrit-Change-Id: I22c98915648760a83abd5f0004af26d426a5e518
Gerrit-Change-Number: 14800
Gerrit-PatchSet: 1
Gerrit-Owner: Hoernchen 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Wed, 17 Jul 2019 10:55:42 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in ...libosmocore[master]: MAXPATHLEN set if not defined

2019-07-17 Thread laforge
laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/libosmocore/+/14797 )

Change subject: MAXPATHLEN set if not defined
..


Patch Set 1: Code-Review+2


--
To view, visit https://gerrit.osmocom.org/c/libosmocore/+/14797
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I1dce8ace228814b5a7246a00b31309ab9461d266
Gerrit-Change-Number: 14797
Gerrit-PatchSet: 1
Gerrit-Owner: Hoernchen 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Ruben Undheim 
Gerrit-Reviewer: laforge 
Gerrit-CC: pespin 
Gerrit-Comment-Date: Wed, 17 Jul 2019 10:54:52 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in ...libosmocore[master]: TIMER constants not on some architectures

2019-07-17 Thread laforge
laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/libosmocore/+/14796 )

Change subject: TIMER constants not on some architectures
..


Patch Set 1: Code-Review-1

Historically, we once supported (in form of build-test verification integrated 
into gerrit) FreeBSD.  However, it was causing significant effort to maintain 
FreeBSD compatibility, while nobody in the active Osmocom developer community 
is using FreeBSD for their development.  Also, we never ever received report 
from even a single user of Osmocom on FreeBSD.

As a result, we decided to drop FreeBSD compatibility and stated we'd 
re-introduce it if somebody from the user/developer community was willing to 
actively maintain it and provide related patches.  This hasn't happened so far, 
so I'm against merging the occasional one-off patch which creates the 
impression of supporting non-Linux platforms while in reality nobody is 
testing/maintaining it.


--
To view, visit https://gerrit.osmocom.org/c/libosmocore/+/14796
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: If24616c3dbf9f1eb6053caed90a63b590d3f5833
Gerrit-Change-Number: 14796
Gerrit-PatchSet: 1
Gerrit-Owner: Hoernchen 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Ruben Undheim 
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Wed, 17 Jul 2019 10:54:42 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in ...libosmocore[master]: Set HTML_TIMESTAMP to NO to make build reproducible

2019-07-17 Thread laforge
laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/libosmocore/+/14795 )

Change subject: Set HTML_TIMESTAMP to NO to make build reproducible
..


Patch Set 1: Code-Review+2


--
To view, visit https://gerrit.osmocom.org/c/libosmocore/+/14795
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I56cce5cda0621f5e9f1c28aaad4646e9e267d17d
Gerrit-Change-Number: 14795
Gerrit-PatchSet: 1
Gerrit-Owner: Hoernchen 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Ruben Undheim 
Gerrit-Reviewer: laforge 
Gerrit-CC: pespin 
Gerrit-Comment-Date: Wed, 17 Jul 2019 10:51:21 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in ...osmo-msc[master]: replace osmo_counter with stat_items

2019-07-17 Thread laforge
laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-msc/+/13802 )

Change subject: replace osmo_counter with stat_items
..


Patch Set 11: Code-Review+2


--
To view, visit https://gerrit.osmocom.org/c/osmo-msc/+/13802
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-msc
Gerrit-Branch: master
Gerrit-Change-Id: I6a20123b263f4f808153794ee8a735092deb399e
Gerrit-Change-Number: 13802
Gerrit-PatchSet: 11
Gerrit-Owner: lynxis lazus 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: daniel 
Gerrit-Reviewer: fixeria 
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: lynxis lazus 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Wed, 17 Jul 2019 10:50:28 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in ...osmocom-bb[master]: target/*.py: shebang: use /usr/bin/env python

2019-07-17 Thread laforge
laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmocom-bb/+/14759 )

Change subject: target/*.py: shebang: use /usr/bin/env python
..


Patch Set 4: Code-Review+1


--
To view, visit https://gerrit.osmocom.org/c/osmocom-bb/+/14759
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Change-Id: Iab185759b574eff1ca1b189dcbb4e1a3eec52132
Gerrit-Change-Number: 14759
Gerrit-PatchSet: 4
Gerrit-Owner: lynxis lazus 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria 
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: lynxis lazus 
Gerrit-CC: pespin 
Gerrit-Comment-Date: Wed, 17 Jul 2019 10:49:45 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in ...osmo-msc[master]: replace osmo_counter with stat_items

2019-07-17 Thread fixeria
fixeria has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-msc/+/13802 )

Change subject: replace osmo_counter with stat_items
..


Patch Set 11: Code-Review+1


--
To view, visit https://gerrit.osmocom.org/c/osmo-msc/+/13802
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-msc
Gerrit-Branch: master
Gerrit-Change-Id: I6a20123b263f4f808153794ee8a735092deb399e
Gerrit-Change-Number: 13802
Gerrit-PatchSet: 11
Gerrit-Owner: lynxis lazus 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: daniel 
Gerrit-Reviewer: fixeria 
Gerrit-Reviewer: lynxis lazus 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Wed, 17 Jul 2019 07:02:53 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in ...osmocom-bb[master]: target/*.py: shebang: use /usr/bin/env python

2019-07-17 Thread fixeria
fixeria has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmocom-bb/+/14759 )

Change subject: target/*.py: shebang: use /usr/bin/env python
..


Patch Set 4: Code-Review+2


--
To view, visit https://gerrit.osmocom.org/c/osmocom-bb/+/14759
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Change-Id: Iab185759b574eff1ca1b189dcbb4e1a3eec52132
Gerrit-Change-Number: 14759
Gerrit-PatchSet: 4
Gerrit-Owner: lynxis lazus 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria 
Gerrit-Reviewer: lynxis lazus 
Gerrit-CC: pespin 
Gerrit-Comment-Date: Wed, 17 Jul 2019 06:59:12 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment