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

Re: [XEN PATCH 1/4] automation: Remove clang-8 from Debian unstable container


  • To: anthony.perard@xxxxxxxxx
  • From: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
  • Date: Wed, 15 Feb 2023 18:15:19 +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=arcselector9901; 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=SGml/FRtRXc79ISGXpgod5+2YxpDDwinJNAZJ/N96xw=; b=mgQa2EmKOB0CCWetA/nokzi+aWrDIkO51v1BveziyFSTsqPcBa3zf4FXm9bMeTzfWIJotUNkIs8inDIzkYcQwIJ5Wju95kiWI8cextSu4h5ZNYXPNHpw+p2V3AjkVHIKdLihyjNsgN7E+Q12G5O+pN30RmAeqEy25G2Aam6KlgjWFgsShPMo+3BgmpLJsRrbkxqfqCtz7DYPvdono+UA7sUhhMnfWS/P3zraEXPsGDl5SO8Va/pqVqDk0UI8s9oKsufQaGili1Yrx4Mmu2TpIr/w3NJRmWHuF8n/X6QR5CoLbVlu2UVgIPtSloc634PheAQImklHxRtHj5huym+kOQ==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=OhYLawFfGyGkzHCman9Q8hxPzBhIHMQnKgZmMsBeRiDtnslmQFrd8RIXZdDbT5o+mmBN7knu7SH9iclTHgBCYkJ7/JEXpdAxoYZ/ejQHR3m0dzbRXeKvxMLXAQGBSHtYP7ZZ5CbIzqApqCXAEdnJ1aAvhDyGq9Fp9yGLMIskKETX7R0Tfj3Hz1sd4l/MjlwxTMNVyTqzc9WBYWRgVsxi0mxXj/3mTUQuX9gOU58QXy1BKWQuqmkPpMXEvafnWoDJecG7Lv0zgJk0lzv+gW25mjs828hk8uzsuvjgVh2XKNGdoF37tC4TjiDNVBWcRe1XSsbnvvAOZXdiaY1YOgQL4w==
  • Authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=citrix.com;
  • Cc: xen-devel@xxxxxxxxxxxxxxxxxxxx, Doug Goldstein <cardoe@xxxxxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>
  • Delivery-date: Wed, 15 Feb 2023 18:16:02 +0000
  • Ironport-data: A9a23:m+ebVK0sFauqzItCM/bD5Rtwkn2cJEfYwER7XKvMYLTBsI5bpzYBz TQcDG2BaamJZWr1ftEgaIWxphlX75fQzNIwSQc/pC1hF35El5HIVI+TRqvS04F+DeWYFR46s J9OAjXkBJppJpMJjk71atANlVEliefTAOK6ULWeUsxIbVcMYD87jh5+kPIOjIdtgNyoayuAo tq3qMDEULOf82cc3lk8tuTS93uDgNyo4GlD5gZnOKgR1LPjvyJ94Kw3dPnZw0TQGuG4LsbiL 87fwbew+H/u/htFIrtJRZ6iLyXm6paLVeS/oiI+t5qK23CulQRrukoPD9IOaF8/ttm8t4sZJ OOhF3CHYVxB0qXkwIzxWvTDes10FfUuFLTveRBTvSEPpqFvnrSFL/hGVSkL0YMkFulfW2MSr tUhbxw3XzOqpMSpyeywUrVWmZF2RCXrFNt3VnBI6xj8VK9jareaBqLA6JlfwSs6gd1IEbDGf c0FZDFzbRPGJRpSJlMQD5F4l+Ct7pX9W2QA9BTJ+uxqvS6Kk1EZPLvFabI5fvSjQ8lPk1nej WXB52njWTkRNcCFyCrD+XWp7gPKtXKlANhPRODjnhJsqACS4U4dUicQaXGyvt6drWr9WtEHI lNBr0LCqoB3riRHVOLVVhukoHmCr1gEVsBZCeES5wSEy66S6AGcbkAUQzgEZNE4ucseQT0xy kTPj97vHSZosrCeVTSa7Lj8hTi7IyQSIEcJbDUISgZD6N7myKkolQ7GRNtnFK+zj/X2FCv2z jTMqzIx74j/luYO3qS/uFvA2jSlo8GQShZvv1qKGGW48gl+eYipIZSy7kTW5upBK4DfSUSdu H8DmI6V6+Vm4YyxqRFhid4lRNmBj8tp+hWF6bKzN/HNLwiQxkM=
  • Ironport-hdrordr: A9a23:1voBh6p5CuRxSH2/t3Ig/WoaV5o6eYIsimQD101hICG9E/bo9P xG+c5xvyMc5wx9ZJheo6HjBEDtex/hHP1OgbX5X43NYOCOggLBEGgI1+rfKlPbdBEW/9QtsJ tdTw==
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

On 15/02/2023 4:31 pm, Andrew Cooper wrote:
> On 15/02/2023 4:28 pm, Anthony PERARD wrote:
>> On Wed, Feb 15, 2023 at 12:26:40PM +0000, Andrew Cooper wrote:
>>> On 15/02/2023 12:02 pm, Anthony PERARD wrote:
>>>> First, apt complain that it isn't the right way to add keys anymore,
>>>> but hopefully that's just a warning.
>>>>
>>>> Second, we can't install clang-8:
>>>> The following packages have unmet dependencies:
>>>>  clang-8 : Depends: libstdc++-8-dev but it is not installable
>>>>            Depends: libgcc-8-dev but it is not installable
>>>>            Depends: libobjc-8-dev but it is not installable
>>>>            Recommends: llvm-8-dev but it is not going to be installed
>>>>            Recommends: libomp-8-dev but it is not going to be installed
>>>>  libllvm8 : Depends: libffi7 (>= 3.3~20180313) but it is not installable
>>>> E: Unable to correct problems, you have held broken packages.
>>>>
>>>> clang on Debian unstable is now version 14.0.6.
>>>>
>>>> Signed-off-by: Anthony PERARD <anthony.perard@xxxxxxxxxx>
>>> Yeah that's because clang 8 is way obsolete, and http://apt.llvm.org
>>> only supports Clang 15 and later now.
>>>
>>> In hindsight, it was a mistake to take unstable-llvm-8.list in the first
>>> place.
>>>
>>> Acked-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
>> I just realize that this change is going to break gitlab testing in the
>> staging-4* branches. So I guess this mean that we will have to backport
>> this patch, or at least the gitlab-ci part.
> I don't think it will.  The currently cached container is still "good"
> for the older branches to use, but we ought to backport the change
> nevertheless.
>
> Given that the container is un(re)buildable anyway, we're not losing
> anything, I don't think.

Ah, so.  Applying just this patch won't break older trees.

But applying this patch, and patch 3, then rebuilding the the debian
unstable container will break older branches, because there will no
longer be anything called clang-8 in the container.

This demonstrates even more that referencing the compiler by exact
version is a mistake.  As is using names like unstable.

As I said up front when we added CI of this form, we were always going
to retrofit some changes to all branches (including the security-only
branches) to keep this working.  These chicken are coming back to roost.

~Andrew



 


Rackspace

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