fixeria has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/libosmo-gprs/+/29683 )


Change subject: tests/rlcmac/ts_44_018_test: do not depend on libosmogsm
......................................................................

tests/rlcmac/ts_44_018_test: do not depend on libosmogsm

Header file <osmocom/gsm/protocol/gsm_04_08.h> belongs to libosmogsm.
However, file 'contrib/libosmo-gprs.spec.in' does not list this lib
as a build dependency.  So building the RPM package currently fails.

We could have listed libosmogsm as 'BuildRequires', but given that
the only thing we use from this lib is the GSM_MACBLOCK_LEN macro,
there is no real need to depend on it.

Change-Id: Ieac350e7a05965c64e8ccf8dd17a569f0afd9514
---
M tests/rlcmac/ts_44_018_test.c
1 file changed, 1 insertion(+), 2 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/libosmo-gprs refs/changes/83/29683/1

diff --git a/tests/rlcmac/ts_44_018_test.c b/tests/rlcmac/ts_44_018_test.c
index dd6f4f7..d1cc170 100644
--- a/tests/rlcmac/ts_44_018_test.c
+++ b/tests/rlcmac/ts_44_018_test.c
@@ -23,7 +23,6 @@
 #include <osmocom/core/msgb.h>

 #include <osmocom/csn1/csn1.h>
-#include <osmocom/gsm/protocol/gsm_04_08.h>
 #include <osmocom/gprs/rlcmac/gprs_rlcmac.h>

 static void *tall_ctx = NULL;
@@ -41,7 +40,7 @@
        };

        for (unsigned int i = 0; i < ARRAY_SIZE(testData); i++) {
-               uint8_t buf[GSM_MACBLOCK_LEN - 3];
+               uint8_t buf[20]; /* GSM_MACBLOCK_LEN - 3 */
                SI_13_t si13ro = { 0 };
                int rc;


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

Gerrit-Project: libosmo-gprs
Gerrit-Branch: master
Gerrit-Change-Id: Ieac350e7a05965c64e8ccf8dd17a569f0afd9514
Gerrit-Change-Number: 29683
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <vyanits...@sysmocom.de>
Gerrit-MessageType: newchange

Reply via email to