qmk_firmware icon indicating copy to clipboard operation
qmk_firmware copied to clipboard

[Bug] Making annepro2/c18 with keymap thomazmoura

Open natenho opened this issue 3 years ago • 3 comments

Trying to build using the alternative Dockerfile from https://openannepro.github.io/install_docker.html the following output is shown:

#8 129.1 Making annepro2/c18 with keymap thomazmoura                                                            [ERRORS]
#8 129.7 In file included from quantum/quantum.h:28,
#8 129.7                  from keyboards/annepro2/annepro2.h:19,
#8 129.7                  from keyboards/annepro2/annepro2.c:3:
#8 129.7 keyboards/annepro2/annepro2.c: In function 'keyboard_pre_init_kb':
#8 129.7 keyboards/annepro2/annepro2.c:56:13: error: 'ANNEPRO2_LED_MCU_WAKEUP_DELAY' undeclared (first use in this function)
#8 129.7      wait_ms(ANNEPRO2_LED_MCU_WAKEUP_DELAY);
#8 129.7              ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
#8 129.7 tmk_core/common/wait.h:17:17: note: in definition of macro 'wait_ms'
#8 129.7              if (ms != 0) {                  \
#8 129.7                  ^~
#8 129.7 keyboards/annepro2/annepro2.c:56:13: note: each undeclared identifier is reported only once for each function it appears in
#8 129.7      wait_ms(ANNEPRO2_LED_MCU_WAKEUP_DELAY);
#8 129.7              ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
#8 129.7 tmk_core/common/wait.h:17:17: note: in definition of macro 'wait_ms'
#8 129.7              if (ms != 0) {                  \
#8 129.7                  ^~
#8 129.7  | 
#8 129.7  | 
#8 129.7  | 
#8 129.7 make[1]: *** [tmk_core/rules.mk:412: .build/obj_annepro2_c18_thomazmoura/keyboards/annepro2/annepro2.o] Error 1
#8 129.7 Make finished with errors
#8 129.7 make: *** [Makefile:523: annepro2/c18] Error 1

natenho avatar Dec 24 '21 19:12 natenho

Similar to https://github.com/OpenAnnePro/qmk_firmware/issues/43, I had to remove the keymap folder in order to build with no errors

natenho avatar Dec 24 '21 19:12 natenho

Gosh, haven't updated my branch for a while now and it seems some breaking changes were made since the last time I changed it.

I'll fix it as soon as I have some free time to check what's new, but on the mean time, it's important to note that you don't have to compile all the keymaps - it might be interesting if you want to flash them all for fun or anything like it, but it should work if you just run the specific keymap you want, such as:

make annepro2/c18:default

Not only it will work even if some keymaps are broken, but it will also compile much faster (since it won't compile every keymap available).

thomazmoura avatar Dec 28 '21 16:12 thomazmoura

Thank you @thomazmoura, I was able to make only one keymap and I'm doing some improvents into @zinosat Dockerfile Making just one keymap is indeed much faster.

At least now we know that something is broken ;P Have a good new year!

natenho avatar Dec 28 '21 17:12 natenho