Skip to content

Bug: Remove checkShebangLine multi arguments check #5459

Closed
@jonathan-rosa

Description

@jonathan-rosa

Input Code

$ ./a.coffee aaa

a.coffee: (--no-header used as proof)

#!/usr/bin/env -S npx coffee --no-header
console.log process.argv

Expected Behavior

$ ./a.coffee aaa
[ 'coffee', '[...]/a.coffee', 'aaa' ]

Current Behavior

$ ./a.coffee aaa
The script to be run begins with a shebang line with more than one
argument. This script will fail on platforms such as Linux which only
allow a single argument.
The shebang line was: '#!/usr/bin/env -S npx coffee --no-header' in file '[...]/a.coffee'
The arguments were: ["-S","npx","coffee","--no-header"]
[ 'coffee', '[...]/a.coffee', 'aaa' ]

Possible Solution

Remove checkShebangLine in coffeescript.coffee as it's no longer necessary.

Context

Linux now allows multiple arguments since coreutils 8.30 (July 1st, 2018), this function can be removed now.

Environment

  • Debian GNU/Linux 12 (bookworm)
  • CoffeeScript version: 2.7.0
  • Node.js version: 18.13.0

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions