Remove BR2_DEPRECATED
authorArnout Vandecappelle <arnout@mind.be>
Sat, 15 Oct 2016 14:51:07 +0000 (16:51 +0200)
committerPeter Korsgaard <peter@korsgaard.com>
Sat, 15 Oct 2016 21:14:45 +0000 (23:14 +0200)
The BR2_DEPRECATED logic is a lot less useful than the legacy handling,
because the symbols just disappears without warning to the user. For
example, we had a few defconfigs that were using deprecated symbols
(which were not actually used because BR2_DEPRECATED wasn't set) so
these didn't build the expected code anymore.

Also, the idea behind BR2_DEPRECATED is that you can easily revive it
again if there is interest. However, it is relatively easy to revert
the removal of a package as well.

The deprecation is also more effort because it has to be removed twice:
once when deprecating, and once when really removing.

It doesn't make sense to add a legacy entry for BR2_DEPRECATED. Users
who actually used it will get legacy warnings instead.

Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
Signed-off-by: Peter Korsgaard <peter@korsgaard.com>
Config.in
Makefile
Makefile.legacy

index 8d37c93806d6589265f2fc59e4e5ed71c94105dc..d9fc40027647f97b0eb2eb61ac840b41736f75ab 100644 (file)
--- a/Config.in
+++ b/Config.in
@@ -361,36 +361,6 @@ config BR2_CCACHE_USE_BASEDIR
 
 endif
 
-config BR2_DEPRECATED
-       bool "Show options and packages that are deprecated or obsolete"
-       help
-         This option shows outdated/obsolete versions of packages and
-         options that are otherwise hidden.
-
-if BR2_DEPRECATED
-
-config BR2_DEPRECATED_SINCE_2015_11
-       bool
-       default y
-
-config BR2_DEPRECATED_SINCE_2016_02
-       bool
-       default y
-
-config BR2_DEPRECATED_SINCE_2016_05
-       bool
-       default y
-
-config BR2_DEPRECATED_SINCE_2016_08
-       bool
-       default y
-
-config BR2_DEPRECATED_SINCE_2016_11
-       bool
-       default y
-
-endif
-
 config BR2_ENABLE_DEBUG
        bool "build packages with debugging symbols"
        help
index 74b831eadf188e92252ebc4566f7c5d15acd29e9..411086186bff63edad869447667952858c986229 100644 (file)
--- a/Makefile
+++ b/Makefile
@@ -436,9 +436,7 @@ all: world
 
 # Include legacy before the other things, because package .mk files
 # may rely on it.
-ifneq ($(BR2_DEPRECATED),y)
 include Makefile.legacy
-endif
 
 include package/Makefile.in
 include support/dependencies/dependencies.mk
index 764cfcd117bcacf08e9a14a3e9f28e215c9f0ee6..800c08926ca99bfaa569b06a7a10d52349dc2186 100644 (file)
@@ -4,9 +4,6 @@
 # This file contains placeholders to detect backward-compatibility problems.
 # When a buildroot "API" feature is being deprecated, a rule should be added
 # here that issues an error when the old feature is used.
-#
-# This file is not included if BR2_DEPRECATED is selected, so it is possible
-# to bypass the errors (although that's usually a bad idea).
 
 ifeq ($(BR2_LEGACY),y)
 $(error "You have legacy configuration in your .config! Please check your configuration.")