diff options
author | Boris Kolpackov <boris@codesynthesis.com> | 2016-04-23 07:46:46 +0200 |
---|---|---|
committer | Boris Kolpackov <boris@codesynthesis.com> | 2016-04-23 16:42:26 +0200 |
commit | 72fdf2163fb1a6d70b1be567eb6b8ec586d766f0 (patch) | |
tree | ede20f485852e8a7e5b47e7914598e9265a0599a /bpkg/pkg-configure.cxx | |
parent | cfc2d16905adb1a97e8bc61b0144f5f763809025 (diff) |
Revert revision ignoring in satisfy(constraint, constraint)
Diffstat (limited to 'bpkg/pkg-configure.cxx')
-rw-r--r-- | bpkg/pkg-configure.cxx | 11 |
1 files changed, 6 insertions, 5 deletions
diff --git a/bpkg/pkg-configure.cxx b/bpkg/pkg-configure.cxx index 8cccddf..abb0ff7 100644 --- a/bpkg/pkg-configure.cxx +++ b/bpkg/pkg-configure.cxx @@ -72,8 +72,9 @@ namespace bpkg auto r (p->prerequisites.emplace (dp, d.constraint)); - // If we already have a dependency on this package, pick the - // stricter of the two constraints. + // Currently we can only capture a single constraint, so if we + // already have a dependency on this package and one constraint is + // not a subset of the other, complain. // if (!r.second) { @@ -83,9 +84,9 @@ namespace bpkg bool s2 (satisfies (d.constraint, c)); if (!s1 && !s2) - fail << "incompatible constraints " - << "(" << n << " " << *c << ") and " - << "(" << n << " " << *d.constraint << ")"; + fail << "multiple dependencies on package " << n << + info << n << " " << *c << + info << n << " " << *d.constraint; if (s2 && !s1) c = d.constraint; |