Skip to content
This repository has been archived by the owner on Aug 5, 2024. It is now read-only.

Warning 41 is misleading #18

Open
Chris00 opened this issue Nov 15, 2017 · 2 comments
Open

Warning 41 is misleading #18

Chris00 opened this issue Nov 15, 2017 · 2 comments

Comments

@Chris00
Copy link

Chris00 commented Nov 15, 2017

Warning 41

warning 41: Some packages are mentionned in package scripts of features,
but there is no dependency or depopt toward them: "user-setup"

seem to imply that user-setup should be mentioned as a dependency — and indeed doing so makes the warning go away — but doing so creates a circular dependency. An improvement would be welcome.

@AltGr
Copy link
Collaborator

AltGr commented Nov 15, 2017

Indeed, it's often likely an error, but not in this case. Probably post-messages, which is not evaluated in the scope of the package but in the final state of the install, shouldn't be considered for warning 41.

@AltGr
Copy link
Collaborator

AltGr commented Nov 16, 2017

This has actually been fixed already in opam, which I forgot; not backported yet to Camelus / 1.2.2 format though. Thanks for reporting!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants