search

mmccoyd / zmk-config

ZMK firmware for Hillside ergonomic keyboards
MIT License
12 stars 114 forks source link

Build fail after dependency zmk-dev-arm update to 3.5 #9

Open FilippeMafra opened 4 months ago

FilippeMafra commented 4 months ago

This was the change on :

chore(ci): Bump to 3.5 Docker images: https://github.com/zmkfirmware/zmk/commit/94aa7d8d532f1e84e8e8b9fa96c522468dfc259d

I have a fork from this repo that also depends on https://github.com/zmkfirmware/zmk and started to fail on build:

Failing build example

The summary of the error:

'label' is marked as deprecated in 'properties:' in /__w/hillside-zmk-config/hillside-zmk-config/zephyr/dts/bindings/display/solomon,ssd1306fb-i2c.yaml for node /soc/i2c@40003000/ssd1306@3c.
devicetree error: 'pinctrl-0' is marked as required in 'properties:' in /__w/hillside-zmk-config/hillside-zmk-config/zephyr/dts/bindings/spi/nordic,nrf-spim.yaml, but does not appear in <Node /soc/spi@40004000 in '/__w/hillside-zmk-config/hillside-zmk-config/zephyr/misc/empty_file.c'>
CMake Error at /__w/hillside-zmk-config/hillside-zmk-config/zephyr/cmake/modules/dts.cmake:279 (message):
  gen_defines.py failed with return code: 1
-- In: /tmp/tmp.k5MDIcFl7Q/zephyr, command: /usr/bin/python3;/__w/hillside-zmk-config/hillside-zmk-config/zephyr/scripts/dts/gen_defines.py;--dts;/tmp/tmp.k5MDIcFl7Q/zephyr/zephyr.dts.pre;--dtc-flags;'';--bindings-dirs;/__w/hillside-zmk-config/hillside-zmk-config/zmk/app/module/dts/bindings;/__w/hillside-zmk-config/hillside-zmk-config/zmk/app/dts/bindings;/__w/hillside-zmk-config/hillside-zmk-config/zephyr/dts/bindings;--header-out;/tmp/tmp.k5MDIcFl7Q/zephyr/include/generated/devicetree_generated.h.new;--dts-out;/tmp/tmp.k5MDIcFl7Q/zephyr/zephyr.dts.new;--edt-pickle-out;/tmp/tmp.k5MDIcFl7Q/zephyr/edt.pickle;--vendor-prefixes;/__w/hillside-zmk-config/hillside-zmk-config/zmk/app/dts/bindings/vendor-prefixes.txt;--vendor-prefixes;/__w/hillside-zmk-config/hillside-zmk-config/zephyr/dts/bindings/vendor-prefixes.txt
Call Stack (most recent call first):
  /__w/hillside-zmk-config/hillside-zmk-config/zephyr/cmake/modules/zephyr_default.cmake:129 (include)
  /__w/hillside-zmk-config/hillside-zmk-config/zephyr/share/zephyr-package/cmake/ZephyrConfig.cmake:66 (include)
  /__w/hillside-zmk-config/hillside-zmk-config/zephyr/share/zephyr-package/cmake/ZephyrConfig.cmake:92 (include_boilerplate)
  CMakeLists.txt:8 (find_package)
ramnathk commented 3 months ago

I forked this project today and tried to build the 52 key keymap and ran into the same issue. @mmccoyd can you please help?

FilippeMafra commented 3 months ago

I'm checking the PR that made the change:

We could take some inspiration from there.

alparo commented 2 months ago

It can be fixed by #10

FilippeMafra commented 1 month ago

Thank you @alparo. This PR fixes it: https://github.com/mmccoyd/zmk-config/pull/10