On Tue, Apr 09, 2019 at 11:57:04AM +0200, David Marchand wrote:
>    On Tue, Apr 9, 2019 at 11:30 AM Bruce Richardson
>    <[1]bruce.richard...@intel.com> wrote:
> 
>      The pipefile option does not seem to exist on FreeBSD, giving an
>      error
>      when running the script.
>      Fixes: 5d9176673008 ("devtools: fix meson build test to exit on
>      failure")
>      Signed-off-by: Bruce Richardson <[2]bruce.richard...@intel.com>
>      ---
>       devtools/test-meson-builds.sh | 5 ++++-
>       1 file changed, 4 insertions(+), 1 deletion(-)
>      diff --git a/devtools/test-meson-builds.sh
>      b/devtools/test-meson-builds.sh
>      index b3b5cfbb5..1fd23fac7 100755
>      --- a/devtools/test-meson-builds.sh
>      +++ b/devtools/test-meson-builds.sh
>      @@ -7,7 +7,10 @@
>       # * if a build-directory already exists we assume it was properly
>      configured
>       # Run ninja after configuration is done.
>      -set -o pipefail
>      +# this option gives an error on BSD
>      +if [ "`uname`" == "Linux" ] ; then
>      +       set -o pipefail
>      +fi
> 
>    == is a bashism, can't you just use = ?
>    --
Yes, my bad, I'll do a v2.

/Bruce

Reply via email to