https://issues.apache.org/SpamAssassin/show_bug.cgi?id=6889
Kevin A. McGrail <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED CC| |[email protected] Resolution|--- |FIXED --- Comment #2 from Kevin A. McGrail <[email protected]> --- (In reply to comment #1) > > One of them: xt/50_dkim..........................skipped > > This rule just runs t/dkim.t. However, that rule appears to be disabled > > with an && 0 in the conditions test to run. > > > > Should this rule be removed in favor of dkim2? This test hasn't worked > > since 2007. > > Yes, please do remove xt/50_dkim.t and t/50_dkim.t, > they have been superseded by 50_dkim2.t > (or, drop 50_dkim and rename 50_dkim2 to 50_dkim). > > Remember to update the MANIFEST accordingly. I had to do two commits to handle the deletion but should be done now: svn commit -m 'replacing dkim test with dkim2' xt t MANIFEST Sending MANIFEST Adding t/dkim.t Deleting t/dkim2.t Deleting xt/50_dkim2.t Transmitting file data . Committed revision 1432183. Running a disttest and all xt tests now to confirm all is still ok. -- You are receiving this mail because: You are the assignee for the bug.
