From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 53C8143A35; Thu, 1 Feb 2024 10:23:57 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 1F82140299; Thu, 1 Feb 2024 10:23:57 +0100 (CET) Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.129.124]) by mails.dpdk.org (Postfix) with ESMTP id 48AA740275 for ; Thu, 1 Feb 2024 10:23:55 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1706779434; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=uDoTrGWor0YKH0lZEOhElnEGG+VN8J6OlRC70eJqSuU=; b=TODWmP0owbW7a7jObRw/FI627zTpTkEkarWDy6YPHzkyJnCQM2vhq+CM0/pcw/16VkCNy4 jin42TJ9tRNd2ucZa1a2SrJg7/adXDRB7vFQfP/8WuxceKWmjJUqadC1H3h2v0sZQzkJBB FGiksEh5w/hAV9GFMiBs7VubRI+sdgw= Received: from mail-lj1-f200.google.com (mail-lj1-f200.google.com [209.85.208.200]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-413-1F8_aTfgOJ6UqiSwepyGaQ-1; Thu, 01 Feb 2024 04:23:53 -0500 X-MC-Unique: 1F8_aTfgOJ6UqiSwepyGaQ-1 Received: by mail-lj1-f200.google.com with SMTP id 38308e7fff4ca-2d07398a5c0so3500651fa.1 for ; Thu, 01 Feb 2024 01:23:53 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1706779432; x=1707384232; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=uDoTrGWor0YKH0lZEOhElnEGG+VN8J6OlRC70eJqSuU=; b=tbS2cGX99TQ44bjZmhfylrV11hHT1OBXsG5Vh7mRSDDDOIeiWoB7ZePVULybHDyBMQ eTkS4ImtvXWsKNNU0z0A0vFJzvagcaBiK7XVKuUiNsVl3UnP2vjwcxca16huDmpteUh8 GJ4M4B/vXg/LW3gxnXZbSl3kA9wzqkoAacZYXwXYaoh+qOupdg5gYY/XpF4y1EsZxyFO m/ok4Wlmw4/YyPkVnJS3Nm6vZRWZQ5Znlm2PfSJ38uT780ygl2sxsJo8nfHijeumiE5U QFFW+6mLD2kVbZwhyuwwiEsJsjIMV8bmLg52wbppKmcSQD7t1o/nno2pBzWNXd8b37UJ 9gDg== X-Gm-Message-State: AOJu0YwDrZUQsZLQrpCzzsqscYikSM+DzgBhB7tFfzsB7FiFExjhKQ62 CR7W4e1r2BqcWOdXfrOu+KAVuX/aAbwiTVEvskI3+W85hxbmhroNQFCqrh6JtFlDHMgQtkX89mq Q/l1jQxkGgW3cxfk/3dE2aFnGFP8GE8hOjl15npWBXzhhkOvEXttjXWH0fyJZCyy46EPnsHSiET sZseiVftdv6nvUO+M= X-Received: by 2002:a2e:6817:0:b0:2cd:cda6:55de with SMTP id c23-20020a2e6817000000b002cdcda655demr1343754lja.14.1706779432172; Thu, 01 Feb 2024 01:23:52 -0800 (PST) X-Google-Smtp-Source: AGHT+IG7YrGw2p17kBXa2aqNPhHDK/HQ0PrY9TcCR8hyZwX4oj68EGJmzhLhBHCc9U8d50VGtZbDF3WBK4L3/GnZWZQ= X-Received: by 2002:a2e:6817:0:b0:2cd:cda6:55de with SMTP id c23-20020a2e6817000000b002cdcda655demr1343747lja.14.1706779431881; Thu, 01 Feb 2024 01:23:51 -0800 (PST) MIME-Version: 1.0 References: <20231103162830.593702-1-bruce.richardson@intel.com> <20231220142152.492556-1-bruce.richardson@intel.com> In-Reply-To: <20231220142152.492556-1-bruce.richardson@intel.com> From: David Marchand Date: Thu, 1 Feb 2024 10:23:40 +0100 Message-ID: Subject: Re: [PATCH v3 0/3] Improve optional lib support To: Bruce Richardson Cc: dev@dpdk.org, mb@smartsharesystems.com X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org On Wed, Dec 20, 2023 at 3:22=E2=80=AFPM Bruce Richardson wrote: > > This patchset builds on the previous v1 and v2, and the discussion > which followed about what libraries should be optional. While still > a long way to go from the position suggested in the v2 discussion, > this set moves us a bit further along the road. > > The first patch is a direct v3 from previous versions, changing the > tracking from a list of optional libs to a list of mandatory ones. > The second patch then knocks off another 5 libs off the mandatory > list - all are libs which should have few dependencies in other > components, but which some common sample apps like l3fwd depend upon. > It should be possible to make other libs optional in future, but to > do so likely requires code-changes to #ifdef out support in places. > > To that end, patch 3 is an RFC to add support for "optional libs" to > the build system. For now, nothing uses this, but it may be useful if > someone does want to take on the task of removing a hard dependency > from one component on another and making it optional. > > Bruce Richardson (3): > build: track mandatory rather than optional libs > build: remove 5 libs from mandatory list > build: RFC - add support for optional dependencies I was not sure if I should take the third patch, but the two others make sense to me and I applied them. The third one can be reconsidered when the need arises. Thanks Bruce. --=20 David Marchand