Re: [PATCH] Geolocation support
- From
Ævar Arnfjörð Bjarmason <avarab@gmail.com>
- Date
- Feb 9, 2015, 13:57 UTC
- Message-ID
- <CACBZZX5rgah7Ucek8dicaDeqvjdkR5ybBC5O7PUxE8+KBwrpgA@mail.gmail.com>
- In-Reply-To
- <CAPc5daW5jYAj5z6UMJAQLoWS4L6OH-NSvbGBMd3LO0qCV40a4A@mail.gmail.com>
On Mon, Feb 9, 2015 at 2:24 AM, Junio C Hamano <gitster@pobox.com> wrote:
> In case I was not clear, I do not think it is likely for us to accept > a patch that mucks with object header fields with this information. > Have them in the log text and let UI interpret them.
We've already told clients for a long time to ignore fields they don't know about, why would we not store what's intended to be machine-readable key-value pair data in the commit object itself, as opposed to sticking it in the log message where parsing it is always going to be a bit more tricky & distracting, since users will have to look at this arbitrary metadata when they do "git log" or "git show".