Re: [libav-devel] [PATCH 5/5] fate.sh: add support for build-only FATE instances

2012-10-04 Thread Martin Storsjö 

On Tue, 20 Sep 2011, Måns Rullgård wrote:


Diego Biurrun di...@biurrun.de writes:


If the build_only variable is set in the configuration file, the FATE
client will skip running tests and just compile all targets.
---
 tests/fate.sh |1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/tests/fate.sh b/tests/fate.sh
index b0df4df..7b1823c 100755
--- a/tests/fate.sh
+++ b/tests/fate.sh
@@ -65,6 +65,7 @@ compile()(
 )

 fate()(
+test -n $build_only  return
 cd ${build} || return
 ${make} ${makeopts} -k fate
 )
--


Let me think about this.  Remind me again tomorrow.


Ping - something like this would be good to get in at one point or 
another.


// Martin___
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

[libav-devel] [PATCH 5/5] fate.sh: add support for build-only FATE instances

2011-09-20 Thread Diego Biurrun 
If the build_only variable is set in the configuration file, the FATE
client will skip running tests and just compile all targets.
---
 tests/fate.sh |1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/tests/fate.sh b/tests/fate.sh
index b0df4df..7b1823c 100755
--- a/tests/fate.sh
+++ b/tests/fate.sh
@@ -65,6 +65,7 @@ compile()(
 )
 
 fate()(
+test -n $build_only  return
 cd ${build} || return
 ${make} ${makeopts} -k fate
 )
-- 
1.7.3.4

___
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Re: [libav-devel] [PATCH 5/5] fate.sh: add support for build-only FATE instances

2011-09-20 Thread Måns Rullgård 
Diego Biurrun di...@biurrun.de writes:

 If the build_only variable is set in the configuration file, the FATE
 client will skip running tests and just compile all targets.
 ---
  tests/fate.sh |1 +
  1 files changed, 1 insertions(+), 0 deletions(-)

 diff --git a/tests/fate.sh b/tests/fate.sh
 index b0df4df..7b1823c 100755
 --- a/tests/fate.sh
 +++ b/tests/fate.sh
 @@ -65,6 +65,7 @@ compile()(
  )

  fate()(
 +test -n $build_only  return
  cd ${build} || return
  ${make} ${makeopts} -k fate
  )
 -- 

Let me think about this.  Remind me again tomorrow.

-- 
Måns Rullgård
m...@mansr.com
___
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Re: [libav-devel] [PATCH 5/5] fate.sh: add support for build-only FATE instances

2011-09-20 Thread Diego Biurrun 
On Tue, Sep 20, 2011 at 04:14:29PM +0100, Måns Rullgård wrote:
 Diego Biurrun di...@biurrun.de writes:
 
  If the build_only variable is set in the configuration file, the FATE
  client will skip running tests and just compile all targets.
 
  --- a/tests/fate.sh
  +++ b/tests/fate.sh
  @@ -65,6 +65,7 @@ compile()(
 
   fate()(
  +test -n $build_only  return
   cd ${build} || return
   ${make} ${makeopts} -k fate
   )
 
 Let me think about this.  Remind me again tomorrow.

To elaborate slightly: This should maybe had an RFC tag.  It's the
minimally intrusive way to implement this.  It could also check if
build_only is set to a specific value (yes comes to mind) and
not just set to any value.

Diego
___
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel