Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

repeated make clean should not show error #132

Closed
TsinghuaRoger opened this issue Jun 17, 2022 · 2 comments
Closed

repeated make clean should not show error #132

TsinghuaRoger opened this issue Jun 17, 2022 · 2 comments

Comments

@TsinghuaRoger
Copy link
Contributor

Repeat "make clean" should not show error
@TsinghuaRoger
Copy link
Contributor Author

$make clean
GO111MODULE=on go clean -i ./...
rm -rf *.out
cd parser && make clean
go clean -i ./...
rm -rf *.out
rm parser.go
rm: parser.go: No such file or directory
make[1]: *** [clean] Error 1
make: *** [clean] Error 2

TsinghuaRoger pushed a commit to TsinghuaRoger/tinysql-official that referenced this issue Jun 17, 2022
@yanguwan
issue talent-plan#132
22c5ace
yanguwan added a commit that referenced this issue Jun 17, 2022
@TsinghuaRoger @yanguwan
issue #132 (#133)
9ef1e9d 
Co-authored-by: yanguwan <[email protected]>
@yanguwan
Copy link
Contributor

solved in pr repeated "make clean" silently #133

yanguwan added a commit that referenced this issue Jun 17, 2022
@TsinghuaRoger @yanguwan
add fail point-disable as dependency to clean in Makefile (#134)
62fc082 
* issue #132

* add failpoint-disable as dependeny of clean

Co-authored-by: yanguwan <[email protected]>
pupillord pushed a commit to pupillord/tinysql that referenced this issue Jul 10, 2022
@TsinghuaRoger @yanguwan @pupillord
issue talent-plan#132 (talent-plan#133)
56141c1 
Co-authored-by: yanguwan <[email protected]>
pupillord pushed a commit to pupillord/tinysql that referenced this issue Jul 10, 2022
@TsinghuaRoger @yanguwan @pupillord
add fail point-disable as dependency to clean in Makefile (talent-pla…
5d6fe9c 
…n#134)

* issue talent-plan#132

* add failpoint-disable as dependeny of clean

Co-authored-by: yanguwan <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@yanguwan @TsinghuaRoger