ricekuo / zhpy

Automatically exported from code.google.com/p/zhpy
0 stars 0 forks source link

python-daemon & zhpy conflict #30

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
python-daemon : http://pypi.python.org/pypi/python-daemon/

Bug Report: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=544391

Suggests :
./tests/ -> ./zhpy-tests/

Original issue reported on code.google.com by dreamerwolf.tw@gmail.com on 31 Aug 2009 at 9:45

GoogleCodeExporter commented 8 years ago
Tests folder is the default name of nose test tool, this folder is for 
unit-test only and users won't access to this 
folder.

If you want to package zhpy for debian, just don't put the tests folder into 
the build. (if you install zhpy by 
easy_install, 'tests' won't copy to the install folder)

Is it acceptable? Or do I miss the point?

Original comment by gasolin on 31 Aug 2009 at 4:11

GoogleCodeExporter commented 8 years ago
Okay ...
First I guess that it should not in the src tarball, and will take ./tests/ out 
from
the original tarball (for Debian) .
Thank you for review this bug.
:)

Original comment by dreamerwolf.tw@gmail.com on 31 Aug 2009 at 4:51

GoogleCodeExporter commented 8 years ago
PS. I got a suggestion from Debian.

It is from bugreport.

2009/9/3 Ben Finney <ben+debian@benfinney.id.au>:
>
> I think the best solution is for neither package to install the unit
> test suite (or, at least, not install it outside a package-specific
> directory).
>
> In the ‘python-daemon’ package, this needs to be fixed upstream by
> causing the distribution's ‘setup’ to exclude the unit test suite
> from installation. The ‘setup.py’ should have the following change:
>
> -    packages=find_packages(),
> +    packages=find_packages(exclude=["tests"]),
>
> Something like this is probably the best solution for ‘python-zhpy’ as
> well.
>

Original comment by dreamerwolf.tw@gmail.com on 3 Sep 2009 at 5:31

GoogleCodeExporter commented 8 years ago
ok, checked in http://code.google.com/p/zhpy/source/detail?r=826

Original comment by gasolin on 3 Sep 2009 at 9:33