[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:
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:
-------------- 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