Skip to content

Conversation

@compnerd
Copy link
Member

Deal with different python versions being installed. GYB is python2 only and breaks with python 3.x. Ensure that we pick up python 2.7 when building (this is the version vended by Visual Studio and is the stable 2.x series release).

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

Deal with different python versions being installed.  GYB is python2 only and breaks with python 3.x.  Ensure that we pick up python 2.7 when building (this is the version vended by Visual Studio and is the stable 2.x series release).
@compnerd
Copy link
Member Author

CC: @gwynne

@compnerd
Copy link
Member Author

@swift-ci please smoke test

@gwynne
Copy link
Contributor

gwynne commented Feb 11, 2019

Visual Studio also vends 64-bit Python 2.7 at C:\Python27amd64; it may be worth including a note to adjust one's invocation accordingly if needed.

@compnerd
Copy link
Member Author

@gwynne ... don't think that the 32-bit vs 64-bit matters here, its used for gyb

@gwynne
Copy link
Contributor

gwynne commented Feb 11, 2019

@compnerd Oh how well I know! But I was thinking more of configurations where only the 64-bit version is installed (such as mine).

@compnerd
Copy link
Member Author

Okay, I think that adding that as a follow up is reasonable.

@compnerd compnerd merged commit 0cf79bb into master Feb 11, 2019
@compnerd compnerd deleted the python-your-gyb branch February 11, 2019 21:05
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.

3 participants