[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn commit: r9622 - trunk/subversion/po

From: <kfogel_at_collab.net>
Date: 2004-05-07 14:43:44 CEST

Branko Čibej <brane@xbc.nu> writes:
> You're trying to forbid empty log messages here, yes? I'd suggest not
> opening an attack path by letting echo expand shell-metas in the $LOG
> variable; so,
> /usr/local/bin/svnlook log ${REPOS} -t ${TXN} | \
> grep '[a-zA-Z0-9]' > /dev/null || exit 1

Thanks! And now that I've posted the old code to a public list, I
think I'll, uh... make this change right now. AHem.

> Yup. Although I'd do something similar as before:
> /mysterious/super/secret/path/to/verify-po.py ${REPOS} ${TXN} || exit 1

Cleaner idiom anyway.

> >By the way, is there any reason not to put "#!/usr/bin/env python" at
> >the top of verify-po.py?
> None at all.

Then consider it done :-).

To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Fri May 7 15:58:51 2004

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.