Skip to content

Commit

Permalink
Split up dependency graph generation to part inside and part outside …
Browse files Browse the repository at this point in the history
…docker
  • Loading branch information
yforster committed Sep 5, 2023
1 parent 0c15131 commit 6c0a345
Show file tree
Hide file tree
Showing 2 changed files with 15 additions and 2 deletions.
9 changes: 9 additions & 0 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -47,6 +47,7 @@ jobs:
startGroup "Build project"
opam exec -- make -j 2
opam exec -- make coqdoc
opam exec -- make docs/dependency_graph.pre
endGroup
uninstall: |
startGroup "Clean project"
Expand All @@ -62,6 +63,14 @@ jobs:
- name: Build dependency graph
run: make depgraph

- name: Install graphviz
uses: ts-graphviz/setup-graphviz@v1

- name: Build dependency graph
run: |
tred docs/dependency_graph.pre > docs/dependency_graph.dot
dot -T png docs/dependency_graph.dot -o docs/dependency_graph.png
- name: Build doc overview
uses: docker://pandoc/core:2.9
with:
Expand Down
8 changes: 6 additions & 2 deletions Makefile.coq.local
Original file line number Diff line number Diff line change
Expand Up @@ -24,8 +24,12 @@ coqdoc: $(GLOBFILES) $(VFILES) $(HEADER) $(FOOTER) $(RESOURCES) $(FRONTPAGE)
clean::
$(HIDE)rm -rf $(COQDOCDIR)/*.html

$(DOCDIR)/dependency_graph.dot: $(VFILES)
coqdep -Q theories LogRel -R coq-partialfun/theories PartialFun $(VFILES) | perl generate_deps.pl | tred > $(DOCDIR)/dependency_graph.dot
$(DOCDIR)/dependency_graph.pre: $(VFILES)
coqdep -Q theories LogRel -R coq-partialfun/theories PartialFun $(VFILES) | perl generate_deps.pl > $(DOCDIR)/dependency_graph.pre

$(DOCDIR)/dependency_graph.dot: $(DOCDIR)/dependency_graph.pre
rm -f $(DOCDIR)/dependency_graph.dot
tred $(DOCDIR)/dependency_graph.pre > $(DOCDIR)/dependency_graph.dot

$(DOCDIR)/dependency_graph.png: $(DOCDIR)/dependency_graph.dot
dot -T png $(DOCDIR)/dependency_graph.dot -o $(DOCDIR)/dependency_graph.png
Expand Down

0 comments on commit 6c0a345

Please sign in to comment.