diff mbox series

[meta-networking] meta-networking: mbedtls: Remove tag from SRC_URI

Message ID 20250814225042.3310913-1-d.schultz@phytec.de
State New
Headers show
Series [meta-networking] meta-networking: mbedtls: Remove tag from SRC_URI | expand

Commit Message

Daniel Schultz Aug. 14, 2025, 10:50 p.m. UTC
This recipe uses SRCREV as well as defines a tag in the SRC_URI.
Only use SRCREV since this will generate an error.

Fix following error:
  [...] FetchError: Fetcher failure: Conflicting revisions
  (c765c831e5c2a0971410692f92f7a81d6ec65ec2 from SRCREV and v3.6.4
  from the url) found, please specify one valid value

Signed-off-by: Daniel Schultz <d.schultz@phytec.de>
---
 meta-networking/recipes-connectivity/mbedtls/mbedtls_3.6.4.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Khem Raj Aug. 15, 2025, 2:06 a.m. UTC | #1
I wonder why it works on autobuilder. Can you try with bitbake -ccleanall ?
and see if fetching works. I can see the SRCREV is same as v3.6.4 tag.

On Thu, Aug 14, 2025 at 3:51 PM Daniel Schultz via
lists.openembedded.org <d.schultz=phytec.de@lists.openembedded.org>
wrote:
>
> This recipe uses SRCREV as well as defines a tag in the SRC_URI.
> Only use SRCREV since this will generate an error.
>
> Fix following error:
>   [...] FetchError: Fetcher failure: Conflicting revisions
>   (c765c831e5c2a0971410692f92f7a81d6ec65ec2 from SRCREV and v3.6.4
>   from the url) found, please specify one valid value
>
> Signed-off-by: Daniel Schultz <d.schultz@phytec.de>
> ---
>  meta-networking/recipes-connectivity/mbedtls/mbedtls_3.6.4.bb | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/meta-networking/recipes-connectivity/mbedtls/mbedtls_3.6.4.bb b/meta-networking/recipes-connectivity/mbedtls/mbedtls_3.6.4.bb
> index 88b50a406b..a0dc6a689a 100644
> --- a/meta-networking/recipes-connectivity/mbedtls/mbedtls_3.6.4.bb
> +++ b/meta-networking/recipes-connectivity/mbedtls/mbedtls_3.6.4.bb
> @@ -23,7 +23,7 @@ LIC_FILES_CHKSUM = "file://LICENSE;md5=379d5819937a6c2f1ef1630d341e026d"
>  SECTION = "libs"
>
>  S = "${WORKDIR}/git"
> -SRC_URI = "gitsm://github.com/Mbed-TLS/mbedtls.git;protocol=https;branch=mbedtls-3.6;tag=v${PV} \
> +SRC_URI = "gitsm://github.com/Mbed-TLS/mbedtls.git;protocol=https;branch=mbedtls-3.6 \
>             file://run-ptest \
>             "
>
> --
> 2.25.1
>
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#118951): https://lists.openembedded.org/g/openembedded-devel/message/118951
> Mute This Topic: https://lists.openembedded.org/mt/114709547/1997914
> Group Owner: openembedded-devel+owner@lists.openembedded.org
> Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub [raj.khem@gmail.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>
Peter Marko Aug. 15, 2025, 5:50 a.m. UTC | #2
That error is shown only when ";rev=${PV}" is specified (which is not the case in this recipe) or a layer modifies something.
Fetching works for me without this patch just fine.

> -----Original Message-----
> From: openembedded-devel@lists.openembedded.org <openembedded-
> devel@lists.openembedded.org> On Behalf Of Khem Raj via
> lists.openembedded.org
> Sent: Friday, August 15, 2025 4:06
> To: d.schultz@phytec.de
> Cc: openembedded-devel@lists.openembedded.org; upstream@lists.phytec.de
> Subject: Re: [oe] [meta-networking][PATCH] meta-networking: mbedtls: Remove
> tag from SRC_URI
> 
> I wonder why it works on autobuilder. Can you try with bitbake -ccleanall ?
> and see if fetching works. I can see the SRCREV is same as v3.6.4 tag.
> 
> On Thu, Aug 14, 2025 at 3:51 PM Daniel Schultz via
> lists.openembedded.org <d.schultz=phytec.de@lists.openembedded.org>
> wrote:
> >
> > This recipe uses SRCREV as well as defines a tag in the SRC_URI.
> > Only use SRCREV since this will generate an error.
> >
> > Fix following error:
> >   [...] FetchError: Fetcher failure: Conflicting revisions
> >   (c765c831e5c2a0971410692f92f7a81d6ec65ec2 from SRCREV and v3.6.4
> >   from the url) found, please specify one valid value
> >
> > Signed-off-by: Daniel Schultz <d.schultz@phytec.de>
> > ---
> >  meta-networking/recipes-connectivity/mbedtls/mbedtls_3.6.4.bb | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/meta-networking/recipes-connectivity/mbedtls/mbedtls_3.6.4.bb
> b/meta-networking/recipes-connectivity/mbedtls/mbedtls_3.6.4.bb
> > index 88b50a406b..a0dc6a689a 100644
> > --- a/meta-networking/recipes-connectivity/mbedtls/mbedtls_3.6.4.bb
> > +++ b/meta-networking/recipes-connectivity/mbedtls/mbedtls_3.6.4.bb
> > @@ -23,7 +23,7 @@ LIC_FILES_CHKSUM =
> "file://LICENSE;md5=379d5819937a6c2f1ef1630d341e026d"
> >  SECTION = "libs"
> >
> >  S = "${WORKDIR}/git"
> > -SRC_URI = "gitsm://github.com/Mbed-
> TLS/mbedtls.git;protocol=https;branch=mbedtls-3.6;tag=v${PV} \
> > +SRC_URI = "gitsm://github.com/Mbed-
> TLS/mbedtls.git;protocol=https;branch=mbedtls-3.6 \
> >             file://run-ptest \
> >             "
> >
> > --
> > 2.25.1
> >
> >
> >
> >
Gyorgy Sarvari Aug. 15, 2025, 5:53 a.m. UTC | #3
On 8/15/25 04:06, Khem Raj via lists.openembedded.org wrote:
> I wonder why it works on autobuilder. Can you try with bitbake -ccleanall ?
> and see if fetching works. I can see the SRCREV is same as v3.6.4 tag.

Was this patch meant for scarthgap, where mbedtls got just broken due to
this rev vs tag issue?


> On Thu, Aug 14, 2025 at 3:51 PM Daniel Schultz via
> lists.openembedded.org <d.schultz=phytec.de@lists.openembedded.org>
> wrote:
>> This recipe uses SRCREV as well as defines a tag in the SRC_URI.
>> Only use SRCREV since this will generate an error.
>>
>> Fix following error:
>>   [...] FetchError: Fetcher failure: Conflicting revisions
>>   (c765c831e5c2a0971410692f92f7a81d6ec65ec2 from SRCREV and v3.6.4
>>   from the url) found, please specify one valid value
>>
>> Signed-off-by: Daniel Schultz <d.schultz@phytec.de>
>> ---
>>  meta-networking/recipes-connectivity/mbedtls/mbedtls_3.6.4.bb | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/meta-networking/recipes-connectivity/mbedtls/mbedtls_3.6.4.bb b/meta-networking/recipes-connectivity/mbedtls/mbedtls_3.6.4.bb
>> index 88b50a406b..a0dc6a689a 100644
>> --- a/meta-networking/recipes-connectivity/mbedtls/mbedtls_3.6.4.bb
>> +++ b/meta-networking/recipes-connectivity/mbedtls/mbedtls_3.6.4.bb
>> @@ -23,7 +23,7 @@ LIC_FILES_CHKSUM = "file://LICENSE;md5=379d5819937a6c2f1ef1630d341e026d"
>>  SECTION = "libs"
>>
>>  S = "${WORKDIR}/git"
>> -SRC_URI = "gitsm://github.com/Mbed-TLS/mbedtls.git;protocol=https;branch=mbedtls-3.6;tag=v${PV} \
>> +SRC_URI = "gitsm://github.com/Mbed-TLS/mbedtls.git;protocol=https;branch=mbedtls-3.6 \
>>             file://run-ptest \
>>             "
>>
>> --
>> 2.25.1
>>
>>
>>
>>
>>
>> -=-=-=-=-=-=-=-=-=-=-=-
>> Links: You receive all messages sent to this group.
>> View/Reply Online (#118953): https://lists.openembedded.org/g/openembedded-devel/message/118953
>> Mute This Topic: https://lists.openembedded.org/mt/114709547/6084445
>> Group Owner: openembedded-devel+owner@lists.openembedded.org
>> Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub [skandigraun@gmail.com]
>> -=-=-=-=-=-=-=-=-=-=-=-
>>
Alexander Kanavin Aug. 15, 2025, 9:08 a.m. UTC | #4
Same as the other proposed patch. I would not take it before there is
an explanation of why the issue suddenly appeared. Did something
change in scarthgap, or is there a mismatch between the tag and
revision?

Alex

On Fri, 15 Aug 2025 at 07:53, Gyorgy Sarvari via
lists.openembedded.org <skandigraun=gmail.com@lists.openembedded.org>
wrote:
>
> On 8/15/25 04:06, Khem Raj via lists.openembedded.org wrote:
> > I wonder why it works on autobuilder. Can you try with bitbake -ccleanall ?
> > and see if fetching works. I can see the SRCREV is same as v3.6.4 tag.
>
> Was this patch meant for scarthgap, where mbedtls got just broken due to
> this rev vs tag issue?
>
>
> > On Thu, Aug 14, 2025 at 3:51 PM Daniel Schultz via
> > lists.openembedded.org <d.schultz=phytec.de@lists.openembedded.org>
> > wrote:
> >> This recipe uses SRCREV as well as defines a tag in the SRC_URI.
> >> Only use SRCREV since this will generate an error.
> >>
> >> Fix following error:
> >>   [...] FetchError: Fetcher failure: Conflicting revisions
> >>   (c765c831e5c2a0971410692f92f7a81d6ec65ec2 from SRCREV and v3.6.4
> >>   from the url) found, please specify one valid value
> >>
> >> Signed-off-by: Daniel Schultz <d.schultz@phytec.de>
> >> ---
> >>  meta-networking/recipes-connectivity/mbedtls/mbedtls_3.6.4.bb | 2 +-
> >>  1 file changed, 1 insertion(+), 1 deletion(-)
> >>
> >> diff --git a/meta-networking/recipes-connectivity/mbedtls/mbedtls_3.6.4.bb b/meta-networking/recipes-connectivity/mbedtls/mbedtls_3.6.4.bb
> >> index 88b50a406b..a0dc6a689a 100644
> >> --- a/meta-networking/recipes-connectivity/mbedtls/mbedtls_3.6.4.bb
> >> +++ b/meta-networking/recipes-connectivity/mbedtls/mbedtls_3.6.4.bb
> >> @@ -23,7 +23,7 @@ LIC_FILES_CHKSUM = "file://LICENSE;md5=379d5819937a6c2f1ef1630d341e026d"
> >>  SECTION = "libs"
> >>
> >>  S = "${WORKDIR}/git"
> >> -SRC_URI = "gitsm://github.com/Mbed-TLS/mbedtls.git;protocol=https;branch=mbedtls-3.6;tag=v${PV} \
> >> +SRC_URI = "gitsm://github.com/Mbed-TLS/mbedtls.git;protocol=https;branch=mbedtls-3.6 \
> >>             file://run-ptest \
> >>             "
> >>
> >> --
> >> 2.25.1
> >>
> >>
> >>
> >>
> >>
> >>
> >>
>
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#118958): https://lists.openembedded.org/g/openembedded-devel/message/118958
> Mute This Topic: https://lists.openembedded.org/mt/114709547/1686489
> Group Owner: openembedded-devel+owner@lists.openembedded.org
> Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub [alex.kanavin@gmail.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>
Gyorgy Sarvari Aug. 15, 2025, 9:24 a.m. UTC | #5
On 8/15/25 11:08, Alexander Kanavin wrote:
> Same as the other proposed patch. I would not take it before there is
> an explanation of why the issue suddenly appeared. Did something
> change in scarthgap, or is there a mismatch between the tag and
> revision?

The tag+revision matching was introduced in walnascar, before that it
was either/or. This patch got into scarthgap, which uses this:
https://lists.openembedded.org/g/openembedded-devel/message/118611

>
> Alex
>
> On Fri, 15 Aug 2025 at 07:53, Gyorgy Sarvari via
> lists.openembedded.org <skandigraun=gmail.com@lists.openembedded.org>
> wrote:
>> On 8/15/25 04:06, Khem Raj via lists.openembedded.org wrote:
>>> I wonder why it works on autobuilder. Can you try with bitbake -ccleanall ?
>>> and see if fetching works. I can see the SRCREV is same as v3.6.4 tag.
>> Was this patch meant for scarthgap, where mbedtls got just broken due to
>> this rev vs tag issue?
>>
>>
>>> On Thu, Aug 14, 2025 at 3:51 PM Daniel Schultz via
>>> lists.openembedded.org <d.schultz=phytec.de@lists.openembedded.org>
>>> wrote:
>>>> This recipe uses SRCREV as well as defines a tag in the SRC_URI.
>>>> Only use SRCREV since this will generate an error.
>>>>
>>>> Fix following error:
>>>>   [...] FetchError: Fetcher failure: Conflicting revisions
>>>>   (c765c831e5c2a0971410692f92f7a81d6ec65ec2 from SRCREV and v3.6.4
>>>>   from the url) found, please specify one valid value
>>>>
>>>> Signed-off-by: Daniel Schultz <d.schultz@phytec.de>
>>>> ---
>>>>  meta-networking/recipes-connectivity/mbedtls/mbedtls_3.6.4.bb | 2 +-
>>>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>>>
>>>> diff --git a/meta-networking/recipes-connectivity/mbedtls/mbedtls_3.6.4.bb b/meta-networking/recipes-connectivity/mbedtls/mbedtls_3.6.4.bb
>>>> index 88b50a406b..a0dc6a689a 100644
>>>> --- a/meta-networking/recipes-connectivity/mbedtls/mbedtls_3.6.4.bb
>>>> +++ b/meta-networking/recipes-connectivity/mbedtls/mbedtls_3.6.4.bb
>>>> @@ -23,7 +23,7 @@ LIC_FILES_CHKSUM = "file://LICENSE;md5=379d5819937a6c2f1ef1630d341e026d"
>>>>  SECTION = "libs"
>>>>
>>>>  S = "${WORKDIR}/git"
>>>> -SRC_URI = "gitsm://github.com/Mbed-TLS/mbedtls.git;protocol=https;branch=mbedtls-3.6;tag=v${PV} \
>>>> +SRC_URI = "gitsm://github.com/Mbed-TLS/mbedtls.git;protocol=https;branch=mbedtls-3.6 \
>>>>             file://run-ptest \
>>>>             "
>>>>
>>>> --
>>>> 2.25.1
>>>>
>>>>
>>>>
>>>>
>>>>
>>>>
>>>>
>>
>> -=-=-=-=-=-=-=-=-=-=-=-
>> Links: You receive all messages sent to this group.
>> View/Reply Online (#118958): https://lists.openembedded.org/g/openembedded-devel/message/118958
>> Mute This Topic: https://lists.openembedded.org/mt/114709547/1686489
>> Group Owner: openembedded-devel+owner@lists.openembedded.org
>> Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub [alex.kanavin@gmail.com]
>> -=-=-=-=-=-=-=-=-=-=-=-
>>
Daniel Schultz Aug. 15, 2025, 10:55 p.m. UTC | #6
Still the same error. I can also see this issue in our nightly builds 
and they're fetching from scratch.

- Daniel

On 8/14/25 19:06, Khem Raj wrote:
> I wonder why it works on autobuilder. Can you try with bitbake -ccleanall ?
> and see if fetching works. I can see the SRCREV is same as v3.6.4 tag.
>
> On Thu, Aug 14, 2025 at 3:51 PM Daniel Schultz via
> lists.openembedded.org <d.schultz=phytec.de@lists.openembedded.org>
> wrote:
>> This recipe uses SRCREV as well as defines a tag in the SRC_URI.
>> Only use SRCREV since this will generate an error.
>>
>> Fix following error:
>>    [...] FetchError: Fetcher failure: Conflicting revisions
>>    (c765c831e5c2a0971410692f92f7a81d6ec65ec2 from SRCREV and v3.6.4
>>    from the url) found, please specify one valid value
>>
>> Signed-off-by: Daniel Schultz <d.schultz@phytec.de>
>> ---
>>   meta-networking/recipes-connectivity/mbedtls/mbedtls_3.6.4.bb | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/meta-networking/recipes-connectivity/mbedtls/mbedtls_3.6.4.bb b/meta-networking/recipes-connectivity/mbedtls/mbedtls_3.6.4.bb
>> index 88b50a406b..a0dc6a689a 100644
>> --- a/meta-networking/recipes-connectivity/mbedtls/mbedtls_3.6.4.bb
>> +++ b/meta-networking/recipes-connectivity/mbedtls/mbedtls_3.6.4.bb
>> @@ -23,7 +23,7 @@ LIC_FILES_CHKSUM = "file://LICENSE;md5=379d5819937a6c2f1ef1630d341e026d"
>>   SECTION = "libs"
>>
>>   S = "${WORKDIR}/git"
>> -SRC_URI = "gitsm://github.com/Mbed-TLS/mbedtls.git;protocol=https;branch=mbedtls-3.6;tag=v${PV} \
>> +SRC_URI = "gitsm://github.com/Mbed-TLS/mbedtls.git;protocol=https;branch=mbedtls-3.6 \
>>              file://run-ptest \
>>              "
>>
>> --
>> 2.25.1
>>
>>
>> -=-=-=-=-=-=-=-=-=-=-=-
>> Links: You receive all messages sent to this group.
>> View/Reply Online (#118951): https://lists.openembedded.org/g/openembedded-devel/message/118951
>> Mute This Topic: https://lists.openembedded.org/mt/114709547/1997914
>> Group Owner: openembedded-devel+owner@lists.openembedded.org
>> Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub [raj.khem@gmail.com]
>> -=-=-=-=-=-=-=-=-=-=-=-
>>
Daniel Schultz Aug. 15, 2025, 10:55 p.m. UTC | #7
we're also using meta-arm and apparently TF-A is also fetching mbedtls. 
Maybe it's somehow related to that layer?

- Daniel

On 8/14/25 22:50, Marko, Peter wrote:
> That error is shown only when ";rev=${PV}" is specified (which is not the case in this recipe) or a layer modifies something.
> Fetching works for me without this patch just fine.
>
>> -----Original Message-----
>> From: openembedded-devel@lists.openembedded.org <openembedded-
>> devel@lists.openembedded.org> On Behalf Of Khem Raj via
>> lists.openembedded.org
>> Sent: Friday, August 15, 2025 4:06
>> To: d.schultz@phytec.de
>> Cc: openembedded-devel@lists.openembedded.org; upstream@lists.phytec.de
>> Subject: Re: [oe] [meta-networking][PATCH] meta-networking: mbedtls: Remove
>> tag from SRC_URI
>>
>> I wonder why it works on autobuilder. Can you try with bitbake -ccleanall ?
>> and see if fetching works. I can see the SRCREV is same as v3.6.4 tag.
>>
>> On Thu, Aug 14, 2025 at 3:51 PM Daniel Schultz via
>> lists.openembedded.org <d.schultz=phytec.de@lists.openembedded.org>
>> wrote:
>>> This recipe uses SRCREV as well as defines a tag in the SRC_URI.
>>> Only use SRCREV since this will generate an error.
>>>
>>> Fix following error:
>>>    [...] FetchError: Fetcher failure: Conflicting revisions
>>>    (c765c831e5c2a0971410692f92f7a81d6ec65ec2 from SRCREV and v3.6.4
>>>    from the url) found, please specify one valid value
>>>
>>> Signed-off-by: Daniel Schultz <d.schultz@phytec.de>
>>> ---
>>>   meta-networking/recipes-connectivity/mbedtls/mbedtls_3.6.4.bb | 2 +-
>>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/meta-networking/recipes-connectivity/mbedtls/mbedtls_3.6.4.bb
>> b/meta-networking/recipes-connectivity/mbedtls/mbedtls_3.6.4.bb
>>> index 88b50a406b..a0dc6a689a 100644
>>> --- a/meta-networking/recipes-connectivity/mbedtls/mbedtls_3.6.4.bb
>>> +++ b/meta-networking/recipes-connectivity/mbedtls/mbedtls_3.6.4.bb
>>> @@ -23,7 +23,7 @@ LIC_FILES_CHKSUM =
>> "file://LICENSE;md5=379d5819937a6c2f1ef1630d341e026d"
>>>   SECTION = "libs"
>>>
>>>   S = "${WORKDIR}/git"
>>> -SRC_URI = "gitsm://github.com/Mbed-
>> TLS/mbedtls.git;protocol=https;branch=mbedtls-3.6;tag=v${PV} \
>>> +SRC_URI = "gitsm://github.com/Mbed-
>> TLS/mbedtls.git;protocol=https;branch=mbedtls-3.6 \
>>>              file://run-ptest \
>>>              "
>>>
>>> --
>>> 2.25.1
>>>
>>>
>>>
>>>
diff mbox series

Patch

diff --git a/meta-networking/recipes-connectivity/mbedtls/mbedtls_3.6.4.bb b/meta-networking/recipes-connectivity/mbedtls/mbedtls_3.6.4.bb
index 88b50a406b..a0dc6a689a 100644
--- a/meta-networking/recipes-connectivity/mbedtls/mbedtls_3.6.4.bb
+++ b/meta-networking/recipes-connectivity/mbedtls/mbedtls_3.6.4.bb
@@ -23,7 +23,7 @@  LIC_FILES_CHKSUM = "file://LICENSE;md5=379d5819937a6c2f1ef1630d341e026d"
 SECTION = "libs"
 
 S = "${WORKDIR}/git"
-SRC_URI = "gitsm://github.com/Mbed-TLS/mbedtls.git;protocol=https;branch=mbedtls-3.6;tag=v${PV} \
+SRC_URI = "gitsm://github.com/Mbed-TLS/mbedtls.git;protocol=https;branch=mbedtls-3.6 \
            file://run-ptest \
            "