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.

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

[Makefile/checklicense] Scalability limit on files to check #9763

Closed
djaglowski opened this issue May 5, 2022 · 0 comments · Fixed by #9775
Closed

[Makefile/checklicense] Scalability limit on files to check #9763

djaglowski opened this issue May 5, 2022 · 0 comments · Fixed by #9775
Assignees
Labels
bug Something isn't working

Comments

@djaglowski
Copy link
Member

This issue was noticed while attempting to bring the log-collection library into contrib. This change would bring in a substantial number of files, but as it turns out, the checklicense target fails due to an argument list that is too long. The failure message is very unhelpful as well.

make: execvp: /bin/sh: Argument list too long
make: *** Makefile.Common:79: checklicense] Error 127

Ultimately, the problem is that checklicense is being run with a list of files that is too long.

It turns out that when running checklicense in the root module, all files within the repo are listed and checked. However, golint also checks all files within each module's root directory. This isn't necessarily a problem in itself - it just means that most files are checked twice. However, it seems we cannot continue to increase the number of go files in the project without excluding at least some of this duplication, such that when checklicense runs for the root module, the list of files is limited.

#9762 demonstrates the problem in an isolated having the golint target running checklicense on an artificially long list of files.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant