<div dir="ltr"><div dir="ltr">Sean - thanks for the feedback.  All proposed changes available here --> <a href="https://github.com/spwhitton/git-remote-gcrypt/commit/9b3ad476c08237103d98ad0a3694f9595b94d9d7">https://github.com/spwhitton/git-remote-gcrypt/commit/9b3ad476c08237103d98ad0a3694f9595b94d9d7</a></div><div dir="ltr"><br></div><div>Cheers!</div><div>Jay</div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Jul 22, 2020 at 11:53 PM Sean Whitton <<a href="mailto:spwhitton@spwhitton.name">spwhitton@spwhitton.name</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hello Jay,<br>
<br>
Thank you for the patch, but unfortunately there are still some issues.<br>
<br>
On Wed 22 Jul 2020 at 11:16AM +01, Jay Colson wrote:<br>
<br>
> diff --git a/git-remote-gcrypt b/git-remote-gcrypt<br>
> index 689e025..42c3d5e 100755<br>
> --- a/git-remote-gcrypt<br>
> +++ b/git-remote-gcrypt<br>
> @@ -748,6 +750,11 @@ do_push()<br>
><br>
>       while IFS=: read -r src_ dst_ # << +src:dst<br>
>       do<br>
> +             if [ "+" = "${src_:0:1}" ]<br>
> +             then<br>
> +                     Force_passed=true<br>
> +             fi<br>
> +<br>
>               src_=${src_#+}<br>
>               filter_to ! @Refslist "$Hex40 $dst_" "$Refslist"<br>
<br>
What happens if some of the refspecs have a + but not all of them do?<br>
I think you need to check for a missing +, not the presence of one.<br>
<br>
> @@ -761,6 +768,14 @@ do_push()<br>
>  $1<br>
>  EOF<br>
><br>
> +     if [ "$Conf_force_required" = true ] && [ "$Force_passed" != true ]<br>
> +     then<br>
> +             echo_die "gcrypt.force-required is set to true.  Explicitly force push by passing --force in order to override this error."<br>
> +     elif [ "$Force_passed" != true ]<br>
> +     then<br>
> +             echo_info "An implicit --force push is occuring. Please set 'git config --global add gcrypt.force-required true' to protect from implicit force pushes."<br>
> +     fi<br>
> +<br>
>       tmp_encrypted="$Tempdir/packP"<br>
>       tmp_objlist="$Tempdir/objlP"<br>
<br>
$Force_passed should be localised to the function -- see the first few<br>
lines of do_push for how this is done.<br>
<br>
Rather than checking the value of $Force_passed twice, please nest the<br>
$Conf_force_required check inside a $Force_passed check.<br>
<br>
-- <br>
Sean Whitton<br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><a href="tel:+353866993893" target="_blank"><img src="https://signature.karma.net/signature/" style="margin: auto;"></a><br></div></div></div></div>