diff mbox

[ovs-dev,2/2] travis: Fail the build if any of the Linux build preparations fail.

Message ID 20170727204106.7772-2-blp@ovn.org
State Accepted
Headers show

Commit Message

Ben Pfaff July 27, 2017, 8:41 p.m. UTC
We want the build to fail if we can't prepare properly for it, but
linux-prepare.sh ignored errors.  This fixes the problem.

This would have made it more obvious where the problem fixed by the
previous commit originated.

(osx-prepare.sh already does the right thing.)

Signed-off-by: Ben Pfaff <blp@ovn.org>
---
 .travis/linux-prepare.sh | 2 ++
 1 file changed, 2 insertions(+)

Comments

Justin Pettit July 27, 2017, 9:17 p.m. UTC | #1
> On Jul 27, 2017, at 1:41 PM, Ben Pfaff <blp@ovn.org> wrote:
> 
> We want the build to fail if we can't prepare properly for it, but
> linux-prepare.sh ignored errors.  This fixes the problem.
> 
> This would have made it more obvious where the problem fixed by the
> previous commit originated.
> 
> (osx-prepare.sh already does the right thing.)
> 
> Signed-off-by: Ben Pfaff <blp@ovn.org>

Acked-by: Justin Pettit <jpettit@ovn.org>

--Justin
diff mbox

Patch

diff --git a/.travis/linux-prepare.sh b/.travis/linux-prepare.sh
index b73088d4a132..3295a5b5c7fb 100755
--- a/.travis/linux-prepare.sh
+++ b/.travis/linux-prepare.sh
@@ -1,5 +1,7 @@ 
 #!/bin/bash
 
+set -ev
+
 # Build and install sparse.
 # 
 # Explicitly disable sparse support for llvm because some travis