build: make crypto libraries optional

Message ID 20230825145701.694047-1-bruce.richardson@intel.com (mailing list archive)
State Accepted, archived
Delegated to: David Marchand
Headers
Series build: make crypto libraries optional |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/loongarch-compilation success Compilation OK
ci/loongarch-unit-testing success Unit Testing PASS
ci/github-robot: build success github build: passed
ci/iol-compile-amd64-testing success Testing PASS
ci/iol-unit-amd64-testing success Testing PASS
ci/iol-unit-arm64-testing success Testing PASS
ci/iol-sample-apps-testing success Testing PASS
ci/iol-mellanox-Performance success Performance Testing PASS
ci/iol-compile-arm64-testing success Testing PASS
ci/iol-broadcom-Performance success Performance Testing PASS
ci/iol-broadcom-Functional success Functional Testing PASS
ci/iol-intel-Performance success Performance Testing PASS
ci/iol-intel-Functional success Functional Testing PASS
ci/Intel-compilation warning apply issues

Commit Message

Bruce Richardson Aug. 25, 2023, 2:57 p.m. UTC
  Cryptodev and the libraries which depend on it can be made optional,
as they can be disabled without breaking the build.

Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
---
 lib/meson.build | 3 +++
 1 file changed, 3 insertions(+)
  

Comments

Morten Brørup Aug. 25, 2023, 4:24 p.m. UTC | #1
> From: Bruce Richardson [mailto:bruce.richardson@intel.com]
> Sent: Friday, 25 August 2023 16.57
> 
> Cryptodev and the libraries which depend on it can be made optional,
> as they can be disabled without breaking the build.
> 
> Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
> ---

Thank you, Bruce. You know I like it! :-)

Acked-by: Morten Brørup <mb@smartsharesystems.com>
  
David Marchand Aug. 28, 2023, 9:31 a.m. UTC | #2
On Fri, Aug 25, 2023 at 5:10 PM Bruce Richardson
<bruce.richardson@intel.com> wrote:
>
> Cryptodev and the libraries which depend on it can be made optional,
> as they can be disabled without breaking the build.
>
> Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>

Acked-by: David Marchand <david.marchand@redhat.com>
  
David Marchand Aug. 29, 2023, 8:04 a.m. UTC | #3
On Fri, Aug 25, 2023 at 5:10 PM Bruce Richardson
<bruce.richardson@intel.com> wrote:
>
> Cryptodev and the libraries which depend on it can be made optional,
> as they can be disabled without breaking the build.
>
> Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>

Applied, thanks.
  

Patch

diff --git a/lib/meson.build b/lib/meson.build
index c1602e818a..099b0ed18a 100644
--- a/lib/meson.build
+++ b/lib/meson.build
@@ -80,6 +80,7 @@  optional_libs = [
         'bpf',
         'cfgfile',
         'compressdev',
+        'cryptodev',
         'distributor',
         'dmadev',
         'efd',
@@ -89,6 +90,7 @@  optional_libs = [
         'gro',
         'gso',
         'ip_frag',
+        'ipsec',
         'jobstats',
         'latencystats',
         'member',
@@ -105,6 +107,7 @@  optional_libs = [
         'regexdev',
         'reorder',
         'sched',
+        'security',
         'table',
         'vhost',
 ]