Re: [GSOC PATCH v2 2/2] builtin/prune: stop depending on 'the_repository'
- From
Ayush Chandekar <ayu.chandekar@gmail.com>
- Date
- Jul 1, 2025, 22:04 UTC
- Message-ID
- <CAE7as+ZC-sXs11YN-247fG5ikFUNxcGTgH2a8SQUaeAuUN7=YQ@mail.gmail.com>
- In-Reply-To
- <CAPSxiM8EkctHgD=to=JxwDC8L6=5xgu=vpb5=bfhQ7bL592eWQ@mail.gmail.com>
On Wed, Jul 2, 2025 at 1:14 AM Usman Akinyemi <usmanakinyemi202@gmail.com> wrote:
Show 24 quoted lines
> > On Tue, Jul 1, 2025 at 11:40 PM Ayush Chandekar <ayu.chandekar@gmail.com> wrote: > > > > On Tue, Jul 1, 2025 at 10:12 PM Junio C Hamano <gitster@pobox.com> wrote: > > > > > > Patrick Steinhardt <ps@pks.im> writes: > > > > > > > On Mon, Jun 30, 2025 at 10:11:05PM +0530, Ayush Chandekar wrote: > > > >> @@ -173,20 +171,19 @@ int cmd_prune(int argc, > > > >> expire = TIME_MAX; > > > >> save_commit_buffer = 0; > > > >> disable_replace_refs(); > > > >> - repo_init_revisions(the_repository, &revs, prefix); > > > >> + repo_init_revisions(repo, &revs, prefix); > > > > > > > > Does this work correctly when running outside of a repository? In > > > > general `cmd_prune()` is not executed and would instead die as it is > > > > declared as `RUN_SETUP`, without the `_GENTLY` suffix. But when the user > > > > asks for help we may still execute the function with a NULL pointer. > > > > > > Good eyes. "git prune -h" would safely exit in parse_options() in > > > such a case, but this part happens before the parse_options() call. > > > > Hello Ayush,
Hi Usman,
Show 10 quoted lines
> > > > Thanks for pointing that out, Patrick. Right now, `parse_options()` is > > called just after the `repo_init_revisions()`. I can move the call to > > it before this. > > > > Although when I tried running "git prune -h", it still gave me the > > expected output. > Do try this outside a repo, i.e outside a git repository and try to > run the all the test if it works. > >
Yes, I tried it. All my tests passed as well, you can check it here: https://github.com/ayu-ch/git/commits/precious-objects-3
Thanks, Ayush