Bug 185905 - [PATCH] security/logcheck: Use "USES=shebangfix" to fix perl path
Summary: [PATCH] security/logcheck: Use "USES=shebangfix" to fix perl path
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: Normal Affects Only Me
Assignee: Gerald Pfeifer
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-01-20 11:10 UTC by Yasuhiro Kimura
Modified: 2014-03-02 12:22 UTC (History)
0 users

See Also:

Attachments
patch-logcheck (960 bytes, text/plain)
2014-01-20 11:10 UTC, Yasuhiro Kimura 		no flags Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
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?