summaryrefslogtreecommitdiff
path: root/mk/check/check-headers.sh
blob: d8dbb500764139bc792f3c3d2f27f39a053a8df8 (plain)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
# $NetBSD: check-headers.sh,v 1.6 2006/11/09 14:41:18 rillig Exp $
#
# This program checks the header files for possible problems.
#
# When a macro definition contains the characters "${", it is likely
# that is comes from a GNU-style configure script that didn't use the
# ${prefix} or ${exec_prefix} variable correctly.
#

set -eu

. "${PKGSRCDIR}/mk/check/check-subr.sh"
cs_setprogname "$0"

found_unresolved_variable=no

# usage: check_header <fname>
check_header() {
	# See the end of the loop for the redirection.
	while read line; do

		# Check for "${" in macro definitions.
		case "$line" in
		"#define"*"\"\${"[a-z]*"}"*"\""*)
			found_unresolved_variable=yes
			cs_error_heading "Found unresolved variable in macro:"
			cs_error_msg "$fname: $line"
			;;
		esac

	done < "$1"
}

find * -type f -print 2>/dev/null \
| {
	while read fname; do

		skip=no
		eval "case \"\$fname\" in $SKIP_FILTER *.orig) skip=yes;; esac"
		[ $skip = no ] || continue

		case "$fname" in
		*.h | *.hpp | *.h++ | *.hxx)
			check_header "$fname"
			;;
		esac
	done

	if [ $found_unresolved_variable = yes ]; then
		cs_explain <<EOF
The above macros may contain references to shell variables.

The cause of this problem is usually that in a configure.ac or
configure.in file, there is some code like

    FOO_DIR="\${bindir}"
    AC_DEFINE_UNQUOTED(FOO_DIR, "\$FOO_DIR", [Directory where foo files go])

You can fix this by telling the original package author not to use
AC_DEFINE_UNQUOTED for directories. Instead, {he,she} should do
something like this:

    # in configure.ac:
    foodir="\${bindir}"
    AC_SUBST(FOO_DIR)

    # in the Makefile.am files (can be more than one):
    AM_CPPFLAGS= -DFOO_DIR=\\"@FOO_DIR@\\"

If this check is wrong and the package really wants to have "\${" in the
macros, append the above filenames to the CHECK_HEADERS_SKIP variable in
the package Makefile.
EOF

	fi
	cs_exit
}