Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Feb 8, 2026

  • Set DMBOOT_CONFIG_DIR to default to ${CMAKE_BINARY_DIR}/configs instead of empty string
  • Update modules/CMakeLists.txt to add a new custom command to download configuration files using dmf-get with --type dmfc
  • Ensure the config directory is created before dmf-get runs
  • Update romfs.cmake to create directory if it doesn't exist and fix make_dmffs command parameters
  • Remove non-fatal fallbacks from module and config downloads (build will fail if files are missing)
  • Update README to document the default behavior
  • Test the build to ensure configs are downloaded and embedded correctly
  • Address code review feedback (fix circular dependency and improve error message)
  • Run security checks (no vulnerabilities found)

Summary: Configuration files from the DMD file are now automatically downloaded to build/configs and embedded in the firmware at /configs. The default config directory is now inside the build directory. Downloads are now fatal - build will fail if required modules or configs are missing.

Original prompt

This section details on the original issue you should resolve

<issue_title>Katalog /configs powinien domyślnie wskazywać na katalog tymczasowy wewnątrz build</issue_title>
<issue_description>Katalog z konfiguracjami, który jest montowany w /configs, powinien domyślnie znajdować się wewnątrz katalogu build (tego w którym budujemy projekt). Do tego katalogu też powinny być ściągane pliki konfiguracyjne z pliku dmd. Więcej na ten temat można poczytać w dmod:
https://github.com/choco-technologies/dmod/blob/develop/docs/dmd-file-format.md

Chodzi o to, że plilki konfiguracyjne mogą znajdować się wewnątrz paczek z driverami i mogą być instalowane przez narzędzie dmf-get. Powinniśmy podać ten nasz katalog jako parametr do dmf-get tak, aby ściągnięte konfiguracje znajdowały się tam. </issue_description>

Comments on the Issue (you are @copilot in this section)


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: JohnAmadis <17320783+JohnAmadis@users.noreply.github.com>
@JohnAmadis JohnAmadis marked this pull request as ready for review February 8, 2026 11:24
Copilot AI and others added 2 commits February 8, 2026 11:25
Co-authored-by: JohnAmadis <17320783+JohnAmadis@users.noreply.github.com>
Co-authored-by: JohnAmadis <17320783+JohnAmadis@users.noreply.github.com>
Copilot AI changed the title [WIP] Update config directory to default to build folder Default config directory to build/configs and auto-download config files Feb 8, 2026
Copilot AI requested a review from JohnAmadis February 8, 2026 11:27
@JohnAmadis
Copy link
Contributor

@copilot - rozumiem powód dla którego zrobiłeś to tak, żeby pobieranie konfiguracji nie failowało builda (dmclk w wersji 0.3 nie zawierał plików konfiguracyjnych), ale z racji tego, że to ważne żeby mieć wszystkie konfiguracje, build powinien wtedy failować, jeśli brakuje plików konfiguracyjnych. Ponadto problem z dmclk został rozwiązany w wersji 0.4, która przed chwilą została opublikowana - możesz sprawdzić teraz jeszcze raz, powinno działać

Co-authored-by: JohnAmadis <17320783+JohnAmadis@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Feb 8, 2026

@copilot - rozumiem powód dla którego zrobiłeś to tak, żeby pobieranie konfiguracji nie failowało builda (dmclk w wersji 0.3 nie zawierał plików konfiguracyjnych), ale z racji tego, że to wa�...

Done. Removed the non-fatal fallbacks from both module and config downloads. The build will now fail if any required modules or configuration files are missing. (commit cfb769c)

@JohnAmadis JohnAmadis merged commit c86ea88 into master Feb 8, 2026
1 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Katalog /configs powinien domyślnie wskazywać na katalog tymczasowy wewnątrz build

2 participants