-
Notifications
You must be signed in to change notification settings - Fork 5
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
coq-htt as package depending on coq-htt-core #28
Comments
That's a good point too. Yes, let's do it. Do you already have the pull request? |
Actually wait. Now I'm not sure how the github repo should look like? Should there suddenly be two different repos, one for coq-htt-core, and a separate one for coq-htt? Hmm. The core part is just 2 files, it seems an overkill to have a separate repo just for them. |
@aleksnanevski the only difference is how
It seems there is green light to put the pull request together, it will take up to an hour in the worst case (CI fiddling), so that's why I ask before. |
Hmm, ok, let me give it a try. I have a feeling it will take me much longer than 1 hour, as our yaml files aren't configured for this (meta.yml in the main repo, as well as the files in templates-extra). But maybe you have a suggestion how to do this better. |
Unfortunately, the templates currently don't support the build approach used in the graph-theory project, so I decided manually define You probably want to have |
I'm not succeeding with doing this on my own (too green with dune). If you have a quick way to generate what's needed in a pull request, that would teach me a lot. |
Closed by #29 |
For the recent release shared at coq/opam#3169
coq-htt
andcoq-htt-core
are independent but mutually exclusive packages. This means that anyone using the core files of htt via opam needs to write that they depend on("coq-htt" | "coq-htt-core")
and test with both packages, which is complication compared to before.A better solution in my view would be if
coq-htt
actually depended oncoq-htt-core
. Then both these packages could be installable at the same time and handle disjoint sets of files (no conflicts). Would a pull request to this effect be welcome @aleksnanevski?The text was updated successfully, but these errors were encountered: