Skip to content

Commit

Permalink
Update shebang to use /usr/bin/env
Browse files Browse the repository at this point in the history
  • Loading branch information
Juerd committed Aug 21, 2023
1 parent c43764a commit 6f110ee
Show file tree
Hide file tree
Showing 2 changed files with 18 additions and 2 deletions.
16 changes: 16 additions & 0 deletions UPGRADING.md
Original file line number Diff line number Diff line change
@@ -1,3 +1,19 @@
# (2023-08-21) RevBank 3.9

A tiny change that could break things: the shebang was changed from
`#!/usr/bin/perl` to the more modern `#!/usr/bin/env perl`.

In the unlikely event that your system has multiple perl executables in the
search path of `PATH`, this change could mean that revbank now uses a different
perl, in which case you may have to reinstall the required Perl libraries.

Background: NixOS doesn't follow the previously uni(x)versal convention that,
when Perl is available, an executable exists at `/usr/bin/perl`. The only
stable paths that NixOS provides for shebangs are `#!/bin/sh` or
`#!/usr/bin/env`. There were always pros and cons to switching the shebang to
`env` (e.g. for use with perlbrew), but learning about Nix has tipped the
scales for me. (The performance penalty is not relevant for RevBank.)

# (2023-02-12) RevBank 3.8

## Update your `revbank.plugins`
Expand Down
4 changes: 2 additions & 2 deletions revbank
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
#!/usr/bin/perl
#!/usr/bin/env perl

use v5.28;
use warnings;
Expand All @@ -18,7 +18,7 @@ use RevBank::Global;
use RevBank::Messages;
use RevBank::Cart;

our $VERSION = "3.8";
our $VERSION = "3.9";
our %HELP1 = (
"abort" => "Abort the current transaction",
);
Expand Down

0 comments on commit 6f110ee

Please sign in to comment.