Date: Fri, 20 Nov 2020 14:16:28 +0000 (UTC) From: Kyle Evans <kevans@FreeBSD.org> To: ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org Subject: svn commit: r555761 - in head/textproc/bsdgrep: . files Message-ID: <202011201416.0AKEGSNI095391@repo.freebsd.org>
next in thread | raw e-mail | index | archive | help
Author: kevans Date: Fri Nov 20 14:16:28 2020 New Revision: 555761 URL: https://svnweb.freebsd.org/changeset/ports/555761 Log: textproc/bsdgrep: fix the build on recent -CURRENT A seatbelt was added to throw a warning if a "SUBDIR." variable appeared non-empty, since that indicates something was trying to expand a MK_ variable most likely and either opts.mk wasn't included or the name of the variable used is wrong. In this particular case, we're repackaging a part from base. I had stripped the src.opts.mk include but hadn't thought to replace it with bsd.opts.mk for userlandy stuff. This also goes as far as fixing the test build, but doesn't yet hook up the tests. On FreeBSD <13 these need a kyua/atf test dep, but I'd like to do a sweep and see if there are any other ports (e.g. from NetBSD base or FreeBSD base) that also have a dependency to weigh in on if this should just be a one-off osversion check or a kyua/atf USES. MFH: 2020Q4 (blanket: build fix) Added: head/textproc/bsdgrep/files/patch-tests_Makefile (contents, props changed) Modified: head/textproc/bsdgrep/Makefile head/textproc/bsdgrep/files/patch-Makefile Modified: head/textproc/bsdgrep/Makefile ============================================================================== --- head/textproc/bsdgrep/Makefile Fri Nov 20 12:56:48 2020 (r555760) +++ head/textproc/bsdgrep/Makefile Fri Nov 20 14:16:28 2020 (r555761) @@ -19,7 +19,8 @@ USES= uidfix MAKE_ENV+= BINDIR="${PREFIX}/bin" \ MANDIR="${MANPREFIX}/man/man" \ CATDIR="${MANPREFIX}/man/man" \ - NLSDIR="${PREFIX}/share/nls" + NLSDIR="${PREFIX}/share/nls" \ + WITHOUT_TESTS=yes CFLAGS+= -std=gnu99 OPTIONS_DEFINE= GNU_COMPAT NLS @@ -38,6 +39,6 @@ pre-install-NLS-on: ${MKDIR} ${STAGEDIR}${PREFIX}/share/nls/gl_ES.ISO8859-1 post-install: - ${LN} -sf ${PREFIX}/bin/grep ${STAGEDIR}${PREFIX}/bin/bsdgrep + ${RLN} ${STAGEDIR}${PREFIX}/bin/grep ${STAGEDIR}${PREFIX}/bin/bsdgrep .include <bsd.port.mk> Modified: head/textproc/bsdgrep/files/patch-Makefile ============================================================================== --- head/textproc/bsdgrep/files/patch-Makefile Fri Nov 20 12:56:48 2020 (r555760) +++ head/textproc/bsdgrep/files/patch-Makefile Fri Nov 20 14:16:28 2020 (r555761) @@ -1,10 +1,11 @@ --- Makefile.orig 2017-10-19 20:07:52 UTC +++ Makefile -@@ -2,31 +2,13 @@ +@@ -2,31 +2,14 @@ # $FreeBSD: head/usr.bin/grep/Makefile 322515 2017-08-14 19:21:37Z ngie $ # $OpenBSD: Makefile,v 1.6 2003/06/25 15:00:04 millert Exp $ -.include <src.opts.mk> ++.include <bsd.opts.mk> - -.if ${MK_BSD_GREP} == "yes" PROG= grep Added: head/textproc/bsdgrep/files/patch-tests_Makefile ============================================================================== --- /dev/null 00:00:00 1970 (empty, because file is newly added) +++ head/textproc/bsdgrep/files/patch-tests_Makefile Fri Nov 20 14:16:28 2020 (r555761) @@ -0,0 +1,58 @@ +--- tests/Makefile.orig 2020-11-20 14:03:37 UTC ++++ tests/Makefile +@@ -3,55 +3,5 @@ + PACKAGE= tests + + ATF_TESTS_SH+= grep_freebsd_test +-NETBSD_ATF_TESTS_SH= grep_test +- +-${PACKAGE}FILES+= d_basic.out +-${PACKAGE}FILES+= d_begin_end_a.out +-${PACKAGE}FILES+= d_begin_end_b.out +-${PACKAGE}FILES+= d_binary.out +-${PACKAGE}FILES+= d_color_a.in +-${PACKAGE}FILES+= d_color_a.out +-${PACKAGE}FILES+= d_color_b.in +-${PACKAGE}FILES+= d_color_b.out +-${PACKAGE}FILES+= d_color_c.out +-${PACKAGE}FILES+= d_context2_a.out +-${PACKAGE}FILES+= d_context2_b.out +-${PACKAGE}FILES+= d_context2_c.out +-${PACKAGE}FILES+= d_context_a.in +-${PACKAGE}FILES+= d_context_a.out +-${PACKAGE}FILES+= d_context_b.in +-${PACKAGE}FILES+= d_context_e.in +-${PACKAGE}FILES+= d_context_b.out +-${PACKAGE}FILES+= d_context_c.out +-${PACKAGE}FILES+= d_context_d.out +-${PACKAGE}FILES+= d_context_e.out +-${PACKAGE}FILES+= d_context_f.out +-${PACKAGE}FILES+= d_context_g.out +-${PACKAGE}FILES+= d_egrep.out +-${PACKAGE}FILES+= d_escmap.in +-${PACKAGE}FILES+= d_f_file_empty.in +-${PACKAGE}FILES+= d_file_exp.in +-${PACKAGE}FILES+= d_file_exp.out +-${PACKAGE}FILES+= d_ignore_case.out +-${PACKAGE}FILES+= d_input +-${PACKAGE}FILES+= d_invert.in +-${PACKAGE}FILES+= d_invert.out +-${PACKAGE}FILES+= d_oflag_zerolen_a.in +-${PACKAGE}FILES+= d_oflag_zerolen_a.out +-${PACKAGE}FILES+= d_oflag_zerolen_b.in +-${PACKAGE}FILES+= d_oflag_zerolen_b.out +-${PACKAGE}FILES+= d_oflag_zerolen_c.in +-${PACKAGE}FILES+= d_oflag_zerolen_c.out +-${PACKAGE}FILES+= d_oflag_zerolen_d.in +-${PACKAGE}FILES+= d_oflag_zerolen_e.in +-${PACKAGE}FILES+= d_oflag_zerolen_e.out +-${PACKAGE}FILES+= d_recurse.out +-${PACKAGE}FILES+= d_recurse_symlink.err +-${PACKAGE}FILES+= d_recurse_symlink.out +-${PACKAGE}FILES+= d_whole_line.out +-${PACKAGE}FILES+= d_word_regexps.out +-${PACKAGE}FILES+= d_zgrep.out +- +-.include <netbsd-tests.test.mk> + + .include <bsd.test.mk>
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?202011201416.0AKEGSNI095391>