<div dir="ltr">I think that re-factoring amavis into smaller chunks so that spamassassin can «use» those modules will reduce the duplicated code, if not the duplicated effort.<div><br></div><div>What is the benefit of having all of the 71 packages in the same file?  This will make it difficult to write and exercise unit tests.  If it's the concern of handling 71 different file handles, the build system could concatenate all of the individual files to a single monolithic script for distribution.</div><div><br></div><div>Can you tell I've been thinking about this the last 24 hours? :-)</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:02 PM, Josh Hamell <span dir="ltr"><<a href="mailto:jhamell@sift.net" target="_blank">jhamell@sift.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 3/8/2016 3:00 AM, <a href="mailto:amavis-users-request@amavis.org">amavis-users-request@amavis.org</a> wrote:<br>
> On 7 Mar 2016, at 12:13, Josh Hamell <<a href="mailto:jhamell@sift.net">jhamell@sift.net</a>> wrote:<br>
>> ><br>
>> > Amavis headers are injected in immediately before delivery, and<br>
>> > therefore aren't available for SA to analyze.<br>
> This is my understanding, amavis headers aren't there until after SA<br>
<br>
Thank you - SA is batting about 92% detection rate for the resulting<br>
BANNED messages.  If it does become an issue, I will need to configure<br>
SA to look at attachments (duplicating the work of AMAVIS), but this may<br>
be enough for now.<br>
<span class="HOEnZb"><font color="#888888"><br>
-Josh<br>
</font></span></blockquote></div><br></div>