Patch Set 4:
FYI: I've created OS#2334 to track jenkins build slave package updates.
--
To view, visit https://gerrit.osmocom.org/2825
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ifb59b79021b2c935b326496ed339f12e13c96b8e
Gerrit-PatchS
Patch Set 9:
The patch order is also visible in top right corner of web UI in "Related
changes". I do agree though that it would be easier if gerrit would
automatically mark patches submitted together explicitly like [1/3].
--
To view, visit https://gerrit.osmocom.org/2894
To unsubscribe, vis
Patch Set 9: Code-Review-1
(4 comments)
Again apologies, in my last comment I didn't notice that the patches are now
independent. Still there is no way to commit the lc15 without the code undup,
which I would have liked to do now. I would accept both at this point, but
there's still a problem
Patch Set 9:
Hmm... you're right. It's weird that it doesn't cause build failure.
--
To view, visit https://gerrit.osmocom.org/2894
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic48b1f75179b9008d65219dd5a47c1ab3b886408
Gerrit-PatchSet:
Hello Neels Hofmeyr, Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/2894
to look at the new patch set (#10).
Move common steps into common jenkins helper
Move value_string termination check and OpenBSC headers checkout into
shared jenkins help
Hello Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/2974
to look at the new patch set (#5).
Use generic L1 headers helper
Change-Id: I2253df3509595d7393ec6631ef846dbebae73ca4
---
M contrib/jenkins_lc15.sh
M contrib/jenkins_oct.sh
M contrib/je
Hello Neels Hofmeyr, Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/2894
to look at the new patch set (#11).
Move common steps into common jenkins helper
Move value_string termination check and OpenBSC headers checkout into
shared jenkins help
dexter has submitted this change and it was merged.
Change subject: vty: additional nodes for sccp-address configuration
..
vty: additional nodes for sccp-address configuration
The planned sccp-addressbook implementation in lib
Patch Set 2: Code-Review-1
For me this clearly should happen by simply calling self.set_powered(False) in
Modem.cleanup(). The aim is to tear down all "things" after each test script,
and Modem.cleanup() is called after each test. Am I missing something?
--
To view, visit https://gerrit.osmoc
Hello Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/2983
to look at the new patch set (#2).
doxygen: enable AUTOBRIEF, drop \brief
Especially for short descriptions, it is annoying to have to type \brief for
every single API doc.
Drop all \b
Hello Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/2982
to look at the new patch set (#2).
doxygen: add missing gb API doc generation
Files in include/osmocom/gprs/ and src/gb/ are not included in any doxygen
generated API docs. Add Doxyfile
Patch Set 4:
(1 comment)
https://gerrit.osmocom.org/#/c/2825/4/tests/auc/gen_ts_55_205_test_sets/pdftxt_2_c.py
File tests/auc/gen_ts_55_205_test_sets/pdftxt_2_c.py:
Line 2
> since the code seems to work with python2 as well as python3, why not go fo
I remember that in an unrelated patch this ca
Hello Neels Hofmeyr, Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/2820
to look at the new patch set (#7).
gsm0808: fix AoIP speech codec element parser/generator
The implementation of the parser/generator for the speech codec
information ele
Hello Neels Hofmeyr, Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/2830
to look at the new patch set (#8).
gsm0808: add function to translate perm speech to speech codec type
The permitted speech field used in channel type element (struct
gsm
Hello Neels Hofmeyr, Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/2938
to look at the new patch set (#6).
cosmetic: Add info about coresponding codecs to permitted speech consts
The constants in enum gsm0808_permitted_speech are not very exp
Hello Neels Hofmeyr, Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/2831
to look at the new patch set (#8).
gsm0808: add function gsm0808_extrapolate_speech_codec()
The contents of the speech codec element (struct gsm0808_speech_codec),
that i
Hello Neels Hofmeyr, Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/2829
to look at the new patch set (#7).
gsm0808: add default speech codec configuration
In an AoIP scenario, the speech codec list field specifies the
codecs used in the RTP i
Patch Set 4:
I think we already have some tests disabled on FreeBSD in some project. If so,
than we can do the same here (if the tests would actually fail of course).
--
To view, visit https://gerrit.osmocom.org/2825
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType
Patch Set 1: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/2964
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I1d6cb31f38721f79e2cf93f9b8e4776f3720aa07
Gerrit-PatchSet: 1
Gerrit-Project: libosmocore
Gerrit-Branch: master
Ger
Patch Set 4: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/2958
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I70f917e05ad46049487710d1752aad2505c96696
Gerrit-PatchSet: 4
Gerrit-Project: libosmocore
Gerrit-Branch: master
Ger
Patch Set 5: Code-Review+1
--
To view, visit https://gerrit.osmocom.org/2954
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I1752cad1e34cb6e5f36edf3f538d803eff29d18c
Gerrit-PatchSet: 5
Gerrit-Project: libosmocore
Gerrit-Branch: master
Ger
Patch Set 1: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/2959
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Iced451be255cfde8a6cf38380bef71ef29673994
Gerrit-PatchSet: 1
Gerrit-Project: libosmocore
Gerrit-Branch: master
Ger
Patch Set 1: Code-Review+1
(1 comment)
https://gerrit.osmocom.org/#/c/2963/1/src/logging.c
File src/logging.c:
Line 850: name[j] = tolower((unsigned char)
info->cat[i].name[j]);
(usually no space in-between cast and casted value, in several places)
--
To view,
Patch Set 1: Code-Review-1
(2 comments)
https://gerrit.osmocom.org/#/c/2961/1/src/pseudotalloc/pseudotalloc.c
File src/pseudotalloc/pseudotalloc.c:
Line 72:while (1) {
since this is for an embedded libosmocore, it doesn't make sense to allow any
size of string. I would use a smaller le
Patch Set 1: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/2953
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I24c11226cd5bb5b6032f4ece7d1a0695dc82f0dd
Gerrit-PatchSet: 1
Gerrit-Project: libosmocore
Gerrit-Branch: master
Ger
Patch Set 1: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/2960
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I4c441b20b250c34656f1e8330d6bb4b1ce2b8423
Gerrit-PatchSet: 1
Gerrit-Project: libosmocore
Gerrit-Branch: master
Ger
Patch Set 1: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/2962
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic1e3255800999669ca9619bfceb4124c773eff2d
Gerrit-PatchSet: 1
Gerrit-Project: libosmocore
Gerrit-Branch: master
Ger
Patch Set 3: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/2957
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I3060cfee8ece86c4c89a65f93c9f5424df0c410b
Gerrit-PatchSet: 3
Gerrit-Project: libosmocore
Gerrit-Branch: master
Ger
Patch Set 1: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/2956
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I57fd3e1bf74ec24dec692710359a1ce00ecde75a
Gerrit-PatchSet: 1
Gerrit-Project: libosmocore
Gerrit-Branch: master
Ger
Patch Set 1:
(1 comment)
https://gerrit.osmocom.org/#/c/2961/1//COMMIT_MSG
Commit Message:
Line 7: pseudotalloc: Add poor implementation of talloc_asprintf
better word would be "simplistic". "poor" always means bad, we don't want to
merge poor code.
--
To view, visit https://gerrit.osmocom.
Patch Set 4:
It's not th efirst because I was first fixing the ones I got without -Wall,
then enabled -Wall and continued fixing. Should not be any issue, the important
one is -Werror which needs to be the last one.
--
To view, visit https://gerrit.osmocom.org/2958
To unsubscribe, visit https
Patch Set 1:
I agree with Neels' comments
--
To view, visit https://gerrit.osmocom.org/2961
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I61cd83d4c8570af1e452c945ac6f194baf7e6ac4
Gerrit-PatchSet: 1
Gerrit-Project: libosmocore
Gerrit-Br
Patch Set 2:
(1 comment)
https://gerrit.osmocom.org/#/c/2697/2/src/osmo_gsm_tester/resource.py
File src/osmo_gsm_tester/resource.py:
Line 122: 'origin' a log.Origin() instance which can be sued to log
information.
To sue someone means to take them to court. Ah! It's a typo! "used" lol
Pau Espin Pedrol has submitted this change and it was merged.
Change subject: lapd_test.c: Convert compilation warning to level message
..
lapd_test.c: Convert compilation warning to level message
This way we clean the build of
Review at https://gerrit.osmocom.org/2989
osmo_ortp.c: Fix warning: implicit use of function
Explicitly state that we are using an internal API from ortp, and
define the function we are using to avoid printing a warning.
Change-Id: I9cadcb31ce7ade3632d83753be97fdc9ea518b5b
---
M src/trau/osmo_
Review at https://gerrit.osmocom.org/2990
osmo_ortp.c: Fix warning: use updated ortp API
Since ortp commit 5ab29c50adc7948b11449e3aeee66010379ff7dc preceeding
ortp 0.22, the API is fixed and requests a pointer instead of an
unsigned long.
As we now require at least ortp 0.22 since libosmo-abis
Review at https://gerrit.osmocom.org/2992
configure.ac: Use -Werror in C(PP)FLAGS
Change-Id: Ic674bf28d781b9806dc023777e1fea4ee1a0257a
---
M configure.ac
1 file changed, 2 insertions(+), 2 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/libosmo-abis refs/changes/92/2992/1
diff --git a
Review at https://gerrit.osmocom.org/2991
configure.ac: Use -Wall in C(PP)FLAGS
Change-Id: I5033387e4a4206f7d34d1db3c6ede9255d199941
---
M configure.ac
1 file changed, 3 insertions(+), 0 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/libosmo-abis refs/changes/91/2991/1
diff --git a/c
Review at https://gerrit.osmocom.org/2993
sccp_scrc: Fix warning: uneeded constant evaulation
translate function is always present in the same file, thus we can
remove this check.
Fixes following warning:
warning: the address of ‘translate’ will always evaluate as ‘true’ [-Waddress]
if (tr
Review at https://gerrit.osmocom.org/2994
sccp_test_vty: Fix compilation warning: Unused variable
Instead of removing the variable, leave it there with unused attribute
set because it is an example and gives extra information to potential
readers of the function (it provides you with the actual
Review at https://gerrit.osmocom.org/2997
xua_test.c: remove useless keyword in struct declaration
Fixes following warning:
warning: useless storage class specifier in empty declaratio
Change-Id: I5cea7b6ec2cc755ad8cec69b771f6e3b297e607d
---
M tests/xua/xua_test.c
1 file changed, 1 insertion(+
Review at https://gerrit.osmocom.org/2996
stp_main.c: Add missing header
stp_main uses osmo_fsm_vty_add_cmds which is defined in /osmocom/vty/misc.h
Fixes 'implicit declaration of function' compilation warning.
Change-Id: I3cc9543fcb908eeb172d36fa472a19b01bf52c46
---
M stp/stp_main.c
1 file c
Review at https://gerrit.osmocom.org/2995
xua_internal.h: Add missing functions required by xua_test
Change-Id: Ic97de20c567bf67c513a45060e1e96e4b67b2311
---
M src/xua_internal.h
1 file changed, 5 insertions(+), 0 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/libosmo-sccp refs/change
Review at https://gerrit.osmocom.org/2999
configure.ac: Use -Werror in C(PP)FLAGS
Change-Id: Ia753a3080ef54ac248dacce5cd692fdd38f7649a
---
M configure.ac
1 file changed, 2 insertions(+), 2 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/libosmo-sccp refs/changes/99/2999/1
diff --git a
Review at https://gerrit.osmocom.org/2998
configure.ac: Use -Wall in C(PP)FLAGS
Change-Id: I1e49427b8267bbdb05d0121971a56359dbecd690
---
M configure.ac
1 file changed, 3 insertions(+), 1 deletion(-)
git pull ssh://gerrit.osmocom.org:29418/libosmo-sccp refs/changes/98/2998/1
diff --git a/co
Patch Set 7:
(1 comment)
https://gerrit.osmocom.org/#/c/2924/7/src/osmo_ss7_vty.c
File src/osmo_ss7_vty.c:
Line 1250: memcpy(ip_addr_backup,&entry->addr.ip,sizeof(entry->addr.ip));
space, space, space. Please consider instructing your editor to do this for
you, if you don't do this automa
Hello Neels Hofmeyr, Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/2924
to look at the new patch set (#8).
sccp: add addressbook functionality for sccp addresses
SCCP addresses are defined through a number of compoinents, not
just an IP-Addre
Hello Neels Hofmeyr, Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/2924
to look at the new patch set (#9).
sccp: add addressbook functionality for sccp addresses
SCCP addresses are defined through a number of compoinents, not
just an IP-Addre
Patch Set 1:
I can confirm that the same warning appears
when compiling with clang-4.0:
warning: suggest braces around initialization
of subobject [-Wmissing-braces]
--
To view, visit https://gerrit.osmocom.org/2955
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType:
Patch Set 2: Code-Review-1
I think we should drop this, avoiding any unneccessary incompatibility between
lc15 and sbts2100
--
To view, visit https://gerrit.osmocom.org/2935
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I6208a1f7d62cac
Harald Welte has submitted this change and it was merged.
Change subject: sccp: add addressbook functionality for sccp addresses
..
sccp: add addressbook functionality for sccp addresses
SCCP addresses are defined through a num
Patch Set 9: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/2924
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I068ed7f7d113dab88424a9d47bab7fc703bb7942
Gerrit-PatchSet: 9
Gerrit-Project: libosmo-sccp
Gerrit-Branch: master
Ge
Harald Welte has submitted this change and it was merged.
Change subject: cosmetic: Add info about coresponding codecs to permitted
speech consts
..
cosmetic: Add info about coresponding codecs to permitted speech consts
The c
Harald Welte has submitted this change and it was merged.
Change subject: gsm0408: update bearer capability speech version
..
gsm0408: update bearer capability speech version
3GPP TS 24.008 specifies two new speech versions for
Harald Welte has submitted this change and it was merged.
Change subject: gsm0808: add speech codec type constants
..
gsm0808: add speech codec type constants
The codec type to be set in member type in struct gsm0808_speech_cod
Harald Welte has submitted this change and it was merged.
Change subject: cosmetic: Add codec references to Speech Version Indication
..
cosmetic: Add codec references to Speech Version Indication
Add the information which GSM
Patch Set 7: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/2820
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Idabb0f9620659557672e1c6b90c75481192e5c89
Gerrit-PatchSet: 7
Gerrit-Project: libosmocore
Gerrit-Branch: master
Ger
Patch Set 7: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/2829
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I9dc0165d76a022b2c1b7418bc3133407e61b7261
Gerrit-PatchSet: 7
Gerrit-Project: libosmocore
Gerrit-Branch: master
Ger
Patch Set 8:
(1 comment)
https://gerrit.osmocom.org/#/c/2830/8/src/gsm/gsm0808_utils.c
File src/gsm/gsm0808_utils.c:
Line 653: int gsm0808_convert_to_speech_codec_type(uint8_t perm_spch)
I would call it 'gsm0808_chan_type_to_speech_codec() to make it more clear.
thanks.
--
To view, visit ht
Patch Set 8:
(1 comment)
https://gerrit.osmocom.org/#/c/2831/8/src/gsm/gsm0808_utils.c
File src/gsm/gsm0808_utils.c:
Line 691: int gsm0808_extrapolate_speech_codec(struct gsm0808_speech_codec *sc,
name: what about gsm0808_speech_codec_from_chan_type() ? I think current name
leaves it unclear f
Patch Set 11: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/2891
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ib47f5a6cc88e784c5662b0dab4ddc03ce9a35132
Gerrit-PatchSet: 11
Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerr
Harald Welte has submitted this change and it was merged.
Change subject: Move common steps into common jenkins helper
..
Move common steps into common jenkins helper
Move value_string termination check and OpenBSC headers chec
Harald Welte has submitted this change and it was merged.
Change subject: lc15: add jenkins helper
..
lc15: add jenkins helper
Add jenkins build helper for lc15 and separate header with BTS-agnostic
functions.
Change-Id: Ib47f
Patch Set 5: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/2974
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I2253df3509595d7393ec6631ef846dbebae73ca4
Gerrit-PatchSet: 5
Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit
Patch Set 11: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/2894
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic48b1f75179b9008d65219dd5a47c1ab3b886408
Gerrit-PatchSet: 11
Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerr
Harald Welte has submitted this change and it was merged.
Change subject: Use generic L1 headers helper
..
Use generic L1 headers helper
Change-Id: I2253df3509595d7393ec6631ef846dbebae73ca4
---
M contrib/jenkins_lc15.sh
M contr
Patch Set 2:
we've had this discussion before, and I remember not being convinced about it.
How does AUTOBRIEF decide which part is "brief" and which part is "verbose"?
--
To view, visit https://gerrit.osmocom.org/2983
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageT
Patch Set 2: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/2980
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I5dc22ca65740a88540dacc61c0742fede172de9c
Gerrit-PatchSet: 2
Gerrit-Project: libosmocore
Gerrit-Branch: master
Ger
Patch Set 2: Code-Review+2
slightly unrelated: we should probably disable the EXTRACT_STATIC part to avoid
documenting functions that users of the library cannot call anyway?
--
To view, visit https://gerrit.osmocom.org/2981
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-Mes
Patch Set 2: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/2982
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ieb64f497f55368e396872083237c9ff28da2dd93
Gerrit-PatchSet: 2
Gerrit-Project: libosmocore
Gerrit-Branch: master
Ger
Patch Set 2: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/2985
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I3f1cd5b10e7beccb61f370d7f8fa2666e56542cf
Gerrit-PatchSet: 2
Gerrit-Project: libosmocore
Gerrit-Branch: master
Ger
Patch Set 2:
the rationale of not changing this code is that it was inherited from the Linux
kernel, and all related whitespace changes will make it more difficult to
re-sync with a more recent kernel version of that file. So I'm a bit undecided
here...
--
To view, visit https://gerrit.osmo
Patch Set 2: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/2986
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I3ea5b88cbe9cb54702429158bf47e768e04e8fe7
Gerrit-PatchSet: 2
Gerrit-Project: libosmocore
Gerrit-Branch: master
Ger
Patch Set 2: Code-Review+2
Ok, I read
https://osmocom.org/projects/cellular-infrastructure/wiki/Guidelines_for_API_documentation
now and it is the first full stop. Works for me.
--
To view, visit https://gerrit.osmocom.org/2983
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerri
Review at https://gerrit.osmocom.org/3000
osmo_sock_get_name(): clearly indicate local and remote
In a string like
127.0.0.1:2905<->127.0.0.1:60661
it is hard to tell which is the local part. I'd have expected it on the left,
but it is actually on the right.
To avoid doubt and bypass bikes
Review at https://gerrit.osmocom.org/3001
osmo_sock_get_name(): clearly indicate local and remote
In a string like
(127.0.0.1:2905<->127.0.0.1:60661)
it is hard to tell which is the local part. I'd have expected it on the left,
but it is actually on the right.
To avoid doubt and bypass bik
Patch Set 1:
Another contestant for this change is https://gerrit.osmocom.org/3000 -- let
the votes decide which one wins.
--
To view, visit https://gerrit.osmocom.org/3001
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Id3e32de19fe07ed
Patch Set 1:
Another contestant for this change is https://gerrit.osmocom.org/3001 -- let
the votes decide which one wins.
--
To view, visit https://gerrit.osmocom.org/3000
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I43dcc6a1906429b
Patch Set 1: Code-Review+1
I prefer this one.
--
To view, visit https://gerrit.osmocom.org/3001
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Id3e32de19fe07ed5b19ee5e67494926a511266b5
Gerrit-PatchSet: 1
Gerrit-Project: libosmocore
Gerri
Hello Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/3001
to look at the new patch set (#2).
osmo_sock_get_name(): clearly indicate local and remote
In a string like
(127.0.0.1:2905<->127.0.0.1:60661)
it is hard to tell which is the local
Hello Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/3000
to look at the new patch set (#2).
osmo_sock_get_name(): clearly indicate local and remote
In a string like
127.0.0.1:2905<->127.0.0.1:60661
it is hard to tell which is the local pa
Patch Set 2:
the reason to exclude it is that this directory contains definitions taken 1:1
from related GSM specifications and thus not some APIs or data types that we
have come up with. I think it's best to not replicate what's written in 3GPP
specs. However, we can of course include it me
Patch Set 2: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/3000
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I43dcc6a1906429bd0955fd7fe2eb5b8495b592d8
Gerrit-PatchSet: 2
Gerrit-Project: libosmocore
Gerrit-Branch: master
Ger
Patch Set 1: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/2955
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I31d5ffc0cf461d207d6c3d43aa0a25a5448d7000
Gerrit-PatchSet: 1
Gerrit-Project: libosmocore
Gerrit-Branch: master
Ger
Patch Set 1: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/2999
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ia753a3080ef54ac248dacce5cd692fdd38f7649a
Gerrit-PatchSet: 1
Gerrit-Project: libosmo-sccp
Gerrit-Branch: master
Ge
Patch Set 1: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/2998
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I1e49427b8267bbdb05d0121971a56359dbecd690
Gerrit-PatchSet: 1
Gerrit-Project: libosmo-sccp
Gerrit-Branch: master
Ge
Patch Set 1: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/2996
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I3cc9543fcb908eeb172d36fa472a19b01bf52c46
Gerrit-PatchSet: 1
Gerrit-Project: libosmo-sccp
Gerrit-Branch: master
Ge
Patch Set 1: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/2995
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic97de20c567bf67c513a45060e1e96e4b67b2311
Gerrit-PatchSet: 1
Gerrit-Project: libosmo-sccp
Gerrit-Branch: master
Ge
Patch Set 1: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/2994
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Id5fddf261a7a75ee1d6e9aff0734065ebf8551ab
Gerrit-PatchSet: 1
Gerrit-Project: libosmo-sccp
Gerrit-Branch: master
Ge
Patch Set 1: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/2997
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I5cea7b6ec2cc755ad8cec69b771f6e3b297e607d
Gerrit-PatchSet: 1
Gerrit-Project: libosmo-sccp
Gerrit-Branch: master
Ge
Patch Set 1: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/2993
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I48a711f3a43496e2c0c637f34f56784f765938c4
Gerrit-PatchSet: 1
Gerrit-Project: libosmo-sccp
Gerrit-Branch: master
Ge
Harald Welte has submitted this change and it was merged.
Change subject: xua_internal.h: Add missing functions required by xua_test
..
xua_internal.h: Add missing functions required by xua_test
Change-Id: Ic97de20c567bf67c513a
Harald Welte has submitted this change and it was merged.
Change subject: stp_main.c: Add missing header
..
stp_main.c: Add missing header
stp_main uses osmo_fsm_vty_add_cmds which is defined in /osmocom/vty/misc.h
Fixes 'impl
Harald Welte has submitted this change and it was merged.
Change subject: sccp_test_vty: Fix compilation warning: Unused variable
..
sccp_test_vty: Fix compilation warning: Unused variable
Instead of removing the variable, leav
Harald Welte has submitted this change and it was merged.
Change subject: configure.ac: Use -Wall in C(PP)FLAGS
..
configure.ac: Use -Wall in C(PP)FLAGS
Change-Id: I1e49427b8267bbdb05d0121971a56359dbecd690
---
M configure.ac
1
Harald Welte has submitted this change and it was merged.
Change subject: configure.ac: Use -Werror in C(PP)FLAGS
..
configure.ac: Use -Werror in C(PP)FLAGS
Change-Id: Ia753a3080ef54ac248dacce5cd692fdd38f7649a
---
M configure.a
Harald Welte has submitted this change and it was merged.
Change subject: xua_test.c: remove useless keyword in struct declaration
..
xua_test.c: remove useless keyword in struct declaration
Fixes following warning:
warning: us
Harald Welte has submitted this change and it was merged.
Change subject: sccp_scrc: Fix warning: uneeded constant evaulation
..
sccp_scrc: Fix warning: uneeded constant evaulation
translate function is always present in the sa
Patch Set 2: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/2988
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I63d6f42567f532851e19eb8a6c842e418ce666af
Gerrit-PatchSet: 2
Gerrit-Project: libosmocore
Gerrit-Branch: master
Ger
Patch Set 2: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/2987
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ifa70e77e90462b5eb2b0457c70fd25275910c72b
Gerrit-PatchSet: 2
Gerrit-Project: libosmocore
Gerrit-Branch: master
Ger
1 - 100 of 134 matches
Mail list logo