On Sun, Jul 29, 2007 at 12:50:18AM -0700, Eric Lesh wrote:
> Make guilt-push respect guards. Also teach guilt-header, guilt-next,
> and guilt-unapplied to grok patches that are skipped because of
> guards.
>
> Signed-off-by: Eric Lesh <eclesh@xxxxxxxx>
> ---
> guilt-header | 7 ++++---
> guilt-next | 2 +-
> guilt-push | 8 ++++----
> guilt-unapplied | 2 +-
> 4 files changed, 10 insertions(+), 9 deletions(-)
>
> diff --git a/guilt-header b/guilt-header
> index d07e2be..ef7f55e 100755
> --- a/guilt-header
> +++ b/guilt-header
> @@ -15,15 +15,16 @@ patch="$1"
> if [ -z "$patch" ]; then
> # use the patch that's on the top of the stack
>
> - eidx=`wc -l < $applied`
> - if [ $eidx -eq 0 ]; then
> + patch=`get_top`
> + if [ -z "$patch" ]; then
> die "Status file is empty"
> fi
> + eidx=`get_series | grep -ne "^$patch\$" | cut -d: -f1`
> else
> # use the specified patch
>
> eidx=`get_series | grep -ne "^$patch\$" | cut -d: -f1`
> - if [ $eidx -eq 0 ]; then
> + if [ -z "$eidx" ]; then
> die "Patch $patch is not in the series"
> fi
> fi
> diff --git a/guilt-next b/guilt-next
> index f38f1cc..38f57fa 100755
> --- a/guilt-next
> +++ b/guilt-next
> @@ -13,5 +13,5 @@ fi
> n=`wc -l < $applied`
> n=$(($n + 1))
>
> -get_series | awk "{ if (NR == $n) print \$0}"
> +get_guarded_series | awk "{ if (NR == $n) print \$0}"
Seeing this almost makes me thing that get_series should give you the
guarded series unless you poke it the right way...something like:
get_series --full
or
get_full_series
The guarded series is what most commands care about, right?
> diff --git a/guilt-push b/guilt-push
> index ad3616b..ce928e3 100755
> --- a/guilt-push
> +++ b/guilt-push
> @@ -24,7 +24,7 @@ if [ "$patch" = "--all" ] || [ "$patch" = "-a" ]; then
> # we are supposed to push all patches, get the last one out of
> # series
>
> - eidx=`get_series | wc -l`
> + eidx=`get_guarded_series | wc -l`
> if [ $eidx -eq 0 ]; then
> die "There are no patches to push"
> fi
> @@ -37,9 +37,9 @@ else
> # we're supposed to push only up to a patch, make sure the patch is
> # in the series
>
> - eidx=`get_series | grep -ne "^$patch\$" | cut -d: -f1`
> + eidx=`get_guarded_series | grep -ne "^$patch\$" | cut -d: -f1`
> if [ -z "$eidx" ]; then
> - die "Patch $patch is not in the series"
> + die "Patch $patch is not in the series or is guarded"
> fi
> fi
>
> @@ -52,7 +52,7 @@ fi
> sidx=`wc -l < $applied`
> sidx=`expr $sidx + 1`
>
> -get_series | sed -n -e "${sidx},${eidx}p" | while read p
> +get_guarded_series | sed -n -e "${sidx},${eidx}p" | while read p
> do
> echo "Applying patch..$p"
> if [ ! -f "$GUILT_DIR/$branch/$p" ]; then
> diff --git a/guilt-unapplied b/guilt-unapplied
> index 192a7e5..6904360 100755
> --- a/guilt-unapplied
> +++ b/guilt-unapplied
> @@ -13,4 +13,4 @@ fi
> n=`wc -l < $applied`
> n=`expr $n + 1`
>
> -get_series | sed -n -e "$n,\$p"
> +get_guarded_series | sed -n -e "$n,\$p"
> --
> 1.5.2
--
Debugging is twice as hard as writing the code in the first place.
Therefore, if you write the code as cleverly as possible, you are, by
definition, not smart enough to debug it.
- Brian W. Kernighan
-
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
