Login | Register
My pages Projects Community openCollabNet

Discussions > users > Re: User Hook Scripts

Project highlights: :. Download .: :. Support .: :. FAQ .: :. Translations .: :. Donate .: :. Report Bug .:

tortoisesvn
Discussion topic

Back to topic list

Re: User Hook Scripts

Author golan davidovits <golandav at yahoo dot com>
Full name golan davidovits <golandav at yahoo dot com>
Date 2009-01-05 00:34:36 PST
Message Hi Stefan,

You are right, i did mean to delete only the svn:mergeinfo property.
Anyway, I continued with my experiment and encountered a problem.


First, my intention to match against CWD (top level) is not enough since i can't control the depth were the user will try to commit.
I resolved it by checking if <line> is DIR and then if <line>\..\.svn exist. if not -> then it's the top level folder that should not be deleted.
All other properties changes should be removed.

Now, i see that if i run the "start commit" hook from check for modifications->commit, the MESSAGEFILE list is populated with all the changed paths, but if i run it by standing the on top level folder and select commit from there, only the top folder list is written in the MESSAGEFILE. This is blocking me from completing the hook.

In addition, it would be very useful if any such client hook will have a disable\enable checkbox. It could make debugging easier.

Thanks,
Golan



----- Original Message ----
From: Stefan Kueng <tortoisesvn at gmail dot com>
To: users at tortoisesvn dot tigris dot org
Sent: Friday, January 2, 2009 9:53:24 AM
Subject: Re: User Hook Scripts

golan davidovits wrote:
> Cool Thanks :-)
>
> This is my intention: We have many issues with the properties changes on inner folders.
> And we don't do inner folders merges. So until SVN 1.6 will minimize these properties creation i thought running "Start Merge" user hook that will do:
>
> go over the file list <lines> coming from MESSAGEFILE and if path <> CWD (top level folder) run "svn propdel <line>"
>
>
> Do you see any issue with that?

Only one small issue: you should check first whether the property is
really svn:mergeinfo before you remove it!

Stefan


--
       ___
  oo // \\ "De Chelonian Mobile"
(_,\/ \_/ \ TortoiseSVN
   \ \_/_\_/> The coolest Interface to (Sub)Version Control
   /_/ \_\ http://tortoisesvn.net

--------------------​--------------------​--------------
http://tortoisesvn.t​igris.org/ds/viewMes​sage.do?dsForumId=40​61&dsMessageId=9​99419

To unsubscribe from this discussion, e-mail: [users-unsubscribe@t​ortoisesvn.tigris.or​g].

« Previous message in topic | 5 of 5 | Next message in topic »

Messages

Show all messages in topic

User Hook Scripts golan davidovits <golandav at yahoo dot com> golan davidovits <golandav at yahoo dot com> 2009-01-01 06:19:55 PST
     Re: User Hook Scripts steveking Stefan Küng 2009-01-01 06:21:36 PST
     Re: User Hook Scripts golan davidovits <golandav at yahoo dot com> golan davidovits <golandav at yahoo dot com> 2009-01-01 06:52:09 PST
         Re: User Hook Scripts steveking Stefan Küng 2009-01-01 23:53:44 PST
     Re: User Hook Scripts golan davidovits <golandav at yahoo dot com> golan davidovits <golandav at yahoo dot com> 2009-01-05 00:34:36 PST
Messages per page: