build: use integer for new max_ethports option too

Message ID 20190301154651.16742-1-luca.boccassi@gmail.com
State Accepted, archived
Delegated to: Thomas Monjalon
Headers show
Series
  • build: use integer for new max_ethports option too
Related show

Checks

Context Check Description
ci/mellanox-Performance-Testing success Performance Testing PASS
ci/intel-Performance-Testing success Performance Testing PASS
ci/Intel-compilation success Compilation OK
ci/checkpatch warning coding style issues

Commit Message

Luca Boccassi March 1, 2019, 3:46 p.m.
From: Luca Boccassi <bluca@debian.org>

max_ethports was merged after the other patch was written:

e04ea7fcf03c ("build: use integers for numerical options")

So convert this one too like the others have already been.

Signed-off-by: Luca Boccassi <bluca@debian.org>
---
 meson_options.txt | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Thomas Monjalon March 26, 2019, 10:10 p.m. | #1
01/03/2019 16:46, luca.boccassi@gmail.com:
> From: Luca Boccassi <bluca@debian.org>
> 
> max_ethports was merged after the other patch was written:
> 
> e04ea7fcf03c ("build: use integers for numerical options")
> 
> So convert this one too like the others have already been.
> 
> Signed-off-by: Luca Boccassi <bluca@debian.org>

Applied, thanks

Patch

diff --git a/meson_options.txt b/meson_options.txt
index dc7346fe0..16d9f92c6 100644
--- a/meson_options.txt
+++ b/meson_options.txt
@@ -20,7 +20,7 @@  option('lib_musdk_dir', type: 'string', value: '',
 	description: 'path to the MUSDK library installation directory')
 option('machine', type: 'string', value: 'native',
 	description: 'set the target machine type')
-option('max_ethports', type: 'string', value: '32',
+option('max_ethports', type: 'integer', value: 32,
 	description: 'maximum number of Ethernet devices')
 option('max_lcores', type: 'integer', value: 128,
 	description: 'maximum number of cores/threads supported by EAL')