Re: Add commitid support to loginfo in CVS

2014-08-19 Thread Vadim Zhukov
2014-08-18 9:23 GMT+04:00 Kent R. Spillner kspill...@acm.org: Haven't tested, just eyeballing the diff; I have two questions inline below. 1). Did you want to use proglen here? Yep, good catch, thanks! 2). proglen is declared as an int; is there any danger malicious values could cause an

Re: Add commitid support to loginfo in CVS

2014-08-17 Thread Kent R. Spillner
Haven't tested, just eyeballing the diff; I have two questions inline below. (Apologies if iOS Mail.app mangles the quoting :() On Aug 14, 2014, at 17:39, Vadim Zhukov persg...@gmail.com wrote: 2014-08-11 4:37 GMT+04:00 Vadim Zhukov persg...@gmail.com: This patch adds support for handling

Re: Add commitid support to loginfo in CVS

2014-08-14 Thread Vadim Zhukov
2014-08-11 4:37 GMT+04:00 Vadim Zhukov persg...@gmail.com: This patch adds support for handling commit IDs in loginfo scripts. Now, if you add, e.g., the following line to loginfo... date %{isVv} /tmp/commitlog ... then you'll get something like: DOQ1DVGdv6tda71v ports/x11/kde4/libs

Add commitid support to loginfo in CVS

2014-08-10 Thread Vadim Zhukov
This patch adds support for handling commit IDs in loginfo scripts. Now, if you add, e.g., the following line to loginfo... date %{isVv} /tmp/commitlog ... then you'll get something like: DOQ1DVGdv6tda71v ports/x11/kde4/libs TODO,1.14,1.15 As a bonus, this adds support for '%%' escaping, so