-
Notifications
You must be signed in to change notification settings - Fork 121
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
west update loops infinitely over the first repository #690
Comments
Thanks for the report. Is this something specific to your (corrupted somehow?) west workspace or can you reproduce from scratch in another and brand new west workspace? In the latter case please share the manifest so we can reproduce too. In any case never remove the workspace that does reproduce. Move it aside if needed. |
Forgot the most obvious sorry: please try |
I just tried to reproduce this in a brand new west workspace, but didn't manage. Maybe I have something corrupted.
I forgot to specify that this does not happen if I set |
I don't think you're supposed to change this; maybe one of these changes contributed to corrupting your workspace. The If you can't find it in the documentation then it's not supported. Manifest resolution is notoriously complicated: |
This being said, infinite loops should never happen. At least not before a considerable amount of abuse. I found this line TWICE in the logs; it looks bad:
Are you actually importing sdk-nrf twice somehow? What happens when you run |
I used
Indeed, and the second one doesn't have a matching
I don't think so?
|
The sequence of the last two lines below is also really strange:
Have you dropped repetitive logging between these line or did you copy them exactly as they were in
|
I copied them exactly as they were. I was also surprised that there were no logs between the repeated updates. |
I had quick look at the code. The Update and Manifest class both call each other so there is potential for some infinite recursion. I can't see exactly where though. Some other ideas, can you try variations of these and look for any clue? west update -vvv # yes you can pass 3 -v
west update --stats -v
west update -v sdk-nrf |
The only difference with
Gives the same output than the previous command, with the only exception of
Gives the following diff with
One thing I noticed while doing these is that the first
and:
|
This is happening again now right after I changed |
Can you share detailed, step-by-step commands that reproduce every time? Including SHA1s. If you can then please edit the description at the top. |
Unfortunately the issue doesn't seem to easily reproduce right away in a workspace that is not already corrupted. |
This is the only west "data" I can think of:
@mbolivar-ampere anything I forgot? |
This loop goes on forever.
Happens with version 1.0.0, but also with a manual build from https://github.com/zephyrproject-rtos/west/tree/v1.2.99.
Contents of
.west/config
(located alongsidemy_dir
andnrf
):Contents of
my_dir/west.yml
:The text was updated successfully, but these errors were encountered: