[FFmpeg-devel] [PATCH 1/2] tool/bisect: add some magic to make the bisect tool be useable when its not part of a checkout
eclipse7 at gmx.net
Tue Jul 3 22:00:41 CEST 2012
Michael Niedermayer wrote:
> diff --git a/tools/bisect b/tools/bisect
> index de43f68..cdf3226 100755
> --- a/tools/bisect
> +++ b/tools/bisect
> @@ -2,6 +2,23 @@
> set -e
> +if test "bisect" = "`basename $0`" ; then
> + echo WARNING, trying to execute tools/bisect directly this cannot work as
> + echo the script itself would not be available in older checkouts
> + echo please use tools/bisec
> + git show master:tools/bisect > tools/bisec
> + chmod u+x tools/bisec
> + exit 1
The distinction bisect/bisec seems a bit too subtle too me. Especially as
the latter looks like a typo at first.
Would it be a solution to rename the checked in file into bisect-create?
In that case the warning message could be replaced by
While thinking more about it naming the created file bisect could cause
trouble for the few revisions where tools/bisect is part of the source.
More information about the ffmpeg-devel