@matheusbsbs / matheus-bagliano / commits / e1588c270d

scalar: alphabetize and simplify config

The config values set by Scalar went through an audit in the previous changes, so now reorganize the settings and simplify their purpose. First, alphabetize the config options, except put the platform-specific options at the end. This groups two Windows-specific settings and only one non-Windows setting. Also, this removes the 'overwrite_on_reconfigure' setting for many of these options. That setting made nearly all of these options "required" for scalar enlistments, restricting use for users. Instead, now nearly all options have removed this setting. However, there is one setting that still has this, which is index.skipHash, which was previously being set to _false_ when we actually prefer the value of true. Keep the overwrite here to help Scalar users upgrade to the new version. We may remove that overwrite in the future once we belive that most of the users who have the false value have upgraded to a version that overwrites that to 'true'. Signed-off-by: Derrick Stolee <stolee@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>

Derrick Stolee committed Dec 12, 2025 at 15:15 UTC e1588c270d584fff0ddf1da684515cd218a0718b
1 file changed +30 -30
scalar.c
+30 -30
@@ -133,10 +133,33 @@ static int have_fsmonitor_support(void)
133 static int set_recommended_config(int reconfigure)
134 {
135 struct scalar_config config[] = {
136 - /* Required */
137 - { "am.keepCR", "true", 1 },
136 + { "am.keepCR", "true" },
137 + { "commitGraph.changedPaths", "true" },
138 + { "commitGraph.generationVersion", "1" },
139 + { "core.autoCRLF", "false" },
140 + { "core.logAllRefUpdates", "true" },
141 + { "core.safeCRLF", "false" },
142 + { "credential.https://dev.azure.com.useHttpPath", "true" },
143 + { "feature.experimental", "false" },
144 + { "feature.manyFiles", "false" },
145 + { "fetch.showForcedUpdates", "false" },
146 + { "fetch.unpackLimit", "1" },
147 + { "fetch.writeCommitGraph", "false" },
148 + { "gc.auto", "0" },
149 + { "gui.GCWarning", "false" },
150 + { "index.skipHash", "true", 1 /* Fix previous setting. */ },
151 + { "index.threads", "true"},
152 + { "index.version", "4" },
153 + { "merge.renames", "true" },
154 + { "merge.stat", "false" },
155 + { "pack.useBitmaps", "false" },
156 + { "pack.usePathWalk", "true" },
157 + { "receive.autoGC", "false" },
158 + { "status.aheadBehind", "false" },
159 +
160 + /* platform-specific */
161 #ifndef WIN32
139 - { "core.untrackedCache", "true", 1 },
162 + { "core.untrackedCache", "true" },
163 #else
164 /*
165 * Unfortunately, Scalar's Functional Tests demonstrated
@@ -150,34 +173,11 @@ static int set_recommended_config(int reconfigure)
173 * Therefore, with a sad heart, we disable this very useful
174 * feature on Windows.
175 */
153 - { "core.untrackedCache", "false", 1 },
154 -#endif
155 - { "core.logAllRefUpdates", "true", 1 },
156 - { "credential.https://dev.azure.com.useHttpPath", "true", 1 },
157 - { "gc.auto", "0", 1 },
158 - { "gui.GCWarning", "false", 1 },
159 - { "index.skipHash", "true", 1 },
160 - { "index.threads", "true", 1 },
161 - { "index.version", "4", 1 },
162 - { "merge.stat", "false", 1 },
163 - { "merge.renames", "true", 1 },
164 - { "pack.useBitmaps", "false", 1 },
165 - { "receive.autoGC", "false", 1 },
166 - { "feature.manyFiles", "false", 1 },
167 - { "feature.experimental", "false", 1 },
168 - { "fetch.unpackLimit", "1", 1 },
169 - { "fetch.writeCommitGraph", "false", 1 },
170 -#ifdef WIN32
171 - { "http.sslBackend", "schannel", 1 },
176 + { "core.untrackedCache", "false" },
177 +
178 + /* Other Windows-specific required settings: */
179 + { "http.sslBackend", "schannel" },
180 #endif
173 - /* Optional */
174 - { "status.aheadBehind", "false" },
175 - { "commitGraph.changedPaths", "true" },
176 - { "commitGraph.generationVersion", "1" },
177 - { "core.autoCRLF", "false" },
178 - { "core.safeCRLF", "false" },
179 - { "fetch.showForcedUpdates", "false" },
180 - { "pack.usePathWalk", "true" },
181 { NULL, NULL },
182 };
183 int i;