On 03/06/2012 10:54 PM, Dominic Cleal wrote:
The new lens is here:
http://git.fedorahosted.org/git/?p=augeas.git;a=blob;f=lenses/shellvars.aug

Could you please try it out (augtool -I<lens dir>) and let me know if
there are any more exclusions required for non-shellvars files caught
with the glob or if valid files fail to parse?

Run "print /augeas//error" in augtool to capture a list of files and errors.

I haven't had time to test it yet, but I think you also need to exclude bootloader as it is also present in shellvars_list.lns. Same bug was found by David about the kernel file. Patch attached

--
Duncan Mac-Vicar P. - http://www.suse.com/

SUSE LINUX Products GmbH, GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer, HRB 16746 (AG Nürnberg)
Maxfeldstraße 5, 90409 Nürnberg, Germany

>From 47eb6fdbe134d8add59ed26e1a21c2703373e250 Mon Sep 17 00:00:00 2001
From: Duncan Mac-Vicar P <[email protected]>
Date: Wed, 28 Mar 2012 12:44:33 +0200
Subject: [PATCH] bootloader is also present in shellvars_list, exclude from
 shellvars.

---
 lenses/shellvars.aug |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/lenses/shellvars.aug b/lenses/shellvars.aug
index 7812b4d..0a5092a 100644
--- a/lenses/shellvars.aug
+++ b/lenses/shellvars.aug
@@ -158,6 +158,7 @@ module Shellvars =
 
   let filter_sysconfig =
       sc_incl "*" .
+      sc_excl "bootloader" .
       sc_excl "hw-uuid" .
       sc_excl "hwconf" .
       sc_excl "ip*tables" .
-- 
1.7.7

_______________________________________________
augeas-devel mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/augeas-devel

Reply via email to