Attention is currently required from: neels, laforge, fixeria.

dexter has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-pcu/+/31176 )

Change subject: support for Ericsson RBS E1 CCU
......................................................................


Patch Set 25:

(2 comments)

File contrib/osmo-pcu.spec.in:

https://gerrit.osmocom.org/c/osmo-pcu/+/31176/comment/8dc8da90_c294b6f0
PS24, Line 49: configure
> I think we do.
Done


File src/Makefile.am:

https://gerrit.osmocom.org/c/osmo-pcu/+/31176/comment/c9559ad8_4be29f16
PS24, Line 218: EXTRA_DIST
> agreed, c+h files should not be in EXTRA_DIST. […]
interesting. All other BTS implementations also have c+h files in EXTRA_DIST. 
Its always the sum of noinst_HEADERS and xyz_SOURCES



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

Gerrit-Project: osmo-pcu
Gerrit-Branch: master
Gerrit-Change-Id: I5c0a76667339ca984a12cbd2052f5d9e5b0f9c4d
Gerrit-Change-Number: 31176
Gerrit-PatchSet: 25
Gerrit-Owner: dexter <pma...@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <lafo...@osmocom.org>
Gerrit-CC: fixeria <vyanits...@sysmocom.de>
Gerrit-CC: neels <nhofm...@sysmocom.de>
Gerrit-Attention: neels <nhofm...@sysmocom.de>
Gerrit-Attention: laforge <lafo...@osmocom.org>
Gerrit-Attention: fixeria <vyanits...@sysmocom.de>
Gerrit-Comment-Date: Tue, 07 Mar 2023 16:38:53 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: laforge <lafo...@osmocom.org>
Comment-In-Reply-To: fixeria <vyanits...@sysmocom.de>
Gerrit-MessageType: comment

Reply via email to