Re: [PATCH 5/5] index: offer advice for unknown index extensions

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

 



Junio C Hamano <gitster@xxxxxxxxx> writes:

> As the deployed versions of Git will keep sending the wrong message,
> I do not mind applying 1/5 and 2/5, given especially that Ben seems
> to be OK with the plan.  I however do not think 3 thru 5 is ready
> yet with this round---there were some discussions on phrasing in
> this thread.

I ran out of time looking at the surrounding code, but I think 1, 2
and 4 form a set that would give us immediate benefit to fast track
to the upcoming release.

I do not know if it makes sense to have 3 and 5 separate; I suspect
a single patch that does "clarify the warning, and allow those who
have no choice in which version of Git to choose squelch it" would
suffice.

The phrasing in 5 received a couple of good concrete suggestions
already, so it is not ready in its current form but need a bit of
wordsmithing.  I also do not think a new "trace_printf" would
particularly help.  If I stared it a lot longer, I may spot more
issues in it.

But what that step does primarily would help long after the upcoming
release and in that sense can wait a bit longer than 1, 2 & 4 (which
I am hoping can be merged in -rc1).



[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]

  Powered by Linux