[prev in list] [next in list] [prev in thread] [next in thread] 

List:       collectd
Subject:    Re: [collectd] Bareword "LOG_ERR" not allowed while "strict subs"
From:       Casey Vega <cvega () mediatemple ! net>
Date:       2012-07-18 15:26:54
Message-ID: CAHb=1CE1FLVo-ZWpf9XvrkZzF_zrj65XRAYC_rVSgfU=U+1Ktw () mail ! gmail ! com
[Download RAW message or body]

[Attachment #2 (multipart/alternative)]


Excellent. Testing is good and I like your approach.

I guess I'm still baffled as to why the Collectd module would use strict
and warning pragma when it doesn't actually adhere to either. In a nutshell
why not remove these all together?

I did try and removing both statements from the module appear to fix my
issue. I'm not sure what effect this would have in production and I surely
don't want to manage a custom package. In the meantime I've opted to build
a more simple package.

Thanks for the test module! Much appreciated.

On Wed, Jul 18, 2012 at 3:16 AM, <collectd@faxm0dem.org> wrote:

> Hi,
>
> On Fri, Jul 13, 2012 at 04:51:29PM +0200, Poil wrote:
> > I'm really interested by your module, keep me up to date about it
>
> It's far from perfect, but you can grab a copy on github.
> Patches/bugreports are very welcome.
>
> https://github.com/faxm0dem/Test-Collectd-Plugins
>
>
> _______________________________________________
> collectd mailing list
> collectd@verplant.org
> http://mailman.verplant.org/listinfo/collectd
>



-- 
*Casey Vega*
Software QA Engineer | (mt) Media Temple
Office: 3108415611
www.mediatemple.com | @mediatemple <http://twitter.com/mediatemple>

[Attachment #5 (text/html)]

Excellent. Testing is good and I like your approach. <div><br></div><div>I guess \
I&#39;m still baffled as to why the Collectd module would use strict and warning \
pragma when it doesn&#39;t actually adhere to either. In a nutshell why not remove \
these all together?</div> <div><br></div><div>I did try and removing both statements \
from the module appear to fix my issue. I&#39;m not sure what effect this would have \
in production and I surely don&#39;t want to manage a custom package. In the meantime \
I&#39;ve opted to build a more simple package.</div> <div><br></div><div>Thanks for \
the test module! Much appreciated.<br><br><div class="gmail_quote">On Wed, Jul 18, \
2012 at 3:16 AM,  <span dir="ltr">&lt;<a href="mailto:collectd@faxm0dem.org" \
target="_blank">collectd@faxm0dem.org</a>&gt;</span> wrote:<br> <blockquote \
class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc \
solid;padding-left:1ex">Hi,<br> <div class="im"><br>
On Fri, Jul 13, 2012 at 04:51:29PM +0200, Poil wrote:<br>
&gt; I&#39;m really interested by your module, keep me up to date about it<br>
<br>
</div>It&#39;s far from perfect, but you can grab a copy on github.<br>
Patches/bugreports are very welcome.<br>
<br>
<a href="https://github.com/faxm0dem/Test-Collectd-Plugins" \
target="_blank">https://github.com/faxm0dem/Test-Collectd-Plugins</a><br> <div \
class="HOEnZb"><div class="h5"><br> <br>
_______________________________________________<br>
collectd mailing list<br>
<a href="mailto:collectd@verplant.org">collectd@verplant.org</a><br>
<a href="http://mailman.verplant.org/listinfo/collectd" \
target="_blank">http://mailman.verplant.org/listinfo/collectd</a><br> \
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div \
style="width:470px;font-family:&#39;Lucida \
Grande&#39;,sans-serif;font-size:13px;line-height:18px;color:rgb(46,46,46);border-top- \
width:1px;border-top-style:dashed;border-top-color:rgb(218,218,218);border-bottom-widt \
h:1px;border-bottom-style:dashed;border-bottom-color:rgb(218,218,218);padding:5px \
0px"> <b>Casey Vega</b><br>Software QA Engineer | (mt) Media Temple<br>Office: \
3108415611<br><a href="http://www.mediatemple.com/" \
style="text-decoration:none;color:rgb(12,111,206)" \
target="_blank">www.mediatemple.com</a> | <a href="http://twitter.com/mediatemple" \
style="text-decoration:none;color:rgb(12,111,174)" \
target="_blank">@mediatemple</a></div> <br \
style="font-family:Times;font-size:medium"><br> </div>



_______________________________________________
collectd mailing list
collectd@verplant.org
http://mailman.verplant.org/listinfo/collectd


[prev in list] [next in list] [prev in thread] [next in thread]