diff mbox series

[1/1] package/easyframes: fix missing dependency

Message ID 20200907202756.1516192-1-horatiu.vultur@microchip.com
State Accepted
Headers show
Series [1/1] package/easyframes: fix missing dependency | expand

Commit Message

Horatiu Vultur Sept. 7, 2020, 8:27 p.m. UTC
Easyframes uses fork when capturing frames in a pcap file, therefore add
the dependency BR2_USE_MMU.

Signed-off-by: Horatiu Vultur <horatiu.vultur@microchip.com>
---
 package/easyframes/Config.in | 4 ++++
 1 file changed, 4 insertions(+)

Comments

Thomas Petazzoni Sept. 7, 2020, 8:50 p.m. UTC | #1
On Mon, 7 Sep 2020 22:27:55 +0200
Horatiu Vultur via buildroot <buildroot@busybox.net> wrote:

> Easyframes uses fork when capturing frames in a pcap file, therefore add
> the dependency BR2_USE_MMU.
> 
> Signed-off-by: Horatiu Vultur <horatiu.vultur@microchip.com>
> ---
>  package/easyframes/Config.in | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/package/easyframes/Config.in b/package/easyframes/Config.in
> index b2c45496b4..b2307fbf59 100644
> --- a/package/easyframes/Config.in
> +++ b/package/easyframes/Config.in
> @@ -1,5 +1,6 @@
>  config BR2_PACKAGE_EASYFRAMES
>  	bool "easyframes"
> +	depends on BR2_USE_MMU # fork()
>  	select BR2_PACKAGE_LIBPCAP
>  	help
>  	  This is a small and simple command-line tool for network
> @@ -8,3 +9,6 @@ config BR2_PACKAGE_EASYFRAMES
>  	  received.
>  
>  	  https://github.com/microchip-ung/easyframes
> +
> +comment "easyframe needs a toolchain w/ MMU"
> +	depends on !BR2_USE_MMU

We don't add Config.in comments for the MMU dependency. Indeed, you
cannot change a toolchain to have MMU support: it's a characteristic of
the CPU architecture. If you CPU architecture has no MMU, there's
nothing you can do about it.

So I've dropped the Config.in comment, and applied. Thanks!

Thomas
diff mbox series

Patch

diff --git a/package/easyframes/Config.in b/package/easyframes/Config.in
index b2c45496b4..b2307fbf59 100644
--- a/package/easyframes/Config.in
+++ b/package/easyframes/Config.in
@@ -1,5 +1,6 @@ 
 config BR2_PACKAGE_EASYFRAMES
 	bool "easyframes"
+	depends on BR2_USE_MMU # fork()
 	select BR2_PACKAGE_LIBPCAP
 	help
 	  This is a small and simple command-line tool for network
@@ -8,3 +9,6 @@  config BR2_PACKAGE_EASYFRAMES
 	  received.
 
 	  https://github.com/microchip-ung/easyframes
+
+comment "easyframe needs a toolchain w/ MMU"
+	depends on !BR2_USE_MMU