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

2.2rc installation takes a long time because of sphinx dependency #1363

Closed
spyder-bot opened this issue Feb 17, 2015 · 8 comments
Closed

2.2rc installation takes a long time because of sphinx dependency #1363

spyder-bot opened this issue Feb 17, 2015 · 8 comments

Comments

@spyder-bot
Copy link
Collaborator

From lewis_le...@hotmail.com on 2013-04-23T12:16:56Z

Please note that reporting bugs from Spyder's help menu ("?" > "Report
issue...") will fill version details for Spyder, Python and Qt in this description for you.

What steps will reproduce the problem?

  1. python setup.py install 2. 3. What is the expected output? What do you see instead? Install stops on: no module sphinx

Then attempt to install sphinx. because of pypi stability and performance problems this may fail. When it works in brings in many dependencies including docutils, pyments, jinja2. Any of these may fail if the download location times out.

Would be desireable to eliminate the sphinx dependency or make it optional--especially for installation. What version of the product are you using? On what operating system? 2.2rc0 on Windows 7, Python 2.7.3

Please provide any additional information below

.

Original issue: http://code.google.com/p/spyderlib/issues/detail?id=1363

@spyder-bot
Copy link
Collaborator Author

From ccordoba12 on 2013-04-23T10:20:23Z

Thanks for opening a new issue. I'll try to come up with a solution as soon as possible.

Labels: -Priority-Critical Priority-High MS-v2.2

@spyder-bot
Copy link
Collaborator Author

From lewis_le...@hotmail.com on 2013-04-23T10:21:02Z

I did eventually install "part" of sphinx with some dependencies missing. But, the sphinx module itself was present so that the install could complete.

rc0 is installed and works.

documentation launches.

object inspector works with plain text and rich text (looks nice with rich text)

Not clear which sphinx dependencies are actually necessary. Seems like pip may bring in a bunch of stuff that is not needed by Spyder.

Would be great to fix before release as this will be a common problem when installing from sources.

Of course, binary installer avoids this, but that is only a solution for Windows.

@spyder-bot
Copy link
Collaborator Author

From ccordoba12 on 2013-04-23T10:29:43Z

And Mac too! (thanks to our new native app).

We need Sphinx for the rich text view you mention. From its deps we are using Jinja2 and Pygments. I don't know what deps they bring in turn.

But yeah, Sphinx shouldn't be mandatory to just install Spyder.

@spyder-bot
Copy link
Collaborator Author

From ccordoba12 on 2013-04-23T14:24:44Z

Labels: Cat-Miscelleneous

@spyder-bot
Copy link
Collaborator Author

From ccordoba12 on 2013-04-24T09:31:15Z

Jed, do you think we should remove sphinx a build time dep? Lewis is right in that the docs can be easily read online.

Besides we are not imposing sphinx as a runtime dep, but with this requirement we are really doing it.

Summary: 2.2rc installation takes a long time because of sphinx dependency (was: Trouble installing rc0 because of sphinx dependency)
Cc: jed.lud...@gmail.com

@spyder-bot
Copy link
Collaborator Author

From jed.lud...@gmail.com on 2013-04-24T21:51:38Z

Sounds okay to me. Isn't it just a matter of moving "from sphinx import setup_command" inside the MyBuildDoc class defined in setup.py?

@spyder-bot
Copy link
Collaborator Author

From ccordoba12 on 2013-04-26T09:10:50Z

I don't know because MyBuildDoc is subclassing setup_command.BuildDoc.

Since I don't feel comfortable enough with distutils to make this change so late at the release process, I'm going to leave it for 2.2.1 or .2.

I hope Pierre shows up before that too, to give us a hand with it :)

Status: Accepted

@spyder-bot
Copy link
Collaborator Author

From pierre.raybaut on 2013-04-28T05:39:18Z

This issue was updated by revision c7dd7e0ffd72 .

Status: Fixed

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

No branches or pull requests

1 participant