1. 12 9月, 2008 1 次提交
  2. 11 9月, 2008 6 次提交
  3. 10 9月, 2008 4 次提交
  4. 07 9月, 2008 3 次提交
  5. 05 9月, 2008 4 次提交
    • J
      Mention the fact that 'git annotate' is only for backward compatibility. · f22a432b
      Junio C Hamano 提交于
      When somebody is reading git-blame.txt (or git-annotate.txt) for the first
      time, the message we would like to send is:
      
       (1) Here is why you would want to use this command, what it can do
           (perhaps more than what you would have expected from "$scm blame"),
           and how you tell it to do what it does.
      
           This is obvious.
      
       (2) You might have heard of the command with the other name.  There is no
           difference between the two, except they differ in their default
           output formats.
      
           This is essential to answer: "git has both?  how are they different?"
      
       (3) We tend to encourage blame over annotate for new scripts and new
           people, but there is no reason to choose one over the other.
      
           This is not as important as (2), but would be useful to avoid
           repeated questions about "when will we start deprecating this?"
      
      As long as we describe (2) on git-annotate page clearly enough, people who
      read git-blame page first and get curious can refer to git-annotate page.
      While at it, subtly hint (3) without being overly explicit.
      Signed-off-by: NJunio C Hamano <gitster@pobox.com>
      f22a432b
    • J
      "blame -c" should be compatible with "annotate" · 7ceacdff
      Junio C Hamano 提交于
      There is no reason to have a separate variable cmd_is_annotate;
      OUTPUT_ANNOTATE_COMPAT option is supposed to produce the compatibility
      output, and we should produce the same output even when the command was
      not invoked as "annotate" but as "blame -c".
      
      Noticed by Pasky.
      Signed-off-by: NJunio C Hamano <gitster@pobox.com>
      7ceacdff
    • J
      log --author/--committer: really match only with name part · a4d7d2c6
      Junio C Hamano 提交于
      When we tried to find commits done by AUTHOR, the first implementation
      tried to pattern match a line with "^author .*AUTHOR", which later was
      enhanced to strip leading caret and look for "^author AUTHOR" when the
      search pattern was anchored at the left end (i.e. --author="^AUTHOR").
      
      This had a few problems:
      
       * When looking for fixed strings (e.g. "git log -F --author=x --grep=y"),
         the regexp internally used "^author .*x" would never match anything;
      
       * To match at the end (e.g. "git log --author='google.com>$'"), the
         generated regexp has to also match the trailing timestamp part the
         commit header lines have.  Also, in order to determine if the '$' at
         the end means "match at the end of the line" or just a literal dollar
         sign (probably backslash-quoted), we would need to parse the regexp
         ourselves.
      
      An earlier alternative tried to make sure that a line matches "^author "
      (to limit by field name) and the user supplied pattern at the same time.
      While it solved the -F problem by introducing a special override for
      matching the "^author ", it did not solve the trailing timestamp nor tail
      match problem.  It also would have matched every commit if --author=author
      was asked for, not because the author's email part had this string, but
      because every commit header line that talks about the author begins with
      that field name, regardleses of who wrote it.
      
      Instead of piling more hacks on top of hacks, this rethinks the grep
      machinery that is used to look for strings in the commit header, and makes
      sure that (1) field name matches literally at the beginning of the line,
      followed by a SP, and (2) the user supplied pattern is matched against the
      remainder of the line, excluding the trailing timestamp data.
      Signed-off-by: NJunio C Hamano <gitster@pobox.com>
      a4d7d2c6
    • S
      git-gui: Fix diff parsing for lines starting with "--" or "++" · ca53c3fd
      Shawn O. Pearce 提交于
      Languages like Lua and SQL use "--" to mark a line as commented out.
      If this appears at column 0 and is part of the pre-image we may see
      "--- foo" in the diff, indicating that the line whose content is
       "-- foo" has been removed from the new version.
      
      git-gui was incorrectly parsing "--- foo" as the old file name
      in the file header, causing it to generate a bad patch file when
      the user tried to stage or unstage a hunk or the selected line.
      We need to keep track of where we are in the parsing so that we do
      not misread a deletion or addition record as part of the header.
      Reported-by: NAlexander Gladysh <agladysh@gmail.com>
      Signed-off-by: NShawn O. Pearce <spearce@spearce.org>
      ca53c3fd
  6. 04 9月, 2008 13 次提交
  7. 03 9月, 2008 1 次提交
  8. 02 9月, 2008 7 次提交
  9. 01 9月, 2008 1 次提交