<feed xmlns='http://www.w3.org/2005/Atom'>
<title>user/sven/git.git/bundle.c, branch v1.6.5.4</title>
<subtitle>Git
</subtitle>
<id>https://git.stealer.net/cgit.cgi/user/sven/git.git/atom?h=v1.6.5.4</id>
<link rel='self' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/atom?h=v1.6.5.4'/>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/'/>
<updated>2009-11-24T05:33:09Z</updated>
<entry>
<title>pack-objects: split implications of --all-progress from progress activation</title>
<updated>2009-11-24T05:33:09Z</updated>
<author>
<name>Nicolas Pitre</name>
<email>nico@fluxnic.net</email>
</author>
<published>2009-11-23T17:43:50Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/commit/?id=4f366275189c06ec26c01ee5ace2f3831b2aa46a'/>
<id>urn:sha1:4f366275189c06ec26c01ee5ace2f3831b2aa46a</id>
<content type='text'>
Currently the --all-progress flag is used to use force progress display
during the writing object phase even if output goes to stdout which is
primarily the case during a push operation.  This has the unfortunate
side effect of forcing progress display even if stderr is not a
terminal.

Let's introduce the --all-progress-implied argument which has the same
intent except for actually forcing the activation of any progress
display.  With this, progress display will be automatically inhibited
whenever stderr is not a terminal, or full progress display will be
included otherwise.  This should let people use 'git push' within a cron
job without filling their logs with useless percentage displays.

Signed-off-by: Nicolas Pitre &lt;nico@fluxnic.net&gt;
Tested-by: Jeff King &lt;peff@peff.net&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>don't dereference NULL upon fdopen failure</title>
<updated>2009-09-13T08:32:20Z</updated>
<author>
<name>Jim Meyering</name>
<email>meyering@redhat.com</email>
</author>
<published>2009-09-12T08:43:27Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/commit/?id=41698375ada57d51cf6db5cf25970ccd1b918180'/>
<id>urn:sha1:41698375ada57d51cf6db5cf25970ccd1b918180</id>
<content type='text'>
There were several unchecked use of fdopen(); replace them with xfdopen()
that checks and dies.

Signed-off-by: Jim Meyering &lt;meyering@redhat.com&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>Terminate argv with NULL before calling setup_revisions()</title>
<updated>2009-05-21T15:56:24Z</updated>
<author>
<name>Nguyễn Thái Ngọc Duy</name>
<email>pclouds@gmail.com</email>
</author>
<published>2009-05-21T09:32:44Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/commit/?id=a80aad7b85fc560451e07792d64ab6cb15a39914'/>
<id>urn:sha1:a80aad7b85fc560451e07792d64ab6cb15a39914</id>
<content type='text'>
It is convention that argv should be terminated with NULL, even if
argc is used to specify the size of argv. setup_revisions() requires
this and may segfault otherwise.

This patch makes sure that all argv (that I can find) is NULL terminated.

Signed-off-by: Nguyễn Thái Ngọc Duy &lt;pclouds@gmail.com&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>Merge branch 'js/maint-all-implies-HEAD'</title>
<updated>2009-01-26T01:13:02Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2009-01-26T01:13:02Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/commit/?id=f18e6bef23809d2823c1a687f375b22c6af0e735'/>
<id>urn:sha1:f18e6bef23809d2823c1a687f375b22c6af0e735</id>
<content type='text'>
* js/maint-all-implies-HEAD:
  bundle: allow the same ref to be given more than once
  revision walker: include a detached HEAD in --all
</content>
</entry>
<entry>
<title>bundle: allow the same ref to be given more than once</title>
<updated>2009-01-18T07:00:31Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2009-01-18T06:27:08Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/commit/?id=b2a6d1c6868b6d5e7d2b4fa9129341220a1e848a'/>
<id>urn:sha1:b2a6d1c6868b6d5e7d2b4fa9129341220a1e848a</id>
<content type='text'>
"git bundle create x master master" used to create a bundle that lists
the same branch (master) twice.  Cloning from such a bundle resulted in
a needless warning "warning: Duplicated ref: refs/remotes/origin/master".

Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>bundle: allow rev-list options to exclude annotated tags</title>
<updated>2009-01-05T18:47:07Z</updated>
<author>
<name>Johannes Schindelin</name>
<email>Johannes.Schindelin@gmx.de</email>
</author>
<published>2009-01-02T18:08:46Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/commit/?id=c9a42c4a12cec17e858b76d6c68aa17c12c0aef1'/>
<id>urn:sha1:c9a42c4a12cec17e858b76d6c68aa17c12c0aef1</id>
<content type='text'>
With options such as "--all --since=2.weeks.ago", annotated tags used to
be included, when they should have been excluded.  The reason is that we
heavily abuse the revision walker to determine what needs to be included
or excluded.  And the revision walker does not show tags at all (and
therefore never marks tags as uninteresting).

Signed-off-by: Johannes Schindelin &lt;johannes.schindelin@gmx.de&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>Fix non-literal format in printf-style calls</title>
<updated>2008-11-11T22:43:59Z</updated>
<author>
<name>Daniel Lowe</name>
<email>dlowe@bitmuse.com</email>
</author>
<published>2008-11-10T21:07:52Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/commit/?id=9db56f71b91153f4076a796c80c61f00edd8b700'/>
<id>urn:sha1:9db56f71b91153f4076a796c80c61f00edd8b700</id>
<content type='text'>
These were found using gcc 4.3.2-1ubuntu11 with the warning:

    warning: format not a string literal and no format arguments

Incorporated suggestions from Brandon Casey &lt;casey@nrlssc.navy.mil&gt;.

Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>Enhance hold_lock_file_for_{update,append}() API</title>
<updated>2008-10-19T19:35:37Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2008-10-17T22:44:39Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/commit/?id=acd3b9eca82e38950f94e4708b528b7dae09a7c8'/>
<id>urn:sha1:acd3b9eca82e38950f94e4708b528b7dae09a7c8</id>
<content type='text'>
This changes the "die_on_error" boolean parameter to a mere "flags", and
changes the existing callers of hold_lock_file_for_update/append()
functions to pass LOCK_DIE_ON_ERROR.

Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>Teach git-bundle to read revision arguments from stdin like git-rev-list.</title>
<updated>2008-07-07T08:31:24Z</updated>
<author>
<name>Adam Brewster</name>
<email>adambrewster@gmail.com</email>
</author>
<published>2008-07-05T21:26:40Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/commit/?id=22568f0a336ac37ae7329c917857b455839d1d09'/>
<id>urn:sha1:22568f0a336ac37ae7329c917857b455839d1d09</id>
<content type='text'>
This patch allows the caller to feed the revision parameters to git-bundle
from its standard input.  This way, a script do not have to worry about
limitation of the length of command line.

Documentation/git-bundle.txt says that git-bundle takes arguments acceptable
to git-rev-list.  Obviously some arguments that git-rev-list handles don't
make sense for git-bundle (e.g. --bisect) but --stdin is pretty reasonable.

Signed-off-by: Adam Brewster &lt;adambrewster@gmail.com&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>start_command(), if .in/.out &gt; 0, closes file descriptors, not the callers</title>
<updated>2008-02-23T19:59:44Z</updated>
<author>
<name>Johannes Sixt</name>
<email>johannes.sixt@telecom.at</email>
</author>
<published>2008-02-21T22:42:56Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/commit/?id=c20181e3a3e39f2c8874567c219e9edddb84393a'/>
<id>urn:sha1:c20181e3a3e39f2c8874567c219e9edddb84393a</id>
<content type='text'>
Callers of start_command() can set the members .in and .out of struct
child_process to a value &gt; 0 to specify that this descriptor is used as
the stdin or stdout of the child process.

Previously, if start_command() was successful, this descriptor was closed
upon return. Here we now make sure that the descriptor is also closed in
case of failures. All callers are updated not to close the file descriptor
themselves after start_command() was called.

Note that earlier run_gpg_verify() of git-verify-tag set .out = 1, which
worked because start_command() treated this as a special case, but now
this is incorrect because it closes the descriptor. The intent here is to
inherit stdout to the child, which is achieved by .out = 0.

Signed-off-by: Johannes Sixt &lt;johannes.sixt@telecom.at&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
</feed>
