Skip to content

Commit

Permalink
fix: revert "fix: cleanup the clean target"
Browse files Browse the repository at this point in the history
This reverts commit 97aafa0.

As @viktorvoltaire pointed out, if we clean at the `.mage` level, we
will "clean" away any work-in-progress sagefiles - that would not be
good!
  • Loading branch information
odsod committed Jan 28, 2022
1 parent 97aafa0 commit 5d71b4f
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ $(sagefile): .sage/go.mod $(shell find .sage/.. -type f -name '*.go')

.PHONY: clean-sage
clean-sage:
@git clean -fdx .sage
@git clean -fdx .sage/tools .sage/bin

.PHONY: all
all: $(sagefile)
Expand Down
2 changes: 1 addition & 1 deletion sg/makefile.go
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@ func generateMakefile(g *codegen.File, pkg *doc.Package, mk Makefile, mks ...Mak
g.P()
g.P(".PHONY: clean-sage")
g.P("clean-sage:")
g.P("\t@git clean -fdx ", includePath)
g.P("\t@git clean -fdx ", filepath.Join(includePath, ToolsDir), " ", filepath.Join(includePath, BinDir))
forEachTargetFunction(pkg, func(function *doc.Func, namespace *doc.Type) bool {
if function.Recv == mk.namespaceName() {
g.P()
Expand Down

0 comments on commit 5d71b4f

Please sign in to comment.