Skip to content

checksrc-all: rewrite in Perl, remove checksrc.bat#17882

Closed
vszakats wants to merge 3 commits intocurl:masterfrom
vszakats:drop-checksrc-bat
Closed

checksrc-all: rewrite in Perl, remove checksrc.bat#17882
vszakats wants to merge 3 commits intocurl:masterfrom
vszakats:drop-checksrc-bat

Conversation

@vszakats
Copy link
Copy Markdown
Member

@vszakats vszakats commented Jul 10, 2025 •
edited
Loading

checksrc.bat was outdated and required Perl for checksrc.pl anyway.
Rewrite checksrc-all.sh in Perl, making it usable in envs without
a POSIX shell.

@github-actions github-actions Bot added the CI Continuous Integration label Jul 10, 2025
@vszakats vszakats force-pushed the drop-checksrc-bat branch from ca7624a to 70bbd7b Compare July 10, 2025 08:48
@vszakats vszakats added the Windows Windows-specific label Jul 10, 2025
@vszakats vszakats changed the title checksrc-all: rewrite in Perl, drop checksrc.bat checksrc-all: rewrite in Perl, remove checksrc.bat Jul 10, 2025
@vszakats vszakats closed this in 88ff396 Jul 11, 2025
@vszakats vszakats deleted the drop-checksrc-bat branch July 11, 2025 10:29
@vszakats vszakats mentioned this pull request Jan 26, 2026
Closed
vszakats added a commit that referenced this pull request Jan 26, 2026
@vszakats
checksrc-all.pl: skip non-repository files
33f606c 
To avoid noise due to local C files when using automatic local checksrc
checks (e.g. via CMake `-DCURL_LINT=ON` option, or `curl-lint` target).

Also replace single-quote with double-quote in external git command, for
portability.

Follow-up to 88ff396 #17882
Follow-up to e785e89 #17376

Closes #20439
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

CI Continuous Integration script Windows Windows-specific

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@vszakats