Skip to content

use #!/usr/bin/perl instead of #!/usr/bin/env perl#80

Merged
toddr merged 1 commit intomainfrom
koan.toddr.bot/fix-shebang-lines
Apr 23, 2026
Merged

use #!/usr/bin/perl instead of #!/usr/bin/env perl#80
toddr merged 1 commit intomainfrom
koan.toddr.bot/fix-shebang-lines

Conversation

@toddr-bot
Copy link
Copy Markdown
Contributor

@toddr-bot toddr-bot commented Apr 23, 2026

What

Replace #!/usr/bin/env perl with #!/usr/bin/perl in all scripts and tests, and document the convention in CLAUDE.md.

Why

MakeMaker's fixin rewrites shebangs at install time to point to the perl that built the distribution, but it only recognizes the canonical #!/usr/bin/perl form. Scripts using #!/usr/bin/env perl bypass this mechanism, so installed scripts may find the wrong perl.

How

Straightforward sed across 27 files (tests + examples + one .pm in examples/lib). Added a Shebangs bullet to the Conventions section of CLAUDE.md.

Testing

Full test suite passes after the change.

🤖 Generated with Claude Code


Quality Report

Changes: 28 files changed, 30 insertions(+), 27 deletions(-)

Code scan: clean

Tests: passed (OK)

Branch hygiene: clean

Generated by Kōan post-mission quality pipeline

@toddr toddr marked this pull request as ready for review April 23, 2026 14:25
MakeMaker's fixin adjusts the shebang at install time to point
to the perl that built the distribution, but it only recognizes
the canonical #!/usr/bin/perl form. Document this in CLAUDE.md
and fix all 27 scripts/tests.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@toddr-bot
Copy link
Copy Markdown
Contributor Author

Rebase with conflict resolution

Branch koan.toddr.bot/fix-shebang-lines was rebased onto main with automatic conflict resolution.

Stats

27 files changed, 29 insertions(+), 26 deletions(-)
Actions performed
  • Already-solved check: negative (confidence=high, reasoning=None of the recent commits on main address shebang line changes from env perl to direct perl paths; )
  • Resolved merge conflicts (1 round(s))
  • Rebased koan.toddr.bot/fix-shebang-lines onto origin/main
  • Pre-push CI check: no CI runs found
  • Force-pushed koan.toddr.bot/fix-shebang-lines to origin
  • CI check enqueued in ## CI (async)

CI status

CI will be checked asynchronously.


Automated by Kōan

@toddr-bot toddr-bot force-pushed the koan.toddr.bot/fix-shebang-lines branch from 04b616c to d7c3699 Compare April 23, 2026 14:26
@toddr toddr merged commit f4f2b7a into main Apr 23, 2026
@toddr toddr deleted the koan.toddr.bot/fix-shebang-lines branch April 23, 2026 14:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants