Date: Wed, 7 Sep 2011 17:10:33 +0800 From: Li-Wen Hsu <lwhsu@FreeBSD.org> To: Sofian Brabez <sbz@freebsd.org> Cc: FreeBSD Python Team <freebsd-python@freebsd.org> Subject: Re: Update of devel/py-twisted to 11.0.0 Message-ID: <CAKBkRUwU8i2Sq3zmZs9fuHaDzodg7rd9Q%2BVGxtRGLaim5mGGRA@mail.gmail.com> In-Reply-To: <20110907110255.GA1795@freebsd.ifr.lan> References: <20110906205704.GA38513@freebsd.ifr.lan> <CAKBkRUyaDNYUHW1B%2BkH-b3E9C298tGFMiYgVqPXR8dsXuiC_tQ@mail.gmail.com> <20110907110255.GA1795@freebsd.ifr.lan>
next in thread | previous in thread | raw e-mail | index | archive | help
Looks good, please go ahead commit and close the related PRs.
Thanks for your help!
On Wed, Sep 7, 2011 at 19:02, Sofian Brabez <sbz@freebsd.org> wrote:
> Hi python@,
>
> On Wed, Sep 07, 2011 at 02:02:43PM +0800, Li-Wen Hsu wrote:
>> Hello Sofian,
>>
>> Thanks for working on this, python@ definitely need more power!
>>
>> Please see the reply inline.
>>
>> On Wed, Sep 7, 2011 at 04:57, Sofian Brabez <sbz@freebsd.org> wrote:
>> > Hi python@,
>> >
>> > This afternoon, I've merged my initial update to the twisted related p=
orts with
>> > the patch of lwhsu@ [1]. My patch is there [2].
>> >
>> > My patch make happy portlint, add LICENSE in each twisted ports and ad=
d new
>> > OPTIONS for twistedPair port.
>>
>> And please also add pair_DEPENDS in bsd.python.mk, that is around line 6=
41.
>
> Done
>>
>> > Please lwhsu@ notice than all twisted ports are sharing the
>> > ${PYTHON_SITELIBDIR}/twisted/plugins folder that's why I've replaced t=
he @dirrm
>> > by a @dirrmtry in pkg-plist.
>>
>> Thanks for reminding me this, I think we have
>>
>> =C2=A0 =C2=A0 @dirrm %%PYTHON_SITELIBDIR%%/twisted/plugins in
>>
>> in devel/py-twistedCore/pkg-plist, and all other twisted ports depends
>> on twistedCore, so we probably don't need to have @dirrm nor @dirrmtry
>> in other ports.
>> They sneaked in my patch because I forget this, so just removing them
>> is fine. (I was planning getting maintainer's approval on general idea
>> then doing a more substantial patch.)
>> Using @dirrmtry is fine, too.
>>
>
> Imho, we should keep the @dirrmtry directive to remember us that all twis=
ted
> ports install files in ${PYTHON_SITELIBDIR}/twisted/plugins and agreed th=
e only
> @dirrm should be in devel/py-twistedCore/pkg-plist file.
>> > I would like to have your approbation to commit it, and sent it to my =
mentors
>> > for their approbations, I start working on this one because I took the=
pr
>> > ports/155570 but lwhsu@ inform me about ports/159954 on #freebsd-pytho=
n.
>> >
>> > Below the tinderbox build logs:
>> >
>> > http://regis.goodking.org/tb/index.php?action=3Dlist_buildports&build=
=3D7-sbz&search_port_name=3Dtwisted&Go=3DGo
>> > http://regis.goodking.org/tb/index.php?action=3Dlist_buildports&build=
=3D8-sbz&search_port_name=3Dtwisted&Go=3DGo
>> > http://regis.goodking.org/tb/index.php?action=3Dlist_buildports&build=
=3D9-sbz&search_port_name=3Dtwisted&Go=3DGo
>>
>> All looks good, and I also tested building all other ports depends on
>> twisted, all are fine. =C2=A0Is anybody aware non-backward-compatible
>> change?
>>
>> Another thing, I found there is no twisted 11.0.0 distfiles in
>> http://tmrc.mit.edu/mirror/twisted/, that's why I removed them from
>> MASTER_SITES, could you check it again?
>
> Indeed, I remove them, we will see in the future if they remirrored them.
>>
>> Please consider my suggestions, thanks!
>>
>
> Patch update with your suggestions, now I'm waiting your go ;), I save th=
e old
> version [1] and the new [2] there.
>> > Regards
>> >
>> > [1] http://people.freebsd.org/~lwhsu/twisted-11.0.0.diff
>> > [2] http://people.freebsd.org/~sbz/twisted-11.0.0.diff
>> >
>> > --
>> > Sofian Brabez
>> >
>>
>>
>>
>> --
>> Li-Wen Hsu
>> http://lwhsu.org
>
> Regards
>
> [1] http://people.freebsd.org/~sbz/twisted-11.0.0.old.diff
> [2] http://people.freebsd.org/~sbz/twisted-11.0.0.diff
>
> --
> Sofian Brabez
>
--=20
Li-Wen Hsu
http://lwhsu.org
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAKBkRUwU8i2Sq3zmZs9fuHaDzodg7rd9Q%2BVGxtRGLaim5mGGRA>