search

mrtazz / checkmake

experimental linter/analyzer for Makefiles
MIT License
1.02k stars 44 forks source link

Support PHONY variable #73

Closed rasa closed 1 year ago

rasa commented 1 year ago

Support:

PHONY += all
all:
   true

.PHONY: ${PHONY}

It currently gives the error: Target "all" should be declared PHONY

mrtazz commented 1 year ago

I don't think we can reasonably support runtime variables in a static linter like this because this more or less requires us to be able to execute the Makefile and inspect generated variable content.

Please feel free to reopen this issue if anyone has a suggestion how to do this.