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;