13c79ad4a5
There are two legitimate cases to prefer ifdef over ifeq in package
recipes: command-line overrides are allowed for busybox and uclibc
configs.
Except for that, all package in tree already use ifeq, so warn the
developer adding/changing a package to use ifeq instead of ifdef, in
order to keep consistence across packages.
file.mk:2: use ifeq ($(SYMBOL),y) instead of ifdef SYMBOL
file.mk:5: use ifneq ($(SYMBOL),y) instead of ifndef SYMBOL
The difference between ifeq and ifdef is that ifdef doesn't expand
recursively.
Add comments to busybox and uclibc packages to avoid a warning in such
special cases.
Cc: Arnout Vandecappelle <arnout@mind.be>
Signed-off-by: Ricardo Martincoski <ricardo.martincoski@gmail.com>
Signed-off-by: Peter Korsgaard <peter@korsgaard.com>
(cherry picked from commit
|
||
|---|---|---|
| .. | ||
| 0001-networking-libiproute-use-linux-if_packet.h-instead-.patch | ||
| 0002-Makefile.flags-strip-non-l-arguments-returned-by-pkg.patch | ||
| 0003-awk-fix-use-after-free-CVE-2022-30065.patch | ||
| 0004-libbb-sockaddr2str-ensure-only-printable-characters-.patch | ||
| 0005-nslookup-sanitize-all-printed-strings-with-printable.patch | ||
| Config.in | ||
| S01syslogd | ||
| S02klogd | ||
| S02sysctl | ||
| S10mdev | ||
| S15watchdog | ||
| S50telnet | ||
| busybox-minimal.config | ||
| busybox.config | ||
| busybox.hash | ||
| busybox.mk | ||
| inittab | ||
| mdev.conf | ||
| telnetd.service | ||
| udhcpc.script | ||