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

Re: [PATCH] build: always use BASEDIR for xen sub-directory


  • To: Jan Beulich <jbeulich@xxxxxxxx>
  • From: Bertrand Marquis <Bertrand.Marquis@xxxxxxx>
  • Date: Fri, 2 Oct 2020 13:39:32 +0000
  • Accept-language: en-GB, en-US
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=arm.com; dmarc=pass action=none header.from=arm.com; dkim=pass header.d=arm.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=H+S3Vu23I+R+RrBoxjqBi0zqDdivtx/4Um+qF6Mi2hE=; b=FxqK3ilEBAtrQfTWkEXuuLPobZ+RdltHcBnPAe4LE5KXv10h7kQIKmedk+WSBT6ldpuhIVjpI9ZHisIgB/wGgQRY0e7zpFxnJEJEfGWwpuwb0gktq4K04UY65sR+BTSM2newVbNK09RLifYgcUnKrJol3+R/kKZtFLXvdA/UMlXc7vTBpm7nFujuP9AMxFljIeuHYhajvxOzViEu9L8fjslHi7R2Ux84w5JMFDKTwAT2DRg8RUkISZrU6iYrB+nQRcjKghZGgcIaw08GsLv2IOtenb8Q2pg/i5dxjXb1T/m3KI6yHOgkXdO4cBD7qadcK4WKHaondBhTiFzEv9kmOw==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=Gh/zgC4Wry1lNzN+PDBdeOA5pMOHIxZBBmXxUb+aE14zop5GiRzUFDMrAz0UclhZ2qhdie4a/BqLwn2M9rxLrUTRYD4FWDC+J5agCX+RLgMKI6T0X9g/EfBXuh8nDXF7MiDVrPXjoE9pL8k2KCcwc5/3OaapOCMHBdKFB1EPxQ1ygn/vvh/qEK1uq6oabypfZoPF/nPO6SR2MWt4ce+noC+uQEy1luzWDi0UVXF8RmvHfot2EfnE9YVEFMXHlMR4/mdiPU7rUGyNNtbqs7cago4xVQTsVwsN3APLLvEdOA3ge79A0rB/wdf2jbV0WA5wbck/ldaFzYs/gz8JApFL4g==
  • Authentication-results-original: suse.com; dkim=none (message not signed) header.d=none;suse.com; dmarc=none action=none header.from=arm.com;
  • Cc: "open list:X86" <xen-devel@xxxxxxxxxxxxxxxxxxxx>
  • Delivery-date: Fri, 02 Oct 2020 13:39:46 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
  • Nodisclaimer: true
  • Original-authentication-results: suse.com; dkim=none (message not signed) header.d=none;suse.com; dmarc=none action=none header.from=arm.com;
  • Thread-index: AQHWmKjTj7VmlPYcLkSjqrrhNFG5S6mEOTwAgAAGTYCAAAEPgIAAAYQAgAAJbQCAAAYOgA==
  • Thread-topic: [PATCH] build: always use BASEDIR for xen sub-directory


> On 2 Oct 2020, at 14:17, Jan Beulich <jbeulich@xxxxxxxx> wrote:
> 
> On 02.10.2020 14:44, Bertrand Marquis wrote:
>> 
>> 
>>> On 2 Oct 2020, at 13:38, Jan Beulich <jbeulich@xxxxxxxx> wrote:
>>> 
>>> On 02.10.2020 14:34, Bertrand Marquis wrote:
>>>>> On 2 Oct 2020, at 13:12, Jan Beulich <jbeulich@xxxxxxxx> wrote:
>>>>> 
>>>>> On 02.10.2020 12:42, Bertrand Marquis wrote:
>>>>>> Modify Makefiles using $(XEN_ROOT)/xen to use $(BASEDIR) instead.
>>>>>> 
>>>>>> This is removing the dependency to xen subdirectory preventing using a
>>>>>> wrong configuration file when xen subdirectory is duplicated for
>>>>>> compilation tests.
>>>>>> 
>>>>>> Signed-off-by: Bertrand Marquis <bertrand.marquis@xxxxxxx>
>>>>> 
>>>>> Acked-by: Jan Beulich <jbeulich@xxxxxxxx>
>>>> 
>>>> Thanks :-)
>>>> 
>>>>> 
>>>>> (but more for the slight tidying than the purpose you name)
>>>> 
>>>> Feel free to remove the justification from the commit message if
>>>> you think it is not usefull.
>>> 
>>> Oh, no, it's not like I consider it not useful. It shows how you
>>> arrived at making the change. It's just that I didn't consider
>>> making copies of xen/ something we mean to be supported. I wouldn't
>>> be surprised if it got broken again ...
>> 
>> basically i do this a “cp -rs” of xen subdirectory so that i can have 
>> directories
>> in which xen is compiled for x86, arm32 and arm64 and recompile all of them
>> quickly without having to go through distclean, config, make each time or 
>> modify
>> the original tree.
> 
> But then you must have adjustments also in the top level makefile,
> such that besides "make xen", "make xen-xyz" also works.

No there i am only compiling the hypervisor, not the tools, so only xen subdir.

Bertrand


 


Rackspace

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