Skip to content

Rustdoc tests fail due to htmlparser python dependency #35760

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.

Sign up for GitHub

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

Jump to bottom
Closed
nagisa opened this issue Aug 17, 2016 · 3 comments
Closed

Rustdoc tests fail due to htmlparser python dependency #35760

nagisa opened this issue Aug 17, 2016 · 3 comments
Labels
T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap)

Comments

@nagisa
Copy link
Member

nagisa commented Aug 17, 2016 

Traceback (most recent call last):
  File "/home/nagisa/Documents/rust/rust/src/etc/htmldocck.py", line 113, in <module>
    from HTMLParser import HTMLParser
ImportError: No module named 'HTMLParser'
@jonas-schievink
Copy link
Contributor

HTMLParser seems to be a part of Python 2, and was renamed to html.parser in Python 3. I have both Python versions installed, so maybe the build system is using the wrong executable? I'm hitting this with rustbuild, not sure about the old build system.

@jonas-schievink
Copy link
Contributor

The old build system is probing python2 first, but it's not used by rustbuild

@euclio
Copy link
Contributor

euclio commented Nov 13, 2016 •
edited
Loading

Just ran into this. Should probably be tagged with A-rustbuild.

@sfackler sfackler added the T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap) label Nov 14, 2016
alexcrichton added a commit to alexcrichton/rust that referenced this issue Nov 14, 2016
@alexcrichton
rustbuild: Allow configuration of python interpreter
5f62613 
Add a configuration key to `config.toml`, read it from `./configure`, and add
auto-detection if none of those were specified.

Closes rust-lang#35760
@alexcrichton alexcrichton mentioned this issue Nov 14, 2016
Merged
bors added a commit that referenced this issue Nov 18, 2016
@bors
Auto merge of #37769 - alexcrichton:rustbuild-python, r=brson
2a6d02e 
rustbuild: Allow configuration of python interpreter

Add a configuration key to `config.toml`, read it from `./configure`, and add
auto-detection if none of those were specified.

Closes #35760
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap)
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@nagisa @euclio @sfackler @jonas-schievink