Thread (58 messages) 58 messages, 7 authors, 2021-12-23

Re: [PATCH 4/6] git-sparse-checkout.txt: update to document that set handles init

From: Elijah Newren <hidden>
Date: 2021-12-07 21:14:10

On Tue, Dec 7, 2021 at 12:42 PM Lessley Dennington
[off-list ref] wrote:
On 12/4/21 12:00 PM, Elijah Newren via GitGitGadget wrote:
quoted
From: Elijah Newren <redacted>
...
quoted
++
+Historically, `set` did not used to handle all the necessary config
+settings, which meant that both `init` and `set` had to be called.
+Invoking both meant the `init` step would first remove nearly all
+tracked files (and in cone mode, ignored files too), then the `set`
+step would add many of the tracked files (but not ignored files) back.
+In addition to the lost files, the performance and UI of this
+combination was poor.
++
Super nit: Perhaps update '`set` did not used to handle' to '`set` did not
handle'.
Yeah, that sounds better.  I'll wait a bit longer for other comments,
then send a fix for this along with any other needed fixes (even if
just included Acks & Reviewed-bys I get).  Thanks!
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help