Skip to content

Commit

Permalink
Fixup documentation build scripts (#70)
Browse files Browse the repository at this point in the history
* add website related targets to top level Makefile for convenient full project clean

* restore a file revision controlled file after we clean the rest of the sphinx output

FIXME: however, running the sphinx build still alters that file
  • Loading branch information
bpkroth authored Sep 12, 2020
1 parent 9150f10 commit fc25569
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 1 deletion.
10 changes: 9 additions & 1 deletion Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@ handledtargets += cmake-build cmake-test cmake-clean \
cmake-buildfiles clean-cmake-buildfiles \
cmake-distclean $(CMAKE) \
python-checks python-test \
website website-clean \
grpc-clean mlos-codegen-clean

# Build using dotnet and the Makefile produced by cmake.
Expand All @@ -28,7 +29,7 @@ test: dotnet-test cmake-test python-test
check: all test

.PHONY: clean
clean: cmake-clean dotnet-clean grpc-clean mlos-codegen-clean
clean: cmake-clean dotnet-clean grpc-clean mlos-codegen-clean website-clean

.PHONY: distclean
distclean: clean cmake-distclean
Expand All @@ -46,6 +47,13 @@ mlos-codegen-clean: dotnet-clean
grpc-clean:
@ $(RM) $(MLOS_ROOT)/Grpc.out

.PHONY: website
website:
$(MAKE) -C website

.PHONY: website-clean
website-clean:
$(MAKE) -C website clean

# Build the dirs.proj file in this directory with "dotnet build"
include $(MLOS_ROOT)/build/DotnetWrapperRules.mk
Expand Down
1 change: 1 addition & 0 deletions website/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -40,6 +40,7 @@ public/index.html:
clean-sphinx-site:
rm -rf sphinx/_build
rm -rf sphinx/api
git restore sphinx/api/mlos.rst
rm -rf public/python_api

# Note: this is an expensive target.
Expand Down

0 comments on commit fc25569

Please sign in to comment.