Mercurial > gnulib
annotate top/maint.mk @ 14394:74d1558ae43f
maint.mk: detect missing-NL-at-EOF, too
* top/maint.mk (sc_prohibit_empty_lines_at_EOF): Adjust so that
it also detects when a file lacks a newline at EOF.
(require_exactly_one_NL_at_EOF_): Renamed from
detect_empty_lines_at_EOF_. I opted not to rename the rule,
since people may well have .x-sc_... file names tied to the
existing name. Suggested by Eric Blake.
author | Jim Meyering <meyering@redhat.com> |
---|---|
date | Fri, 25 Feb 2011 20:18:02 +0100 |
parents | e52d28567d4e |
children | 29f8fb6ed4b4 |
rev | line source |
---|---|
9811
5f79eb63c21e
Alter GNUmakefile to install into top directory.
Eric Blake <ebb9@byu.net>
parents:
diff
changeset
|
1 # -*-Makefile-*- |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
2 # This Makefile fragment tries to be general-purpose enough to be |
11501
5223ecd9ec17
maint.mk: import improvements from m4
Eric Blake <ebb9@byu.net>
parents:
11500
diff
changeset
|
3 # used by many projects via the gnulib maintainer-makefile module. |
9811
5f79eb63c21e
Alter GNUmakefile to install into top directory.
Eric Blake <ebb9@byu.net>
parents:
diff
changeset
|
4 |
14079
97fc9a21a8fb
maint: update almost all copyright ranges to include 2011
Jim Meyering <meyering@redhat.com>
parents:
13880
diff
changeset
|
5 ## Copyright (C) 2001-2011 Free Software Foundation, Inc. |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
6 ## |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
7 ## This program is free software: you can redistribute it and/or modify |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
8 ## it under the terms of the GNU General Public License as published by |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
9 ## the Free Software Foundation, either version 3 of the License, or |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
10 ## (at your option) any later version. |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
11 ## |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
12 ## This program is distributed in the hope that it will be useful, |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
13 ## but WITHOUT ANY WARRANTY; without even the implied warranty of |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
14 ## MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
15 ## GNU General Public License for more details. |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
16 ## |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
17 ## You should have received a copy of the GNU General Public License |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
18 ## along with this program. If not, see <http://www.gnu.org/licenses/>. |
9811
5f79eb63c21e
Alter GNUmakefile to install into top directory.
Eric Blake <ebb9@byu.net>
parents:
diff
changeset
|
19 |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
20 # This is reported not to work with make-3.79.1 |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
21 # ME := $(word $(words $(MAKEFILE_LIST)),$(MAKEFILE_LIST)) |
9811
5f79eb63c21e
Alter GNUmakefile to install into top directory.
Eric Blake <ebb9@byu.net>
parents:
diff
changeset
|
22 ME := maint.mk |
5f79eb63c21e
Alter GNUmakefile to install into top directory.
Eric Blake <ebb9@byu.net>
parents:
diff
changeset
|
23 |
11563
861b812ba15f
top/maint.mk: Make $(srcdir)/build-aux configurable.
Simon Josefsson <simon@josefsson.org>
parents:
11547
diff
changeset
|
24 # Override this in cfg.mk if you use a non-standard build-aux directory. |
861b812ba15f
top/maint.mk: Make $(srcdir)/build-aux configurable.
Simon Josefsson <simon@josefsson.org>
parents:
11547
diff
changeset
|
25 build_aux ?= $(srcdir)/build-aux |
861b812ba15f
top/maint.mk: Make $(srcdir)/build-aux configurable.
Simon Josefsson <simon@josefsson.org>
parents:
11547
diff
changeset
|
26 |
9811
5f79eb63c21e
Alter GNUmakefile to install into top directory.
Eric Blake <ebb9@byu.net>
parents:
diff
changeset
|
27 # Do not save the original name or timestamp in the .tar.gz file. |
5f79eb63c21e
Alter GNUmakefile to install into top directory.
Eric Blake <ebb9@byu.net>
parents:
diff
changeset
|
28 # Use --rsyncable if available. |
5f79eb63c21e
Alter GNUmakefile to install into top directory.
Eric Blake <ebb9@byu.net>
parents:
diff
changeset
|
29 gzip_rsyncable := \ |
13376 | 30 $(shell gzip --help 2>/dev/null|grep rsyncable >/dev/null \ |
31 && printf %s --rsyncable) | |
9811
5f79eb63c21e
Alter GNUmakefile to install into top directory.
Eric Blake <ebb9@byu.net>
parents:
diff
changeset
|
32 GZIP_ENV = '--no-name --best $(gzip_rsyncable)' |
5f79eb63c21e
Alter GNUmakefile to install into top directory.
Eric Blake <ebb9@byu.net>
parents:
diff
changeset
|
33 |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
34 GIT = git |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
35 VC = $(GIT) |
11501
5223ecd9ec17
maint.mk: import improvements from m4
Eric Blake <ebb9@byu.net>
parents:
11500
diff
changeset
|
36 VC-tag = git tag -s -m '$(VERSION)' -u '$(gpg_key_ID)' |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
37 |
11563
861b812ba15f
top/maint.mk: Make $(srcdir)/build-aux configurable.
Simon Josefsson <simon@josefsson.org>
parents:
11547
diff
changeset
|
38 VC_LIST = $(build_aux)/vc-list-files -C $(srcdir) |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
39 |
12736
123f8b43a3aa
top/maint.mk (VC_LIST_EXCEPT): Filter list through VC_LIST_ALWAYS_EXCLUDE_REGEX.
Simon Josefsson <simon@josefsson.org>
parents:
12608
diff
changeset
|
40 # You can override this variable in cfg.mk to set your own regexp |
123f8b43a3aa
top/maint.mk (VC_LIST_EXCEPT): Filter list through VC_LIST_ALWAYS_EXCLUDE_REGEX.
Simon Josefsson <simon@josefsson.org>
parents:
12608
diff
changeset
|
41 # matching files to ignore. |
123f8b43a3aa
top/maint.mk (VC_LIST_EXCEPT): Filter list through VC_LIST_ALWAYS_EXCLUDE_REGEX.
Simon Josefsson <simon@josefsson.org>
parents:
12608
diff
changeset
|
42 VC_LIST_ALWAYS_EXCLUDE_REGEX ?= ^$$ |
123f8b43a3aa
top/maint.mk (VC_LIST_EXCEPT): Filter list through VC_LIST_ALWAYS_EXCLUDE_REGEX.
Simon Josefsson <simon@josefsson.org>
parents:
12608
diff
changeset
|
43 |
12768
f12919d53a60
maint.mk: make VC_LIST_EXCEPT robustly handle a srcdir containing "."
Jim Meyering <meyering@redhat.com>
parents:
12767
diff
changeset
|
44 # This is to preprocess robustly the output of $(VC_LIST), so that even |
f12919d53a60
maint.mk: make VC_LIST_EXCEPT robustly handle a srcdir containing "."
Jim Meyering <meyering@redhat.com>
parents:
12767
diff
changeset
|
45 # when $(srcdir) is a pathological name like "....", the leading sed command |
f12919d53a60
maint.mk: make VC_LIST_EXCEPT robustly handle a srcdir containing "."
Jim Meyering <meyering@redhat.com>
parents:
12767
diff
changeset
|
46 # removes only the intended prefix. |
12771
a11a67aec9bf
maint.mk: fix syntax-check in a non-srcdir build directory
Jim Meyering <meyering@redhat.com>
parents:
12768
diff
changeset
|
47 _dot_escaped_srcdir = $(subst .,\.,$(srcdir)) |
12768
f12919d53a60
maint.mk: make VC_LIST_EXCEPT robustly handle a srcdir containing "."
Jim Meyering <meyering@redhat.com>
parents:
12767
diff
changeset
|
48 |
12773
1fedbaac4fa9
maint.mk: do not prepend "./" after filtering
Jim Meyering <meyering@redhat.com>
parents:
12771
diff
changeset
|
49 # Post-process $(VC_LIST) output, prepending $(srcdir)/, but only |
1fedbaac4fa9
maint.mk: do not prepend "./" after filtering
Jim Meyering <meyering@redhat.com>
parents:
12771
diff
changeset
|
50 # when $(srcdir) is not ".". |
1fedbaac4fa9
maint.mk: do not prepend "./" after filtering
Jim Meyering <meyering@redhat.com>
parents:
12771
diff
changeset
|
51 ifeq ($(srcdir),.) |
1fedbaac4fa9
maint.mk: do not prepend "./" after filtering
Jim Meyering <meyering@redhat.com>
parents:
12771
diff
changeset
|
52 _prepend_srcdir_prefix = |
1fedbaac4fa9
maint.mk: do not prepend "./" after filtering
Jim Meyering <meyering@redhat.com>
parents:
12771
diff
changeset
|
53 else |
1fedbaac4fa9
maint.mk: do not prepend "./" after filtering
Jim Meyering <meyering@redhat.com>
parents:
12771
diff
changeset
|
54 _prepend_srcdir_prefix = | sed 's|^|$(srcdir)/|' |
1fedbaac4fa9
maint.mk: do not prepend "./" after filtering
Jim Meyering <meyering@redhat.com>
parents:
12771
diff
changeset
|
55 endif |
1fedbaac4fa9
maint.mk: do not prepend "./" after filtering
Jim Meyering <meyering@redhat.com>
parents:
12771
diff
changeset
|
56 |
1fedbaac4fa9
maint.mk: do not prepend "./" after filtering
Jim Meyering <meyering@redhat.com>
parents:
12771
diff
changeset
|
57 # In order to be able to consistently filter "."-relative names, |
1fedbaac4fa9
maint.mk: do not prepend "./" after filtering
Jim Meyering <meyering@redhat.com>
parents:
12771
diff
changeset
|
58 # (i.e., with no $(srcdir) prefix), this definition is careful to |
1fedbaac4fa9
maint.mk: do not prepend "./" after filtering
Jim Meyering <meyering@redhat.com>
parents:
12771
diff
changeset
|
59 # remove any $(srcdir) prefix, and to restore what it removes. |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
60 VC_LIST_EXCEPT = \ |
12768
f12919d53a60
maint.mk: make VC_LIST_EXCEPT robustly handle a srcdir containing "."
Jim Meyering <meyering@redhat.com>
parents:
12767
diff
changeset
|
61 $(VC_LIST) | sed 's|^$(_dot_escaped_srcdir)/||' \ |
12767
941bc7981053
maint.mk: fix VC_LIST_EXCEPT for srcdir != builddir
Jiri Denemark <jdenemar@redhat.com>
parents:
12736
diff
changeset
|
62 | if test -f $(srcdir)/.x-$@; then grep -vEf $(srcdir)/.x-$@; \ |
941bc7981053
maint.mk: fix VC_LIST_EXCEPT for srcdir != builddir
Jiri Denemark <jdenemar@redhat.com>
parents:
12736
diff
changeset
|
63 else grep -Ev -e "$${VC_LIST_EXCEPT_DEFAULT-ChangeLog}"; fi \ |
941bc7981053
maint.mk: fix VC_LIST_EXCEPT for srcdir != builddir
Jiri Denemark <jdenemar@redhat.com>
parents:
12736
diff
changeset
|
64 | grep -Ev -e '$(VC_LIST_ALWAYS_EXCLUDE_REGEX)' \ |
12773
1fedbaac4fa9
maint.mk: do not prepend "./" after filtering
Jim Meyering <meyering@redhat.com>
parents:
12771
diff
changeset
|
65 $(_prepend_srcdir_prefix) |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
66 |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
67 ifeq ($(origin prev_version_file), undefined) |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
68 prev_version_file = $(srcdir)/.prev-version |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
69 endif |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
70 |
11736
9f4d092a26a4
maint.mk: avoid warnings about missing files
Jim Meyering <meyering@redhat.com>
parents:
11733
diff
changeset
|
71 PREV_VERSION := $(shell cat $(prev_version_file) 2>/dev/null) |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
72 VERSION_REGEXP = $(subst .,\.,$(VERSION)) |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
73 PREV_VERSION_REGEXP = $(subst .,\.,$(PREV_VERSION)) |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
74 |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
75 ifeq ($(VC),$(GIT)) |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
76 this-vc-tag = v$(VERSION) |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
77 this-vc-tag-regexp = v$(VERSION_REGEXP) |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
78 else |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
79 tag-package = $(shell echo "$(PACKAGE)" | tr '[:lower:]' '[:upper:]') |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
80 tag-this-version = $(subst .,_,$(VERSION)) |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
81 this-vc-tag = $(tag-package)-$(tag-this-version) |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
82 this-vc-tag-regexp = $(this-vc-tag) |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
83 endif |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
84 my_distdir = $(PACKAGE)-$(VERSION) |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
85 |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
86 # Old releases are stored here. |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
87 release_archive_dir ?= ../release |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
88 |
12329
1d8d8b72ddd8
build: mention ftp redirector in release announcements
Eric Blake <ebb9@byu.net>
parents:
12325
diff
changeset
|
89 # Override gnu_rel_host and url_dir_list in cfg.mk if these are not right. |
1d8d8b72ddd8
build: mention ftp redirector in release announcements
Eric Blake <ebb9@byu.net>
parents:
12325
diff
changeset
|
90 # Use alpha.gnu.org for alpha and beta releases. |
1d8d8b72ddd8
build: mention ftp redirector in release announcements
Eric Blake <ebb9@byu.net>
parents:
12325
diff
changeset
|
91 # Use ftp.gnu.org for stable releases. |
1d8d8b72ddd8
build: mention ftp redirector in release announcements
Eric Blake <ebb9@byu.net>
parents:
12325
diff
changeset
|
92 gnu_ftp_host-alpha = alpha.gnu.org |
1d8d8b72ddd8
build: mention ftp redirector in release announcements
Eric Blake <ebb9@byu.net>
parents:
12325
diff
changeset
|
93 gnu_ftp_host-beta = alpha.gnu.org |
1d8d8b72ddd8
build: mention ftp redirector in release announcements
Eric Blake <ebb9@byu.net>
parents:
12325
diff
changeset
|
94 gnu_ftp_host-stable = ftp.gnu.org |
1d8d8b72ddd8
build: mention ftp redirector in release announcements
Eric Blake <ebb9@byu.net>
parents:
12325
diff
changeset
|
95 gnu_rel_host ?= $(gnu_ftp_host-$(RELEASE_TYPE)) |
1d8d8b72ddd8
build: mention ftp redirector in release announcements
Eric Blake <ebb9@byu.net>
parents:
12325
diff
changeset
|
96 |
1d8d8b72ddd8
build: mention ftp redirector in release announcements
Eric Blake <ebb9@byu.net>
parents:
12325
diff
changeset
|
97 ifeq ($(gnu_rel_host),ftp.gnu.org) |
1d8d8b72ddd8
build: mention ftp redirector in release announcements
Eric Blake <ebb9@byu.net>
parents:
12325
diff
changeset
|
98 url_dir_list ?= http://ftpmirror.gnu.org/$(PACKAGE) |
1d8d8b72ddd8
build: mention ftp redirector in release announcements
Eric Blake <ebb9@byu.net>
parents:
12325
diff
changeset
|
99 else |
1d8d8b72ddd8
build: mention ftp redirector in release announcements
Eric Blake <ebb9@byu.net>
parents:
12325
diff
changeset
|
100 url_dir_list ?= ftp://$(gnu_rel_host)/gnu/$(PACKAGE) |
1d8d8b72ddd8
build: mention ftp redirector in release announcements
Eric Blake <ebb9@byu.net>
parents:
12325
diff
changeset
|
101 endif |
1d8d8b72ddd8
build: mention ftp redirector in release announcements
Eric Blake <ebb9@byu.net>
parents:
12325
diff
changeset
|
102 |
12383
a22f69c2029c
maintainer-makefile: allow customization of NEWS entry format
Alfred M. Szmidt <ams@gnu.org>
parents:
12329
diff
changeset
|
103 # Override this in cfg.mk if you are using a different format in your |
a22f69c2029c
maintainer-makefile: allow customization of NEWS entry format
Alfred M. Szmidt <ams@gnu.org>
parents:
12329
diff
changeset
|
104 # NEWS file. |
12384
8620ff59758a
maint.mk: news-check: use grep -E
Jim Meyering <meyering@redhat.com>
parents:
12383
diff
changeset
|
105 today = $(shell date +%Y-%m-%d) |
13393
50e9a2259a7d
maint.mk: make the news-check rule more configurable
Peter Simons <simons@cryp.to>
parents:
13378
diff
changeset
|
106 |
50e9a2259a7d
maint.mk: make the news-check rule more configurable
Peter Simons <simons@cryp.to>
parents:
13378
diff
changeset
|
107 # Select which lines of NEWS are searched for $(news-check-regexp). |
50e9a2259a7d
maint.mk: make the news-check rule more configurable
Peter Simons <simons@cryp.to>
parents:
13378
diff
changeset
|
108 # This is a sed line number spec. The default says that we search |
50e9a2259a7d
maint.mk: make the news-check rule more configurable
Peter Simons <simons@cryp.to>
parents:
13378
diff
changeset
|
109 # lines 1..10 of NEWS for $(news-check-regexp). |
50e9a2259a7d
maint.mk: make the news-check rule more configurable
Peter Simons <simons@cryp.to>
parents:
13378
diff
changeset
|
110 # If you want to search only line 3 or only lines 20-22, use "3" or "20,22". |
50e9a2259a7d
maint.mk: make the news-check rule more configurable
Peter Simons <simons@cryp.to>
parents:
13378
diff
changeset
|
111 news-check-lines-spec ?= 1,10 |
12385
ad3ca6ce84b0
maint.mk: backslash-escape parens in default regexp
Jim Meyering <meyering@redhat.com>
parents:
12384
diff
changeset
|
112 news-check-regexp ?= '^\*.* $(VERSION_REGEXP) \($(today)\)' |
12383
a22f69c2029c
maintainer-makefile: allow customization of NEWS entry format
Alfred M. Szmidt <ams@gnu.org>
parents:
12329
diff
changeset
|
113 |
9811
5f79eb63c21e
Alter GNUmakefile to install into top directory.
Eric Blake <ebb9@byu.net>
parents:
diff
changeset
|
114 # Prevent programs like 'sort' from considering distinct strings to be equal. |
5f79eb63c21e
Alter GNUmakefile to install into top directory.
Eric Blake <ebb9@byu.net>
parents:
diff
changeset
|
115 # Doing it here saves us from having to set LC_ALL elsewhere in this file. |
5f79eb63c21e
Alter GNUmakefile to install into top directory.
Eric Blake <ebb9@byu.net>
parents:
diff
changeset
|
116 export LC_ALL = C |
5f79eb63c21e
Alter GNUmakefile to install into top directory.
Eric Blake <ebb9@byu.net>
parents:
diff
changeset
|
117 |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
118 ## --------------- ## |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
119 ## Sanity checks. ## |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
120 ## --------------- ## |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
121 |
11736
9f4d092a26a4
maint.mk: avoid warnings about missing files
Jim Meyering <meyering@redhat.com>
parents:
11733
diff
changeset
|
122 _cfg_mk := $(shell test -f $(srcdir)/cfg.mk && echo '$(srcdir)/cfg.mk') |
9f4d092a26a4
maint.mk: avoid warnings about missing files
Jim Meyering <meyering@redhat.com>
parents:
11733
diff
changeset
|
123 |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
124 # Collect the names of rules starting with `sc_'. |
11927
234941dec230
maint.mk: sort the list of syntax-check rules
Daniel P. Berrange <berrange@redhat.com>
parents:
11909
diff
changeset
|
125 syntax-check-rules := $(sort $(shell sed -n 's/^\(sc_[a-zA-Z0-9_-]*\):.*/\1/p' \ |
234941dec230
maint.mk: sort the list of syntax-check rules
Daniel P. Berrange <berrange@redhat.com>
parents:
11909
diff
changeset
|
126 $(srcdir)/$(ME) $(_cfg_mk))) |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
127 .PHONY: $(syntax-check-rules) |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
128 |
14357
e52d28567d4e
maintainer-makefile: make syntax-check a no-op from tarballs
Eric Blake <eblake@redhat.com>
parents:
14216
diff
changeset
|
129 ifeq ($(shell $(VC_LIST) >/dev/null 2>&1; echo $$?),0) |
e52d28567d4e
maintainer-makefile: make syntax-check a no-op from tarballs
Eric Blake <eblake@redhat.com>
parents:
14216
diff
changeset
|
130 local-checks-available += $(syntax-check-rules) |
e52d28567d4e
maintainer-makefile: make syntax-check a no-op from tarballs
Eric Blake <eblake@redhat.com>
parents:
14216
diff
changeset
|
131 else |
e52d28567d4e
maintainer-makefile: make syntax-check a no-op from tarballs
Eric Blake <eblake@redhat.com>
parents:
14216
diff
changeset
|
132 local-checks-available += no-vc-detected |
e52d28567d4e
maintainer-makefile: make syntax-check a no-op from tarballs
Eric Blake <eblake@redhat.com>
parents:
14216
diff
changeset
|
133 no-vc-detected: |
e52d28567d4e
maintainer-makefile: make syntax-check a no-op from tarballs
Eric Blake <eblake@redhat.com>
parents:
14216
diff
changeset
|
134 @echo "No version control files detected; skipping syntax check" |
e52d28567d4e
maintainer-makefile: make syntax-check a no-op from tarballs
Eric Blake <eblake@redhat.com>
parents:
14216
diff
changeset
|
135 endif |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
136 .PHONY: $(local-checks-available) |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
137 |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
138 # Arrange to print the name of each syntax-checking rule just before running it. |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
139 $(syntax-check-rules): %: %.m |
12866
bedb32c7fa1b
maint.mk: mark syntax-check sc_*.m rules as .PHONY
Jim Meyering <meyering@redhat.com>
parents:
12865
diff
changeset
|
140 sc_m_rules_ = $(patsubst %, %.m, $(syntax-check-rules)) |
bedb32c7fa1b
maint.mk: mark syntax-check sc_*.m rules as .PHONY
Jim Meyering <meyering@redhat.com>
parents:
12865
diff
changeset
|
141 .PHONY: $(sc_m_rules_) |
bedb32c7fa1b
maint.mk: mark syntax-check sc_*.m rules as .PHONY
Jim Meyering <meyering@redhat.com>
parents:
12865
diff
changeset
|
142 $(sc_m_rules_): |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
143 @echo $(patsubst sc_%.m, %, $@) |
13208
85be704c72eb
maint.mk: print the elapsed time for each syntax-check rule
Jim Meyering <meyering@redhat.com>
parents:
13207
diff
changeset
|
144 @date +%s.%N > .sc-start-$(basename $@) |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
145 |
13208
85be704c72eb
maint.mk: print the elapsed time for each syntax-check rule
Jim Meyering <meyering@redhat.com>
parents:
13207
diff
changeset
|
146 # Compute and print the elapsed time for each syntax-check rule. |
85be704c72eb
maint.mk: print the elapsed time for each syntax-check rule
Jim Meyering <meyering@redhat.com>
parents:
13207
diff
changeset
|
147 sc_z_rules_ = $(patsubst %, %.z, $(syntax-check-rules)) |
85be704c72eb
maint.mk: print the elapsed time for each syntax-check rule
Jim Meyering <meyering@redhat.com>
parents:
13207
diff
changeset
|
148 .PHONY: $(sc_z_rules_) |
85be704c72eb
maint.mk: print the elapsed time for each syntax-check rule
Jim Meyering <meyering@redhat.com>
parents:
13207
diff
changeset
|
149 $(sc_z_rules_): %.z: % |
85be704c72eb
maint.mk: print the elapsed time for each syntax-check rule
Jim Meyering <meyering@redhat.com>
parents:
13207
diff
changeset
|
150 @end=$$(date +%s.%N); \ |
85be704c72eb
maint.mk: print the elapsed time for each syntax-check rule
Jim Meyering <meyering@redhat.com>
parents:
13207
diff
changeset
|
151 start=$$(cat .sc-start-$*); \ |
85be704c72eb
maint.mk: print the elapsed time for each syntax-check rule
Jim Meyering <meyering@redhat.com>
parents:
13207
diff
changeset
|
152 rm -f .sc-start-$*; \ |
85be704c72eb
maint.mk: print the elapsed time for each syntax-check rule
Jim Meyering <meyering@redhat.com>
parents:
13207
diff
changeset
|
153 awk -v s=$$start -v e=$$end \ |
13221
6b5e6ecef4f5
maint.mk: generate more concise timing data for syntax-check rules
Jim Meyering <meyering@redhat.com>
parents:
13210
diff
changeset
|
154 'END {printf "%.2f $(patsubst sc_%,%,$*)\n", e - s}' < /dev/null |
13208
85be704c72eb
maint.mk: print the elapsed time for each syntax-check rule
Jim Meyering <meyering@redhat.com>
parents:
13207
diff
changeset
|
155 |
85be704c72eb
maint.mk: print the elapsed time for each syntax-check rule
Jim Meyering <meyering@redhat.com>
parents:
13207
diff
changeset
|
156 # The patsubst here is to replace each sc_% rule with its sc_%.z wrapper |
85be704c72eb
maint.mk: print the elapsed time for each syntax-check rule
Jim Meyering <meyering@redhat.com>
parents:
13207
diff
changeset
|
157 # that computes and prints elapsed time. |
85be704c72eb
maint.mk: print the elapsed time for each syntax-check rule
Jim Meyering <meyering@redhat.com>
parents:
13207
diff
changeset
|
158 local-check := \ |
85be704c72eb
maint.mk: print the elapsed time for each syntax-check rule
Jim Meyering <meyering@redhat.com>
parents:
13207
diff
changeset
|
159 $(patsubst sc_%, sc_%.z, \ |
85be704c72eb
maint.mk: print the elapsed time for each syntax-check rule
Jim Meyering <meyering@redhat.com>
parents:
13207
diff
changeset
|
160 $(filter-out $(local-checks-to-skip), $(local-checks-available))) |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
161 |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
162 syntax-check: $(local-check) |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
163 |
13103
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
164 # _sc_search_regexp |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
165 # |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
166 # This macro searches for a given construct in the selected files and |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
167 # then takes some action. |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
168 # |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
169 # Parameters (shell variables): |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
170 # |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
171 # prohibit | require |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
172 # |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
173 # Regular expression (ERE) denoting either a forbidden construct |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
174 # or a required construct. Those arguments are exclusive. |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
175 # |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
176 # in_vc_files | in_files |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
177 # |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
178 # grep-E-style regexp denoting the files to check. If no files |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
179 # are specified the default are all the files that are under |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
180 # version control. |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
181 # |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
182 # containing | non_containing |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
183 # |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
184 # Select the files (non) containing strings matching this regexp. |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
185 # If both arguments are specified then CONTAINING takes |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
186 # precedence. |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
187 # |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
188 # with_grep_options |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
189 # |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
190 # Extra options for grep. |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
191 # |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
192 # ignore_case |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
193 # |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
194 # Ignore case. |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
195 # |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
196 # halt |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
197 # |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
198 # Message to display before to halting execution. |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
199 |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
200 # By default, _sc_search_regexp does not ignore case. |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
201 export ignore_case = |
13376 | 202 _ignore_case = $$(test -n "$$ignore_case" && printf %s -i || :) |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
203 |
13103
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
204 define _sc_say_and_exit |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
205 dummy=; : so we do not need a semicolon before each use; \ |
13376 | 206 { printf '%s\n' "$(ME): $$msg" 1>&2; exit 1; }; |
13103
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
207 endef |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
208 |
13126
5f4b249b16d0
maint.mk: don't silently disable project-specific syntax-check rules
Jim Meyering <meyering@redhat.com>
parents:
13104
diff
changeset
|
209 # _sc_search_regexp used to be named _prohibit_regexp. However, |
5f4b249b16d0
maint.mk: don't silently disable project-specific syntax-check rules
Jim Meyering <meyering@redhat.com>
parents:
13104
diff
changeset
|
210 # upgrading to the new definition and leaving the old name undefined |
5f4b249b16d0
maint.mk: don't silently disable project-specific syntax-check rules
Jim Meyering <meyering@redhat.com>
parents:
13104
diff
changeset
|
211 # would usually convert each custom rule using $(_prohibit_regexp) |
5f4b249b16d0
maint.mk: don't silently disable project-specific syntax-check rules
Jim Meyering <meyering@redhat.com>
parents:
13104
diff
changeset
|
212 # (usually defined in cfg.mk) into a no-op. This definition ensures |
5f4b249b16d0
maint.mk: don't silently disable project-specific syntax-check rules
Jim Meyering <meyering@redhat.com>
parents:
13104
diff
changeset
|
213 # that people know right away if they're still using the old name. |
5f4b249b16d0
maint.mk: don't silently disable project-specific syntax-check rules
Jim Meyering <meyering@redhat.com>
parents:
13104
diff
changeset
|
214 # FIXME: remove in 2012. |
5f4b249b16d0
maint.mk: don't silently disable project-specific syntax-check rules
Jim Meyering <meyering@redhat.com>
parents:
13104
diff
changeset
|
215 _prohibit_regexp = \ |
5f4b249b16d0
maint.mk: don't silently disable project-specific syntax-check rules
Jim Meyering <meyering@redhat.com>
parents:
13104
diff
changeset
|
216 $(error '*** you need to s/_prohibit_regexp/_sc_search_regexp/, and adapt') |
5f4b249b16d0
maint.mk: don't silently disable project-specific syntax-check rules
Jim Meyering <meyering@redhat.com>
parents:
13104
diff
changeset
|
217 |
13103
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
218 define _sc_search_regexp |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
219 dummy=; : so we do not need a semicolon before each use; \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
220 \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
221 : Check arguments; \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
222 test -n "$$prohibit" && test -n "$$require" \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
223 && { msg='Cannot specify both prohibit and require' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
224 $(_sc_say_and_exit) } || :; \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
225 test -z "$$prohibit" && test -z "$$require" \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
226 && { msg='Should specify either prohibit or require' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
227 $(_sc_say_and_exit) } || :; \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
228 test -n "$$in_vc_files" && test -n "$$in_files" \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
229 && { msg='Cannot specify both in_vc_files and in_files' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
230 $(_sc_say_and_exit) } || :; \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
231 test "x$$halt" != x \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
232 || { msg='halt not defined' $(_sc_say_and_exit) }; \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
233 \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
234 : Filter by file name; \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
235 if test -n "$$in_files"; then \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
236 files=$$(find $(srcdir) | grep -E "$$in_files"); \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
237 else \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
238 files=$$($(VC_LIST_EXCEPT)); \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
239 if test -n "$$in_vc_files"; then \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
240 files=$$(echo "$$files" | grep -E "$$in_vc_files"); \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
241 fi; \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
242 fi; \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
243 \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
244 : Filter by content; \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
245 test -n "$$files" && test -n "$$containing" \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
246 && { files=$$(grep -l "$$containing" $$files); } || :; \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
247 test -n "$$files" && test -n "$$non_containing" \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
248 && { files=$$(grep -vl "$$non_containing" $$files); } || :; \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
249 \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
250 : Check for the construct; \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
251 if test -n "$$files"; then \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
252 if test -n "$$prohibit"; then \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
253 grep $$with_grep_options $(_ignore_case) -nE "$$prohibit" $$files \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
254 && { msg="$$halt" $(_sc_say_and_exit) } || :; \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
255 else \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
256 grep $$with_grep_options $(_ignore_case) -LE "$$require" $$files \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
257 | grep . \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
258 && { msg="$$halt" $(_sc_say_and_exit) } || :; \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
259 fi \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
260 else :; \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
261 fi || :; |
11468
65ab7e6dcafe
maint.mk: import changes to syntax-check macros from coreutils
Reuben Thomas <rrt@sc3d.org>
parents:
11406
diff
changeset
|
262 endef |
65ab7e6dcafe
maint.mk: import changes to syntax-check macros from coreutils
Reuben Thomas <rrt@sc3d.org>
parents:
11406
diff
changeset
|
263 |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
264 sc_avoid_if_before_free: |
11564
ab18b3d5e34a
* top/maint.mk: Adjust backslash alignment.
Jim Meyering <meyering@redhat.com>
parents:
11563
diff
changeset
|
265 @$(build_aux)/useless-if-before-free \ |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
266 $(useless_free_options) \ |
11518
956eb61203ab
top/maint.mk (sc_avoid_if_before_free): Except useless-if-before-free script.
Simon Josefsson <simon@josefsson.org>
parents:
11517
diff
changeset
|
267 $$($(VC_LIST_EXCEPT) | grep -v useless-if-before-free) && \ |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
268 { echo '$(ME): found useless "if" before "free" above' 1>&2; \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
269 exit 1; } || : |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
270 |
9811
5f79eb63c21e
Alter GNUmakefile to install into top directory.
Eric Blake <ebb9@byu.net>
parents:
diff
changeset
|
271 sc_cast_of_argument_to_free: |
13103
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
272 @prohibit='\<free *\( *\(' halt='don'\''t cast free argument' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
273 $(_sc_search_regexp) |
9811
5f79eb63c21e
Alter GNUmakefile to install into top directory.
Eric Blake <ebb9@byu.net>
parents:
diff
changeset
|
274 |
5f79eb63c21e
Alter GNUmakefile to install into top directory.
Eric Blake <ebb9@byu.net>
parents:
diff
changeset
|
275 sc_cast_of_x_alloc_return_value: |
13103
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
276 @prohibit='\*\) *x(m|c|re)alloc\>' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
277 halt='don'\''t cast x*alloc return value' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
278 $(_sc_search_regexp) |
9811
5f79eb63c21e
Alter GNUmakefile to install into top directory.
Eric Blake <ebb9@byu.net>
parents:
diff
changeset
|
279 |
5f79eb63c21e
Alter GNUmakefile to install into top directory.
Eric Blake <ebb9@byu.net>
parents:
diff
changeset
|
280 sc_cast_of_alloca_return_value: |
13103
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
281 @prohibit='\*\) *alloca\>' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
282 halt='don'\''t cast alloca return value' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
283 $(_sc_search_regexp) |
9811
5f79eb63c21e
Alter GNUmakefile to install into top directory.
Eric Blake <ebb9@byu.net>
parents:
diff
changeset
|
284 |
5f79eb63c21e
Alter GNUmakefile to install into top directory.
Eric Blake <ebb9@byu.net>
parents:
diff
changeset
|
285 sc_space_tab: |
13103
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
286 @prohibit='[ ] ' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
287 halt='found SPACE-TAB sequence; remove the SPACE' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
288 $(_sc_search_regexp) |
9811
5f79eb63c21e
Alter GNUmakefile to install into top directory.
Eric Blake <ebb9@byu.net>
parents:
diff
changeset
|
289 |
11468
65ab7e6dcafe
maint.mk: import changes to syntax-check macros from coreutils
Reuben Thomas <rrt@sc3d.org>
parents:
11406
diff
changeset
|
290 # Don't use *scanf or the old ato* functions in `real' code. |
9811
5f79eb63c21e
Alter GNUmakefile to install into top directory.
Eric Blake <ebb9@byu.net>
parents:
diff
changeset
|
291 # They provide no error checking mechanism. |
5f79eb63c21e
Alter GNUmakefile to install into top directory.
Eric Blake <ebb9@byu.net>
parents:
diff
changeset
|
292 # Instead, use strto* functions. |
5f79eb63c21e
Alter GNUmakefile to install into top directory.
Eric Blake <ebb9@byu.net>
parents:
diff
changeset
|
293 sc_prohibit_atoi_atof: |
13103
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
294 @prohibit='\<([fs]?scanf|ato([filq]|ll)) *\(' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
295 halt='do not use *scan''f, ato''f, ato''i, ato''l, ato''ll or ato''q' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
296 $(_sc_search_regexp) |
9811
5f79eb63c21e
Alter GNUmakefile to install into top directory.
Eric Blake <ebb9@byu.net>
parents:
diff
changeset
|
297 |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
298 # Use STREQ rather than comparing strcmp == 0, or != 0. |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
299 sc_prohibit_strcmp: |
14140
1ca84334d3eb
maint.mk: improve sc_prohibit_strcmp regex
Eric Blake <eblake@redhat.com>
parents:
14137
diff
changeset
|
300 @grep -nE '! *str''cmp *\(|\<str''cmp *\(.+\) *[!=]=' \ |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
301 $$($(VC_LIST_EXCEPT)) \ |
14140
1ca84334d3eb
maint.mk: improve sc_prohibit_strcmp regex
Eric Blake <eblake@redhat.com>
parents:
14137
diff
changeset
|
302 | grep -vE ':# *define STRN?EQ\(' && \ |
1ca84334d3eb
maint.mk: improve sc_prohibit_strcmp regex
Eric Blake <eblake@redhat.com>
parents:
14137
diff
changeset
|
303 { echo '$(ME): replace str''cmp calls above with STREQ/STRNEQ' \ |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
304 1>&2; exit 1; } || : |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
305 |
12208
a377ab624fcc
maint: detect usage(1) and other suspicious exits
Eric Blake <ebb9@byu.net>
parents:
12083
diff
changeset
|
306 # Pass EXIT_*, not number, to usage, exit, and error (when exiting) |
12325
444e305ec8cc
maint.mk: improve sc_prohibit_magic_number_exit
Jim Meyering <meyering@redhat.com>
parents:
12287
diff
changeset
|
307 # Convert all uses automatically, via these two commands: |
444e305ec8cc
maint.mk: improve sc_prohibit_magic_number_exit
Jim Meyering <meyering@redhat.com>
parents:
12287
diff
changeset
|
308 # git grep -l '\<exit *(1)' \ |
444e305ec8cc
maint.mk: improve sc_prohibit_magic_number_exit
Jim Meyering <meyering@redhat.com>
parents:
12287
diff
changeset
|
309 # | grep -vEf .x-sc_prohibit_magic_number_exit \ |
444e305ec8cc
maint.mk: improve sc_prohibit_magic_number_exit
Jim Meyering <meyering@redhat.com>
parents:
12287
diff
changeset
|
310 # | xargs --no-run-if-empty \ |
444e305ec8cc
maint.mk: improve sc_prohibit_magic_number_exit
Jim Meyering <meyering@redhat.com>
parents:
12287
diff
changeset
|
311 # perl -pi -e 's/(^|[^.])\b(exit ?)\(1\)/$1$2(EXIT_FAILURE)/' |
444e305ec8cc
maint.mk: improve sc_prohibit_magic_number_exit
Jim Meyering <meyering@redhat.com>
parents:
12287
diff
changeset
|
312 # git grep -l '\<exit *(0)' \ |
444e305ec8cc
maint.mk: improve sc_prohibit_magic_number_exit
Jim Meyering <meyering@redhat.com>
parents:
12287
diff
changeset
|
313 # | grep -vEf .x-sc_prohibit_magic_number_exit \ |
444e305ec8cc
maint.mk: improve sc_prohibit_magic_number_exit
Jim Meyering <meyering@redhat.com>
parents:
12287
diff
changeset
|
314 # | xargs --no-run-if-empty \ |
444e305ec8cc
maint.mk: improve sc_prohibit_magic_number_exit
Jim Meyering <meyering@redhat.com>
parents:
12287
diff
changeset
|
315 # perl -pi -e 's/(^|[^.])\b(exit ?)\(0\)/$1$2(EXIT_SUCCESS)/' |
12208
a377ab624fcc
maint: detect usage(1) and other suspicious exits
Eric Blake <ebb9@byu.net>
parents:
12083
diff
changeset
|
316 sc_prohibit_magic_number_exit: |
13103
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
317 @prohibit='(^|[^.])\<(usage|exit) ?\([0-9]|\<error ?\([1-9][0-9]*,' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
318 halt='use EXIT_* values rather than magic number' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
319 $(_sc_search_regexp) |
12208
a377ab624fcc
maint: detect usage(1) and other suspicious exits
Eric Blake <ebb9@byu.net>
parents:
12083
diff
changeset
|
320 |
9811
5f79eb63c21e
Alter GNUmakefile to install into top directory.
Eric Blake <ebb9@byu.net>
parents:
diff
changeset
|
321 # Using EXIT_SUCCESS as the first argument to error is misleading, |
5f79eb63c21e
Alter GNUmakefile to install into top directory.
Eric Blake <ebb9@byu.net>
parents:
diff
changeset
|
322 # since when that parameter is 0, error does not exit. Use `0' instead. |
5f79eb63c21e
Alter GNUmakefile to install into top directory.
Eric Blake <ebb9@byu.net>
parents:
diff
changeset
|
323 sc_error_exit_success: |
13103
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
324 @prohibit='error *\(EXIT_SUCCESS,' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
325 in_vc_files='\.[chly]$$' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
326 halt='found error (EXIT_SUCCESS' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
327 $(_sc_search_regexp) |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
328 |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
329 # `FATAL:' should be fully upper-cased in error messages |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
330 # `WARNING:' should be fully upper-cased, or fully lower-cased |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
331 sc_error_message_warn_fatal: |
13102
9c58202271c4
maint.mk: match 0 or more whitespace-before-function-call '('
Jose E. Marchesi <jemarch@gnu.org>
parents:
13101
diff
changeset
|
332 @grep -nEA2 '[^rp]error *\(' $$($(VC_LIST_EXCEPT)) \ |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
333 | grep -E '"Warning|"Fatal|"fatal' && \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
334 { echo '$(ME): use FATAL, WARNING or warning' 1>&2; \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
335 exit 1; } || : |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
336 |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
337 # Error messages should not start with a capital letter |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
338 sc_error_message_uppercase: |
13102
9c58202271c4
maint.mk: match 0 or more whitespace-before-function-call '('
Jose E. Marchesi <jemarch@gnu.org>
parents:
13101
diff
changeset
|
339 @grep -nEA2 '[^rp]error *\(' $$($(VC_LIST_EXCEPT)) \ |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
340 | grep -E '"[A-Z]' \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
341 | grep -vE '"FATAL|"WARNING|"Java|"C#|PRIuMAX' && \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
342 { echo '$(ME): found capitalized error message' 1>&2; \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
343 exit 1; } || : |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
344 |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
345 # Error messages should not end with a period |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
346 sc_error_message_period: |
13102
9c58202271c4
maint.mk: match 0 or more whitespace-before-function-call '('
Jose E. Marchesi <jemarch@gnu.org>
parents:
13101
diff
changeset
|
347 @grep -nEA2 '[^rp]error *\(' $$($(VC_LIST_EXCEPT)) \ |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
348 | grep -E '[^."]\."' && \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
349 { echo '$(ME): found error message ending in period' 1>&2; \ |
9811
5f79eb63c21e
Alter GNUmakefile to install into top directory.
Eric Blake <ebb9@byu.net>
parents:
diff
changeset
|
350 exit 1; } || : |
5f79eb63c21e
Alter GNUmakefile to install into top directory.
Eric Blake <ebb9@byu.net>
parents:
diff
changeset
|
351 |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
352 sc_file_system: |
13103
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
353 @prohibit=file''system \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
354 ignore_case=1 \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
355 halt='found use of "file''system"; spell it "file system"' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
356 $(_sc_search_regexp) |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
357 |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
358 # Don't use cpp tests of this symbol. All code assumes config.h is included. |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
359 sc_prohibit_have_config_h: |
13103
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
360 @prohibit='^# *if.*HAVE''_CONFIG_H' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
361 halt='found use of HAVE''_CONFIG_H; remove' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
362 $(_sc_search_regexp) |
9811
5f79eb63c21e
Alter GNUmakefile to install into top directory.
Eric Blake <ebb9@byu.net>
parents:
diff
changeset
|
363 |
11519
49521800ab0c
maint.mk: allow package-specific header to provide <config.h>
Eric Blake <ebb9@byu.net>
parents:
11518
diff
changeset
|
364 # Nearly all .c files must include <config.h>. However, we also permit this |
49521800ab0c
maint.mk: allow package-specific header to provide <config.h>
Eric Blake <ebb9@byu.net>
parents:
11518
diff
changeset
|
365 # via inclusion of a package-specific header, if cfg.mk specified one. |
49521800ab0c
maint.mk: allow package-specific header to provide <config.h>
Eric Blake <ebb9@byu.net>
parents:
11518
diff
changeset
|
366 # config_h_header must be suitable for grep -E. |
49521800ab0c
maint.mk: allow package-specific header to provide <config.h>
Eric Blake <ebb9@byu.net>
parents:
11518
diff
changeset
|
367 config_h_header ?= <config\.h> |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
368 sc_require_config_h: |
13103
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
369 @require='^# *include $(config_h_header)' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
370 in_vc_files='\.c$$' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
371 halt='the above files do not include <config.h>' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
372 $(_sc_search_regexp) |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
373 |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
374 # You must include <config.h> before including any other header file. |
11519
49521800ab0c
maint.mk: allow package-specific header to provide <config.h>
Eric Blake <ebb9@byu.net>
parents:
11518
diff
changeset
|
375 # This can possibly be via a package-specific header, if given by cfg.mk. |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
376 sc_require_config_h_first: |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
377 @if $(VC_LIST_EXCEPT) | grep -l '\.c$$' > /dev/null; then \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
378 fail=0; \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
379 for i in $$($(VC_LIST_EXCEPT) | grep '\.c$$'); do \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
380 grep '^# *include\>' $$i | sed 1q \ |
11519
49521800ab0c
maint.mk: allow package-specific header to provide <config.h>
Eric Blake <ebb9@byu.net>
parents:
11518
diff
changeset
|
381 | grep -E '^# *include $(config_h_header)' > /dev/null \ |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
382 || { echo $$i; fail=1; }; \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
383 done; \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
384 test $$fail = 1 && \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
385 { echo '$(ME): the above files include some other header' \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
386 'before <config.h>' 1>&2; exit 1; } || :; \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
387 else :; \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
388 fi |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
389 |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
390 sc_prohibit_HAVE_MBRTOWC: |
13103
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
391 @prohibit='\bHAVE_MBRTOWC\b' \ |
13202
69a6d1611d1b
maint.mk: correct a diagnostic
Jim Meyering <meyering@redhat.com>
parents:
13192
diff
changeset
|
392 halt="do not use $$prohibit; it is always defined" \ |
13103
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
393 $(_sc_search_regexp) |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
394 |
11468
65ab7e6dcafe
maint.mk: import changes to syntax-check macros from coreutils
Reuben Thomas <rrt@sc3d.org>
parents:
11406
diff
changeset
|
395 # To use this "command" macro, you must first define two shell variables: |
65ab7e6dcafe
maint.mk: import changes to syntax-check macros from coreutils
Reuben Thomas <rrt@sc3d.org>
parents:
11406
diff
changeset
|
396 # h: the header, enclosed in <> or "" |
65ab7e6dcafe
maint.mk: import changes to syntax-check macros from coreutils
Reuben Thomas <rrt@sc3d.org>
parents:
11406
diff
changeset
|
397 # re: a regular expression that matches IFF something provided by $h is used. |
13097
ef132cd978fc
maint.mk: s/_header_without_use/_sc_header_without_use/
Jose E. Marchesi <jemarch@gnu.org>
parents:
13085
diff
changeset
|
398 define _sc_header_without_use |
11853
97a383c7cec4
maint.mk: _header_without_use: fix a quoting bug and remove a bash'ism
Jim Meyering <meyering@redhat.com>
parents:
11847
diff
changeset
|
399 dummy=; : so we do not need a semicolon before each use; \ |
97a383c7cec4
maint.mk: _header_without_use: fix a quoting bug and remove a bash'ism
Jim Meyering <meyering@redhat.com>
parents:
11847
diff
changeset
|
400 h_esc=`echo "$$h"|sed 's/\./\\\\./g'`; \ |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
401 if $(VC_LIST_EXCEPT) | grep -l '\.c$$' > /dev/null; then \ |
11468
65ab7e6dcafe
maint.mk: import changes to syntax-check macros from coreutils
Reuben Thomas <rrt@sc3d.org>
parents:
11406
diff
changeset
|
402 files=$$(grep -l '^# *include '"$$h_esc" \ |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
403 $$($(VC_LIST_EXCEPT) | grep '\.c$$')) && \ |
11468
65ab7e6dcafe
maint.mk: import changes to syntax-check macros from coreutils
Reuben Thomas <rrt@sc3d.org>
parents:
11406
diff
changeset
|
404 grep -LE "$$re" $$files | grep . && \ |
65ab7e6dcafe
maint.mk: import changes to syntax-check macros from coreutils
Reuben Thomas <rrt@sc3d.org>
parents:
11406
diff
changeset
|
405 { echo "$(ME): the above files include $$h but don't use it" \ |
65ab7e6dcafe
maint.mk: import changes to syntax-check macros from coreutils
Reuben Thomas <rrt@sc3d.org>
parents:
11406
diff
changeset
|
406 1>&2; exit 1; } || :; \ |
65ab7e6dcafe
maint.mk: import changes to syntax-check macros from coreutils
Reuben Thomas <rrt@sc3d.org>
parents:
11406
diff
changeset
|
407 else :; \ |
65ab7e6dcafe
maint.mk: import changes to syntax-check macros from coreutils
Reuben Thomas <rrt@sc3d.org>
parents:
11406
diff
changeset
|
408 fi |
65ab7e6dcafe
maint.mk: import changes to syntax-check macros from coreutils
Reuben Thomas <rrt@sc3d.org>
parents:
11406
diff
changeset
|
409 endef |
65ab7e6dcafe
maint.mk: import changes to syntax-check macros from coreutils
Reuben Thomas <rrt@sc3d.org>
parents:
11406
diff
changeset
|
410 |
9811
5f79eb63c21e
Alter GNUmakefile to install into top directory.
Eric Blake <ebb9@byu.net>
parents:
diff
changeset
|
411 # Prohibit the inclusion of assert.h without an actual use of assert. |
5f79eb63c21e
Alter GNUmakefile to install into top directory.
Eric Blake <ebb9@byu.net>
parents:
diff
changeset
|
412 sc_prohibit_assert_without_use: |
13097
ef132cd978fc
maint.mk: s/_header_without_use/_sc_header_without_use/
Jose E. Marchesi <jemarch@gnu.org>
parents:
13085
diff
changeset
|
413 @h='<assert.h>' re='\<assert *\(' $(_sc_header_without_use) |
9811
5f79eb63c21e
Alter GNUmakefile to install into top directory.
Eric Blake <ebb9@byu.net>
parents:
diff
changeset
|
414 |
12245
86929e33bdad
maint.mk: Prohibit inclusion of "close-stream.h" without use.
Jim Meyering <meyering@redhat.com>
parents:
12208
diff
changeset
|
415 # Prohibit the inclusion of close-stream.h without an actual use. |
86929e33bdad
maint.mk: Prohibit inclusion of "close-stream.h" without use.
Jim Meyering <meyering@redhat.com>
parents:
12208
diff
changeset
|
416 sc_prohibit_close_stream_without_use: |
13097
ef132cd978fc
maint.mk: s/_header_without_use/_sc_header_without_use/
Jose E. Marchesi <jemarch@gnu.org>
parents:
13085
diff
changeset
|
417 @h='"close-stream.h"' re='\<close_stream *\(' $(_sc_header_without_use) |
12245
86929e33bdad
maint.mk: Prohibit inclusion of "close-stream.h" without use.
Jim Meyering <meyering@redhat.com>
parents:
12208
diff
changeset
|
418 |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
419 # Prohibit the inclusion of getopt.h without an actual use. |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
420 sc_prohibit_getopt_without_use: |
13097
ef132cd978fc
maint.mk: s/_header_without_use/_sc_header_without_use/
Jose E. Marchesi <jemarch@gnu.org>
parents:
13085
diff
changeset
|
421 @h='<getopt.h>' re='\<getopt(_long)? *\(' $(_sc_header_without_use) |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
422 |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
423 # Don't include quotearg.h unless you use one of its functions. |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
424 sc_prohibit_quotearg_without_use: |
13097
ef132cd978fc
maint.mk: s/_header_without_use/_sc_header_without_use/
Jose E. Marchesi <jemarch@gnu.org>
parents:
13085
diff
changeset
|
425 @h='"quotearg.h"' re='\<quotearg(_[^ ]+)? *\(' $(_sc_header_without_use) |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
426 |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
427 # Don't include quote.h unless you use one of its functions. |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
428 sc_prohibit_quote_without_use: |
13097
ef132cd978fc
maint.mk: s/_header_without_use/_sc_header_without_use/
Jose E. Marchesi <jemarch@gnu.org>
parents:
13085
diff
changeset
|
429 @h='"quote.h"' re='\<quote(_n)? *\(' $(_sc_header_without_use) |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
430 |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
431 # Don't include this header unless you use one of its functions. |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
432 sc_prohibit_long_options_without_use: |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
433 @h='"long-options.h"' re='\<parse_long_options *\(' \ |
13097
ef132cd978fc
maint.mk: s/_header_without_use/_sc_header_without_use/
Jose E. Marchesi <jemarch@gnu.org>
parents:
13085
diff
changeset
|
434 $(_sc_header_without_use) |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
435 |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
436 # Don't include this header unless you use one of its functions. |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
437 sc_prohibit_inttostr_without_use: |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
438 @h='"inttostr.h"' re='\<(off|[iu]max|uint)tostr *\(' \ |
13097
ef132cd978fc
maint.mk: s/_header_without_use/_sc_header_without_use/
Jose E. Marchesi <jemarch@gnu.org>
parents:
13085
diff
changeset
|
439 $(_sc_header_without_use) |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
440 |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
441 # Don't include this header unless you use one of its functions. |
12861
2afdfba76c99
maint.mk: prohibit inclusion of "ignore-value.h" without_use
Jim Meyering <meyering@redhat.com>
parents:
12827
diff
changeset
|
442 sc_prohibit_ignore_value_without_use: |
2afdfba76c99
maint.mk: prohibit inclusion of "ignore-value.h" without_use
Jim Meyering <meyering@redhat.com>
parents:
12827
diff
changeset
|
443 @h='"ignore-value.h"' re='\<ignore_(value|ptr) *\(' \ |
13097
ef132cd978fc
maint.mk: s/_header_without_use/_sc_header_without_use/
Jose E. Marchesi <jemarch@gnu.org>
parents:
13085
diff
changeset
|
444 $(_sc_header_without_use) |
12861
2afdfba76c99
maint.mk: prohibit inclusion of "ignore-value.h" without_use
Jim Meyering <meyering@redhat.com>
parents:
12827
diff
changeset
|
445 |
2afdfba76c99
maint.mk: prohibit inclusion of "ignore-value.h" without_use
Jim Meyering <meyering@redhat.com>
parents:
12827
diff
changeset
|
446 # Don't include this header unless you use one of its functions. |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
447 sc_prohibit_error_without_use: |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
448 @h='"error.h"' \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
449 re='\<error(_at_line|_print_progname|_one_per_line|_message_count)? *\('\ |
13097
ef132cd978fc
maint.mk: s/_header_without_use/_sc_header_without_use/
Jose E. Marchesi <jemarch@gnu.org>
parents:
13085
diff
changeset
|
450 $(_sc_header_without_use) |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
451 |
12287
0c467c7c0132
maint.mk: Prohibit inclusion of "xalloc.h" without use.
Jim Meyering <meyering@redhat.com>
parents:
12245
diff
changeset
|
452 # Don't include xalloc.h unless you use one of its functions. |
0c467c7c0132
maint.mk: Prohibit inclusion of "xalloc.h" without use.
Jim Meyering <meyering@redhat.com>
parents:
12245
diff
changeset
|
453 # Consider these symbols: |
0c467c7c0132
maint.mk: Prohibit inclusion of "xalloc.h" without use.
Jim Meyering <meyering@redhat.com>
parents:
12245
diff
changeset
|
454 # perl -lne '/^# *define (\w+)\(/ and print $1' lib/xalloc.h|grep -v '^__'; |
13103
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
455 # perl -lne '/^(?:extern )?(?:void|char) \*?(\w+) *\(/ and print $1' lib/xalloc.h |
12287
0c467c7c0132
maint.mk: Prohibit inclusion of "xalloc.h" without use.
Jim Meyering <meyering@redhat.com>
parents:
12245
diff
changeset
|
456 # Divide into two sets on case, and filter each through this: |
0c467c7c0132
maint.mk: Prohibit inclusion of "xalloc.h" without use.
Jim Meyering <meyering@redhat.com>
parents:
12245
diff
changeset
|
457 # | sort | perl -MRegexp::Assemble -le \ |
0c467c7c0132
maint.mk: Prohibit inclusion of "xalloc.h" without use.
Jim Meyering <meyering@redhat.com>
parents:
12245
diff
changeset
|
458 # 'print Regexp::Assemble->new(file => "/dev/stdin")->as_string'|sed 's/\?://g' |
0c467c7c0132
maint.mk: Prohibit inclusion of "xalloc.h" without use.
Jim Meyering <meyering@redhat.com>
parents:
12245
diff
changeset
|
459 # Note this was produced by the above: |
12576
51b39cac47f1
maint.mk: include 4 more function names in alloca.h-checking regexp
Jim Meyering <meyering@redhat.com>
parents:
12559
diff
changeset
|
460 # _xa1 = \ |
51b39cac47f1
maint.mk: include 4 more function names in alloca.h-checking regexp
Jim Meyering <meyering@redhat.com>
parents:
12559
diff
changeset
|
461 #x(((2n?)?re|c(har)?|n(re|m)|z)alloc|alloc_(oversized|die)|m(alloc|emdup)|strdup) |
51b39cac47f1
maint.mk: include 4 more function names in alloca.h-checking regexp
Jim Meyering <meyering@redhat.com>
parents:
12559
diff
changeset
|
462 # But we can do better, in at least two ways: |
51b39cac47f1
maint.mk: include 4 more function names in alloca.h-checking regexp
Jim Meyering <meyering@redhat.com>
parents:
12559
diff
changeset
|
463 # 1) take advantage of two "dup"-suffixed strings: |
51b39cac47f1
maint.mk: include 4 more function names in alloca.h-checking regexp
Jim Meyering <meyering@redhat.com>
parents:
12559
diff
changeset
|
464 # x(((2n?)?re|c(har)?|n(re|m)|[mz])alloc|alloc_(oversized|die)|(mem|str)dup) |
51b39cac47f1
maint.mk: include 4 more function names in alloca.h-checking regexp
Jim Meyering <meyering@redhat.com>
parents:
12559
diff
changeset
|
465 # 2) notice that "c(har)?|[mz]" is equivalent to the shorter and more readable |
51b39cac47f1
maint.mk: include 4 more function names in alloca.h-checking regexp
Jim Meyering <meyering@redhat.com>
parents:
12559
diff
changeset
|
466 # "char|[cmz]" |
51b39cac47f1
maint.mk: include 4 more function names in alloca.h-checking regexp
Jim Meyering <meyering@redhat.com>
parents:
12559
diff
changeset
|
467 # x(((2n?)?re|char|n(re|m)|[cmz])alloc|alloc_(oversized|die)|(mem|str)dup) |
51b39cac47f1
maint.mk: include 4 more function names in alloca.h-checking regexp
Jim Meyering <meyering@redhat.com>
parents:
12559
diff
changeset
|
468 _xa1 = x(((2n?)?re|char|n(re|m)|[cmz])alloc|alloc_(oversized|die)|(mem|str)dup) |
12287
0c467c7c0132
maint.mk: Prohibit inclusion of "xalloc.h" without use.
Jim Meyering <meyering@redhat.com>
parents:
12245
diff
changeset
|
469 _xa2 = X([CZ]|N?M)ALLOC |
0c467c7c0132
maint.mk: Prohibit inclusion of "xalloc.h" without use.
Jim Meyering <meyering@redhat.com>
parents:
12245
diff
changeset
|
470 sc_prohibit_xalloc_without_use: |
0c467c7c0132
maint.mk: Prohibit inclusion of "xalloc.h" without use.
Jim Meyering <meyering@redhat.com>
parents:
12245
diff
changeset
|
471 @h='"xalloc.h"' \ |
0c467c7c0132
maint.mk: Prohibit inclusion of "xalloc.h" without use.
Jim Meyering <meyering@redhat.com>
parents:
12245
diff
changeset
|
472 re='\<($(_xa1)|$(_xa2)) *\('\ |
13097
ef132cd978fc
maint.mk: s/_header_without_use/_sc_header_without_use/
Jose E. Marchesi <jemarch@gnu.org>
parents:
13085
diff
changeset
|
473 $(_sc_header_without_use) |
12287
0c467c7c0132
maint.mk: Prohibit inclusion of "xalloc.h" without use.
Jim Meyering <meyering@redhat.com>
parents:
12245
diff
changeset
|
474 |
12863
8fd1fe5e9683
maint.mk: prohibit inclusion of "hash.h" without_use
Jim Meyering <meyering@redhat.com>
parents:
12861
diff
changeset
|
475 # Extract function names: |
13103
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
476 # perl -lne '/^(?:extern )?(?:void|char) \*?(\w+) *\(/ and print $1' lib/hash.h |
12863
8fd1fe5e9683
maint.mk: prohibit inclusion of "hash.h" without_use
Jim Meyering <meyering@redhat.com>
parents:
12861
diff
changeset
|
477 _hash_re = \ |
8fd1fe5e9683
maint.mk: prohibit inclusion of "hash.h" without_use
Jim Meyering <meyering@redhat.com>
parents:
12861
diff
changeset
|
478 clear|delete|free|get_(first|next)|insert|lookup|print_statistics|reset_tuning |
8fd1fe5e9683
maint.mk: prohibit inclusion of "hash.h" without_use
Jim Meyering <meyering@redhat.com>
parents:
12861
diff
changeset
|
479 _hash_fn = \<($(_hash_re)) *\( |
8fd1fe5e9683
maint.mk: prohibit inclusion of "hash.h" without_use
Jim Meyering <meyering@redhat.com>
parents:
12861
diff
changeset
|
480 _hash_struct = (struct )?\<[Hh]ash_(table|tuning)\> |
8fd1fe5e9683
maint.mk: prohibit inclusion of "hash.h" without_use
Jim Meyering <meyering@redhat.com>
parents:
12861
diff
changeset
|
481 sc_prohibit_hash_without_use: |
12865
19a113553866
maint.mk: prohibit inclusion of "hash-pjw.h" without_use
Jim Meyering <meyering@redhat.com>
parents:
12864
diff
changeset
|
482 @h='"hash.h"' \ |
12863
8fd1fe5e9683
maint.mk: prohibit inclusion of "hash.h" without_use
Jim Meyering <meyering@redhat.com>
parents:
12861
diff
changeset
|
483 re='$(_hash_fn)|$(_hash_struct)'\ |
13097
ef132cd978fc
maint.mk: s/_header_without_use/_sc_header_without_use/
Jose E. Marchesi <jemarch@gnu.org>
parents:
13085
diff
changeset
|
484 $(_sc_header_without_use) |
12863
8fd1fe5e9683
maint.mk: prohibit inclusion of "hash.h" without_use
Jim Meyering <meyering@redhat.com>
parents:
12861
diff
changeset
|
485 |
12865
19a113553866
maint.mk: prohibit inclusion of "hash-pjw.h" without_use
Jim Meyering <meyering@redhat.com>
parents:
12864
diff
changeset
|
486 sc_prohibit_hash_pjw_without_use: |
19a113553866
maint.mk: prohibit inclusion of "hash-pjw.h" without_use
Jim Meyering <meyering@redhat.com>
parents:
12864
diff
changeset
|
487 @h='"hash-pjw.h"' \ |
19a113553866
maint.mk: prohibit inclusion of "hash-pjw.h" without_use
Jim Meyering <meyering@redhat.com>
parents:
12864
diff
changeset
|
488 re='\<hash_pjw *\(' \ |
13097
ef132cd978fc
maint.mk: s/_header_without_use/_sc_header_without_use/
Jose E. Marchesi <jemarch@gnu.org>
parents:
13085
diff
changeset
|
489 $(_sc_header_without_use) |
12865
19a113553866
maint.mk: prohibit inclusion of "hash-pjw.h" without_use
Jim Meyering <meyering@redhat.com>
parents:
12864
diff
changeset
|
490 |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
491 sc_prohibit_safe_read_without_use: |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
492 @h='"safe-read.h"' re='(\<SAFE_READ_ERROR\>|\<safe_read *\()' \ |
13097
ef132cd978fc
maint.mk: s/_header_without_use/_sc_header_without_use/
Jose E. Marchesi <jemarch@gnu.org>
parents:
13085
diff
changeset
|
493 $(_sc_header_without_use) |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
494 |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
495 sc_prohibit_argmatch_without_use: |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
496 @h='"argmatch.h"' \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
497 re='(\<(ARRAY_CARDINALITY|X?ARGMATCH(|_TO_ARGUMENT|_VERIFY))\>|\<argmatch(_exit_fn|_(in)?valid) *\()' \ |
13097
ef132cd978fc
maint.mk: s/_header_without_use/_sc_header_without_use/
Jose E. Marchesi <jemarch@gnu.org>
parents:
13085
diff
changeset
|
498 $(_sc_header_without_use) |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
499 |
12035
2898ee238452
syntax-check: detect unnecessary inclusion of canonicalize.h
Jim Meyering <meyering@redhat.com>
parents:
11969
diff
changeset
|
500 sc_prohibit_canonicalize_without_use: |
2898ee238452
syntax-check: detect unnecessary inclusion of canonicalize.h
Jim Meyering <meyering@redhat.com>
parents:
11969
diff
changeset
|
501 @h='"canonicalize.h"' \ |
2898ee238452
syntax-check: detect unnecessary inclusion of canonicalize.h
Jim Meyering <meyering@redhat.com>
parents:
11969
diff
changeset
|
502 re='CAN_(EXISTING|ALL_BUT_LAST|MISSING)|canonicalize_(mode_t|filename_mode)' \ |
13097
ef132cd978fc
maint.mk: s/_header_without_use/_sc_header_without_use/
Jose E. Marchesi <jemarch@gnu.org>
parents:
13085
diff
changeset
|
503 $(_sc_header_without_use) |
12035
2898ee238452
syntax-check: detect unnecessary inclusion of canonicalize.h
Jim Meyering <meyering@redhat.com>
parents:
11969
diff
changeset
|
504 |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
505 sc_prohibit_root_dev_ino_without_use: |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
506 @h='"root-dev-ino.h"' \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
507 re='(\<ROOT_DEV_INO_(CHECK|WARN)\>|\<get_root_dev_ino *\()' \ |
13097
ef132cd978fc
maint.mk: s/_header_without_use/_sc_header_without_use/
Jose E. Marchesi <jemarch@gnu.org>
parents:
13085
diff
changeset
|
508 $(_sc_header_without_use) |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
509 |
11954
c2aabb82023d
syntax-check: detect unnecessary inclusion of openat.h
Jim Meyering <meyering@redhat.com>
parents:
11927
diff
changeset
|
510 sc_prohibit_openat_without_use: |
c2aabb82023d
syntax-check: detect unnecessary inclusion of openat.h
Jim Meyering <meyering@redhat.com>
parents:
11927
diff
changeset
|
511 @h='"openat.h"' \ |
11969
25d12bf7e5bf
openat: provide more convenience names
Eric Blake <ebb9@byu.net>
parents:
11954
diff
changeset
|
512 re='\<(openat_(permissive|needs_fchdir|(save|restore)_fail)|l?(stat|ch(own|mod))at|(euid)?accessat)\>' \ |
13097
ef132cd978fc
maint.mk: s/_header_without_use/_sc_header_without_use/
Jose E. Marchesi <jemarch@gnu.org>
parents:
13085
diff
changeset
|
513 $(_sc_header_without_use) |
11954
c2aabb82023d
syntax-check: detect unnecessary inclusion of openat.h
Jim Meyering <meyering@redhat.com>
parents:
11927
diff
changeset
|
514 |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
515 # Prohibit the inclusion of c-ctype.h without an actual use. |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
516 ctype_re = isalnum|isalpha|isascii|isblank|iscntrl|isdigit|isgraph|islower\ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
517 |isprint|ispunct|isspace|isupper|isxdigit|tolower|toupper |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
518 sc_prohibit_c_ctype_without_use: |
13098
8d61d11c6cf0
maint.mk: split a long line
Jim Meyering <meyering@redhat.com>
parents:
13097
diff
changeset
|
519 @h='[<"]c-ctype.h[">]' re='\<c_($(ctype_re)) *\(' \ |
8d61d11c6cf0
maint.mk: split a long line
Jim Meyering <meyering@redhat.com>
parents:
13097
diff
changeset
|
520 $(_sc_header_without_use) |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
521 |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
522 _empty = |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
523 _sp = $(_empty) $(_empty) |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
524 # The following list was generated by running: |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
525 # man signal.h|col -b|perl -ne '/bsd_signal.*;/.../sigwaitinfo.*;/ and print' \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
526 # | perl -lne '/^\s+(?:int|void).*?(\w+).*/ and print $1' | fmt |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
527 _sig_functions = \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
528 bsd_signal kill killpg pthread_kill pthread_sigmask raise sigaction \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
529 sigaddset sigaltstack sigdelset sigemptyset sigfillset sighold sigignore \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
530 siginterrupt sigismember signal sigpause sigpending sigprocmask sigqueue \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
531 sigrelse sigset sigsuspend sigtimedwait sigwait sigwaitinfo |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
532 _sig_function_re = $(subst $(_sp),|,$(strip $(_sig_functions))) |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
533 # The following were extracted from "man signal.h" manually. |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
534 _sig_types_and_consts = \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
535 MINSIGSTKSZ SA_NOCLDSTOP SA_NOCLDWAIT SA_NODEFER SA_ONSTACK \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
536 SA_RESETHAND SA_RESTART SA_SIGINFO SIGEV_NONE SIGEV_SIGNAL \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
537 SIGEV_THREAD SIGSTKSZ SIG_BLOCK SIG_SETMASK SIG_UNBLOCK SS_DISABLE \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
538 SS_ONSTACK mcontext_t pid_t sig_atomic_t sigevent siginfo_t sigset_t \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
539 sigstack sigval stack_t ucontext_t |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
540 # generated via this: |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
541 # perl -lne '/^#ifdef (SIG\w+)/ and print $1' lib/sig2str.c|sort -u|fmt -70 |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
542 _sig_names = \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
543 SIGABRT SIGALRM SIGALRM1 SIGBUS SIGCANCEL SIGCHLD SIGCLD SIGCONT \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
544 SIGDANGER SIGDIL SIGEMT SIGFPE SIGFREEZE SIGGRANT SIGHUP SIGILL \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
545 SIGINFO SIGINT SIGIO SIGIOT SIGKAP SIGKILL SIGKILLTHR SIGLOST SIGLWP \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
546 SIGMIGRATE SIGMSG SIGPHONE SIGPIPE SIGPOLL SIGPRE SIGPROF SIGPWR \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
547 SIGQUIT SIGRETRACT SIGSAK SIGSEGV SIGSOUND SIGSTKFLT SIGSTOP SIGSYS \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
548 SIGTERM SIGTHAW SIGTRAP SIGTSTP SIGTTIN SIGTTOU SIGURG SIGUSR1 \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
549 SIGUSR2 SIGVIRT SIGVTALRM SIGWAITING SIGWINCH SIGWIND SIGWINDOW \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
550 SIGXCPU SIGXFSZ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
551 _sig_syms_re = $(subst $(_sp),|,$(strip $(_sig_names) $(_sig_types_and_consts))) |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
552 |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
553 # Prohibit the inclusion of signal.h without an actual use. |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
554 sc_prohibit_signal_without_use: |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
555 @h='<signal.h>' \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
556 re='\<($(_sig_function_re)) *\(|\<($(_sig_syms_re))\>' \ |
13097
ef132cd978fc
maint.mk: s/_header_without_use/_sc_header_without_use/
Jose E. Marchesi <jemarch@gnu.org>
parents:
13085
diff
changeset
|
557 $(_sc_header_without_use) |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
558 |
13304
e5593199cd7e
maint.mk: prohibit unwarranted use of <strings.h>
Jim Meyering <meyering@redhat.com>
parents:
13302
diff
changeset
|
559 # Prohibit the inclusion of strings.h without a sensible use. |
e5593199cd7e
maint.mk: prohibit unwarranted use of <strings.h>
Jim Meyering <meyering@redhat.com>
parents:
13302
diff
changeset
|
560 # Using the likes of bcmp, bcopy, bzero, index or rindex is not sensible. |
e5593199cd7e
maint.mk: prohibit unwarranted use of <strings.h>
Jim Meyering <meyering@redhat.com>
parents:
13302
diff
changeset
|
561 sc_prohibit_strings_without_use: |
e5593199cd7e
maint.mk: prohibit unwarranted use of <strings.h>
Jim Meyering <meyering@redhat.com>
parents:
13302
diff
changeset
|
562 @h='<strings.h>' \ |
e5593199cd7e
maint.mk: prohibit unwarranted use of <strings.h>
Jim Meyering <meyering@redhat.com>
parents:
13302
diff
changeset
|
563 re='\<(strn?casecmp|ffs(ll)?)\>' \ |
e5593199cd7e
maint.mk: prohibit unwarranted use of <strings.h>
Jim Meyering <meyering@redhat.com>
parents:
13302
diff
changeset
|
564 $(_sc_header_without_use) |
e5593199cd7e
maint.mk: prohibit unwarranted use of <strings.h>
Jim Meyering <meyering@redhat.com>
parents:
13302
diff
changeset
|
565 |
13029
44ae789f77b0
maint.mk: prohibit inclusion of "intprops.h" without use
Jim Meyering <meyering@redhat.com>
parents:
12866
diff
changeset
|
566 # Get the list of symbol names with this: |
44ae789f77b0
maint.mk: prohibit inclusion of "intprops.h" without use
Jim Meyering <meyering@redhat.com>
parents:
12866
diff
changeset
|
567 # perl -lne '/^# *define (\w+)\(/ and print $1' lib/intprops.h|grep -v '^s'|fmt |
44ae789f77b0
maint.mk: prohibit inclusion of "intprops.h" without use
Jim Meyering <meyering@redhat.com>
parents:
12866
diff
changeset
|
568 _intprops_names = \ |
44ae789f77b0
maint.mk: prohibit inclusion of "intprops.h" without use
Jim Meyering <meyering@redhat.com>
parents:
12866
diff
changeset
|
569 TYPE_IS_INTEGER TYPE_TWOS_COMPLEMENT TYPE_ONES_COMPLEMENT \ |
44ae789f77b0
maint.mk: prohibit inclusion of "intprops.h" without use
Jim Meyering <meyering@redhat.com>
parents:
12866
diff
changeset
|
570 TYPE_SIGNED_MAGNITUDE TYPE_SIGNED TYPE_MINIMUM TYPE_MAXIMUM \ |
44ae789f77b0
maint.mk: prohibit inclusion of "intprops.h" without use
Jim Meyering <meyering@redhat.com>
parents:
12866
diff
changeset
|
571 INT_STRLEN_BOUND INT_BUFSIZE_BOUND |
44ae789f77b0
maint.mk: prohibit inclusion of "intprops.h" without use
Jim Meyering <meyering@redhat.com>
parents:
12866
diff
changeset
|
572 _intprops_syms_re = $(subst $(_sp),|,$(strip $(_intprops_names))) |
44ae789f77b0
maint.mk: prohibit inclusion of "intprops.h" without use
Jim Meyering <meyering@redhat.com>
parents:
12866
diff
changeset
|
573 # Prohibit the inclusion of intprops.h without an actual use. |
44ae789f77b0
maint.mk: prohibit inclusion of "intprops.h" without use
Jim Meyering <meyering@redhat.com>
parents:
12866
diff
changeset
|
574 sc_prohibit_intprops_without_use: |
44ae789f77b0
maint.mk: prohibit inclusion of "intprops.h" without use
Jim Meyering <meyering@redhat.com>
parents:
12866
diff
changeset
|
575 @h='"intprops.h"' \ |
44ae789f77b0
maint.mk: prohibit inclusion of "intprops.h" without use
Jim Meyering <meyering@redhat.com>
parents:
12866
diff
changeset
|
576 re='\<($(_intprops_syms_re)) *\(' \ |
13097
ef132cd978fc
maint.mk: s/_header_without_use/_sc_header_without_use/
Jose E. Marchesi <jemarch@gnu.org>
parents:
13085
diff
changeset
|
577 $(_sc_header_without_use) |
13029
44ae789f77b0
maint.mk: prohibit inclusion of "intprops.h" without use
Jim Meyering <meyering@redhat.com>
parents:
12866
diff
changeset
|
578 |
14216
676f66180a36
maint.mk: add syntax-check rule: detect unnecessary #include <stddef.h>
Jim Meyering <meyering@redhat.com>
parents:
14210
diff
changeset
|
579 _stddef_syms_re = NULL|offsetof|ptrdiff_t|size_t|wchar_t |
676f66180a36
maint.mk: add syntax-check rule: detect unnecessary #include <stddef.h>
Jim Meyering <meyering@redhat.com>
parents:
14210
diff
changeset
|
580 # Prohibit the inclusion of stddef.h without an actual use. |
676f66180a36
maint.mk: add syntax-check rule: detect unnecessary #include <stddef.h>
Jim Meyering <meyering@redhat.com>
parents:
14210
diff
changeset
|
581 sc_prohibit_stddef_without_use: |
676f66180a36
maint.mk: add syntax-check rule: detect unnecessary #include <stddef.h>
Jim Meyering <meyering@redhat.com>
parents:
14210
diff
changeset
|
582 @h='<stddef.h>' \ |
676f66180a36
maint.mk: add syntax-check rule: detect unnecessary #include <stddef.h>
Jim Meyering <meyering@redhat.com>
parents:
14210
diff
changeset
|
583 re='\<($(_stddef_syms_re)) *\(' \ |
676f66180a36
maint.mk: add syntax-check rule: detect unnecessary #include <stddef.h>
Jim Meyering <meyering@redhat.com>
parents:
14210
diff
changeset
|
584 $(_sc_header_without_use) |
676f66180a36
maint.mk: add syntax-check rule: detect unnecessary #include <stddef.h>
Jim Meyering <meyering@redhat.com>
parents:
14210
diff
changeset
|
585 |
9811
5f79eb63c21e
Alter GNUmakefile to install into top directory.
Eric Blake <ebb9@byu.net>
parents:
diff
changeset
|
586 sc_obsolete_symbols: |
13103
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
587 @prohibit='\<(HAVE''_FCNTL_H|O''_NDELAY)\>' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
588 halt='do not use HAVE''_FCNTL_H or O'_NDELAY \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
589 $(_sc_search_regexp) |
9811
5f79eb63c21e
Alter GNUmakefile to install into top directory.
Eric Blake <ebb9@byu.net>
parents:
diff
changeset
|
590 |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
591 # FIXME: warn about definitions of EXIT_FAILURE, EXIT_SUCCESS, STREQ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
592 |
11468
65ab7e6dcafe
maint.mk: import changes to syntax-check macros from coreutils
Reuben Thomas <rrt@sc3d.org>
parents:
11406
diff
changeset
|
593 # Each nonempty ChangeLog line must start with a year number, or a TAB. |
9811
5f79eb63c21e
Alter GNUmakefile to install into top directory.
Eric Blake <ebb9@byu.net>
parents:
diff
changeset
|
594 sc_changelog: |
13103
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
595 @prohibit='^[^12 ]' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
596 in_vc_files='^ChangeLog$$' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
597 halt='found unexpected prefix in a ChangeLog' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
598 $(_sc_search_regexp) |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
599 |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
600 # Ensure that each .c file containing a "main" function also |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
601 # calls set_program_name. |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
602 sc_program_name: |
13103
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
603 @require='set_program_name *\(m?argv\[0\]\);' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
604 in_vc_files='\.c$$' \ |
13853
2f3dbd409322
maintainer-makefile: check for i18n setup
Eric Blake <eblake@redhat.com>
parents:
13704
diff
changeset
|
605 containing='\<main *(' \ |
13103
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
606 halt='the above files do not call set_program_name' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
607 $(_sc_search_regexp) |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
608 |
13853
2f3dbd409322
maintainer-makefile: check for i18n setup
Eric Blake <eblake@redhat.com>
parents:
13704
diff
changeset
|
609 # Ensure that each .c file containing a "main" function also |
2f3dbd409322
maintainer-makefile: check for i18n setup
Eric Blake <eblake@redhat.com>
parents:
13704
diff
changeset
|
610 # calls bindtextdomain. |
2f3dbd409322
maintainer-makefile: check for i18n setup
Eric Blake <eblake@redhat.com>
parents:
13704
diff
changeset
|
611 sc_bindtextdomain: |
2f3dbd409322
maintainer-makefile: check for i18n setup
Eric Blake <eblake@redhat.com>
parents:
13704
diff
changeset
|
612 @require='bindtextdomain *\(' \ |
2f3dbd409322
maintainer-makefile: check for i18n setup
Eric Blake <eblake@redhat.com>
parents:
13704
diff
changeset
|
613 in_vc_files='\.c$$' \ |
2f3dbd409322
maintainer-makefile: check for i18n setup
Eric Blake <eblake@redhat.com>
parents:
13704
diff
changeset
|
614 containing='\<main *(' \ |
2f3dbd409322
maintainer-makefile: check for i18n setup
Eric Blake <eblake@redhat.com>
parents:
13704
diff
changeset
|
615 halt='the above files do not call bindtextdomain' \ |
2f3dbd409322
maintainer-makefile: check for i18n setup
Eric Blake <eblake@redhat.com>
parents:
13704
diff
changeset
|
616 $(_sc_search_regexp) |
2f3dbd409322
maintainer-makefile: check for i18n setup
Eric Blake <eblake@redhat.com>
parents:
13704
diff
changeset
|
617 |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
618 # Require that the final line of each test-lib.sh-using test be this one: |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
619 # Exit $fail |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
620 # Note: this test requires GNU grep's --label= option. |
11646
fc7e7bde323e
tests: maint.mk: make sc_require_test_exit_idiom more generic
Jim Meyering <meyering@redhat.com>
parents:
11575
diff
changeset
|
621 Exit_witness_file ?= tests/test-lib.sh |
fc7e7bde323e
tests: maint.mk: make sc_require_test_exit_idiom more generic
Jim Meyering <meyering@redhat.com>
parents:
11575
diff
changeset
|
622 Exit_base := $(notdir $(Exit_witness_file)) |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
623 sc_require_test_exit_idiom: |
11646
fc7e7bde323e
tests: maint.mk: make sc_require_test_exit_idiom more generic
Jim Meyering <meyering@redhat.com>
parents:
11575
diff
changeset
|
624 @if test -f $(srcdir)/$(Exit_witness_file); then \ |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
625 die=0; \ |
11646
fc7e7bde323e
tests: maint.mk: make sc_require_test_exit_idiom more generic
Jim Meyering <meyering@redhat.com>
parents:
11575
diff
changeset
|
626 for i in $$(grep -l -F 'srcdir/$(Exit_base)' \ |
fc7e7bde323e
tests: maint.mk: make sc_require_test_exit_idiom more generic
Jim Meyering <meyering@redhat.com>
parents:
11575
diff
changeset
|
627 $$($(VC_LIST) tests)); do \ |
fc7e7bde323e
tests: maint.mk: make sc_require_test_exit_idiom more generic
Jim Meyering <meyering@redhat.com>
parents:
11575
diff
changeset
|
628 tail -n1 $$i | grep '^Exit .' > /dev/null \ |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
629 && : || { die=1; echo $$i; } \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
630 done; \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
631 test $$die = 1 && \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
632 { echo 1>&2 '$(ME): the final line in each of the above is not:'; \ |
11646
fc7e7bde323e
tests: maint.mk: make sc_require_test_exit_idiom more generic
Jim Meyering <meyering@redhat.com>
parents:
11575
diff
changeset
|
633 echo 1>&2 'Exit something'; \ |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
634 exit 1; } || :; \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
635 fi |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
636 |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
637 sc_the_the: |
13103
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
638 @prohibit='\<the ''the\>' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
639 ignore_case=1 \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
640 halt='found use of "the ''the";' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
641 $(_sc_search_regexp) |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
642 |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
643 sc_trailing_blank: |
13103
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
644 @prohibit='[ ]$$' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
645 halt='found trailing blank(s)' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
646 $(_sc_search_regexp) |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
647 |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
648 # Match lines like the following, but where there is only one space |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
649 # between the options and the description: |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
650 # -D, --all-repeated[=delimit-method] print all duplicate lines\n |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
651 longopt_re = --[a-z][0-9A-Za-z-]*(\[?=[0-9A-Za-z-]*\]?)? |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
652 sc_two_space_separator_in_usage: |
13103
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
653 @prohibit='^ *(-[A-Za-z],)? $(longopt_re) [^ ].*\\$$' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
654 halt='help2man requires at least two spaces between an option and its description'\ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
655 $(_sc_search_regexp) |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
656 |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
657 # Look for diagnostics that aren't marked for translation. |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
658 # This won't find any for which error's format string is on a separate line. |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
659 sc_unmarked_diagnostics: |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
660 @grep -nE \ |
13103
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
661 '\<error *\([^"]*"[^"]*[a-z]{3}' $$($(VC_LIST_EXCEPT)) \ |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
662 | grep -v '_''(' && \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
663 { echo '$(ME): found unmarked diagnostic(s)' 1>&2; \ |
9811
5f79eb63c21e
Alter GNUmakefile to install into top directory.
Eric Blake <ebb9@byu.net>
parents:
diff
changeset
|
664 exit 1; } || : |
5f79eb63c21e
Alter GNUmakefile to install into top directory.
Eric Blake <ebb9@byu.net>
parents:
diff
changeset
|
665 |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
666 # Avoid useless parentheses like those in this example: |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
667 # #if defined (SYMBOL) || defined (SYM2) |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
668 sc_useless_cpp_parens: |
13103
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
669 @prohibit='^# *if .*defined *\(' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
670 halt='found useless parentheses in cpp directive' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
671 $(_sc_search_regexp) |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
672 |
13275
2917aab6182c
maint.mk: syntax-check: prohibit HAVE_<header>_H that are always true
Jim Meyering <meyering@redhat.com>
parents:
13253
diff
changeset
|
673 # List headers for which HAVE_HEADER_H is always true, assuming you are |
2917aab6182c
maint.mk: syntax-check: prohibit HAVE_<header>_H that are always true
Jim Meyering <meyering@redhat.com>
parents:
13253
diff
changeset
|
674 # using the appropriate gnulib module. CAUTION: for each "unnecessary" |
2917aab6182c
maint.mk: syntax-check: prohibit HAVE_<header>_H that are always true
Jim Meyering <meyering@redhat.com>
parents:
13253
diff
changeset
|
675 # #if HAVE_HEADER_H that you remove, be sure that your project explicitly |
2917aab6182c
maint.mk: syntax-check: prohibit HAVE_<header>_H that are always true
Jim Meyering <meyering@redhat.com>
parents:
13253
diff
changeset
|
676 # requires the gnulib module that guarantees the usability of that header. |
13277
b07571bc9838
maint.mk: avoid side-effect in latest syntax-check
Jim Meyering <meyering@redhat.com>
parents:
13275
diff
changeset
|
677 gl_assured_headers_ = \ |
13278
a7cc50668746
maint.mk: fix a bug introduced in last change
Jim Meyering <meyering@redhat.com>
parents:
13277
diff
changeset
|
678 cd $(gnulib_dir)/lib && echo *.in.h|sed 's/\.in\.h//g' |
13275
2917aab6182c
maint.mk: syntax-check: prohibit HAVE_<header>_H that are always true
Jim Meyering <meyering@redhat.com>
parents:
13253
diff
changeset
|
679 |
2917aab6182c
maint.mk: syntax-check: prohibit HAVE_<header>_H that are always true
Jim Meyering <meyering@redhat.com>
parents:
13253
diff
changeset
|
680 # Convert the list of names to upper case, and replace each space with "|". |
2917aab6182c
maint.mk: syntax-check: prohibit HAVE_<header>_H that are always true
Jim Meyering <meyering@redhat.com>
parents:
13253
diff
changeset
|
681 az_ = abcdefghijklmnopqrstuvwxyz |
2917aab6182c
maint.mk: syntax-check: prohibit HAVE_<header>_H that are always true
Jim Meyering <meyering@redhat.com>
parents:
13253
diff
changeset
|
682 AZ_ = ABCDEFGHIJKLMNOPQRSTUVWXYZ |
13277
b07571bc9838
maint.mk: avoid side-effect in latest syntax-check
Jim Meyering <meyering@redhat.com>
parents:
13275
diff
changeset
|
683 gl_header_upper_case_or_ = \ |
b07571bc9838
maint.mk: avoid side-effect in latest syntax-check
Jim Meyering <meyering@redhat.com>
parents:
13275
diff
changeset
|
684 $$($(gl_assured_headers_) \ |
13275
2917aab6182c
maint.mk: syntax-check: prohibit HAVE_<header>_H that are always true
Jim Meyering <meyering@redhat.com>
parents:
13253
diff
changeset
|
685 | tr $(az_)/.- $(AZ_)___ \ |
2917aab6182c
maint.mk: syntax-check: prohibit HAVE_<header>_H that are always true
Jim Meyering <meyering@redhat.com>
parents:
13253
diff
changeset
|
686 | tr -s ' ' '|' \ |
13277
b07571bc9838
maint.mk: avoid side-effect in latest syntax-check
Jim Meyering <meyering@redhat.com>
parents:
13275
diff
changeset
|
687 ) |
13275
2917aab6182c
maint.mk: syntax-check: prohibit HAVE_<header>_H that are always true
Jim Meyering <meyering@redhat.com>
parents:
13253
diff
changeset
|
688 sc_prohibit_always_true_header_tests: |
13277
b07571bc9838
maint.mk: avoid side-effect in latest syntax-check
Jim Meyering <meyering@redhat.com>
parents:
13275
diff
changeset
|
689 @or=$(gl_header_upper_case_or_); \ |
b07571bc9838
maint.mk: avoid side-effect in latest syntax-check
Jim Meyering <meyering@redhat.com>
parents:
13275
diff
changeset
|
690 re="HAVE_($$or)_H"; \ |
b07571bc9838
maint.mk: avoid side-effect in latest syntax-check
Jim Meyering <meyering@redhat.com>
parents:
13275
diff
changeset
|
691 prohibit='\<'"$$re"'\>' \ |
13704
fe063c25c0ce
maint.mk: avoid unexpanded \n in two diagnostics
Jim Meyering <meyering@redhat.com>
parents:
13635
diff
changeset
|
692 halt=$$(printf '%s\n' \ |
fe063c25c0ce
maint.mk: avoid unexpanded \n in two diagnostics
Jim Meyering <meyering@redhat.com>
parents:
13635
diff
changeset
|
693 'do not test the above HAVE_<header>_H symbol(s);' \ |
fe063c25c0ce
maint.mk: avoid unexpanded \n in two diagnostics
Jim Meyering <meyering@redhat.com>
parents:
13635
diff
changeset
|
694 ' with the corresponding gnulib module, they are always true') \ |
13275
2917aab6182c
maint.mk: syntax-check: prohibit HAVE_<header>_H that are always true
Jim Meyering <meyering@redhat.com>
parents:
13253
diff
changeset
|
695 $(_sc_search_regexp) |
2917aab6182c
maint.mk: syntax-check: prohibit HAVE_<header>_H that are always true
Jim Meyering <meyering@redhat.com>
parents:
13253
diff
changeset
|
696 |
13329
256d057c7f1f
maint.mk: prohibit definition of symbols defined by gnulib
Jim Meyering <meyering@redhat.com>
parents:
13309
diff
changeset
|
697 # ================================================================== |
256d057c7f1f
maint.mk: prohibit definition of symbols defined by gnulib
Jim Meyering <meyering@redhat.com>
parents:
13309
diff
changeset
|
698 gl_other_headers_ ?= \ |
256d057c7f1f
maint.mk: prohibit definition of symbols defined by gnulib
Jim Meyering <meyering@redhat.com>
parents:
13309
diff
changeset
|
699 intprops.h \ |
256d057c7f1f
maint.mk: prohibit definition of symbols defined by gnulib
Jim Meyering <meyering@redhat.com>
parents:
13309
diff
changeset
|
700 openat.h \ |
256d057c7f1f
maint.mk: prohibit definition of symbols defined by gnulib
Jim Meyering <meyering@redhat.com>
parents:
13309
diff
changeset
|
701 stat-macros.h |
256d057c7f1f
maint.mk: prohibit definition of symbols defined by gnulib
Jim Meyering <meyering@redhat.com>
parents:
13309
diff
changeset
|
702 |
256d057c7f1f
maint.mk: prohibit definition of symbols defined by gnulib
Jim Meyering <meyering@redhat.com>
parents:
13309
diff
changeset
|
703 # Perl -lne code to extract "significant" cpp-defined symbols from a |
256d057c7f1f
maint.mk: prohibit definition of symbols defined by gnulib
Jim Meyering <meyering@redhat.com>
parents:
13309
diff
changeset
|
704 # gnulib header file, eliminating a few common false-positives. |
256d057c7f1f
maint.mk: prohibit definition of symbols defined by gnulib
Jim Meyering <meyering@redhat.com>
parents:
13309
diff
changeset
|
705 gl_extract_significant_defines_ = \ |
13337
23825f79a03a
maint.mk: gnulib doesn't guarantee NSIG
Eric Blake <eblake@redhat.com>
parents:
13333
diff
changeset
|
706 /^\# *define ([^_ (][^ (]*)(\s*\(|\s+\w+)/\ |
23825f79a03a
maint.mk: gnulib doesn't guarantee NSIG
Eric Blake <eblake@redhat.com>
parents:
13333
diff
changeset
|
707 && $$2 !~ /(?:rpl_|_used_without_)/\ |
13338
43bae0ee08b9
maint.mk: tweak preceding change
Jim Meyering <meyering@redhat.com>
parents:
13337
diff
changeset
|
708 && $$1 !~ /^(?:NSIG|ATTRIBUTE_NORETURN)$$/\ |
13329
256d057c7f1f
maint.mk: prohibit definition of symbols defined by gnulib
Jim Meyering <meyering@redhat.com>
parents:
13309
diff
changeset
|
709 and print $$1 |
256d057c7f1f
maint.mk: prohibit definition of symbols defined by gnulib
Jim Meyering <meyering@redhat.com>
parents:
13309
diff
changeset
|
710 |
256d057c7f1f
maint.mk: prohibit definition of symbols defined by gnulib
Jim Meyering <meyering@redhat.com>
parents:
13309
diff
changeset
|
711 # Create a list of regular expressions matching the names |
256d057c7f1f
maint.mk: prohibit definition of symbols defined by gnulib
Jim Meyering <meyering@redhat.com>
parents:
13309
diff
changeset
|
712 # of macros that are guaranteed to be defined by parts of gnulib. |
13333
dd75a1afe536
maint.mk: avoid using a temporary file in the always-defined-macros check
Jim Meyering <meyering@redhat.com>
parents:
13331
diff
changeset
|
713 define def_sym_regex |
dd75a1afe536
maint.mk: avoid using a temporary file in the always-defined-macros check
Jim Meyering <meyering@redhat.com>
parents:
13331
diff
changeset
|
714 gen_h=$(gl_generated_headers_); \ |
13329
256d057c7f1f
maint.mk: prohibit definition of symbols defined by gnulib
Jim Meyering <meyering@redhat.com>
parents:
13309
diff
changeset
|
715 (cd $(gnulib_dir)/lib; \ |
256d057c7f1f
maint.mk: prohibit definition of symbols defined by gnulib
Jim Meyering <meyering@redhat.com>
parents:
13309
diff
changeset
|
716 for f in *.in.h $(gl_other_headers_); do \ |
256d057c7f1f
maint.mk: prohibit definition of symbols defined by gnulib
Jim Meyering <meyering@redhat.com>
parents:
13309
diff
changeset
|
717 perl -lne '$(gl_extract_significant_defines_)' $$f; \ |
256d057c7f1f
maint.mk: prohibit definition of symbols defined by gnulib
Jim Meyering <meyering@redhat.com>
parents:
13309
diff
changeset
|
718 done; \ |
256d057c7f1f
maint.mk: prohibit definition of symbols defined by gnulib
Jim Meyering <meyering@redhat.com>
parents:
13309
diff
changeset
|
719 ) | sort -u \ |
13378
c4a059473950
maint.mk: also prohibit "#undef" of always-defined symbols
Jim Meyering <meyering@redhat.com>
parents:
13376
diff
changeset
|
720 | grep -Ev '^ATTRIBUTE_NORETURN' \ |
c4a059473950
maint.mk: also prohibit "#undef" of always-defined symbols
Jim Meyering <meyering@redhat.com>
parents:
13376
diff
changeset
|
721 | sed 's/^/^ *# *(define|undef) */;s/$$/\\>/' |
13329
256d057c7f1f
maint.mk: prohibit definition of symbols defined by gnulib
Jim Meyering <meyering@redhat.com>
parents:
13309
diff
changeset
|
722 endef |
256d057c7f1f
maint.mk: prohibit definition of symbols defined by gnulib
Jim Meyering <meyering@redhat.com>
parents:
13309
diff
changeset
|
723 |
256d057c7f1f
maint.mk: prohibit definition of symbols defined by gnulib
Jim Meyering <meyering@redhat.com>
parents:
13309
diff
changeset
|
724 # Don't define macros that we already get from gnulib header files. |
13333
dd75a1afe536
maint.mk: avoid using a temporary file in the always-defined-macros check
Jim Meyering <meyering@redhat.com>
parents:
13331
diff
changeset
|
725 sc_prohibit_always-defined_macros: |
13329
256d057c7f1f
maint.mk: prohibit definition of symbols defined by gnulib
Jim Meyering <meyering@redhat.com>
parents:
13309
diff
changeset
|
726 @if test -d $(gnulib_dir); then \ |
13333
dd75a1afe536
maint.mk: avoid using a temporary file in the always-defined-macros check
Jim Meyering <meyering@redhat.com>
parents:
13331
diff
changeset
|
727 case $$(echo all: | grep -l -f - Makefile) in Makefile);; *) \ |
dd75a1afe536
maint.mk: avoid using a temporary file in the always-defined-macros check
Jim Meyering <meyering@redhat.com>
parents:
13331
diff
changeset
|
728 echo '$(ME): skipping $@: you lack GNU grep' 1>&2; exit 0;; \ |
dd75a1afe536
maint.mk: avoid using a temporary file in the always-defined-macros check
Jim Meyering <meyering@redhat.com>
parents:
13331
diff
changeset
|
729 esac; \ |
13378
c4a059473950
maint.mk: also prohibit "#undef" of always-defined symbols
Jim Meyering <meyering@redhat.com>
parents:
13376
diff
changeset
|
730 $(def_sym_regex) | grep -E -f - $$($(VC_LIST_EXCEPT)) \ |
13329
256d057c7f1f
maint.mk: prohibit definition of symbols defined by gnulib
Jim Meyering <meyering@redhat.com>
parents:
13309
diff
changeset
|
731 && { echo '$(ME): define the above via some gnulib .h file' \ |
256d057c7f1f
maint.mk: prohibit definition of symbols defined by gnulib
Jim Meyering <meyering@redhat.com>
parents:
13309
diff
changeset
|
732 1>&2; exit 1; } || :; \ |
256d057c7f1f
maint.mk: prohibit definition of symbols defined by gnulib
Jim Meyering <meyering@redhat.com>
parents:
13309
diff
changeset
|
733 fi |
256d057c7f1f
maint.mk: prohibit definition of symbols defined by gnulib
Jim Meyering <meyering@redhat.com>
parents:
13309
diff
changeset
|
734 # ================================================================== |
256d057c7f1f
maint.mk: prohibit definition of symbols defined by gnulib
Jim Meyering <meyering@redhat.com>
parents:
13309
diff
changeset
|
735 |
13253
51e653bb55dd
top/maint.mk (sc_prohibit_backup_files): Prohibit checked in backup files.
Simon Josefsson <simon@josefsson.org>
parents:
13241
diff
changeset
|
736 # Prohibit checked in backup files. |
51e653bb55dd
top/maint.mk (sc_prohibit_backup_files): Prohibit checked in backup files.
Simon Josefsson <simon@josefsson.org>
parents:
13241
diff
changeset
|
737 sc_prohibit_backup_files: |
51e653bb55dd
top/maint.mk (sc_prohibit_backup_files): Prohibit checked in backup files.
Simon Josefsson <simon@josefsson.org>
parents:
13241
diff
changeset
|
738 @$(VC_LIST) | grep '~$$' && \ |
51e653bb55dd
top/maint.mk (sc_prohibit_backup_files): Prohibit checked in backup files.
Simon Josefsson <simon@josefsson.org>
parents:
13241
diff
changeset
|
739 { echo '$(ME): found version controlled backup file' 1>&2; \ |
51e653bb55dd
top/maint.mk (sc_prohibit_backup_files): Prohibit checked in backup files.
Simon Josefsson <simon@josefsson.org>
parents:
13241
diff
changeset
|
740 exit 1; } || : |
51e653bb55dd
top/maint.mk (sc_prohibit_backup_files): Prohibit checked in backup files.
Simon Josefsson <simon@josefsson.org>
parents:
13241
diff
changeset
|
741 |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
742 # Require the latest GPL. |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
743 sc_GPL_version: |
13103
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
744 @prohibit='either ''version [^3]' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
745 halt='GPL vN, N!=3' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
746 $(_sc_search_regexp) |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
747 |
12593
e8847ec6bc92
maint.mk: detect incorrect GFDL usage
Eric Blake <ebb9@byu.net>
parents:
12592
diff
changeset
|
748 # Require the latest GFDL. Two regexp, since some .texi files end up |
e8847ec6bc92
maint.mk: detect incorrect GFDL usage
Eric Blake <ebb9@byu.net>
parents:
12592
diff
changeset
|
749 # line wrapping between 'Free Documentation License,' and 'Version'. |
e8847ec6bc92
maint.mk: detect incorrect GFDL usage
Eric Blake <ebb9@byu.net>
parents:
12592
diff
changeset
|
750 _GFDL_regexp = (Free ''Documentation.*Version 1\.[^3]|Version 1\.[^3] or any) |
e8847ec6bc92
maint.mk: detect incorrect GFDL usage
Eric Blake <ebb9@byu.net>
parents:
12592
diff
changeset
|
751 sc_GFDL_version: |
13103
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
752 @prohibit='$(_GFDL_regexp)' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
753 halt='GFDL vN, N!=3' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
754 $(_sc_search_regexp) |
12593
e8847ec6bc92
maint.mk: detect incorrect GFDL usage
Eric Blake <ebb9@byu.net>
parents:
12592
diff
changeset
|
755 |
13880
fe3adfe381fa
maint.mk: add a URL discussing the no-@acronym policy
Jim Meyering <meyering@redhat.com>
parents:
13871
diff
changeset
|
756 # Don't use Texinfo's @acronym{}. |
fe3adfe381fa
maint.mk: add a URL discussing the no-@acronym policy
Jim Meyering <meyering@redhat.com>
parents:
13871
diff
changeset
|
757 # http://lists.gnu.org/archive/html/bug-gnulib/2010-03/msg00321.html |
13233
2376509f2722
maint.mk: texinfo_suffix_re_: correct the default regexp
Jim Meyering <meyering@redhat.com>
parents:
13232
diff
changeset
|
758 texinfo_suffix_re_ ?= \.(txi|texi(nfo)?)$$ |
13045
04f53e0b1556
top/maint.mk (sc_texinfo_acronym): Add rule.
Simon Josefsson <simon@josefsson.org>
parents:
13029
diff
changeset
|
759 sc_texinfo_acronym: |
13232
80668c656e53
top/maint.mk (sc_texinfo_acronym): Improve filename regexp.
Simon Josefsson <simon@josefsson.org>
parents:
13221
diff
changeset
|
760 @prohibit='@acronym\{' \ |
80668c656e53
top/maint.mk (sc_texinfo_acronym): Improve filename regexp.
Simon Josefsson <simon@josefsson.org>
parents:
13221
diff
changeset
|
761 in_vc_files='$(texinfo_suffix_re_)' \ |
13103
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
762 halt='found use of Texinfo @acronym{}' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
763 $(_sc_search_regexp) |
13045
04f53e0b1556
top/maint.mk (sc_texinfo_acronym): Add rule.
Simon Josefsson <simon@josefsson.org>
parents:
13029
diff
changeset
|
764 |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
765 cvs_keywords = \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
766 Author|Date|Header|Id|Name|Locker|Log|RCSfile|Revision|Source|State |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
767 |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
768 sc_prohibit_cvs_keyword: |
13103
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
769 @prohibit='\$$($(cvs_keywords))\$$' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
770 halt='do not use CVS keyword expansion' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
771 $(_sc_search_regexp) |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
772 |
13209
473dffe238d2
maint.mk: improve empty-line-at-EOF check
Jim Meyering <meyering@redhat.com>
parents:
13208
diff
changeset
|
773 # This Perl code is slightly obfuscated. Not only is each "$" doubled |
473dffe238d2
maint.mk: improve empty-line-at-EOF check
Jim Meyering <meyering@redhat.com>
parents:
13208
diff
changeset
|
774 # because it's in a Makefile, but the $$c's are comments; we cannot |
473dffe238d2
maint.mk: improve empty-line-at-EOF check
Jim Meyering <meyering@redhat.com>
parents:
13208
diff
changeset
|
775 # use "#" due to the way the script ends up concatenated onto one line. |
473dffe238d2
maint.mk: improve empty-line-at-EOF check
Jim Meyering <meyering@redhat.com>
parents:
13208
diff
changeset
|
776 # It would be much more concise, and would produce better output (including |
473dffe238d2
maint.mk: improve empty-line-at-EOF check
Jim Meyering <meyering@redhat.com>
parents:
13208
diff
changeset
|
777 # counts) if written as: |
13203
0f51b742ab58
maint.mk: new syntax-check rule: prohibit empty lines at end of file
Jim Meyering <meyering@redhat.com>
parents:
13202
diff
changeset
|
778 # perl -ln -0777 -e '/\n(\n+)$/ and print "$ARGV: ".length $1' ... |
0f51b742ab58
maint.mk: new syntax-check rule: prohibit empty lines at end of file
Jim Meyering <meyering@redhat.com>
parents:
13202
diff
changeset
|
779 # but that would be far less efficient, reading the entire contents |
13209
473dffe238d2
maint.mk: improve empty-line-at-EOF check
Jim Meyering <meyering@redhat.com>
parents:
13208
diff
changeset
|
780 # of each file, rather than just the last two bytes of each. |
14394
74d1558ae43f
maint.mk: detect missing-NL-at-EOF, too
Jim Meyering <meyering@redhat.com>
parents:
14357
diff
changeset
|
781 # In addition, while the code below detects both blank lines and a missing |
74d1558ae43f
maint.mk: detect missing-NL-at-EOF, too
Jim Meyering <meyering@redhat.com>
parents:
14357
diff
changeset
|
782 # newline at EOF, the above detects only the former. |
13203
0f51b742ab58
maint.mk: new syntax-check rule: prohibit empty lines at end of file
Jim Meyering <meyering@redhat.com>
parents:
13202
diff
changeset
|
783 # |
13209
473dffe238d2
maint.mk: improve empty-line-at-EOF check
Jim Meyering <meyering@redhat.com>
parents:
13208
diff
changeset
|
784 # This is a perl script that is expected to be the single-quoted argument |
473dffe238d2
maint.mk: improve empty-line-at-EOF check
Jim Meyering <meyering@redhat.com>
parents:
13208
diff
changeset
|
785 # to a command-line "-le". The remaining arguments are file names. |
14394
74d1558ae43f
maint.mk: detect missing-NL-at-EOF, too
Jim Meyering <meyering@redhat.com>
parents:
14357
diff
changeset
|
786 # Print the name of each file that ends in exactly one newline byte. |
74d1558ae43f
maint.mk: detect missing-NL-at-EOF, too
Jim Meyering <meyering@redhat.com>
parents:
14357
diff
changeset
|
787 # I.e., warn if there are blank lines (2 or more newlines), or if the |
74d1558ae43f
maint.mk: detect missing-NL-at-EOF, too
Jim Meyering <meyering@redhat.com>
parents:
14357
diff
changeset
|
788 # last byte is not a newline. However, currently we don't complain |
74d1558ae43f
maint.mk: detect missing-NL-at-EOF, too
Jim Meyering <meyering@redhat.com>
parents:
14357
diff
changeset
|
789 # about any file that contains exactly one byte. |
13203
0f51b742ab58
maint.mk: new syntax-check rule: prohibit empty lines at end of file
Jim Meyering <meyering@redhat.com>
parents:
13202
diff
changeset
|
790 # Exit nonzero if at least one such file is found, otherwise, exit 0. |
13209
473dffe238d2
maint.mk: improve empty-line-at-EOF check
Jim Meyering <meyering@redhat.com>
parents:
13208
diff
changeset
|
791 # Warn about, but otherwise ignore open failure. Ignore seek/read failure. |
13203
0f51b742ab58
maint.mk: new syntax-check rule: prohibit empty lines at end of file
Jim Meyering <meyering@redhat.com>
parents:
13202
diff
changeset
|
792 # |
0f51b742ab58
maint.mk: new syntax-check rule: prohibit empty lines at end of file
Jim Meyering <meyering@redhat.com>
parents:
13202
diff
changeset
|
793 # Use this if you want to remove trailing empty lines from selected files: |
0f51b742ab58
maint.mk: new syntax-check rule: prohibit empty lines at end of file
Jim Meyering <meyering@redhat.com>
parents:
13202
diff
changeset
|
794 # perl -pi -0777 -e 's/\n\n+$/\n/' files... |
0f51b742ab58
maint.mk: new syntax-check rule: prohibit empty lines at end of file
Jim Meyering <meyering@redhat.com>
parents:
13202
diff
changeset
|
795 # |
14394
74d1558ae43f
maint.mk: detect missing-NL-at-EOF, too
Jim Meyering <meyering@redhat.com>
parents:
14357
diff
changeset
|
796 require_exactly_one_NL_at_EOF_ = \ |
13241
cf12b14b472b
maint.mk: use gnu-style indentation in an embedded perl script
Jim Meyering <meyering@redhat.com>
parents:
13233
diff
changeset
|
797 foreach my $$f (@ARGV) \ |
cf12b14b472b
maint.mk: use gnu-style indentation in an embedded perl script
Jim Meyering <meyering@redhat.com>
parents:
13233
diff
changeset
|
798 { \ |
cf12b14b472b
maint.mk: use gnu-style indentation in an embedded perl script
Jim Meyering <meyering@redhat.com>
parents:
13233
diff
changeset
|
799 open F, "<", $$f or (warn "failed to open $$f: $$!\n"), next; \ |
cf12b14b472b
maint.mk: use gnu-style indentation in an embedded perl script
Jim Meyering <meyering@redhat.com>
parents:
13233
diff
changeset
|
800 my $$p = sysseek (F, -2, 2); \ |
cf12b14b472b
maint.mk: use gnu-style indentation in an embedded perl script
Jim Meyering <meyering@redhat.com>
parents:
13233
diff
changeset
|
801 my $$c = "seek failure probably means file has < 2 bytes; ignore"; \ |
cf12b14b472b
maint.mk: use gnu-style indentation in an embedded perl script
Jim Meyering <meyering@redhat.com>
parents:
13233
diff
changeset
|
802 my $$last_two_bytes; \ |
cf12b14b472b
maint.mk: use gnu-style indentation in an embedded perl script
Jim Meyering <meyering@redhat.com>
parents:
13233
diff
changeset
|
803 defined $$p and $$p = sysread F, $$last_two_bytes, 2; \ |
cf12b14b472b
maint.mk: use gnu-style indentation in an embedded perl script
Jim Meyering <meyering@redhat.com>
parents:
13233
diff
changeset
|
804 close F; \ |
cf12b14b472b
maint.mk: use gnu-style indentation in an embedded perl script
Jim Meyering <meyering@redhat.com>
parents:
13233
diff
changeset
|
805 $$c = "ignore read failure"; \ |
14394
74d1558ae43f
maint.mk: detect missing-NL-at-EOF, too
Jim Meyering <meyering@redhat.com>
parents:
14357
diff
changeset
|
806 $$p && ($$last_two_bytes eq "\n\n" \ |
74d1558ae43f
maint.mk: detect missing-NL-at-EOF, too
Jim Meyering <meyering@redhat.com>
parents:
14357
diff
changeset
|
807 || substr ($$last_two_bytes,1) ne "\n") \ |
74d1558ae43f
maint.mk: detect missing-NL-at-EOF, too
Jim Meyering <meyering@redhat.com>
parents:
14357
diff
changeset
|
808 and (print $$f), $$fail=1; \ |
13241
cf12b14b472b
maint.mk: use gnu-style indentation in an embedded perl script
Jim Meyering <meyering@redhat.com>
parents:
13233
diff
changeset
|
809 } \ |
cf12b14b472b
maint.mk: use gnu-style indentation in an embedded perl script
Jim Meyering <meyering@redhat.com>
parents:
13233
diff
changeset
|
810 END { exit defined $$fail } |
13203
0f51b742ab58
maint.mk: new syntax-check rule: prohibit empty lines at end of file
Jim Meyering <meyering@redhat.com>
parents:
13202
diff
changeset
|
811 sc_prohibit_empty_lines_at_EOF: |
14394
74d1558ae43f
maint.mk: detect missing-NL-at-EOF, too
Jim Meyering <meyering@redhat.com>
parents:
14357
diff
changeset
|
812 @perl -le '$(require_exactly_one_NL_at_EOF_)' $$($(VC_LIST_EXCEPT)) \ |
74d1558ae43f
maint.mk: detect missing-NL-at-EOF, too
Jim Meyering <meyering@redhat.com>
parents:
14357
diff
changeset
|
813 || { echo '$(ME): empty line(s) or no newline at EOF' \ |
13203
0f51b742ab58
maint.mk: new syntax-check rule: prohibit empty lines at end of file
Jim Meyering <meyering@redhat.com>
parents:
13202
diff
changeset
|
814 1>&2; exit 1; } || :; \ |
0f51b742ab58
maint.mk: new syntax-check rule: prohibit empty lines at end of file
Jim Meyering <meyering@redhat.com>
parents:
13202
diff
changeset
|
815 |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
816 # Make sure we don't use st_blocks. Use ST_NBLOCKS instead. |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
817 # This is a bit of a kludge, since it prevents use of the string |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
818 # even in comments, but for now it does the job with no false positives. |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
819 sc_prohibit_stat_st_blocks: |
13103
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
820 @prohibit='[.>]st_blocks' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
821 halt='do not use st_blocks; use ST_NBLOCKS' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
822 $(_sc_search_regexp) |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
823 |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
824 # Make sure we don't define any S_IS* macros in src/*.c files. |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
825 # They're already defined via gnulib's sys/stat.h replacement. |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
826 sc_prohibit_S_IS_definition: |
13103
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
827 @prohibit='^ *# *define *S_IS' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
828 halt='do not define S_IS* macros; include <sys/stat.h>' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
829 $(_sc_search_regexp) |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
830 |
13085
fb5165836aea
maint.mk: prohibit use of test's -o and -a operators
Jim Meyering <meyering@redhat.com>
parents:
13058
diff
changeset
|
831 _ptm1 = use "test C1 && test C2", not "test C1 -''a C2" |
fb5165836aea
maint.mk: prohibit use of test's -o and -a operators
Jim Meyering <meyering@redhat.com>
parents:
13058
diff
changeset
|
832 _ptm2 = use "test C1 || test C2", not "test C1 -''o C2" |
fb5165836aea
maint.mk: prohibit use of test's -o and -a operators
Jim Meyering <meyering@redhat.com>
parents:
13058
diff
changeset
|
833 # Using test's -a and -o operators is not portable. |
13101
8dad7d30953c
maint.mk: check for [ as well as test
Eric Blake <eblake@redhat.com>
parents:
13098
diff
changeset
|
834 # We prefer test over [, since the latter is spelled [[ in configure.ac. |
13085
fb5165836aea
maint.mk: prohibit use of test's -o and -a operators
Jim Meyering <meyering@redhat.com>
parents:
13058
diff
changeset
|
835 sc_prohibit_test_minus_ao: |
13104
3f4bdc04f3f1
maint.mk: fix some fallout
Eric Blake <eblake@redhat.com>
parents:
13103
diff
changeset
|
836 @prohibit='(\<test| \[+) .+ -[ao] ' \ |
3f4bdc04f3f1
maint.mk: fix some fallout
Eric Blake <eblake@redhat.com>
parents:
13103
diff
changeset
|
837 halt='$(_ptm1); $(_ptm2)' \ |
3f4bdc04f3f1
maint.mk: fix some fallout
Eric Blake <eblake@redhat.com>
parents:
13103
diff
changeset
|
838 $(_sc_search_regexp) |
13085
fb5165836aea
maint.mk: prohibit use of test's -o and -a operators
Jim Meyering <meyering@redhat.com>
parents:
13058
diff
changeset
|
839 |
13855
78d646edb95c
maintainer-makefile: prohibit test x == x
Eric Blake <eblake@redhat.com>
parents:
13853
diff
changeset
|
840 # Avoid a test bashism. |
78d646edb95c
maintainer-makefile: prohibit test x == x
Eric Blake <eblake@redhat.com>
parents:
13853
diff
changeset
|
841 sc_prohibit_test_double_equal: |
78d646edb95c
maintainer-makefile: prohibit test x == x
Eric Blake <eblake@redhat.com>
parents:
13853
diff
changeset
|
842 @prohibit='(\<test| \[+) .+ == ' \ |
13865
85922ab33c1c
maintainer-makefile: cover a few more files
Eric Blake <eblake@redhat.com>
parents:
13858
diff
changeset
|
843 containing='#! */bin/[a-z]*sh' \ |
13855
78d646edb95c
maintainer-makefile: prohibit test x == x
Eric Blake <eblake@redhat.com>
parents:
13853
diff
changeset
|
844 halt='use "test x = x", not "test x =''= x"' \ |
78d646edb95c
maintainer-makefile: prohibit test x == x
Eric Blake <eblake@redhat.com>
parents:
13853
diff
changeset
|
845 $(_sc_search_regexp) |
78d646edb95c
maintainer-makefile: prohibit test x == x
Eric Blake <eblake@redhat.com>
parents:
13853
diff
changeset
|
846 |
12608
d2cfa946257a
maint.mk: extend proper_name_utf8-vs-LIBICONV-checking rule
Jim Meyering <meyering@redhat.com>
parents:
12606
diff
changeset
|
847 # Each program that uses proper_name_utf8 must link with one of the |
d2cfa946257a
maint.mk: extend proper_name_utf8-vs-LIBICONV-checking rule
Jim Meyering <meyering@redhat.com>
parents:
12606
diff
changeset
|
848 # ICONV libraries. Otherwise, some ICONV library must appear in LDADD. |
d2cfa946257a
maint.mk: extend proper_name_utf8-vs-LIBICONV-checking rule
Jim Meyering <meyering@redhat.com>
parents:
12606
diff
changeset
|
849 # The perl -0777 invocation below extracts the possibly-multi-line |
d2cfa946257a
maint.mk: extend proper_name_utf8-vs-LIBICONV-checking rule
Jim Meyering <meyering@redhat.com>
parents:
12606
diff
changeset
|
850 # definition of LDADD from the appropriate Makefile.am and exits 0 |
d2cfa946257a
maint.mk: extend proper_name_utf8-vs-LIBICONV-checking rule
Jim Meyering <meyering@redhat.com>
parents:
12606
diff
changeset
|
851 # when it contains "ICONV". |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
852 sc_proper_name_utf8_requires_ICONV: |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
853 @progs=$$(grep -l 'proper_name_utf8 ''("' $$($(VC_LIST_EXCEPT)));\ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
854 if test "x$$progs" != x; then \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
855 fail=0; \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
856 for p in $$progs; do \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
857 dir=$$(dirname "$$p"); \ |
12608
d2cfa946257a
maint.mk: extend proper_name_utf8-vs-LIBICONV-checking rule
Jim Meyering <meyering@redhat.com>
parents:
12606
diff
changeset
|
858 perl -0777 \ |
d2cfa946257a
maint.mk: extend proper_name_utf8-vs-LIBICONV-checking rule
Jim Meyering <meyering@redhat.com>
parents:
12606
diff
changeset
|
859 -ne 'exit !(/^LDADD =(.+?[^\\]\n)/ms && $$1 =~ /ICONV/)' \ |
d2cfa946257a
maint.mk: extend proper_name_utf8-vs-LIBICONV-checking rule
Jim Meyering <meyering@redhat.com>
parents:
12606
diff
changeset
|
860 $$dir/Makefile.am && continue; \ |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
861 base=$$(basename "$$p" .c); \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
862 grep "$${base}_LDADD.*ICONV)" $$dir/Makefile.am > /dev/null \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
863 || { fail=1; echo 1>&2 "$(ME): $$p uses proper_name_utf8"; }; \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
864 done; \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
865 test $$fail = 1 && \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
866 { echo 1>&2 '$(ME): the above do not link with any ICONV library'; \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
867 exit 1; } || :; \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
868 fi |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
869 |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
870 # Warn about "c0nst struct Foo const foo[]", |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
871 # but not about "char const *const foo" or "#define const const". |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
872 sc_redundant_const: |
13103
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
873 @prohibit='\bconst\b[[:space:][:alnum:]]{2,}\bconst\b' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
874 halt='redundant "const" in declarations' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
875 $(_sc_search_regexp) |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
876 |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
877 sc_const_long_option: |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
878 @grep '^ *static.*struct option ' $$($(VC_LIST_EXCEPT)) \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
879 | grep -Ev 'const struct option|struct option const' && { \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
880 echo 1>&2 '$(ME): add "const" to the above declarations'; \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
881 exit 1; } || : |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
882 |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
883 NEWS_hash = \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
884 $$(sed -n '/^\*.* $(PREV_VERSION_REGEXP) ([0-9-]*)/,$$p' \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
885 $(srcdir)/NEWS \ |
12592
f46f6a6b078c
maint.mk: ignore multi-line copyright in NEWS
Eric Blake <ebb9@byu.net>
parents:
12581
diff
changeset
|
886 | perl -0777 -pe \ |
f46f6a6b078c
maint.mk: ignore multi-line copyright in NEWS
Eric Blake <ebb9@byu.net>
parents:
12581
diff
changeset
|
887 's/^Copyright.+?Free\sSoftware\sFoundation,\sInc\.\n//ms' \ |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
888 | md5sum - \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
889 | sed 's/ .*//') |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
890 |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
891 # Ensure that we don't accidentally insert an entry into an old NEWS block. |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
892 sc_immutable_NEWS: |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
893 @if test -f $(srcdir)/NEWS; then \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
894 test "$(NEWS_hash)" = '$(old_NEWS_hash)' && : || \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
895 { echo '$(ME): you have modified old NEWS' 1>&2; exit 1; }; \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
896 fi |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
897 |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
898 # Update the hash stored above. Do this after each release and |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
899 # for any corrections to old entries. |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
900 update-NEWS-hash: NEWS |
11750
246d2e2f523d
maint.mk: relax update-NEWS-hash to accept more syntax
Peter Simons <simons@cryp.to>
parents:
11736
diff
changeset
|
901 perl -pi -e 's/^(old_NEWS_hash[ \t]+:?=[ \t]+).*/$${1}'"$(NEWS_hash)/" \ |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
902 $(srcdir)/cfg.mk |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
903 |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
904 # Ensure that we use only the standard $(VAR) notation, |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
905 # not @...@ in Makefile.am, now that we can rely on automake |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
906 # to emit a definition for each substituted variable. |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
907 # We use perl rather than "grep -nE ..." to exempt a single |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
908 # use of an @...@-delimited variable name in src/Makefile.am. |
12606
47c45341eca0
maint.mk: allow packages to add makefile @@ exceptions
Eric Blake <ebb9@byu.net>
parents:
12593
diff
changeset
|
909 # Allow the package to add exceptions via a hook in cfg.mk; |
47c45341eca0
maint.mk: allow packages to add makefile @@ exceptions
Eric Blake <ebb9@byu.net>
parents:
12593
diff
changeset
|
910 # for example, @PRAGMA_SYSTEM_HEADER@ can be permitted by |
47c45341eca0
maint.mk: allow packages to add makefile @@ exceptions
Eric Blake <ebb9@byu.net>
parents:
12593
diff
changeset
|
911 # setting this to ' && !/PRAGMA_SYSTEM_HEADER/'. |
47c45341eca0
maint.mk: allow packages to add makefile @@ exceptions
Eric Blake <ebb9@byu.net>
parents:
12593
diff
changeset
|
912 _makefile_at_at_check_exceptions ?= |
47c45341eca0
maint.mk: allow packages to add makefile @@ exceptions
Eric Blake <ebb9@byu.net>
parents:
12593
diff
changeset
|
913 sc_makefile_at_at_check: |
47c45341eca0
maint.mk: allow packages to add makefile @@ exceptions
Eric Blake <ebb9@byu.net>
parents:
12593
diff
changeset
|
914 @perl -ne '/\@[A-Z_0-9]+\@/'$(_makefile_at_at_check_exceptions) \ |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
915 -e 'and (print "$$ARGV:$$.: $$_"), $$m=1; END {exit !$$m}' \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
916 $$($(VC_LIST_EXCEPT) | grep -E '(^|/)Makefile\.am$$') \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
917 && { echo '$(ME): use $$(...), not @...@' 1>&2; exit 1; } || : |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
918 |
12384
8620ff59758a
maint.mk: news-check: use grep -E
Jim Meyering <meyering@redhat.com>
parents:
12383
diff
changeset
|
919 news-check: NEWS |
13393
50e9a2259a7d
maint.mk: make the news-check rule more configurable
Peter Simons <simons@cryp.to>
parents:
13378
diff
changeset
|
920 if sed -n $(news-check-lines-spec)p $(srcdir)/NEWS \ |
50e9a2259a7d
maint.mk: make the news-check rule more configurable
Peter Simons <simons@cryp.to>
parents:
13378
diff
changeset
|
921 | grep -E $(news-check-regexp) >/dev/null; then \ |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
922 :; \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
923 else \ |
12384
8620ff59758a
maint.mk: news-check: use grep -E
Jim Meyering <meyering@redhat.com>
parents:
12383
diff
changeset
|
924 echo 'NEWS: $$(news-check-regexp) failed to match' 1>&2; \ |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
925 exit 1; \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
926 fi |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
927 |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
928 sc_makefile_TAB_only_indentation: |
13103
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
929 @prohibit='^ [ ]{8}' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
930 in_vc_files='akefile|\.mk$$' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
931 halt='found TAB-8-space indentation' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
932 $(_sc_search_regexp) |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
933 |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
934 sc_m4_quote_check: |
13103
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
935 @prohibit='(AC_DEFINE(_UNQUOTED)?|AC_DEFUN)\([^[]' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
936 in_vc_files='(^configure\.ac|\.m4)$$' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
937 halt='quote the first arg to AC_DEF*' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
938 $(_sc_search_regexp) |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
939 |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
940 fix_po_file_diag = \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
941 'you have changed the set of files with translatable diagnostics;\n\ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
942 apply the above patch\n' |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
943 |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
944 # Verify that all source files using _() are listed in po/POTFILES.in. |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
945 po_file = po/POTFILES.in |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
946 sc_po_check: |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
947 @if test -f $(po_file); then \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
948 grep -E -v '^(#|$$)' $(po_file) \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
949 | grep -v '^src/false\.c$$' | sort > $@-1; \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
950 files=; \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
951 for file in $$($(VC_LIST_EXCEPT)) lib/*.[ch]; do \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
952 test -r $$file || continue; \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
953 case $$file in \ |
11710
e59c94826772
maint.mk: don't look for translatable strings in *.m4 or *.mk
Jim Meyering <meyering@redhat.com>
parents:
11646
diff
changeset
|
954 *.m4|*.mk) continue ;; \ |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
955 *.?|*.??) ;; \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
956 *) continue;; \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
957 esac; \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
958 case $$file in \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
959 *.[ch]) \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
960 base=`expr " $$file" : ' \(.*\)\..'`; \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
961 { test -f $$base.l || test -f $$base.y; } && continue;; \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
962 esac; \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
963 files="$$files $$file"; \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
964 done; \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
965 grep -E -l '\b(N?_|gettext *)\([^)"]*("|$$)' $$files \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
966 | sort -u > $@-2; \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
967 diff -u -L $(po_file) -L $(po_file) $@-1 $@-2 \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
968 || { printf '$(ME): '$(fix_po_file_diag) 1>&2; exit 1; }; \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
969 rm -f $@-1 $@-2; \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
970 fi |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
971 |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
972 # Sometimes it is useful to change the PATH environment variable |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
973 # in Makefiles. When doing so, it's better not to use the Unix-centric |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
974 # path separator of `:', but rather the automake-provided `$(PATH_SEPARATOR)'. |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
975 msg = '$(ME): Do not use `:'\'' above; use $$(PATH_SEPARATOR) instead' |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
976 sc_makefile_path_separator_check: |
13103
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
977 @prohibit='PATH[=].*:' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
978 in_vc_files='akefile|\.mk$$' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
979 halt=$(msg) \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
980 $(_sc_search_regexp) |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
981 |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
982 # Check that `make alpha' will not fail at the end of the process. |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
983 writable-files: |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
984 if test -d $(release_archive_dir); then :; else \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
985 for file in $(distdir).tar.gz \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
986 $(release_archive_dir)/$(distdir).tar.gz; do \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
987 test -e $$file || continue; \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
988 test -w $$file \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
989 || { echo ERROR: $$file is not writable; fail=1; }; \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
990 done; \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
991 test "$$fail" && exit 1 || : ; \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
992 fi |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
993 |
13635
102e01a1ca93
maint.mk: avoid obscure sc_copyright_check failure in coreutils
Jim Meyering <meyering@redhat.com>
parents:
13393
diff
changeset
|
994 v_etc_file = $(gnulib_dir)/lib/version-etc.c |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
995 sample-test = tests/sample-test |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
996 texi = doc/$(PACKAGE).texi |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
997 # Make sure that the copyright date in $(v_etc_file) is up to date. |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
998 # Do the same for the $(sample-test) and the main doc/.texi file. |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
999 sc_copyright_check: |
13103
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
1000 @require='enum { COPYRIGHT_YEAR = '$$(date +%Y)' };' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
1001 in_files=$(v_etc_file) \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
1002 halt='out of date copyright in $(v_etc_file); update it' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
1003 $(_sc_search_regexp) |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
1004 @require='# Copyright \(C\) '$$(date +%Y)' Free' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
1005 in_vc_files=$(sample-test) \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
1006 halt='out of date copyright in $(sample-test); update it' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
1007 $(_sc_search_regexp) |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
1008 @require='Copyright @copyright\{\} .*'$$(date +%Y)' Free' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
1009 in_vc_files=$(texi) \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
1010 halt='out of date copyright in $(texi); update it' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
1011 $(_sc_search_regexp) |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
1012 |
13185
8845faccc61b
maint.mk: add a PATH crosschecking syntax-check rule
Jim Meyering <meyering@redhat.com>
parents:
13126
diff
changeset
|
1013 # If tests/help-version exists and seems to be new enough, assume that its |
8845faccc61b
maint.mk: add a PATH crosschecking syntax-check rule
Jim Meyering <meyering@redhat.com>
parents:
13126
diff
changeset
|
1014 # use of init.sh and path_prepend_ is correct, and ensure that every other |
8845faccc61b
maint.mk: add a PATH crosschecking syntax-check rule
Jim Meyering <meyering@redhat.com>
parents:
13126
diff
changeset
|
1015 # use of init.sh is identical. |
8845faccc61b
maint.mk: add a PATH crosschecking syntax-check rule
Jim Meyering <meyering@redhat.com>
parents:
13126
diff
changeset
|
1016 # This is useful because help-version cross-checks prog --version |
8845faccc61b
maint.mk: add a PATH crosschecking syntax-check rule
Jim Meyering <meyering@redhat.com>
parents:
13126
diff
changeset
|
1017 # with $(VERSION), which verifies that its path_prepend_ invocation |
8845faccc61b
maint.mk: add a PATH crosschecking syntax-check rule
Jim Meyering <meyering@redhat.com>
parents:
13126
diff
changeset
|
1018 # sets PATH correctly. This is an inexpensive way to ensure that |
8845faccc61b
maint.mk: add a PATH crosschecking syntax-check rule
Jim Meyering <meyering@redhat.com>
parents:
13126
diff
changeset
|
1019 # the other init.sh-using tests also get it right. |
8845faccc61b
maint.mk: add a PATH crosschecking syntax-check rule
Jim Meyering <meyering@redhat.com>
parents:
13126
diff
changeset
|
1020 _hv_file ?= $(srcdir)/tests/help-version |
13331
dc2c7e3048ec
maint.mk: improve sc_cross_check_PATH_usage_in_tests
Jim Meyering <meyering@redhat.com>
parents:
13329
diff
changeset
|
1021 _hv_regex_weak ?= ^ *\. .*/init\.sh" |
dc2c7e3048ec
maint.mk: improve sc_cross_check_PATH_usage_in_tests
Jim Meyering <meyering@redhat.com>
parents:
13329
diff
changeset
|
1022 _hv_regex_strong ?= ^ *\. "\$${srcdir=\.}/init\.sh" |
13185
8845faccc61b
maint.mk: add a PATH crosschecking syntax-check rule
Jim Meyering <meyering@redhat.com>
parents:
13126
diff
changeset
|
1023 sc_cross_check_PATH_usage_in_tests: |
13192
ba35e9647b3b
maint.mk: print better diagnostic when there is no $(_hv_file)
Jim Meyering <meyering@redhat.com>
parents:
13185
diff
changeset
|
1024 @if test -f $(_hv_file); then \ |
13331
dc2c7e3048ec
maint.mk: improve sc_cross_check_PATH_usage_in_tests
Jim Meyering <meyering@redhat.com>
parents:
13329
diff
changeset
|
1025 grep -l 'VERSION mismatch' $(_hv_file) >/dev/null \ |
dc2c7e3048ec
maint.mk: improve sc_cross_check_PATH_usage_in_tests
Jim Meyering <meyering@redhat.com>
parents:
13329
diff
changeset
|
1026 || { echo "$@: skipped: no such file: $(_hv_file)" 1>&2; \ |
dc2c7e3048ec
maint.mk: improve sc_cross_check_PATH_usage_in_tests
Jim Meyering <meyering@redhat.com>
parents:
13329
diff
changeset
|
1027 exit 0; }; \ |
dc2c7e3048ec
maint.mk: improve sc_cross_check_PATH_usage_in_tests
Jim Meyering <meyering@redhat.com>
parents:
13329
diff
changeset
|
1028 grep -lE '$(_hv_regex_strong)' $(_hv_file) >/dev/null \ |
dc2c7e3048ec
maint.mk: improve sc_cross_check_PATH_usage_in_tests
Jim Meyering <meyering@redhat.com>
parents:
13329
diff
changeset
|
1029 || { echo "$@: $(_hv_file) lacks conforming use of init.sh" 1>&2; \ |
dc2c7e3048ec
maint.mk: improve sc_cross_check_PATH_usage_in_tests
Jim Meyering <meyering@redhat.com>
parents:
13329
diff
changeset
|
1030 exit 1; }; \ |
dc2c7e3048ec
maint.mk: improve sc_cross_check_PATH_usage_in_tests
Jim Meyering <meyering@redhat.com>
parents:
13329
diff
changeset
|
1031 good=$$(grep -E '$(_hv_regex_strong)' $(_hv_file)); \ |
dc2c7e3048ec
maint.mk: improve sc_cross_check_PATH_usage_in_tests
Jim Meyering <meyering@redhat.com>
parents:
13329
diff
changeset
|
1032 grep -LFx "$$good" \ |
dc2c7e3048ec
maint.mk: improve sc_cross_check_PATH_usage_in_tests
Jim Meyering <meyering@redhat.com>
parents:
13329
diff
changeset
|
1033 $$(grep -lE '$(_hv_regex_weak)' $$($(VC_LIST_EXCEPT))) \ |
dc2c7e3048ec
maint.mk: improve sc_cross_check_PATH_usage_in_tests
Jim Meyering <meyering@redhat.com>
parents:
13329
diff
changeset
|
1034 | grep . && \ |
dc2c7e3048ec
maint.mk: improve sc_cross_check_PATH_usage_in_tests
Jim Meyering <meyering@redhat.com>
parents:
13329
diff
changeset
|
1035 { echo "$(ME): the above files use path_prepend_ inconsistently" \ |
dc2c7e3048ec
maint.mk: improve sc_cross_check_PATH_usage_in_tests
Jim Meyering <meyering@redhat.com>
parents:
13329
diff
changeset
|
1036 1>&2; exit 1; } || :; \ |
13185
8845faccc61b
maint.mk: add a PATH crosschecking syntax-check rule
Jim Meyering <meyering@redhat.com>
parents:
13126
diff
changeset
|
1037 fi |
8845faccc61b
maint.mk: add a PATH crosschecking syntax-check rule
Jim Meyering <meyering@redhat.com>
parents:
13126
diff
changeset
|
1038 |
12816
7a4d2814715b
syntax-check: detect incorrect boolean macro values in config.h
Pádraig Brady <P@draigBrady.com>
parents:
12773
diff
changeset
|
1039 # #if HAVE_... will evaluate to false for any non numeric string. |
7a4d2814715b
syntax-check: detect incorrect boolean macro values in config.h
Pádraig Brady <P@draigBrady.com>
parents:
12773
diff
changeset
|
1040 # That would be flagged by using -Wundef, however gnulib currently |
7a4d2814715b
syntax-check: detect incorrect boolean macro values in config.h
Pádraig Brady <P@draigBrady.com>
parents:
12773
diff
changeset
|
1041 # tests many undefined macros, and so we can't enable that option. |
7a4d2814715b
syntax-check: detect incorrect boolean macro values in config.h
Pádraig Brady <P@draigBrady.com>
parents:
12773
diff
changeset
|
1042 # So at least preclude common boolean strings as macro values. |
7a4d2814715b
syntax-check: detect incorrect boolean macro values in config.h
Pádraig Brady <P@draigBrady.com>
parents:
12773
diff
changeset
|
1043 sc_Wundef_boolean: |
13103
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
1044 @prohibit='^#define.*(yes|no|true|false)$$' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
1045 in_files='$(CONFIG_INCLUDE)' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
1046 halt='Use 0 or 1 for macro values' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
1047 $(_sc_search_regexp) |
12816
7a4d2814715b
syntax-check: detect incorrect boolean macro values in config.h
Pádraig Brady <P@draigBrady.com>
parents:
12773
diff
changeset
|
1048 |
12827
d844fd639561
maint: add a syntax-check rule to check for vulnerable Makefile.in
Jim Meyering <meyering@redhat.com>
parents:
12816
diff
changeset
|
1049 sc_vulnerable_makefile_CVE-2009-4029: |
13103
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
1050 @prohibit='perm -777 -exec chmod a\+rwx|chmod 777 \$$\(distdir\)' \ |
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
1051 in_files=$$(find $(srcdir) -name Makefile.in) \ |
13704
fe063c25c0ce
maint.mk: avoid unexpanded \n in two diagnostics
Jim Meyering <meyering@redhat.com>
parents:
13635
diff
changeset
|
1052 halt=$$(printf '%s\n' \ |
fe063c25c0ce
maint.mk: avoid unexpanded \n in two diagnostics
Jim Meyering <meyering@redhat.com>
parents:
13635
diff
changeset
|
1053 'the above files are vulnerable; beware of running' \ |
fe063c25c0ce
maint.mk: avoid unexpanded \n in two diagnostics
Jim Meyering <meyering@redhat.com>
parents:
13635
diff
changeset
|
1054 ' "make dist*" rules, and upgrade to fixed automake' \ |
fe063c25c0ce
maint.mk: avoid unexpanded \n in two diagnostics
Jim Meyering <meyering@redhat.com>
parents:
13635
diff
changeset
|
1055 ' see http://bugzilla.redhat.com/542609 for details') \ |
13103
3622812c3a89
maint.mk: _sc_search_regexp: generalize and rename from _prohibit_regexp
Jose E. Marchesi <jemarch@gnu.org>
parents:
13102
diff
changeset
|
1056 $(_sc_search_regexp) |
12827
d844fd639561
maint: add a syntax-check rule to check for vulnerable Makefile.in
Jim Meyering <meyering@redhat.com>
parents:
12816
diff
changeset
|
1057 |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
1058 vc-diff-check: |
11501
5223ecd9ec17
maint.mk: import improvements from m4
Eric Blake <ebb9@byu.net>
parents:
11500
diff
changeset
|
1059 (unset CDPATH; cd $(srcdir) && $(VC) diff) > vc-diffs || : |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
1060 if test -s vc-diffs; then \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
1061 cat vc-diffs; \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
1062 echo "Some files are locally modified:" 1>&2; \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
1063 exit 1; \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
1064 else \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
1065 rm vc-diffs; \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
1066 fi |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
1067 |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
1068 rel-files = $(DIST_ARCHIVES) |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
1069 |
11575
cfe435cf936c
maint.mk: allow gnulib_dir in VPATH build
Eric Blake <ebb9@byu.net>
parents:
11573
diff
changeset
|
1070 gnulib_dir ?= $(srcdir)/gnulib |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
1071 gnulib-version = $$(cd $(gnulib_dir) && git describe) |
11501
5223ecd9ec17
maint.mk: import improvements from m4
Eric Blake <ebb9@byu.net>
parents:
11500
diff
changeset
|
1072 bootstrap-tools ?= autoconf,automake,gnulib |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
1073 |
12539
dafd055586ef
maint.mk: don't require explicit gpg_key_ID in cfg.mk
Jim Meyering <meyering@redhat.com>
parents:
12538
diff
changeset
|
1074 # If it's not already specified, derive the GPG key ID from |
dafd055586ef
maint.mk: don't require explicit gpg_key_ID in cfg.mk
Jim Meyering <meyering@redhat.com>
parents:
12538
diff
changeset
|
1075 # the signed tag we've just applied to mark this release. |
dafd055586ef
maint.mk: don't require explicit gpg_key_ID in cfg.mk
Jim Meyering <meyering@redhat.com>
parents:
12538
diff
changeset
|
1076 gpg_key_ID ?= \ |
dafd055586ef
maint.mk: don't require explicit gpg_key_ID in cfg.mk
Jim Meyering <meyering@redhat.com>
parents:
12538
diff
changeset
|
1077 $$(git cat-file tag v$(VERSION) > .ann-sig \ |
dafd055586ef
maint.mk: don't require explicit gpg_key_ID in cfg.mk
Jim Meyering <meyering@redhat.com>
parents:
12538
diff
changeset
|
1078 && gpgv .ann-sig - < /dev/null 2>&1 \ |
dafd055586ef
maint.mk: don't require explicit gpg_key_ID in cfg.mk
Jim Meyering <meyering@redhat.com>
parents:
12538
diff
changeset
|
1079 | sed -n '/.*key ID \([0-9A-F]*\)/s//\1/p'; rm -f .ann-sig) |
dafd055586ef
maint.mk: don't require explicit gpg_key_ID in cfg.mk
Jim Meyering <meyering@redhat.com>
parents:
12538
diff
changeset
|
1080 |
13306
3b72fc82a799
maint.mk, announce-gen: allow project-specific announcement mail headers
Jim Meyering <meyering@redhat.com>
parents:
13304
diff
changeset
|
1081 translation_project_ ?= coordinator@translationproject.org |
3b72fc82a799
maint.mk, announce-gen: allow project-specific announcement mail headers
Jim Meyering <meyering@redhat.com>
parents:
13304
diff
changeset
|
1082 announcement_Cc_ ?= $(translation_project_), $(PACKAGE_BUGREPORT) |
3b72fc82a799
maint.mk, announce-gen: allow project-specific announcement mail headers
Jim Meyering <meyering@redhat.com>
parents:
13304
diff
changeset
|
1083 announcement_mail_headers_ ?= \ |
3b72fc82a799
maint.mk, announce-gen: allow project-specific announcement mail headers
Jim Meyering <meyering@redhat.com>
parents:
13304
diff
changeset
|
1084 To: info-gnu@gnu.org \ |
3b72fc82a799
maint.mk, announce-gen: allow project-specific announcement mail headers
Jim Meyering <meyering@redhat.com>
parents:
13304
diff
changeset
|
1085 Cc: $(announcement_Cc_) \ |
3b72fc82a799
maint.mk, announce-gen: allow project-specific announcement mail headers
Jim Meyering <meyering@redhat.com>
parents:
13304
diff
changeset
|
1086 Mail-Followup-To: $(PACKAGE_BUGREPORT) |
3b72fc82a799
maint.mk, announce-gen: allow project-specific announcement mail headers
Jim Meyering <meyering@redhat.com>
parents:
13304
diff
changeset
|
1087 |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
1088 announcement: NEWS ChangeLog $(rel-files) |
11564
ab18b3d5e34a
* top/maint.mk: Adjust backslash alignment.
Jim Meyering <meyering@redhat.com>
parents:
11563
diff
changeset
|
1089 @$(build_aux)/announce-gen \ |
13306
3b72fc82a799
maint.mk, announce-gen: allow project-specific announcement mail headers
Jim Meyering <meyering@redhat.com>
parents:
13304
diff
changeset
|
1090 --mail-headers='$(announcement_mail_headers_)' \ |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
1091 --release-type=$(RELEASE_TYPE) \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
1092 --package=$(PACKAGE) \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
1093 --prev=$(PREV_VERSION) \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
1094 --curr=$(VERSION) \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
1095 --gpg-key-id=$(gpg_key_ID) \ |
12581
25232b487199
maint: support 'make announcement' from a VPATH build
Eric Blake <ebb9@byu.net>
parents:
12576
diff
changeset
|
1096 --news=$(srcdir)/NEWS \ |
11501
5223ecd9ec17
maint.mk: import improvements from m4
Eric Blake <ebb9@byu.net>
parents:
11500
diff
changeset
|
1097 --bootstrap-tools=$(bootstrap-tools) \ |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
1098 --gnulib-version=$(gnulib-version) \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
1099 --no-print-checksums \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
1100 $(addprefix --url-dir=, $(url_dir_list)) |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
1101 |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
1102 ## ---------------- ## |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
1103 ## Updating files. ## |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
1104 ## ---------------- ## |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
1105 |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
1106 ftp-gnu = ftp://ftp.gnu.org/gnu |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
1107 www-gnu = http://www.gnu.org |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
1108 |
12864
9ab78f8f0368
maint.mk: allow the default upload destination dir to be overridden
Jim Meyering <meyering@redhat.com>
parents:
12863
diff
changeset
|
1109 upload_dest_dir_ ?= $(PACKAGE) |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
1110 emit_upload_commands: |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
1111 @echo ===================================== |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
1112 @echo ===================================== |
11563
861b812ba15f
top/maint.mk: Make $(srcdir)/build-aux configurable.
Simon Josefsson <simon@josefsson.org>
parents:
11547
diff
changeset
|
1113 @echo "$(build_aux)/gnupload $(GNUPLOADFLAGS) \\" |
12864
9ab78f8f0368
maint.mk: allow the default upload destination dir to be overridden
Jim Meyering <meyering@redhat.com>
parents:
12863
diff
changeset
|
1114 @echo " --to $(gnu_rel_host):$(upload_dest_dir_) \\" |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
1115 @echo " $(rel-files)" |
12538
a8059f620e8f
maint.mk: create announcement template in ~/, not in /tmp
Jim Meyering <meyering@redhat.com>
parents:
12443
diff
changeset
|
1116 @echo '# send the ~/announce-$(my_distdir) e-mail' |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
1117 @echo ===================================== |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
1118 @echo ===================================== |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
1119 |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
1120 define emit-commit-log |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
1121 printf '%s\n' 'post-release administrivia' '' \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
1122 '* NEWS: Add header line for next release.' \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
1123 '* .prev-version: Record previous version.' \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
1124 '* cfg.mk (old_NEWS_hash): Auto-update.' |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
1125 endef |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
1126 |
11534
9f608fba8b1c
build: ensure that a release build fails when a submodule is unclean
Jim Meyering <meyering@redhat.com>
parents:
11519
diff
changeset
|
1127 .PHONY: no-submodule-changes |
9f608fba8b1c
build: ensure that a release build fails when a submodule is unclean
Jim Meyering <meyering@redhat.com>
parents:
11519
diff
changeset
|
1128 no-submodule-changes: |
11547
3f43faadaf9d
maint: make the new no-submodule-changes rule VPATH-safe
Eric Blake <ebb9@byu.net>
parents:
11534
diff
changeset
|
1129 if test -d $(srcdir)/.git; then \ |
3f43faadaf9d
maint: make the new no-submodule-changes rule VPATH-safe
Eric Blake <ebb9@byu.net>
parents:
11534
diff
changeset
|
1130 diff=$$(cd $(srcdir) && git submodule -q foreach \ |
3f43faadaf9d
maint: make the new no-submodule-changes rule VPATH-safe
Eric Blake <ebb9@byu.net>
parents:
11534
diff
changeset
|
1131 git diff-index --name-only HEAD) \ |
11534
9f608fba8b1c
build: ensure that a release build fails when a submodule is unclean
Jim Meyering <meyering@redhat.com>
parents:
11519
diff
changeset
|
1132 || exit 1; \ |
9f608fba8b1c
build: ensure that a release build fails when a submodule is unclean
Jim Meyering <meyering@redhat.com>
parents:
11519
diff
changeset
|
1133 case $$diff in '') ;; \ |
9f608fba8b1c
build: ensure that a release build fails when a submodule is unclean
Jim Meyering <meyering@redhat.com>
parents:
11519
diff
changeset
|
1134 *) echo '$(ME): submodule files are locally modified:'; \ |
9f608fba8b1c
build: ensure that a release build fails when a submodule is unclean
Jim Meyering <meyering@redhat.com>
parents:
11519
diff
changeset
|
1135 echo "$$diff"; exit 1;; esac; \ |
9f608fba8b1c
build: ensure that a release build fails when a submodule is unclean
Jim Meyering <meyering@redhat.com>
parents:
11519
diff
changeset
|
1136 else \ |
9f608fba8b1c
build: ensure that a release build fails when a submodule is unclean
Jim Meyering <meyering@redhat.com>
parents:
11519
diff
changeset
|
1137 : ; \ |
9f608fba8b1c
build: ensure that a release build fails when a submodule is unclean
Jim Meyering <meyering@redhat.com>
parents:
11519
diff
changeset
|
1138 fi |
9f608fba8b1c
build: ensure that a release build fails when a submodule is unclean
Jim Meyering <meyering@redhat.com>
parents:
11519
diff
changeset
|
1139 |
14137
2dbfa19b11d0
maint.mk: add pre-release check to ensure submodule commits are public
Eric Blake <eblake@redhat.com>
parents:
14079
diff
changeset
|
1140 submodule-checks ?= no-submodule-changes public-submodule-commit |
2dbfa19b11d0
maint.mk: add pre-release check to ensure submodule commits are public
Eric Blake <eblake@redhat.com>
parents:
14079
diff
changeset
|
1141 |
2dbfa19b11d0
maint.mk: add pre-release check to ensure submodule commits are public
Eric Blake <eblake@redhat.com>
parents:
14079
diff
changeset
|
1142 # Ensure that each sub-module commit we're using is public. |
2dbfa19b11d0
maint.mk: add pre-release check to ensure submodule commits are public
Eric Blake <eblake@redhat.com>
parents:
14079
diff
changeset
|
1143 # Without this, it is too easy to tag and release code that |
2dbfa19b11d0
maint.mk: add pre-release check to ensure submodule commits are public
Eric Blake <eblake@redhat.com>
parents:
14079
diff
changeset
|
1144 # cannot be built from a fresh clone. |
2dbfa19b11d0
maint.mk: add pre-release check to ensure submodule commits are public
Eric Blake <eblake@redhat.com>
parents:
14079
diff
changeset
|
1145 .PHONY: public-submodule-commit |
2dbfa19b11d0
maint.mk: add pre-release check to ensure submodule commits are public
Eric Blake <eblake@redhat.com>
parents:
14079
diff
changeset
|
1146 public-submodule-commit: |
14190
6699cf2b13db
maint.mk: improve the public-submodule-commit rule
Jim Meyering <meyering@redhat.com>
parents:
14140
diff
changeset
|
1147 $(AM_V_GEN)if test -d $(srcdir)/.git; then \ |
6699cf2b13db
maint.mk: improve the public-submodule-commit rule
Jim Meyering <meyering@redhat.com>
parents:
14140
diff
changeset
|
1148 cd $(srcdir) && \ |
14210
59d4ae9d48d6
maintainer-makefile: work with older git for submodule check
Eric Blake <eblake@redhat.com>
parents:
14201
diff
changeset
|
1149 git submodule --quiet foreach test '$$(git rev-parse $$sha1)' \ |
59d4ae9d48d6
maintainer-makefile: work with older git for submodule check
Eric Blake <eblake@redhat.com>
parents:
14201
diff
changeset
|
1150 = '$$(git merge-base origin $$sha1)' \ |
14137
2dbfa19b11d0
maint.mk: add pre-release check to ensure submodule commits are public
Eric Blake <eblake@redhat.com>
parents:
14079
diff
changeset
|
1151 || { echo '$(ME): found non-public submodule commit' >&2; \ |
2dbfa19b11d0
maint.mk: add pre-release check to ensure submodule commits are public
Eric Blake <eblake@redhat.com>
parents:
14079
diff
changeset
|
1152 exit 1; }; \ |
2dbfa19b11d0
maint.mk: add pre-release check to ensure submodule commits are public
Eric Blake <eblake@redhat.com>
parents:
14079
diff
changeset
|
1153 else \ |
2dbfa19b11d0
maint.mk: add pre-release check to ensure submodule commits are public
Eric Blake <eblake@redhat.com>
parents:
14079
diff
changeset
|
1154 : ; \ |
2dbfa19b11d0
maint.mk: add pre-release check to ensure submodule commits are public
Eric Blake <eblake@redhat.com>
parents:
14079
diff
changeset
|
1155 fi |
14199
d6780f2a0ebc
maint.mk: make "check" depend on public-submodule-commit by default
Jim Meyering <meyering@redhat.com>
parents:
14190
diff
changeset
|
1156 # This rule has a high enough utility/cost ratio that it should be a |
d6780f2a0ebc
maint.mk: make "check" depend on public-submodule-commit by default
Jim Meyering <meyering@redhat.com>
parents:
14190
diff
changeset
|
1157 # dependent of "check" by default. However, some of us do occasionally |
d6780f2a0ebc
maint.mk: make "check" depend on public-submodule-commit by default
Jim Meyering <meyering@redhat.com>
parents:
14190
diff
changeset
|
1158 # commit a temporary change that deliberately points to a non-public |
d6780f2a0ebc
maint.mk: make "check" depend on public-submodule-commit by default
Jim Meyering <meyering@redhat.com>
parents:
14190
diff
changeset
|
1159 # submodule commit, and want to be able to use rules like "make check". |
14201
0efdc4f3700e
maint.mk: adjust variable name to be consistent with other gl_ vars
Jim Meyering <meyering@redhat.com>
parents:
14199
diff
changeset
|
1160 # In that case, run e.g., "make check gl_public_submodule_commit=" |
14199
d6780f2a0ebc
maint.mk: make "check" depend on public-submodule-commit by default
Jim Meyering <meyering@redhat.com>
parents:
14190
diff
changeset
|
1161 # to disable this test. |
14201
0efdc4f3700e
maint.mk: adjust variable name to be consistent with other gl_ vars
Jim Meyering <meyering@redhat.com>
parents:
14199
diff
changeset
|
1162 gl_public_submodule_commit ?= public-submodule-commit |
0efdc4f3700e
maint.mk: adjust variable name to be consistent with other gl_ vars
Jim Meyering <meyering@redhat.com>
parents:
14199
diff
changeset
|
1163 check: $(gl_public_submodule_commit) |
14137
2dbfa19b11d0
maint.mk: add pre-release check to ensure submodule commits are public
Eric Blake <eblake@redhat.com>
parents:
14079
diff
changeset
|
1164 |
12083
b7b81ca175f4
maint.mk: requiring "make major" was annoying, for a "minor" release.
Jim Meyering <meyering@redhat.com>
parents:
12035
diff
changeset
|
1165 .PHONY: alpha beta stable |
b7b81ca175f4
maint.mk: requiring "make major" was annoying, for a "minor" release.
Jim Meyering <meyering@redhat.com>
parents:
12035
diff
changeset
|
1166 ALL_RECURSIVE_TARGETS += alpha beta stable |
14137
2dbfa19b11d0
maint.mk: add pre-release check to ensure submodule commits are public
Eric Blake <eblake@redhat.com>
parents:
14079
diff
changeset
|
1167 alpha beta stable: $(local-check) writable-files $(submodule-checks) |
12083
b7b81ca175f4
maint.mk: requiring "make major" was annoying, for a "minor" release.
Jim Meyering <meyering@redhat.com>
parents:
12035
diff
changeset
|
1168 test $@ = stable \ |
b7b81ca175f4
maint.mk: requiring "make major" was annoying, for a "minor" release.
Jim Meyering <meyering@redhat.com>
parents:
12035
diff
changeset
|
1169 && { echo $(VERSION) | grep -E '^[0-9]+(\.[0-9]+)+$$' \ |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
1170 || { echo "invalid version string: $(VERSION)" 1>&2; exit 1;};}\ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
1171 || : |
11909
2cd9af4a1b6a
maint.mk: remove more coreutils-specific targets; XZ_OPT=-9ev
Jim Meyering <meyering@redhat.com>
parents:
11853
diff
changeset
|
1172 $(MAKE) vc-diff-check |
12384
8620ff59758a
maint.mk: news-check: use grep -E
Jim Meyering <meyering@redhat.com>
parents:
12383
diff
changeset
|
1173 $(MAKE) news-check |
11909
2cd9af4a1b6a
maint.mk: remove more coreutils-specific targets; XZ_OPT=-9ev
Jim Meyering <meyering@redhat.com>
parents:
11853
diff
changeset
|
1174 $(MAKE) distcheck |
2cd9af4a1b6a
maint.mk: remove more coreutils-specific targets; XZ_OPT=-9ev
Jim Meyering <meyering@redhat.com>
parents:
11853
diff
changeset
|
1175 $(MAKE) dist XZ_OPT=-9ev |
12443
3a97f22a64eb
maint.mk: allow a project to override release-prep commands
Alfred M. Szmidt <ams@gnu.org>
parents:
12441
diff
changeset
|
1176 $(MAKE) $(release-prep-hook) RELEASE_TYPE=$@ |
3a97f22a64eb
maint.mk: allow a project to override release-prep commands
Alfred M. Szmidt <ams@gnu.org>
parents:
12441
diff
changeset
|
1177 $(MAKE) -s emit_upload_commands RELEASE_TYPE=$@ |
3a97f22a64eb
maint.mk: allow a project to override release-prep commands
Alfred M. Szmidt <ams@gnu.org>
parents:
12441
diff
changeset
|
1178 |
3a97f22a64eb
maint.mk: allow a project to override release-prep commands
Alfred M. Szmidt <ams@gnu.org>
parents:
12441
diff
changeset
|
1179 # Override this in cfg.mk if you follow different procedures. |
3a97f22a64eb
maint.mk: allow a project to override release-prep commands
Alfred M. Szmidt <ams@gnu.org>
parents:
12441
diff
changeset
|
1180 release-prep-hook ?= release-prep |
3a97f22a64eb
maint.mk: allow a project to override release-prep commands
Alfred M. Szmidt <ams@gnu.org>
parents:
12441
diff
changeset
|
1181 |
13871
8c865631b168
maint.mk: rename variable: s/noteworthy/gl_noteworthy_news_/
Jim Meyering <meyering@redhat.com>
parents:
13865
diff
changeset
|
1182 gl_noteworthy_news_ = * Noteworthy changes in release ?.? (????-??-??) [?] |
12443
3a97f22a64eb
maint.mk: allow a project to override release-prep commands
Alfred M. Szmidt <ams@gnu.org>
parents:
12441
diff
changeset
|
1183 .PHONY: release-prep |
3a97f22a64eb
maint.mk: allow a project to override release-prep commands
Alfred M. Szmidt <ams@gnu.org>
parents:
12441
diff
changeset
|
1184 release-prep: |
3a97f22a64eb
maint.mk: allow a project to override release-prep commands
Alfred M. Szmidt <ams@gnu.org>
parents:
12441
diff
changeset
|
1185 case $$RELEASE_TYPE in alpha|beta|stable) ;; \ |
3a97f22a64eb
maint.mk: allow a project to override release-prep commands
Alfred M. Szmidt <ams@gnu.org>
parents:
12441
diff
changeset
|
1186 *) echo "invalid RELEASE_TYPE: $$RELEASE_TYPE" 1>&2; exit 1;; esac |
12538
a8059f620e8f
maint.mk: create announcement template in ~/, not in /tmp
Jim Meyering <meyering@redhat.com>
parents:
12443
diff
changeset
|
1187 $(MAKE) -s announcement > ~/announce-$(my_distdir) |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
1188 if test -d $(release_archive_dir); then \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
1189 ln $(rel-files) $(release_archive_dir); \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
1190 chmod a-w $(rel-files); \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
1191 fi |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
1192 echo $(VERSION) > $(prev_version_file) |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
1193 $(MAKE) update-NEWS-hash |
13871
8c865631b168
maint.mk: rename variable: s/noteworthy/gl_noteworthy_news_/
Jim Meyering <meyering@redhat.com>
parents:
13865
diff
changeset
|
1194 perl -pi -e '$$. == 3 and print "$(gl_noteworthy_news_)\n\n\n"' NEWS |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
1195 $(emit-commit-log) > .ci-msg |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
1196 $(VC) commit -F .ci-msg -a |
12538
a8059f620e8f
maint.mk: create announcement template in ~/, not in /tmp
Jim Meyering <meyering@redhat.com>
parents:
12443
diff
changeset
|
1197 rm .ci-msg |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
1198 |
13309
eeb72cb4f562
maint.mk: allow to pass options to gendocs.sh
Jim Meyering <meyering@redhat.com>
parents:
13306
diff
changeset
|
1199 # Override this with e.g., -s $(srcdir)/some_other_name.texi |
eeb72cb4f562
maint.mk: allow to pass options to gendocs.sh
Jim Meyering <meyering@redhat.com>
parents:
13306
diff
changeset
|
1200 # if the default $(PACKAGE)-derived name doesn't apply. |
eeb72cb4f562
maint.mk: allow to pass options to gendocs.sh
Jim Meyering <meyering@redhat.com>
parents:
13306
diff
changeset
|
1201 gendocs_options_ ?= |
eeb72cb4f562
maint.mk: allow to pass options to gendocs.sh
Jim Meyering <meyering@redhat.com>
parents:
13306
diff
changeset
|
1202 |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
1203 .PHONY: web-manual |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
1204 web-manual: |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
1205 @test -z "$(manual_title)" \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
1206 && { echo define manual_title in cfg.mk 1>&2; exit 1; } || : |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
1207 @cd '$(srcdir)/doc'; \ |
13309
eeb72cb4f562
maint.mk: allow to pass options to gendocs.sh
Jim Meyering <meyering@redhat.com>
parents:
13306
diff
changeset
|
1208 $(SHELL) ../build-aux/gendocs.sh $(gendocs_options_) \ |
eeb72cb4f562
maint.mk: allow to pass options to gendocs.sh
Jim Meyering <meyering@redhat.com>
parents:
13306
diff
changeset
|
1209 -o '$(abs_builddir)/doc/manual' \ |
11495
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
1210 --email $(PACKAGE_BUGREPORT) $(PACKAGE) \ |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
1211 "$(PACKAGE_NAME) - $(manual_title)" |
d88c5ec9288d
Sync maint.mk with coreutils.
Simon Josefsson <simon@josefsson.org>
parents:
11484
diff
changeset
|
1212 @echo " *** Upload the doc/manual directory to web-cvs." |
9811
5f79eb63c21e
Alter GNUmakefile to install into top directory.
Eric Blake <ebb9@byu.net>
parents:
diff
changeset
|
1213 |
10738
af888e12d10d
top/maint.mk: Add coverage rules, inspired by scripts in gnupdf.
Simon Josefsson <simon@josefsson.org>
parents:
10368
diff
changeset
|
1214 # Code Coverage |
af888e12d10d
top/maint.mk: Add coverage rules, inspired by scripts in gnupdf.
Simon Josefsson <simon@josefsson.org>
parents:
10368
diff
changeset
|
1215 |
af888e12d10d
top/maint.mk: Add coverage rules, inspired by scripts in gnupdf.
Simon Josefsson <simon@josefsson.org>
parents:
10368
diff
changeset
|
1216 init-coverage: |
11406
ae9bcb808cfd
maint.mk: really add $(AM_MAKEFLAGS)
Ralf Wildenhues <Ralf.Wildenhues@gmx.de>
parents:
11405
diff
changeset
|
1217 $(MAKE) $(AM_MAKEFLAGS) clean |
10738
af888e12d10d
top/maint.mk: Add coverage rules, inspired by scripts in gnupdf.
Simon Josefsson <simon@josefsson.org>
parents:
10368
diff
changeset
|
1218 lcov --directory . --zerocounters |
af888e12d10d
top/maint.mk: Add coverage rules, inspired by scripts in gnupdf.
Simon Josefsson <simon@josefsson.org>
parents:
10368
diff
changeset
|
1219 |
10757
0affc3e42aa9
maint.mk: Use --coverage.
Simon Josefsson <simon@josefsson.org>
parents:
10738
diff
changeset
|
1220 COVERAGE_CCOPTS ?= "-g --coverage" |
10738
af888e12d10d
top/maint.mk: Add coverage rules, inspired by scripts in gnupdf.
Simon Josefsson <simon@josefsson.org>
parents:
10368
diff
changeset
|
1221 COVERAGE_OUT ?= doc/coverage |
af888e12d10d
top/maint.mk: Add coverage rules, inspired by scripts in gnupdf.
Simon Josefsson <simon@josefsson.org>
parents:
10368
diff
changeset
|
1222 |
af888e12d10d
top/maint.mk: Add coverage rules, inspired by scripts in gnupdf.
Simon Josefsson <simon@josefsson.org>
parents:
10368
diff
changeset
|
1223 build-coverage: |
11406
ae9bcb808cfd
maint.mk: really add $(AM_MAKEFLAGS)
Ralf Wildenhues <Ralf.Wildenhues@gmx.de>
parents:
11405
diff
changeset
|
1224 $(MAKE) $(AM_MAKEFLAGS) CFLAGS=$(COVERAGE_CCOPTS) CXXFLAGS=$(COVERAGE_CCOPTS) |
ae9bcb808cfd
maint.mk: really add $(AM_MAKEFLAGS)
Ralf Wildenhues <Ralf.Wildenhues@gmx.de>
parents:
11405
diff
changeset
|
1225 $(MAKE) $(AM_MAKEFLAGS) CFLAGS=$(COVERAGE_CCOPTS) CXXFLAGS=$(COVERAGE_CCOPTS) check |
10738
af888e12d10d
top/maint.mk: Add coverage rules, inspired by scripts in gnupdf.
Simon Josefsson <simon@josefsson.org>
parents:
10368
diff
changeset
|
1226 mkdir -p $(COVERAGE_OUT) |
af888e12d10d
top/maint.mk: Add coverage rules, inspired by scripts in gnupdf.
Simon Josefsson <simon@josefsson.org>
parents:
10368
diff
changeset
|
1227 lcov --directory . --output-file $(COVERAGE_OUT)/$(PACKAGE).info \ |
af888e12d10d
top/maint.mk: Add coverage rules, inspired by scripts in gnupdf.
Simon Josefsson <simon@josefsson.org>
parents:
10368
diff
changeset
|
1228 --capture |
af888e12d10d
top/maint.mk: Add coverage rules, inspired by scripts in gnupdf.
Simon Josefsson <simon@josefsson.org>
parents:
10368
diff
changeset
|
1229 |
af888e12d10d
top/maint.mk: Add coverage rules, inspired by scripts in gnupdf.
Simon Josefsson <simon@josefsson.org>
parents:
10368
diff
changeset
|
1230 gen-coverage: |
af888e12d10d
top/maint.mk: Add coverage rules, inspired by scripts in gnupdf.
Simon Josefsson <simon@josefsson.org>
parents:
10368
diff
changeset
|
1231 genhtml --output-directory $(COVERAGE_OUT) \ |
af888e12d10d
top/maint.mk: Add coverage rules, inspired by scripts in gnupdf.
Simon Josefsson <simon@josefsson.org>
parents:
10368
diff
changeset
|
1232 $(COVERAGE_OUT)/$(PACKAGE).info \ |
af888e12d10d
top/maint.mk: Add coverage rules, inspired by scripts in gnupdf.
Simon Josefsson <simon@josefsson.org>
parents:
10368
diff
changeset
|
1233 --highlight --frames --legend \ |
af888e12d10d
top/maint.mk: Add coverage rules, inspired by scripts in gnupdf.
Simon Josefsson <simon@josefsson.org>
parents:
10368
diff
changeset
|
1234 --title "$(PACKAGE_NAME)" |
af888e12d10d
top/maint.mk: Add coverage rules, inspired by scripts in gnupdf.
Simon Josefsson <simon@josefsson.org>
parents:
10368
diff
changeset
|
1235 |
af888e12d10d
top/maint.mk: Add coverage rules, inspired by scripts in gnupdf.
Simon Josefsson <simon@josefsson.org>
parents:
10368
diff
changeset
|
1236 coverage: init-coverage build-coverage gen-coverage |
af888e12d10d
top/maint.mk: Add coverage rules, inspired by scripts in gnupdf.
Simon Josefsson <simon@josefsson.org>
parents:
10368
diff
changeset
|
1237 |
9811
5f79eb63c21e
Alter GNUmakefile to install into top directory.
Eric Blake <ebb9@byu.net>
parents:
diff
changeset
|
1238 # Update gettext files. |
5f79eb63c21e
Alter GNUmakefile to install into top directory.
Eric Blake <ebb9@byu.net>
parents:
diff
changeset
|
1239 PACKAGE ?= $(shell basename $(PWD)) |
11484
489151a3b578
maintainer-makefile: (PO_DOMAIN): New variable, allows overriding of gettext domain.
Simon Josefsson <simon@josefsson.org>
parents:
11468
diff
changeset
|
1240 PO_DOMAIN ?= $(PACKAGE) |
489151a3b578
maintainer-makefile: (PO_DOMAIN): New variable, allows overriding of gettext domain.
Simon Josefsson <simon@josefsson.org>
parents:
11468
diff
changeset
|
1241 POURL = http://translationproject.org/latest/$(PO_DOMAIN)/ |
9811
5f79eb63c21e
Alter GNUmakefile to install into top directory.
Eric Blake <ebb9@byu.net>
parents:
diff
changeset
|
1242 PODIR ?= po |
5f79eb63c21e
Alter GNUmakefile to install into top directory.
Eric Blake <ebb9@byu.net>
parents:
diff
changeset
|
1243 refresh-po: |
5f79eb63c21e
Alter GNUmakefile to install into top directory.
Eric Blake <ebb9@byu.net>
parents:
diff
changeset
|
1244 rm -f $(PODIR)/*.po && \ |
5f79eb63c21e
Alter GNUmakefile to install into top directory.
Eric Blake <ebb9@byu.net>
parents:
diff
changeset
|
1245 echo "$(ME): getting translations into po (please ignore the robots.txt ERROR 404)..." && \ |
5f79eb63c21e
Alter GNUmakefile to install into top directory.
Eric Blake <ebb9@byu.net>
parents:
diff
changeset
|
1246 wget --no-verbose --directory-prefix $(PODIR) --no-directories --recursive --level 1 --accept .po --accept .po.1 $(POURL) && \ |
5f79eb63c21e
Alter GNUmakefile to install into top directory.
Eric Blake <ebb9@byu.net>
parents:
diff
changeset
|
1247 echo 'en@boldquot' > $(PODIR)/LINGUAS && \ |
5f79eb63c21e
Alter GNUmakefile to install into top directory.
Eric Blake <ebb9@byu.net>
parents:
diff
changeset
|
1248 echo 'en@quot' >> $(PODIR)/LINGUAS && \ |
10828
45b8dcc40064
maintainer-makefile: Fix sed regexp.
Simon Josefsson <simon@josefsson.org>
parents:
10757
diff
changeset
|
1249 ls $(PODIR)/*.po | sed 's/\.po//' | sed 's,$(PODIR)/,,' | sort >> $(PODIR)/LINGUAS |
9811
5f79eb63c21e
Alter GNUmakefile to install into top directory.
Eric Blake <ebb9@byu.net>
parents:
diff
changeset
|
1250 |
13286
bacd8565eeaa
top/maint.mk (indent): Run twice to produce idempotent results.
Simon Josefsson <simon@josefsson.org>
parents:
13278
diff
changeset
|
1251 # Running indent once is not idempotent, but running it twice is. |
9811
5f79eb63c21e
Alter GNUmakefile to install into top directory.
Eric Blake <ebb9@byu.net>
parents:
diff
changeset
|
1252 INDENT_SOURCES ?= $(C_SOURCES) |
5f79eb63c21e
Alter GNUmakefile to install into top directory.
Eric Blake <ebb9@byu.net>
parents:
diff
changeset
|
1253 .PHONY: indent |
5f79eb63c21e
Alter GNUmakefile to install into top directory.
Eric Blake <ebb9@byu.net>
parents:
diff
changeset
|
1254 indent: |
5f79eb63c21e
Alter GNUmakefile to install into top directory.
Eric Blake <ebb9@byu.net>
parents:
diff
changeset
|
1255 indent $(INDENT_SOURCES) |
13286
bacd8565eeaa
top/maint.mk (indent): Run twice to produce idempotent results.
Simon Josefsson <simon@josefsson.org>
parents:
13278
diff
changeset
|
1256 indent $(INDENT_SOURCES) |
11776
9deb84e458cb
maint.mk: add a copyright-updating rule
Jim Meyering <meyering@redhat.com>
parents:
11750
diff
changeset
|
1257 |
11834
5c2855361aa1
maint.mk: support update-copyright-env
Joel E. Denny <jdenny@clemson.edu>
parents:
11788
diff
changeset
|
1258 # If you want to set UPDATE_COPYRIGHT_* environment variables, |
5c2855361aa1
maint.mk: support update-copyright-env
Joel E. Denny <jdenny@clemson.edu>
parents:
11788
diff
changeset
|
1259 # put the assignments in this variable. |
5c2855361aa1
maint.mk: support update-copyright-env
Joel E. Denny <jdenny@clemson.edu>
parents:
11788
diff
changeset
|
1260 update-copyright-env ?= |
5c2855361aa1
maint.mk: support update-copyright-env
Joel E. Denny <jdenny@clemson.edu>
parents:
11788
diff
changeset
|
1261 |
11776
9deb84e458cb
maint.mk: add a copyright-updating rule
Jim Meyering <meyering@redhat.com>
parents:
11750
diff
changeset
|
1262 # Run this rule once per year (usually early in January) |
9deb84e458cb
maint.mk: add a copyright-updating rule
Jim Meyering <meyering@redhat.com>
parents:
11750
diff
changeset
|
1263 # to update all FSF copyright year lists in your project. |
11836
4103ca3d0881
maint.mk: simplify update-copyright rule
Eric Blake <ebb9@byu.net>
parents:
11834
diff
changeset
|
1264 # If you have an additional project-specific rule, |
4103ca3d0881
maint.mk: simplify update-copyright rule
Eric Blake <ebb9@byu.net>
parents:
11834
diff
changeset
|
1265 # add it in cfg.mk along with a line 'update-copyright: prereq'. |
4103ca3d0881
maint.mk: simplify update-copyright rule
Eric Blake <ebb9@byu.net>
parents:
11834
diff
changeset
|
1266 # By default, exclude all variants of COPYING; you can also |
4103ca3d0881
maint.mk: simplify update-copyright rule
Eric Blake <ebb9@byu.net>
parents:
11834
diff
changeset
|
1267 # add exemptions (such as ChangeLog..* for rotated change logs) |
4103ca3d0881
maint.mk: simplify update-copyright rule
Eric Blake <ebb9@byu.net>
parents:
11834
diff
changeset
|
1268 # in the file .x-update-copyright. |
11776
9deb84e458cb
maint.mk: add a copyright-updating rule
Jim Meyering <meyering@redhat.com>
parents:
11750
diff
changeset
|
1269 .PHONY: update-copyright |
11836
4103ca3d0881
maint.mk: simplify update-copyright rule
Eric Blake <ebb9@byu.net>
parents:
11834
diff
changeset
|
1270 update-copyright: |
11847
5dd8e8cf05db
maint.mk: give full control over update-copyright exclusions
Joel E. Denny <jdenny@clemson.edu>
parents:
11836
diff
changeset
|
1271 grep -l -w Copyright \ |
5dd8e8cf05db
maint.mk: give full control over update-copyright exclusions
Joel E. Denny <jdenny@clemson.edu>
parents:
11836
diff
changeset
|
1272 $$(export VC_LIST_EXCEPT_DEFAULT=COPYING && $(VC_LIST_EXCEPT)) \ |
11834
5c2855361aa1
maint.mk: support update-copyright-env
Joel E. Denny <jdenny@clemson.edu>
parents:
11788
diff
changeset
|
1273 | $(update-copyright-env) xargs $(build_aux)/$@ |