-
Notifications
You must be signed in to change notification settings - Fork 21
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
recursive clone fails #30
Comments
Someone could send a pull request to |
I can send a PR to |
I sent a PR to https://github.com/antmicro/litex-buildenv. Should I have sent it to https://github.com/timvideos/litex-buildenv? |
@wohlbier - The timvideos repo is probably the best idea? |
I imagine so, but it's not clear that it is maintained. I made a PR at timvideos. |
timvideos/litex-buildenv#754 fixes this. Shall I do a PR to update the hash here? |
@wohlbier thanks for your involvement. A PR fixing the state of this repo is very welcomed! |
Ok, I'll work on that. I'm working through the mechanics of how to create the PR when the submodule (https://github.com/antmicro/litex-buildenv) does not have the changes from its base. If I demonstrate that it works when using the base (https://github.com/timvideos/litex-buildenv) is that sufficient information to show that it works? Meantime, I opened a PR for the antmicro fork at antmicro/litex-buildenv#4, which I assume would be merged after demonstration. |
I don't have a device yet and was just planning to do the The only error I saw executing the steps was this, but it did not appear to be fatal.
|
See #31 |
The text was updated successfully, but these errors were encountered: