-
Notifications
You must be signed in to change notification settings - Fork 31
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
update Bender.yml
by using flist and vendor
deps
#9
Conversation
ead5c1d
to
fe11e33
Compare
Hi @joennlae, I opened #7 on your and @Vyeow's behalf. Feel free to modify it. I would propose using this thread for discussions on updating Bender.yml to avoid fragmentation. Regarding this change: I think larger projects with CVA6 as a dependency might not want to be forced to use the vendored dependencies of common cells etc. But for a standalone instantiation, this might be favourable. My proposal from #7 stands: Add a target |
Hi, @niwis. I am aware of #7. That is also why this pull request is against the I tried to make it work with bendered deps, but I got duplicate module declarations. Due to nonconforming versions. Which cannot be easily fixed because the others are vendored. But I understand that we need to have the possibility. So do you want to have the |
I don't have any strong opinions on this. My main question is: is this change compatible with a system like Cheshire, which has common_cells as a dependency? |
2a7009c
to
51849e7
Compare
afa5f5e
to
af0b930
Compare
Superseded by #11 |
This is a version of
Bender.yml
that works. However, it does only use the vendored in IPs. I know kind of defying the purpose of bender but still useful for a bender pipeline when cva6 is included.