Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/63780/?format=api
{ "id": 63780, "url": "http://patches.dpdk.org/api/patches/63780/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/patch/20191211182147.19355-4-kevin.laatz@intel.com/", "project": { "id": 1, "url": "http://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": "<20191211182147.19355-4-kevin.laatz@intel.com>", "list_archive_url": "https://inbox.dpdk.org/dev/20191211182147.19355-4-kevin.laatz@intel.com", "date": "2019-12-11T18:21:47", "name": "[v4,3/3] build: clean up experimental syms check", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": true, "hash": "63a694b66ac2299a44a342d4d82564f1e971b98d", "submitter": { "id": 921, "url": "http://patches.dpdk.org/api/people/921/?format=api", "name": "Kevin Laatz", "email": "kevin.laatz@intel.com" }, "delegate": { "id": 24651, "url": "http://patches.dpdk.org/api/users/24651/?format=api", "username": "dmarchand", "first_name": "David", "last_name": "Marchand", "email": "david.marchand@redhat.com" }, "mbox": "http://patches.dpdk.org/project/dpdk/patch/20191211182147.19355-4-kevin.laatz@intel.com/mbox/", "series": [ { "id": 7802, "url": "http://patches.dpdk.org/api/series/7802/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/list/?series=7802", "date": "2019-12-11T18:21:44", "name": "Add ABI compatibility checks to the meson build", "version": 4, "mbox": "http://patches.dpdk.org/series/7802/mbox/" } ], "comments": "http://patches.dpdk.org/api/patches/63780/comments/", "check": "success", "checks": "http://patches.dpdk.org/api/patches/63780/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 dpdk.org (dpdk.org [92.243.14.124])\n\tby inbox.dpdk.org (Postfix) with ESMTP id 92F83A04F5;\n\tWed, 11 Dec 2019 19:22:33 +0100 (CET)", "from [92.243.14.124] (localhost [127.0.0.1])\n\tby dpdk.org (Postfix) with ESMTP id A184E1BF31;\n\tWed, 11 Dec 2019 19:22:25 +0100 (CET)", "from mga12.intel.com (mga12.intel.com [192.55.52.136])\n by dpdk.org (Postfix) with ESMTP id 0DE681BE3D\n for <dev@dpdk.org>; Wed, 11 Dec 2019 19:22:22 +0100 (CET)", "from orsmga007.jf.intel.com ([10.7.209.58])\n by fmsmga106.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384;\n 11 Dec 2019 10:22:22 -0800", "from silpixa00399838.ir.intel.com (HELO\n silpixa00399838.ger.corp.intel.com) ([10.237.222.157])\n by orsmga007.jf.intel.com with ESMTP; 11 Dec 2019 10:22:20 -0800" ], "X-Amp-Result": "SKIPPED(no attachment in message)", "X-Amp-File-Uploaded": "False", "X-ExtLoop1": "1", "X-IronPort-AV": "E=Sophos;i=\"5.69,302,1571727600\"; d=\"scan'208\";a=\"203655561\"", "From": "Kevin Laatz <kevin.laatz@intel.com>", "To": "dev@dpdk.org", "Cc": "thomas@monjalon.net, david.marchand@redhat.com,\n bruce.richardson@intel.com,\n ray.kinsella@intel.com, Kevin Laatz <kevin.laatz@intel.com>", "Date": "Wed, 11 Dec 2019 18:21:47 +0000", "Message-Id": "<20191211182147.19355-4-kevin.laatz@intel.com>", "X-Mailer": "git-send-email 2.17.1", "In-Reply-To": "<20191211182147.19355-1-kevin.laatz@intel.com>", "References": "<20191129210905.1865-1-kevin.laatz@intel.com>\n <20191211182147.19355-1-kevin.laatz@intel.com>", "Subject": "[dpdk-dev] [PATCH v4 3/3] build: clean up experimental syms check", "X-BeenThere": "dev@dpdk.org", "X-Mailman-Version": "2.1.15", "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", "Sender": "\"dev\" <dev-bounces@dpdk.org>" }, "content": "This patch cleans up the meson build files in lib and drivers by moving the\ncustom target for checking the experimental syms next to the abi compat\nchecks. This also removes the dependency on the check for the shared build,\nwhich was not required by anything, but was only added to force the\nexperimental syms check run.\n\nSigned-off-by: Kevin Laatz <kevin.laatz@intel.com>\n---\n drivers/meson.build | 21 +++++++++++----------\n lib/meson.build | 21 +++++++++++----------\n 2 files changed, 22 insertions(+), 20 deletions(-)", "diff": "diff --git a/drivers/meson.build b/drivers/meson.build\nindex db58a67b9..4f4c93b73 100644\n--- a/drivers/meson.build\n+++ b/drivers/meson.build\n@@ -163,15 +163,6 @@ foreach class:dpdk_driver_classes\n \t\t\t\t\t'-Wl,/implib:lib\\\\' + implib]\n \t\t\telse\n \t\t\t\tlk_args = ['-Wl,--version-script=' + version_map]\n-\t\t\t\t# on unix systems check the output of the\n-\t\t\t\t# experimental syms script, using it as a\n-\t\t\t\t# dependency of the .so build\n-\t\t\t\tlk_deps += custom_target(lib_name + '.exp_chk',\n-\t\t\t\t\tcommand: [check_experimental_syms,\n-\t\t\t\t\t\tversion_map, '@INPUT@'],\n-\t\t\t\t\tcapture: true,\n-\t\t\t\t\tinput: static_lib,\n-\t\t\t\t\toutput: lib_name + '.exp_chk')\n \t\t\tendif\n \n \t\t\tshared_lib = shared_library(lib_name,\n@@ -181,7 +172,6 @@ foreach class:dpdk_driver_classes\n \t\t\t\tdependencies: shared_deps,\n \t\t\t\tc_args: cflags,\n \t\t\t\tlink_args: lk_args,\n-\t\t\t\tlink_depends: lk_deps,\n \t\t\t\tversion: lib_version,\n \t\t\t\tsoversion: so_version,\n \t\t\t\tinstall: true,\n@@ -197,6 +187,17 @@ foreach class:dpdk_driver_classes\n \t\t\t\t\tdependencies: static_deps)\n \n \t\t\tif abidiff.found()\n+\t\t\t\t# on unix systems check the output of the\n+\t\t\t\t# experimental syms script\n+\t\t\t\tcustom_target(lib_name + '.exp_chk',\n+\t\t\t\t\tcommand: [check_experimental_syms,\n+\t\t\t\t\t\tversion_map, '@INPUT@'],\n+\t\t\t\t\tcapture: true,\n+\t\t\t\t\tinput: static_lib,\n+\t\t\t\t\toutput: lib_name + '.exp_chk',\n+\t\t\t\t\tinstall: false,\n+\t\t\t\t\tbuild_by_default: true)\n+\n \t\t\t\tcustom_target('lib' + lib_name + '.abi_chk',\n \t\t\t\t\tcommand: [abidiff, '--no-added-syms',\n \t\t\t\t\t\t'--suppr', abignore,\ndiff --git a/lib/meson.build b/lib/meson.build\nindex f0aba6635..b0f915969 100644\n--- a/lib/meson.build\n+++ b/lib/meson.build\n@@ -154,15 +154,6 @@ foreach l:libraries\n \t\t\t\t\t'-Wl,/implib:lib\\\\' + implib]\n \t\t\telse\n \t\t\t\tlk_args = ['-Wl,--version-script=' + version_map]\n-\t\t\t\t# on unix systems check the output of the\n-\t\t\t\t# experimental syms script, using it as a\n-\t\t\t\t# dependency of the .so build\n-\t\t\t\tlk_deps += custom_target(name + '.exp_chk',\n-\t\t\t\t\tcommand: [check_experimental_syms,\n-\t\t\t\t\t\tversion_map, '@INPUT@'],\n-\t\t\t\t\tcapture: true,\n-\t\t\t\t\tinput: static_lib,\n-\t\t\t\t\toutput: name + '.exp_chk')\n \t\t\tendif\n \n \t\t\tshared_lib = shared_library(libname,\n@@ -172,7 +163,6 @@ foreach l:libraries\n \t\t\t\t\tdependencies: shared_deps,\n \t\t\t\t\tinclude_directories: includes,\n \t\t\t\t\tlink_args: lk_args,\n-\t\t\t\t\tlink_depends: lk_deps,\n \t\t\t\t\tversion: lib_version,\n \t\t\t\t\tsoversion: so_version,\n \t\t\t\t\tinstall: true)\n@@ -181,6 +171,17 @@ foreach l:libraries\n \t\t\t\t\tdependencies: shared_deps)\n \n \t\t\tif abidiff.found()\n+\t\t\t\t# on unix systems check the output of the\n+\t\t\t\t# experimental syms script\n+\t\t\t\tcustom_target(name + '.exp_chk',\n+\t\t\t\t\tcommand: [check_experimental_syms,\n+\t\t\t\t\t\tversion_map, '@INPUT@'],\n+\t\t\t\t\tcapture: true,\n+\t\t\t\t\tinput: static_lib,\n+\t\t\t\t\toutput: name + '.exp_chk',\n+\t\t\t\t\tinstall: false,\n+\t\t\t\t\tbuild_by_default: true)\n+\n \t\t\t\tcustom_target(dir_name + '.abi_chk',\n \t\t\t\t\tcommand: [abidiff, '--no-added-syms',\n \t\t\t\t\t\t'--suppr', abignore,\n", "prefixes": [ "v4", "3/3" ] }