commit: Clarify reasoning for "commit-update-summary" default
In case anyone else gets the idea of changing the default for archive-z2 repos. Should have the rationale captured in the source code somewhere.
This commit is contained in:
parent
bddb25f79e
commit
2233d443e7
|
|
@ -524,6 +524,11 @@ ostree_builtin_commit (int argc, char **argv, GCancellable *cancellable, GError
|
||||||
if (!ostree_repo_commit_transaction (repo, &stats, cancellable, error))
|
if (!ostree_repo_commit_transaction (repo, &stats, cancellable, error))
|
||||||
goto out;
|
goto out;
|
||||||
|
|
||||||
|
/* The default for this option is FALSE, even for archive-z2 repos,
|
||||||
|
* because ostree supports multiple processes committing to the same
|
||||||
|
* repo (but different refs) concurrently, and in fact gnome-continuous
|
||||||
|
* actually does this. In that context it's best to update the summary
|
||||||
|
* explicitly instead of automatically here. */
|
||||||
if (!ot_keyfile_get_boolean_with_default (ostree_repo_get_config (repo), "core",
|
if (!ot_keyfile_get_boolean_with_default (ostree_repo_get_config (repo), "core",
|
||||||
"commit-update-summary", FALSE,
|
"commit-update-summary", FALSE,
|
||||||
&update_summary, error))
|
&update_summary, error))
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue