Repository navigation
Bug: Remove checkShebangLine multi arguments check #5459
Description
Input Code
$ ./a.coffee aaa
a.coffee: (--no-header used as proof)
#!/usr/bin/env -S npx coffee --no-header
console.log process.argvExpected 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
Activity
GeoffreyBooth commentedon Dec 11, 2023
As of 2023 there are still many Linux distros where it’s unsupported: nodejs/node#49295 (comment).