Bug 207265

Summary: [PATCH] irc/ircd-ratbox pre-build target broken
Product: Ports & Packages Reporter: John Marshall <john>
Component: Individual Port(s)Assignee: Bernard Spil <brnrd>
Status: Closed FIXED    
Severity: Affects Some People CC: brnrd, moggie
Priority: --- Keywords: patch
Version: LatestFlags: bugzilla: maintainer-feedback? (moggie)
Hardware: Any   
OS: Any   
URL: https://reviews.freebsd.org/D5479
Attachments:
Description Flags
Extract bsd.port.options.mk from the middle of the pre-build target script none

Description John Marshall 2016-02-17 00:37:37 UTC

    
Comment 1 John Marshall 2016-02-17 00:45:59 UTC
Created attachment 167103 [details]
Extract bsd.port.options.mk from the middle of the pre-build target script

Guessing that moving the bsd.port.options.mk .include into the middle of the pre-build target script in r408952 was unintentional. This is what it does...

rwsrv04# cd irc/ircd-ratbox
rwsrv04# make clean
make: "/build/ports/irc/ircd-ratbox/Makefile" line 135: Unassociated shell command "@${ECHO_MSG} "Executing ircd-shortcut.pl for ircd-shortcuts generation.""
make: "/build/ports/irc/ircd-ratbox/Makefile" line 136: Unassociated shell command "${PERL} ${WRKSRC}/contrib/ircd-shortcut.pl"
make: Fatal errors encountered -- cannot continue
make: stopped in /build/ports/irc/ircd-ratbox

The attached patch relocates the .include, hopefully to its proper place.
Comment 2 Bernard Spil freebsd_committer freebsd_triage 2016-02-27 20:30:23 UTC
Hi John,

Thanks for the patch. The include must indeed happen before the target definitions.

See review D5479

Hope to commit this soon (pending mentor review)
Comment 3 Bernard Spil freebsd_committer freebsd_triage 2016-07-03 12:23:04 UTC
Closed by http://svnweb.freebsd.org/changeset/ports/410482