[license-header] refactor: excludes, updating, better logic #2780

Merged
crueter merged 3 commits from license-skips into master 2025-10-20 02:59:44 +02:00
Owner

Rewrite of license-header.sh. Lots of new goodies:

  • Fully POSIX compliant
  • supports dash arguments (-u/--update, -c/--commit, -uc for both)
  • can update year or malformed headers in-place now
  • accounts for shell scripts
  • exclusion of external files (e.g. sse2neon, my CMake modules/scripts)
  • better, more extensible logic all around
Rewrite of license-header.sh. Lots of new goodies: - Fully POSIX compliant - supports dash arguments (-u/--update, -c/--commit, -uc for both) - can update year or malformed headers in-place now - accounts for shell scripts - exclusion of external files (e.g. sse2neon, my CMake modules/scripts) - better, more extensible logic all around
Rewrite of license-header.sh. Lots of new goodies:
- Fully POSIX compliant
- supports dash arguments (-u/--update, -c/--commit, -uc for both)
- can update year or malformed headers in-place now
- accounts for shell scripts
- exclusion of external files (e.g. sse2neon, my CMake modules/scripts)
- better, more extensible logic all around

Signed-off-by: crueter <crueter@eden-emu.dev>
docs, fixups
All checks were successful
eden-license / license-header (pull_request) Successful in 30s
GitHub Actions [CI] Build succeeded
e2b5397500
Signed-off-by: crueter <crueter@eden-emu.dev>
nuke leftovers and add more externals
All checks were successful
eden-license / license-header (pull_request) Successful in 28s
GitHub Actions [CI] Build succeeded
d7f3e42a68
Signed-off-by: crueter <crueter@eden-emu.dev>
crueter merged commit d7cd7c6313 into master 2025-10-20 02:59:44 +02:00
crueter deleted branch license-skips 2025-10-20 02:59:45 +02:00
Member

Maybe we could move it to tools and nuke .ci and .github, and reference build docs to Eden-CI/Workflow repo?

Maybe we could move it to `tools` and nuke `.ci` and `.github`, and reference build docs to Eden-CI/Workflow repo?
Author
Owner

no, CI stuff should stay in-tree always. We can just update it if needed

no, CI stuff should stay in-tree always. We can just update it if needed
Member

@crueter wrote in #2780 (comment):

no, CI stuff should stay in-tree always. We can just update it if needed

Then we need to update it, the version locally don't include freebsd and android scripts

@crueter wrote in https://git.eden-emu.dev/eden-emu/eden/pulls/2780#issuecomment-7459: > no, CI stuff should stay in-tree always. We can just update it if needed Then we need to update it, the version locally don't include freebsd and android scripts
Sign in to join this conversation.
No description provided.