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

XcodeVersion interprets version 10.0 as version 1.0 #1459

Closed
codeWonderland opened this issue Jun 8, 2018 · 1 comment
Closed

XcodeVersion interprets version 10.0 as version 1.0 #1459

codeWonderland opened this issue Jun 8, 2018 · 1 comment

Comments

@codeWonderland
Copy link

Platform: MacOS Mojave
Compiler: Apple LLVM version 10.0.0 (clang-1000.10.25.5)

I was attempting to reinstall node after updating my Mac to the new developer release, and I kept getting an issue that my compiler was out of date. After digging for a while, I began to play around with the functions in gyp, and I noticed that the issue resides in pylib/xcode_emulation.py.

In said file, on line 1273, the script attempts to take the output of xcodebuild -version and translate that into a numerical value that the program can use. To test my theory, I printed the version of before and after that statement. The output was as follows:

>>> import gyp.xcode_emulation
>>> gyp.xcode_emulation.XcodeVersion()
Xcode 10.0
100
('0100', '10L176w')

As you can see, the system is interpreting version 10.0 the same as it would version 1.0, causing anything that is based off of this to think that the system is out of date.

The line currently exists as version = version.split()[-1].replace('.', '') I can rewrite this and make a pull request on this repo. I just wanted to make sure there was an issue in place before I make any changes.

@richardlau
Copy link
Member

See #1454. This was fixed by #1455 and released in node-gyp 3.6.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

No branches or pull requests

2 participants