Introducing keepalive_interval = -1 breaks to augeas lens. Fix the lens by allowing signed ints in the regular expression. --- daemon/libvirtd.aug | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/daemon/libvirtd.aug b/daemon/libvirtd.aug index a70aa1d..4d40ee2 100644 --- a/daemon/libvirtd.aug +++ b/daemon/libvirtd.aug @@ -13,7 +13,7 @@ module Libvirtd = let str_val = del /\"/ "\"" . store /[^\"]*/ . del /\"/ "\"" let bool_val = store /0|1/ - let int_val = store /[0-9]+/ + let int_val = store /-?[0-9]+/ let str_array_element = [ seq "el" . str_val ] . del /[ \t\n]*/ "" let str_array_val = counter "el" . array_start . ( str_array_element . ( array_sep . str_array_element ) * ) ? . array_end

On Mon, May 02, 2016 at 09:42:58AM +0200, Michal Privoznik wrote: > On 15.04.2016 17:03, Cédric Bosdonnat wrote: > > Introducing keepalive_interval = -1 breaks to augeas lens. Fix > > the lens > > by allowing signed ints in the regular expression. > > --- > > daemon/libvirtd.aug | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/daemon/libvirtd.aug b/daemon/libvirtd.aug > > index a70aa1d..4d40ee2 100644 > > --- a/daemon/libvirtd.aug > > +++ b/daemon/libvirtd.aug > > @@ -13,7 +13,7 @@ module Libvirtd = > > > > let str_val = del /\"/ "\"" . store /[^\"]*/ . del /\"/ "\"" > > let bool_val = store /0|1/ > > - let int_val = store /[0-9]+/ > > + let int_val = store /-?[0-9]+/ > > let str_array_element = [ seq "el" . str_val ] . del /[ > > \t\n]*/ "" > > let str_array_val = counter "el" . array_start . ( > > str_array_element . ( array_sep . str_array_element ) * ) ? . > > array_end > > > > > > ACK > Should we also create a uint_cal and uint_entry? Because this way you allow other things to be set to negative values as well, e.g. max_anonymous_clients, log_level.