On Tue, Mar 23, 2021 at 10:22 AM Tomasz Dziendzielski
<tomasz.dziendziel...@gmail.com> wrote:
>
> From: Wladislav Wiebe <wladislav...@gmail.com>
>
> Introduced poky upstream commit:
> 14df6d53b6 linux-yocto: improve checkout error handling and reporting
>
> Converts all remote branches to local branches.
> Our Linux repo has ~4500 remote branches:
> git branch -a | grep remotes |wc -l  -> 4575
>
> This costs our best workstations ~20 minutes to simply do that conversion
> in do_kernel_checkout which is not needed (at least I do not see any
> benefit after testing with this patch).
>

Unfortunately, this is needed for some workflows on top of linux-yocto.

So this can't go in as-is.

Why you have 4500 remotes is another question of course ;)

We could introduce a variable that when enabled makes the functionality
optional, and would fix your use case.

Bruce

> Signed-off-by: Wladislav Wiebe <wladislav...@gmail.com>
> Signed-off-by: Tomasz Dziendzielski <tomasz.dziendziel...@gmail.com>
> ---
>  meta/classes/kernel-yocto.bbclass | 9 ---------
>  1 file changed, 9 deletions(-)
>
> diff --git a/meta/classes/kernel-yocto.bbclass 
> b/meta/classes/kernel-yocto.bbclass
> index 35587dd564..e3bc9938ff 100644
> --- a/meta/classes/kernel-yocto.bbclass
> +++ b/meta/classes/kernel-yocto.bbclass
> @@ -363,15 +363,6 @@ do_kernel_checkout() {
>                 git clean -d -f
>         fi
>
> -       # convert any remote branches to local tracking ones
> -       for i in `git branch -a --no-color | grep remotes | grep -v HEAD`; do
> -               b=`echo $i | cut -d' ' -f2 | sed 's%remotes/origin/%%'`;
> -               git show-ref --quiet --verify -- "refs/heads/$b"
> -               if [ $? -ne 0 ]; then
> -                       git branch $b $i > /dev/null
> -               fi
> -       done
> -
>         # Create a working tree copy of the kernel by checking out a branch
>         machine_branch="${@ get_machine_branch(d, "${KBRANCH}" )}"
>
> --
> 2.31.0
>
>
> 
>


-- 
- Thou shalt not follow the NULL pointer, for chaos and madness await
thee at its end
- "Use the force Harry" - Gandalf, Star Trek II
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#149824): 
https://lists.openembedded.org/g/openembedded-core/message/149824
Mute This Topic: https://lists.openembedded.org/mt/81551851/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to