Source-Changes-HG archive
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]
[src/trunk]: src/usr.bin/make make: remove period from end of error messages ...
details: https://anonhg.NetBSD.org/src/rev/bf3296d3607b
branches: trunk
changeset: 1027178:bf3296d3607b
user: rillig <rillig%NetBSD.org@localhost>
date: Thu Dec 09 20:13:09 2021 +0000
description:
make: remove period from end of error messages and warnings
The majority of the existing error messages and warnings does not
include a period at the end. Follow this style consistently.
diffstat:
usr.bin/make/cond.c | 10 +++++-----
usr.bin/make/parse.c | 6 +++---
usr.bin/make/suff.c | 6 +++---
usr.bin/make/unit-tests/directive-else.exp | 6 +++---
usr.bin/make/unit-tests/directive-endif.exp | 8 ++++----
usr.bin/make/unit-tests/directive-for-null.exp | 2 +-
usr.bin/make/unit-tests/lint.exp | 2 +-
usr.bin/make/unit-tests/var-eval-short.exp | 2 +-
usr.bin/make/unit-tests/varmod-loop-delete.exp | 2 +-
usr.bin/make/unit-tests/varmod-loop-varname.exp | 8 ++++----
usr.bin/make/var.c | 8 ++++----
11 files changed, 30 insertions(+), 30 deletions(-)
diffs (216 lines):
diff -r 8c57efe1a301 -r bf3296d3607b usr.bin/make/cond.c
--- a/usr.bin/make/cond.c Thu Dec 09 14:33:19 2021 +0000
+++ b/usr.bin/make/cond.c Thu Dec 09 20:13:09 2021 +0000
@@ -1,4 +1,4 @@
-/* $NetBSD: cond.c,v 1.281 2021/12/05 10:11:31 rillig Exp $ */
+/* $NetBSD: cond.c,v 1.282 2021/12/09 20:13:09 rillig Exp $ */
/*
* Copyright (c) 1988, 1989, 1990 The Regents of the University of California.
@@ -95,7 +95,7 @@
#include "dir.h"
/* "@(#)cond.c 8.2 (Berkeley) 1/2/94" */
-MAKE_RCSID("$NetBSD: cond.c,v 1.281 2021/12/05 10:11:31 rillig Exp $");
+MAKE_RCSID("$NetBSD: cond.c,v 1.282 2021/12/09 20:13:09 rillig Exp $");
/*
* The parsing of conditional expressions is based on this grammar:
@@ -1211,7 +1211,7 @@
if (IsEndif(p)) { /* It is an '.endif'. */
if (p[5] != '\0') {
Parse_Error(PARSE_FATAL,
- "The .endif directive does not take arguments.");
+ "The .endif directive does not take arguments");
}
if (cond_depth == cond_min_depth) {
@@ -1242,8 +1242,8 @@
if (p[2] != '\0')
Parse_Error(PARSE_FATAL,
- "The .else directive "
- "does not take arguments.");
+ "The .else directive "
+ "does not take arguments");
if (cond_depth == cond_min_depth) {
Parse_Error(PARSE_FATAL, "if-less else");
diff -r 8c57efe1a301 -r bf3296d3607b usr.bin/make/parse.c
--- a/usr.bin/make/parse.c Thu Dec 09 14:33:19 2021 +0000
+++ b/usr.bin/make/parse.c Thu Dec 09 20:13:09 2021 +0000
@@ -1,4 +1,4 @@
-/* $NetBSD: parse.c,v 1.571 2021/12/07 23:56:06 rillig Exp $ */
+/* $NetBSD: parse.c,v 1.572 2021/12/09 20:13:09 rillig Exp $ */
/*
* Copyright (c) 1988, 1989, 1990, 1993
@@ -109,7 +109,7 @@
#include "pathnames.h"
/* "@(#)parse.c 8.3 (Berkeley) 3/19/94" */
-MAKE_RCSID("$NetBSD: parse.c,v 1.571 2021/12/07 23:56:06 rillig Exp $");
+MAKE_RCSID("$NetBSD: parse.c,v 1.572 2021/12/09 20:13:09 rillig Exp $");
/* types and constants */
@@ -2895,7 +2895,7 @@
line = ParseGetLine(GLM_FOR_BODY);
if (line == NULL) {
Parse_Error(PARSE_FATAL,
- "Unexpected end of file in for loop.");
+ "Unexpected end of file in .for loop");
break;
}
} while (For_Accum(line));
diff -r 8c57efe1a301 -r bf3296d3607b usr.bin/make/suff.c
--- a/usr.bin/make/suff.c Thu Dec 09 14:33:19 2021 +0000
+++ b/usr.bin/make/suff.c Thu Dec 09 20:13:09 2021 +0000
@@ -1,4 +1,4 @@
-/* $NetBSD: suff.c,v 1.355 2021/11/28 22:48:06 rillig Exp $ */
+/* $NetBSD: suff.c,v 1.356 2021/12/09 20:13:09 rillig Exp $ */
/*
* Copyright (c) 1988, 1989, 1990, 1993
@@ -115,7 +115,7 @@
#include "dir.h"
/* "@(#)suff.c 8.4 (Berkeley) 3/21/94" */
-MAKE_RCSID("$NetBSD: suff.c,v 1.355 2021/11/28 22:48:06 rillig Exp $");
+MAKE_RCSID("$NetBSD: suff.c,v 1.356 2021/12/09 20:13:09 rillig Exp $");
typedef List SuffixList;
typedef ListNode SuffixListNode;
@@ -2072,7 +2072,7 @@
Suffix *suff = FindSuffixByName(name);
if (suff == NULL) {
Parse_Error(PARSE_WARNING,
- "Desired null suffix %s not defined.",
+ "Desired null suffix %s not defined",
name);
return;
}
diff -r 8c57efe1a301 -r bf3296d3607b usr.bin/make/unit-tests/directive-else.exp
--- a/usr.bin/make/unit-tests/directive-else.exp Thu Dec 09 14:33:19 2021 +0000
+++ b/usr.bin/make/unit-tests/directive-else.exp Thu Dec 09 20:13:09 2021 +0000
@@ -1,11 +1,11 @@
-make: "directive-else.mk" line 14: The .else directive does not take arguments.
+make: "directive-else.mk" line 14: The .else directive does not take arguments
make: "directive-else.mk" line 15: ok
make: "directive-else.mk" line 19: ok
-make: "directive-else.mk" line 21: The .else directive does not take arguments.
+make: "directive-else.mk" line 21: The .else directive does not take arguments
make: "directive-else.mk" line 26: if-less else
make: "directive-else.mk" line 32: ok
make: "directive-else.mk" line 33: warning: extra else
-make: "directive-else.mk" line 45: The .else directive does not take arguments.
+make: "directive-else.mk" line 45: The .else directive does not take arguments
make: Fatal errors encountered -- cannot continue
make: stopped in unit-tests
exit status 1
diff -r 8c57efe1a301 -r bf3296d3607b usr.bin/make/unit-tests/directive-endif.exp
--- a/usr.bin/make/unit-tests/directive-endif.exp Thu Dec 09 14:33:19 2021 +0000
+++ b/usr.bin/make/unit-tests/directive-endif.exp Thu Dec 09 20:13:09 2021 +0000
@@ -1,7 +1,7 @@
-make: "directive-endif.mk" line 18: The .endif directive does not take arguments.
-make: "directive-endif.mk" line 23: The .endif directive does not take arguments.
-make: "directive-endif.mk" line 33: The .endif directive does not take arguments.
-make: "directive-endif.mk" line 39: The .endif directive does not take arguments.
+make: "directive-endif.mk" line 18: The .endif directive does not take arguments
+make: "directive-endif.mk" line 23: The .endif directive does not take arguments
+make: "directive-endif.mk" line 33: The .endif directive does not take arguments
+make: "directive-endif.mk" line 39: The .endif directive does not take arguments
make: "directive-endif.mk" line 45: Unknown directive "endifx"
make: Fatal errors encountered -- cannot continue
make: stopped in unit-tests
diff -r 8c57efe1a301 -r bf3296d3607b usr.bin/make/unit-tests/directive-for-null.exp
--- a/usr.bin/make/unit-tests/directive-for-null.exp Thu Dec 09 14:33:19 2021 +0000
+++ b/usr.bin/make/unit-tests/directive-for-null.exp Thu Dec 09 20:13:09 2021 +0000
@@ -1,5 +1,5 @@
make: "(stdin)" line 2: Zero byte read from file
-make: "(stdin)" line 2: Unexpected end of file in for loop.
+make: "(stdin)" line 2: Unexpected end of file in .for loop
make: "(stdin)" line 3: Zero byte read from file
make: Fatal errors encountered -- cannot continue
make: stopped in unit-tests
diff -r 8c57efe1a301 -r bf3296d3607b usr.bin/make/unit-tests/lint.exp
--- a/usr.bin/make/unit-tests/lint.exp Thu Dec 09 14:33:19 2021 +0000
+++ b/usr.bin/make/unit-tests/lint.exp Thu Dec 09 20:13:09 2021 +0000
@@ -1,4 +1,4 @@
-make: In the :@ modifier of "VAR", the variable name "${:Ubar:S,b,v,}" must not contain a dollar.
+make: In the :@ modifier of "VAR", the variable name "${:Ubar:S,b,v,}" must not contain a dollar
y@:Q}
xvaluey
exit status 2
diff -r 8c57efe1a301 -r bf3296d3607b usr.bin/make/unit-tests/var-eval-short.exp
--- a/usr.bin/make/unit-tests/var-eval-short.exp Thu Dec 09 14:33:19 2021 +0000
+++ b/usr.bin/make/unit-tests/var-eval-short.exp Thu Dec 09 20:13:09 2021 +0000
@@ -1,4 +1,4 @@
-make: "var-eval-short.mk" line 41: In the :@ modifier of "", the variable name "${FAIL}" must not contain a dollar.
+make: "var-eval-short.mk" line 41: In the :@ modifier of "", the variable name "${FAIL}" must not contain a dollar
make: "var-eval-short.mk" line 41: Malformed conditional (0 && ${:Uword:@${FAIL}@expr@})
make: "var-eval-short.mk" line 81: Invalid time value: ${FAIL}}
make: "var-eval-short.mk" line 81: Malformed conditional (0 && ${:Uword:gmtime=${FAIL}})
diff -r 8c57efe1a301 -r bf3296d3607b usr.bin/make/unit-tests/varmod-loop-delete.exp
--- a/usr.bin/make/unit-tests/varmod-loop-delete.exp Thu Dec 09 14:33:19 2021 +0000
+++ b/usr.bin/make/unit-tests/varmod-loop-delete.exp Thu Dec 09 20:13:09 2021 +0000
@@ -1,4 +1,4 @@
-make: "varmod-loop-delete.mk" line 19: Cannot delete variable "VAR" while it is used.
+make: "varmod-loop-delete.mk" line 19: Cannot delete variable "VAR" while it is used
make: Fatal errors encountered -- cannot continue
make: stopped in unit-tests
exit status 1
diff -r 8c57efe1a301 -r bf3296d3607b usr.bin/make/unit-tests/varmod-loop-varname.exp
--- a/usr.bin/make/unit-tests/varmod-loop-varname.exp Thu Dec 09 14:33:19 2021 +0000
+++ b/usr.bin/make/unit-tests/varmod-loop-varname.exp Thu Dec 09 20:13:09 2021 +0000
@@ -1,10 +1,10 @@
-make: "varmod-loop-varname.mk" line 16: In the :@ modifier of "", the variable name "${:Ubar:S,b,v,}" must not contain a dollar.
+make: "varmod-loop-varname.mk" line 16: In the :@ modifier of "", the variable name "${:Ubar:S,b,v,}" must not contain a dollar
make: "varmod-loop-varname.mk" line 16: Malformed conditional (${:Uone two three:@${:Ubar:S,b,v,}@+${var}+@} != "+one+ +two+ +three+")
-make: "varmod-loop-varname.mk" line 85: In the :@ modifier of "1 2 3", the variable name "v$" must not contain a dollar.
+make: "varmod-loop-varname.mk" line 85: In the :@ modifier of "1 2 3", the variable name "v$" must not contain a dollar
make: "varmod-loop-varname.mk" line 85: Malformed conditional (${1 2 3:L:@v$@($v)@} != "(1) (2) (3)")
-make: "varmod-loop-varname.mk" line 90: In the :@ modifier of "1 2 3", the variable name "v$$" must not contain a dollar.
+make: "varmod-loop-varname.mk" line 90: In the :@ modifier of "1 2 3", the variable name "v$$" must not contain a dollar
make: "varmod-loop-varname.mk" line 90: Malformed conditional (${1 2 3:L:@v$$@($v)@} != "() () ()")
-make: "varmod-loop-varname.mk" line 95: In the :@ modifier of "1 2 3", the variable name "v$$$" must not contain a dollar.
+make: "varmod-loop-varname.mk" line 95: In the :@ modifier of "1 2 3", the variable name "v$$$" must not contain a dollar
make: "varmod-loop-varname.mk" line 95: Malformed conditional (${1 2 3:L:@v$$$@($v)@} != "() () ()")
make: Fatal errors encountered -- cannot continue
make: stopped in unit-tests
diff -r 8c57efe1a301 -r bf3296d3607b usr.bin/make/var.c
--- a/usr.bin/make/var.c Thu Dec 09 14:33:19 2021 +0000
+++ b/usr.bin/make/var.c Thu Dec 09 20:13:09 2021 +0000
@@ -1,4 +1,4 @@
-/* $NetBSD: var.c,v 1.968 2021/12/07 21:47:21 rillig Exp $ */
+/* $NetBSD: var.c,v 1.969 2021/12/09 20:13:10 rillig Exp $ */
/*
* Copyright (c) 1988, 1989, 1990, 1993
@@ -140,7 +140,7 @@
#include "metachar.h"
/* "@(#)var.c 8.3 (Berkeley) 3/19/94" */
-MAKE_RCSID("$NetBSD: var.c,v 1.968 2021/12/07 21:47:21 rillig Exp $");
+MAKE_RCSID("$NetBSD: var.c,v 1.969 2021/12/09 20:13:10 rillig Exp $");
/*
* Variables are defined using one of the VAR=value assignments. Their
@@ -498,7 +498,7 @@
v = he->value;
if (v->inUse) {
Parse_Error(PARSE_FATAL,
- "Cannot delete variable \"%s\" while it is used.",
+ "Cannot delete variable \"%s\" while it is used",
v->name.str);
return;
}
@@ -2449,7 +2449,7 @@
if (strchr(args.var, '$') != NULL) {
Parse_Error(PARSE_FATAL,
"In the :@ modifier of \"%s\", the variable name \"%s\" "
- "must not contain a dollar.",
+ "must not contain a dollar",
expr->name, args.var);
return AMR_CLEANUP;
}
Home |
Main Index |
Thread Index |
Old Index