This is a re-roll of:
http://article.gmane.org/gmane.comp.version-control.git/288017
The changes from v1 are:
- dropped original 10/10 that got rid of GIT_REPO_VERSION defines
(David mentioned that he's going to be expanding their use, so the
argument that they are not used does not make sense)
- added new 10/10 marking messages for translation (suggested by Duy)
- added int return value to new read_repository_format(); this is
redundant with the error value returned in the struct, but allows a
more idiomatic:
if (read_repository_format(&format, file) < 0)
...
- drop confusing comment from 06/10, in favor of a better explanation
in the commit message
- fixed newline regression from v1 when printing out unknown
extensions. Note that because we switch from looping over warning()
or die() to sticking errors into a strbuf (which the caller then
feeds to warning/die), the format changed a bit. Naively, it would
become:
warning: unknown extension: foo
unknown:extension: bar
but I turned it into the more pleasant:
warning: unknown repository extensions found:
foo
bar
I think it would be better still if warning() was smart enough to
stick its prefix in front of all lines (like advise() does). But
that's outside the scope of this series. And it probably doesn't
matter much either way; this is not a message we'd expect anyone to
see routinely.
[01/10]: setup: document check_repository_format()
[02/10]: wrap shared_repository global in get/set accessors
[03/10]: lazily load core.sharedrepository
[04/10]: check_repository_format_gently: stop using git_config_early
[05/10]: config: drop git_config_early
[06/10]: setup: refactor repo format reading and verification
[07/10]: init: use setup.c's repo version verification
[08/10]: setup: unify repository version callbacks
[09/10]: setup: drop repository_format_version global
[10/10]: verify_repository_format: mark messages for translation
-Peff
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
