build: update gnulib submodule to latest
[coreutils.git] / cfg.mk
blob6db0daf71128de5b4f7c62ee01910c5ba49901ed
1 # Customize maint.mk -*- makefile -*-
2 # Copyright (C) 2003-2023 Free Software Foundation, Inc.
4 # This program is free software: you can redistribute it and/or modify
5 # it under the terms of the GNU General Public License as published by
6 # the Free Software Foundation, either version 3 of the License, or
7 # (at your option) any later version.
9 # This program is distributed in the hope that it will be useful,
10 # but WITHOUT ANY WARRANTY; without even the implied warranty of
11 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
12 # GNU General Public License for more details.
14 # You should have received a copy of the GNU General Public License
15 # along with this program. If not, see <https://www.gnu.org/licenses/>.
17 # Used in maint.mk's web-manual rule
18 manual_title = Core GNU utilities
20 # Use the direct link. This is guaranteed to work immediately, while
21 # it can take a while for the faster mirror links to become usable.
22 url_dir_list = https://ftp.gnu.org/gnu/$(PACKAGE)
24 # Exclude bundled external projects from syntax checks
25 VC_LIST_ALWAYS_EXCLUDE_REGEX = src/blake2/.*$$
27 # Tests not to run as part of "make distcheck".
28 local-checks-to-skip = \
29 sc_proper_name_utf8_requires_ICONV \
30 sc_indent
32 # Tools used to bootstrap this package, used for "announcement".
33 bootstrap-tools = autoconf,automake,gnulib,bison
35 # Now that we have better tests, make this the default.
36 export VERBOSE = yes
38 # Comparing tarball sizes compressed using different xz presets, we see that
39 # an -8e-compressed tarball is only 9KiB larger than the -9e-compressed one.
40 # Using -8e is preferred, since that lets the decompression process use half
41 # the memory (32MiB rather than 64MiB).
42 # $ for i in {7,8,9}{e,}; do \
43 # (n=$(xz -$i < coreutils-8.15*.tar|wc -c);echo $n $i) & done |sort -nr
44 # 5129388 7
45 # 5036524 7e
46 # 5017476 8
47 # 5010604 9
48 # 4923016 8e
49 # 4914152 9e
50 export XZ_OPT = -8e
52 old_NEWS_hash = ac6cb1c35890b53e56acbfbb078fcd53
54 # Add an exemption for sc_makefile_at_at_check.
55 _makefile_at_at_check_exceptions = \
56 ' && !/MAKEINFO/ && !/^cu_install_prog/ && !/dynamic-dep/'
58 # Our help-version script is in a slightly different location.
59 _hv_file ?= $(srcdir)/tests/misc/help-version
61 # Ensure that the list of O_ symbols used to compute O_FULLBLOCK is complete.
62 dd = $(srcdir)/src/dd.c
63 sc_dd_O_FLAGS:
64 @rm -f $@.1 $@.2
65 @{ echo O_FULLBLOCK; echo O_NOCACHE; \
66 perl -nle '/^ +\| (O_\w*)$$/ and print $$1' $(dd); } | sort > $@.1
67 @{ echo O_NOFOLLOW; perl -nle '/{"[a-z]+",\s*(O_\w+)},/ and print $$1' \
68 $(dd); } | sort > $@.2
69 @diff -u $@.1 $@.2; diff=$$?; \
70 rm -f $@.1 $@.2; \
71 test "$$diff" = 0 \
72 || { echo '$(ME): $(dd) has inconsistent O_ flag lists'>&2; \
73 exit 1; }
75 # Ensure that dd's definition of LONGEST_SYMBOL stays in sync
76 # with the strings from the two affected variables.
77 dd_c = $(srcdir)/src/dd.c
78 sc_dd_max_sym_length:
79 ifneq ($(wildcard $(dd_c)),)
80 @len=$$( (sed -n '/conversions\[] =$$/,/^};/p' $(dd_c);\
81 sed -n '/flags\[] =$$/,/^};/p' $(dd_c) ) \
82 |sed -n '/"/s/^[^"]*"\([^"]*\)".*/\1/p'| wc -L);\
83 max=$$(sed -n '/^#define LONGEST_SYMBOL /s///p' $(dd_c) \
84 |tr -d '"' | wc -L); \
85 if test "$$len" = "$$max"; then :; else \
86 echo 'dd.c: LONGEST_SYMBOL is not longest' 1>&2; \
87 exit 1; \
89 endif
91 # Many m4 macros names once began with 'jm_'.
92 # On 2004-04-13, they were all changed to start with gl_ instead.
93 # Make sure that none are inadvertently reintroduced.
94 sc_prohibit_jm_in_m4:
95 @grep -nE 'jm_[A-Z]' \
96 $$($(VC_LIST) m4 |grep '\.m4$$'; echo /dev/null) && \
97 { echo '$(ME): do not use jm_ in m4 macro names' \
98 1>&2; exit 1; } || :
100 # Ensure that each root-requiring test is run via the "check-root" rule.
101 sc_root_tests:
102 @t1=sc-root.expected; t2=sc-root.actual; \
103 grep -nl '^ *require_root_$$' `$(VC_LIST) tests` | \
104 sed 's|.*/tests/|tests/|' | sort > $$t1; \
105 for t in $(all_root_tests); do echo $$t; done | sort > $$t2; \
106 st=0; diff -u $$t1 $$t2 || st=1; \
107 rm -f $$t1 $$t2; \
108 exit $$st
110 # Ensure that all version-controlled test cases are listed in $(all_tests).
111 sc_tests_list_consistency:
112 @bs="\\"; \
113 test_extensions_rx=`echo $(TEST_EXTENSIONS) \
114 | sed -e "s/ /|/g" -e "s/$$bs./$$bs$$bs./g"`; \
116 for t in $(all_tests); do echo $$t; done; \
117 cd $(top_srcdir); \
118 $(SHELL) build-aux/vc-list-files tests \
119 | grep -Ev '^tests/(factor/(run|create-test)|init)\.sh$$' \
120 | grep -E "$$test_extensions_rx\$$"; \
121 } | sort | uniq -u | grep . && exit 1; :
123 # Ensure that all version-controlled test scripts are executable.
124 sc_tests_executable:
125 @set -o noglob 2>/dev/null || set -f; \
126 cd $(srcdir); \
127 find_ext="-name '' "`printf -- "-o -name *%s " $(TEST_EXTENSIONS)`;\
128 find $(srcdir)/tests/ \( $$find_ext \) \! -perm -u+x -print \
129 | { sed "s|^$(srcdir)/||"; git ls-files $(srcdir)/tests/; } \
130 | sort | uniq -d \
131 | sed -e "s/^/$(ME): Please make test executable: /" | grep . \
132 && exit 1; :
134 # Ensure all gnulib patches apply cleanly
135 sc_ensure_gl_diffs_apply_cleanly:
136 @find $(srcdir)/gl/ -name '*.diff' | while read p; do \
137 patch --fuzz=0 -f -s -d $(srcdir)/gnulib/ -p1 --dry-run < "$$p" >&2 \
138 || { echo "$$p" >&2; echo 'To refresh all gl patches run:' \
139 'make refresh-gnulib-patches' >&2; exit 1; } \
140 done
142 # Avoid :>file which doesn't propagate errors
143 sc_prohibit_colon_redirection:
144 @cd $(srcdir)/tests && GIT_PAGER= git grep -En ': *>.*\|\|' \
145 && { echo '$(ME): '"The leading colon in :> will hide errors" >&2; \
146 exit 1; } \
147 || :
149 # Ensure emit_mandatory_arg_note() is called if required
150 sc_ensure_emit_mandatory_arg_note:
151 @cd $(srcdir)/src && GIT_PAGER= git \
152 grep -l -- '^ *-[^-].*--.*[^[]=' *.c \
153 | xargs grep -L emit_mandatory_arg_note | grep . \
154 && { echo '$(ME): '"emit_mandatory_arg_note() missing" 1>&2; \
155 exit 1; } || :
157 # Create a list of regular expressions matching the names
158 # of files included from system.h. Exclude a couple.
159 .re-list:
160 @sed -n '/^# *include /s///p' $(srcdir)/src/system.h \
161 | grep -Ev 'sys/(param|file)\.h' \
162 | sed 's/ .*//;;s/^["<]/^# *include [<"]/;s/\.h[">]$$/\\.h[">]/' \
163 > $@-t
164 @mv $@-t $@
166 define gl_trap_
167 Exit () { set +e; (exit $$1); exit $$1; }; \
168 for sig in 1 2 3 13 15; do \
169 eval "trap 'Exit $$(expr $$sig + 128)' $$sig"; \
170 done
171 endef
173 # Files in src/ should not include directly any of
174 # the headers already included via system.h.
175 sc_system_h_headers: .re-list
176 @if test -f $(srcdir)/src/system.h; then \
177 trap 'rc=$$?; rm -f .re-list; exit $$rc' 0; \
178 $(gl_trap_); \
179 grep -nE -f .re-list \
180 $$($(VC_LIST_EXCEPT) | grep -E '^($(srcdir)/)?src/') \
181 && { echo '$(ME): the above are already included via system.h'\
182 1>&2; exit 1; } || :; \
185 # Files in src/ should not use '%s' notation in format strings,
186 # i.e., single quotes around %s (or similar) should be avoided.
187 sc_prohibit_quotes_notation:
188 @cd $(srcdir)/src && GIT_PAGER= git grep -n "\".*[\`']%s'.*\"" *.c \
189 && { echo '$(ME): '"Use quote() to avoid quoted '%s' notation" 1>&2; \
190 exit 1; } \
191 || :
193 error_fns = (error|diagnose)
195 # Files in src/ should quote all strings in error() output, so that
196 # unexpected input chars like \r etc. don't corrupt the error.
197 # In edge cases this can be avoided by putting the format string
198 # on a separate line to the arguments, or the arguments in parenthesis.
199 sc_error_quotes:
200 @cd $(srcdir)/src \
201 && GIT_PAGER= git grep -E -n '$(error_fns) *\(.*%s.*, [^(]*\);$$' \
202 *.c | grep -v ', q' \
203 && { echo '$(ME): '"Use quote() for error string arguments" 1>&2; \
204 exit 1; } \
205 || :
207 # Files in src/ should quote all file names in error() output
208 # using quotef(), to provide quoting only when necessary,
209 # but also provide better support for copy and paste when used.
210 sc_error_shell_quotes:
211 @cd $(srcdir)/src && \
212 { GIT_PAGER= git grep -E \
213 '$(error_fns) \(.*%s[:"], .*(name|file)[^"]*\);$$' *.c; \
214 GIT_PAGER= git grep -E \
215 ' quote[ _].*file' *.c; } \
216 | grep -Ev '(quotef|q[^ ]*name)' \
217 && { echo '$(ME): '"Use quotef() for colon delimited names" 1>&2; \
218 exit 1; } \
219 || :
221 # Files in src/ should quote all file names in error() output
222 # using quoteaf() when the name is separated with spaces,
223 # to distinguish the file name at issue and
224 # to provide better support for copy and paste.
225 sc_error_shell_always_quotes:
226 @cd $(srcdir)/src && GIT_PAGER= git grep -E \
227 '$(error_fns) \(.*[^:] %s[ "].*, .*(name|file)[^"]*\);$$' \
228 *.c | grep -Ev '(quoteaf|q[^ ]*name)' \
229 && { echo '$(ME): '"Use quoteaf() for space delimited names" 1>&2; \
230 exit 1; } \
231 || :
232 @cd $(srcdir)/src && GIT_PAGER= git grep -E -A1 \
233 '$(error_fns) \([^%]*[^:] %s[ "]' *.c | grep 'quotef' \
234 && { echo '$(ME): '"Use quoteaf() for space delimited names" 1>&2; \
235 exit 1; } \
236 || :
238 # Avoid unstyled quoting to internal slots and thus destined for diagnostics
239 # as that can leak unescaped control characters to the output, when using
240 # the default "literal" quoting style.
241 # Instead use quotef(), or quoteaf() or in edge cases quotearg_n_style_colon().
242 # A more general PCRE would be @prohibit='quotearg_.*(?!(style|buffer))'
243 sc_prohibit-quotearg:
244 @prohibit='quotearg(_n)?(|_colon|_char|_mem) ' \
245 in_vc_files='\.c$$' \
246 halt='Unstyled diagnostic quoting detected' \
247 $(_sc_search_regexp)
249 sc_prohibit-skip:
250 @prohibit='\|\| skip ' \
251 halt='Use skip_ not skip' \
252 $(_sc_search_regexp)
254 sc_sun_os_names:
255 @grep -nEi \
256 'solaris[^[:alnum:]]*2\.(7|8|9|[1-9][0-9])|sunos[^[:alnum:]][6-9]' \
257 $$($(VC_LIST_EXCEPT)) && \
258 { echo '$(ME): found misuse of Sun OS version numbers' 1>&2; \
259 exit 1; } || :
261 # Ensure that the list of programs and author names is accurate.
262 # We need a UTF8 locale. If a lack of locale support or a missing
263 # translation inhibits printing of UTF-8 names, just skip this test.
264 au_dotdot = authors-dotdot
265 au_actual = authors-actual
266 sc_check-AUTHORS: $(all_programs)
267 @locale=en_US.UTF-8; \
268 LC_ALL=$$locale ./src/factor --version \
269 | grep ' Torbjorn ' > /dev/null \
270 && { echo "$@: skipping this check"; exit 0; }; \
271 rm -f $(au_actual) $(au_dotdot); \
272 for i in `ls $(all_programs) \
273 | sed -e 's,^src/,,' -e 's,$(EXEEXT)$$,,' \
274 | sed /libstdbuf/d \
275 | $(ASSORT) -u`; do \
276 test "$$i" = '[' && continue; \
277 exe=$$i; \
278 if test "$$i" = install; then \
279 exe=ginstall; \
280 elif test "$$i" = test; then \
281 exe='['; \
282 fi; \
283 LC_ALL=$$locale ./src/$$exe --version \
284 | perl -0 -p -e 's/,\n/, /gm' \
285 | sed -n -e '/Written by /{ s//'"$$i"': /;' \
286 -e 's/,* and /, /; s/\.$$//; p; }'; \
287 done > $(au_actual) && \
288 sed -n '/^[^ ][^ ]*:/p' $(srcdir)/AUTHORS > $(au_dotdot) \
289 && diff $(au_actual) $(au_dotdot) \
290 && rm -f $(au_actual) $(au_dotdot)
292 # Each program with a non-ASCII author name must link with LIBICONV.
293 sc_check-I18N-AUTHORS:
294 @cd $(srcdir)/src && \
295 for i in $$(git grep -l -w proper_name_utf8 *.c|sed 's/\.c//'); do \
296 grep -E "^src_$${i}_LDADD"' .?= .*\$$\(LIBICONV\)' local.mk \
297 > /dev/null \
298 || { echo "$(ME): link rules for $$i do not include" \
299 '$$(LIBICONV)' 1>&2; exit 1; }; \
300 done
302 # Ensure the alternative __attribute (keyword) form isn't used as
303 # that form is not elided where required. Also ensure that we don't
304 # directly use attributes already defined by gnulib.
305 # TODO: move the check for _GL... attributes to gnulib.
306 sc_prohibit-gl-attributes:
307 @prohibit='__attribute |__(unused|pure|const)__' \
308 in_vc_files='\.[ch]$$' \
309 halt='Use _GL... attribute macros' \
310 $(_sc_search_regexp)
312 # Prefer the const declaration form, with const following the type
313 sc_prohibit-const-char:
314 @prohibit='const char \*' \
315 in_vc_files='\.[ch]$$' \
316 halt='Use char const *, not const char *' \
317 $(_sc_search_regexp)
319 # Look for lines longer than 80 characters, except omit:
320 # - urls
321 # - the fdl.texi file copied from gnulib,
322 # - the help2man script copied from upstream,
323 # - tests involving long checksum lines, and
324 # - the 'pr' test cases.
325 FILTER_LONG_LINES = \
326 \|^[^:]*NEWS:.*https\{,1\}://| d; \
327 \|^[^:]*doc/fdl.texi:| d; \
328 \|^[^:]*man/help2man:| d; \
329 \|^[^:]*tests/cksum/sha[0-9]*sum.*\.pl[-:]| d; \
330 \|^[^:]*tests/pr/|{ \|^[^:]*tests/pr/pr-tests:| !d; };
331 sc_long_lines:
332 @wc -L /dev/null >/dev/null 2>/dev/null \
333 || { echo "$@: skipping: wc -L not supported"; exit 0; }; \
334 sed -r 1q /dev/null 2>/dev/null \
335 || { echo "$@: skipping: sed -r not supported"; exit 0; }; \
336 files=$$($(VC_LIST_EXCEPT) | xargs wc -L | sed -rn '/ total$$/d;\
337 s/^ *(8[1-9]|9[0-9]|[0-9]{3,}) //p'); \
338 halt='line(s) with more than 80 characters; reindent'; \
339 for file in $$files; do \
340 expand $$file | grep -nE '^.{80}.' | \
341 sed -e "s|^|$$file:|" -e '$(FILTER_LONG_LINES)'; \
342 done | grep . && { msg="$$halt" $(_sc_say_and_exit) } || :
344 # Option descriptions should not start with a capital letter.
345 # One could grep source directly as follows:
346 # grep -E " {2,6}-.*[^.] [A-Z][a-z]" $$($(VC_LIST_EXCEPT) | grep '\.c$$')
347 # but that would miss descriptions not on the same line as the -option.
348 sc_option_desc_uppercase: $(ALL_MANS)
349 @grep '^\\fB\\-' -A1 man/*.1 | LC_ALL=C grep '\.1.[A-Z][a-z]' \
350 && { echo 1>&2 '$@: found initial capitals in --help'; exit 1; } || :
352 # '--' should not be treated as '–' (U+2013 EN DASH) in long option names.
353 sc_texi_long_option_escaped: doc/coreutils.info
354 @grep ' –[^ ]' '$<' \
355 && { echo 1>&2 '$@: found unquoted --long-option'; exit 1; } || :
357 # Ensure all man/*.[1x] files are present.
358 sc_man_file_correlation: check-x-vs-1 check-programs-vs-x
360 # Ensure that for each .x file in the 'man/' subdirectory, there is a
361 # corresponding .1 file in the definition of $(EXTRA_MANS).
362 # But since that expansion usually lacks programs like arch and hostname,
363 # add them here manually.
364 .PHONY: check-x-vs-1
365 check-x-vs-1:
366 @PATH=./src$(PATH_SEPARATOR)$$PATH; export PATH; \
367 t=$@-t; \
368 (cd $(srcdir)/man && ls -1 *.x) \
369 | sed 's/\.x$$//' | $(ASSORT) > $$t; \
370 (echo $(patsubst man/%,%,$(ALL_MANS)) \
371 | tr -s ' ' '\n' | sed 's/\.1$$//') \
372 | $(ASSORT) -u | diff - $$t || { rm $$t; exit 1; }; \
373 rm $$t
375 # Ensure that non-trivial .x files in the 'man/' subdirectory,
376 # i.e., files exceeding a line count of 20 or a byte count of 1000,
377 # contain a Copyright notice.
378 .PHONY: sc_man_check_x_copyright
379 sc_man_check_x_copyright:
380 @status=0; \
381 cd $(srcdir) && wc -cl man/*.x | head -n-1 \
382 | awk '$$1 >= 20 || $$2 >= 1000 {print $$3}' \
383 | xargs grep -L 'Copyright .* Free Software Foundation' \
384 | grep . \
385 && { echo 1>&2 '$@: exceeding file size/line count limit' \
386 '- please add a copyright note'; status=1; }; \
387 exit $$status
389 # Writing a portable rule to generate a manpage like '[.1' would be
390 # a nightmare, so filter that out.
391 all-progs-but-lbracket = $(filter-out [,$(patsubst src/%,%,$(all_programs)))
393 # Ensure that for each coreutils program there is a corresponding
394 # '.x' file in the 'man/' subdirectory.
395 .PHONY: check-programs-vs-x
396 check-programs-vs-x:
397 @status=0; \
398 for p in dummy $(all-progs-but-lbracket); do \
399 case $$p in *.so) continue;; esac; \
400 test $$p = dummy && continue; \
401 test $$p = ginstall && p=install || : ; \
402 test -f $(srcdir)/man/$$p.x \
403 || { echo missing $$p.x 1>&2; status=1; }; \
404 done; \
405 exit $$status
407 # Ensure we can check out on case insensitive file systems
408 sc_case_insensitive_file_names: src/uniq
409 @git -C $(srcdir) ls-files | sort -f | src/uniq -Di | grep . && \
410 { echo "$(ME): the above file(s) conflict on case insensitive" \
411 " file systems" 1>&2; exit 1; } || :
413 # Ensure that the end of each release's section is marked by two empty lines.
414 sc_NEWS_two_empty_lines:
415 @sed -n 4,/Noteworthy/p $(srcdir)/NEWS \
416 | perl -n0e '/(^|\n)\n\n\* Noteworthy/ or exit 1' \
417 || { echo '$(ME): use two empty lines to separate NEWS sections' \
418 1>&2; exit 1; } || :
420 # With split lines, don't leave an operator at end of line.
421 # Instead, put it on the following line, where it is more apparent.
422 # Don't bother checking for "*" at end of line, since it provokes
423 # far too many false positives, matching constructs like "TYPE *".
424 # Similarly, omit "=" (initializers).
425 binop_re_ ?= [-/+^!<>]|[-/+*^!<>=]=|&&?|\|\|?|<<=?|>>=?
426 sc_prohibit_operator_at_end_of_line:
427 @prohibit='. ($(binop_re_))$$' \
428 in_vc_files='\.[chly]$$' \
429 halt='found operator at end of line' \
430 $(_sc_search_regexp)
432 # Partial substitutes for GNU extensions \< and \> in regexps.
433 begword = (^|[^_[:alnum:]])
434 endword = ($$|[^_[:alnum:]])
436 # Don't use address of "stat" or "lstat" functions
437 sc_prohibit_stat_macro_address:
438 @prohibit='$(begword)l?stat '':|&l?stat$(endword)' \
439 halt='stat() and lstat() may be function-like macros' \
440 $(_sc_search_regexp)
442 # Ensure that date's --help output stays in sync with the info
443 # documentation for GNU strftime. The only exception is %N and %q,
444 # which date accepts but GNU strftime does not.
446 # "info foo" fails with error, but not "info foo >/dev/null".
447 extract_char = sed 's/^[^%][^%]*%\(.\).*/\1/'
448 sc_strftime_check:
449 @if test -f $(srcdir)/src/date.c; then \
450 if info libc date calendar format 2>/dev/null | \
451 grep "^ *['\`]%.'$$" >$@-tmp; then \
452 { echo N; echo q; $(extract_char) $@-tmp; }| sort \
453 >$@-info; \
454 grep '^ %. ' $(srcdir)/src/date.c | sort \
455 | $(extract_char) > $@-src; \
456 diff -u $@-src $@-info || exit 1; \
457 else \
458 echo '$(ME): skipping $@: libc info not installed' 1>&2; \
459 fi; \
460 rm -f $@-info $@-src $@-tmp; \
463 # Indent only with spaces.
464 sc_prohibit_tab_based_indentation:
465 @prohibit='^ * ' \
466 halt='TAB in indentation; use only spaces' \
467 $(_sc_search_regexp)
469 # Enforce lowercase 'e' in "I.e.".
470 sc_prohibit_uppercase_id_est:
471 @prohibit='I\.E\.' \
472 halt='Uppercase "Id Est" abbreviation; use "I.e.," instead' \
473 $(_sc_search_regexp)
475 # Enforce double-space before "I.e." at the beginning of a sentence.
476 sc_ensure_dblspace_after_dot_before_id_est:
477 @prohibit='\. I\.e\.' \
478 halt='Single space after dot before "i.e."; use ". i.e." instead' \
479 $(_sc_search_regexp)
481 # Enforce comma after "i.e." (at least before a blank or at EOL).
482 sc_ensure_comma_after_id_est:
483 @prohibit='[Ii]\.e\.( |$$)' \
484 halt='Missing comma after "i.e."; use "i.e.," instead' \
485 $(_sc_search_regexp)
487 # The SEE ALSO section of a man page should not be terminated with
488 # a period. Check the first line after each "SEE ALSO" line in man/*.x:
489 sc_prohibit_man_see_also_period:
490 @grep -nB1 '\.$$' $$($(VC_LIST_EXCEPT) | grep 'man/.*\.x$$') \
491 | grep -A1 -e '-\[SEE ALSO]' | grep '\.$$' && \
492 { echo '$(ME): do not end "SEE ALSO" section with a period' \
493 1>&2; exit 1; } || :
495 sc_prohibit_exit_write_error:
496 @prohibit='error.*EXIT_FAILURE.*write error' \
497 in_vc_files='\.c$$' \
498 halt='Use write_error() instead' \
499 $(_sc_search_regexp)
501 sc_prohibit_NULL:
502 @prohibit='$(begword)NULL$(endword)' \
503 in_vc_files='\.[ch]$$' \
504 halt='use nullptr instead' \
505 $(_sc_search_regexp)
507 # Don't use "indent-tabs-mode: nil" anymore. No longer needed.
508 sc_prohibit_emacs__indent_tabs_mode__setting:
509 @prohibit='^( *[*#] *)?indent-tabs-mode:' \
510 halt='use of emacs indent-tabs-mode: setting' \
511 $(_sc_search_regexp)
513 # Ensure that tests don't include a redundant fail=0.
514 sc_prohibit_fail_0:
515 @prohibit='$(begword)fail=0$(endword)' \
516 halt='fail=0 initialization' \
517 $(_sc_search_regexp)
519 # Ensure that tests don't use `cmd ... && fail=1` as that hides crashes.
520 # The "exclude" expression allows common idioms like `test ... && fail=1`
521 # and the 2>... portion allows commands that redirect stderr and so probably
522 # independently check its contents and thus detect any crash messages.
523 sc_prohibit_and_fail_1:
524 @prohibit='&& fail=1' \
525 exclude='(returns_|stat|kill|test |EGREP|grep|compare|2> *[^/])' \
526 halt='&& fail=1 detected. Please use: returns_ 1 ... || fail=1' \
527 in_vc_files='^tests/' \
528 $(_sc_search_regexp)
530 # Ensure that tests don't use `cmd ... || fail` as that's a noop.
531 sc_prohibit_or_fail:
532 @prohibit='\|\| fail$$' \
533 exclude=':#' \
534 halt='|| fail detected. Please use: || fail=1' \
535 in_vc_files='^tests/' \
536 $(_sc_search_regexp)
538 # Ensure that env vars are not passed through returns_ as
539 # that was seen to fail on FreeBSD /bin/sh at least
540 sc_prohibit_env_returns:
541 @prohibit='=[^ ]* returns_ ' \
542 exclude='_ returns_ ' \
543 halt='Passing env vars to returns_ is non portable' \
544 in_vc_files='^tests/' \
545 $(_sc_search_regexp)
547 # The mode part of a setfacl -m option argument must be three bytes long.
548 # I.e., an argument of user:bin:rw or user:bin:r will make Solaris 10's
549 # setfacl reject it with: "Unrecognized character found in mode field".
550 # Use hyphens to give it a length of 3: "...:rw-" or "...:r--".
551 sc_prohibit_short_facl_mode_spec:
552 @prohibit='$(begword)setfacl .*-m.*:.*:[rwx-]{1,2} ' \
553 halt='setfacl mode string length < 3; extend with hyphen(s)' \
554 $(_sc_search_regexp)
556 # Ensure that "stdio--.h" is used where appropriate.
557 sc_require_stdio_safer:
558 @if $(VC_LIST_EXCEPT) | grep -l '\.[ch]$$' > /dev/null; then \
559 files=$$(grep -El '$(begword)freopen ?\(' $$($(VC_LIST_EXCEPT)\
560 | grep '\.[ch]$$')); \
561 test -n "$$files" && grep -LE 'include "stdio--.h"' $$files \
562 | grep . && \
563 { echo '$(ME): the above files should use "stdio--.h"' \
564 1>&2; exit 1; } || :; \
565 else :; \
568 # Ensure that "stdlib--.h" is used where appropriate.
569 sc_require_stdlib_safer:
570 @if $(VC_LIST_EXCEPT) | grep -l '\.[ch]$$' > /dev/null; then \
571 files=$$(grep -El '$(begword)mkstemp ?\(' $$($(VC_LIST_EXCEPT)\
572 | grep '\.[ch]$$')); \
573 test -n "$$files" && grep -LE 'include "stdlib--.h"' $$files \
574 | grep . && \
575 { echo '$(ME): the above files should use "stdlib--.h"' \
576 1>&2; exit 1; } || :; \
577 else :; \
580 sc_prohibit_perl_hash_quotes:
581 @prohibit="\{'[A-Z_]+' *[=}]" \
582 halt="in Perl code, write \$$hash{KEY}, not \$$hash{'K''EY'}" \
583 $(_sc_search_regexp)
585 # Prefer xnanosleep over other less-precise sleep methods
586 sc_prohibit_sleep:
587 @prohibit='$(begword)(nano|u)?sleep \(' \
588 halt='prefer xnanosleep over other sleep interfaces' \
589 $(_sc_search_regexp)
591 # Use print_ver_ (from init.cfg), not open-coded $VERBOSE check.
592 sc_prohibit_verbose_version:
593 @prohibit='test "\$$VERBOSE" = yes && .* --version' \
594 halt='use the print_ver_ function instead...' \
595 $(_sc_search_regexp)
597 # Enforce print_ver_ tracking of dependencies
598 # Each coreutils specific program a test requires
599 # should be tagged by calling through env(1).
600 sc_env_test_dependencies:
601 @cd $(top_srcdir) && GIT_PAGER= git grep -E \
602 "env ($$(build-aux/gen-lists-of-programs.sh --list-progs | \
603 grep -vF '[' |paste -d'|' -s))" tests | \
604 sed "s/\([^:]\):.*env \([^)' ]*\).*/\1 \2/" | uniq | \
605 while read test prog; do \
606 printf '%s' $$test | grep '\.pl$$' >/dev/null && continue; \
607 grep "print_ver_.* $$prog" $$test >/dev/null \
608 || echo $$test should call: print_ver_ $$prog; \
609 done | grep . && exit 1 || :
611 # Use framework_failure_, not the old name without the trailing underscore.
612 sc_prohibit_framework_failure:
613 @prohibit='$(begword)framework_''failure$(endword)' \
614 halt='use framework_failure_ instead' \
615 $(_sc_search_regexp)
617 # Prohibit the use of `...` in tests/. Use $(...) instead.
618 sc_prohibit_test_backticks:
619 @prohibit='`' in_vc_files='^tests/' \
620 halt='use $$(...), not `...` in tests/' \
621 $(_sc_search_regexp)
623 # Ensure that compare is used to check empty files
624 # so that the unexpected contents are displayed
625 sc_prohibit_test_empty:
626 @prohibit='test -s.*&&' in_vc_files='^tests/' \
627 halt='use `compare /dev/null ...`, not `test -s ...` in tests/' \
628 $(_sc_search_regexp)
630 # Programs like sort, ls, expr use PROG_FAILURE in place of EXIT_FAILURE.
631 # Others, use the EXIT_CANCELED, EXIT_ENOENT, etc. macros defined in system.h.
632 # In those programs, ensure that EXIT_FAILURE is not used by mistake.
633 sc_some_programs_must_avoid_exit_failure:
634 @cd $(srcdir) \
635 && grep -nw EXIT_FAILURE \
636 $$(git grep -El '[^T]_FAILURE|EXIT_CANCELED' src/) \
637 | grep -v '^src/system\.h:' \
638 | grep -vE '= EXIT_FAILURE|return .* \?' | grep . \
639 && { echo '$(ME): do not use EXIT_FAILURE in the above' \
640 1>&2; exit 1; } || :
642 # Ensure that tests call the get_min_ulimit_v_ function if using ulimit -v
643 sc_prohibit_test_ulimit_without_require_:
644 @(git -C $(srcdir) grep -l get_min_ulimit_v_ tests; \
645 git -C $(srcdir) grep -l 'ulimit -v' tests) \
646 | sort | uniq -u | grep . && { echo "$(ME): the above test(s)"\
647 " should match get_min_ulimit_v_ with ulimit -v" 1>&2; exit 1; } || :
649 # Ensure that tests call the cleanup_ function if using background processes
650 sc_prohibit_test_background_without_cleanup_:
651 @(git -C $(srcdir) grep -El '( &$$|&[^&]*=\$$!)' tests; \
652 git -C $(srcdir) grep -l 'cleanup_()' tests | sed p) \
653 | sort | uniq -u | grep . && { echo "$(ME): the above test(s)"\
654 " should use cleanup_ for background processes" 1>&2; exit 1; } || :
656 # Ensure that tests call the print_ver_ function for programs which are
657 # actually used in that test.
658 sc_prohibit_test_calls_print_ver_with_irrelevant_argument:
659 @git -C $(srcdir) grep -w print_ver_ tests \
660 | sed 's#:print_ver_##' \
661 | { fail=0; \
662 while read file name; do \
663 for i in $$name; do \
664 case "$$i" in install) i=ginstall;; esac; \
665 grep -w "$$i" $$file|grep -vw print_ver_|grep . >/dev/null \
666 || { fail=1; \
667 echo "*** Test: $$file, offending: $$i." 1>&2; };\
668 done; \
669 done; \
670 test $$fail = 0 || exit 1; \
671 } || { echo "$(ME): the above test(s) call print_ver_ for" \
672 "program(s) they don't use" 1>&2; exit 1; }
674 # Exempt the contents of any usage function from the following.
675 _continued_string_col_1 = \
676 s/^usage .*?\n}//ms;/\\\n\w/ and print ("$$ARGV\n"),$$e=1;END{$$e||=0;exit $$e}
677 # Ding any source file that has a continued string with an alphabetic in the
678 # first column of the following line. We prohibit them because they usually
679 # trigger false positives in tools that try to map an arbitrary line number
680 # to the enclosing function name. Of course, very many strings do precisely
681 # this, *when they are part of the usage function*. That is why we exempt
682 # the contents of any function named "usage".
683 sc_prohibit_continued_string_alpha_in_column_1:
684 @perl -0777 -ne '$(_continued_string_col_1)' \
685 $$($(VC_LIST_EXCEPT) | grep '\.[ch]$$') \
686 || { echo '$(ME): continued string with word in first column' \
687 1>&2; exit 1; } || :
688 # Use this to list offending lines:
689 # git ls-files |grep '\.[ch]$' | xargs \
690 # perl -n -0777 -e 's/^usage.*?\n}//ms;/\\\n\w/ and print "$ARGV\n"' \
691 # | xargs grep -A1 '\\$'|grep '\.[ch][:-][_a-zA-Z]'
694 ###########################################################
695 _p0 = \([^"'/]\|"\([^\"]\|[\].\)*"\|'\([^\']\|[\].\)*'
696 _pre = $(_p0)\|[/][^"'/*]\|[/]"\([^\"]\|[\].\)*"\|[/]'\([^\']\|[\].\)*'\)*
697 _pre_anchored = ^\($(_pre)\)
698 _comment_and_close = [^*]\|[*][^/*]\)*[*][*]*/
699 # help font-lock mode: '
701 # A sed expression that removes ANSI C and ISO C99 comments.
702 # Derived from the one in GNU gettext's 'moopp' preprocessor.
703 _sed_remove_comments = \
704 /[/][/*]/{ \
705 ta; \
706 :a; \
707 s,$(_pre_anchored)//.*,\1,; \
708 te; \
709 s,$(_pre_anchored)/[*]\($(_comment_and_close),\1 ,; \
710 ta; \
711 /^$(_pre)[/][*]/{ \
712 s,$(_pre_anchored)/[*].*,\1 ,; \
713 tu; \
714 :u; \
715 n; \
716 s,^\($(_comment_and_close),,; \
717 tv; \
718 s,^.*$$,,; \
719 bu; \
720 :v; \
721 }; \
722 :e; \
724 # Quote all single quotes.
725 _sed_rm_comments_q = $(subst ','\'',$(_sed_remove_comments))
726 # help font-lock mode: '
728 _space_before_paren_exempt =? \\n\\$$
729 _space_before_paren_exempt = \
730 (^ *\#|\\n\\$$|%s\(to %s|(date|group|character)\(s\))
731 # Ensure that there is a space before each open parenthesis in C code.
732 sc_space_before_open_paren:
733 @if $(VC_LIST_EXCEPT) | grep -l '\.[ch]$$' > /dev/null; then \
734 fail=0; \
735 for c in $$($(VC_LIST_EXCEPT) | grep '\.[ch]$$'); do \
736 sed '$(_sed_rm_comments_q)' $$c 2>/dev/null \
737 | grep -i '[[:alnum:]](' \
738 | grep -vE '$(_space_before_paren_exempt)' \
739 | grep . && { fail=1; echo "*** $$c"; }; \
740 done; \
741 test $$fail = 1 && \
742 { echo '$(ME): the above files lack a space-before-open-paren' \
743 1>&2; exit 1; } || :; \
744 else :; \
747 # Similar to the gnulib maint.mk rule for sc_prohibit_strcmp
748 # Use STREQ_LEN or STRPREFIX rather than comparing strncmp == 0, or != 0.
749 sc_prohibit_strncmp:
750 @prohibit='^[^#].*str''ncmp *\(' \
751 halt='use STREQ_LEN or STRPREFIX instead of str''ncmp' \
752 $(_sc_search_regexp)
754 # Enforce recommended preprocessor indentation style.
755 sc_preprocessor_indentation:
756 @if cppi --version >/dev/null 2>&1; then \
757 $(VC_LIST_EXCEPT) | grep '\.[ch]$$' | xargs cppi -a -c \
758 || { echo '$(ME): incorrect preprocessor indentation' 1>&2; \
759 exit 1; }; \
760 else \
761 echo '$(ME): skipping test $@: cppi not installed' 1>&2; \
764 # THANKS.in is a list of name/email pairs for people who are mentioned in
765 # commit logs (and generated ChangeLog), but who are not also listed as an
766 # author of a commit. Name/email pairs of commit authors are automatically
767 # extracted from the repository. As a very minor factorization, when
768 # someone who was initially listed only in THANKS.in later authors a commit,
769 # this rule detects that their pair may now be removed from THANKS.in.
770 sc_THANKS_in_duplicates:
771 @{ git -C $(srcdir) log --pretty=format:%aN | sort -u; \
772 cut -b-36 $(srcdir)/THANKS.in \
773 | sed '/^$$/,/^$$/!d;/^$$/d;s/ *$$//'; } \
774 | sort | uniq -d | grep . \
775 && { echo '$(ME): remove the above names from THANKS.in' \
776 1>&2; exit 1; } || :
778 # Ensure the contributor list stays sorted. However, if the system's
779 # en_US.UTF-8 locale data is erroneous, give a diagnostic and skip
780 # this test. This affects OS X, up to at least 10.11.6.
781 # Use our sort as other implementations may result in a different order.
782 sc_THANKS_in_sorted:
783 @printf 'a\n.b\n'|LC_ALL=en_US.UTF-8 src/sort -c 2> /dev/null \
784 && { \
785 sed '/^$$/,/^$$/!d;/^$$/d' $(srcdir)/THANKS.in > $@.1 && \
786 LC_ALL=en_US.UTF-8 src/sort -f -k1,1 $@.1 > $@.2 && \
787 diff -u $@.1 $@.2; diff=$$?; \
788 rm -f $@.1 $@.2; \
789 test "$$diff" = 0 \
790 || { echo '$(ME): THANKS.in is unsorted' 1>&2; exit 1; }; \
792 || { echo '$(ME): this system has erroneous locale data;' \
793 'skipping $@' 1>&2; }
795 # Look for developer diagnostics that are marked for translation.
796 # This won't find any for which devmsg's format string is on a separate line.
797 sc_marked_devdiagnostics:
798 @prohibit='$(begword)devmsg *\(.*_\(' \
799 halt='found marked developer diagnostic(s)' \
800 $(_sc_search_regexp)
802 # Ensure we keep hex constants as 4 or 8 bytes for consistency
803 # and so that make src/fs-magic-compare works consistently
804 sc_fs-magic-compare:
805 @sed -n 's|.*/\* \(0x[0-9A-Fa-f]\{1,\}\) .*\*/|\1|p' \
806 $(srcdir)/src/stat.c | grep -Ev '^0x([0-9A-F]{4}){1,2}$$' \
807 && { echo '$(ME): Constants in src/stat.c should be 4 or 8' \
808 'upper-case chars' 1>&2; exit 1; } || :
810 # Ensure gnulib generated files are ignored
811 # TODO: Perhaps augment gnulib-tool to do this in lib/.gitignore?
812 sc_gitignore_missing:
813 @{ sed -n '/^\/lib\/.*\.h$$/{p;p}' $(srcdir)/.gitignore; \
814 find lib -name '*.in*' ! -name '*~' ! -name 'sys_*' | \
815 sed 's|^|/|; s|_\(.*in\.h\)|/\1|; s/\.in//'; } | \
816 sort | uniq -u | grep . && { echo '$(ME): Add above' \
817 'entries to .gitignore' >&2; exit 1; } || :
819 # Flag redundant entries in .gitignore
820 # Disabled for now as too aggressive flagging
821 # entries like /lib/arg-nonnull.h
822 #sc_gitignore_redundant:
823 # @{ grep ^/lib $(srcdir)/.gitignore; \
824 # sed 's|^|/lib|' $(srcdir)/lib/.gitignore; } | \
825 # sort | uniq -d | grep . && { echo '$(ME): Remove above' \
826 # 'entries from .gitignore' >&2; exit 1; } || :
828 sc_prohibit-form-feed:
829 @prohibit=$$'\f' \
830 in_vc_files='\.[chly]$$' \
831 halt='Form Feed (^L) detected' \
832 $(_sc_search_regexp)
834 # Override the default Cc: used in generating an announcement.
835 announcement_Cc_ = $(translation_project_), \
836 coreutils@gnu.org, coreutils-announce@gnu.org
838 -include $(srcdir)/dist-check.mk
840 update-copyright-env = \
841 UPDATE_COPYRIGHT_FORCE=1 \
842 UPDATE_COPYRIGHT_USE_INTERVALS=2 \
843 UPDATE_COPYRIGHT_MAX_LINE_LENGTH=79
845 # List syntax-check exemptions.
846 exclude_file_name_regexp--sc_space_tab = \
847 ^(tests/pr/|tests/misc/nl\.sh$$|gl/.*\.diff$$|man/help2man$$)
848 exclude_file_name_regexp--sc_bindtextdomain = \
849 ^(gl/.*|lib/euidaccess-stat|src/make-prime-list|src/cksum)\.c$$
850 exclude_file_name_regexp--sc_trailing_blank = \
851 ^(tests/pr/|gl/.*\.diff$$|man/help2man)
852 exclude_file_name_regexp--sc_system_h_headers = \
853 ^src/((system|copy|chown-core|find-mount-point)\.h|make-prime-list\.c)$$
855 _src = (false|lbracket|ls-(dir|ls|vdir)|tac-pipe|uname-(arch|uname))
856 _gl_src = (xdecto.max|cl-strtold)
857 exclude_file_name_regexp--sc_require_config_h_first = \
858 (^lib/buffer-lcm\.c|gl/lib/$(_gl_src)\.c|src/$(_src)\.c)$$
859 exclude_file_name_regexp--sc_require_config_h = \
860 $(exclude_file_name_regexp--sc_require_config_h_first)
862 exclude_file_name_regexp--sc_po_check = ^(gl/|man/help2man)
863 exclude_file_name_regexp--sc_prohibit_always-defined_macros = \
864 ^src/(seq|remove)\.c$$
865 exclude_file_name_regexp--sc_prohibit_empty_lines_at_EOF = ^tests/pr/
866 exclude_file_name_regexp--sc_program_name = \
867 ^(gl/.*|lib/euidaccess-stat|src/make-prime-list)\.c$$
868 exclude_file_name_regexp--sc_file_system = \
869 NEWS|^(init\.cfg|src/df\.c|tests/df/df-P\.sh|tests/df/df-output\.sh)$$
870 exclude_file_name_regexp--sc_prohibit_always_true_header_tests = \
871 ^m4/stat-prog\.m4$$
872 exclude_file_name_regexp--sc_prohibit_fail_0 = \
873 (^.*/git-hooks/commit-msg|^tests/init\.sh|Makefile\.am|\.mk|.*\.texi)$$
874 exclude_file_name_regexp--sc_prohibit_test_minus_ao = doc/.*\.texi$$
875 exclude_file_name_regexp--sc_prohibit_atoi_atof = ^lib/euidaccess-stat\.c$$
877 # longlong.h is maintained elsewhere.
878 _ll = ^src/longlong\.h$$
879 exclude_file_name_regexp--sc_useless_cpp_parens = $(_ll)
880 exclude_file_name_regexp--sc_space_before_open_paren = $(_ll)
882 tbi_1 = ^tests/pr/|(\.mk|^man/help2man)$$
883 tbi_2 = ^scripts/git-hooks/(pre-commit|pre-applypatch|applypatch-msg)$$
884 tbi_3 = (GNU)?[Mm]akefile(\.am)?$$|$(_ll)
885 exclude_file_name_regexp--sc_prohibit_tab_based_indentation = \
886 $(tbi_1)|$(tbi_2)|$(tbi_3)
888 exclude_file_name_regexp--sc_preprocessor_indentation = \
889 ^(gl/lib/rand-isaac\.[ch]|gl/tests/test-rand-isaac\.c)$$|$(_ll)
890 exclude_file_name_regexp--sc_prohibit_stat_st_blocks = \
891 ^(src/system\.h|tests/du/2g\.sh)$$
893 exclude_file_name_regexp--sc_prohibit_continued_string_alpha_in_column_1 = \
894 ^src/(system\.h|od\.c|printf\.c|getlimits\.c)$$
896 _cksum = ^tests/cksum/cksum-base64\.pl$$
897 exclude_file_name_regexp--sc_prohibit_test_backticks = \
898 ^tests/(local\.mk|(init|misc/stdbuf|factor/create-test)\.sh)$$|$(_cksum)
900 # Exempt test.c, since it's nominally shared, and relatively static.
901 exclude_file_name_regexp--sc_prohibit_operator_at_end_of_line = \
902 ^src/(ptx|test|head)\.c$$
904 exclude_file_name_regexp--sc_error_message_uppercase = ^src/factor\.c$$
905 exclude_file_name_regexp--sc_prohibit_atoi_atof = ^src/make-prime-list\.c$$
907 # Exception here as we don't want __attribute elided on non GCC
908 exclude_file_name_regexp--sc_prohibit-gl-attributes = ^src/libstdbuf\.c$$
910 exclude_file_name_regexp--sc_prohibit_uppercase_id_est = \.diff$$
911 exclude_file_name_regexp--sc_ensure_dblspace_after_dot_before_id_est = \.diff$$
912 exclude_file_name_regexp--sc_ensure_comma_after_id_est = \.diff|$(_ll)$$
913 exclude_file_name_regexp--sc_long_lines = \.diff$$|$(_ll)|$(_cksum)
915 # `grep . -q` is not exactly equivalent to `grep . >/dev/null`
916 # and this difference is significant in the NEWS description
917 exclude_file_name_regexp--sc_unportable_grep_q = NEWS
919 # Augment AM_CFLAGS to include our per-directory options:
920 AM_CFLAGS += $($(@D)_CFLAGS)
922 src_CFLAGS = $(WARN_CFLAGS)
923 lib_CFLAGS = $(GNULIB_WARN_CFLAGS)
924 gnulib-tests_CFLAGS = $(GNULIB_TEST_WARN_CFLAGS)
926 # Configuration to make the tight-scope syntax-check rule work with
927 # non-recursive make.
928 # Note _gl_TS_headers use _single line_ extern function declarations,
929 # while *_SOURCES use the _two line_ form.
930 export _gl_TS_headers = $(noinst_HEADERS)
931 # Add exceptions for --enable-single-binary renamed functions.
932 _gl_TS_unmarked_extern_functions = main usage
933 _gl_TS_unmarked_extern_functions += single_binary_main_.* _usage_.*
934 # Headers to search for single line extern _data_ declarations.
935 _gl_TS_other_headers = $(srcdir)/src/*.h src/*.h
936 # Tell the tight_scope rule about an exceptional "extern" variable.
937 # Normally, the rule would detect its declaration, but that uses a
938 # different name, __clz_tab.
939 _gl_TS_unmarked_extern_vars = factor_clz_tab
940 # Other tight_scope settings
941 _gl_TS_dir = .
942 _gl_TS_obj_files = src/*.$(OBJEXT)