search

ocaml-opam / Camelus

Bot posting reports on opam-repository pull-request using a web hook
Other
18 stars 9 forks source link

Warning 41 is misleading #18

Open Chris00 opened 7 years ago

Chris00 commented 7 years ago

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 commented 7 years ago

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 commented 7 years ago

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