I would keep it as their is *.rst files in the pylern2 repo. Itis the
readme files and I think they should get installed.


On Tue, Apr 8, 2014 at 3:22 PM, Bart <notifications@github.com> wrote:

> In setup.py:
>
> >      license='BSD 3-clause license',
> >      long_description=open('README.rst').read(),
> >      install_requires=['numpy>=1.5', 'theano', 'pyyaml', 'argparse'],
> >      package_data={
> > -        '': ['*.txt', '*.rst', '*.cu', '*.cuh', '*.h'],
> > +        '': ['*.txt', '*.rst', '*.cu', '*.cuh', '*.h', '*.so'],
>
> I just checked, and I believe David is right, it's not needed. It copies
> the *.so files because they are part of an extension. Here is the ouput
> with and without package_data defined: http://www.diffchecker.com/1oegf6iw
>
> There is no need for *.rst either though, shall I get rid of that too?
>
> --
> Reply to this email directly or view it on GitHub<https://github.com/lisa-lab/pylearn2/pull/774/files#r11406970>
> .
>