-
Notifications
You must be signed in to change notification settings - Fork 365
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
Update MinGW build instructions and Makefile #459
Comments
3 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I could not get this project to compile as-is under a Windows + MSys2 MinGW environment due to multiple errors:
WIN
macro responsible for detection of MSys is commented out in theMakefile.common
. Therefore it doesn't even try and set the Windows / MinGW specific options.simavr/Makefile.common
Lines 87 to 90 in a56b550
pacman -S mingw-w64-x86_64-toolchain
pacman -S mingw-w64-x86_64-libelf
pacman -S mingw-w64-x86_64-avr-toolchain
pacman -S mingw-w64-x86_64-freeglut
mingw-w64-x86_64-gcc 10.3.0-2
), fails to findstrsep
as reported in undefined reference tostrsep
#403. The issue was closed without giving a resolution other than "old libc, not worth my time", making compilation on Windows impossible. PR Check if environment is MinGW and implement missing strsep() #404 is partly correct here, in my case I also needed to ammend the header file to expose the function.avr_usb_ep_read()
fails to compile under the above GCC option because it thinksv
might be uninitialized upon return, and that warning is turned into an error by-Werror
.mingw-w64-x86_64-avr-toolchain
(AVR-GCC 8.4.0). All it matters that there'savr-gcc
and avr-libc. A slight adaption in the Makefile to fix the AVR include path to/mingw64/avr
and AVR compiler path / prefix fixes this.make build-simavr
can be done, a normalmake
fails while compiling an example requiring the Unix headerpty.h
. Non-compatible examples should be excluded from the build. Or, https://github.com/rprichard/winpty can be substituted.The text was updated successfully, but these errors were encountered: