[riot-notifications] [RIOT-OS/RIOT] make: don't execute tilde files on test target (#11414)

Martine Lenders notifications at github.com
Thu Apr 18 12:50:34 CEST 2019


> It is however the only place where we need to do this. As all other `wildcard` are used with a file extension.
> I would prefer to replace it by matching only `python` executable scripts as there are only python tests.
> 
> ```
> find tests/*/tests/  -type f -executable -exec basename {} \; | sort -u
> 01-make.py
> 01-run.py
> 02-bench.py
> 02-run.py
> test.py
> ```
> If at some point we need to add other tests file extensions, we could add them to the accepted list or maintain different lists with `PYTHONTESTS`, `SHELLTESTS` to allow maybe different handling.

I don't see your point. Why should we go away from the general approach now only to reintroduce it in a non-scalable manner later.


-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/RIOT-OS/RIOT/pull/11414#issuecomment-484451017
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190418/5c6c738e/attachment.html>


More information about the notifications mailing list