-
Notifications
You must be signed in to change notification settings - Fork 7
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
Missing deps on test sections? #27
Comments
Sadly, this is a known and undocumented problem. The problem is just in the |
To be honest, I think it is better in every way to fix |
The package had been released in setup-dev mode. It seems there is a bug with deps computation when there is a flag involved. Consider filling a bug to remind me to solve this. |
Thanks for commenting on this Sylvain. If I understand you right, the lib part of oasis, which is used to compute dependencies, is at fault here. That would mean that the REST API needs no change, it should just provide enough dependencies for the package to compile, even if the package is in |
OK, I totally mismatch my sentence. "The package had NOT been released in setup-dev mode". The rest of the comment stands. There is a bug in dependency calculation with flags. Please fill a bug against oasis. The quick solution is to have "Default: false" for flag Test in odn. |
Bug reported, thanks! |
Hi,
starting from a fresh install, I fail to install oasis:
It seems that odb do not add deps from the test section (fileutils and oUnit), while the configure script of odn requires them. This issue is related to #25, in that there is a need to access test dependencies with oasis-db.
Easy workaround of course, first install the missing dependencies, by invoking
ocaml odb.ml fileutils
directly. I'm posting it here for the record (hope I'm not duplicating a previous bug report ...)The text was updated successfully, but these errors were encountered: