1. 28 2月, 2009 4 次提交
  2. 27 2月, 2009 5 次提交
    • J
      Merge branch 'cc/maint-1.6.0-bisect-fix' · 0c347356
      Junio C Hamano 提交于
      * cc/maint-1.6.0-bisect-fix:
        bisect: fix quoting TRIED revs when "bad" commit is also "skip"ped
      
      Conflicts:
      	git-bisect.sh
      0c347356
    • C
      bisect: fix quoting TRIED revs when "bad" commit is also "skip"ped · 1b249ffe
      Christian Couder 提交于
      When the "bad" commit was also "skip"ped and when more than one
      commit was skipped, the "filter_skipped" function would have
      printed something like:
      
          bisect_rev=<hash1>|<hash2>
      
      (where <hash1> and <hash2> are hexadecimal sha1 hashes)
      
      and this would have been evaled later as piping "bisect_rev=<hash1>"
      into "<hash2>", which would have failed.
      
      So this patch makes the "filter_skipped" function properly quote
      what it outputs, so that it will print something like:
      
      bisect_rev='<hash1>|<hash2>'
      
      which will be properly evaled later.  The caller was not stopping
      properly because the scriptlet this function returned to be evaled
      was not strung together with && and because of this, an error in
      an earlier part of the output was simply ignored.
      
      A test case is added to the test suite.
      
      And while at it, we also initialize the VARS, FOUND and TRIED
      variables, so that we protect ourselves from environment variables
      the user may have with these names.
      Signed-off-by: NChristian Couder <chriscool@tuxfamily.org>
      Signed-off-by: NJunio C Hamano <gitster@pobox.com>
      1b249ffe
    • J
      git-am: make --abort less dangerous · 2d56a136
      Junio C Hamano 提交于
      When you are in the middle of "git rebase", "git am --abort" by mistake
      would have referred to nonexistent ORIG_HEAD and barfed, or worse yet, used
      a stale ORIG_HEAD and taken you to an unexpected commit.
      
      Also the option parsing did not reject "git am --abort --skip".
      Signed-off-by: NJunio C Hamano <gitster@pobox.com>
      2d56a136
    • M
      git-am: Keep index in case of abort with dirty index · c767184d
      Michael J Gruber 提交于
      git am --abort resets the index unconditionally. But in case a previous
      git am exited due to a dirty index it is preferable to keep that index.
      Make it so.
      Signed-off-by: NMichael J Gruber <git@drmicha.warpmail.net>
      Signed-off-by: NJunio C Hamano <gitster@pobox.com>
      c767184d
    • J
      t5540-http-push.sh: avoid non-portable grep -P · 2591838b
      Jay Soffian 提交于
      OS X's GNU grep does not support -P/--perl-regexp.
      
      We use a basic RE instead, and simplify the pattern slightly by
      replacing '+' with '*' so it can be more easily expressed using a basic
      RE.  The important part of pattern, checking for a SHA-1 has suffix in
      the successful PUT/MOVE operations, remains the same.  Also, a-z instead
      of a-f was an obvious mistake in the original RE. Here are samples of
      what we want to match:
      
      127.0.0.1 - - [26/Feb/2009:22:38:13 +0000] "PUT /test_repo.git/objects/3e/a4fbb9e18a401a6463c595d08118fcb9fb7426_fab55116904c665a95438bcc78521444a7db6096 HTTP/1.1" 201 277
      127.0.0.1 - - [26/Feb/2009:22:38:13 +0000] "MOVE /test_repo.git/objects/3e/a4fbb9e18a401a6463c595d08118fcb9fb7426_fab55116904c665a95438bcc78521444a7db6096 HTTP/1.1" 201 277
      Signed-off-by: NJay Soffian <jaysoffian@gmail.com>
      Signed-off-by: NJunio C Hamano <gitster@pobox.com>
      2591838b
  3. 26 2月, 2009 5 次提交
    • J
      GIT 1.6.2-rc2 · 661763ab
      Junio C Hamano 提交于
      Signed-off-by: NJunio C Hamano <gitster@pobox.com>
      661763ab
    • J
      Merge branch 'jc/maint-1.6.0-pack-directory' · bb0cebd7
      Junio C Hamano 提交于
      * jc/maint-1.6.0-pack-directory:
        Make sure objects/pack exists before creating a new pack
      bb0cebd7
    • J
      Make sure objects/pack exists before creating a new pack · 6e180cdc
      Junio C Hamano 提交于
      In a repository created with git older than f49fb35d (git-init-db: create
      "pack" subdirectory under objects, 2005-06-27), objects/pack/ directory is
      not created upon initialization.  It was Ok because subdirectories are
      created as needed inside directories init-db creates, and back then,
      packfiles were recent invention.
      
      After the said commit, new codepaths started relying on the presense of
      objects/pack/ directory in the repository.  This was exacerbated with
      8b4eb6b6 (Do not perform cross-directory renames when creating packs,
      2008-09-22) that moved the location temporary pack files are created from
      objects/ directory to objects/pack/ directory, because moving temporary to
      the final location was done carefully with lazy leading directory creation.
      
      Many packfile related operations in such an old repository can fail
      mysteriously because of this.
      
      This commit introduces two helper functions to make things work better.
      
       - odb_mkstemp() is a specialized version of mkstemp() to refactor the
         code and teach it to create leading directories as needed;
      
       - odb_pack_keep() refactors the code to create a ".keep" file while
         create leading directories as needed.
      Signed-off-by: NJunio C Hamano <gitster@pobox.com>
      6e180cdc
    • J
      Allow HTTP tests to run on Darwin · 69707d61
      Jay Soffian 提交于
      This patch allows the HTTP tests to run on OS X 10.5. It is not
      sufficient to be able to pass in LIB_HTTPD_PATH and
      LIB_HTTPD_MODULE_PATH alone, as the apache.conf also needs a couple
      tweaks.
      
      These changes are put into an <IfDefine> to keep them Darwin specific,
      but this means lib-httpd.sh needs to be modified to pass -DDarwin to
      apache when running on Darwin. As long as we're making this change to
      lib-httpd.sh, we may as well set LIB_HTTPD_PATH and
      LIB_HTTPD_MODULE_PATH to appropriate default values for the platform.
      
      Note that we now pass HTTPD_PARA to apache at shutdown as well.
      Otherwise apache will emit a harmless, but noisy warning that LogFormat
      is an unknown directive.
      Signed-off-by: NJay Soffian <jaysoffian@gmail.com>
      Signed-off-by: NJunio C Hamano <gitster@pobox.com>
      69707d61
    • M
      3dbe1165
  4. 25 2月, 2009 11 次提交
  5. 24 2月, 2009 3 次提交
  6. 23 2月, 2009 2 次提交
  7. 22 2月, 2009 1 次提交
  8. 21 2月, 2009 4 次提交
  9. 20 2月, 2009 5 次提交
    • J
      Merge branch 'maint' · 8c5b85ce
      Junio C Hamano 提交于
      * maint:
        More friendly message when locking the index fails.
        Document git blame --reverse.
        Documentation: Note file formats send-email accepts
      8c5b85ce
    • M
      More friendly message when locking the index fails. · e43a6fd3
      Matthieu Moy 提交于
      Just saying that index.lock exists doesn't tell the user _what_ to do
      to fix the problem. We should give an indication that it's normally
      safe to delete index.lock after making sure git isn't running here.
      Signed-off-by: NMatthieu Moy <Matthieu.Moy@imag.fr>
      Signed-off-by: NJunio C Hamano <gitster@pobox.com>
      e43a6fd3
    • M
      Document git blame --reverse. · b452cc16
      Matthieu Moy 提交于
      This was introduced in 85af7929 but
      not documented outside the commit message.
      Signed-off-by: NMatthieu Moy <Matthieu.Moy@imag.fr>
      Signed-off-by: NJunio C Hamano <gitster@pobox.com>
      b452cc16
    • M
      gitweb: Hyperlink multiple git hashes on the same commit message line · 7d233dea
      Marcel M. Cary 提交于
      The current implementation only hyperlinks the first hash on
      a given line of the commit message.  It seems sensible to
      highlight all of them if there are multiple, and it seems
      plausible that there would be multiple even with a tidy line
      length limit, because they can be abbreviated as short as 8
      characters.
      
      Benchmark:
      
      I wanted to make sure that using the 'e' switch to the Perl regex
      wasn't going to kill performance, since this is called once per commit
      message line displayed.
      
      In all three A/B scenarios I tried, the A and B yielded the same
      results within 2%, where A is the version of code before this patch
      and B is the version after.
      
      1: View a commit message containing the last 1000 commit hashes
      2: View a commit message containing 1000 lines of 40 dots to avoid
         hyperlinking at the same message length
      3: View a short merge commit message with a few lines of text and
         no hashes
      
      All were run in CGI mode on my sub-production hardware on a recent
      clone of git.git.  Numbers are the average of 10 reqests per second
      with the first request discarded, since I expect this change to affect
      primarily CPU usage.  Measured with ApacheBench.
      
      Note that the web page rendered was the same; while the new code
      supports multiple hashes per line, there was at most one per line.
      
      The primary purpose of scenarios 2 and 3 were to verify that the
      addition of 1000 commit messages had an impact on how much of the time
      was spent rendering commit messages.  They were all within 2% of 0.80
      requests per second (much faster).
      
      So I think the patch has no noticeable effect on performance.
      Signed-off-by: NMarcel M. Cary <marcel@oak.homeunix.org>
      Acked-by: NJakub Narebski <jnareb@gmail.com>
      Signed-off-by: NJunio C Hamano <gitster@pobox.com>
      7d233dea
    • J
      system_path(): simplify using strip_path_suffix(), and add suffix "git" · 024aa7d8
      Johannes Schindelin 提交于
      At least for the author of this patch, the logic in system_path() was
      too hard to understand.  Using the function strip_path_suffix() documents
      the idea of the code better.
      
      The real change is to add the suffix "git", so that a runtime prefix will
      be computed correctly even when the executable was called in /git/ as is
      the case in msysGit (Windows insists to search the current directory
      before the PATH when looking for an executable).
      Signed-off-by: NJohannes Schindelin <johannes.schindelin@gmx.de>
      Acked-by: NJohannes Sixt <j6t@kdbg.org>
      Signed-off-by: NJunio C Hamano <gitster@pobox.com>
      024aa7d8