[PATCH] build: move enabled applications list declaration

[PATCH] build: move enabled applications list declaration

[David Marchand]

No functional impact foreseen. Simply move this variable and rename it
for consistency with other similar variables.

Signed-off-by: David Marchand <david.marchand@redhat.com>
---
 app/meson.build | 4 +---
 meson.build     | 3 ++-
 2 files changed, 3 insertions(+), 4 deletions(-)

diff --git a/app/meson.build b/app/meson.build
index 4fc1a83eba..0c2e7e4c00 100644
--- a/app/meson.build
+++ b/app/meson.build
@@ -38,8 +38,6 @@ if get_option('default_library') == 'static' and not is_windows
     default_ldflags += ['-Wl,--export-dynamic']
 endif
 
-enabled_apps = [] # used to print summary at the end
-
 foreach app:apps
     name = app
     build = true
@@ -90,7 +88,7 @@ foreach app:apps
         continue
     endif
 
-    enabled_apps += app
+    dpdk_apps_enabled += app
     link_libs = []
     if get_option('default_library') == 'static'
         link_libs = dpdk_static_libraries + dpdk_drivers
diff --git a/meson.build b/meson.build
index 70b54f0c98..2e6e546d20 100644
--- a/meson.build
+++ b/meson.build
@@ -44,6 +44,7 @@ dpdk_drivers = []
 dpdk_extra_ldflags = []
 dpdk_libs_deprecated = []
 dpdk_apps_disabled = []
+dpdk_apps_enabled = []
 dpdk_libs_disabled = []
 dpdk_libs_enabled = []
 dpdk_drvs_disabled = []
@@ -123,7 +124,7 @@ endif
 output_message = '\n=================\nApplications Enabled\n=================\n'
 output_message += '\napps:\n\t'
 output_count = 0
-foreach app:enabled_apps
+foreach app:dpdk_apps_enabled
     output_message += app + ', '
     output_count += 1
     if output_count == 8
-- 
2.41.0

Re: [PATCH] build: move enabled applications list declaration

[Bruce Richardson]

On Fri, Aug 11, 2023 at 03:28:05PM +0200, David Marchand wrote:
> No functional impact foreseen. Simply move this variable and rename it
> for consistency with other similar variables.
> 
> Signed-off-by: David Marchand <david.marchand@redhat.com>
> ---
Acked-by: Bruce Richardson <bruce.richardson@intel.com>

Re: [PATCH] build: move enabled applications list declaration

[Tyler Retzlaff]

On Fri, Aug 11, 2023 at 03:28:05PM +0200, David Marchand wrote:
> No functional impact foreseen. Simply move this variable and rename it
> for consistency with other similar variables.
> 
> Signed-off-by: David Marchand <david.marchand@redhat.com>
> ---
Acked-by: Tyler Retzlaff <roretzla@linux.microsoft.com>

Re: [PATCH] build: move enabled applications list declaration

[David Marchand]

On Fri, Aug 11, 2023 at 3:28 PM David Marchand
<david.marchand@redhat.com> wrote:
>
> No functional impact foreseen. Simply move this variable and rename it
> for consistency with other similar variables.
>
> Signed-off-by: David Marchand <david.marchand@redhat.com>

Acked-by: Bruce Richardson <bruce.richardson@intel.com>
Acked-by Tyler Retzlaff <roretzla@linux.microsoft.com>

Applied, thanks.


-- 
David Marchand