Attention is currently required from: fixeria.
laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bts/+/34597?usp=email )
Change subject: meas: also match stderr logging for meas_test
......................................................................
Patch Set 2: Code-Review+1
(1 comment)
Patchset:
PS2:
not entirely sure if we really want to match 1:1 each character in hundreds of lines of measurement logging output. But' let's see if and how often it breaks...
--
To view, visit https://gerrit.osmocom.org/c/osmo-bts/+/34597?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: Id1ce154a65b194ce5fd9257c9c1b26d62aad1e0d
Gerrit-Change-Number: 34597
Gerrit-PatchSet: 2
Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Comment-Date: Tue, 03 Oct 2023 12:50:54 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
Attention is currently required from: Sebastian Schmidt.
laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/pysim/+/34600?usp=email )
Change subject: Add swig to Debian installation instructions
......................................................................
Patch Set 1: Code-Review-1
(1 comment)
Patchset:
PS1:
I agree. It is not our job to list dependencies of other packages (which may or may not change over time anyway). If one of our dependencies misses its upstream dependency, the problem is there.
--
To view, visit https://gerrit.osmocom.org/c/pysim/+/34600?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: I989dd8ccf8d089c784d566fe0359b85df7575fc3
Gerrit-Change-Number: 34600
Gerrit-PatchSet: 1
Gerrit-Owner: Sebastian Schmidt <yath+osmocom(a)yath.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Attention: Sebastian Schmidt <yath+osmocom(a)yath.de>
Gerrit-Comment-Date: Tue, 03 Oct 2023 12:49:17 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
Attention is currently required from: Sebastian Schmidt.
laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/pysim/+/34601?usp=email )
Change subject: Add pyshark to requirements.
......................................................................
Patch Set 1: Code-Review-1
(1 comment)
Patchset:
PS1:
as @dexter stated, we prefer to not have a hard dependency. most pysim users will likely never use pySim-trace, and hence never need that dependency.
--
To view, visit https://gerrit.osmocom.org/c/pysim/+/34601?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: Ic93e83195feb7ada3b364bae86442dae90d81d90
Gerrit-Change-Number: 34601
Gerrit-PatchSet: 1
Gerrit-Owner: Sebastian Schmidt <yath+osmocom(a)yath.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-CC: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Attention: Sebastian Schmidt <yath+osmocom(a)yath.de>
Gerrit-Comment-Date: Tue, 03 Oct 2023 12:48:09 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment