From owner-freebsd-python@FreeBSD.ORG Wed Oct 19 11:26:16 2011 Return-Path: Delivered-To: python@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 3851E106586F for ; Wed, 19 Oct 2011 11:26:16 +0000 (UTC) (envelope-from wenheping@gmail.com) Received: from mail-vx0-f182.google.com (mail-vx0-f182.google.com [209.85.220.182]) by mx1.freebsd.org (Postfix) with ESMTP id E71398FC24 for ; Wed, 19 Oct 2011 11:26:15 +0000 (UTC) Received: by vcbfo13 with SMTP id fo13so2121277vcb.13 for ; Wed, 19 Oct 2011 04:26:15 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type:content-transfer-encoding; bh=EH9oXLIrBO05JI5OJJgwm8Doslj7i3ajUCVcgULa1vA=; b=T3Ii5jEgqvH7yybZbq8ScrlTjzRQ5ysYoMb4ArXH8UULcDsOAAsyEn+1Uica54Yr71 iziaOUiGXrkYZla6WyHebNAKzg6RRUjLUxJ83/KHy7mDaB7jiFY7GTLyEvaEXofD3GFp JOrzNsqcov2+LLDKjy5BN56ZslyI7e1Mu8LxU= MIME-Version: 1.0 Received: by 10.182.114.67 with SMTP id je3mr870554obb.30.1319021861870; Wed, 19 Oct 2011 03:57:41 -0700 (PDT) Received: by 10.182.89.5 with HTTP; Wed, 19 Oct 2011 03:57:41 -0700 (PDT) In-Reply-To: <4E9EA6B9.7010203@yandex.ru> References: <4E9B2A8D.5000108@yandex.ru> <4E9EA6B9.7010203@yandex.ru> Date: Wed, 19 Oct 2011 18:57:41 +0800 Message-ID: From: wen heping To: Ruslan Mahmatkhanov Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Cc: Li-Wen Hsu , python@freebsd.org, Martin Wilke Subject: Re: Re: About zope and plone X-BeenThere: freebsd-python@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: FreeBSD-specific Python issues List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 19 Oct 2011 11:26:16 -0000 Yes, I agree with this PR that we should add -N to easy_install's deinstall argument. And shall we ask for a exp-run to test it ? wen 2011/10/19 Ruslan Mahmatkhanov : > Good day, gentlemen. > > Please tell what do you think about message bellow and this pr: > http://bugs.freebsd.org/159962 > > Message bellow is somewhat complements this pr. Please also note that Wen > agree with this pr, but he is busy for this right now. Also note that thi= s > pr is a stopper wrt importing of new zope and plone. > > Thanks in advance. > > -------- =D0=98=D1=81=D1=85=D0=BE=D0=B4=D0=BD=D0=BE=D0=B5 =D1=81=D0=BE=D0= =BE=D0=B1=D1=89=D0=B5=D0=BD=D0=B8=D0=B5 -------- > =D0=A2=D0=B5=D0=BC=D0=B0: Re: About zope and plone > =D0=94=D0=B0=D1=82=D0=B0: Sun, 16 Oct 2011 23:03:41 +0400 > =D0=9E=D1=82: Ruslan Mahmatkhanov > =D0=9A=D0=BE=D0=BC=D1=83: wen heping > > wen heping wrote on 14.10.2011 13:15: > >> Today I tested devel/py-zope.minmax in Tinderbox, it has the same plist >> error. >> Now I need such a port exist in current FreeBSD portstree to be a demo >> to python@. >> >> If python@ does not disagree I would ask portmgr@ for an exprun to test >> it. >> Then I would commit it. > > Hi Wen, > > i'm finally get it. > While seeking an example for you i took some broken/outdated ports, that = i > just can't left in this state :), so i apologize for delay. > > You was right, it's a kind of upstream problem, but bsd.python.mk still > contains a problem too, because this error still exists on deinstall of > such silly ports (that may potentially exist in future). > > Here is how to reproduce: > Pick any port, that > =C2=A0a) using setuptools for installing > =C2=A0b) has non-empty install_requires[] list in it's setup.py > =C2=A0c) does not define BUILD or RUN DEPENDS for deps, that are listed > =C2=A0 =C2=A0 in it's setup.py > > Ok, i wasn't able to find such port in the tree. But you can take, say, > devel/py-daemon, remove BUILD/RUN_DEPENDS in it's Makefile and try to > build it in tinderbox - it will builds and installs fine, but you'll get > extra files installed on deinstall. > > The only port, that looked promising, was devel/py-Jinja. It defines > this lines: > > PYEASYINSTALL_INSTALLARGS=3D =C2=A0 =C2=A0 =C2=A0-N ${PYTHON_SITELIBDIR}/= ${PYEASYINSTALL_EGG} > PYEASYINSTALL_UNINSTALLARGS=3D =C2=A0 =C2=A0-q -m -N > ${PYTHON_SITELIBDIR}/${PYEASYINSTALL_EGG} > > Note adding the -N key on deinstall. But right now they are not needed, > because this port packages successfully without them (it has empty > install_requires[]). > > The problem with Products.MailHost, for example, is that if i add zope > as dependency of this port, we well have cyclic deps and broken build. > Resolving this issue with upstream will take much time (i know this > because i already have 3 patches that hanging in zope/plone bugtrackers > more than month). > > I can avoid this a different way, by defining > PYEASYINSTALL_UNINSTALLARGS, but i believe that more correct and more > easy is to define this -N in bsd.python.mk, because it's just two bytes > in one file - and this is all, problem solved. Otherwise, i will be > forced to patch many ports with this UNINSTALLARGS line, effectively > littering the tree and making it hard to maintain. We already have it in > PYEASYINSTALL_INSTALLARGS for some reason anyway, so why to not define it= in > UNINSTALLARGS too for symmetry? > > I hope this arguments are sufficient to convince portmgr@ and make this > change. > > -- > Regards, > Ruslan > > Tinderboxing kills... the drives. >