<div dir="ltr"><div dir="ltr"><div dir="auto">Hi Sean,<div dir="auto"><br></div><div dir="auto">> I already discussed a very similar patch with one of your colleagues, I believe.</div><div dir="auto"><br></div><div dir="auto">Oh nice so he did submit. We weren't sure.</div><div dir="auto">We improve upon the previous version with several bugfixes, the test suite and the documentation. We also tested everything against a GitLab server.</div><div dir="auto"><br></div><div dir="auto">> We concluded that the pre-receive hook and the commit history</div><div dir="auto">> in the commit message did not achieve its intended purposes, once you</div><div dir="auto">> think carefully about exactly what is trusting what.</div><div dir="auto"> </div><div dir="auto">Would you be so kind and provide a log or summary of your discussion so that we can properly respond? Unfortunately we do not have a way to contact the mentioned colleague.</div><div dir="auto"><br></div><div dir="auto">Best,</div><div dir="auto">Tobias</div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, 16 Sept 2022, 02:01 Sean Whitton, <<a href="mailto:spwhitton@spwhitton.name" target="_blank">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 Tobias,<br>
<br>
On Thu 15 Sep 2022 at 04:40PM +02, Tobias Birchler wrote:<br>
<br>
> We implemented an additional feature for git-remote-gcrypt. The attached patch<br>
> adds the (optional) functionality of maintaining a commit history in the commit<br>
> message.<br>
<br>
Thank you for the patch.<br>
<br>
I already discussed a very similar patch with one of your colleagues, I<br>
believe.  We concluded that the pre-receive hook and the commit history<br>
in the commit message did not achieve its intended purposes, once you<br>
think carefully about exactly what is trusting what.<br>
<br>
Could you explain how your patch is different, and how it overcomes<br>
those difficulties, please?<br>
<br>
> Furthermore it adds information to the README about the additional<br>
> functionality<br>
> and other things that were unclear to us.<br>
<br>
If you could prepare these as separate patch(es) we can get them applied<br>
in parallel with the above discussion.<br>
<br>
-- <br>
Sean Whitton<br>
</blockquote></div>
</div>