Conversation
|
@palmskog Hi Karl. I implemented the changes that you recommended yesterday, I think. Before I merge, would you please mind checking if I did so correctly, and if the setup will correctly package the files. |
palmskog
left a comment
There was a problem hiding this comment.
Looks good to me, but why remove Dune-related files? There is no harm in keeping them around, and it gives more options what do with the release, e.g., use Dune locally.
|
Hm, ok, I can try to put the dune back. The first few attempts to build have failed, so I erased the dune files because I was unsure as to how they work. The error ended up being elsewhere in the end. |
|
@palmskog Hmm, I'm having a problem with using the package, and I wonder if it has to do with the change in the build. Once I install coq-htt-core and coq-htt, the system, for some reason, doesn't assign them logical names. So I can't import the files into other projects. The behavior is different in fcsl-pcm, which does receive a logical name. Have you experienced this before? |
|
@aleksnanevski the only difference I can see is that the package build now uses From htt Require Import quicksort options. |
|
Yes, the problem was that I just didn't refresh the opam environment variables via eval $(opam env). Sorry for the false alarm. |
Uh oh!
There was an error while loading. Please reload this page.