You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by Panu Outinen <pa...@vertex.fi> on 2008/10/07 13:17:17 UTC

svnlook 1.5.x & contrib hook-script 'enforcer', problem with properties ...

Hi!

I just upgraded our Subversion server version from 1.4.3 to 1.5.1 and 
ran into some problems with svn-properties with the contributed 
'enforcer' script:
http://svn.collab.net/repos/svn/branches/1.5.x/contrib/hook-scripts/enforcer/enforcer

Looks like that r27808 with svnlook has broken it since enforcer's 
parser expects to have the string "Name: " always in front of them.

   - Panu

----------------------------------------------

r27808 comments (the beginning!)

Finish issue #3019 by disambiguating property change actions in 'svn
diff' and 'svnlook diff' output.  Now instead of seeing:

    Property changes on: iota
    ___________________________________________________________________
    Name: changedprop
       - oldvalue
       + newvalue
    Name: deletedprop
       - oldvalue
    Name: addedprop
       + newvalue

you'll see the following (note the action string):

    Property changes on: iota
    ___________________________________________________________________
    Modified: changedprop
       - oldvalue
       + newvalue
    Deleted: deletedprop
       - oldvalue
    Added: addedprop
       + newvalue

...


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org

Re: svnlook 1.5.x & contrib hook-script 'enforcer', problem with properties ...

Posted by Karl Fogel <kf...@red-bean.com>.
Panu Outinen <pa...@vertex.fi> writes:
> Hi!
>
> I just upgraded our Subversion server version from 1.4.3 to 1.5.1 and
> ran into some problems with svn-properties with the contributed
> enforcer' script:
> http://svn.collab.net/repos/svn/branches/1.5.x/contrib/hook-scripts/enforcer/enforcer
>
> Looks like that r27808 with svnlook has broken it since enforcer's
> parser expects to have the string "Name: " always in front of them.

Yup.  Are you reasonably good at Python?  Would you like to have a go at
a patch?  See http://subversion.tigris.org/hacking.html#patches .

I'm CC'ing Wilfredo Sanchez Vega <ws...@wsanchez.net>, who is the
original author of that script.

-Karl

> ----------------------------------------------
>
> r27808 comments (the beginning!)
>
> Finish issue #3019 by disambiguating property change actions in 'svn
> diff' and 'svnlook diff' output.  Now instead of seeing:
>
>    Property changes on: iota
>    ___________________________________________________________________
>    Name: changedprop
>       - oldvalue
>       + newvalue
>    Name: deletedprop
>       - oldvalue
>    Name: addedprop
>       + newvalue
>
> you'll see the following (note the action string):
>
>    Property changes on: iota
>    ___________________________________________________________________
>    Modified: changedprop
>       - oldvalue
>       + newvalue
>    Deleted: deletedprop
>       - oldvalue
>    Added: addedprop
>       + newvalue
>
> ...
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
> For additional commands, e-mail: dev-help@subversion.tigris.org

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org