ports/62931: Time::HiRes is not required for Perl 5.8 users to build/run File::Tail

[Sunpoet Po-Chuan Hsieh]

>Number:         62931
>Category:       ports
>Synopsis:       Time::HiRes is not required for Perl 5.8 users to build/run File::Tail
>Confidential:   no
>Severity:       non-critical
>Priority:       medium
>Responsible:    freebsd-ports-bugs
>State:          open
>Quarter:        
>Keywords:       
>Date-Required:
>Class:          sw-bug
>Submitter-Id:   current-users
>Arrival-Date:   Mon Feb 16 14:00:35 PST 2004
>Closed-Date:
>Last-Modified:
>Originator:     Sunpoet Po-Chuan Hsieh
>Release:        FreeBSD 5.2-CURRENT i386
>Organization:
SUNPOET.net
>Environment:
System: FreeBSD IRIS.sunpoet.net 5.2-CURRENT FreeBSD 5.2-CURRENT #0: Sun Dec 21 18:12:22 CST 2003 root@:/usr/obj/usr/src/sys/sunpoet i386

	
>Description:
Perl 5.8 comes with Time::HiRes already. That means ports/devel/p5-Time-HiRes
is not required for perl 5.8 users to build and run ports/devel/p5-File-Tail
(File::Tail).

% ls /usr/local/lib/perl5/5.8.2/mach/Time/HiRes.pm
/usr/local/lib/perl5/5.8.2/mach/Time/HiRes.pm

	
>How-To-Repeat:
% cd /usr/ports/devel/p5-File-Tail
% make build-depends-list
/usr/ports/devel/p5-Time-HiRes
/usr/ports/lang/perl5.8
% make run-depends-list
/usr/ports/devel/p5-Time-HiRes
/usr/ports/lang/perl5.8

	
>Fix:
% diff -u Makefile.orig Makefile
--- Makefile.orig	Fri Oct 24 20:01:51 2003
+++ Makefile	Thu Feb  5 20:01:12 2004
@@ -15,13 +15,16 @@
 MAINTAINER=	jeremy at external.org
 COMMENT=	Perl Module to read the end of a file as it's appended to
 
-BUILD_DEPENDS=	${site_perl}/Time/HiRes.pm:${PORTSDIR}/devel/p5-Time-HiRes
-RUN_DEPENDS=	${BUILD_DEPENDS}
-
-site_perl=	${SITE_PERL}/${PERL_ARCH}
-
 PERL_CONFIGURE=	YES
 
 MAN3=		File::Tail.3
 
-.include <bsd.port.mk>
+.include <bsd.port.pre.mk>
+
+.if ${PERL_LEVEL} < 500800
+site_perl=	${SITE_PERL}/${PERL_ARCH}
+BUILD_DEPENDS=	${site_perl}/Time/HiRes.pm:${PORTSDIR}/devel/p5-Time-HiRes
+RUN_DEPENDS=	${BUILD_DEPENDS}
+.endif
+
+.include <bsd.port.post.mk>

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