Re: More pre-revprop-change hook woes

On Tue, 17 Feb 2004 14:09:07 +0100
Juanma Barranquero <jmbarranquero@wke.es> wrote:

> This simple pre-revprop-change.bat hook:

[Somewhat related]

From check_hook_cmd() (in trunk/subversion/libsvn_repos/hooks.c):

    static const char* const check_extns[] = {
  #ifdef WIN32
    /* For WIN32 we need to check with an added extension(s). */
      ".exe", ".cmd", ".bat", /* ### Any other extensions? */
  #else
      "",
  #endif
      NULL
    };

May I suggest that .py and .pl would be sensible extensions on Windows?
Both ActiveState Perl and Python installers, as well as the standard
Python Windows installer from www.python.org, register the corresponding
extension, so the scripts can be executed by clicking on them or from
the CMD command line.

Certainly not all executable extensions can be defined in
check_hook_cmd(), but Python and Perl are very popular and that would
avoid the current "trick" of executing a .BAT to start the script.

                                                                Juanma

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Tue Feb 17 16:29:15 2004