diff options
| author | Matthew Kenigsberg <matthew@floxdev.com> | 2024-10-09 16:26:24 -0600 |
|---|---|---|
| committer | Matthew Kenigsberg <matthew@floxdev.com> | 2024-10-11 08:07:43 -0600 |
| commit | 2fc61b161dfaddc4455fce44749e391934ce58e1 (patch) | |
| tree | 7f6af4afa55d84296be78b7a63327d50e4bb9f2e /pkgs/development/python-modules/python-mapnik/python-mapnik_std_optional.patch | |
| parent | 48c8d415bee57982a7659fe865fc52e521a18139 (diff) | |
granted: fix wrapping of assume script
This fixes a few problems with the assume script in the granted package.
- assume is currently being exec'ed by a wrapper script created with
wrapProgram, but it should be sourced. This causes the error:
```
line 172: return: can only `return' from a function or sourced script
```
- assume inspects argv[0] to check if an alias has been added to the
user's shell profile. This means a wrapper of any kind won't work,
even if it called source instead of exec, because source will still
set argv[0]. This means assume always tries to add the alias, but it
fails when it has already added the alias.
```
[i] To assume roles with Granted, we need to add an alias to your shell profile (https://docs.commonfate.io/granted/internals/shell-alias)
? Install zsh alias at /Users/matthew/.zshenv Yes
[✘] the Granted alias has already been added to /Users/matthew/.zshenv
```
- assume depends on assumego, so if it is run directly from e.g. a
result directory created with `nix build`, it will output
```
line 22: assumego: command not found
```
Fix all of these problems by dropping the wrapper around assume, and
instead insert a few lines into the start of the assume script that add
xdg-utils and granted to assume's PATH.
Diffstat (limited to 'pkgs/development/python-modules/python-mapnik/python-mapnik_std_optional.patch')
0 files changed, 0 insertions, 0 deletions
