Re: [PATCH v3 5/5] repo-settings: create feature.experimental setting

2019-08-08 Thread Derrick Stolee
On 8/8/2019 4:31 PM, Elijah Newren wrote: > On Thu, Aug 8, 2019 at 12:12 PM Derrick Stolee wrote: >> >> On 8/8/2019 2:59 PM, Junio C Hamano wrote: >>> Elijah Newren writes: >>> > --- a/Documentation/config/merge.txt > +++ b/Documentation/config/merge.txt > @@ -54,7 +54,8 @@ merge.dire

Re: [PATCH v3 5/5] repo-settings: create feature.experimental setting

2019-08-08 Thread Derrick Stolee
On 8/8/2019 4:07 PM, Junio C Hamano wrote: > Elijah Newren writes: > >> However, Stollee's commit message started with "The >> 'feature.experimental' setting includes config options that are not >> committed to become defaults". >> >> If these are settings that are "intended" but not "committed"

Re: [PATCH v3 5/5] repo-settings: create feature.experimental setting

2019-08-08 Thread Elijah Newren
On Thu, Aug 8, 2019 at 12:12 PM Derrick Stolee wrote: > > On 8/8/2019 2:59 PM, Junio C Hamano wrote: > > Elijah Newren writes: > > > >>> --- a/Documentation/config/merge.txt > >>> +++ b/Documentation/config/merge.txt > >>> @@ -54,7 +54,8 @@ merge.directoryRenames:: > >>> moved into the ne

Re: [PATCH v3 5/5] repo-settings: create feature.experimental setting

2019-08-08 Thread Junio C Hamano
Elijah Newren writes: > However, Stollee's commit message started with "The > 'feature.experimental' setting includes config options that are not > committed to become defaults". > > If these are settings that are "intended" but not "committed" to > become defaults, then yes, you're absolutely ri

Re: [PATCH v3 5/5] repo-settings: create feature.experimental setting

2019-08-08 Thread Elijah Newren
On Thu, Aug 8, 2019 at 12:00 PM Junio C Hamano wrote: > > Elijah Newren writes: > > >> --- a/Documentation/config/merge.txt > >> +++ b/Documentation/config/merge.txt > >> @@ -54,7 +54,8 @@ merge.directoryRenames:: > >> moved into the new directory. If set to "conflict", a conflict > >>

Re: [PATCH v3 5/5] repo-settings: create feature.experimental setting

2019-08-08 Thread Derrick Stolee
On 8/8/2019 2:59 PM, Junio C Hamano wrote: > Elijah Newren writes: > >>> --- a/Documentation/config/merge.txt >>> +++ b/Documentation/config/merge.txt >>> @@ -54,7 +54,8 @@ merge.directoryRenames:: >>> moved into the new directory. If set to "conflict", a conflict >>> will be rep

Re: [PATCH v3 5/5] repo-settings: create feature.experimental setting

2019-08-08 Thread Junio C Hamano
Elijah Newren writes: >> --- a/Documentation/config/merge.txt >> +++ b/Documentation/config/merge.txt >> @@ -54,7 +54,8 @@ merge.directoryRenames:: >> moved into the new directory. If set to "conflict", a conflict >> will be reported for such paths. If merge.renames is false, >>

Re: [PATCH v3 5/5] repo-settings: create feature.experimental setting

2019-08-08 Thread Derrick Stolee
On 8/8/2019 2:34 PM, Elijah Newren wrote: > Sorry for the late reply... > > On Tue, Jul 30, 2019 at 3:49 PM Derrick Stolee via GitGitGadget > wrote: >> >> --- a/Documentation/config/merge.txt >> +++ b/Documentation/config/merge.txt >> @@ -54,7 +54,8 @@ merge.directoryRenames:: >> moved in

Re: [PATCH v3 5/5] repo-settings: create feature.experimental setting

2019-08-08 Thread Elijah Newren
Sorry for the late reply... On Tue, Jul 30, 2019 at 3:49 PM Derrick Stolee via GitGitGadget wrote: > > --- a/Documentation/config/merge.txt > +++ b/Documentation/config/merge.txt > @@ -54,7 +54,8 @@ merge.directoryRenames:: > moved into the new directory. If set to "conflict", a conflict

[PATCH v3 5/5] repo-settings: create feature.experimental setting

2019-07-30 Thread Derrick Stolee via GitGitGadget
From: Derrick Stolee The 'feature.experimental' setting includes config options that are not committed to become defaults, but could use additional testing. Update the following config settings to take new defaults, and to use the repo_settings struct if not already using them: * 'pack.useSpars