<Is it a bug?> Re: User Hook Scripts

Hi,

I'm resending because i didn't get any reply and it's very important to me to resolve this issue.
The mentioned problem is that firing "Start Commit" user hook from check for modifications-> commit populates the MESSAGEFILE list with all changed paths, but running it directly from right click->commit populates MESSAGEFILE file with only the CWD path (even if it didn't change).
 
Thanks,
Golan

----- Original Message ----
From: golan davidovits <golandav_at_yahoo.com>
To: users_at_tortoisesvn.tigris.org
Sent: Monday, January 5, 2009 10:34:34 AM
Subject: Re: User Hook Scripts

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.com>
To: users_at_tortoisesvn.tigris.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.tigris.org/ds/viewMessage.do?dsForumId=4061&dsMessageId=999419
To unsubscribe from this discussion, e-mail: [users-unsubscribe_at_tortoisesvn.tigris.org].
------------------------------------------------------
http://tortoisesvn.tigris.org/ds/viewMessage.do?dsForumId=4061&dsMessageId=1004530
To unsubscribe from this discussion, e-mail: [users-unsubscribe_at_tortoisesvn.tigris.org].
------------------------------------------------------
http://tortoisesvn.tigris.org/ds/viewMessage.do?dsForumId=4061&dsMessageId=1007050
To unsubscribe from this discussion, e-mail: [users-unsubscribe_at_tortoisesvn.tigris.org].