Attention is currently required from: osmith, pespin.
dexter has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bsc/+/34192 )
Change subject: pcuif_proto: use confirmation flag in struct gsm_pcu_if_pch
......................................................................
Patch Set 1:
(1 comment)
Patchset:
PS1:
> Same as I said on the other patch: I'm not liking this approach. […]
I am not happy with this either. I will rename the "confirmed_imm_ass" to "confirm" and add some logic to osmo-bsc to make the behavior look like the one on osmo-bts as good as possible. I already have an idea.
--
To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/34192
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: I3d2842626b7e8325860ea3160c7d900d39e953a0
Gerrit-Change-Number: 34192
Gerrit-PatchSet: 1
Gerrit-Owner: dexter <pmaier(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: osmith <osmith(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Fri, 25 Aug 2023 08:41:01 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: comment
Attention is currently required from: laforge.
dexter has posted comments on this change. ( https://gerrit.osmocom.org/c/pysim/+/34154 )
Change subject: sim-rest-server: fix REST method info
......................................................................
Patch Set 1:
(1 comment)
Patchset:
PS1:
> I'm not entirely sure this is an improvement. […]
I agree with that, but I am not sure how much effort this would be.
Quickly looking at pySim-shellp.py I would say we need a runtime state (rs) object and a profile (see init_card()) Then we could use the select method in lchan[0].
--
To view, visit https://gerrit.osmocom.org/c/pysim/+/34154
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: Ib0178823abb18187404249cfed71cfb3123d1d74
Gerrit-Change-Number: 34154
Gerrit-PatchSet: 1
Gerrit-Owner: dexter <pmaier(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>
Gerrit-Attention: laforge <laforge(a)osmocom.org>
Gerrit-Comment-Date: Fri, 25 Aug 2023 08:08:58 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: laforge <laforge(a)osmocom.org>
Gerrit-MessageType: comment
pespin has uploaded this change for review. ( https://gerrit.osmocom.org/c/docker-playground/+/34200 )
Change subject: debian-*-erlang: Install on libsctp-dev to enable sctp support in erlang apps
......................................................................
debian-*-erlang: Install on libsctp-dev to enable sctp support in erlang apps
The current "--enable-sctp" flag when building erlang consists of
supporting SCTP if the libsctp library is found at runtime, by
dynamically linking to it only when SCTP features are requested.
The erlang package in debian only lists libsctp as a Build-Depends, but
it is not installed by default, so apps wishing to use SCTP will fail:
"""
[error] CRASH REPORT Process <0.155.0> with 0 neighbours crashed with reason: no match of right hand value {error,{shutdown,{open,eprotonosupport}
"""
Installing the libsctp library fixes the issue.
Related: OS#5757
Change-Id: I5ca332f9064c16491116608577b3f68379d840e6
---
M debian-bookworm-erlang/Dockerfile
M debian-bullseye-erlang/Dockerfile
2 files changed, 24 insertions(+), 0 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/docker-playground refs/changes/00/34200/1
diff --git a/debian-bookworm-erlang/Dockerfile b/debian-bookworm-erlang/Dockerfile
index 0c13fd5..1ccc6cb 100644
--- a/debian-bookworm-erlang/Dockerfile
+++ b/debian-bookworm-erlang/Dockerfile
@@ -46,6 +46,7 @@
apt-get update && \
apt-get install -y --no-install-recommends \
libpcap-dev \
+ libsctp-dev \
&& \
apt-get clean
diff --git a/debian-bullseye-erlang/Dockerfile b/debian-bullseye-erlang/Dockerfile
index 9477ebc..de8a111 100644
--- a/debian-bullseye-erlang/Dockerfile
+++ b/debian-bullseye-erlang/Dockerfile
@@ -37,6 +37,7 @@
RUN apt-get update && \
apt-get -y install \
libpcap-dev \
+ libsctp-dev \
&& \
apt-get clean
--
To view, visit https://gerrit.osmocom.org/c/docker-playground/+/34200
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: docker-playground
Gerrit-Branch: master
Gerrit-Change-Id: I5ca332f9064c16491116608577b3f68379d840e6
Gerrit-Change-Number: 34200
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: newchange