Re: [libav-devel] [PATCH] fate.sh: add support for build-only FATE instances
On 05/20/2013 10:36 AM, Martin Storsjö wrote: From: 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. --- doc/fate.texi |1 + tests/fate.sh |1 + 2 files changed, 2 insertions(+) Fine with me, *surprise* :) Diego ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel
[libav-devel] [PATCH] fate.sh: add support for build-only FATE instances
From: 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.
---
doc/fate.texi |1 +
tests/fate.sh |1 +
2 files changed, 2 insertions(+)
diff --git a/doc/fate.texi b/doc/fate.texi
index bc6ab3e..f076acd 100644
--- a/doc/fate.texi
+++ b/doc/fate.texi
@@ -120,6 +120,7 @@ samples=/path/to/fate/samples
workdir=# directory in which to do all the work
fate_recv="ssh -T fate@@fate.libav.org" # command to submit report
comment=# optional description
+build_only= # set to "yes" for a compile-only instance that skips tests
# the following are optional and map to configure options
arch=
diff --git a/tests/fate.sh b/tests/fate.sh
index 86e4178..d2bc178 100755
--- a/tests/fate.sh
+++ b/tests/fate.sh
@@ -66,6 +66,7 @@ compile()(
)
fate()(
+test "$build_only" = "yes" && return
cd ${build} || return
${make} ${makeopts} -k fate
)
--
1.7.9.4
___
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel
Re: [libav-devel] [PATCH] fate.sh: add support for build-only FATE instances
On Sat, Oct 06, 2012 at 05:26:15PM +0100, Måns Rullgård wrote: > Diego Biurrun writes: > > If the "build_only" variable is set in the configuration file, the > > FATE client will skip running tests and just compile all targets. > > --- > > This is a slightly cleaned-up version of an RFC patch I sent earlier. > > This comes with documentation, so it should be good to go. > > Please don't say that things "should be good" when nothing of substance > has changed from a rejected version. The previous incarnation of this patch was not really rejected. You said: Let me think about this. Remind me again tomorrow. That was on September 20th last year, which turns today into the 382nd tomorrow. In the meantime everybody is running custom hacked-up fate.sh scripts for their build-only instances. Martin even sent a similar patch in the meantime, duplicating my - admittedly limited - work. This is not a change that will cause maintenance headaches going forward. Your choice of "dd" or "C-k C-k" editor commands will remove it again... Diego ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel
Re: [libav-devel] [PATCH] fate.sh: add support for build-only FATE instances
Diego Biurrun writes: > If the "build_only" variable is set in the configuration file, the > FATE client will skip running tests and just compile all targets. > --- > This is a slightly cleaned-up version of an RFC patch I sent earlier. > This comes with documentation, so it should be good to go. Please don't say that things "should be good" when nothing of substance has changed from a rejected version. -- Måns Rullgård m...@mansr.com ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel
[libav-devel] [PATCH] fate.sh: add support for build-only FATE instances
If the "build_only" variable is set in the configuration file, the
FATE client will skip running tests and just compile all targets.
---
This is a slightly cleaned-up version of an RFC patch I sent earlier.
This comes with documentation, so it should be good to go.
doc/fate.texi |1 +
tests/fate.sh |1 +
2 files changed, 2 insertions(+), 0 deletions(-)
diff --git a/doc/fate.texi b/doc/fate.texi
index 975f40a..5b08d9a 100644
--- a/doc/fate.texi
+++ b/doc/fate.texi
@@ -118,6 +118,7 @@ samples=/path/to/fate/samples
workdir=# directory in which to do all the work
fate_recv="ssh -T fate@@fate.libav.org" # command to submit report
comment=# optional description
+build_only= # set to "yes" for a compile-only instance that skips tests
# the following are optional and map to configure options
arch=
diff --git a/tests/fate.sh b/tests/fate.sh
index 86e4178..d2bc178 100755
--- a/tests/fate.sh
+++ b/tests/fate.sh
@@ -66,6 +66,7 @@ compile()(
)
fate()(
+test "$build_only" = "yes" && return
cd ${build} || return
${make} ${makeopts} -k fate
)
--
1.7.1
___
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel
