[prev in list] [next in list] [prev in thread] [next in thread] 

List:       gentoo-dev
Subject:    Re: [gentoo-dev] python-namespaces.eclass
From:       Arfrever Frehtes Taifersar Arahesis <Arfrever () gentoo ! org>
Date:       2011-04-30 22:49:49
Message-ID: 201105010049.51614.Arfrever () gentoo ! org
[Download RAW message or body]


2011-05-01 00:32:13 Brian Harring napisał(a):
> On Sat, Apr 30, 2011 at 11:27:47PM +0200, Arfrever Frehtes Taifersar Arahesis wrote:
> > 2011-04-04 13:48:43 Brian Harring napisał(a):
> > > > # @ECLASS: python-namespaces.eclass
> > > > # @MAINTAINER:
> > > > # Gentoo Python Project <python@gentoo.org>
> > > > # @BLURB: Eclass for packages installing Python namespaces
> > > > # @DESCRIPTION:
> > > > # The python-namespaces eclass defines phase functions for packages installing Python namespaces.
> > > 
> > > ^^^ This isn't a useful description.
> > 
> > IMHO it's sufficient, but could you suggest some sentences of description?
> 
> It probably is sufficient for *you*- you're knee deep in the guts of 
> python, and know it's purpose.  Plus you wrote the eclass ;)
> 
> The purpose of the description, and general code comments is for 
> *other* folk who may be looking at that code/problem for the first 
> time.  It needs to be written aimed at them.
> 
> I'd suggest doing a grep of DESCRIPTION w/in eclasses and working from 
> the clearer examples- just looking at the first few examples returned, 
> alternatives for example has enough in the opening description to 
> understand exactly what it's for, same for apache-2.
> 
> One thing to keep in mind is that even for folk who know python, this 
> is actually an area that doesn't match the normal verbage, and is a 
> bit niche in it's usage- try googling 'python namespaces' sometime, 
> note that it's scope discussions rather than pkgutil/distribute 
> importation across multiple directories. 
> 
> To be clear, 'python namespaces' is a whole other thing from what this 
> is doing- this is manipulation of importation pathways (the 
> module/import hierarchy/namespace, rather than the common scope 
> terminology).
> 
> Either way, rough suggestion:
> """
> This eclass handles installation/creation of python 2.7 and higher

__init__.py files generated by this eclass work with Python 2.4.
(I haven't tested them with older versions.)

> pkgutil namespaces, and the equivalent distibute functionality.  See 
> zope's (example ebuild) for examples of usage.
> """

-- 
Arfrever Frehtes Taifersar Arahesis

["signature.asc" (application/pgp-signature)]

[prev in list] [next in list] [prev in thread] [next in thread]