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 : > 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 overflow, or ar

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 wrote: > > 2014-08-11 4:37 GMT+04:00 Vadim Zhukov : >> This patch adds support for handling commit IDs in loginfo scripts.

Re: Add commitid support to loginfo in CVS

2014-08-14 Thread Vadim Zhukov
2014-08-11 4:37 GMT+04:00 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

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 n