git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Jan Wielemaker <wielemak@science.uva.nl>
To: Johannes Schindelin <Johannes.Schindelin@gmx.de>
Cc: Frank Lichtenheld <frank@lichtenheld.de>,
	Martin Langhoff <martin.langhoff@gmail.com>,
	git@vger.kernel.org
Subject: Re: [PATCH/RFT] cvsserver: only allow checkout of branches
Date: Thu, 4 Oct 2007 19:06:20 +0200	[thread overview]
Message-ID: <200710041906.21004.wielemak@science.uva.nl> (raw)
In-Reply-To: <Pine.LNX.4.64.0710041622070.4174@racer.site>

On Thursday 04 October 2007 17:29, Johannes Schindelin wrote:
> It does not make sense to check out tags or "HEAD".
>
> Noticed by Jan Wielemaker.

Thanks (though it won't bite me a second time :-)

> ) +    {
> +	$log->warn("Checkout failed: $module is not a branch");
> +	print "error 1 Checkout failed: $module is not a branch\n";

There are various places where it says print "...", one that even
started some suggestion on what might be wrong, but these things don't
end up at a visible place when using ssh as transport. Its a good idea
this one is at least in the logfile as well. Not sure, but might
printing to stderr work better?

	Cheers --- Jan

  parent reply	other threads:[~2007-10-04 17:06 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-10-03 11:48 git-cvsserver commit trouble (unexpected end of file in client) Jan Wielemaker
2007-10-03 13:13 ` Jan Wielemaker
2007-10-03 14:57   ` Jan Wielemaker
2007-10-03 16:11 ` Johannes Schindelin
2007-10-03 18:42   ` Jan Wielemaker
2007-10-03 18:55     ` Johannes Schindelin
     [not found] ` <200710032325.55128.wielemak@science.uva.nl>
     [not found]   ` <Pine.LNX.4.64.0710032311480.28395@racer.site>
2007-10-04  7:27     ` Jan Wielemaker
2007-10-04 11:18       ` git-cvsserver commit trouble BUG+Work-around Jan Wielemaker
2007-10-04 12:56         ` Johannes Schindelin
2007-10-04 13:06           ` Jan Wielemaker
2007-10-04 15:29             ` [PATCH/RFT] cvsserver: only allow checkout of branches Johannes Schindelin
2007-10-04 16:18               ` Frank Lichtenheld
2007-10-04 16:32                 ` Johannes Schindelin
2007-10-04 17:06               ` Jan Wielemaker [this message]
2007-10-04 17:18                 ` Johannes Schindelin
2007-10-04 18:04                 ` Frank Lichtenheld
2007-10-04 21:15               ` Frank Lichtenheld
2007-10-04 21:49                 ` Johannes Schindelin
2007-10-04 21:55                   ` Shawn O. Pearce

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=200710041906.21004.wielemak@science.uva.nl \
    --to=wielemak@science.uva.nl \
    --cc=Johannes.Schindelin@gmx.de \
    --cc=frank@lichtenheld.de \
    --cc=git@vger.kernel.org \
    --cc=martin.langhoff@gmail.com \
    /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).