]> rtime.felk.cvut.cz Git - coffee/buildroot.git/commit
Config.in files: use if/endif instead of 'depends on' for main symbol
authorThomas De Schampheleire <patrickdepinguin@gmail.com>
Fri, 20 Dec 2013 21:31:30 +0000 (22:31 +0100)
committerThomas Petazzoni <thomas.petazzoni@free-electrons.com>
Wed, 25 Dec 2013 11:21:39 +0000 (12:21 +0100)
commit35eaed8d07bdc73a83990bc306fcdb08b2e70eaf
treefe38a982159d30eeadbc0a30462da1676bdd11b0
parent142a81d60533ef179300c44fd29aaefbfae62d3a
Config.in files: use if/endif instead of 'depends on' for main symbol

In the Config.in file of package foo, it often happens that there are other
symbols besides BR2_PACKAGE_FOO. Typically, these symbols only make sense
when foo itself is enabled. There are two ways to express this: with
    depends on BR2_PACKAGE_FOO
in each extra symbol, or with
    if BR2_PACKAGE_FOO
        ...
    endif
around the entire set of extra symbols.

The if/endif approach avoids the repetition of 'depends on' statements on
multiple symbols, so this is clearly preferred. But even when there is only
one extra symbol, if/endif is a more logical choice:
- it is future-proof for when extra symbols are added
- it allows to have just one strategy instead of two (less confusion)

This patch modifies the Config.in files accordingly.

Signed-off-by: Thomas De Schampheleire <thomas.de.schampheleire@gmail.com>
Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
60 files changed:
boot/syslinux/Config.in
package/binutils/Config.in
package/cppcms/Config.in
package/cvs/Config.in
package/dbus/Config.in
package/dhcp/Config.in
package/directfb-examples/Config.in
package/e2fsprogs/Config.in
package/efl/libedje/Config.in
package/evtest/Config.in
package/gadgetfs-test/Config.in
package/gettext/Config.in
package/gnutls/Config.in
package/googlefontdirectory/Config.in
package/gpsd/Config.in
package/haserl/Config.in
package/input-tools/Config.in
package/kexec/Config.in
package/kismet/Config.in
package/kmod/Config.in
package/lcdproc/Config.in
package/libarchive/Config.in
package/libcurl/Config.in
package/libdnet/Config.in
package/libidn/Config.in
package/libmicrohttpd/Config.in
package/libmpeg2/Config.in
package/libnfc/Config.in
package/libnspr/Config.in
package/lighttpd/Config.in
package/links/Config.in
package/live555/Config.in
package/madplay/Config.in
package/matchbox/Config.in
package/nano/Config.in
package/ncftp/Config.in
package/netsnmp/Config.in
package/nfs-utils/Config.in
package/openssl/Config.in
package/polarssl/Config.in
package/pppd/Config.in
package/proftpd/Config.in
package/pulseaudio/Config.in
package/qtuio/Config.in
package/rsh-redone/Config.in
package/schifra/Config.in
package/sdl_image/Config.in
package/sdl_sound/Config.in
package/sg3_utils/Config.in
package/speex/Config.in
package/taglib/Config.in
package/tcl/Config.in
package/tcpdump/Config.in
package/tiff/Config.in
package/tzdata/Config.in
package/udpcast/Config.in
package/usbutils/Config.in
package/vim/Config.in
package/wireless_tools/Config.in
package/zeromq/Config.in