diff mbox

[1/3] travis: do not print dpdk build log

Message ID 1482521000-17301-1-git-send-email-maxim.uvarov@linaro.org
State Accepted
Commit ad6219e3682d94fb8971261936ace46399ed2085
Headers show

Commit Message

Maxim Uvarov Dec. 23, 2016, 7:23 p.m. UTC
print only stderr on building dpdk to have log more
readable for odp things.

Signed-off-by: Maxim Uvarov <maxim.uvarov@linaro.org>

---
 .travis.yml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

-- 
2.7.1.250.gff4ea60

Comments

Maxim Uvarov Jan. 9, 2017, 2:25 p.m. UTC | #1
Please review.

Maxim.

On 12/23/16 22:23, Maxim Uvarov wrote:
> print only stderr on building dpdk to have log more

> readable for odp things.

> 

> Signed-off-by: Maxim Uvarov <maxim.uvarov@linaro.org>

> ---

>  .travis.yml | 2 +-

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

> 

> diff --git a/.travis.yml b/.travis.yml

> index 3283e02..cb8484f 100644

> --- a/.travis.yml

> +++ b/.travis.yml

> @@ -43,7 +43,7 @@ before_install:

>          - pushd ${TARGET}

>          - sed -ri 's,(CONFIG_RTE_LIBRTE_PMD_PCAP=).*,\1y,' .config

>          - popd

> -        - make install T=${TARGET} EXTRA_CFLAGS="-fPIC"

> +        - make install T=${TARGET} EXTRA_CFLAGS="-fPIC" > /dev/null

>          - popd

>  

>  script:

>
Mike Holmes Jan. 9, 2017, 6:20 p.m. UTC | #2
On 23 December 2016 at 14:23, Maxim Uvarov <maxim.uvarov@linaro.org> wrote:

> print only stderr on building dpdk to have log more

> readable for odp things.

>

> Signed-off-by: Maxim Uvarov <maxim.uvarov@linaro.org>

>


Reviewed-by: Mike Holmes <mike.holmes@linaro.org>



> ---

>  .travis.yml | 2 +-

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

>

> diff --git a/.travis.yml b/.travis.yml

> index 3283e02..cb8484f 100644

> --- a/.travis.yml

> +++ b/.travis.yml

> @@ -43,7 +43,7 @@ before_install:

>          - pushd ${TARGET}

>          - sed -ri 's,(CONFIG_RTE_LIBRTE_PMD_PCAP=).*,\1y,' .config

>          - popd

> -        - make install T=${TARGET} EXTRA_CFLAGS="-fPIC"

> +        - make install T=${TARGET} EXTRA_CFLAGS="-fPIC" > /dev/null

>


When this fails there will be no indication why since it will be lost to
/dev/null, but since that is for the supporting DPDK I think it is ok.




>          - popd

>

>  script:

> --

> 2.7.1.250.gff4ea60

>

>



-- 
Mike Holmes
Program Manager - Linaro Networking Group
Linaro.org <http://www.linaro.org/> *│ *Open source software for ARM SoCs
"Work should be fun and collaborative, the rest follows"
Maxim Uvarov Jan. 10, 2017, 3:04 p.m. UTC | #3
Merged,
Maxim.

On 01/09/17 21:20, Mike Holmes wrote:
> 

> 

> On 23 December 2016 at 14:23, Maxim Uvarov <maxim.uvarov@linaro.org

> <mailto:maxim.uvarov@linaro.org>> wrote:

> 

>     print only stderr on building dpdk to have log more

>     readable for odp things.

> 

>     Signed-off-by: Maxim Uvarov <maxim.uvarov@linaro.org

>     <mailto:maxim.uvarov@linaro.org>>

> 

> 

> Reviewed-by: Mike Holmes <mike.holmes@linaro.org

> <mailto:mike.holmes@linaro.org>>

>  

> 

>     ---

>      .travis.yml | 2 +-

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

> 

>     diff --git a/.travis.yml b/.travis.yml

>     index 3283e02..cb8484f 100644

>     --- a/.travis.yml

>     +++ b/.travis.yml

>     @@ -43,7 +43,7 @@ before_install:

>              - pushd ${TARGET}

>              - sed -ri 's,(CONFIG_RTE_LIBRTE_PMD_PCAP=).*,\1y,' .config

>              - popd

>     -        - make install T=${TARGET} EXTRA_CFLAGS="-fPIC"

>     +        - make install T=${TARGET} EXTRA_CFLAGS="-fPIC" > /dev/null

> 

> 

> When this fails there will be no indication why since it will be lost to

> /dev/null, but since that is for the supporting DPDK I think it is ok. 

> 

> 

>  

> 

>              - popd

> 

>      script:

>     --

>     2.7.1.250.gff4ea60

> 

> 

> 

> 

> -- 

> Mike Holmes

> Program Manager - Linaro Networking Group

> Linaro.org <http://www.linaro.org/>* **│ *Open source software for ARM SoCs

> "Work should be fun and collaborative, the rest follows"

> 

> __

> 

>
diff mbox

Patch

diff --git a/.travis.yml b/.travis.yml
index 3283e02..cb8484f 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -43,7 +43,7 @@  before_install:
         - pushd ${TARGET}
         - sed -ri 's,(CONFIG_RTE_LIBRTE_PMD_PCAP=).*,\1y,' .config
         - popd
-        - make install T=${TARGET} EXTRA_CFLAGS="-fPIC"
+        - make install T=${TARGET} EXTRA_CFLAGS="-fPIC" > /dev/null
         - popd
 
 script: