[meta-rockchip][PATCH v2] linux-torvalds-next: use tag

[meta-rockchip][PATCH v2] linux-torvalds-next: use tag

[Trevor Woerner]

Use the tag name specifically; note that it is no longer reachable from any
branch.

Signed-off-by: Trevor Woerner <twoerner@gmail.com>
---
 recipes-kernel/linux/linux-torvalds-next_git.bb | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/recipes-kernel/linux/linux-torvalds-next_git.bb b/recipes-kernel/linux/linux-torvalds-next_git.bb
index 9a95eb5bc4d8..1369ba1d707f 100644
--- a/recipes-kernel/linux/linux-torvalds-next_git.bb
+++ b/recipes-kernel/linux/linux-torvalds-next_git.bb
@@ -10,13 +10,12 @@ COMPATIBLE_MACHINE = "^$"
 COMPATIBLE_MACHINE:radxa-zero-3 = "radxa-zero-3"
 
 LINUX_VERSION = "6.10-rc3"
+KERNEL_VERSION_SANITY_SKIP = "1"
 PV = "${LINUX_VERSION}+git${SRCPV}"
 SRC_URI = " \
-	git://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git;protocol=https;branch=master \
+	git://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git;protocol=https;nobranch=1;tag=next-20240611 \
 	file://rockchip-kmeta;type=kmeta;name=rockchip-kmeta;destsuffix=rockchip-kmeta \
 	"
-# this is tag 'next-20240611'
-SRCREV = "a957267fa7e9159d3d2ee1421359ebf228570c68"
 
 inherit kernel
 inherit kernel-yocto
-- 
2.44.0.478.g7774cfed6261

Re: [yocto-patches] [meta-rockchip][PATCH v2] linux-torvalds-next: use tag

[Quentin Schulz]

Hi Trevor,

On 6/18/24 7:16 PM, Trevor Woerner via lists.yoctoproject.org wrote:
> Use the tag name specifically; note that it is no longer reachable from any
> branch.
> 
> Signed-off-by: Trevor Woerner <twoerner@gmail.com>
> ---
>   recipes-kernel/linux/linux-torvalds-next_git.bb | 5 ++---
>   1 file changed, 2 insertions(+), 3 deletions(-)
> 
> diff --git a/recipes-kernel/linux/linux-torvalds-next_git.bb b/recipes-kernel/linux/linux-torvalds-next_git.bb
> index 9a95eb5bc4d8..1369ba1d707f 100644
> --- a/recipes-kernel/linux/linux-torvalds-next_git.bb
> +++ b/recipes-kernel/linux/linux-torvalds-next_git.bb
> @@ -10,13 +10,12 @@ COMPATIBLE_MACHINE = "^$"
>   COMPATIBLE_MACHINE:radxa-zero-3 = "radxa-zero-3"
>   
>   LINUX_VERSION = "6.10-rc3"
> +KERNEL_VERSION_SANITY_SKIP = "1"
>   PV = "${LINUX_VERSION}+git${SRCPV}"
>   SRC_URI = " \
> -	git://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git;protocol=https;branch=master \
> +	git://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git;protocol=https;nobranch=1;tag=next-20240611 \
>   	file://rockchip-kmeta;type=kmeta;name=rockchip-kmeta;destsuffix=rockchip-kmeta \
>   	"
> -# this is tag 'next-20240611'
> -SRCREV = "a957267fa7e9159d3d2ee1421359ebf228570c68"
>   

Reviewed-by: Quentin Schulz <quentin.schulz@cherry.de>

Thanks!
Quentin

Re: [yocto-patches] [meta-rockchip][PATCH v2] linux-torvalds-next: use tag

[Martin Jansa]

Why is this better? Now bitbake will always run git ls-remote to
covert next-20240611 into a957267fa7e9159d3d2ee1421359ebf228570c68 and
break parsing this recipe without network access.

On Wed, Jun 19, 2024 at 10:59 AM Quentin Schulz via
lists.yoctoproject.org
<quentin.schulz=cherry.de@lists.yoctoproject.org> wrote:
>
> Hi Trevor,
>
> On 6/18/24 7:16 PM, Trevor Woerner via lists.yoctoproject.org wrote:
> > Use the tag name specifically; note that it is no longer reachable from any
> > branch.
> >
> > Signed-off-by: Trevor Woerner <twoerner@gmail.com>
> > ---
> >   recipes-kernel/linux/linux-torvalds-next_git.bb | 5 ++---
> >   1 file changed, 2 insertions(+), 3 deletions(-)
> >
> > diff --git a/recipes-kernel/linux/linux-torvalds-next_git.bb b/recipes-kernel/linux/linux-torvalds-next_git.bb
> > index 9a95eb5bc4d8..1369ba1d707f 100644
> > --- a/recipes-kernel/linux/linux-torvalds-next_git.bb
> > +++ b/recipes-kernel/linux/linux-torvalds-next_git.bb
> > @@ -10,13 +10,12 @@ COMPATIBLE_MACHINE = "^$"
> >   COMPATIBLE_MACHINE:radxa-zero-3 = "radxa-zero-3"
> >
> >   LINUX_VERSION = "6.10-rc3"
> > +KERNEL_VERSION_SANITY_SKIP = "1"
> >   PV = "${LINUX_VERSION}+git${SRCPV}"
> >   SRC_URI = " \
> > -     git://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git;protocol=https;branch=master \
> > +     git://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git;protocol=https;nobranch=1;tag=next-20240611 \
> >       file://rockchip-kmeta;type=kmeta;name=rockchip-kmeta;destsuffix=rockchip-kmeta \
> >       "
> > -# this is tag 'next-20240611'
> > -SRCREV = "a957267fa7e9159d3d2ee1421359ebf228570c68"
> >
>
> Reviewed-by: Quentin Schulz <quentin.schulz@cherry.de>
>
> Thanks!
> Quentin
>
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#348): https://lists.yoctoproject.org/g/yocto-patches/message/348
> Mute This Topic: https://lists.yoctoproject.org/mt/106745214/3617156
> Group Owner: yocto-patches+owner@lists.yoctoproject.org
> Unsubscribe: https://lists.yoctoproject.org/g/yocto-patches/leave/13388836/3617156/66960823/xyzzy [martin.jansa@gmail.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>
>

Re: [yocto-patches] [meta-rockchip][PATCH v2] linux-torvalds-next: use tag

[Trevor Woerner]

On Wed 2024-06-19 @ 01:56:14 PM, Martin Jansa via lists.yoctoproject.org wrote:
> Why is this better?

Because in a repository like linux-next
a957267fa7e9159d3d2ee1421359ebf228570c68 could be rebased out of existence?
Although now that makes we wonder what would happen to the tag in this
situation.

Three days ago this hash was reachable from master, now it is not. That's a
lot of churn.

> Now bitbake will always run git ls-remote to
> covert next-20240611 into a957267fa7e9159d3d2ee1421359ebf228570c68 and
> break parsing this recipe without network access.

Is that how you're building with meta-rockchip?

> On Wed, Jun 19, 2024 at 10:59 AM Quentin Schulz via
> lists.yoctoproject.org
> <quentin.schulz=cherry.de@lists.yoctoproject.org> wrote:
> >
> > Hi Trevor,
> >
> > On 6/18/24 7:16 PM, Trevor Woerner via lists.yoctoproject.org wrote:
> > > Use the tag name specifically; note that it is no longer reachable from any
> > > branch.
> > >
> > > Signed-off-by: Trevor Woerner <twoerner@gmail.com>
> > > ---
> > >   recipes-kernel/linux/linux-torvalds-next_git.bb | 5 ++---
> > >   1 file changed, 2 insertions(+), 3 deletions(-)
> > >
> > > diff --git a/recipes-kernel/linux/linux-torvalds-next_git.bb b/recipes-kernel/linux/linux-torvalds-next_git.bb
> > > index 9a95eb5bc4d8..1369ba1d707f 100644
> > > --- a/recipes-kernel/linux/linux-torvalds-next_git.bb
> > > +++ b/recipes-kernel/linux/linux-torvalds-next_git.bb
> > > @@ -10,13 +10,12 @@ COMPATIBLE_MACHINE = "^$"
> > >   COMPATIBLE_MACHINE:radxa-zero-3 = "radxa-zero-3"
> > >
> > >   LINUX_VERSION = "6.10-rc3"
> > > +KERNEL_VERSION_SANITY_SKIP = "1"
> > >   PV = "${LINUX_VERSION}+git${SRCPV}"
> > >   SRC_URI = " \
> > > -     git://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git;protocol=https;branch=master \
> > > +     git://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git;protocol=https;nobranch=1;tag=next-20240611 \
> > >       file://rockchip-kmeta;type=kmeta;name=rockchip-kmeta;destsuffix=rockchip-kmeta \
> > >       "
> > > -# this is tag 'next-20240611'
> > > -SRCREV = "a957267fa7e9159d3d2ee1421359ebf228570c68"
> > >
> >
> > Reviewed-by: Quentin Schulz <quentin.schulz@cherry.de>
> >
> > Thanks!
> > Quentin
> >
> >
> > 
> >
> >
> 
> 
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#351): https://lists.yoctoproject.org/g/yocto-patches/message/351
> Mute This Topic: https://lists.yoctoproject.org/mt/106745214/900817
> Group Owner: yocto-patches+owner@lists.yoctoproject.org
> Unsubscribe: https://lists.yoctoproject.org/g/yocto-patches/leave/13168745/900817/63955952/xyzzy [twoerner@gmail.com]
> -=-=-=-=-=-=-=-=-=-=-=-
> 
> 

Re: [yocto-patches] [meta-rockchip][PATCH v2] linux-torvalds-next: use tag

[Martin Jansa]

On Wed, Jun 19, 2024 at 2:17 PM Trevor Woerner via
lists.yoctoproject.org <twoerner=gmail.com@lists.yoctoproject.org>
wrote:
>
> On Wed 2024-06-19 @ 01:56:14 PM, Martin Jansa via lists.yoctoproject.org wrote:
> > Why is this better?
>
> Because in a repository like linux-next
> a957267fa7e9159d3d2ee1421359ebf228570c68 could be rebased out of existence?

Really? It shouldn't disappear while the tag is still referencing it
and if the tag is deleted than this change won't help you as well
after first gc in upstream repo.

> Although now that makes we wonder what would happen to the tag in this
> situation.
>
> Three days ago this hash was reachable from master, now it is not. That's a
> lot of churn.
>
> > Now bitbake will always run git ls-remote to
> > covert next-20240611 into a957267fa7e9159d3d2ee1421359ebf228570c68 and
> > break parsing this recipe without network access.
>
> Is that how you're building with meta-rockchip?

I don't even use meta-rockchip, but I'm a bit allergic to ls-remote
while parsing recipes I'm not using (and random build failures caused
by that when there is a network glitch or something during re-parsing
of the recipes), especially when I notice a change like this which
IMHO made it worse.

Regards,

> > On Wed, Jun 19, 2024 at 10:59 AM Quentin Schulz via
> > lists.yoctoproject.org
> > <quentin.schulz=cherry.de@lists.yoctoproject.org> wrote:
> > >
> > > Hi Trevor,
> > >
> > > On 6/18/24 7:16 PM, Trevor Woerner via lists.yoctoproject.org wrote:
> > > > Use the tag name specifically; note that it is no longer reachable from any
> > > > branch.
> > > >
> > > > Signed-off-by: Trevor Woerner <twoerner@gmail.com>
> > > > ---
> > > >   recipes-kernel/linux/linux-torvalds-next_git.bb | 5 ++---
> > > >   1 file changed, 2 insertions(+), 3 deletions(-)
> > > >
> > > > diff --git a/recipes-kernel/linux/linux-torvalds-next_git.bb b/recipes-kernel/linux/linux-torvalds-next_git.bb
> > > > index 9a95eb5bc4d8..1369ba1d707f 100644
> > > > --- a/recipes-kernel/linux/linux-torvalds-next_git.bb
> > > > +++ b/recipes-kernel/linux/linux-torvalds-next_git.bb
> > > > @@ -10,13 +10,12 @@ COMPATIBLE_MACHINE = "^$"
> > > >   COMPATIBLE_MACHINE:radxa-zero-3 = "radxa-zero-3"
> > > >
> > > >   LINUX_VERSION = "6.10-rc3"
> > > > +KERNEL_VERSION_SANITY_SKIP = "1"
> > > >   PV = "${LINUX_VERSION}+git${SRCPV}"
> > > >   SRC_URI = " \
> > > > -     git://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git;protocol=https;branch=master \
> > > > +     git://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git;protocol=https;nobranch=1;tag=next-20240611 \
> > > >       file://rockchip-kmeta;type=kmeta;name=rockchip-kmeta;destsuffix=rockchip-kmeta \
> > > >       "
> > > > -# this is tag 'next-20240611'
> > > > -SRCREV = "a957267fa7e9159d3d2ee1421359ebf228570c68"
> > > >
> > >
> > > Reviewed-by: Quentin Schulz <quentin.schulz@cherry.de>
> > >
> > > Thanks!
> > > Quentin
> > >
> > >
> > >
> > >
> > >
> >
> >
> >
> >
> >
>
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#352): https://lists.yoctoproject.org/g/yocto-patches/message/352
> Mute This Topic: https://lists.yoctoproject.org/mt/106745214/3617156
> Group Owner: yocto-patches+owner@lists.yoctoproject.org
> Unsubscribe: https://lists.yoctoproject.org/g/yocto-patches/leave/13388836/3617156/66960823/xyzzy [martin.jansa@gmail.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>
>

