Skip to content
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

CMakeLists.txt should not define a fixed list of dependent applications #165

Open
DavidEdell opened this issue Oct 1, 2023 · 0 comments

Comments

@DavidEdell
Copy link

Describe the bug
If a different set of dependencies is needed, then the build process will fail. This makes it impossible to utilize to_lab in a system without modifying the upstream repository. The TO table can be easily overridden in the build process (ie: copying to sample_defs/tables/to_lab_sub.c), but the include list is currently fixed.

Proposed change
The foreach line in CMakeLists.txt can be changed to use a CMake macro
foreach(EXT_APP ${${TGTSYS_${SYSVAR}}_APPLIST})

NOTE: This issue also applies to sch_lab.

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

No branches or pull requests

1 participant