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.
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
test: remove Perl script errchk #25669
Comments
@ysmolsky You marked this as NeedsDecision but I'm not sure what decision you are looking for. |
If no one in the tree uses errchk, it should be deleted. If it turns out that someone else (some x repo?) uses it, they can just make a copy for themselves. I'm not aware of anywhere else in the tree that uses perl. You could confirm that by removing perl from your PATH and running all.bash. |
Better to make a Docker image with a fake perl in $PATH that returns an error, because some tests (net/http/cgi?) do a LookPath for Perl and skip if not found. |
Change https://golang.org/cl/115955 mentions this issue: |
Perl is still being used in these:
I will stop here with Perl evictions, if anyone feels up to it, please go ahead with test.cgi and host_test.go. |
No test uses the perl script
errchk
anymore. We can remove it from the source tree. Is there any other place from which we should removePerl
as well?The text was updated successfully, but these errors were encountered: