<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, Apr 29, 2016 at 12:41 AM, Raphaël Pinson <span dir="ltr"><<a href="mailto:raphael.pinson@camptocamp.com" target="_blank">raphael.pinson@camptocamp.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi David,<div><br></div><div><br></div><div>Great to know that a release is on the way!</div><span class=""><div><br></div><div><span style="line-height:1.5"><br></span></div><div><span style="line-height:1.5">On Fri, Apr 29, 2016 at 1:45 AM David Lutterkort <<a href="mailto:lutter@watzmann.net" target="_blank">lutter@watzmann.net</a>> wrote:</span></div></span><div class="gmail_quote"><span class=""><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>One thing that I didn't notice until now was that the Json lens has seen quite a bit of change and now accepts comments - while that's definitely a much nicer format than 'standard' Json, I wonder if it wouldn't be better to do that with a different lens, say 'JsonWithComments' and leave the existing Json lens as it was.<br></div></div></blockquote><div><br></div></span><div>I think that depends if one cares about validating the format using the lens or not. Supporting comments only harms if you're using the lens to make sure the file is standard Json, but I'm not sure sure Augeas is the best tool for this anyway.</div></div></div></blockquote><div><br></div><div>It also changes the structure of the tree, though it's all empty nodes. More importantly though, I always looked at it as a fairly straightforward example of how to use recursive lenses, and comments distract a bit from that. Would you be opposed to a PR that adds a JsonCComments lens and restores the Json lens to just plain old json ?<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_quote"><div>On another subject, I'd have loved to get the Lua features in Augeas 1.5, but I'm not sure in what form this would happen anymore…</div></div></div></blockquote><div><br></div><div>From the PR, it looks like htere's still a bit of work left, right ?<br><br></div><div>David<br> <br></div></div></div></div>