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

Re: [PATCH for-4.15] automation: Fix the Alpine clang builds to use clang


  • To: Jan Beulich <jbeulich@xxxxxxxx>
  • From: Roger Pau Monné <roger.pau@xxxxxxxxxx>
  • Date: Mon, 1 Mar 2021 10:31:41 +0100
  • 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-SenderADCheck; bh=KYZkX7geCxLSO9G0LY8wuBLG4qVdOP9FeM2/vBRf5d8=; b=F5DesffL9UHcq9RFWhFVBZI3k9h/EBrhmkPTz48gzLjWoOmt6i9VOQ7fDbYtePeAt1znX16HzvoKqkP3oT10rKvNqzlGUcEQxPlgJHsv/cJdi04IDEpS2rdx4TBlZ0321mYAUuwxguBQJPCpYkVyeZKxUHiwQbX5QbI87NgE5xjGqMXgqImHYjxcM4KxwWRPYlzXwzK8RBbDq+EcBEv7Zs5rVZMSNjOcr2x9okr+Io4bmQctk6p/GmLtlkd4ydLHV6hxtCTZmB88Q+KgSSIvRPMaKMN39IG0LibxS+X4hdzHSE8sbE++/XOoDQlNUNUy6YhxvieW/jlauOyG2rtz9g==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=XWWbJQ6lHDZC7Zoz+xRHSQKeP12OlLsyo/xBp/awUdTRQS15vZUyUin9emIPYk89RPV++5gETNfXWhKa17aQfTBy9XgJV3lT4Rj/SiShScILaaAI9Nq3AVSWV1nUJyS1cqTrWpGo+nceeCivJZnKBzWIvAUMl3qSi0dsKkuP/sSodf1D354AL6ORvoRgvDTlGJR5rZuCuec9xo8bElpWWYZRR5hrrggusHvJjbp6nt36Trn6R9pk5IdN2uJSqECMTcpAWRw2SS1XxKhWqIUFeKZiJU+NeezKc2tv3N4XjbbcCaFt5uQqq5XS8t9jf5UV7gx4sVLAxl817HMuOsp4BA==
  • Authentication-results: esa2.hc3370-68.iphmx.com; dkim=pass (signature verified) header.i=@citrix.onmicrosoft.com
  • Cc: Stefano Stabellini <sstabellini@xxxxxxxxxx>, Xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxxx>, Doug Goldstein <cardoe@xxxxxxxxxx>, Ian Jackson <iwj@xxxxxxxxxxxxxx>, Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
  • Delivery-date: Mon, 01 Mar 2021 09:31:58 +0000
  • Ironport-sdr: UPcTxmatAYx10PeP5PAyrxuPp1nsdqHW1zNsCXkDF6PXx/9UC+pFRUQlEuTYLQFIUeJJlysPec V1pp0RER2+i7UQ9uLsgWjG+xM2dOg4EcQ7/iikkoeNluKvvhaQOb+NHelDgtPaj+BCozFT78CR kgiXUMTQDD8qgKCcGMYhSLN8peW2pj+NcVPcBRQ6NKDNTJPIXxUySN4b639YSfxwQUydTaGm32 vpQkn47YaRQFspMVHTLjwncWqx9TSaThN7rkXzYsNboSwF4mZQo7/6WPKtjbQVsUBt/heHuAMZ IC8=
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

On Mon, Mar 01, 2021 at 09:58:48AM +0100, Jan Beulich wrote:
> On 26.02.2021 23:48, Stefano Stabellini wrote:
> > On Fri, 26 Feb 2021, Andrew Cooper wrote:
> >> Looks like a copy&paste error.
> >>
> >> Fixes: f6e1d8515d7 ("automation: add alpine linux x86 build jobs")
> >> Signed-off-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
> > 
> > Thanks for the patch and of course it is correct
> > 
> > Acked-by: Stefano Stabellini <sstabellini@xxxxxxxxxx>
> > 
> > 
> > However unfortunately it breaks the Alpine Linux gitlab-ci again :-(
> > I created a branch with Roger's patches plus this patch. The two clang
> > Alpine Linux build jobs fail:
> > 
> > https://gitlab.com/xen-project/people/sstabellini/xen/-/jobs/1059686530
> > https://gitlab.com/xen-project/people/sstabellini/xen/-/jobs/1059686532
> > 
> > 
> > The error is the following:
> > 
> > <built-in>:3:10: fatal error: 'cstring' file not found
> > #include "cstring"
> >          ^~~~~~~~~
> > 1 error generated.
> > make[10]: *** [Makefile:120: headers++.chk] Error 1
> > make[10]: *** Waiting for unfinished jobs....
> 
> Doesn't this suggest there's a C++ compiler installed without
> the headers also being available? The headers++.chk rule
> requires neither or both to be present.

Right, that's because the include path for clang++ seems to rely on
the gcc c++ headers:

#include <...> search starts here:
 
/usr/bin/../lib/gcc/x86_64-alpine-linux-musl/10.2.1/../../../../include/c++/10.2.1
 
/usr/bin/../lib/gcc/x86_64-alpine-linux-musl/10.2.1/../../../../include/c++/10.2.1/x86_64-alpine-linux-musl
 
/usr/bin/../lib/gcc/x86_64-alpine-linux-musl/10.2.1/../../../../include/c++/10.2.1/backward
 /usr/include
 /usr/lib/clang/10.0.1/include

It's solved by installing g++, I will prepare a patch against the
docker file. I've got no idea why clang doesn't install the required
c++ headers (or simply pull in g++ if that's required).

It's unrelated to the -ffreestanding issue.

Roger.



 


Rackspace

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