Re: [PATCH v3] doc: prefer installing using meson rather than ninja

[Bruce Richardson <bruce.richardson@intel.com>]

On Thu, Jun 22, 2023 at 08:35:17PM +0200, Thomas Monjalon wrote:
> 09/06/2023 16:51, Bruce Richardson:
> > After doing a build, to install DPDK system-wide our documentation
> > recommended using the "ninja install" command. However, for anyone
> > building as a non-root user and only installing as root, the "meson
> > install" command is a better alternative, as it provides for
> > automatically dropping or elevating privileges as necessary in more
> > recent meson releases [1].
> > 
> > [1] https://mesonbuild.com/Installing.html#installing-as-the-superuser
> > 
> > Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
> > 
> > ---
> > V3:
> > * correct order of arguments to meson in CI scripts. The "-C" option
> >   must follow the meson "install" command. [This is consistent with
> >   other uses e.g. meson compile -C ..., meson test -C ...]
> > 
> > V2:
> > * Fix one missed reference to "ninja install" in Linux GSG
> > * Changed CI scripts to use "meson install" to ensure step is properly
> >   tested.
> > ---
> >  .ci/linux-build.sh                           | 4 ++--
> >  doc/guides/contributing/coding_style.rst     | 2 +-
> >  doc/guides/cryptodevs/uadk.rst               | 2 +-
> >  doc/guides/freebsd_gsg/build_dpdk.rst        | 2 +-
> >  doc/guides/freebsd_gsg/build_sample_apps.rst | 2 +-
> >  doc/guides/linux_gsg/build_dpdk.rst          | 4 ++--
> >  doc/guides/prog_guide/build-sdk-meson.rst    | 4 ++--
> >  7 files changed, 10 insertions(+), 10 deletions(-)
> 
> I see other occurences which could be replaced:
> 
> .ci/linux-build.sh:    [ -d install ] || DESTDIR=$(pwd)/install ninja -C build install

Yes, this could be replaced, I somehow missed it.

> devtools/test-meson-builds.sh:  echo "DESTDIR=$2 $ninja_cmd -C $1 install" >&$verbose
> devtools/test-meson-builds.sh:  DESTDIR=$2 $ninja_cmd -C $1 install >&$veryverbose

While these could be replaced, the use of ninja is not a problem here, as
the whole script runs with user permissions. I'll consider replacing them
in a new version, though.

> doc/guides/nics/mlx4.rst:    ninja install
> doc/guides/platform/mlx5.rst:    ninja install

I looked at these before when doing the patch. They need to stay as they
are. They are part of instructions for installing the mlx driver
dependencies, which use cmake, not meson.

/Bruce