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

Make conf-autoconf a dev dependency #12

Closed
palmskog opened this issue Jan 23, 2022 · 1 comment · Fixed by coq/opam#2077
Closed

Make conf-autoconf a dev dependency #12

palmskog opened this issue Jan 23, 2022 · 1 comment · Fixed by coq/opam#2077

Comments

@palmskog
Copy link

The current released version of validsdp, coq-validsdp.0.7, has conf-autoconf as a build dependency:

"conf-autoconf" {build}

However, it can be an inconvenience to users of the released opam repo to have to install autoconf (a system package), just to install a Coq package. The current best practice is therefore to declare conf-autoconf a development build dependency:

"conf-autoconf" {dev & build}

This is because released packages are expected to be based on tarballs generated by autoconf, which removes the need for autoconf itself. See Flocq for examples of this, e.g., the package coq-flocq.3.4.3.

@proux01
Copy link
Contributor

proux01 commented Jan 24, 2022

@palmskog thanks, did that for the OPAM files in this repo and coq/opam#2077 completely removes the conf-autoconf dependencies as the configure.ac source files of autoconf are not even shipped in the tarballs.

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

Successfully merging a pull request may close this issue.

2 participants