Author: obrien
Date: Thu Jul  5 18:23:36 2012
New Revision: 238143
URL: http://svn.freebsd.org/changeset/base/238143

Log:
  Remove the "funny targets" make check.  We no longer need embedded :: targets
  to build FreeBSD (they are used in Perl man pages).  We never needed embedded
  "!" in targets that I can find.
  
  We got this from OpenBSD and I cannot find any other make that supports
  such things -- contrary to their commit message claim: "This behaviour
  is also consistent with other versions of make.".

Added:
  head/tools/regression/usr.bin/make/syntax/funny-targets/
  head/tools/regression/usr.bin/make/syntax/funny-targets/Makefile   (contents, 
props changed)
  head/tools/regression/usr.bin/make/syntax/funny-targets/expected.status.1   
(contents, props changed)
  head/tools/regression/usr.bin/make/syntax/funny-targets/expected.status.2   
(contents, props changed)
  head/tools/regression/usr.bin/make/syntax/funny-targets/expected.stderr.1   
(contents, props changed)
  head/tools/regression/usr.bin/make/syntax/funny-targets/expected.stderr.2   
(contents, props changed)
  head/tools/regression/usr.bin/make/syntax/funny-targets/expected.stdout.1   
(contents, props changed)
  head/tools/regression/usr.bin/make/syntax/funny-targets/expected.stdout.2   
(contents, props changed)
  head/tools/regression/usr.bin/make/syntax/funny-targets/test.t   (contents, 
props changed)
Modified:
  head/tools/build/make_check/Makefile

Modified: head/tools/build/make_check/Makefile
==============================================================================
--- head/tools/build/make_check/Makefile        Thu Jul  5 18:19:35 2012        
(r238142)
+++ head/tools/build/make_check/Makefile        Thu Jul  5 18:23:36 2012        
(r238143)
@@ -23,7 +23,7 @@ NIL=
 SMAKE= MAKEFLAGS= ${MAKE} -C ${.CURDIR}
 
 all:
-       @echo '1..17'
+       @echo '1..16'
        @${SMAKE} C_check || { cd ${.CURDIR} ; ${MAKE} failure ; }
        @echo "ok 1 - C_check # Test of -C flag existence detected no 
regression."
        @echo 1:${DATA1} 2:${DATA2} 3:${DATA3} 4:${DATA4} 5:${DATA5} | \
@@ -40,26 +40,24 @@ all:
        @echo "ok 6 - notdef # Test notdef detected no regression."
        @${SMAKE} modifiers || ${SMAKE} failure
        @echo "ok 7 - modifiers # Test modifiers detected no regression."
-       @${SMAKE} funny_targets || ${SMAKE} failure
-       @echo "ok 8 funny_targets # Test funny_targets detected no regression."
        @${SMAKE} arith_expr || ${SMAKE} failure
-       @echo "ok 9 arith_expr # Test arith_expr detected no regression."
+       @echo "ok 8 arith_expr # Test arith_expr detected no regression."
        @${SMAKE} PATH_exists || ${SMAKE} failure
-       @echo "ok 10 PATH_exists # Test PATH_exists detected no regression."
+       @echo "ok 9 PATH_exists # Test PATH_exists detected no regression."
        @${SMAKE} double_quotes || ${SMAKE} failure
-       @echo "ok 11 double_quotes # Test double_quotes detected no regression."
+       @echo "ok 10 double_quotes # Test double_quotes detected no regression."
        @! ${SMAKE} double_quotes2 >/dev/null 2>&1 && true || ${SMAKE} failure
-       @echo "ok 12 double_quotes2 # Test double_quotes2 detected no 
regression."
+       @echo "ok 11 double_quotes2 # Test double_quotes2 detected no 
regression."
        @${SMAKE} pass_cmd_vars || ${SMAKE} failure
-       @echo "ok 13 pass_cmd_vars # Test pass_cmd_vars detected no regression."
+       @echo "ok 12 pass_cmd_vars # Test pass_cmd_vars detected no regression."
        @${SMAKE} plus_flag || ${SMAKE} failure
-       @echo "ok 14 plus_flag # Test plus_flag detected no regression."
+       @echo "ok 13 plus_flag # Test plus_flag detected no regression."
        @! ${SMAKE} shell >/dev/null 2>&1 && true || ${SMAKE} failure
-       @echo "ok 15 shell # Test shell detected no regression."
+       @echo "ok 14 shell # Test shell detected no regression."
        @${SMAKE} shell_1 || ${SMAKE} failure
-       @echo "ok 16 shell_1 # Test shell_1 detected no regression."
+       @echo "ok 15 shell_1 # Test shell_1 detected no regression."
        @${SMAKE} shell_2 || ${SMAKE} failure
-       @echo "ok 17 shell_2 # Test shell_2 detected no regression."
+       @echo "ok 16 shell_2 # Test shell_2 detected no regression."
 
 .if make(C_check)
 C_check:
@@ -107,12 +105,6 @@ modifiers:
        fi
 .endif
 
-.if make(funny_targets)
-funny_targets: colons::target exclamation!target
-colons::target:
-exclamation!target: 
-.endif
-
 .if make(arith_expr)
 arith_expr:
 # See if arithmetic expression parsing is broken.

Added: head/tools/regression/usr.bin/make/syntax/funny-targets/Makefile
==============================================================================
--- /dev/null   00:00:00 1970   (empty, because file is newly added)
+++ head/tools/regression/usr.bin/make/syntax/funny-targets/Makefile    Thu Jul 
 5 18:23:36 2012        (r238143)
@@ -0,0 +1,14 @@
+# $FreeBSD$
+#
+# PR bin/6612 / GRN r102178
+#
+
+test1: colons::colons::target
+
+colons::colons::target:
+       @echo '${.TARGET}'
+
+test2: exclamation!target
+
+exclamation!target:
+       @echo '${.TARGET}

Added: head/tools/regression/usr.bin/make/syntax/funny-targets/expected.status.1
==============================================================================
--- /dev/null   00:00:00 1970   (empty, because file is newly added)
+++ head/tools/regression/usr.bin/make/syntax/funny-targets/expected.status.1   
Thu Jul  5 18:23:36 2012        (r238143)
@@ -0,0 +1 @@
+0

Added: head/tools/regression/usr.bin/make/syntax/funny-targets/expected.status.2
==============================================================================
--- /dev/null   00:00:00 1970   (empty, because file is newly added)
+++ head/tools/regression/usr.bin/make/syntax/funny-targets/expected.status.2   
Thu Jul  5 18:23:36 2012        (r238143)
@@ -0,0 +1 @@
+0

Added: head/tools/regression/usr.bin/make/syntax/funny-targets/expected.stderr.1
==============================================================================

Added: head/tools/regression/usr.bin/make/syntax/funny-targets/expected.stderr.2
==============================================================================

Added: head/tools/regression/usr.bin/make/syntax/funny-targets/expected.stdout.1
==============================================================================
--- /dev/null   00:00:00 1970   (empty, because file is newly added)
+++ head/tools/regression/usr.bin/make/syntax/funny-targets/expected.stdout.1   
Thu Jul  5 18:23:36 2012        (r238143)
@@ -0,0 +1 @@
+colons::colons::target

Added: head/tools/regression/usr.bin/make/syntax/funny-targets/expected.stdout.2
==============================================================================
--- /dev/null   00:00:00 1970   (empty, because file is newly added)
+++ head/tools/regression/usr.bin/make/syntax/funny-targets/expected.stdout.2   
Thu Jul  5 18:23:36 2012        (r238143)
@@ -0,0 +1 @@
+exclamation!target

Added: head/tools/regression/usr.bin/make/syntax/funny-targets/test.t
==============================================================================
--- /dev/null   00:00:00 1970   (empty, because file is newly added)
+++ head/tools/regression/usr.bin/make/syntax/funny-targets/test.t      Thu Jul 
 5 18:23:36 2012        (r238143)
@@ -0,0 +1,14 @@
+#!/bin/sh
+
+# $FreeBSD$
+
+cd `dirname $0`
+. ../../common.sh
+
+# Description
+DESC='Target names with "funny" embeded characters.'
+
+# Run
+TEST_N=2
+
+eval_cmd $*
_______________________________________________
[email protected] mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "[email protected]"

Reply via email to