Hi David,<br><br>Thanks for the comments.<br><br>I've squashed the commits into 12 commits which all pass unit tests and added notes in NEWS. See <a href="https://github.com/raphink/augeas/compare/dev;inifile_squashed">https://github.com/raphink/augeas/compare/dev;inifile_squashed</a><br>
<br><br>Raphaël<br><br><br><br><div class="gmail_quote">On Wed, Feb 6, 2013 at 9:29 PM, David Lutterkort <span dir="ltr"><<a href="mailto:lutter@redhat.com" target="_blank">lutter@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi Raphael,<br>
<div class="im"><br>
On Fri, 2013-01-11 at 00:31 +0100, Raphaël Pinson wrote:<br>
> Last update for the day:<br>
><br>
> I've made new changes to support other features by default:<br>
><br>
> * IniFile now supports doseol by default, because there's often<br>
> (sadly) IniFiles can use \r\n as eol. The IniFile-based lenses have<br>
> been fixed accordingly;<br>
> * There are now entry_generic and entry_multiline_generic definitions<br>
> allowing to avoid redefining too much in lenses, when # and ; are<br>
> allowed in values for example.<br>
><br>
> Again, these changes require a few adaptions in existing IniFile-based<br>
> lenses. The diff in the patch series [1] demonstrates the kind of<br>
> changes that are necessary to make, essentially:<br>
><br>
> * Excluding \r as well as \n from values because of the doseol<br>
> support;<br>
> * Fixing definitions using deprecated IniFile definitions (such as<br>
> sto_to_eol or sto_to_comment). I could optionally keep these in<br>
> inifile.aug for compatibility purpose, even though they are not used<br>
> anymore;<br>
<br>
</div>Yes, they should stay around, with a deprecation warning in the comment.<br>
<div class="im"><br>
> * Changing { "#comment" } into {} in test files since empty comments<br>
> are mapped as empty nodes;<br>
> * Fixing double quoted values in tests (and code) since smart quotes<br>
> are used.<br>
><br>
> All these are not huge changes but they do require editing/debugging,<br>
> although I believe the benefits are well worth it (more standard<br>
> lenses, support for smart quotes, simple to use "loose" lenses, doseol<br>
> support).<br>
<br>
</div>All these changes make sense to me; my only comment is that your patch<br>
series needs to be modified a bit: you have to make sure that all the<br>
tests pass after each patch (to preserve bisectability), which means<br>
that you have to combine any change to IniFile with all the fixups<br>
required in lenses.<br>
<br>
We also need to make sure your notes on the changes make it into the<br>
release notes etc. Just add them to the NEWS file already so we won't<br>
forget that.<br>
<span class="HOEnZb"><font color="#888888"><br>
David<br>
<br>
<br>
</font></span></blockquote></div><br><br clear="all"><br>-- <br>Raphaël Pinson<br>Administrateur Systèmes & Réseaux<br>
Camptocamp France<br>
Savoie Technolac<br>
BP 352<br>
48, avenue du Lac du Bourget<br>
73372 Le Bourget du Lac, Cedex<br>
<a href="http://www.camptocamp.com" target="_blank">www.camptocamp.com</a><br>