-
Notifications
You must be signed in to change notification settings - Fork 8
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
Improve expand rule #146
Comments
I have simple question. In given example |
@Mucka at the moment what we are doing is something like this :
where This should be fixed later on. Currently we are testing build system extensively in our hobbie projects to see what we like and what we want to change. |
Thank for your answer! I have system with lots of unit tests. And every module have multiple files with tests grouped like: |
@Mucka Can't all of them just be linked together? Or you get linker conflicts? Usually unit tests are just separate object files that all end up in one application. |
@jimon For now, every each test have separate main function and eventually ends as separate application. This is embedded application, test were designed to be as small as possible, to be able to run them on hardware eventually, for now the unit test platform is qemu so this is not necessity. The unit test platform is custom stuff based on ThrowTheSwitch/Unity. |
Ah ok, indeed this use case is not really trivial for our build system :\ Maybe it's feasible to put every object + test case for it in separate folder? So you can build them folder by folder case. And then build a final application by getting all object files recursively and ignoring specific prefix ( |
Now if you write something like
build objects(build/*): auto src1/*.c src2/temp.c
you will getValueError: cannot expand rule cc because of different amount of explicit generated targets and explicit inputs ...
which is just bad UX. We really need to improve it!The text was updated successfully, but these errors were encountered: