Skip to content

Revert "[Python] Use explicit imports" #757

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

Merged
merged 1 commit into from
Dec 23, 2015

Conversation

eeckstein
Copy link
Contributor

Reverts #752

I got following build error on macos:

Exception occurred:
File "/Users/erik/proj/main-swift/swift/utils/pygments/swift.py", line 5, in
from pygments.lexer import (
ImportError: cannot import name Comment

eeckstein added a commit that referenced this pull request Dec 23, 2015
Revert "[Python] Use explicit imports"
@eeckstein eeckstein merged commit 81fc913 into master Dec 23, 2015
@gribozavr
Copy link
Contributor

@practicalswift Could you look into the failure and resubmit the PR with a fix?

@modocache
Copy link
Contributor

Splitting up the PR would help merge it in a little at a time--I'm particularly interested in removing the wildcards from utils/build-script.

@practicalswift
Copy link
Contributor

@eeckstein Sorry about that! New PR submitted as #758. Split up as per @modocache:s request.

gribozavr added a commit that referenced this pull request Dec 24, 2015
[Python] Use explicit imports (again) - split up to avoid pygments issue in #757
@gribozavr gribozavr deleted the revert-752-explicit-imports branch January 4, 2016 11:20
kateinoigakukun pushed a commit to kateinoigakukun/swift that referenced this pull request Sep 7, 2020
[pull] swiftwasm-release/5.3 from release/5.3
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.

4 participants