From: Marcel Krause <mk+copyleft@pimpmybyte.de>
To: git mailing list <git@vger.kernel.org>
Cc: Junio C Hamano <gitster@pobox.com>,
Marcel Krause <mk+copyleft@pimpmybyte.de>
Subject: Re: [PATCH] doc: make the gitfile syntax easier to discover
Date: Tue, 28 Nov 2023 02:51:03 +0100 [thread overview]
Message-ID: <70125a8e-57ed-2ac6-1260-2aaa10cbc851@pimpmybyte.de> (raw)
In-Reply-To: <xmqqr0kd5i66.fsf@gitster.g>
Thanks for your advice! I'll prepare a new patch.
> The title itself is sufficient, i.e. it had poor visibility, and you
> fix it by giving it better visibility.
Yeah I see now, I should have sent the explanation as cover letter,
not as commit message.
> more relevant is why you needed to find out what the former should
> be in the first place.
My gitfile links worktrees on limited-capability filesystems to a
git repo on a nicer filesystem.
^-- Should I write this in the commit message?
(For the more curious: That way I sync GRUB configs accross lots of
FAT FS thumb drives while having the main repo and main worktree on
an ext3 FS with symlinks and proper owner/group/mode for non-config
files of the project.)
> Do not unnecessary rewrap existing text, only to insert a few
> words, to force reviewers read a lot more than needed.
The shortest diff I can come up with would be:
--- a/Documentation/gitrepository-layout.txt
+++ b/Documentation/gitrepository-layout.txt
@@ -25,3 +25,4 @@ A Git repository comes in two different flavours:
your working tree, containing `gitdir: <path>` to point at the real
-directory that has the repository. This mechanism is often used for
+directory that has the repository.
+This mechanism is called a 'gitfile' and is often used for
a working tree of a submodule checkout, to allow you in the
It loses the double space and thus the sentence boundary information,
but a similar case exists in line 156 (old) so I guess it'll be ok.
> Running "git grep linkgit:gitrepository-layout" would help you find
> the right way to spell this one, I think.
Found and fixed the missing "[5]".
Maybe I'll try and teach our tests to catch that.
next prev parent reply other threads:[~2023-11-28 1:51 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-11-24 19:47 [PATCH] doc: make the gitfile syntax easier to discover Marcel Krause
2023-11-26 3:04 ` Junio C Hamano
2023-11-28 1:51 ` Marcel Krause [this message]
2023-11-28 5:55 ` Marcel Krause
2023-11-28 6:48 ` Marcel Krause
2023-11-28 6:55 ` Marcel Krause
2023-12-03 13:15 ` Junio C Hamano
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=70125a8e-57ed-2ac6-1260-2aaa10cbc851@pimpmybyte.de \
--to=mk+copyleft@pimpmybyte.de \
--cc=git@vger.kernel.org \
--cc=gitster@pobox.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).