[PATCH v2 0/10] cleaning up check_repository_format_gently

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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 majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]