From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: <pve-devel-bounces@lists.proxmox.com> Received: from firstgate.proxmox.com (firstgate.proxmox.com [IPv6:2a01:7e0:0:424::9]) by lore.proxmox.com (Postfix) with ESMTPS id A278E1FF195 for <inbox@lore.proxmox.com>; Fri, 7 Mar 2025 15:07:10 +0100 (CET) Received: from firstgate.proxmox.com (localhost [127.0.0.1]) by firstgate.proxmox.com (Proxmox) with ESMTP id 5D7241C750; Fri, 7 Mar 2025 15:07:03 +0100 (CET) Message-ID: <5cf29bf6-5cab-4f4e-b436-60fd5ed1313a@proxmox.com> Date: Fri, 7 Mar 2025 15:06:58 +0100 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Beta To: Gabriel Goller <g.goller@proxmox.com>, pve-devel@lists.proxmox.com References: <20250307122945.307426-1-g.goller@proxmox.com> Content-Language: en-GB From: Thomas Lamprecht <t.lamprecht@proxmox.com> In-Reply-To: <20250307122945.307426-1-g.goller@proxmox.com> X-SPAM-LEVEL: Spam detection results: 0 AWL -0.039 Adjusted score from AWL reputation of From: address BAYES_00 -1.9 Bayes spam probability is 0 to 1% DMARC_MISSING 0.1 Missing DMARC policy KAM_DMARC_STATUS 0.01 Test Rule for DKIM or SPF Failure with Strict Alignment SPF_HELO_NONE 0.001 SPF: HELO does not publish an SPF Record SPF_PASS -0.001 SPF: sender matches SPF record Subject: [pve-devel] applied-series: [PATCH 1/4] frr: bump from 8.5.2 to 10.2.1 X-BeenThere: pve-devel@lists.proxmox.com X-Mailman-Version: 2.1.29 Precedence: list List-Id: Proxmox VE development discussion <pve-devel.lists.proxmox.com> List-Unsubscribe: <https://lists.proxmox.com/cgi-bin/mailman/options/pve-devel>, <mailto:pve-devel-request@lists.proxmox.com?subject=unsubscribe> List-Archive: <http://lists.proxmox.com/pipermail/pve-devel/> List-Post: <mailto:pve-devel@lists.proxmox.com> List-Help: <mailto:pve-devel-request@lists.proxmox.com?subject=help> List-Subscribe: <https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel>, <mailto:pve-devel-request@lists.proxmox.com?subject=subscribe> Reply-To: Proxmox VE development discussion <pve-devel@lists.proxmox.com> Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: pve-devel-bounces@lists.proxmox.com Sender: "pve-devel" <pve-devel-bounces@lists.proxmox.com> On 07/03/2025 13:29, Gabriel Goller wrote: > This includes a new frr-test-tools package that we are not interested in > (it's a testing package), so we ignore it with a BuildProfile. > > Signed-off-by: Gabriel Goller <g.goller@proxmox.com> > --- > Makefile | 2 +- > debian/control | 9 +++++++++ > frr | 2 +- > 3 files changed, 11 insertions(+), 2 deletions(-) > > applied series, thanks! I adapted the bump commit to use our common commit subject style and more importantly I fixed the versioning. As FRR is not developed by us we cannot use native versioning but need to split the native version and the packaging revision with a minus; this also ensures upgrades are correctly pulled in independently if the native part gets bumped by upstream or we bump the revision again for the same upstream version due to (packaging) changes on top of an upstream version that got already released to our package repos. I got some info for this in the packaging guide in our internal wiki (might make that public in the future) and it basically follows Debian, so you can also check out the respective section [0] of the Debian Policy Manual. [0]: https://www.debian.org/doc/debian-policy/ch-controlfields.html#version I also backported libyang3 to satisfy updated build-dependency requirements of upstream, doing so itself is not a lot of work, but having to find out that this is needed is not only some extra work that can be avoided, it's also rather risky to slip through if it would only matter at runtime, causing some ugly breakage. So, the next time please ensure to build in a clean environment, e.g., using `sbuild` and a relatively minimal base image, and also check upstream for any newer dependency needs and update the Build-Depends section in debian/control accordingly. _______________________________________________ pve-devel mailing list pve-devel@lists.proxmox.com https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel