Skip to content
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

bbv/theories checkin breaks all projects using bbv #10

Closed
vmurali opened this issue Jul 3, 2018 · 2 comments
Closed

bbv/theories checkin breaks all projects using bbv #10

vmurali opened this issue Jul 3, 2018 · 2 comments

Comments

@vmurali
Copy link
Contributor

vmurali commented Jul 3, 2018

Looks like I didn't realize this affects all the projects using bbv :(

In my setup, my _CoqProject using bbv has the following:

-R . Kami
-R ../bbv bbv

And inside my files (in Kami), I do "Require Import bbv/Word.".

The following change to _CoqProject does not work.
-R . Kami
-R ../bbv/theories bbv

Also, why is this a good idea as opposed to storing the .vo files directly in the directories named after the project?

@samuelgruetter
Copy link
Contributor

I'm surprised that this change doesn't work for you, because for bedrock2, it worked: mit-plv/bedrock2@0d1e6eb
And for riscv-coq too: mit-plv/riscv-coq@913b989
I really just had to insert /theories in the right place.
Are you sure that's not just a "should have started from a fresh clone"?

@vmurali
Copy link
Contributor Author

vmurali commented Jul 3, 2018 via email

@vmurali vmurali closed this as completed Jul 3, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants