Hello, I've (mostly) fixed it. The patch is here: http://func.pastebin.com/m2bd4dfb The full new module is here: http://func.pastebin.com/d3dbcb44c The modified/improved test script is here: http://func.pastebin.com/m25900459 Modifications on the module: - fix for trailing / - new function=backup (because in func+augeas we can't use /augeas/save; each action is a new augeas instance) - new function=getenv Modifications on the test script: - the tests are now 'chrooted' with AUGEAS_ROOT (but you need to put something like export AUGEAS_ROOT='/tmp' in /etc/init.d/funcd, I don't know yet of a better way to pass this variable to func) - new tests, in particular for extended paths Todo: - fix for some extended path failures - find a way to use /augeas/save in a persistent way - ability to work on any single conf. file when it will be available in Augeas Louis Le samedi 21 février 2009 à 11:52 +0100, Louis Coilliot a écrit : > Hello, > > great. > > I need to push you a new version because since augeas 0.4 the path > expression is more strict and a trailing / is not allowed anymore. > My module does not handle this. > > So for now it's broken with the last augeas version. But it's easy to > adjust. I'll do this ASAP. > > In addition, I agree with you that the module name is ugly > (confmgt_augeas) > > We could rename it confaug or simply augeas. I like confaug because it > suggests that it deals with conf-iguration management, for people who > would not know about augeas. > > I thought I would provide some other modules for configuration > management (based on regexps, or ConfigParser for ini files), but I was > told by Raphael Pinson at the FOSDEM this month in the Augeas conference > that the developpers were considering letting people load only a > specific file to work on, and specify a lens to handle it. > So ini files and other files with generic formats and not located in the > standard system path could be managed with the same module. I'm looking > forward to being able too use this > > > Louis > > > > Le vendredi 20 février 2009 à 17:38 -0500, Adrian LIkins a écrit : > > Louis Coilliot wrote: > > > > I'm thinking it's about time for another release. > > > > (...) > > > > So if anyone has anything outstanding (patches, modules, > > > > must have features, must fix bugs, docs, etc) it would be > > > > great if you could get it ready soon ;-) > > > >(...) > > > > Adrian > > > > Sorry about the delay, this kind of dropped off my radar. > > > > I merged this and pushed it. Probably need to adopt the docs info to > > the module docs on the wiki, and see if we can integrate the test > > script into the unit tests. > > > > Let me know if there are any changes that need merged. > > > > Adrian > _______________________________________________ Func-list mailing list Func-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/func-list
