Re: Bug#728797: ITP: python-mne -- Python modules for MEG and EEG data analysis
Hi Alexandre,
On Mon, Nov 25, 2013 at 09:47:36AM +0100, Alexandre Gramfort wrote:
> > I just merged this target into override_dh_installman and now it is
> > used.
>
> thanks
:-)
> > BTW, I need to insist on my naming question which finally becomes quite
> > abstruse if you call the source package mne-python but the binary
> > python-mne. Please answer my question (now asked in the third flavour
> > of rewording) why exactly you are specifying the used programming
> > language in the package name.
>
> well mne-python is a python library and all the other python packages I use
> on a daily basis start with python- e.g. python-sklearn, python-numpy,
> python-scipy. The convention seem to be python-"import name"
For libraries this is OK and since you confirmed that it is basically a
library (despite the fact that there is a user application /usr/bin/mne
which was triggering my question) the name python-mne for the binary
package sounds right. I wonder whether we should call the source package
in the same way 'python-mne'. I fail to see a reason to switch the name
parts for the source package.
> and mne-python
> is imported in python with "import mne" hence the package name "python-mne"
>
> does it make sense?
Yes. My question was basically triggered because I was not sure about
the role of /usr/bin/mne. I was wondering whether this might serve as a
user application which could be put into a separate binary package - but
this sounds like overkill for this few-line wrapper. In any case I
decided to create a real manpage - help2man does a too poor job here.
Finally I noticed that no docs are installed in the package. I added
the AUTHORS and README file but I would be happy if you could inject the
proper code to install the "real" documentation into the package (or
provide an extra *-doc package if this might be more apropriate).
If you want to add these docs you most probably want to add the following
to debian/rules:
override_dh_installdocs:
dh_installdocs
additional code to install the documentation
Just keep us informed in case this hint might not be sufficient.
Kind regards
Andreas.
--
http://fam-tille.de
Reply to: