-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA256 On 02/07/14 01:24 AM, Ulrich Mueller wrote: >>>>>> On Tue, 01 Jul 2014, Ian Stakenvicius wrote: > >> On 01/07/14 06:44 PM, Ciaran McCreesh wrote: >>>> || ( amd64? ( >>> >>> I thought we were trying to discourage that abomination... > >> It's still necessary, unfortunately, until such time as >> everything in the tree no longer needs/depends on the emul-* >> packages (see news item #8 for progress). Otherwise, end-users >> would end up with dependency collisions. > > Sure, the second block is needed, but the point is that foo? ( ) > conditionals inside a || ( ) group should be avoided. >
Ahh, sorry, missed that point entirely. > I wonder if the dependency couldn't be written like this, without > the amd64? () conditional: > > || ( ( >> =dev-libs/glib-2.34.3:2[abi_x86_32(-)] >> =virtual/opengl-7.0-r1[abi_x86_32(-)] > ) ( app-emulation/emul-linux-x86-baselibs[-abi_x86_32(-)] > app-emulation/emul-linux-x86-opengl[-abi_x86_32(-)] ) ) > > emul-linux-x86-* doesn't have keywords other than amd64, so it > won't match on non-amd64 anyway. > LGTM, will test tomorrow and confirm. -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.22 (GNU/Linux) iF4EAREIAAYFAlOzm3cACgkQ2ugaI38ACPCloQD/fKfmTkG9yp+e/D+tK7Xmiepb SKPlIk9tcoUzF0h8aToBAIqAjG8hcRKzuwoHOytpEekxVV29HWgWe+2xD6Q37T/r =XYoV -----END PGP SIGNATURE-----
