<div dir="ltr"><div dir="ltr">I've prepared a revised patch that includes a new test script that, when run, verifies the basic functionality of git-remote-gcrypt. It's not the most comprehensive test suite every made (to put it generously), but it generally persuades me that the patch works correctly.</div><div dir="ltr"><br></div><div>I'm going to send the revised patch as a reply to this email.</div><div><br></div><div>Best,</div><div>Cathy Fitzpatrick</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Dec 15, 2023 at 9:17 AM 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,<br>
<br>
On Fri 15 Dec 2023 at 04:27am -08, Cathy J. Fitzpatrick wrote:<br>
<br>
> The standard `git-config(1)` setting of `pack.packSizeLimit` is used to<br>
> specify the maximum size of a pack file when repacking a repository.<br>
> This setting is important when working with a remote git hosting<br>
> provider that imposes a maximum file size on files stored on the remote<br>
> server. For example, GitHub currently imposes a maximum size of 100 MiB<br>
> per file stored on its servers.<br>
><br>
> Until now, gpg-remote-gcrypt has ignored the `pack.packSizeLimit`<br>
> setting when repacking the repository. This setting has been ignored<br>
> because gpg-remote-gcrypt has supplied the `--stdout` flag to<br>
> `gpg-repack(1)`, and that flag implicitly causes `gpg-repack(1)` to<br>
> ignore the value of `pack.packSizeLimit`.<br>
><br>
> This patch modifies gpg-remote-gcrypt so that it will respect the value<br>
> of `pack.packSizeLimit`. This is achieved by modifying the invocation of<br>
> `gpg-repack(1)` so that the `--stdout` argument is not supplied.<br>
> Instead, the pack files are written to the same temporary directory that<br>
> gpg-remote-gcrypt already uses for other purposes.<br>
<br>
Thanks.  I'd like to apply this.<br>
<br>
Could you give me some idea of the testing that you've done?<br>
<br>
-- <br>
Sean Whitton<br>
</blockquote></div></div>