Yes, otherwise it won't install *.so files. It was needed in the past when
we allowed "python setup.py install". Maybe it isn't useful for "develop",
but I think we should keep it.


On Tue, Apr 8, 2014 at 2:55 PM, David Warde-Farley <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'],
>
> Can you confirm the .so is necessary? I'm almost sure it isn't.
>
> --
> Reply to this email directly or view it on GitHub<https://github.com/lisa-lab/pylearn2/pull/774/files#r11405626>
> .
>