git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Clemens Buchacher <drizzd@aon.at>
To: git@vger.kernel.org
Cc: Junio C Hamano <gitster@pobox.com>, tobiasu@tmux.org
Subject: Re: [BUG] git push --quiet fails with older versions
Date: Sat,  3 Sep 2011 18:34:13 +0200	[thread overview]
Message-ID: <1315067656-2846-1-git-send-email-drizzd@aon.at> (raw)
In-Reply-To: <20110903105723.GA16304@tin.tmux.org>

Hello Tobias,

On Sat, Sep 03, 2011 at 12:57:23PM +0200, Tobias Ulmer wrote:
> 
> my distro updated git to 1.7.6.1. Now git push --quiet, used in various
> scripts, blows up against 1.7.6. I don't know how backwards compatible
> git tries to be, but in my opinion that seems like a regression for a
> patchlevel release.

Yes, that's a good point. I did not consider that.

> The problem seems to be that the remote git receive-pack does not
> understand --quiet. Maybe it is possible to check for the remote version
> before sending incompatible commandline parameters?

Indeed, that's what we must do. I am sending three patches in reply
to this email:

[PATCH 1/3] tests for push --quiet
[PATCH 2/3] fix push --quiet via http
[PATCH 3/3] push: old receive-pack does not understand --quiet

The third one fixes this issue on the client side.  It does not
depend on the first two. It applies to v1.7.6.1, maint, and master.
Would be great if you could test.

Thanks,
Clemens

  reply	other threads:[~2011-09-03 16:37 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-09-03 10:57 [BUG] git push --quiet fails with older versions Tobias Ulmer
2011-09-03 16:34 ` Clemens Buchacher [this message]
2011-09-03 16:34   ` [PATCH 1/3] tests for push --quiet Clemens Buchacher
2011-09-03 16:34   ` [PATCH 2/3] fix push --quiet via http Clemens Buchacher
2011-09-03 16:34   ` [PATCH 3/3] push: old receive-pack does not understand --quiet Clemens Buchacher
2011-09-05  2:28     ` Junio C Hamano
2011-09-05  3:01       ` Junio C Hamano
2011-09-05  8:35     ` Junio C Hamano
2011-09-05  9:23       ` Michael J Gruber
2011-09-05 14:17         ` Michael J Gruber
2011-09-05 14:24           ` [PATCH] t5541: avoid TAP test miscounting Michael J Gruber
2011-09-05 19:34         ` [PATCH 3/3] push: old receive-pack does not understand --quiet Junio C Hamano
2011-09-06  6:03           ` Michael J Gruber
2011-09-06 18:01     ` Junio C Hamano
2011-09-06 19:52       ` Clemens Buchacher
2011-09-04 12:08   ` [BUG] git push --quiet fails with older versions Tobias Ulmer
2011-09-04 19:02 ` Junio C Hamano

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1315067656-2846-1-git-send-email-drizzd@aon.at \
    --to=drizzd@aon.at \
    --cc=git@vger.kernel.org \
    --cc=gitster@pobox.com \
    --cc=tobiasu@tmux.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).