1. 03 4月, 2008 1 次提交
  2. 02 4月, 2008 10 次提交
  3. 01 4月, 2008 1 次提交
  4. 31 3月, 2008 7 次提交
    • B
      mktag.c: improve verification of tagger field and tests · e0aaf781
      Brandon Casey 提交于
      Since nearly its birth, git's tags have included a "tagger" field which
      describes the name of tagger, email of tagger, and date and time of tagging.
      But, this field was only loosely tested by git-mktag. Provide some thorough
      testing for this field and also ensure that the tag header is separated
      from the tag body by an empty line to reduce the convenience of creating
      a flawed tag.
      Signed-off-by: NBrandon Casey <casey@nrlssc.navy.mil>
      Signed-off-by: NJunio C Hamano <gitster@pobox.com>
      e0aaf781
    • J
      diff-files: careful when inspecting work tree items · f58dbf23
      Junio C Hamano 提交于
      This fixes the same breakage in diff-files.
      Signed-off-by: NJunio C Hamano <gitster@pobox.com>
      f58dbf23
    • J
      diff-index: careful when inspecting work tree items · 948dd346
      Junio C Hamano 提交于
      Earlier, if you changed a staged path into a directory in the work tree,
      we happily ran lstat(2) on it and found that it exists, and declared that
      the user changed it to a gitlink.
      
      This is wrong for two reasons:
      
       (1) It may be a directory, but it may not be a submodule, and in the
           latter case, the change we need to report is "the blob at the path
           has disappeared".  We need to check with resolve_gitlink_ref() to be
           consistent with what "git add" and "git update-index --add" does.
      
       (2) lstat(2) may have succeeded only because a leading component of the
           path was turned into a symbolic link that points at something that
           exists in the work tree.  In such a case, the path itself does not
           exist anymore, as far as the index is concerned.
      
      This fixes these breakages in diff-index that the previous patch has
      exposed.
      Signed-off-by: NJunio C Hamano <gitster@pobox.com>
      948dd346
    • J
      Add corner case tests for diff-index and diff-files · 6301f303
      Junio C Hamano 提交于
      diff-index and diff-files can get confused in corner cases when an indexed
      blob turns into something else in the work tree.  This patch adds tests to
      expose such breakages.
      
      The test is classified under t2XXX series instead of t4XXX series, because
      the ultimate objective is to fix "add -u" (and "commit -a" that shares the
      same issue).
      Signed-off-by: NJunio C Hamano <gitster@pobox.com>
      6301f303
    • J
      Allow git-cvsserver database table name prefix to be specified. · 6aeeffd1
      Josh Elsasser 提交于
      Adds a gitcvs.dbtablenameprefix config variable, the contents of which
      are prepended to any database tables names used by git-cvsserver. The
      same substutions as gitcvs.dbname and gitcvs.dbuser are supported, and
      any non-alphabetic characters are replaced with underscores.
      
      A typo found in contrib/completion/git-completion.bash is also fixed.
      Signed-off-by: NJunio C Hamano <gitster@pobox.com>
      6aeeffd1
    • B
      Silence cpio's "N blocks" output when cloning locally · c20711d2
      Bryan Donlan 提交于
      Pass --quiet to cpio in git-clone to hide the (confusing) "0 blocks" message.
      For compatibility with operating systems which might not support GNUisms,
      the presence of --quiet is probed for by grepping cpio's --help output.
      Signed-off-by: NBryan Donlan <bdonlan@fushizen.net>
      Signed-off-by: NJunio C Hamano <gitster@pobox.com>
      c20711d2
    • E
      git-svn: remove redundant slashes from show-ignore · 67dac28b
      Eric Wong 提交于
      Jonathan Scott Duff wrote:
      
      > Recently I tried "git svn showignore" on my parrot repository and it
      > failed.  I tracked it down to the prop_walk() sub.  When it recurses,
      > $path has an extra / on the beginning (i.e., when it recurses, it
      > tries to get the props for "//apps" instead of "/apps").   I *think*
      > this is because $path is used in the recursive call rather than $p
      > (which seems to contain a properly transformed $path).  Anyway, I've
      > attached a patch that works for me and I think is generally the right
      > thing.
      
      Patch-submitted-by: Jonathan Scott Duff
      Signed-off-by: NEric Wong <normalperson@yhbt.net>
      Signed-off-by: NJunio C Hamano <gitster@pobox.com>
      67dac28b
  5. 28 3月, 2008 21 次提交