<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Sep 5, 2014 at 6:41 PM, Loïc Hoguin <span dir="ltr"><<a href="mailto:essen@ninenines.eu" target="_blank">essen@ninenines.eu</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 09/05/2014 11:25 PM, Michael Truog wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br></blockquote></span><span class="">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
2) Use a parse transform to rewrite all the source code that wants to do<br>
logging so that it can determine whether it needs to log by checking<br>
global state, probably in ets.  This approach is taken in lager with the<br>
parse transform at<br>
<a href="https://github.com/basho/lager/blob/master/src/lager_transform.erl" target="_blank">https://github.com/basho/<u></u>lager/blob/master/src/lager_<u></u>transform.erl</a> .<br>
</blockquote>
<br></span>
I do not like this one, especially since lager will simply fail if modules were not compiled with the parse_transform (the functions aren't defined). So you have to compile everything with it, can't call from the shell (at least not the same way), yada yada. Very impractical. Efficiency is always good to have but that one is quite costy.<br></blockquote><div><br></div><div>Instead of complaining in text, how about contributing in code? If there's something you don't like about lager, Andrew is easy to find in IRC or on email -- discuss your concerns with him, propose a solution, and submit a PR. Andrew is quite reasonable and pragmatic.</div><div><br></div><div>--steve</div></div></div></div>