Bug 185905

Summary: [PATCH] security/logcheck: Use "USES=shebangfix" to fix perl path
Product: Ports & Packages Reporter: Yasuhiro Kimura <yasu>
Component: Individual Port(s)Assignee: Gerald Pfeifer <gerald>
Status: Closed FIXED    
Severity: Affects Only Me    
Priority: Normal    
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
patch-logcheck none

Description Yasuhiro Kimura freebsd_committer freebsd_triage 2014-01-20 11:10:02 UTC 
	
	Use "USES=shebangfix" to fix perl path of logtail and logtail2.
Comment 1 Gerald Pfeifer freebsd_committer freebsd_triage 2014-03-02 02:04:24 UTC 
Responsible Changed
From-To: freebsd-ports-bugs->gerald

I'll take care of this.
Comment 2 dfilter service freebsd_committer freebsd_triage 2014-03-02 12:10:23 UTC 
Author: gerald
Date: Sun Mar  2 12:10:16 2014
New Revision: 346759
URL: http://svnweb.freebsd.org/changeset/ports/346759
QAT: https://qat.redports.org/buildarchive/r346759/

Log:
  Leverage USES=shebangfix to fix two Perl scripts (logtail, logtail2)
  instead of doing one the manual way and not fixing the other.
  
  PR:		185905
  Submitted by:	Yasuhiro KIMURA <yasu@utahime.org>

Modified:
  head/security/logcheck/Makefile

Modified: head/security/logcheck/Makefile
==============================================================================
--- head/security/logcheck/Makefile	Sun Mar  2 11:57:14 2014	(r346758)
+++ head/security/logcheck/Makefile	Sun Mar  2 12:10:16 2014	(r346759)
@@ -33,7 +33,7 @@ OPTIONS_DEFAULT=CRON
 .endif
 
 # Enable Perl dependency for logtail script
-USES=		perl5
+USES=		perl5 shebangfix
 
 WRKSRC=		${WRKDIR}/${DISTNAME:S!_!-!}
 BINMODE=	755
@@ -45,6 +45,7 @@ SUB_LIST+=	LOGCHECK_USER=${LOGCHECK_USER
 		PW=${PW} CRON=${PORT_OPTIONS:MCRON}
 SUB_FILES=	pkg-install pkg-deinstall pkg-message
 PLIST_SUB+=	DBDIR=${DBDIR} RUNDIR=${RUNDIR}
+SHEBANG_FILES=	src/logtail src/logtail2
 CONFIG_DIRS=	cracking.d ignore.d.paranoid ignore.d.server \
 		ignore.d.workstation violations.d violations.ignore.d
 DOCS=		AUTHORS CHANGES CREDITS LICENSE TODO docs/README*
@@ -72,8 +73,6 @@ post-patch:
 do-build:
 	@${REINPLACE_CMD} -e 's!/var/log/syslog!/var/log/messages!' \
 		${WRKSRC}/etc/logcheck.logfiles
-	@${REINPLACE_CMD} -e "s|#!/usr/bin/perl|#!${PERL}|" \
-		${WRKSRC}/src/logtail
 
 do-install:
 	${INSTALL_SCRIPT} ${WRKSRC}/src/logcheck ${PREFIX}/sbin
_______________________________________________
svn-ports-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-ports-all
To unsubscribe, send any mail to "svn-ports-all-unsubscribe@freebsd.org"
Comment 3 Gerald Pfeifer freebsd_committer freebsd_triage 2014-03-02 12:20:18 UTC 
State Changed
From-To: open->closed

Thank you for working out this nice improvement and submitting it 
to us, Kimura-san.  I just committed your changes. 

I noticed this port has not been converted to staging yet.  Is this 
something you could look into, perhaps?