[riot-notifications] [RIOT-OS/RIOT] doxygen/Makefile: do not rebuild riot.css automatically (#10249)

Gaƫtan Harter notifications at github.com
Mon Feb 11 18:27:52 CET 2019


I think we could keep only the `src/css/riot.css` target and just remove it from the dependencies of `html`. Having a phony target for rebuilding could help as shortcut but not sure if it is necessary.

I also noticed that re-creating the file with `lesscpy` discards the license header so may not even be suitable for generating it. It is documented in the `lesscpy` documentation:
https://lesscpy.readthedocs.io/en/latest/#differences-from-less-js
Re-creating the file with `lessc` gives me the same output as the tracked one.
I would remove the support for `lesscpy` completely in that case, or only support lesscpy and re-add the header manually.


If the file can be stability re-created, a check in static-tests could be nice to check if it was changed but forgotten. I would see it as a different pull request though.
Or a lazyer solution would be that the target does a:

```
    @sed -i '1 i\/* This file was generated by "make $@" it should be manually updated when it dependencies are modified */' $@
```

-- 
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/10249#issuecomment-462417175
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190211/1c6ff62b/attachment.html>


More information about the notifications mailing list