Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

paver ignores install_requires in setup.py #49

Closed
yedpodtrzitko opened this issue Mar 15, 2012 · 3 comments
Closed

paver ignores install_requires in setup.py #49

yedpodtrzitko opened this issue Mar 15, 2012 · 3 comments
Assignees
Milestone

Comments

@yedpodtrzitko
Copy link

Issue is already described over here:
https://code.google.com/p/paver/issues/detail?id=42

it's caused by this conditional in setuptools.command.install: ~70

if caller_module != 'distutils.dist' or caller_name!='run_commands':
    # We weren't called from the command line or setup(), so we
    # should run in backward-compatibility mode to support bdist_*
    # commands.
    _install.run(self)
else:
    self.do_egg_install()

...so when setup.py is dispatched by paver, it uses install() from distribute (if branch), which doesn't process requirements

"solved" by this hack:
https://github.com/yedpodtrzitko/paver/commit/7fe05f04195b6b4af1bf963e792fdf63a4963624

@ghost ghost assigned Almad Mar 16, 2012
@Almad
Copy link
Member

Almad commented Mar 16, 2012

Urgh, let's stack hacks!

Please just add note about this in-setuptools-hack above the code in patch and send pull request, thanks.

@Almad
Copy link
Member

Almad commented Mar 16, 2012

(when you are at it, note to changelog would be also nice :)

@yedpodtrzitko
Copy link
Author

fixed in yedpodtrzitko/paver@7fe05f04195b6b4af1bf963e792fdf63a4963624

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants