Patch Detail
Show a patch.
GET /api/patches/18315/?format=api
{ "id": 18315, "url": "https://patchwork.libcamera.org/api/patches/18315/?format=api", "web_url": "https://patchwork.libcamera.org/patch/18315/", "project": { "id": 1, "url": "https://patchwork.libcamera.org/api/projects/1/?format=api", "name": "libcamera", "link_name": "libcamera", "list_id": "libcamera_core", "list_email": "libcamera-devel@lists.libcamera.org", "web_url": "", "scm_url": "", "webscm_url": "" }, "msgid": "<mailman.34.1677532518.25031.libcamera-devel@lists.libcamera.org>", "date": "2023-02-27T21:15:04", "name": "[libcamera-devel,v2] meson: Align handling of build metadata", "commit_ref": null, "pull_url": null, "state": "accepted", "archived": false, "hash": "aef720157c7c6dbf31dc676f42847b479a41aebe", "submitter": { "id": 135, "url": "https://patchwork.libcamera.org/api/people/135/?format=api", "name": "Michael Riesch", "email": "michael.riesch@wolfvision.net" }, "delegate": null, "mbox": "https://patchwork.libcamera.org/patch/18315/mbox/", "series": [ { "id": 3774, "url": "https://patchwork.libcamera.org/api/series/3774/?format=api", "web_url": "https://patchwork.libcamera.org/project/libcamera/list/?series=3774", "date": "2023-02-27T21:15:04", "name": "[libcamera-devel,v2] meson: Align handling of build metadata", "version": 2, "mbox": "https://patchwork.libcamera.org/series/3774/mbox/" } ], "comments": "https://patchwork.libcamera.org/api/patches/18315/comments/", "check": "pending", "checks": "https://patchwork.libcamera.org/api/patches/18315/checks/", "tags": {}, "headers": { "Return-Path": "<libcamera-devel-bounces@lists.libcamera.org>", "X-Original-To": "parsemail@patchwork.libcamera.org", "Delivered-To": "parsemail@patchwork.libcamera.org", "Received": [ "from lancelot.ideasonboard.com (lancelot.ideasonboard.com\n\t[92.243.16.209])\n\tby patchwork.libcamera.org (Postfix) with ESMTPS id 06553BE08A\n\tfor <parsemail@patchwork.libcamera.org>;\n\tMon, 27 Feb 2023 21:15:20 +0000 (UTC)", "from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 398396265A;\n\tMon, 27 Feb 2023 22:15:19 +0100 (CET)" ], "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/simple; d=libcamera.org;\n\ts=mail; t=1677532519;\n\tbh=jENDUjcdMzFowE17qK9JCOrQmpcbwVX5374cRXtI38I=;\n\th=To:Date:List-Id:List-Post:From:List-Subscribe:List-Unsubscribe:\n\tList-Archive:Reply-To:List-Help:Subject:From;\n\tb=VZUDEC42YfYnIEQjFwyl3X9ALb3CM6KhlFZPiEys522RuWmgBiFTEmTnAna1RN7pa\n\tPgeHJS4q+CXI9+XSUc13i1+xn4seX6SiBq6cuzSjQWVmHYu2TdFvX2ocJUrb2Md86F\n\tKBnxj1kZLFodm9gCau15s2Yr6PP9uf6jf0o2JLzQRYsV+2fuRYwy+qu3z2QRbDTWM8\n\tl09+JS+BobjNovBwjKgFK2JBu9JxYKfQ2DOcPEFrsX2K6q8xWLKz5v0nBHVtL311ck\n\thngM+Q0xdJuuw1KdjT5fqur8kMB7EhkX0R7Bn40LbimTU0drWzDNnlDtL2Hr3XeCNR\n\tX3ML1VtnSGJ5A==", "To": "libcamera-devel@lists.libcamera.org", "Date": "Mon, 27 Feb 2023 22:15:04 +0100", "MIME-Version": "1.0", "Message-ID": "<mailman.34.1677532518.25031.libcamera-devel@lists.libcamera.org>", "List-Id": "<libcamera-devel.lists.libcamera.org>", "List-Post": "<mailto:libcamera-devel@lists.libcamera.org>", "From": "Michael Riesch via libcamera-devel\n\t<libcamera-devel@lists.libcamera.org>", "Precedence": "list", "X-Mailman-Version": "2.1.29", "X-BeenThere": "libcamera-devel@lists.libcamera.org", "List-Subscribe": "<https://lists.libcamera.org/listinfo/libcamera-devel>,\n\t<mailto:libcamera-devel-request@lists.libcamera.org?subject=subscribe>", "List-Unsubscribe": "<https://lists.libcamera.org/options/libcamera-devel>,\n\t<mailto:libcamera-devel-request@lists.libcamera.org?subject=unsubscribe>", "List-Archive": "<https://lists.libcamera.org/pipermail/libcamera-devel/>", "Reply-To": "Michael Riesch <michael.riesch@wolfvision.net>", "List-Help": "<mailto:libcamera-devel-request@lists.libcamera.org?subject=help>", "Subject": "[libcamera-devel] [PATCH v2] meson: Align handling of build metadata", "Content-Type": "message/rfc822", "Content-Disposition": "inline", "Errors-To": "libcamera-devel-bounces@lists.libcamera.org", "Sender": "\"libcamera-devel\" <libcamera-devel-bounces@lists.libcamera.org>" }, "content": "The build metadata is split off correctly from the version string\nobtained with utils/gen-version.sh, but for the meson project version\nthis step is not carried out. However, since libcamera uses Semantic\nVersioning, it should be possible to add build metadata to the meson\nproject version. Align the handling of the build metadata to resolve\nthis mismatch.\n\nSigned-off-by: Michael Riesch <michael.riesch@wolfvision.net>\n---\nv2 of this series follows a discussion with kbingham in #libcamera\n\nChanges in v2:\n - revise patch message to reflect the current mismatch and the\n proposed fix better\n - introduce the 'project_version' variable to avoid three .split()\n operations\n\n meson.build | 9 +++++----\n 1 file changed, 5 insertions(+), 4 deletions(-)", "diff": "diff --git a/meson.build b/meson.build\nindex 9eee9d39..c8222338 100644\n--- a/meson.build\n+++ b/meson.build\n@@ -25,13 +25,14 @@ if libcamera_git_version == ''\n endif\n \n libcamera_version = libcamera_git_version.split('+')[0]\n+project_version = meson.project_version().split('+')[0]\n \n # A shallow clone, or a clone without a reachable tag equivalent to the\n # meson.project_version() could leave the project in a mis-described state.\n # Produce a warning in this event, and fix to a best effort.\n-if libcamera_version != meson.project_version()\n+if libcamera_version != project_version\n warning('The sources disagree about the version: '\n- + libcamera_version + ' != ' + meson.project_version())\n+ + libcamera_version + ' != ' + project_version)\n \n summary({'libcamera git version' : libcamera_git_version,\n 'Source version match' : false,\n@@ -40,9 +41,9 @@ if libcamera_version != meson.project_version()\n \n # Replace the version components reported by git with the release version,\n # but keep all trailing information supplied by git.\n- libcamera_git_version = (meson.project_version() +\n+ libcamera_version = project_version\n+ libcamera_git_version = (libcamera_version +\n libcamera_git_version.strip(libcamera_version))\n- libcamera_version = meson.project_version()\n \n # Append a marker to show we have modified this version string\n libcamera_git_version += '-nvm'\n", "prefixes": [ "libcamera-devel", "v2" ] }