-
Notifications
You must be signed in to change notification settings - Fork 751
Rebased #154
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
Rebased #154
Changes from all commits
c945022
95958bb
d6a782b
9745dcd
343238c
94107e4
ef43a0e
3447b9a
ba6c09a
0cbab89
e43b21a
b869178
0a4fa48
ff86811
bebe07f
611eb08
f2c932b
d2a605a
22d66b9
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -40,6 +40,18 @@ def _find_msbuild_tool(tool="msbuild.exe", use_windows_sdk=False): | |
locappdir, vcpy27, r"x64\mt.exe") | ||
if os.path.exists(mtpath): | ||
return mtpath | ||
if sys.version_info[:2] == (3,5): | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. There shouldn't be any need for hardcoded paths. Not everyone will have installed the windows SDK on the C drive. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @tonyroberts I agree, but I did not find in registry this path. also not sure if windows sdk can be installed to non-default paths?! maybe @zooba can point how this path can be mapped in more generalized way? (r"C:\Program Files (x86)\Windows Kits\8.1\bin") There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. removing from rebase. agree with @tonyroberts There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
|
||
vcpy35 = (r"C:\Program Files (x86)\Windows Kits\8.1\bin") | ||
if PLATFORM == "x86": | ||
mtpath = os.path.join( | ||
vcpy35, r"x86\mt.exe") | ||
elif PLATFORM == "x64": | ||
mtpath = os.path.join( | ||
vcpy35, r"x64\mt.exe") | ||
if os.path.exists(mtpath): | ||
return mtpath | ||
else: | ||
print (mtpath) | ||
value_name = "InstallationFolder" | ||
sdk_name = "Windows SDK" | ||
keys_to_check = [ | ||
|
@@ -304,6 +316,7 @@ def _check_output(*popenargs, **kwargs): | |
'Programming Language :: Python :: 3.2', | ||
'Programming Language :: Python :: 3.3', | ||
'Programming Language :: Python :: 3.4', | ||
'Programming Language :: Python :: 3.5', | ||
'Programming Language :: C#', | ||
'License :: OSI Approved :: Zope Public License', | ||
'Development Status :: 5 - Production/Stable', | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Setting branch to
develop
might make sense, but if you're pushing to the main repo you should keep the reference as it is ...There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Agreed. I think we should merge into master and do a 2.1 release as soon as
the 3.5 stuff is all working and tested, so the CI build statuses should be
left as they were.
On Mon, Feb 22, 2016 at 10:43 AM Benedikt Reinartz [email protected]
wrote:
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@tonyroberts you can see on my branch that py35 is working and tested
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
When this is merged the same will hold for the original links :)