-
Notifications
You must be signed in to change notification settings - Fork 27
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
BUG: .S assembly files not being compiled #181
Comments
Hi. Thanks for opening up an issue. That is an interesting one. I have been trying to compile threadx for a hobby project a while ago and ran into similar issues. Can't promise I will have a fix very soon, however will put it on top of my list. If you have any idea of a quick fix in mind I would be happy to hear it. |
Splitting assembly code and assembly language files is a good idea (.s and .S). Like the original Makefile from STM32CubeMX: ASM_SOURCES are used for .s and ASMM_SOURCES are used for .S. Main issue right now is that threadx uses a lot of .S files from the threadx library. I will do some research over the weekend to find out more. |
I managed to modify STM32Make.make to build firmware. Here are the steps that I did in order to build.
Also, STM32Make.make doesn't set AS_DEFS the same as in the original Makefile. |
…troduced in the out of source build branch. Like using environment variables for GCC and OpenOCD.
As stated in the pull request, which I thank you for tremendously even though it is not merged. I really appreciate people trying to merge in new stuff. So thank you a bunch for your contribution! A new beta version is out which includes some of your fixed described above. If you would like to test it it can be found here: https://github.com/bmd-studio/stm32-for-vscode/releases/tag/v3.2.7-beta |
Platform: Windows 10/Linux
stm32-for-vscode v3.2.6
Way to reproduce:
Output of make -f STM32Make.make -qp
The original makefile from CubeMX has ASMM_SOURCES which stm32-for-vscode is completely ignoring.
The text was updated successfully, but these errors were encountered: