Comments on Troy Hunt: Creating Subversion pre-commit hooks in .NET

It can definitely be pre-commit.cmd or any other e...

2011-06-16T06:44:49.876+10:00

It can definitely be pre-commit.cmd or any other executable variation of the file type (i.e. pre-commit.bat). All my Windows environments are running with a .cmd extension.

Excellent article. ->) Thank you Troy. One th...

2011-06-15T23:58:59.109+10:00

Excellent article. ->)

Thank you Troy.

One thing I learned extra after doing this is : In windows, the name of the exe has to be pre-commit.exe.

That's an interesting question regarding PC-Li...

2011-06-09T19:11:28.775+10:00

That's an interesting question regarding PC-Lint. In theory, you should be able to access the contents of the transaction before it's committed.

Having said that, I'd strongly advise against tying code quality measures to transactions pre-commit. I see three problems with this:

Firstly, an SVN plugin (command line, Tortoise, whatever), is really not the right medium to be communicating potentially numerous breaches of quality standards. It's just too succinct.

Secondly, you run the risk of significantly increasing the time required to commit code. Obviously this will depend on the transaction size and the efficiency of PC-Lint, but it's certainly going to introduce an overhead and you probably want to be asking if this should be happening in the midst of what's meant to be a pretty fluid process.

Finally, if you're just inspecting the transaction, you're going to be assessing one small part of the app. I'm not familiar with PC-Lint but many static analysis tools are only meaningful when they have access to the entire code base. For example, you couldn't identify unused code.

You're really talking about a build server and continuous integration process. Take a look at how I'm doing this with NDepend and TeamCity: http://www.troyhunt.com/2010/12/continuous-code-quality-measurement.html

Hi, You do have a point... Thanks for the valuab...

2011-02-25T18:07:40.767+11:00

Hi,

You do have a point...

Thanks for the valuable advice!

That's an interesting question regarding PC-Li...

2010-12-23T08:16:37.205+11:00

Hi, Any idea on configuring the pre-commit hook t...

2010-12-22T15:51:19.141+11:00

Hi,

Any idea on configuring the pre-commit hook to be used with PC lint?

Basically need to direct SVN to execute PC lint, scan through output report for errors & return a true or false.

Much appreciated if anyone can advise/assist.

Thanks.

Unfortunately all the examples above were from a d...

2010-06-02T07:20:19.665+10:00

Unfortunately all the examples above were from a dummy project I’ve since blown away. Your best bet would be to just create a new console app and copy and paste from above. You shouldn’t need any syntax which isn’t already in this post, fortunately it’s a very simple process.

Would you mind sharing the project file? Thx!

2010-06-02T07:11:53.158+10:00

Would you mind sharing the project file? Thx!

I don’t believe you can modify the comments in the...

2010-06-01T08:26:42.879+10:00

I don’t believe you can modify the comments in the pre-commit stage because the transaction has already been created. Have a look at some of the suggestions on Stack Overflow for changing the log message after it has been committed: http://stackoverflow.com/questions/304383/how-do-i-edit-a-commit-message-that-i-already-committed-in-subversion

Out of interest, what would cause you to want to change someone's commit message?

Is it possible for a pre-commit script to modify t...

2010-05-28T08:02:02.317+10:00

Is it possible for a pre-commit script to modify the commit comments? Is it possible to modify text files being checked in?

The hooks Troy created in our SVN repo generally h...

2010-02-14T14:17:11.566+11:00

The hooks Troy created in our SVN repo generally helped in proper documentation of revisions implemented by the developers and manage code transitions easily. This validation scheme was implemented in our company across Asia and we (Philippines development team) have benefited from it. On the hardware resource side, file inclusion restrictions will efficiently manage server performance and disk space allocations.
Great work Troy! In behalf of the PH team, thank you for provisioning this facility.

-Myla