public inbox for dev@dpdk.org
 help / color / mirror / Atom feed
From: Bruce Richardson <bruce.richardson@intel.com>
To: Patrick Robb <probb@iol.unh.edu>
Cc: David Marchand <david.marchand@redhat.com>, <dev@dpdk.org>,
	<ci@dpdk.org>,  Thomas Monjalon <thomas@monjalon.net>,
	Aaron Conole <aconole@redhat.com>,
	 Luca Boccassi <luca.boccassi@gmail.com>,
	Timothy Redaelli <tredaelli@redhat.com>,
	Matthew Labrecque <mlabrecque@iol.unh.edu>
Subject: Re: [PATCH] build: bump meson version to remove warnings
Date: Fri, 23 Jan 2026 16:11:33 +0000	[thread overview]
Message-ID: <aXOdtd5SjD08kkBv@bricha3-mobl1.ger.corp.intel.com> (raw)
In-Reply-To: <CAJvnSUBOU7QTb6kp2RypHnk7rF5QjSjk5G8WtsEEgT23VmW-vw@mail.gmail.com>

On Fri, Jan 23, 2026 at 10:59:23AM -0500, Patrick Robb wrote:
>    On Fri, Jan 23, 2026 at 10:43 AM Bruce Richardson
>    <[1]bruce.richardson@intel.com> wrote:
> 
>      On Fri, Jan 23, 2026 at 10:35:53AM -0500, Patrick Robb wrote:
>      >    On Fri, Jan 23, 2026 at 4:02 AM Bruce Richardson
>      >    <[1][2]bruce.richardson@intel.com> wrote:
>      >
>      >      On Thu, Jan 22, 2026 at 01:59:19PM -0500, Patrick Robb wrote:
>      >      >    Thanks David for the CC. We need to rebuild all of our
>      DPDK
>      >      >    container images for the distro build testing etc. which
>      takes
>      >      some
>      >      >    time. We should be ready early next week.
>      >      >
>      >      Is rebuilding all container images something that needs to be
>      done
>      >      every
>      >      time we do a minimum meson version bump? If so, we probably
>      need to
>      >      look to
>      >      take steps to make things easier to do.
>      >
>      >    Right now we do build new images any time any DPDK dependencies
>      are
>      >    updated, including meson. That's done with the dpdk-ci template
>      engine
>      >    (an application that templates out Dockerfiles for DPDK CI
>      >    testing):
>      [2][3]https://git.dpdk.org/tools/dpdk-ci/tree/containers
>      >    However, to simplify we can just move to installing meson at
>      runtime in
>      >    all cases. We already have to do this sometimes (like when
>      testing with
>      >    an LTS that requires a different meson) so doing it in all
>      cases
>      >    wouldn't be too disruptive. And, of course the main point is
>      that that
>      >    would allow us to accomodate a version bump like this  without
>      >    rebuilding the CI images. Let me know if you have an opinion.
>      In any
>      >    case I will chat with the CI Lab students about it.
>      >
>      Having it installed at runtime would be definitely good.
>      The other thing I was considering is whether we actually need to use
>      the
>      minimum meson version for all build testing. If one or two
>      containers have
>      the minimum version that should be fine, and others can use the
>      distro
>      supplied version so long as its >= minimum.
> 
>    Okay, both of the options above sound good. From my end if I had to
>    choose I would just do the first solution (install meson at runtime in
>    all cases) because it keeps our process uniform across distro testing
>    and there is no "special" container. But, the second option sounds good
>    too so please let me know what you think is best.
> 

Go with your runtime solution, it's fine - and possibly better.

  reply	other threads:[~2026-01-23 16:11 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-12-05 12:13 [PATCH] build: bump meson version to remove warnings Bruce Richardson
2025-12-05 12:28 ` Bruce Richardson
2026-01-20  8:31   ` David Marchand
2026-01-22 18:59     ` Patrick Robb
2026-01-23  9:02       ` Bruce Richardson
2026-01-23 15:35         ` Patrick Robb
2026-01-23 15:43           ` Bruce Richardson
2026-01-23 15:59             ` Patrick Robb
2026-01-23 16:11               ` Bruce Richardson [this message]
2026-02-03 20:55                 ` Patrick Robb
2026-01-28 11:19   ` Kevin Traynor
2026-01-28 11:37     ` Bruce Richardson

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=aXOdtd5SjD08kkBv@bricha3-mobl1.ger.corp.intel.com \
    --to=bruce.richardson@intel.com \
    --cc=aconole@redhat.com \
    --cc=ci@dpdk.org \
    --cc=david.marchand@redhat.com \
    --cc=dev@dpdk.org \
    --cc=luca.boccassi@gmail.com \
    --cc=mlabrecque@iol.unh.edu \
    --cc=probb@iol.unh.edu \
    --cc=thomas@monjalon.net \
    --cc=tredaelli@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox