Re: [PATCH 06/16] docs/about/emulation: update assets for uftrace plugin documentation

[Pierrick Bouvier <pierrick.bouvier@linaro.org>]

On 2025-11-21 15:44, Philippe Mathieu-Daudé wrote:
> On 12/11/25 02:38, Akihiko Odaki wrote:
>> On 2025/11/12 2:17, Alex Bennée wrote:
>>> From: Pierrick Bouvier <pierrick.bouvier@linaro.org>
>>>
>>> Linaro is discontinuing its fileserver service by end of the year.
>>> Migrate assets to GitHub.
>>
>> I wonder why they are moved to GitHub instead of share.linaro.org. They
>> look like plain binary blobs like other files moved to share.linaro.org
>> and don't need a version control.
>>
>> If they are going to be on Git, why don't you put them on https://
>> gitlab.com/qemu-project instead? I hope creating a repository there is
>> straightforward.
>>
>>>
>>> Signed-off-by: Pierrick Bouvier <pierrick.bouvier@linaro.org>
>>> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
>>> ---
>>>    docs/about/emulation.rst | 20 ++++++++++----------
>>>    1 file changed, 10 insertions(+), 10 deletions(-)
>>>
>>> diff --git a/docs/about/emulation.rst b/docs/about/emulation.rst
>>> index 92c219119e8..4a7d1f41780 100644
>>> --- a/docs/about/emulation.rst
>>> +++ b/docs/about/emulation.rst
>>> @@ -886,24 +886,24 @@ As an example, we can trace qemu itself running
>>> git::
>>>        $ uftrace dump --chrome | gzip > ~/qemu_aarch64_git_help.json.gz
>>>    For convenience, you can download this trace
>>> `qemu_aarch64_git_help.json.gz
>>> -<https://fileserver.linaro.org/s/N8X8fnZ5yGRZLsT/download/
>>> qemu_aarch64_git_help.json.gz>`_.
>>> +<https://github.com/pbo-linaro/qemu-assets/raw/refs/heads/master/
>>> qemu-uftrace/qemu_aarch64_git_help.json.gz>`_.
>>
>> I think it's better to point to a specific commit so that e.g., files
>> can be renamed in the future. This URL will look like:
>> https://github.com/pbo-linaro/qemu-assets/blob/
>> ec68ed241bb303128537ac662d97e38972ff7257/qemu-uftrace/aarch64_boot.json.gz
> 
> Yes please.
> 

If files are renamed/moved, then we probably want to update this URL to 
new one instead of pointing on something obsolete anyway, so I'm not 
sure what is the benefit. Feel free to update this simple patch, so it 
gets pulled in next release, I don't mind the URL, as long as it's 
shipped in.