[Calypso] [PATCH 1/2] Add requires to run tests

Guido Günther agx at sigxcpu.org
Sun Jan 3 04:21:25 PST 2016


On Sat, Jan 02, 2016 at 04:49:53PM +0000, Jelmer Vernooij wrote:
> On Sat, Jan 02, 2016 at 05:10:45PM +0100, Guido Günther wrote:
> > ---
> >  setup.cfg | 0
> >  setup.py  | 1 +
> >  2 files changed, 1 insertion(+)
> >  delete mode 100644 setup.cfg
> > 
> > diff --git a/setup.cfg b/setup.cfg
> > deleted file mode 100644
> > index e69de29..0000000
> > diff --git a/setup.py b/setup.py
> > index 61b5a7c..8ead90f 100755
> > --- a/setup.py
> > +++ b/setup.py
> > @@ -78,6 +78,7 @@ setup(
> >      packages=["calypso", "calypso.acl"],
> >      provides=["calypso"],
> >      install_requires=["daemon","vobject"],
> > +    setup_requires=['nose>=0.11.1'],
> >      scripts=["calypso.py"],
> >      cmdclass={"build_scripts": BuildScripts},
> >      keywords=["calendar", "CalDAV"],
> 
> Wouldn't you want test_requires rather than setup_requies for nose? LGTM otherwise.

Yeah, pushed with that one. Could you have a look at the actual test
case (followup patch) too?

Cheers,
 -- Guido


More information about the Calypso mailing list