Re: [GSoC Patch v2 6/7] repo: add path.grafts with absolute and relative suffix formatting
- From
K Jayatheerth <jayatheerthkulkarni2005@gmail.com>
- Date
- Jul 21, 2026, 02:19 UTC
- Message-ID
- <CA+rGoLcr7KTu5S3GN=VsV8dFF0ww0cew+4TvXtiqjGCXtVpMsQ@mail.gmail.com>
- In-Reply-To
- <xmqq33xejomv.fsf@gitster.g>
Hey Lucas and Junio,
On Mon, Jul 20, 2026 at 9:31 AM Junio C Hamano <gitster@pobox.com> wrote:
Show 24 quoted lines
> > Lucas Seiki Oshiro <lucasseikioshiro@gmail.com> writes: > > >> Introduce `path.grafts.absolute` and `path.grafts.relative` keys to > >> `git repo info`. This allows scripting layers to query the active grafts > >> context cleanly while scaling transparently with active `GIT_GRAFT_FILE` > >> environment variable overrides. > > > > I ran `git repo info path.grafts.relative` in a repository with no > > `grafts` file, and it returned `.git/info/grafts`, which obviously > > doesn't exist. > > > > Wouldn't it be better if we check if that file exists before > > returning this value? > > That is an interesting question, but I think it depends on who is > querying and for what purpose. > > If a script is asking where to write the file, then the author wants > to know where the file is supposed to be, even if no such file > exists yet. Since the file format is public, they are free to write > their own tools to manipulate it. > > Thanks.
That's tough. I think I align with Junio here.
I just used rev-parse as my compass to work on this command. Just to clarify Should I send a v3 changing something?
Regards, - K Jayatheerth