You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@spamassassin.apache.org by bu...@bugzilla.spamassassin.org on 2009/04/22 16:23:21 UTC

[Bug 6095] New: Make Time::HiRes a required module

https://issues.apache.org/SpamAssassin/show_bug.cgi?id=6095

           Summary: Make Time::HiRes a required module
           Product: Spamassassin
           Version: SVN Trunk (Latest Devel Version)
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: enhancement
          Priority: P5
         Component: Libraries
        AssignedTo: dev@spamassassin.apache.org
        ReportedBy: Mark.Martinec@ijs.si


(from a dev@spamassassin.apache.org ML):

Mark:
> Perhaps 3.3 is a good opportunity to make some Perl modules required
> or phased out. One example that comes to mind is making Time::HiRes
> a requirement - several SA modules are now jumping hoops and perform
> suboptimally when they need to deal with integer seconds.

Justin:
> +1, that seems ok to me.  a lot of distros bundle Time::HiRes now.


-- 
Configure bugmail: https://issues.apache.org/SpamAssassin/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

[Bug 6095] Make Time::HiRes a required module

Posted by bu...@bugzilla.spamassassin.org.
https://issues.apache.org/SpamAssassin/show_bug.cgi?id=6095





--- Comment #5 from Karsten Bräckelmann <gu...@rudersport.de>  2009-04-22 08:16:21 PST ---
And M::SA:Util::DependencyInfo.pm, please? :)


-- 
Configure bugmail: https://issues.apache.org/SpamAssassin/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

[Bug 6095] Make Time::HiRes a required module

Posted by bu...@bugzilla.spamassassin.org.
https://issues.apache.org/SpamAssassin/show_bug.cgi?id=6095





--- Comment #4 from Mark Martinec <Ma...@ijs.si>  2009-04-22 07:49:30 PST ---
> quick request -- could you update INSTALL, too, it still lists it under the
> Optional section.  ditto Makefile.PL's PREREQ_PM

Thanks for noticing!

Bug 6095, cont'd, make Time::HiRes a required module
Sending        INSTALL
Sending        META.yml
Sending        Makefile.PL
Transmitting file data ...
Committed revision 767550 ( https://svn.apache.org/viewcvs.cgi?view=rev&rev=767550 ).


-- 
Configure bugmail: https://issues.apache.org/SpamAssassin/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

[Bug 6095] Make Time::HiRes a required module

Posted by bu...@bugzilla.spamassassin.org.
https://issues.apache.org/SpamAssassin/show_bug.cgi?id=6095





--- Comment #7 from Mark Martinec <Ma...@ijs.si>  2009-04-22 10:21:02 PST ---
Looks like the Hudson installation is missing a Time::HiRes module:

  ERROR: the required Time::HiRes module is not installed.
  at lib/Mail/SpamAssassin/Util/DependencyInfo.pm line 304.

Would somebody who knows how to do it please install the module.


-- 
Configure bugmail: https://issues.apache.org/SpamAssassin/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

[Bug 6095] Make Time::HiRes a required module

Posted by bu...@bugzilla.spamassassin.org.
https://issues.apache.org/SpamAssassin/show_bug.cgi?id=6095


Mark Martinec <Ma...@ijs.si> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED




--- Comment #2 from Mark Martinec <Ma...@ijs.si>  2009-04-22 07:24:58 PST ---
closing


-- 
Configure bugmail: https://issues.apache.org/SpamAssassin/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

[Bug 6095] Make Time::HiRes a required module

Posted by bu...@bugzilla.spamassassin.org.
https://issues.apache.org/SpamAssassin/show_bug.cgi?id=6095





--- Comment #6 from Mark Martinec <Ma...@ijs.si>  2009-04-22 09:13:15 PST ---
> And M::SA:Util::DependencyInfo.pm, please? :)

Didn't I do it in the first round?
It is in @MODULES, previously it was in @OPTIONAL_MODULES.


-- 
Configure bugmail: https://issues.apache.org/SpamAssassin/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

[Bug 6095] Make Time::HiRes a required module

Posted by bu...@bugzilla.spamassassin.org.
https://issues.apache.org/SpamAssassin/show_bug.cgi?id=6095





--- Comment #1 from Mark Martinec <Ma...@ijs.si>  2009-04-22 07:23:50 PST ---
Sending        lib/Mail/SpamAssassin/AsyncLoop.pm
Sending        lib/Mail/SpamAssassin/DnsResolver.pm
Sending        lib/Mail/SpamAssassin/Util/DependencyInfo.pm
Sending        lib/Mail/SpamAssassin/Util/Progress.pm
Sending        lib/Mail/SpamAssassin.pm
Transmitting file data .....
Committed revision 767536 ( https://svn.apache.org/viewcvs.cgi?view=rev&rev=767536 ).


-- 
Configure bugmail: https://issues.apache.org/SpamAssassin/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

[Bug 6095] Make Time::HiRes a required module

Posted by bu...@bugzilla.spamassassin.org.
https://issues.apache.org/SpamAssassin/show_bug.cgi?id=6095





--- Comment #3 from Justin Mason <jm...@jmason.org>  2009-04-22 07:34:12 PST ---
quick request -- could you update INSTALL, too, it still lists it under the
Optional section.  ditto Makefile.PL's PREREQ_PM


-- 
Configure bugmail: https://issues.apache.org/SpamAssassin/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

[Bug 6095] Make Time::HiRes a required module

Posted by bu...@bugzilla.spamassassin.org.
https://issues.apache.org/SpamAssassin/show_bug.cgi?id=6095





--- Comment #9 from Justin Mason <jm...@jmason.org>  2009-04-22 11:54:54 PST ---
(In reply to comment #7)
> Looks like the Hudson installation is missing a Time::HiRes module:
> 
>   ERROR: the required Time::HiRes module is not installed.
>   at lib/Mail/SpamAssassin/Util/DependencyInfo.pm line 304.
> 
> Would somebody who knows how to do it please install the module.

doing it now...


-- 
Configure bugmail: https://issues.apache.org/SpamAssassin/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

[Bug 6095] Make Time::HiRes a required module

Posted by bu...@bugzilla.spamassassin.org.
https://issues.apache.org/SpamAssassin/show_bug.cgi?id=6095





--- Comment #8 from Karsten Bräckelmann <gu...@rudersport.de>  2009-04-22 10:35:24 PST ---
(In reply to comment #6)
> > And M::SA:Util::DependencyInfo.pm, please? :)
> 
> Didn't I do it in the first round?

Oops, my bad, sorry. Yes, you did. Somehow I managed to overlook the commit
message in comment 1 *and* svn updated the wrong branch...


-- 
Configure bugmail: https://issues.apache.org/SpamAssassin/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.