Date: Wed, 7 Sep 2011 13:02:55 +0200 From: Sofian Brabez <sbz@FreeBSD.org> To: Li-Wen Hsu <lwhsu@FreeBSD.org> Cc: FreeBSD Python Team <freebsd-python@freebsd.org> Subject: Re: Update of devel/py-twisted to 11.0.0 Message-ID: <20110907110255.GA1795@freebsd.ifr.lan> In-Reply-To: <CAKBkRUyaDNYUHW1B%2BkH-b3E9C298tGFMiYgVqPXR8dsXuiC_tQ@mail.gmail.com> References: <20110906205704.GA38513@freebsd.ifr.lan> <CAKBkRUyaDNYUHW1B%2BkH-b3E9C298tGFMiYgVqPXR8dsXuiC_tQ@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
--pf9I7BMVVzbSWLtt
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable
Hi python@,
On Wed, Sep 07, 2011 at 02:02:43PM +0800, Li-Wen Hsu wrote:
> Hello Sofian,
>=20
> Thanks for working on this, python@ definitely need more power!
>=20
> Please see the reply inline.
>=20
> 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 po=
rts with
> > the patch of lwhsu@ [1]. My patch is there [2].
> >
> > My patch make happy portlint, add LICENSE in each twisted ports and add=
new
> > OPTIONS for twistedPair port.
>=20
> And please also add pair_DEPENDS in bsd.python.mk, that is around line 64=
1.
Done
>=20
> > Please lwhsu@ notice than all twisted ports are sharing the
> > ${PYTHON_SITELIBDIR}/twisted/plugins folder that's why I've replaced th=
e @dirrm
> > by a @dirrmtry in pkg-plist.
>=20
> Thanks for reminding me this, I think we have
>=20
> @dirrm %%PYTHON_SITELIBDIR%%/twisted/plugins in
>=20
> 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.
>=20
Imho, we should keep the @dirrmtry directive to remember us that all twisted
ports install files in ${PYTHON_SITELIBDIR}/twisted/plugins and agreed the =
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 m=
entors
> > 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-python.
> >
> > 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
>=20
> All looks good, and I also tested building all other ports depends on
> twisted, all are fine. Is anybody aware non-backward-compatible
> change?
>=20
> 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.
>=20
> Please consider my suggestions, thanks!
>=20
Patch update with your suggestions, now I'm waiting your go ;), I save the =
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
> >
>=20
>=20
>=20
> --=20
> 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
--pf9I7BMVVzbSWLtt
Content-Type: application/pgp-signature
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.18 (FreeBSD)
iEYEARECAAYFAk5nT14ACgkQc2NR9CSH5X7RagCfU86NDN1oOf50kev13JUk2q1z
+o4AoMKTKlh6KtrJtmAjVAdJMN7pgWiM
=AFkl
-----END PGP SIGNATURE-----
--pf9I7BMVVzbSWLtt--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20110907110255.GA1795>