search

ocaml / opam

opam is a source-based package manager. It supports multiple simultaneous compiler installations, flexible package constraints, and a Git-friendly development workflow.
https://opam.ocaml.org
Other
1.23k stars 351 forks source link

wrong error message for package names instead of wrong dependency #729

Closed smondet closed 11 years ago

smondet commented 11 years ago

When a package has a wrong dependency the error message is still on the "package name".

It would be preferable to not display that the package is "available" and display the Warning as an error while trying to install the package.

$ opam remote add test ort/
test       Synchronizing with /bla/bla/ort
...
[WARNING] Package sexp_parser_edsl.0.0.1 depends on the unknown package sexplib.syntax
The following NEW packages are available:
 - sexp_parser_edsl.0.0.1
Updating the cache of metadata (~/.ocamlbrew/ocaml-4.00.1/.opam/state.cache).

$ opam install sexp_parser_edsl
sexp_parser_edsl is not a valid versioned package name
'/home/smondet/.ocamlbrew/ocaml-4.00.1/bin/opam install sexp_parser_edsl' failed

(I am using master:

$ git log -1
commit 2641bf825908cf4d53e10453c9355c81235c948b
Author: Thomas Gazagnaire <thomas@gazagnaire.org>
Date:   Thu Aug 8 16:10:57 2013 +0200

    If an `opam` file is located at the root of a dev package, use its build instructions instead of the one stored in OPAM global state.

    Fix #671

)

samoht commented 11 years ago

Hum, I am not able to repro this on master. Closing for now on, but please re-open if you happen to hit this again.