<feed xmlns='http://www.w3.org/2005/Atom'>
<title>git/git.c, branch v1.7.2</title>
<subtitle>Mirror of https://git.kernel.org/pub/scm/git/git.git/
</subtitle>
<id>https://git.shady.money/git/atom?h=v1.7.2</id>
<link rel='self' href='https://git.shady.money/git/atom?h=v1.7.2'/>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/'/>
<updated>2010-07-15T19:09:14Z</updated>
<entry>
<title>Merge branch 'jn/paginate-fix'</title>
<updated>2010-07-15T19:09:14Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2010-07-15T19:09:14Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=53b304224a561b5fd4ae35cedc0a978d91d4b1da'/>
<id>urn:sha1:53b304224a561b5fd4ae35cedc0a978d91d4b1da</id>
<content type='text'>
* jn/paginate-fix:
  git --paginate: paginate external commands again
  git --paginate: do not commit pager choice too early
  tests: local config file should be honored from subdirs of toplevel
  t7006: test pager configuration for several git commands
  t7006 (pager): introduce helper for parameterized tests

Conflicts:
	t/t7006-pager.sh
</content>
</entry>
<entry>
<title>git --paginate: paginate external commands again</title>
<updated>2010-07-14T23:07:34Z</updated>
<author>
<name>Jonathan Nieder</name>
<email>jrnieder@gmail.com</email>
</author>
<published>2010-07-14T22:55:12Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=030149a4dcd584f6b47221f5b087d081e582d790'/>
<id>urn:sha1:030149a4dcd584f6b47221f5b087d081e582d790</id>
<content type='text'>
73e25e7c (git --paginate: do not commit pager choice too early,
2010-06-26) failed to take some cases into account.

1b. Builtins that do not use RUN_SETUP (like git config) do
    not find GIT_DIR set correctly when the pager is launched
    from run_builtin().  So the core.pager configuration is
    not honored from subdirectories of the toplevel for them.

4a. External git commands (like git request-pull) relied on the
    early pager launch to take care of handling the -p option.
    Ever since 73e25e7c, they do not honor the -p option at all.

4b. Commands invoked through ! aliases (like ls) were also relying
    on the early pager launch.

Fix (4a) by launching the pager (if requested) before running such a
“dashed external”.  For simplicity, this still does not search for a
.git directory before running the external command; when run from a
subdirectory of the toplevel, therefore, the “[core] pager”
configuration is still not honored.

Fix (4b) by launching pager if requested before carrying out such an
alias.  Actually doing this has no effect, since the pager (if any)
would have already been launched in a failed attempt to try a
dashed external first.  The choice-of-pager-not-honored-from-
subdirectory bug still applies here, too.

(1b) is not a regression.  There is no need to fix it yet.

Noticed by Junio.

Signed-off-by: Jonathan Nieder &lt;jrnieder@gmail.com&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>git --paginate: do not commit pager choice too early</title>
<updated>2010-06-28T17:31:17Z</updated>
<author>
<name>Nguyễn Thái Ngọc Duy</name>
<email>pclouds@gmail.com</email>
</author>
<published>2010-06-26T19:26:37Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=73e25e7cc80e175b5c94678188333e42107fa16e'/>
<id>urn:sha1:73e25e7cc80e175b5c94678188333e42107fa16e</id>
<content type='text'>
When git is passed the --paginate option, starting up a pager requires
deciding what pager to start, which requires access to the core.pager
configuration.

At the relevant moment, the repository has not been searched for yet.
Attempting to access the configuration at this point results in
git_dir being set to .git [*], which is almost certainly not what was
wanted.  In particular, when run from a subdirectory of the toplevel,
git --paginate does not respect the core.pager setting from the
current repository.

[*] unless GIT_DIR or GIT_CONFIG is set

So delay the pager startup when possible:

1. run_argv() already commits pager choice inside run_builtin() if a
   command is found.  For commands that use RUN_SETUP, waiting until
   then fixes the problem described above: once git knows where to
   look, it happily respects the core.pager setting.

2. list_common_cmds_help() prints out 29 lines and exits.  This can
   benefit from pagination, so we need to commit the pager choice
   before writing this output.

   Luckily ‘git’ without subcommand has no other reason to access a
   repository, so it would be intuitive to ignore repository-local
   configuration in this case.  Simpler for now to choose a pager
   using the funny code that notices a repository that happens to be
   at .git.  That this accesses a repository when it is very
   convenient to is a bug but not an important one.

3. help_unknown_cmd() prints out a few lines to stderr.  It is not
   important to paginate this, so don’t.

Signed-off-by: Nguyễn Thái Ngọc Duy &lt;pclouds@gmail.com&gt;
Signed-off-by: Jonathan Nieder &lt;jrnieder@gmail.com&gt;
Acked-by: Jeff King &lt;peff@peff.net&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>grep: Add the option '--open-files-in-pager'</title>
<updated>2010-06-13T16:16:38Z</updated>
<author>
<name>Johannes Schindelin</name>
<email>johannes.schindelin@gmx.de</email>
</author>
<published>2010-06-12T16:36:51Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=678e484b7d4e6388edeec3470bbbcd206817c148'/>
<id>urn:sha1:678e484b7d4e6388edeec3470bbbcd206817c148</id>
<content type='text'>
This adds an option to open the matching files in the pager, and if the
pager happens to be "less" (or "vi") and there is only one grep pattern,
it also jumps to the first match right away.

The short option was chose as '-O' to avoid clashes with GNU grep's
options (as suggested by Junio).

So, 'git grep -O abc' is a short form for 'less +/abc $(grep -l abc)'
except that it works also with spaces in file names, and it does not
start the pager if there was no matching file.

[jn: rebased and added tests; with error handling fix from Junio
squashed in]

Signed-off-by: Johannes Schindelin &lt;johannes.schindelin@gmx.de&gt;
Signed-off-by: Jonathan Nieder &lt;jrnieder@gmail.com&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>Allow passing of configuration parameters in the command line</title>
<updated>2010-03-28T16:48:25Z</updated>
<author>
<name>Alex Riesen</name>
<email>raa.lkml@gmail.com</email>
</author>
<published>2010-03-26T22:53:57Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=8b1fa778676ae94f7a6d4113fa90947b548154dd'/>
<id>urn:sha1:8b1fa778676ae94f7a6d4113fa90947b548154dd</id>
<content type='text'>
The values passed this way will override whatever is defined
in the config files.

Signed-off-by: Alex Riesen &lt;raa.lkml@gmail.com&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>Merge branch 'jh/notes'</title>
<updated>2010-03-15T07:52:06Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2010-03-15T07:52:06Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=2949151fe9d92b2c3405f188e650d9bb7b46c663'/>
<id>urn:sha1:2949151fe9d92b2c3405f188e650d9bb7b46c663</id>
<content type='text'>
* jh/notes: (33 commits)
  Documentation: fix a few typos in git-notes.txt
  notes: fix malformed tree entry
  builtin-notes: Minor (mostly parse_options-related) fixes
  builtin-notes: Add "copy" subcommand for copying notes between objects
  builtin-notes: Misc. refactoring of argc and exit value handling
  builtin-notes: Add -c/-C options for reusing notes
  builtin-notes: Refactor handling of -F option to allow combining -m and -F
  builtin-notes: Deprecate the -m/-F options for "git notes edit"
  builtin-notes: Add "append" subcommand for appending to note objects
  builtin-notes: Add "add" subcommand for adding notes to objects
  builtin-notes: Add --message/--file aliases for -m/-F options
  builtin-notes: Add "list" subcommand for listing note objects
  Documentation: Generalize git-notes docs to 'objects' instead of 'commits'
  builtin-notes: Add "prune" subcommand for removing notes for missing objects
  Notes API: prune_notes(): Prune notes that belong to non-existing objects
  t3305: Verify that removing notes triggers automatic fanout consolidation
  builtin-notes: Add "remove" subcommand for removing existing notes
  Teach builtin-notes to remove empty notes
  Teach notes code to properly preserve non-notes in the notes tree
  t3305: Verify that adding many notes with git-notes triggers increased fanout
  ...

Conflicts:
	Makefile
</content>
</entry>
<entry>
<title>fallback SSH_ASKPASS when GIT_ASKPASS not set</title>
<updated>2010-03-02T20:15:41Z</updated>
<author>
<name>Frank Li</name>
<email>lznuaa@gmail.com</email>
</author>
<published>2010-03-02T11:52:11Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=82cd8358e8203fa02da18e8ee71449cb45c4157d'/>
<id>urn:sha1:82cd8358e8203fa02da18e8ee71449cb45c4157d</id>
<content type='text'>
If GIT_ASKPASS is not set and SSH_ASKPASS set, GIT_ASKPASS will
use SSH_ASKPASS.

Signed-off-by: Frank Li &lt;lznuaa@gmail.com&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>Merge branch 'maint'</title>
<updated>2010-02-17T06:40:45Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2010-02-17T06:40:45Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=72cd63c008c673e185740a694cdf3e85711df3d2'/>
<id>urn:sha1:72cd63c008c673e185740a694cdf3e85711df3d2</id>
<content type='text'>
* maint:
  Prepare 1.7.0.1 release notes
  Fix use of mutex in threaded grep
  dwim_ref: fix dangling symref warning
  stash pop: remove 'apply' options during 'drop' invocation
  diff: make sure --output=/bad/path is caught
  Remove hyphen from "git-command" in two error messages
</content>
</entry>
<entry>
<title>Merge branch 'maint-1.6.6' into maint</title>
<updated>2010-02-16T23:05:02Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2010-02-16T23:05:02Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=e7b3cea0f7589c57f7bb808330e32bdd7f901c95'/>
<id>urn:sha1:e7b3cea0f7589c57f7bb808330e32bdd7f901c95</id>
<content type='text'>
* maint-1.6.6:
  dwim_ref: fix dangling symref warning
  stash pop: remove 'apply' options during 'drop' invocation
  diff: make sure --output=/bad/path is caught
  Remove hyphen from "git-command" in two error messages
</content>
</entry>
<entry>
<title>Remove hyphen from "git-command" in two error messages</title>
<updated>2010-02-16T02:20:54Z</updated>
<author>
<name>Pete Harlan</name>
<email>pgit@pcharlan.com</email>
</author>
<published>2010-02-15T23:33:18Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=7283bbc70a55d7364fbeaefc1009c03fcfc8d929'/>
<id>urn:sha1:7283bbc70a55d7364fbeaefc1009c03fcfc8d929</id>
<content type='text'>
Signed-off-by: Pete Harlan &lt;pgit@pcharlan.com&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
</feed>
