osmith has submitted this change and it was merged. ( https://gerrit.osmocom.org/12082 )
Change subject: jenkins.sh: remove leftover MGCP env variable ...................................................................... jenkins.sh: remove leftover MGCP env variable Jenkins currently runs the build twice, once with MGCP="--enable-mgcp-transcoding" and once with MGCP="--disable-mgcp-transcoding". The configure.ac does not have this parameter, so let's remove the confusing variable. This is in preparation for describing all environment variables used in jenkins.sh (follow up commit). Change-Id: I6223afef0f34d90170a691d7d5b1d9542c34c6b0 --- M contrib/jenkins.sh 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Pau Espin Pedrol: Looks good to me, approved Jenkins Builder: Verified diff --git a/contrib/jenkins.sh b/contrib/jenkins.sh index 222bd8f..b0ef7de 100755 --- a/contrib/jenkins.sh +++ b/contrib/jenkins.sh @@ -37,12 +37,12 @@ cd "$base" autoreconf --install --force -./configure $MGCP --enable-vty-tests --enable-external-tests --enable-werror +./configure --enable-vty-tests --enable-external-tests --enable-werror $MAKE $PARALLEL_MAKE LD_LIBRARY_PATH="$inst/lib" $MAKE check \ || cat-testlogs.sh LD_LIBRARY_PATH="$inst/lib" \ - DISTCHECK_CONFIGURE_FLAGS="$MGCP --enable-vty-tests --enable-external-tests" \ + DISTCHECK_CONFIGURE_FLAGS="--enable-vty-tests --enable-external-tests" \ $MAKE distcheck \ || cat-testlogs.sh -- To view, visit https://gerrit.osmocom.org/12082 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-mgw Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I6223afef0f34d90170a691d7d5b1d9542c34c6b0 Gerrit-Change-Number: 12082 Gerrit-PatchSet: 2 Gerrit-Owner: osmith <[email protected]> Gerrit-Reviewer: Jenkins Builder (1000002) Gerrit-Reviewer: Pau Espin Pedrol <[email protected]> Gerrit-Reviewer: osmith <[email protected]>
