Revert "dev-ml/lablgl: Remove another := slot op from || dep"
authorAlexis Ballier <aballier@gentoo.org>
Thu, 23 Jun 2016 09:00:28 +0000 (11:00 +0200)
committerAlexis Ballier <aballier@gentoo.org>
Thu, 23 Jun 2016 11:23:27 +0000 (13:23 +0200)
This reverts commit 9675d96cfe244033e97dd7034b95c5af251405e3.

This commit is wrong in several aspects:
  - Such changes should come with a revbump
  - Removing := deps causes packages not to be rebuilt on upgrades, causing
    barely understandable build failures when upgrading, later on.
    Thanks to this commit, people that have installed those packages will
    likely experience those failures with the upcoming ocaml 4.03 unmask.
  - This is a non maintainer commit where no discussion at all happened with the
    maintainers during the 3 days between bug report and mass commit.
    Such discussion could have prevented the above mentioned breakage.

Proper fix is to drop the backward compatibility part of the || () dep. This fix
was slowly moving forward whenever possible and noticed. Main blocker for this is
bug #463018 which requires taking extra care for not breaking the stable tree
nor needlessly preventing future stabilization of packages.

dev-ml/lablgl/lablgl-1.05.ebuild

index 33ec4772a0b2d2d8b4f23c0e5552e37097b1c128..bb69c5d38aefc72e3983e24bd65d64d23fac5df0 100644 (file)
@@ -24,7 +24,7 @@ RDEPEND="
        tk? (
                >=dev-lang/tcl-8.3:0=
                >=dev-lang/tk-8.3:0=
-               || ( dev-ml/labltk <dev-lang/ocaml-4.02[tk] )
+               || ( dev-ml/labltk:= <dev-lang/ocaml-4.02[tk] )
        )
        "