ports/103966: devel/p5-IO-Multiplex includes Time::HiRes regardless of perl version

[Jeremy Chadwick]

>Number:         103966
>Category:       ports
>Synopsis:       devel/p5-IO-Multiplex includes Time::HiRes regardless of perl version
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    freebsd-ports-bugs
>State:          open
>Quarter:        
>Keywords:       
>Date-Required:
>Class:          update
>Submitter-Id:   current-users
>Arrival-Date:   Wed Oct 04 06:30:17 GMT 2006
>Closed-Date:
>Last-Modified:
>Originator:     Jeremy Chadwick
>Release:        FreeBSD 6.2-PRERELEASE i386
>Organization:
Parodius Networking
>Environment:
System: FreeBSD icarus.home.lan 6.2-PRERELEASE FreeBSD 6.2-PRERELEASE #0: Mon Oct 2 10:20:50 PDT 2006 root at icarus.home.lan:/usr/obj/usr/src/sys/ICARUS i386
>Description:
	p5-IO-Multiplex blindly includes a dependency on devel/p5-Time-HiRes
	regardless of which version of perl is installed on the system.

	Starting with perl 5.7.2, Time::HiRes is included with perl -- however,
	there are known issues with the base-included Time::HiRes module in
	perl 5.8.4 and below.  A simple .if wrapper which checks PERL_LEVEL
	should solve this.

	If you need something to compare with (for sake of integrity), look at
	devel/p5-Spoon, net/p5-Net-Amazon, net/p5-POE-Component-Client-Traceroute,
	or www/p5-Apache-ASP.

	NOTE: True, perl 5.8.8 includes Time::HiRes version 1.86 while the port
	is for 1.91, but this really should not make a difference on UNIX.  The
	changes beween 1.86 and 1.91 are mainly for Cygwin, minus one fixup in
	1.91 for SuSE Linux, which obviously doesn't apply here.  Here's the
	official Time::HiRes ChangeLog if you have concerns committing this:
	http://search.cpan.org/src/JHI/Time-HiRes-1.91/Changes
>How-To-Repeat:
	n/a
>Fix:
	Apply the below patch.


--- Makefile.orig       Sun Feb 15 06:51:55 2004
+++ Makefile    Tue Oct  3 23:11:02 2006
@@ -15,8 +15,10 @@
 MAINTAINER=    tobez at FreeBSD.org
 COMMENT=       IO::Multiplex - Manage IO on many file handles

+.if ${PERL_LEVEL} < 500805
 BUILD_DEPENDS= ${SITE_PERL}/${PERL_ARCH}/Time/HiRes.pm:${PORTSDIR}/devel/p5-Time-HiRes
 RUN_DEPENDS=   ${BUILD_DEPENDS}
+.endif

 PERL_CONFIGURE=        yes


>Release-Note:
>Audit-Trail:
>Unformatted: