From owner-cvs-all@FreeBSD.ORG Sun Dec 31 17:04:26 2006 Return-Path: X-Original-To: cvs-all@FreeBSD.org Delivered-To: cvs-all@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52]) by hub.freebsd.org (Postfix) with ESMTP id 3BB6A16A40F; Sun, 31 Dec 2006 17:04:26 +0000 (UTC) (envelope-from pav@FreeBSD.org) Received: from hood.oook.cz (ip-160-218-152-218.eurotel.cz [160.218.152.218]) by mx1.freebsd.org (Postfix) with ESMTP id BB44213C44B; Sun, 31 Dec 2006 17:04:20 +0000 (UTC) (envelope-from pav@FreeBSD.org) Received: from ikaros.oook.cz (localhost [127.0.0.1]) by hood.oook.cz (8.13.8/8.13.8) with ESMTP id kBVH4B8B064313 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO); Sun, 31 Dec 2006 18:04:12 +0100 (CET) (envelope-from pav@FreeBSD.org) Received: (from pav@localhost) by ikaros.oook.cz (8.13.8/8.13.8/Submit) id kBVH47xF064311; Sun, 31 Dec 2006 18:04:07 +0100 (CET) (envelope-from pav@FreeBSD.org) X-Authentication-Warning: ikaros.oook.cz: pav set sender to pav@FreeBSD.org using -f From: Pav Lucistnik To: Alejandro Pulver In-Reply-To: <20061231133753.4156b868@phobos.mars.bsd> References: <200612310434.kBV4YRXS037540@repoman.freebsd.org> <1167575429.1320.24.camel@ikaros.oook.cz> <20061231133753.4156b868@phobos.mars.bsd> Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="=-i3GUuChqzAjHvPQ9Olj1" Date: Sun, 31 Dec 2006 18:04:05 +0100 Message-Id: <1167584645.1320.33.camel@ikaros.oook.cz> Mime-Version: 1.0 X-Mailer: Evolution 2.8.2.1 FreeBSD GNOME Team Port Cc: cvs-ports@FreeBSD.org, cvs-all@FreeBSD.org, ports-committers@FreeBSD.org Subject: Re: cvs commit: ports/Mk bsd.wx.mk X-BeenThere: cvs-all@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list Reply-To: pav@FreeBSD.org List-Id: CVS commit messages for the entire tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 31 Dec 2006 17:04:26 -0000 --=-i3GUuChqzAjHvPQ9Olj1 Content-Type: text/plain; charset=ISO-8859-2 Content-Transfer-Encoding: quoted-printable Alejandro Pulver p=ED=B9e v ne 31. 12. 2006 v 13:37 -0300: > On Sun, 31 Dec 2006 15:30:29 +0100 > Pav Lucistnik wrote: >=20 > > > - Remove _WX_ILLEGAL_VERS_* added in the last update to 2.8. The re= ason is > > > that each port should choose the version it can use, that include= s taking > > > into account available components (also this brings much more chi= cken and > > > egg problems so the code could get really messy). > >=20 > > I have reverted this. It's needed as long as you want to let people use > > ranges like 2.4+ together with python component, etc. > >=20 >=20 > It is complex to do that (if you want to do it the right way), because > you need to parse components and dependencies before selecting a > version. The current code is wrong since it will only reject components > without dependency type specifications (and if there were dependencies > then it will also ignore them). >=20 > I think people should select the versions that actually work with the > port, that's why the version specification string is so flexible. >=20 > Also, if versions with invalid components are removed then the reported > error if a user writes USE_WX=3D2.4 and WX_COMPS=3Dsvg would be "selected= a > null or invalid WxWidgets version" instead of "selected a WxWidgets > component (${_WX_COMP}) which is not available for the selected version > (${_WX_VER})". That can be done, I'm confident. > > If you disagree with me, please do it so it errors out with IGNORE > > message, instead of creating a broken dependency line. > >=20 >=20 > It defines IGNORE _and_ adds the broken dependency line. I can add a > "&& !defined(IGNORE)" before adding the wrong component to the list, > but I thought IGNORE was checked when building INDEX. Well, it passed make describe and thus broke INDEX. IGNORE is not checked when running make describe, I believe. But the problem is really this: With a port allowing USE_WX=3D2.4+ and WX_COMPS=3Dpython, it should go for 2.4 or 2.6, instead of selecting 2.8 and then failing. I'm not happy with it setting such port on IGNORE. Why it should fail, when perfectly legit option is available? It should bail out with IGNORE when you select USE_WX=3D2.8 and WX_COMPS=3Dpython, never else. > > Also, the auto-detection does not seem to work: > >=20 > > [Makefile] > > USE_WX=3D 2.4+ > > WX_COMPS=3D wx > >=20 > > [pkg_info] > > wxgtk2-common-2.6.3_1/ > > wxgtk2-unicode-2.6.3_1/ > > wxgtk2-2.6.3_1/ > >=20 > > $ make -V LIB_DEPENDS > > wx_base-2.8:/usr/ports/x11-toolkits/wxgtk28 > >=20 > > What gives? > >=20 >=20 > Sorry, I re-wrote it many times and forgot this: >=20 > -_WX_VER_INSTALLED:=3D ${_HAVE_WX} > +_WX_VER_INSTALLED:=3D ${_HAVE_WX:Mwx-*:S/wx-//} Please commit this fix. --=20 Pav Lucistnik Pain clots and unformed lice pat this train. --=-i3GUuChqzAjHvPQ9Olj1 Content-Type: application/pgp-signature; name=signature.asc Content-Description: Toto je =?UTF-8?Q?digit=C3=A1ln=C4=9B?= =?ISO-8859-1?Q?_podepsan=E1?= =?UTF-8?Q?_=C4=8D=C3=A1st?= =?ISO-8859-1?Q?_zpr=E1vy?= -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.6 (FreeBSD) iD8DBQBFl+2FntdYP8FOsoIRAuraAKDK3otnZLsMF4mmHtuqsAnagJSDOgCeOyi8 I+GvGZmZS5TXLm5A3JgNvDM= =xB4A -----END PGP SIGNATURE----- --=-i3GUuChqzAjHvPQ9Olj1--