-
Notifications
You must be signed in to change notification settings - Fork 16.6k
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
waf: fails to compile on windows #4018
Comments
could you do the following?
|
Yeah, I'm in an airport at the moment, but I can easily run that this
|
|
return code on error is michael@WIN-3NBOUTHN4TN /cygdrive/c/Users/michael/Desktop/DIYDrones/ardupilot/ArduPlane |
additional michael@WIN-3NBOUTHN4TN /cygdrive/c/Users/michael/Desktop/DIYDrones/ardupilot/ArduPlane the crash is in python, not from the script itself. I added a print to the end of mavgen.py. $ uname -a |
@meee1 @WickedShell completely untested in a machine that crashes, but could you give it a try? https://github.com/lucasdemarchi/ardupilot/tree/pr/waf-windows-workaround |
Still no joy: @lucasdemarchi
|
@WickedShell ok... it seems python uses signed. Just pushed a version that may work. Could you try again? |
@lucasdemarchi I'm always happy to test, still seems to be failing
|
@WickedShell actually tridge merged this and fixed that error, can you test master? |
@OXINARF still fails
|
@WickedShell ah just noticed, tridge didn't merge the second version Lucas put, but the first. |
@OXINARF since all of Lucases branches fail for me I'm not to surprised master failed as well :) |
i've pushed some changes which make it pass for both my windows boxes now, at least for building SITL |
@tridge I made comments in the commits: 5ef0c4d#commitcomment-17411303 and 9653aae#commitcomment-17411315 |
master as of ~2 minutes ago made it through mavlink, still compiling the rest now, (I did see an error code, but it continued past it) |
ok... seems to be working now. Thanks. |
Issue details
waf crashes after compiling the mavlink bindings
Version
e797470
Python 2.7.10
CYGWIN_NT-6.1-WOW milton 2.4.1(0.293/5/3) 2016-01-24 11:24 i686 Cygwin
Windows 7 Pro x64
Platform
Logs
The text was updated successfully, but these errors were encountered: