Attention is currently required from: fixeria.
osmith has posted comments on this change. ( https://gerrit.osmocom.org/c/docker-playground/+/36446?usp=email )
Change subject: ttcn3-bts-test: abort if missing osmo-config-merge ......................................................................
Patch Set 2:
(1 comment)
File ttcn3-bts-test/Makefile:
https://gerrit.osmocom.org/c/docker-playground/+/36446/comment/0e858517_f2b8... PS1, Line 15: cfg req-osmo-config-merge
Ah, indeed. So by doing this you allow the user to do: […]
well, it is also allowed without having it in PHONY. The only thing it does, is that if you do:
``` $ touch req-osmo-config-merge $ make osmo-config-merge ```
Then make knows it isn't a real target and it should run anyway, even though the file with the name of the target exists. It's good practice to put it there, for our use case it doesn't make a difference. But since we also have cfg there I put it there as well.