[augeas-devel] Re: [PATCH] Add default lens (the equivalent of sysconfig on Debian systems)

[Free Ekanayaka]

|--==> On Fri, 05 Sep 2008 16:41:17 +0000, David Lutterkort <lutter at redhat.com> said:

  DL> On Fri, 2008-09-05 at 12:11 +0200, Free Ekanayaka wrote:
  >># HG changeset patch
  >># User Free Ekanayaka <free at 64studio.com>
  >># Date 1220609447 -7200
  >># Node ID 5293b6f11f988f2dafc7122c9ec3cae3504a6a63
  >># Parent  76b4a5665cd6a32cc46131091c2860176b864033
  >>Add default lens (the equivalent of sysconfig on Debian systems)

  DL> This looks good, except that the module name is a little too generic.
  DL> Can we call this Debian_default or something similar ?

At the moment the shellvars.aug module is used by

ifcfg.aug
sysconfig.aug
default.aug

but none of them actually adds some more information to Shellvars. So
why not merging all of them directly in shellvars?

I guess just adding something like:

let filter = Util.stdexcl
           . incl "/etc/sysconfig/*"
           . incl "/etc/default/*"
           . incl "/etc/sysconfig/network-scripts/ifcfg"

According to the Debian policy all files /etc/default/ must be
Shellvars-compatible:

"This file should be sourced by the script when the script runs. It
must contain only variable settings and comments in SUSv3 sh format" [0]

I'm not sure about RedHat systems though.

Ciao,

Free

[0] http://www.debian.org/doc/debian-policy/ch-opersys.html (9.3.2 Writing the scripts)