[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Proposal: List files in Xen originated from external sources
- To: Jan Beulich <jbeulich@xxxxxxxx>
- From: Luca Fancellu <Luca.Fancellu@xxxxxxx>
- Date: Mon, 6 Feb 2023 12:48:17 +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-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=Uq5ePRHFEkRDZNX3ott0OvteXRN/5GjcPlXRQpT5hAE=; b=n50dUs4cdo5KPR5QB46F1CgFLisASSvB1ogmJeI5LphEgG+hn2VktB+cJdfs7di4UzNMKgYenHI8Dw/423uiSQJi8BUP/WSS5mBrieqYpU0gUmpi2xZhizPVMRnFjaNRvyol/bclJTW0ENYCkmN1pRaIKrq/IvSfYCk26vDFl1+VjX7U4QTHcoC49Cwvz3UbNcUrR3i7ZJHlCMrkoOUMmeSHT7uUWSMUS19kFPe3/qFE3s8DWoRnXt+Ag8BS5b9A4ocWsISQhD9G8dF7/cdVRQy7qJpAJyO/5CFq4/dENZJf9G9pHyUtVeISKxzvCxEJb9q+sXPqimg1uIaUXFNaTw==
- Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=eC1pAYoe1DW/4DEsyXyJla87fP7GkwcuejV1Iqxy2c5/ZJJ0gKTvSBgZ/d9OgD0oi6T6rRWL4MqDmxm6SdcE+MTnjtvdDAKc0/k0Lc0QEZlrhUDsnYF/8n1nlPlONxsMWJz7klGrptyFiaViTId2zfLQ0QMtvx4Wu38FFyvbx8z0jV1cylCJKGyCK5RaxMXZfH94K2oJ4OGZ5/HA5fQiHFP/1FxwOwlr17mjVT8Mhq8tG5NI/Gv/Nff6vqoi9wU3T3UdJxdQp8di+TXM7vOpUV9mhhaQTNO1STUE56XQg3AvoHWGcd9F66ctlTZ1cpCBgn4nghNah7QFLHhqINi4mQ==
- Authentication-results-original: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=arm.com;
- Cc: Stefano Stabellini <sstabellini@xxxxxxxxxx>, Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, George Dunlap <george.dunlap@xxxxxxxxxx>, Julien Grall <julien@xxxxxxx>, Xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxxx>, Bertrand Marquis <Bertrand.Marquis@xxxxxxx>
- Delivery-date: Mon, 06 Feb 2023 12:48:52 +0000
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
- Nodisclaimer: true
- Original-authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=arm.com;
- Thread-index: AQHZN7IkxuQrtwTv+0WBDKkxqc3Ztq7BtLQAgAAnBYCAAASlAIAAAsKA
- Thread-topic: Proposal: List files in Xen originated from external sources
+ adding Bertrand in the CC, apologies for forgetting you in the first mail
> On 6 Feb 2023, at 12:38, Jan Beulich <jbeulich@xxxxxxxx> wrote:
>
> On 06.02.2023 13:21, Luca Fancellu wrote:
>>> On 6 Feb 2023, at 10:01, Jan Beulich <jbeulich@xxxxxxxx> wrote:
>>> On 03.02.2023 10:30, Luca Fancellu wrote:
>>>> origin_path: relative path in the original project, mostly linked to the
>>>> original_revision field
>>>>
>>>> origin_revision: revision of the changes in the repository when this file
>>>> was taken.
>>>
>>> I guess this needs clarifying: What revision is meant here? The one
>>> originally
>>> imported, the one last updated from, or yet something else? Keep in mind
>>> that
>>> we may also be selectively importing changes, in which case any particular
>>> "revision" can easily end up misleading. The format likely also wants
>>> specifying, e.g. (like we do for Fixes: tags) the first 12 digits of a
>>> commit
>>> hash. For said selective updating this might then allow for something like
>>> <base-hash>+<cherry-pick>+...
>>
>> This is a good point, what would it be the best format in your opinion?
>> Maybe we should have the field as an array of revisions?
>>
>> "origin_revision": [
>> "Revision of latest backport”,
>> [...]
>> "originally imported revision”,
>> ]
>>
>> What should be the best to capture the history of the file?
>
> That would be taking the 2nd step before the 1st one. For now we need to
> determine _what_ information we want to record. Then we can determine how
> to best represent it.
Yes you are right, what informations makes sense to track for a file, my first
thought
is clearly the revision where the file was taken, at first I thought that
updating the revision
to the latest one back ported would have been enough, but from your comment I
see that
it's not so simple.
>
> Jan
|
