Thomas Ackermann <th.acke...@arcor.de> writes:

> Yes; in e2399e9 the following hunks where left out from the patch to 
> update-hook-example.txt:
>
> @@ -111,12 +114,12 @@ then
>  
>        info "Found matching head pattern: '$head_pattern'"
>        for user_pattern in $user_patterns; do
> -     info "Checking user: '$username' against pattern: '$user_pattern'"
> -     matchlen=$(expr "$username" : "$user_pattern")
> -     if test "$matchlen" = "${#username}"
> -     then
> -       grant "Allowing user: '$username' with pattern: '$user_pattern'"
> -     fi
> +        info "Checking user: '$username' against pattern: '$user_pattern'"
> +        matchlen=$(expr "$username" : "$user_pattern")
> +        if test "$matchlen" = "${#username}"
> +        then
> +          grant "Allowing user: '$username' with pattern: '$user_pattern'"
> +        fi
>        done
>        deny "The user is not in the access list for this branch"
>      done
> @@ -149,13 +152,13 @@ then
>  
>        info "Found matching head pattern: '$head_pattern'"
>        for group_pattern in $group_patterns; do
> -     for groupname in $groups; do
> -       info "Checking group: '$groupname' against pattern: '$group_pattern'"
> -       matchlen=$(expr "$groupname" : "$group_pattern")
> -       if test "$matchlen" = "${#groupname}"
> -       then
> -         grant "Allowing group: '$groupname' with pattern: '$group_pattern'"
> -       fi
> +        for groupname in $groups; do
> +          info "Checking group: '$groupname' against pattern: 
> '$group_pattern'"
> +          matchlen=$(expr "$groupname" : "$group_pattern")
> +          if test "$matchlen" = "${#groupname}"
> +          then
> +            grant "Allowing group: '$groupname' with pattern: 
> '$group_pattern'"
> +          fi
>          done
>        done
>        deny "None of the user's groups are in the access list for this branch"

Yuck.  That is because I almost always apply patches with whitespace
breakage fix.  The above two hunks, if taken as patches to shell
script, does nothing but adding whitespace breakages, turning tab
indent into expanded runs of spaces, and that was why the tool
dropped them.

Resurrected; will queue the result later.

Thanks.
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to