test: tm: use script instead of binary when calling traffic manager test

Message ID 1492646405-32739-1-git-send-email-odpbot@yandex.ru
State New
Headers show

Commit Message

Github ODP bot April 20, 2017, midnight
From: Dmitry Eremin-Solenikov <dmitry.ereminsolenikov@linaro.org>


Since 51e3b8776b78180741fa57a621f9d13b9ae8bbfb tm test received wrapper
script checking if it is run under Travis CI. However linux-generic
testsuite was not updated to call script instead of binary, resulting in
test failures.

Signed-off-by: Dmitry Eremin-Solenikov <dmitry.ereminsolenikov@linaro.org>

---
/** Email created from pull request 15 (lumag:fix-tmgr)
 ** https://github.com/Linaro/odp/pull/15
 ** Patch: https://github.com/Linaro/odp/pull/15.patch
 ** Base sha: 5622f689653013b317842386a1aed81dd81fe749
 ** Merge commit sha: d52fad48d4e87f9a79fadb0d6dc217f5bed9b187
 **/
 test/linux-generic/Makefile.am | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Bill Fischofer April 20, 2017, 4:09 p.m. | #1
On Wed, Apr 19, 2017 at 7:00 PM, Github ODP bot <odpbot@yandex.ru> wrote:

> From: Dmitry Eremin-Solenikov <dmitry.ereminsolenikov@linaro.org>

>

> Since 51e3b8776b78180741fa57a621f9d13b9ae8bbfb tm test received wrapper

> script checking if it is run under Travis CI. However linux-generic

> testsuite was not updated to call script instead of binary, resulting in

> test failures.

>

> Signed-off-by: Dmitry Eremin-Solenikov <dmitry.ereminsolenikov@linaro.org>

>


Reviewed-by: Bill Fischofer <bill.fischofer@linaro.org>



> ---

> /** Email created from pull request 15 (lumag:fix-tmgr)

>  ** https://github.com/Linaro/odp/pull/15

>  ** Patch: https://github.com/Linaro/odp/pull/15.patch

>  ** Base sha: 5622f689653013b317842386a1aed81dd81fe749

>  ** Merge commit sha: d52fad48d4e87f9a79fadb0d6dc217f5bed9b187

>  **/

>  test/linux-generic/Makefile.am | 2 +-

>  1 file changed, 1 insertion(+), 1 deletion(-)

>

> diff --git a/test/linux-generic/Makefile.am b/test/linux-generic/Makefile.

> am

> index 0522550..38516a4 100644

> --- a/test/linux-generic/Makefile.am

> +++ b/test/linux-generic/Makefile.am

> @@ -30,7 +30,7 @@ TESTS = validation/api/pktio/pktio_run.sh \

>         $(ALL_API_VALIDATION_DIR)/thread/thread_main$(EXEEXT) \

>         $(ALL_API_VALIDATION_DIR)/time/time_main$(EXEEXT) \

>         $(ALL_API_VALIDATION_DIR)/timer/timer_main$(EXEEXT) \

> -       $(ALL_API_VALIDATION_DIR)/traffic_mngr/traffic_mngr_main$(EXEEXT)

> \

> +       $(ALL_API_VALIDATION_DIR)/traffic_mngr/traffic_mngr.sh \

>         $(ALL_API_VALIDATION_DIR)/shmem/shmem_main$(EXEEXT) \

>         $(ALL_API_VALIDATION_DIR)/system/system_main$(EXEEXT) \

>         ring/ring_main$(EXEEXT)

>

>
Maxim Uvarov April 24, 2017, 4:35 p.m. | #2
That was merged,
Maxim.

On 20.04.2017 19:09, Bill Fischofer wrote:
> On Wed, Apr 19, 2017 at 7:00 PM, Github ODP bot <odpbot@yandex.ru> wrote:

> 

>> From: Dmitry Eremin-Solenikov <dmitry.ereminsolenikov@linaro.org>

>>

>> Since 51e3b8776b78180741fa57a621f9d13b9ae8bbfb tm test received wrapper

>> script checking if it is run under Travis CI. However linux-generic

>> testsuite was not updated to call script instead of binary, resulting in

>> test failures.

>>

>> Signed-off-by: Dmitry Eremin-Solenikov <dmitry.ereminsolenikov@linaro.org>

>>

> 

> Reviewed-by: Bill Fischofer <bill.fischofer@linaro.org>

> 

> 

>> ---

>> /** Email created from pull request 15 (lumag:fix-tmgr)

>>  ** https://github.com/Linaro/odp/pull/15

>>  ** Patch: https://github.com/Linaro/odp/pull/15.patch

>>  ** Base sha: 5622f689653013b317842386a1aed81dd81fe749

>>  ** Merge commit sha: d52fad48d4e87f9a79fadb0d6dc217f5bed9b187

>>  **/

>>  test/linux-generic/Makefile.am | 2 +-

>>  1 file changed, 1 insertion(+), 1 deletion(-)

>>

>> diff --git a/test/linux-generic/Makefile.am b/test/linux-generic/Makefile.

>> am

>> index 0522550..38516a4 100644

>> --- a/test/linux-generic/Makefile.am

>> +++ b/test/linux-generic/Makefile.am

>> @@ -30,7 +30,7 @@ TESTS = validation/api/pktio/pktio_run.sh \

>>         $(ALL_API_VALIDATION_DIR)/thread/thread_main$(EXEEXT) \

>>         $(ALL_API_VALIDATION_DIR)/time/time_main$(EXEEXT) \

>>         $(ALL_API_VALIDATION_DIR)/timer/timer_main$(EXEEXT) \

>> -       $(ALL_API_VALIDATION_DIR)/traffic_mngr/traffic_mngr_main$(EXEEXT)

>> \

>> +       $(ALL_API_VALIDATION_DIR)/traffic_mngr/traffic_mngr.sh \

>>         $(ALL_API_VALIDATION_DIR)/shmem/shmem_main$(EXEEXT) \

>>         $(ALL_API_VALIDATION_DIR)/system/system_main$(EXEEXT) \

>>         ring/ring_main$(EXEEXT)

>>

>>

Patch hide | download patch | download mbox

diff --git a/test/linux-generic/Makefile.am b/test/linux-generic/Makefile.am
index 0522550..38516a4 100644
--- a/test/linux-generic/Makefile.am
+++ b/test/linux-generic/Makefile.am
@@ -30,7 +30,7 @@  TESTS = validation/api/pktio/pktio_run.sh \
 	$(ALL_API_VALIDATION_DIR)/thread/thread_main$(EXEEXT) \
 	$(ALL_API_VALIDATION_DIR)/time/time_main$(EXEEXT) \
 	$(ALL_API_VALIDATION_DIR)/timer/timer_main$(EXEEXT) \
-	$(ALL_API_VALIDATION_DIR)/traffic_mngr/traffic_mngr_main$(EXEEXT) \
+	$(ALL_API_VALIDATION_DIR)/traffic_mngr/traffic_mngr.sh \
 	$(ALL_API_VALIDATION_DIR)/shmem/shmem_main$(EXEEXT) \
 	$(ALL_API_VALIDATION_DIR)/system/system_main$(EXEEXT) \
 	ring/ring_main$(EXEEXT)