Re: [yocto-patches] [meta-rockchip][PATCH v2] linux-torvalds-next: use tag

[Khem Raj]

Why use tags ? they are floating SHA1 and bitbake will need n/w access
all the time.

On Wed, Jun 19, 2024 at 1:59 AM Quentin Schulz via
lists.yoctoproject.org
<quentin.schulz=cherry.de@lists.yoctoproject.org> wrote:
>
> Hi Trevor,
>
> On 6/18/24 7:16 PM, Trevor Woerner via lists.yoctoproject.org wrote:
> > Use the tag name specifically; note that it is no longer reachable from any
> > branch.
> >
> > Signed-off-by: Trevor Woerner <twoerner@gmail.com>
> > ---
> >   recipes-kernel/linux/linux-torvalds-next_git.bb | 5 ++---
> >   1 file changed, 2 insertions(+), 3 deletions(-)
> >
> > diff --git a/recipes-kernel/linux/linux-torvalds-next_git.bb b/recipes-kernel/linux/linux-torvalds-next_git.bb
> > index 9a95eb5bc4d8..1369ba1d707f 100644
> > --- a/recipes-kernel/linux/linux-torvalds-next_git.bb
> > +++ b/recipes-kernel/linux/linux-torvalds-next_git.bb
> > @@ -10,13 +10,12 @@ COMPATIBLE_MACHINE = "^$"
> >   COMPATIBLE_MACHINE:radxa-zero-3 = "radxa-zero-3"
> >
> >   LINUX_VERSION = "6.10-rc3"
> > +KERNEL_VERSION_SANITY_SKIP = "1"
> >   PV = "${LINUX_VERSION}+git${SRCPV}"
> >   SRC_URI = " \
> > -     git://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git;protocol=https;branch=master \
> > +     git://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git;protocol=https;nobranch=1;tag=next-20240611 \
> >       file://rockchip-kmeta;type=kmeta;name=rockchip-kmeta;destsuffix=rockchip-kmeta \
> >       "
> > -# this is tag 'next-20240611'
> > -SRCREV = "a957267fa7e9159d3d2ee1421359ebf228570c68"
> >
>
> Reviewed-by: Quentin Schulz <quentin.schulz@cherry.de>
>
> Thanks!
> Quentin
>
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#348): https://lists.yoctoproject.org/g/yocto-patches/message/348
> Mute This Topic: https://lists.yoctoproject.org/mt/106745214/1997914
> Group Owner: yocto-patches+owner@lists.yoctoproject.org
> Unsubscribe: https://lists.yoctoproject.org/g/yocto-patches/leave/13340786/1997914/645821814/xyzzy [raj.khem@gmail.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>
>

Re: [meta-rockchip][PATCH v2] linux-torvalds-next: use tag

[Trevor Woerner]

On Tue 2024-06-18 @ 01:16:19 PM, Trevor Woerner wrote:
> Use the tag name specifically; note that it is no longer reachable from any
> branch.
> 
> Signed-off-by: Trevor Woerner <twoerner@gmail.com>
> ---
>  recipes-kernel/linux/linux-torvalds-next_git.bb | 5 ++---
>  1 file changed, 2 insertions(+), 3 deletions(-)

Added to meta-rockchip, master branch.
Thanks for the review!