Re: [PATCH v2 0/9] sparse-checkout: make cone mode the default

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

 



Junio C Hamano <gitster@xxxxxxxxx> writes:

> "Elijah Newren via GitGitGadget" <gitgitgadget@xxxxxxxxx> writes:
>
>> == Updates Log ==
>>
>> Changes since v1:
>>
>>  * rebased on master, to remove dependence on en/present-despite-skipped
>>    (which has since merged to master). Earlier version of series wasn't
>>    picked up anyway, so rebasing should be safe.
>>  * Wording and code style tweaks suggested by Stolee in his review
>>
>> == Overview ==
>>
>> This patch changes the default mode for sparse-checkout from non-cone mode
>> to cone-mode, and marks non-cone mode as deprecated. There is no plan to
>> remove non-cone mode, we are merely recommending against its use.
>
> Hopefully there are some convincing explanation to help users
> understand why the cone mode is recommended in the documentation in
> these patches.  Was there a discussion on list to give our unanimous
> support for this plan, or is this only preference by a developer or
> two who have worked on the sparse-checkout code recently?

OK.  There are a few things that I found questionable, but they were
mostly documentation issues (and lack of updates to two places, one
in read-tree and the other in config/core.txt).

Thanks.



[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