Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/128464/?format=api
https://patches.dpdk.org/api/patches/128464/?format=api", "web_url": "https://patches.dpdk.org/project/dpdk/patch/20230609133406.80093-1-bruce.richardson@intel.com/", "project": { "id": 1, "url": "https://patches.dpdk.org/api/projects/1/?format=api", "name": "DPDK", "link_name": "dpdk", "list_id": "dev.dpdk.org", "list_email": "dev@dpdk.org", "web_url": "http://core.dpdk.org", "scm_url": "git://dpdk.org/dpdk", "webscm_url": "http://git.dpdk.org/dpdk", "list_archive_url": "https://inbox.dpdk.org/dev", "list_archive_url_format": "https://inbox.dpdk.org/dev/{}", "commit_url_format": "" }, "msgid": "<20230609133406.80093-1-bruce.richardson@intel.com>", "list_archive_url": "https://inbox.dpdk.org/dev/20230609133406.80093-1-bruce.richardson@intel.com", "date": "2023-06-09T13:34:06", "name": "[v2] doc: prefer installing using meson rather than ninja", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": true, "hash": "afe50954d3e722b14c9371b406d4e541ce994e11", "submitter": { "id": 20, "url": "https://patches.dpdk.org/api/people/20/?format=api", "name": "Bruce Richardson", "email": "bruce.richardson@intel.com" }, "delegate": { "id": 1, "url": "https://patches.dpdk.org/api/users/1/?format=api", "username": "tmonjalo", "first_name": "Thomas", "last_name": "Monjalon", "email": "thomas@monjalon.net" }, "mbox": "https://patches.dpdk.org/project/dpdk/patch/20230609133406.80093-1-bruce.richardson@intel.com/mbox/", "series": [ { "id": 28438, "url": "https://patches.dpdk.org/api/series/28438/?format=api", "web_url": "https://patches.dpdk.org/project/dpdk/list/?series=28438", "date": "2023-06-09T13:34:06", "name": "[v2] doc: prefer installing using meson rather than ninja", "version": 2, "mbox": "https://patches.dpdk.org/series/28438/mbox/" } ], "comments": "https://patches.dpdk.org/api/patches/128464/comments/", "check": "fail", "checks": "https://patches.dpdk.org/api/patches/128464/checks/", "tags": {}, "related": [], "headers": { "Return-Path": "<dev-bounces@dpdk.org>", "X-Original-To": "patchwork@inbox.dpdk.org", "Delivered-To": "patchwork@inbox.dpdk.org", "Received": [ "from mails.dpdk.org (mails.dpdk.org [217.70.189.124])\n\tby inbox.dpdk.org (Postfix) with ESMTP id 0CA6B42C6E;\n\tFri, 9 Jun 2023 15:34:34 +0200 (CEST)", "from mails.dpdk.org (localhost [127.0.0.1])\n\tby mails.dpdk.org (Postfix) with ESMTP id 984F240EDB;\n\tFri, 9 Jun 2023 15:34:33 +0200 (CEST)", "from mga12.intel.com (mga12.intel.com [192.55.52.136])\n by mails.dpdk.org (Postfix) with ESMTP id 4287240A84\n for <dev@dpdk.org>; Fri, 9 Jun 2023 15:34:32 +0200 (CEST)", "from fmsmga003.fm.intel.com ([10.253.24.29])\n by fmsmga106.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384;\n 09 Jun 2023 06:34:31 -0700", "from silpixa00401385.ir.intel.com ([10.237.214.152])\n by FMSMGA003.fm.intel.com with ESMTP; 09 Jun 2023 06:34:30 -0700" ], "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/simple;\n d=intel.com; i=@intel.com; q=dns/txt; s=Intel;\n t=1686317672; x=1717853672;\n h=from:to:cc:subject:date:message-id:in-reply-to:\n references:mime-version:content-transfer-encoding;\n bh=tsCjvpoghKjYMD5YnTFLqF+B02H1XtQ7scFTqwcOD3I=;\n b=OC6OX+oaOmzQBA2QlrrR41O6FNdioYjplFftzAdxO4wWIAJvZriK8dre\n jnA4v7DTjEvnpH7ynD6kb11chg0xE4GHO9mpXppuqd2kaVrJwJzobXCf7\n bDJPhowb9vojzu+vLbdvcvyXuUwOQ7KCu3aWSw3krhIS2XGoW5m7CVTSS\n 8GQt5NwnSgzDf+gF983yOJh+9sJNzeGnEDQZEPlfh9V9UF1Z1C+ZAtHbg\n 1WtrFwNOy9dxayqTeOu3jQLDQ/4nCXLyN60vTvLqsNsTFcUP+WXCmYTmK\n YH3WHevl49YiUDSOxnZCF4Hf1Rs77QsKyrzSAgMOOwG3UIjIE9i4WKeXN g==;", "X-IronPort-AV": [ "E=McAfee;i=\"6600,9927,10736\"; a=\"337231651\"", "E=Sophos;i=\"6.00,229,1681196400\"; d=\"scan'208\";a=\"337231651\"", "E=McAfee;i=\"6600,9927,10736\"; a=\"800218313\"", "E=Sophos;i=\"6.00,229,1681196400\"; d=\"scan'208\";a=\"800218313\"" ], "X-ExtLoop1": "1", "From": "Bruce Richardson <bruce.richardson@intel.com>", "To": "dev@dpdk.org", "Cc": "david.marchand@redhat.com,\n\tBruce Richardson <bruce.richardson@intel.com>", "Subject": "[PATCH v2] doc: prefer installing using meson rather than ninja", "Date": "Fri, 9 Jun 2023 14:34:06 +0100", "Message-Id": "<20230609133406.80093-1-bruce.richardson@intel.com>", "X-Mailer": "git-send-email 2.39.2", "In-Reply-To": "<20230609105139.76239-1-bruce.richardson@intel.com>", "References": "<20230609105139.76239-1-bruce.richardson@intel.com>", "MIME-Version": "1.0", "Content-Transfer-Encoding": "8bit", "X-BeenThere": "dev@dpdk.org", "X-Mailman-Version": "2.1.29", "Precedence": "list", "List-Id": "DPDK patches and discussions <dev.dpdk.org>", "List-Unsubscribe": "<https://mails.dpdk.org/options/dev>,\n <mailto:dev-request@dpdk.org?subject=unsubscribe>", "List-Archive": "<http://mails.dpdk.org/archives/dev/>", "List-Post": "<mailto:dev@dpdk.org>", "List-Help": "<mailto:dev-request@dpdk.org?subject=help>", "List-Subscribe": "<https://mails.dpdk.org/listinfo/dev>,\n <mailto:dev-request@dpdk.org?subject=subscribe>", "Errors-To": "dev-bounces@dpdk.org" }, "content": "After doing a build, to install DPDK system-wide our documentation\nrecommended using the \"ninja install\" command. However, for anyone\nbuilding as a non-root user and only installing as root, the \"meson\ninstall\" command is a better alternative, as it provides for\nautomatically dropping or elevating privileges as necessary in more\nrecent meson releases [1].\n\n[1] https://mesonbuild.com/Installing.html#installing-as-the-superuser\n\nSigned-off-by: Bruce Richardson <bruce.richardson@intel.com>\n\n---\nV2:\n* Fix one missed reference to \"ninja install\" in Linux GSG\n* Changed CI scripts to use \"meson install\" to ensure step is properly\n tested.\n---\n .ci/linux-build.sh | 4 ++--\n doc/guides/contributing/coding_style.rst | 2 +-\n doc/guides/cryptodevs/uadk.rst | 2 +-\n doc/guides/freebsd_gsg/build_dpdk.rst | 2 +-\n doc/guides/freebsd_gsg/build_sample_apps.rst | 2 +-\n doc/guides/linux_gsg/build_dpdk.rst | 4 ++--\n doc/guides/prog_guide/build-sdk-meson.rst | 4 ++--\n 7 files changed, 10 insertions(+), 10 deletions(-)", "diff": "diff --git a/.ci/linux-build.sh b/.ci/linux-build.sh\nindex 9631e342b5..69ca46a6a1 100755\n--- a/.ci/linux-build.sh\n+++ b/.ci/linux-build.sh\n@@ -150,14 +150,14 @@ if [ \"$ABI_CHECKS\" = \"true\" ]; then\n git clone --single-branch -b \"$REF_GIT_TAG\" $REF_GIT_REPO $refsrcdir\n meson setup $OPTS -Dexamples= $refsrcdir $refsrcdir/build\n ninja -C $refsrcdir/build\n- DESTDIR=$(pwd)/reference ninja -C $refsrcdir/build install\n+ DESTDIR=$(pwd)/reference meson -C $refsrcdir/build install\n find reference/usr/local -name '*.a' -delete\n rm -rf reference/usr/local/bin\n rm -rf reference/usr/local/share\n echo $REF_GIT_TAG > reference/VERSION\n fi\n \n- DESTDIR=$(pwd)/install ninja -C build install\n+ DESTDIR=$(pwd)/install meson -C build install\n devtools/check-abi.sh reference install ${ABI_CHECKS_WARN_ONLY:-}\n fi\n \ndiff --git a/doc/guides/contributing/coding_style.rst b/doc/guides/contributing/coding_style.rst\nindex 89db6260cf..00d6270624 100644\n--- a/doc/guides/contributing/coding_style.rst\n+++ b/doc/guides/contributing/coding_style.rst\n@@ -956,7 +956,7 @@ ext_deps\n headers\n \t**Default Value = []**.\n \tUsed to return the list of header files for the library that should be\n-\tinstalled to $PREFIX/include when ``ninja install`` is run. As with\n+\tinstalled to $PREFIX/include when ``meson install`` is run. As with\n \tsource files, these should be specified using the meson ``files()``\n \tfunction.\n \tWhen ``check_includes`` build option is set to ``true``, each header file\ndiff --git a/doc/guides/cryptodevs/uadk.rst b/doc/guides/cryptodevs/uadk.rst\nindex 9af6b88a5a..136ab4be6a 100644\n--- a/doc/guides/cryptodevs/uadk.rst\n+++ b/doc/guides/cryptodevs/uadk.rst\n@@ -90,7 +90,7 @@ Test steps\n meson setup build (--reconfigure)\n cd build\n ninja\n- sudo ninja install\n+ sudo meson install\n \n #. Prepare hugepages for DPDK (see also :doc:`../tools/hugepages`)\n \ndiff --git a/doc/guides/freebsd_gsg/build_dpdk.rst b/doc/guides/freebsd_gsg/build_dpdk.rst\nindex 514d18c870..86e8e5a805 100644\n--- a/doc/guides/freebsd_gsg/build_dpdk.rst\n+++ b/doc/guides/freebsd_gsg/build_dpdk.rst\n@@ -47,7 +47,7 @@ The final, install, step generally needs to be run as root::\n meson setup build\n cd build\n ninja\n- ninja install\n+ meson install\n \n This will install the DPDK libraries and drivers to `/usr/local/lib` with a\n pkg-config file `libdpdk.pc` installed to `/usr/local/lib/pkgconfig`. The\ndiff --git a/doc/guides/freebsd_gsg/build_sample_apps.rst b/doc/guides/freebsd_gsg/build_sample_apps.rst\nindex c87e982759..b1ab7545b1 100644\n--- a/doc/guides/freebsd_gsg/build_sample_apps.rst\n+++ b/doc/guides/freebsd_gsg/build_sample_apps.rst\n@@ -22,7 +22,7 @@ the system when DPDK is installed, and so can be built using GNU make.\n on the FreeBSD system.\n \n The following shows how to compile the helloworld example app, following\n-the installation of DPDK using `ninja install` as described previously::\n+the installation of DPDK using `meson install` as described previously::\n \n $ export PKG_CONFIG_PATH=/usr/local/lib/pkgconfig\n \ndiff --git a/doc/guides/linux_gsg/build_dpdk.rst b/doc/guides/linux_gsg/build_dpdk.rst\nindex bbd2efc9d8..9c0dd9daf6 100644\n--- a/doc/guides/linux_gsg/build_dpdk.rst\n+++ b/doc/guides/linux_gsg/build_dpdk.rst\n@@ -68,11 +68,11 @@ Once configured, to build and then install DPDK system-wide use:\n \n cd build\n ninja\n- ninja install\n+ meson install\n ldconfig\n \n The last two commands above generally need to be run as root,\n-with the `ninja install` step copying the built objects to their final system-wide locations,\n+with the `meson install` step copying the built objects to their final system-wide locations,\n and the last step causing the dynamic loader `ld.so` to update its cache to take account of the new objects.\n \n .. note::\ndiff --git a/doc/guides/prog_guide/build-sdk-meson.rst b/doc/guides/prog_guide/build-sdk-meson.rst\nindex 5deabbe54c..93aa1f80e3 100644\n--- a/doc/guides/prog_guide/build-sdk-meson.rst\n+++ b/doc/guides/prog_guide/build-sdk-meson.rst\n@@ -12,7 +12,7 @@ following set of commands::\n \tmeson setup build\n \tcd build\n \tninja\n-\tninja install\n+\tmeson install\n \n This will compile DPDK in the ``build`` subdirectory, and then install the\n resulting libraries, drivers and header files onto the system - generally\n@@ -165,7 +165,7 @@ printing each command on a new line as it runs.\n Installing the Compiled Files\n ------------------------------\n \n-Use ``ninja install`` to install the required DPDK files onto the system.\n+Use ``meson install`` to install the required DPDK files onto the system.\n The install prefix defaults to ``/usr/local`` but can be used as with other\n options above. The environment variable ``DESTDIR`` can be used to adjust\n the root directory for the install, for example when packaging.\n", "prefixes": [ "v2" ] }{ "id": 128464, "url": "