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

python setup.py build_sphinx failure for Python 3.x #977

Closed
astrofrog opened this issue Apr 13, 2013 · 5 comments
Closed

python setup.py build_sphinx failure for Python 3.x #977

astrofrog opened this issue Apr 13, 2013 · 5 comments
Milestone

Comments

@astrofrog
Copy link
Member

When running:

python setup.py build_sphinx

in Python 3.3, I get the following error:

$ python setup.py build_sphinx
Freezing version number to astropy/version.py
running build_sphinx
creating docs/_build
creating docs/_build/doctrees
creating docs/_build/html
Traceback (most recent call last):
  File "setup.py", line 128, in <module>
    entry_points=entry_points
  File "/opt/local/Library/Frameworks/Python.framework/Versions/3.3/lib/python3.3/distutils/core.py", line 148, in setup
    dist.run_commands()
  File "/opt/local/Library/Frameworks/Python.framework/Versions/3.3/lib/python3.3/distutils/dist.py", line 929, in run_commands
    self.run_command(cmd)
  File "/opt/local/Library/Frameworks/Python.framework/Versions/3.3/lib/python3.3/distutils/dist.py", line 948, in run_command
    cmd_obj.run()
  File "/Users/tom/dropbox/Code/development/Astropy/astropy/astropy/setup_helpers.py", line 772, in run
    from urllib import pathname2url
ImportError: cannot import name pathname2url

Presumably, the import just needs to be adjusted for Python 3.x. I'm tagging this as 'easy' and it would be a possible patch for GSoC students.

@jdnc
Copy link
Contributor

jdnc commented Apr 15, 2013

Hi I am working on this as a GSoC'13 applicant - Madhura Parikh

@astrofrog
Copy link
Member Author

@jdnc - I saw you submitted this as a traditional patch to the mailing list, but we virtually always use pull requests on GitHub to submit the suggested changes, so could you try and open a pull request for this? See this page: http://docs.astropy.org/en/latest/development/workflow/index.html for setting up git properly, then follow the 'workflow for developers'. Thanks!

@jdnc
Copy link
Contributor

jdnc commented Apr 15, 2013

Sure I'll open a pull request for this.

jdnc referenced this issue in jdnc/astropy Apr 15, 2013
@mdboom
Copy link
Contributor

mdboom commented Apr 15, 2013

The pull request is #986 -- @jdnc: note that this creates a handy link from the bug to the new pull request.

@jdnc
Copy link
Contributor

jdnc commented Apr 15, 2013

@mdboom Thanks for sharing that tip. I have added a new commit to #986 by using the defined variable PY3

@eteq eteq closed this as completed in ff0a07c Apr 22, 2013
eteq added a commit that referenced this issue May 13, 2013
Added fix to close #977
Conflicts:

	astropy/setup_helpers.py
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

3 participants