svn pre-commit code review

[wenjin]

hello,

So far, I know how to create patch files for pre-commit code review in crucible. But we can still commit the code to svn repository if the code is not reviewed. I want the codes in svn repository are all reviewed. The codes not reviewed can not commit to svn repository. Should I write svn pre-commit hook? Please help me with this problem. Thanks

[Midori]

Your use case is not totally clear, but it sounds as you were to check if the commit message contains a code review number.

If that's some formatted expression (like "FOO-123"), then you can automate this with the Commit Policy Plugin. It has a condition that requires the commit message to match a regular expression, and you can use this to enforce entering the expression!

[wenjin]

Do you use "grep" in pre-commit.bat to catch the key code review number? Is that review number linked to crucible? Can you show me your pre-commit hook? Thank you very much.

I do not know how to add the link number of crucible in our svn log.

[Norman Abramovitz]

Yes, I would write a svn pre-commit hook because that is the only way to handle all different types of clients.