Netsaint plugins are programs, scripts needed to check various services on monitored hosts. But this software was written to use on Linux, so there are incorrect path to tools like sed, diff, tail or grep in plugins scripts. Fix: Here it is a patch for port Makefile to correct these paths: ----------------- cut here -------------------------------------------- Greetings -- /~\ The ASCII Krzysztof Stryjek \ / Ribbon Campaign wtp@NOSPAM.mud.pl X Against HTML http://mud.pl/~wtp/ / \ Email! GG: 3608113 ICQ: 124986907--drTfIavhWuqqz02TezajFA7S32CNNRXARzKwGNgowkgIffO8 Content-Type: text/plain; name="file.diff" Content-Transfer-Encoding: 7bit Content-Disposition: attachment; filename="file.diff" --- Makefile Thu Feb 20 19:51:00 2003 +++ Makefile.new Wed Oct 1 12:05:28 2003 @@ -18,6 +18,7 @@ USE_SUBMAKE= yes GNU_CONFIGURE= yes +USE_REINPLACE= yes CONFIGURE_ARGS= --sbindir=${PREFIX}/share/netsaint/sbin \ --libexecdir=${PREFIX}/libexec/netsaint \ --datadir=${PREFIX}/share/netsaint \ @@ -35,6 +36,10 @@ .if exists(${WRKDIRPREFIX}${.CURDIR}/Makefile.inc) .include "${WRKDIRPREFIX}${.CURDIR}/Makefile.inc" .endif + +post-patch: + @${FIND} ${WRKSRC}/plugins-scripts/ -type f -perm -+x | ${XARGS} -n1 \ + ${REINPLACE_CMD} -E "s,/bin/(diff|sed|tail|grep),/usr/bin/\1,g" post-clean: @${RM} -f ${WRKDIRPREFIX}${.CURDIR}/Makefile.inc ----------------- cut here -------------------------------------------- How-To-Repeat: Just install netsaint plugins and try run /usr/local/libexec/netsaint/log_check for example. There will be error message: /bin/sed: No such file or directory.
Although the patch applies correctly, I still get the 'can't find /bin/sed' error when using check_log. Can the submitter confirm this? Also, I have patches for pkg-plist that removes the files in libexec/.
Responsible Changed From-To: freebsd-ports-bugs->linimon I'll take a look.
State Changed From-To: open->feedback Note that I've asked the submitter for feedback.
State Changed From-To: feedback->closed I made the regex a bit lamer (because REINPLACE_CMD could still be perl on older systems), and this seems to work for me. Committed, thanks!
Responsible Changed From-To: linimon->petef Race condition, steal this from linimon with his blessing.