- Notifications
You must be signed in to change notification settings - Fork750
Update setup.py - use vswhere for msbuild 15+#540
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
UnmanagedExports complaining about ildasm.exe: UnmanagedExports.1.2.7\tools\RGiesecke.DllExport.targets(58,3): error : Microsoft.Build.Utilities.ToolLocationHelper could not find ildasm.exe. [.\pythonnet\src\clrmodule\clrmodule.csproj] |
den-run-ai commentedSep 11, 2017 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
although unmanaged exports complains, the build looks fine and the "import clr" works. this issue seems very common: https://stackoverflow.com/questions/44836800/could-not-find-ildasm-exe |
dmitriyse commentedSep 11, 2017 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
Build fails due to 12.04 upgraded to Mono 5.2 + travis apt command have a bad thing --no-install-suggest. addons:apt:sources: -sourceline:deb http://download.mono-project.com/repo/ubuntu precise mainkey_url:http://keyserver.ubuntu.com/pks/lookup?op=get&search=0xA6A19B38D3D831EFpackages: -mono-devel -ca-certificates-mono Probably it's time to merge#518 , as it's contain many things like that already debugged. |
codecovbot commentedSep 11, 2017 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
Codecov Report
@@ Coverage Diff @@## master #540 +/- ##==========================================- Coverage 77.12% 76.41% -0.72%========================================== Files 65 64 -1 Lines 5547 5541 -6 Branches 889 889 ==========================================- Hits 4278 4234 -44- Misses 981 1019 +38 Partials 288 288
Continue to review full report at Codecov.
|
closing this in favor#518 assuming@dmitriyse moves vswhere to tools subdirectory |
My suggestion#540 (comment) did not helps. Strange. |
SlimeQ commentedJun 6, 2018
On Windows 10, the ildasm.exe errordoes prevent installation from completing. I'm using the master branch on 6/6/18. I attempted to upgrade DllExport and configure the app using DllExport.bat (which was suggested somewhere). This results in the following error:
|
What does this implement/fix? Explain your changes.
VS 2017 issues with msbuild
Does this close any currently open issues?
#539
Any other comments?
@dmitriyse needs to be added to this commit
Checklist
Check all those that are applicable and complete.
AUTHORS
CHANGELOG