diff mbox series

[scarthgap] Tags from ARM-software repository are now in another branch

Message ID AS4PR10MB5743C880C7F7D12B9AF14DAEE45AA@AS4PR10MB5743.EURPRD10.PROD.OUTLOOK.COM
State Superseded
Delegated to: Ryan Eatmon
Headers show
Series [scarthgap] Tags from ARM-software repository are now in another branch | expand

Commit Message

Miguel Salvatierra Veleiro July 28, 2025, 12:28 p.m. UTC 
Good afternoon,

I tried to fetch the sources from recipe /meta-arago/meta-arago-extras/recipes-devtools/arm-compute-library/arm-compute-library_24.12.bb in scarthgap but the SRCREV are not found in the github.com/ARM-software/ComputeLibrary.git repository. All the tags in this repo are moved to a new branch called archived-releases. I've changed the branch to "archived-releases" parameter in the SRC_URI and now it works. How can I request to merge this change?


Miguel,


Content Classification:



Public

√

Internal



Confidential



Secret



[cid:4724218e-d2f5-4e90-9cca-ab0044278fe0]

CTAG - Centro Tecnológico de Automoción de Galicia
Polígono Industrial "A GRANXA", Calle "A“, 249-250
Porriño 36475 (Pontevedra). SPAIN.
Tfn: (+34) 986 900 300         Privacy Policy<https://ctag.com/wp-content/uploads/politica_integrada_2009.pdf>

Comments

Andrew Davis July 28, 2025, 5:10 p.m. UTC | #1 
On 7/28/25 7:28 AM, Miguel Salvatierra Veleiro via lists.yoctoproject.org wrote:
> Good afternoon,
> 
> I tried to fetch the sources from recipe /meta-arago/meta-arago-extras/recipes-devtools/arm-compute-library/arm-compute-library_24.12.bb in scarthgap but the SRCREV are not found in the github.com/ARM-software/ComputeLibrary.git repository. All the tags in this repo are moved to a new branch called archived-releases. I've changed the branch to "archived-releases" parameter in the SRC_URI and now it works. How can I request to merge this change?
> 

