diff mbox

[ovs-dev] travis: Install `bc` utility for kernel compilation

Message ID 1443549326-945-1-git-send-email-diproiettod@vmware.com
State Accepted
Headers show

Commit Message

Daniele Di Proietto Sept. 29, 2015, 5:55 p.m. UTC
Newer kernels appear to require `bc` to build all the headers

Also, alphabetize the package list

Tested-at: https://travis-ci.org/ddiproietto/ovs/builds/82757574
Signed-off-by: Daniele Di Proietto <diproiettod@vmware.com>
---
 .travis.yml | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

Comments

Ben Pfaff Sept. 29, 2015, 5:56 p.m. UTC | #1
On Tue, Sep 29, 2015 at 06:55:26PM +0100, Daniele Di Proietto wrote:
> Newer kernels appear to require `bc` to build all the headers
> 
> Also, alphabetize the package list
> 
> Tested-at: https://travis-ci.org/ddiproietto/ovs/builds/82757574
> Signed-off-by: Daniele Di Proietto <diproiettod@vmware.com>

Acked-by: Ben Pfaff <blp@nicira.com>
Daniele Di Proietto Sept. 29, 2015, 6:05 p.m. UTC | #2
On 29/09/2015 18:56, "Ben Pfaff" <blp@nicira.com> wrote:

>On Tue, Sep 29, 2015 at 06:55:26PM +0100, Daniele Di Proietto wrote:
>> Newer kernels appear to require `bc` to build all the headers
>> 
>> Also, alphabetize the package list
>> 
>> Tested-at: 
>>https://urldefense.proofpoint.com/v2/url?u=https-3A__travis-2Dci.org_ddip
>>roietto_ovs_builds_82757574&d=BQIBAg&c=Sqcl0Ez6M0X8aeM67LKIiDJAXVeAw-YihV
>>MNtXt-uEs&r=SmB5nZacmXNq0gKCC1s_Cw5yUNjxgD4v5kJqZ2uWLlE&m=fzJ44S-4cavyffw
>>FJ_iYPpICyVwiLs_tgkzVgMwp8GI&s=s1CHl9sphSmCS0M_PrAplwAgqjAGjXeAb2sz_EDXVW
>>w&e= 
>> Signed-off-by: Daniele Di Proietto <diproiettod@vmware.com>
>
>Acked-by: Ben Pfaff <blp@nicira.com>

Thanks! I hope it's the last round...

Applied to master
diff mbox

Patch

diff --git a/.travis.yml b/.travis.yml
index 1c11bb5..30b8713 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -6,9 +6,10 @@  compiler:
 addons:
   apt:
     packages:
+      - bc
+      - gcc-multilib
       - libssl-dev
       - llvm-dev
-      - gcc-multilib
 
 before_install: ./.travis/prepare.sh