[riot-notifications] [RIOT-OS/RIOT] murdock: allow multiple files to be sent along with a test job (#11697)
notifications at github.com
Wed Jun 26 19:32:17 CEST 2019
I noticed one bug I think, we are checking the `ELFFILE` hash and uploading the `FLASHFILE`.
While thinking about it a bit more, also applied to the given example, I imagine somehow two variables for this.
* `APPLICATION_BIN_FILES` or something that described the compiled files that are used. For the normal usage, `FLASHFILE` and should also contain `softdevice.hex` I imagine. And these files should course be uploaded to `bindir`. Putting `ELFFILE` there would be specific to the test if we want one running `debug`. These ones could be automatically added as dependency to `link` in `Makefile.include`.
* `TESTS_INPUT_FILES` for files that should be taken into account when checking the test hash which by default would have `TESTS_INPUT_FILES += $(TESTS) $(APPLICATIONS_TEST_BIN_FILES)`
And could also contain other, non generated files that are part of the repository and need to be taken into account. Currently, the usage PR is using other files not in `TESTS` as test inputs.
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the notifications