<div dir="ltr">On Wed, Jul 23, 2008 at 8:10 PM, David Lutterkort <<a href="mailto:dlutter@redhat.com">dlutter@redhat.com</a>> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="Ih2E3d">On Wed, 2008-07-23 at 16:18 +0200, Raphaël Pinson wrote:<br>
> I can't say I'm a big fan of the oneline (pre|post)rotate hooks (I<br>
> prefer to split them into 'line' nodes) but at least it works now.<br>
<br>
</div>Is it more a taste thing or a usability concern ? If the latter, we<br>
should address that. Though I still think Augeas shouldn't pretend to<br>
understand something it clearly doesn't - the script is just an opaque<br>
blob to Augeas.</blockquote><div><br>I personally see it as a usability matter. I'm just thinking of myself as the user of the API and inserting (or reading) a blog like <br>prerotate = "                if [ -f /var/run/apache2.pid ]; then<br>

                        /etc/init.d/apache2 restart > /dev/null<br>
                fi<br>
"<br><br>instead of<br><br>prerotate/line[1] = "if [ -f /var/run/apache2.pid ]; then"<br>prerotate/line[2] = "/etc/init.d/apache2 restart > /dev/null"<br>prerotate/line[3] = "fi"<br>
<br>To me, the latter is much easier to parse to to read. If I had to deal with the first case, I would be afraid to completely mess up the look of the conffile with spaces and tabs. Maybe that's just my opinion. What do others on the list think?<br>
<br><br>Raphael<br> </div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><br>
<font color="#888888"><br>
David<br>
<br>
<br>
</font></blockquote></div><br></div>