Hmm, so looks like ARM decided to force rebase their main branch.. The development for
arm-compute-library happens over here[0]. The github page is where they put their releases
which I assumed would make it *more* stable, and yet they go and rebase it and force push
removing commits breaking our SDK builds :(

For now, yes, your patch looks correct. To request this change, you simply need to send
it to this same mailing list (meta-arago@lists.yoctoproject.org) as a proper git format-patch
patch. Or if you would like I can make that patch and send it here for you.

Andrew

[0] https://git.mlplatform.org/ml/ComputeLibrary.git/

> diff --git a/meta-arago-extras/recipes-devtools/arm-compute-library/arm-compute-library_24.12.bb b/meta-arago-extras/recipes-devtools/arm-compute-library/arm-compute-library_24.12.bb
> index f4129a75..db376433 100644
> --- a/meta-arago-extras/recipes-devtools/arm-compute-library/arm-compute-library_24.12.bb
> +++ b/meta-arago-extras/recipes-devtools/arm-compute-library/arm-compute-library_24.12.bb
> @@ -4,7 +4,7 @@ LICENSE = "MIT"
>   LIC_FILES_CHKSUM = "file://LICENSES/MIT.txt;md5=35f8944fae972976691f3483b0ac9dba"
> 
>   SRC_URI = " \
> -    git://github.com/ARM-software/ComputeLibrary.git;branch=main;protocol=https \
> +    git://github.com/ARM-software/ComputeLibrary.git;branch=archived-releases;protocol=https \
>       file://0001-fix-Fix-indention-in-cmake-generator-script.patch \
>       file://0002-Use-ARM_COMPUTE_ENABLE_NEON-in-code-for-checking-NEO.patch \
>       file://0003-Use-ARM_COMPUTE_ENABLE_SVE-in-code-for-checking-SVE-.patch \
> 
> Miguel,
> 
> 
> Content Classification:
> 
> 
> 
> Public
> 
> √
> 
> Internal
> 
> 
> 
> Confidential
> 
> 
> 
> Secret
> 
> 
> 
> [cid:4724218e-d2f5-4e90-9cca-ab0044278fe0]
> 
> CTAG - Centro Tecnológico de Automoción de Galicia
> Polígono Industrial "A GRANXA", Calle "A“, 249-250
> Porriño 36475 (Pontevedra). SPAIN.
> Tfn: (+34) 986 900 300         Privacy Policy<https://ctag.com/wp-content/uploads/politica_integrada_2009.pdf>
> 
> 
> 
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#16325): https://lists.yoctoproject.org/g/meta-arago/message/16325
> Mute This Topic: https://lists.yoctoproject.org/mt/114414449/3619733
> Group Owner: meta-arago+owner@lists.yoctoproject.org
> Unsubscribe: https://lists.yoctoproject.org/g/meta-arago/unsub [afd@ti.com]
> -=-=-=-=-=-=-=-=-=-=-=-
> 
>
Andrew Davis July 30, 2025, 6:56 p.m. UTC | #2 
On 7/29/25 2:47 AM, Miguel Salvatierra Veleiro wrote:
> Thanks for your help,
> 
>  From 9374166114be7805d031931ff7ec12bdac2097a4 Mon Sep 17 00:00:00 2001
> From: migsal3222 <migsal3222@ctag.com>
> Date: Mon, 28 Jul 2025 11:19:32 +0000
> Subject: [PATCH] meta-arago-extras: arm-compute-library: The tag v24.12 from
>   github.com/ARM-software/ComputeLibrary.git is now in archived-releases branch
> 
> ---
>   .../arm-compute-library/arm-compute-library_24.12.bb            | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/meta-arago-extras/recipes-devtools/arm-compute-library/arm-compute-library_24.12.bb b/meta-arago-extras/recipes-devtools/arm-compute-library/arm-compute-library_24.12.bb
> index f4129a75..db376433 100644
> --- a/meta-arago-extras/recipes-devtools/arm-compute-library/arm-compute-library_24.12.bb
> +++ b/meta-arago-extras/recipes-devtools/arm-compute-library/arm-compute-library_24.12.bb
> @@ -4,7 +4,7 @@ LICENSE = "MIT"
>   LIC_FILES_CHKSUM = "file://LICENSES/MIT.txt;md5=35f8944fae972976691f3483b0ac9dba"
>   SRC_URI = " \
> -    git://github.com/ARM-software/ComputeLibrary.git;branch=main;protocol=https \
> +    git://github.com/ARM-software/ComputeLibrary.git;branch=archived-releases;protocol=https \
>       file://0001-fix-Fix-indention-in-cmake-generator-script.patch \
>       file://0002-Use-ARM_COMPUTE_ENABLE_NEON-in-code-for-checking-NEO.patch \
>       file://0003-Use-ARM_COMPUTE_ENABLE_SVE-in-code-for-checking-SVE-.patch \
> --
> 2.43.0
> 
> 

The patch doesn't seem to have been sent correctly, you should look into
"git format-patch" and "git send-email" for future patches. For this one
I've taken the liberty of re-sending your patch with the correct format[0],
hope that is okay with you, feel free to ack the patch if so.

Andrew

[0] https://lists.yoctoproject.org/g/meta-arago/message/16344

> Miguel
> 
> ------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
> *De:* Andrew Davis <afd@ti.com>
> *Enviado:* Lunes, 28 de Julio de 2025 19:10
> *Para:* Miguel Salvatierra Veleiro <miguel.salvatierra3222@ctag.com>; meta-arago@lists.yoctoproject.org <meta-arago@lists.yoctoproject.org>
> *Asunto:* Re: [meta-arago] [scarthgap][PATCH] Tags from ARM-software repository are now in another branch
> 
> [No suele recibir correo electrónico de afd@ti.com. Descubra por qué esto es importante en https://aka.ms/LearnAboutSenderIdentification <https://aka.ms/LearnAboutSenderIdentification> ]
> 
> On 7/28/25 7:28 AM, Miguel Salvatierra Veleiro via lists.yoctoproject.org wrote:
>  > Good afternoon,
>  >
>  > I tried to fetch the sources from recipe /meta-arago/meta-arago-extras/recipes-devtools/arm-compute-library/arm-compute-library_24.12.bb in scarthgap but the SRCREV are not found in the github.com/ARM-software/ComputeLibrary.git repository. All the tags in this repo are moved to a new branch called archived-releases. I've changed the branch to "archived-releases" parameter in the SRC_URI and now it works. How can I request to merge this change?
>  >
> 
> Hmm, so looks like ARM decided to force rebase their main branch.. The development for
> arm-compute-library happens over here[0]. The github page is where they put their releases
> which I assumed would make it *more* stable, and yet they go and rebase it and force push
> removing commits breaking our SDK builds :(
> 
> For now, yes, your patch looks correct. To request this change, you simply need to send
> it to this same mailing list (meta-arago@lists.yoctoproject.org) as a proper git format-patch
> patch. Or if you would like I can make that patch and send it here for you.
> 
> Andrew
> 
> [0] https://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgit.mlplatform.org%2Fml%2FComputeLibrary.git%2F&data=05%7C02%7Cmiguel.salvatierra3222%40ctag.com%7C422d822893e945881e5d08ddcdf9a2c3%7C6b2f6648df414bbb8eb7a5be90169d14%7C0%7C0%7C638893194261704887%7CUnknown%7CTWFpbGZsb3d8eyJFbXB0eU1hcGkiOnRydWUsIlYiOiIwLjAuMDAwMCIsIlAiOiJXaW4zMiIsIkFOIjoiTWFpbCIsIldUIjoyfQ%3D%3D%7C0%7C%7C%7C&sdata=rJBrDP9g2sXX5ypBGOzW80EcsHNyTbjKRbfq7ihGLOs%3D&reserved=0 <https://git.mlplatform.org/ml/ComputeLibrary.git/>
> 
>  > diff --git a/meta-arago-extras/recipes-devtools/arm-compute-library/arm-compute-library_24.12.bb b/meta-arago-extras/recipes-devtools/arm-compute-library/arm-compute-library_24.12.bb
>  > index f4129a75..db376433 100644
>  > --- a/meta-arago-extras/recipes-devtools/arm-compute-library/arm-compute-library_24.12.bb
>  > +++ b/meta-arago-extras/recipes-devtools/arm-compute-library/arm-compute-library_24.12.bb
>  > @@ -4,7 +4,7 @@ LICENSE = "MIT"
>  >   LIC_FILES_CHKSUM = "file://LICENSES/MIT.txt;md5=35f8944fae972976691f3483b0ac9dba <file://LICENSES/MIT.txt;md5=35f8944fae972976691f3483b0ac9dba>"
>  >
>  >   SRC_URI = " \
>  > -    git://github.com/ARM-software/ComputeLibrary.git;branch=main;protocol=https \
>  > +    git://github.com/ARM-software/ComputeLibrary.git;branch=archived-releases;protocol=https \
>  > file://0001-fix-Fix-indention-in-cmake-generator-script.patch <file://0001-fix-Fix-indention-in-cmake-generator-script.patch> \
>  > file://0002-Use-ARM_COMPUTE_ENABLE_NEON-in-code-for-checking-NEO.patch <file://0002-Use-ARM_COMPUTE_ENABLE_NEON-in-code-for-checking-NEO.patch> \
>  > file://0003-Use-ARM_COMPUTE_ENABLE_SVE-in-code-for-checking-SVE-.patch <file://0003-Use-ARM_COMPUTE_ENABLE_SVE-in-code-for-checking-SVE-.patch> \
>  >
>  > Miguel,
>  >
>  >
>  > Content Classification:
>  >
>  >
>  >
>  > Public
>  >
>  > √
>  >
>  > Internal
>  >
>  >
>  >
>  > Confidential
>  >
>  >
>  >
>  > Secret
>  >
>  >
>  >
>  > [cid:4724218e-d2f5-4e90-9cca-ab0044278fe0]
>  >
>  > CTAG - Centro Tecnológico de Automoción de Galicia
>  > Polígono Industrial "A GRANXA", Calle "A“, 249-250
>  > Porriño 36475 (Pontevedra). SPAIN.
>  > Tfn: (+34) 986 900 300         Privacy Policy<https://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fctag.com%2Fwp-content%2Fuploads%2Fpolitica_integrada_2009.pdf&data=05%7C02%7Cmiguel.salvatierra3222%40ctag.com%7C422d822893e945881e5d08ddcdf9a2c3%7C6b2f6648df414bbb8eb7a5be90169d14%7C0%7C0%7C638893194261734619%7CUnknown%7CTWFpbGZsb3d8eyJFbXB0eU1hcGkiOnRydWUsIlYiOiIwLjAuMDAwMCIsIlAiOiJXaW4zMiIsIkFOIjoiTWFpbCIsIldUIjoyfQ%3D%3D%7C0%7C%7C%7C&sdata=bF3WFAABVjirrL3ELQtJwhcvtu3T49kLeHKafPmmRJc%3D&reserved=0 <https://ctag.com/wp-content/uploads/politica_integrada_2009.pdf>>
>  >
>  >
>  >
>  > -=-=-=-=-=-=-=-=-=-=-=-
>  > Links: You receive all messages sent to this group.
>  > View/Reply Online (#16325): https://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Flists.yoctoproject.org%2Fg%2Fmeta-arago%2Fmessage%2F16325&data=05%7C02%7Cmiguel.salvatierra3222%40ctag.com%7C422d822893e945881e5d08ddcdf9a2c3%7C6b2f6648df414bbb8eb7a5be90169d14%7C0%7C0%7C638893194261752527%7CUnknown%7CTWFpbGZsb3d8eyJFbXB0eU1hcGkiOnRydWUsIlYiOiIwLjAuMDAwMCIsIlAiOiJXaW4zMiIsIkFOIjoiTWFpbCIsIldUIjoyfQ%3D%3D%7C0%7C%7C%7C&sdata=%2BxG0FXj6XlJ0PgtZDDxJMi1H8NlGYEp6mMBivL6a%2BYA%3D&reserved=0 <https://lists.yoctoproject.org/g/meta-arago/message/16325>
>  > Mute This Topic: https://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Flists.yoctoproject.org%2Fmt%2F114414449%2F3619733&data=05%7C02%7Cmiguel.salvatierra3222%40ctag.com%7C422d822893e945881e5d08ddcdf9a2c3%7C6b2f6648df414bbb8eb7a5be90169d14%7C0%7C0%7C638893194261766633%7CUnknown%7CTWFpbGZsb3d8eyJFbXB0eU1hcGkiOnRydWUsIlYiOiIwLjAuMDAwMCIsIlAiOiJXaW4zMiIsIkFOIjoiTWFpbCIsIldUIjoyfQ%3D%3D%7C0%7C%7C%7C&sdata=wEOkK5EUuwd7jz5KFMcaSQ2pPwiMI6xSv49MXDf5V2s%3D&reserved=0 <https://lists.yoctoproject.org/mt/114414449/3619733>
>  > Group Owner: meta-arago+owner@lists.yoctoproject.org
>  > Unsubscribe: https://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Flists.yoctoproject.org%2Fg%2Fmeta-arago%2Funsub&data=05%7C02%7Cmiguel.salvatierra3222%40ctag.com%7C422d822893e945881e5d08ddcdf9a2c3%7C6b2f6648df414bbb8eb7a5be90169d14%7C0%7C0%7C638893194261779917%7CUnknown%7CTWFpbGZsb3d8eyJFbXB0eU1hcGkiOnRydWUsIlYiOiIwLjAuMDAwMCIsIlAiOiJXaW4zMiIsIkFOIjoiTWFpbCIsIldUIjoyfQ%3D%3D%7C0%7C%7C%7C&sdata=hb5xpoIY%2FeruEZn1YmFlpMANome9jAeQvCkcEc2Sang%3D&reserved=0 <https://lists.yoctoproject.org/g/meta-arago/unsub> [afd@ti.com]
>  > -=-=-=-=-=-=-=-=-=-=-=-
>  >
>  >
> 
> ⚠️ PRECAUCIÓN: Este correo se originó fuera de CTAG.
> No haga clic en enlaces ni abra archivos adjuntos a menos que reconozca al remitente y confíe en su contenido.
diff mbox series

Patch

diff --git a/meta-arago-extras/recipes-devtools/arm-compute-library/arm-compute-library_24.12.bb b/meta-arago-extras/recipes-devtools/arm-compute-library/arm-compute-library_24.12.bb
index f4129a75..db376433 100644
--- a/meta-arago-extras/recipes-devtools/arm-compute-library/arm-compute-library_24.12.bb
+++ b/meta-arago-extras/recipes-devtools/arm-compute-library/arm-compute-library_24.12.bb
@@ -4,7 +4,7 @@  LICENSE = "MIT"
 LIC_FILES_CHKSUM = "file://LICENSES/MIT.txt;md5=35f8944fae972976691f3483b0ac9dba"

 SRC_URI = " \
-    git://github.com/ARM-software/ComputeLibrary.git;branch=main;protocol=https \
+    git://github.com/ARM-software/ComputeLibrary.git;branch=archived-releases;protocol=https \
     file://0001-fix-Fix-indention-in-cmake-generator-script.patch \
     file://0002-Use-ARM_COMPUTE_ENABLE_NEON-in-code-for-checking-NEO.patch \
     file://0003-Use-ARM_COMPUTE_ENABLE_SVE-in-code-for-checking-SVE-.patch \