<div dir="ltr">Thank you Markus,<div><br></div><div>I was just minutes ago looking on the CPAN for amavis-related modules and came across your name.  I'd never seen a collectd plugin before :-)</div><div><br></div><div>I saw your unofficial repository on github as well when I was looking for online repos but moved on when I saw that it is unofficial.  I will plan to submit pull requests against it once I get to the point of submitting patches.</div><div><br></div><div>Cheers,</div><div><br></div><div>C.J.</div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Mar 14, 2016 at 12:32 AM, Benning, Markus <span dir="ltr"><<a href="mailto:ich@markusbenning.de" target="_blank">ich@markusbenning.de</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 2016-03-14 04:06, C.J. Collier wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I'm doing a quick review of the amavisd source here:<br>
<br>
<a href="http://anonscm.debian.org/cgit/amavisd-new/pkg-amavisd-new.git/tree/amavisd" rel="noreferrer" target="_blank">http://anonscm.debian.org/cgit/amavisd-new/pkg-amavisd-new.git/tree/amavisd</a><br>
<br>
First of all, is there an online repository upstream of this one that<br>
I should be using instead?  The only thing I could find was the static<br>
release .tar.xz files referenced here:<br>
<br>
<a href="http://anonscm.debian.org/cgit/amavisd-new/pkg-amavisd-new.git/tree/debian/watch" rel="noreferrer" target="_blank">http://anonscm.debian.org/cgit/amavisd-new/pkg-amavisd-new.git/tree/debian/watch</a><br>
<br>
I would like to make some changes to address some or all of the issues<br>
raised by perlcritic:<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
cjac@hype0:/usr/src/git/alioth/pkg-amavisd-new$ perlcritic amavisd |<br>
wc -l<br>
<br>
288<br>
</blockquote>
<br>
But I'm afraid that I will break things if I just go hacking around in<br>
there.  I would expect there to be a rather extensive set of tests in<br>
the t/ directory for a project of this size.  I'm happy to contribute<br>
some if they don't exist.  I just want to have a bit of a safety net<br>
to fall back on.<br>
</blockquote>
<br></span>
Unfortunately there is no public source code repository which could be used<br>
to directly contribute to amavisd-new.<br>
<br>
I recently created a repository from the tar balls at:<br>
<br>
<a href="https://github.com/benningm/amavisd-new" rel="noreferrer" target="_blank">https://github.com/benningm/amavisd-new</a><br>
<br>
It could be used to collect patches from this mailling list an others.<span class="HOEnZb"><font color="#888888"><br>
<br>
<br>
<br>
Markus<br>
<br>
-- <br>
<a href="https://markusbenning.de/" rel="noreferrer" target="_blank">https://markusbenning.de/</a><br>
</font></span></blockquote></div><br></div>