mirror of
https://github.com/RIOT-OS/RIOT.git
synced 2024-12-29 04:50:03 +01:00
doc: do not rebuild riot.css
- simplify the makefile - changing behavior depending on lessc being installed is surprising and makes the Makefile less readable - failing with `make: lessc: No such file or directory` is helpful, but `make: No rule to make target 'src/css/riot.css'` is not clearly indicating that `lessc` was not found - don't rebuild `riot.css` when generating HTML output - anyone touching the less file will have to manually call `make src/css/riot.css -C doc/doxygen` - this happens so rarely that implementing a convenient mechanism is not worth the trouble Fixes https://github.com/RIOT-OS/RIOT/issues/8122
This commit is contained in:
parent
7d16a1b289
commit
cfee27cfb2
@ -7,7 +7,7 @@ export STRIP_FROM_INC_PATH_LIST=$(shell \
|
||||
|
||||
# use lessc (http://lesscss.org/#using-less) for compiling CSS
|
||||
# It can also be installed in ubuntu with the `node-less` package
|
||||
LESSC ?= $(shell command -v lessc 2>/dev/null)
|
||||
LESSC ?= lessc
|
||||
|
||||
DOCUMENTATION_FORMAT ?= html
|
||||
|
||||
@ -16,7 +16,7 @@ doc: $(DOCUMENTATION_FORMAT)
|
||||
|
||||
# by marking html as phony we force make to re-run Doxygen even if the directory exists.
|
||||
.PHONY: html
|
||||
html: src/css/riot.css src/changelog.md
|
||||
html: src/changelog.md
|
||||
( cat riot.doxyfile ; echo "GENERATE_HTML = yes" ) | doxygen -
|
||||
|
||||
.PHONY: check
|
||||
@ -27,14 +27,12 @@ check: src/changelog.md
|
||||
man: src/changelog.md
|
||||
( cat riot.doxyfile ; echo "GENERATE_MAN = yes" ) | doxygen -
|
||||
|
||||
ifneq (,$(LESSC))
|
||||
src/css/riot.css: src/css/riot.less src/css/variables.less
|
||||
@$(LESSC) $< $@
|
||||
|
||||
src/css/variables.less: src/config.json
|
||||
@grep "^\s*\"@" $< | sed -e 's/^\s*"//g' -e 's/":\s*"/: /g' \
|
||||
-e 's/",\?$$/;/g' -e 's/\\"/"/g' > $@
|
||||
endif
|
||||
|
||||
src/changelog.md: src/changelog.md.tmp ../../release-notes.txt
|
||||
@./generate-changelog.py $+ $@
|
||||
|
Loading…
Reference in New Issue
Block a user