[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [XEN PATCH] CI: Remove `brctl` from Arch Linux container


  • To: Anthony PERARD <anthony@xxxxxxxxxxxxxx>, xen-devel@xxxxxxxxxxxxxxxxxxxx
  • From: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
  • Date: Fri, 13 Feb 2026 11:58:06 +0000
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=citrix.com; dmarc=pass action=none header.from=citrix.com; dkim=pass header.d=citrix.com; arc=none
  • Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=hPJZaV7SexPrpxCoY92LjoZDMc/je4U139Q2UO8WbXQ=; b=uOia21KtwGqiDR9kkjz1YfTcunkgAy+K8PUgOR2LEAn4L4tP9hv3GBgFlbMYhGA36EkffBjzMgvgEFFtQyfG/b4F3pRmmx44MAyEM7WyrnYD2RfQNKcmWOHbjmxQt80sqXnLZHQNwm58frg4xeffS4tMO1kUEUbHiiPzaQgNQIhBJ263uCiJtl2apBU+0ybtfBj3uLKoiOm9reaM4hi+mUdWzzZSzm6XWjRYcwfAVvY4WFyjL18P1hFRVjd3hZBSMG16Tg0+qub7UstUjmnZKm+w7V+ZCNd/SJdWppx4h6JclX6DLAgEz4d1KiBNMzQwnO3+Wqa6mYWha/iTmIe7HQ==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=yc0Ujpqc1MJBsZA+VHpBdp17UWQT4qWzvM5ROIvr0Fdq1NGjqZKE6ezsDr7Pza2t97XQ4gaIjyEq8Co8QDW1f8SJOEHq4PfcgR0pJ3BXQaSlCljx3gaiqyGPDQbXWJBaH8WPUHNGPLjAcL0NbooUOyqtYPgoLaOcOe/MI2MmVhPVH+HIJym9QOVPo3+odRA2dbnsRqzqNH95lruRQd8nxp7gmzmU/KIo/ZCYinDUfyubAh8pEVKmrNMCFziXzmhl2h3IreZmJpR+yb3Uo1fBGl9ws2qjMt0LXpCbsHAILFatYySy7ubqwjXIpQTDfQD2uHfjdRAqAtkM3+TMII/nXg==
  • Authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=citrix.com;
  • Cc: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, Anthony PERARD <anthony.perard@xxxxxxxxxx>, Doug Goldstein <cardoe@xxxxxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>
  • Delivery-date: Fri, 13 Feb 2026 11:58:23 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

On 13/02/2026 10:19 am, Anthony PERARD wrote:
> From: Anthony PERARD <anthony.perard@xxxxxxxxxx>
>
> Package "bridge-utils" have been removed from the official repos, it
> contained `brctl`. That utility is used by network hotplug scripts,
> but they already check if it is available and switch to use "iproute2"
> (`ip`) when not.
>
> Signed-off-by: Anthony PERARD <anthony.perard@xxxxxxxxxx>

Acked-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>

Right now, README lists bridge-utils as a hard prereq, and this is not
true any longer.

iproute is also listed as a hard prereq.  Should we merge these two
lines with a logical or?

> ---
>  automation/build/archlinux/current.dockerfile | 1 -
>  1 file changed, 1 deletion(-)
>
> diff --git a/automation/build/archlinux/current.dockerfile 
> b/automation/build/archlinux/current.dockerfile
> index 4e53c835fa..a257027772 100644
> --- a/automation/build/archlinux/current.dockerfile
> +++ b/automation/build/archlinux/current.dockerfile
> @@ -6,7 +6,6 @@ LABEL maintainer.name="The Xen Project" \
>  RUN pacman-key --init
>  
>  RUN pacman -S --refresh --sysupgrade --noconfirm --noprogressbar --needed \
> -        bridge-utils \
>          bzip2 \
>          cpio \
>          discount \

If you're making this change, I'd like to dust off patch to convert to
heredoc and get both submitted together.

~Andrew



 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.