[pymvpa] Possible Typo

Michael Hanke michael.hanke at gmail.com
Sat Mar 21 07:04:48 UTC 2009


On Thu, Mar 19, 2009 at 10:23:45AM -0400, Yaroslav Halchenko wrote:
> > On Thu, Mar 19, 2009 at 01:09:51PM +0100, Valentin Haenel wrote:
> > > Hi all,
> 
> > > i found a possible typo in setup.py. The file is executable but the she-bang is
> > > missing. Maybe i am wrong, but i have attached a patch in case i am not. ;)
> 
> > Not sure whether it should be non-exec or with shebang, since typically
> > people do:
> >   python setup.py install
> so, since that is the only usecase for it, ie noone does 'import setup'
> and since there is no conditional on __name__, I would actually vote for
> shebang.  I will accept your patch into yoh/master ;) (I just need to
> learn how to do that properly so I could preserve all your commit
> information so you come into the history of pymvpa development ;))

Maybe it is worth putting the corresponding reference here:

  http://www.pymvpa.org/faq.html#i-want-to-develop-a-new-feature-for-pymvpa-how-can-i-do-it-efficiently

Doing it like this preserves the information who actually did the patch
and does not overwrite it with the ID of the person who just commited it
to the repository -- which will eventually lead to an appearance in the
PyMVPA changelog movie ;-)

  http://www.pymvpa.org/history.html



Michael


-- 
GPG key:  1024D/3144BE0F Michael Hanke
http://apsy.gse.uni-magdeburg.de/hanke
ICQ: 48230050



More information about the Pkg-ExpPsy-PyMVPA mailing list