[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: pre-revprop-change stdin wrong?

From: <kmradke_at_rockwellcollins.com>
Date: Thu, 12 Feb 2009 16:14:32 -0600

kmradke_at_rockwellcollins.com wrote on 02/12/2009 08:22:59 AM:
> Ryan Schmidt <subversion-2009a_at_ryandesign.com> wrote on 02/12/2009
01:17:49 AM:
> > On Feb 12, 2009, at 00:31, Daniel Shahaf wrote:
>
> [snip]
>
> > >>> I see a few things wrong here:
> > >>>
> > >>> 1) That is an ugly error message
> > >>> 2) The pre-revprop-change script gets the OLD log message passed
to
> > >>> stdin when it should
> > >>> get the NEW log message according to the tmpl file.
> > >>
> > >> On Subversion 1.5.5 with Mac OS X 10.4.11 Intel, I see:
> > >>
> > >> $ svn propedit --revprop -r 1 svn:log
> > >> svn: Revprop change blocked by pre-revprop-change hook (exit code
1)
> > >> with output:
> > >> This stdin is from pre-revprop-change:
> > >> -----
> > >> new log message
> > >>
> > >> -----
> > >>
> > >> So, my error is shorter (do you have some debug options turned on
> > >> causing more verbose error messages?)
> > >
> > > In maintainer mode, you get file.c:line numbers in the output.
> > > Anyway,
> > > in this case you also see a slightly different error chain,
indicating
> > > that either you use different RA layers or different svn releases
> > > (there
> > > was recently a fix to remove some DAV details from error messages).
> >
> > Oh, indeed. I tested on a file:///-protocol repository.
>
> Yeah, I forgot I compile with maintainer mode. That explains
> the ugly error message.
>
> I believe this is only an issue with mod_dav, not svn:// or file://...

I have confirmed that using file:// works as expected but http://
does not with the same hook script.

Kevin R.

------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=1065&dsMessageId=1147532

To unsubscribe from this discussion, e-mail: [users-unsubscribe_at_subversion.tigris.org].
Received on 2009-02-13 03:32:48 CET

This is an archived mail posted to the Subversion Users mailing list.