Merge pull request #504 from engnadeau:build/phony-targets

PiperOrigin-RevId: 477022393
Change-Id: I3c808e475e7af4fabb09d4cb1b045538dced3045

doc/Makefile

@@ -1,18 +1,19 @@
 # Minimal makefile for Sphinx documentation
 
+.DEFAULT_GOAL := help
+
 # You can set these variables from the command line.
 SPHINXOPTS    =
 SPHINXBUILD   = sphinx-build
 SOURCEDIR     = .
 BUILDDIR      = _build
 
-# Put it first so that "make" without argument is like "make help".
+.PHONY: help
 help:
 	@$(SPHINXBUILD) -M help "$(SOURCEDIR)" "$(BUILDDIR)" $(SPHINXOPTS) $(O)
 
-.PHONY: help Makefile
-
 # Catch-all target: route all unknown targets to Sphinx using the new
 # "make mode" option.  $(O) is meant as a shortcut for $(SPHINXOPTS).
+.PHONY: Makefile
 %: Makefile
 	@$(SPHINXBUILD) -M $@ "$(SOURCEDIR)" "$(BUILDDIR)" $(SPHINXOPTS) $(O)

sample/Makefile

@@ -4,6 +4,7 @@
 
 COMMON=-O2 -I../include -L../lib -std=c++17 -pthread -Wl,-no-as-needed -Wl,-rpath,'$$ORIGIN'/../lib
 
+.PHONY: all
 all:
 	$(CXX) $(COMMON) testxml.cc            -lmujoco          -o ../bin/testxml
 	$(CXX) $(COMMON) testspeed.cc          -lmujoco          -o ../bin/testspeed

sample/Makefile.macos

@@ -10,6 +10,7 @@ CFLAGS=-O2 -F$(MUJOCOPATH) -I$(GLFWROOT)/include -pthread
 CXXFLAGS=$(CFLAGS) -std=c++17 -stdlib=libc++
 ALLFLAGS=$(CXXFLAGS) -L$(GLFWROOT)/lib -Wl,-rpath,$(MUJOCOPATH)
 
+.PHONY: all
 all:
 	clang++ $(ALLFLAGS)    testxml.cc    -framework mujoco        -o testxml
 	clang++ $(ALLFLAGS)    testspeed.cc  -framework mujoco        -o testspeed

sample/Makefile.windows

@@ -8,6 +8,7 @@
 
 COMMON=/O2 /MT /EHsc /arch:AVX /I../include /Fe../bin/
 
+.PHONY: all
 all:
 	cl $(COMMON) testxml.cc                                ../lib/mujoco.lib
 	cl $(COMMON) testspeed.cc                              ../lib/mujoco.lib