Re: [PATCH v5 0/4] refs: allow setting the reference directory
- From
Karthik Nayak <karthik.188@gmail.com>
- Date
- Feb 10, 2026, 13:02 UTC
- Message-ID
- <CAOLa=ZRPLB-jLJ=4cdtO0DuDD=+tPp7t-Kei5pMsF0en6i2jJg@mail.gmail.com>
- In-Reply-To
- <xmqqtsvp4wxv.fsf@gitster.g>
Junio C Hamano <gitster@pobox.com> writes:
Show 22 quoted lines
> Karthik Nayak <karthik.188@gmail.com> writes: > >> Changes in v5: >> - Moved around the commits, to ensure that the code to handle the config >> in the backend is first. Previously, we added the config first, which >> meant the commit allowed users to provide a URI but it was simply >> ignore. >> - Fix typos and grammar and rename variables. >> - Clean up the description and documentation to actually specify >> protocol over location. > > This one looked good. > >> - Avoid an extra memory allocation by detaching the strbuf value. > > So did this (thanks Stolee for spotting the opportunity). > >> - Link to v4: https://patch.msgid.link/20260202-kn-alternate-ref-dir-v4-0-3b30430411e3@gmail.com > > Replaced. Hopefully this is now ready for 'next'? > > Thanks.
Hello Junio, I don't think so, I think we still need to address few things as per the latest review. So let's hold off on it.
Karthik