diff mbox

Config.in.legacy: select BR2_LEGACY for module-init-tools legacy option

Message ID 1383851388-28586-1-git-send-email-thomas.petazzoni@free-electrons.com
State Accepted
Commit 0f401f914669b2d697f6efcfc0d2888a4c118f4d
Headers show

Commit Message

Thomas Petazzoni Nov. 7, 2013, 7:09 p.m. UTC
Commit 94c72087f3 ("module-init-tools: remove package") removed the
module-init-tools package and therefore added the corresponding config
option in Config.in.legacy. However, the commit forgot to add the
"select BR2_LEGACY" that ensures the build cannot proceed until the
user has acknowledged the change (i.e kmod replacing
module-init-tools).

This commit adds this missing "select BR2_LEGACY".

Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
---
 Config.in.legacy | 1 +
 1 file changed, 1 insertion(+)

Comments

Thomas De Schampheleire Nov. 7, 2013, 7:35 p.m. UTC | #1
On Thu, Nov 7, 2013 at 8:09 PM, Thomas Petazzoni
<thomas.petazzoni@free-electrons.com> wrote:
> Commit 94c72087f3 ("module-init-tools: remove package") removed the
> module-init-tools package and therefore added the corresponding config
> option in Config.in.legacy. However, the commit forgot to add the
> "select BR2_LEGACY" that ensures the build cannot proceed until the
> user has acknowledged the change (i.e kmod replacing
> module-init-tools).
>
> This commit adds this missing "select BR2_LEGACY".
>
> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>

Acked-by: Thomas De Schampheleire <thomas.de.schampheleire@gmail.com>
Arnout Vandecappelle Nov. 8, 2013, 12:27 a.m. UTC | #2
On 07/11/13 20:09, Thomas Petazzoni wrote:
> Commit 94c72087f3 ("module-init-tools: remove package") removed the
> module-init-tools package and therefore added the corresponding config
> option in Config.in.legacy. However, the commit forgot to add the
> "select BR2_LEGACY" that ensures the build cannot proceed until the
> user has acknowledged the change (i.e kmod replacing
> module-init-tools).
>
> This commit adds this missing "select BR2_LEGACY".
>
> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>

Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>

> ---
>   Config.in.legacy | 1 +
>   1 file changed, 1 insertion(+)
>
> diff --git a/Config.in.legacy b/Config.in.legacy
> index d02f0a6..587afe1 100644
> --- a/Config.in.legacy
> +++ b/Config.in.legacy
> @@ -123,6 +123,7 @@ config BR2_PACKAGE_MODULE_INIT_TOOLS
>   	bool "module-init-tools replaced by kmod"
>   	select BR2_PACKAGE_KMOD
>   	select BR2_PACKAGE_KMOD_TOOLS
> +	select BR2_LEGACY
>   	help
>   	  The 'module-init-tools' package has been removed, since it
>   	  has been depracated upstream and replaced by 'kmod'.
>
Peter Korsgaard Nov. 10, 2013, 11:35 a.m. UTC | #3
>>>>> "Thomas" == Thomas Petazzoni <thomas.petazzoni@free-electrons.com> writes:

 > Commit 94c72087f3 ("module-init-tools: remove package") removed the
 > module-init-tools package and therefore added the corresponding config
 > option in Config.in.legacy. However, the commit forgot to add the
 > "select BR2_LEGACY" that ensures the build cannot proceed until the
 > user has acknowledged the change (i.e kmod replacing
 > module-init-tools).

 > This commit adds this missing "select BR2_LEGACY".

 > Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>

Committed, thanks.
diff mbox

Patch

diff --git a/Config.in.legacy b/Config.in.legacy
index d02f0a6..587afe1 100644
--- a/Config.in.legacy
+++ b/Config.in.legacy
@@ -123,6 +123,7 @@  config BR2_PACKAGE_MODULE_INIT_TOOLS
 	bool "module-init-tools replaced by kmod"
 	select BR2_PACKAGE_KMOD
 	select BR2_PACKAGE_KMOD_TOOLS
+	select BR2_LEGACY
 	help
 	  The 'module-init-tools' package has been removed, since it
 	  has been depracated upstream and replaced by 'kmod'.