On 07/23/2014 12:13 AM, Martin Kletzander wrote: > On Fri, Jul 18, 2014 at 03:23:51PM -0600, Eric Blake wrote: >> Commit 5028160 accidentally weakened the strtol prohibitions to >> skip ALL files under src/util instead of the former situation of >> just protecting util/virsexpr.c; even though NONE of the files >> in that directory need any protection. >> >> Shorten some long lines while at it. >> >> * cfg.mk (exclude_file_name_regexp--sc_prohibit_strtol): No need >> to exclude all of util. >> (exclude_file_name_regexp--sc_prohibit_sprintf): Reduce long line. >> (exclude_file_name_regexp--sc_prohibit_raw_allocation): Likewise. >> >> + ^(docs/hacking\.html\.in|.*stp|.pl)$$ >> > > This fails for me even though the regexp makes sense. Either (a) > adding an asterisk before 'pl' (the same way as for > '.*stp') or (b) encapsulating the caret in the parentheses should > work, probably one of those you meant to do. Oops, serves me right for testing one version, then tweaking, then posting, without retesting. > > ACK with that changed, Fixed and pushed. -- Eric Blake eblake redhat com +1-919-301-3266 Libvirt virtualization library http://libvirt.org
Attachment:
signature.asc
Description: OpenPGP digital signature
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list