-
Notifications
You must be signed in to change notification settings - Fork 44
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
Compiled Windows version is missing from releases #74
Comments
Yeah, the windows cross-complilation seems to be broken at the moment for some reason. Needs to be studied. Help is welcome as I don't run X-Plane or develop on Windows. |
DISCLAIMER: I know very little about QT and Docker and cross-compilation etc... but I found that the Windows xpl was missing due to the DESTDIR = $$PWD line that was commented out in the extplane-server.pro file. I think this resulted in the libextplane-server.a file not being built or something. (like I said I know very little about this stuff :-)) |
Would you be able to create a PR for that? |
I have never done anything like that before, but I can give it a try. |
You can even go to the file in this repo, press edit and GitHub will guide you through the process (sort of).
… Op 26 aug. 2021 om 00:03 heeft jfjoubert ***@***.***> het volgende geschreven:
I have never done anything like that before, but I can give it a try.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
|
This should fix the missing win.xpl and extplane-transformer.exe in the final build. Relates to vranki#74
Any updates on this? I would love to use this, but I'm less fond of signing up for an account just so I can compile it. |
The readme mentions Linux and Windows versions are in the releases. But it's only Linux. Since it's possible to compile for those two systems using Docker, you could have Github Actions do it for you. I know you're looking for developers, but I'm afraid I can't help out with this, though I'll try to look into it.
The text was updated successfully, but these errors were encountered: