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

Re: [PATCH] xen: remove the usage of the P ar option


  • To: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
  • From: Roger Pau Monné <roger.pau@xxxxxxxxxx>
  • Date: Thu, 31 Dec 2020 10:20:15 +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=R3mxOnLO74MPNjud4xKFP0mDDrraeYSt6AZo5I4xlSQ=; b=ISza6vBwz+QiVSyHIKJjZFGHePbM6qiIlXxofItzWqrncz3g/045CasDj79TQjcs+X2S1zjQUvb2gkzCpqiEGVuowFWxRq7tL89MtvmKTWjYRZXpGyhbdpskS3UOk2hVjAMscIzGIql7KJLco14Gmh2dSgCfqR/xnnyMqAiJCSD5vXJfL6nWZMZIsK82pojHHMNzuLbQqi3GIJwUmrYOucr5ltRkbhzdU4h97SN3+vEorxk+RRWv37B8dIL12yJTuGQB3hhwrHaTDd5SMaRWGI5BD2O9Lvnreif+QWwYg5e5dDG1DzZacPOWTwUem33iFWfSz3W5nSBCiNZuA/NWjw==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=XC0JzoMQEa10fcDz2fj2zA3dw4/A5XpnuNkpTlqRDPhe+B42RGucC3+vMv5ZYELmjDMR3L5bivo93M/YIUiywEA67XQhzPcxkfJ2q15tIB3eAgd6mtRB/pN2VroGbPaWfZ1XhxzguIPgpGTIwZNg2TZMWEPjyVDr2nBA+J01YylHs+9ROXAb8XoXkumS/8ZedrpqT8Y49nUVt6+Kc2kynxzlG9eTgEWgFSoi4AVWqy3Hs9meNRqmY+ys4H8LAUtaXcEAtSjgAsIm6tSn2UZPpet5icku5cKHXgpwxyECulguIJzaGHhEZ5HNl/wHqGjwLhLsa2VT7QzN75UxuFqKOw==
  • Authentication-results: esa4.hc3370-68.iphmx.com; dkim=pass (signature verified) header.i=@citrix.onmicrosoft.com
  • Cc: <xen-devel@xxxxxxxxxxxxxxxxxxxx>, George Dunlap <george.dunlap@xxxxxxxxxx>, Ian Jackson <iwj@xxxxxxxxxxxxxx>, Jan Beulich <jbeulich@xxxxxxxx>, Julien Grall <julien@xxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Wei Liu <wl@xxxxxxx>
  • Delivery-date: Thu, 31 Dec 2020 09:20:37 +0000
  • Ironport-sdr: y/QRrfxr3OiAkFFQNis31YGruPcxEiROiOz+SnKJzpZYuZo+h/NoKiXOT/gcyCpPYNZ6R4XwSh +ggq7srKfuNaAQ4bnhkCYDuQmRMZgB+6OS9ZdqH3RahTIH7dzH2sb202iF7VoRAQJnyoEvzEW/ djxY1TdgOYIQH3eFuQTVTubNnsJicH54jkwiP78h9j+29shTWtjbX+zg2wCcwKr4wwIWf/rBp4 wziNIIX9mNZQcTVExkga0b5yXMz9nwAa/diy5pUP4XWrmKdLL7sYKKEnkMImsRErgX26p1qE5g UR0=
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

On Wed, Dec 30, 2020 at 06:32:58PM +0000, Andrew Cooper wrote:
> On 30/12/2020 17:34, Roger Pau Monne wrote:
> > It's not part of the POSIX standard [0] and as such non GNU ar
> > implementations don't usually have it.
> >
> > It's not relevant for the use case here anyway, as the archive file is
> > recreated every time due to the rm invocation before the ar call. No
> > file name matching should happen so matching using the full path name
> > or a relative one should yield the same result.
> >
> > This fixes the build on FreeBSD.
> >
> > Signed-off-by: Roger Pau Monné <roger.pau@xxxxxxxxxx>
> 
> Acked-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, although...
> 
> We really need some kind of BSD build in CI.  This kind of breakage
> shouldn't get into master to begin with.

Fully agree. I'm not that familiar with gitlab CI, but since we have
our own runners there, could we also launch VMs instead of just using
containers?

It might be easier to integrate with osstest in the future, since
FreeBSD has now switched to git.

> >
> > [0] https://pubs.opengroup.org/onlinepubs/9699919799/utilities/ar.html
> > ---
> > I'm unsure whether the r and s options are also needed, since they
> > seem to only be relevant when updating a library, and Xen build system
> > always removes the old library prior to any ar call.
> 
> ... I think r should be dropped, because we're not replacing any files. 
> However, I expect the index file is still relevant, because without it,
> you've got to perform an O(n) search through the archive to find a file.

Right, the description of 's' between opengroup and the Linux man page
seems to be slightly different. From the opengroup description it seems
like s should be used to force the generation of a symbol table when
no changes are made to the archive, but otherwise ar should already
generate such symbol table.

Thanks, Roger.



 


Rackspace

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