Re: [PATCH] Geolocation support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Ævar Arnfjörð Bjarmason <avarab@xxxxxxxxx> writes:

> On Mon, Feb 9, 2015 at 2:24 AM, Junio C Hamano <gitster@xxxxxxxxx> 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, 

Yes, but the reason that mechanism is there is not because we want
to add random cruft Git does not have to know about.  It is to avoid
older Git from suddenly stop working when we really need to add new
essential things.

This was discussed in great length on this list already.

cf. http://thread.gmane.org/gmane.comp.version-control.git/138848/focus=138852

More importantly, adding non-essential stuff left and right will
force third party Git reimplementations to pay attention to them and
also will leave room for them to make mistakes when deciding what to
propagate, what to drop and what to update when rewriting commits
via rebase, cherry-pick, etc.

> why would we not store what's intended to be
> machine-readable key-value pair data in the commit object itself,

I think this sentence gets it backwards.  The question to ask is if
it is an arbitrary cruft that the end users are better off if they
can easily typofix in the commit message log editor, or is it
essential for Git to operate correctly and end users shouldn't be
allowed to muck with in the editor?

>> The expected location format is "CITY, COUNTRY (LAT, LON)".

I would expect that I can typofix "Les Angeles" to "Los Angeles",
if I were using this feature.
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]