Bug 42981 - Update port: security/hping strip binary
Summary: Update port: security/hping strip binary
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: freebsd-ports (Nobody)
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2002-09-20 10:40 UTC by Nicolas Jombart
Modified: 2002-10-22 02:03 UTC (History)
0 users

See Also:

Attachments
hping.diff (683 bytes, patch)
2002-09-20 10:40 UTC, Nicolas Jombart 		no flags Details | Diff
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 Nicolas Jombart 2002-09-20 10:40:01 UTC 
	. Strip binary
	. Bump PORTREVISION

	No files added or deleted.
Comment 1 Nicolas.Jombart 2002-09-20 10:47:55 UTC 
on Fri, Sep 20, 2002 at 11:33:00AM +0200, Nicolas Jombart wrote :
> +post-install:
> +	strip ${PREFIX}/sbin/hping

Sorry :

diff -ruN /usr/ports/security/hping/Makefile hping/Makefile
--- /usr/ports/security/hping/Makefile	Mon Sep 17 07:50:40 2001
+++ hping/Makefile	Fri Sep 20 11:43:50 2002
@@ -7,7 +7,7 @@
 
 PORTNAME=	hping
 PORTVERSION=	2.0.0r1
-PORTREVISION=	1
+PORTREVISION=	2
 PORTEPOCH=	1
 CATEGORIES=	security net
 MASTER_SITES=	http://www.hping.org/
@@ -20,7 +20,7 @@
 CONFIGURE_ARGS+=	--force-libpcap
 USE_GMAKE= 	yes
 
-MAN8=	hping.8
+MAN8=		hping.8
 
 do-install:
 	${INSTALL_PROGRAM} ${WRKSRC}/hping2 ${PREFIX}/sbin/hping
@@ -31,5 +31,8 @@
 	${INSTALL_DATA} ${WRKSRC}/docs/${i} ${PREFIX}/share/doc/hping/
 .endfor
 .endif
+
+post-install:
+	${STRIP_CMD} ${PREFIX}/sbin/hping
 
 .include <bsd.port.mk>
Comment 2 Adam Weinberger freebsd_committer freebsd_triage 2002-10-22 02:02:33 UTC 
State Changed
From-To: open->closed

Even though the program's build says "use `make strip' to strip hping2 binary" 
at the end, it is not necessary to strip it manually. the INSTALL_PROGRAM 
directive strips executables automatically. Adding in this patch would be 
redundant.