Skip to content

Autodep #520

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

Closed
wants to merge 2 commits into from
Closed

Autodep #520

wants to merge 2 commits into from

Conversation

gregglind
Copy link
Contributor

still need numpy for setup, but at least we get the right version of python-dateutil

Others could be:

* have requirements{2,3}.txt, and read from them with a function

setuptools_args = {
'install_requires': ['python-dateutil < 2','numpy'],
}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think this will work because I don't use setuptools in Python 2.x-- something about setuptools messes up the Cython-ing step and I've never found the time to dig in and figure out why

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pip in virtualenv comes with distribute which has no conflict with Cython at all. I just built the Cython bits on OSX (Py2.7). here is my pip freeze before the build:

Cython==0.15.1
distribute==0.6.24
numpy==1.6.1

pip install -e . # works fine.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Got it, thanks

@wesm
Copy link
Member

wesm commented Dec 25, 2011

install_requires is invalid with regular distutils. Is there a way to check if pip/distribute is in use?

@gregglind
Copy link
Contributor Author

Sorry for the delay here!

As with usual in packaging, there are a million solutions :)

  • turn it off, and let people figure this stuff out themselves. Something like "if there is no setuptools, then tell them eveyrthing they need to get going"
  • just require one of setuptools or distribute. It think this is the case for most people.

Look for some inbound code... I think this is an improvement...

@gregglind gregglind closed this Jan 4, 2012
gregglind added a commit to gregglind/pandas that referenced this pull request Jan 4, 2012
wesm pushed a commit that referenced this pull request Jan 6, 2012
dan-nadler pushed a commit to dan-nadler/pandas that referenced this pull request Sep 23, 2019
…bols-new

Fix for list_symbols reaching memory limit of MongoDB
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

Successfully merging this pull request may close these issues.

2 participants