On 26.05.2013 23:22, Daniel Shahaf wrote:
> On Sun, May 26, 2013 at 08:06:47PM -0000, br...@apache.org wrote:
>> +def main(repo, revision, author, propname, action):
>> +    else:
>> +        sys.stderr.write('Unknown revprop change action "%s"\n' % action)
>> +        return
> Maybe sys.exit(1)?  Otherwise the stderr output will likely go unnoticed
> (libsvn_repos discards stderr when the exit code is zero).
>
>> +if __name__ == "__main__":
>> +    if len(sys.argv) != 6:
>> +        sys.stderr.write("invalid args\n")
>> +        sys.exit(0)
>> +
> Same.

I was wondering about that myself. I basically just copied the behaviour
of the commit-hook.

-- Brane

-- 
Branko Čibej
Director of Subversion | WANdisco | www.wandisco.com

Reply via email to