qmk / qmk_configurator

The QMK Configurator
http://config.qmk.fm
687 stars 343 forks source link

Build error when compiling any of the Helix/Rev2 keyboards #714

Closed agaisin closed 1 year ago

agaisin commented 4 years ago
Errors here: **Linking: .build/helix_rev2_layout_mine.json.elf  .build/obj_helix_rev2_layout_mine.json/keyboards/helix/rev2/rev2.o: In function `matrix_init_kb': /qmk_compiler/qmk_firmware/keyboards/helix/rev2/rev2.c:25: undefined reference to `is_helix_master' collect2: error: ld returned 1 exit status

makeine.json.elf] Error 1 make: * ine.json] Error 1 ake finished with errors

Compiler output below.

Helix

Helix

A compact split ortholinear keyboard.

Keyboard Maintainer: Makoto Kurauchi @pluis9
Hardware Supported: Helix PCB Alpha, Beta, Pro Micro
Hardware Availability: PCB & Case Data, Yushakobo Shop, Little Keyboards

How to build

See build environment setup then the make instructions for more information.

Loaded 6 layers and 384 keycodes. Defined 4 Any key keycodes

Making 

avr-gcc (GCC) 5.4.0 Copyright (C) 2015 Free Software Foundation, Inc. This is free software; see the source for copying conditions. There is NO warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

Compiling: keyboards/helix/helix.c  Compiling: keyboards/helix/rev2/rev2.c  Compiling: keyboards/helix/rev2/keymaps/layout_mine.json/keymap.c  Compiling: quantum/quantum.c  Compiling: quantum/keymap_common.c  Compiling: quantum/keycode_config.c  Compiling: quantum/matrix_common.c  Compiling: quantum/matrix.c  Compiling: quantum/debounce/sym_g.c  Compiling: quantum/process_keycode/process_space_cadet.c  Compiling: quantum/process_keycode/process_magic.c  Compiling: quantum/process_keycode/process_grave_esc.c  Compiling: tmk_core/common/host.c  Compiling: tmk_core/common/keyboard.c  Compiling: tmk_core/common/action.c  Compiling: tmk_core/common/action_tapping.c  Compiling: tmk_core/common/action_macro.c  Compiling: tmk_core/common/action_layer.c  Compiling: tmk_core/common/action_util.c  Compiling: tmk_core/common/print.c  Compiling: tmk_core/common/debug.c  Compiling: tmk_core/common/util.c  Compiling: tmk_core/common/eeconfig.c  Compiling: tmk_core/common/report.c  Compiling: tmk_core/common/avr/suspend.c  Compiling: tmk_core/common/avr/timer.c  Compiling: tmk_core/common/avr/bootloader.c  Assembling: tmk_core/common/avr/xprintf.S  Compiling: tmk_core/common/magic.c  Compiling: tmk_core/protocol/lufa/lufa.c  Compiling: tmk_core/protocol/usb_descriptor.c  Compiling: tmk_core/protocol/lufa/outputselect.c  Compiling: lib/lufa/LUFA/Drivers/USB/Class/Common/HIDParser.c  Compiling: lib/lufa/LUFA/Drivers/USB/Core/AVR8/Device_AVR8.c  Compiling: lib/lufa/LUFA/Drivers/USB/Core/AVR8/EndpointStream_AVR8.c  Compiling: lib/lufa/LUFA/Drivers/USB/Core/AVR8/Endpoint_AVR8.c  Compiling: lib/lufa/LUFA/Drivers/USB/Core/AVR8/Host_AVR8.c  Compiling: lib/lufa/LUFA/Drivers/USB/Core/AVR8/PipeStream_AVR8.c  Compiling: lib/lufa/LUFA/Drivers/USB/Core/AVR8/Pipe_AVR8.c  Compiling: lib/lufa/LUFA/Drivers/USB/Core/AVR8/USBController_AVR8.c  Compiling: lib/lufa/LUFA/Drivers/USB/Core/AVR8/USBInterrupt_AVR8.c  Compiling: lib/lufa/LUFA/Drivers/USB/Core/ConfigDescriptors.c  Compiling: lib/lufa/LUFA/Drivers/USB/Core/DeviceStandardReq.c  Compiling: lib/lufa/LUFA/Drivers/USB/Core/Events.c  Compiling: lib/lufa/LUFA/Drivers/USB/Core/HostStandardReq.c  Compiling: lib/lufa/LUFA/Drivers/USB/Core/USBTask.c  **Linking: .build/helix_rev2_layout_mine.json.elf  .build/obj_helix_rev2_layout_mine.json/keyboards/helix/rev2/rev2.o: In function `matrix_init_kb': /qmk_compiler/qmk_firmware/keyboards/helix/rev2/rev2.c:25: undefined reference to `is_helix_master' collect2: error: ld returned 1 exit status

makeine.json.elf] Error 1 make: * ine.json] Error 1 ake finished with errors 

skullydazed commented 4 years ago

The helix board requires special flags passed to the make command which the API can't support. I'm afraid someone will have to some work to make it work on configurator.

mtei commented 4 years ago

The helix board requires special flags passed to the make command which the API can't support. I'm afraid someone will have to some work to make it work on configurator.

I have published https://github.com/qmk/qmk_firmware/pull/8422. This is a change on the QMK CORE side to resolve this issue. (I think so.)

And I think the change on the Helix keyboard side would be solved by adding two small files after merging https://github.com/qmk/qmk_firmware/pull/8669.

mtei commented 3 years ago

I made a workaround for the compile error on QMK configurator.

Currently the following two targets are compilable.

fauxpark commented 1 year ago

I believe this is fixed now.