search

siemens / kas

Setup tool for bitbake based projects
MIT License
371 stars 148 forks source link

config: Fix include file handling if no config file is given #82

Closed jneuhauser closed 2 years ago

jneuhauser commented 2 years ago

If we do not specify a configuration file, the repository anchor is set to the current working directory, which breaks the resolution of repo-/file-relative include file paths in the IncludeHandler class.

Signed-off-by: Johann Neuhauser jneuhauser@dh-electronics.com

henning-schild commented 2 years ago

This should go via the Mailinglist. I do not yet understand the problem to be solved. Maybe bring an example to repro with "expected result" and "actual result". Ideally that would be a testcase which fails before your patch and works after your patch.

That Signed-Off looks a bit weird, but it might just be the github html rendering playing tricks.

jneuhauser commented 2 years ago

Thanks for your feedback, I'll try to create a minimal example and will discuss this on the mailing list.

jneuhauser commented 2 years ago

Moved to the mailing list https://groups.google.com/g/kas-devel/c/kqZx7N7oQtI