Pass tool directory and version as environment variable into postinstall hook

[iilyak]

I am trying to write an aqua registry.yaml for https://github.com/spotbugs/spotbugs. The release of spotbugs doesn't have an executable flag set. So I am trying to use the posthook to set it.

[tools]
"aqua:iilyak/spotbugs/spotbugs" = { version = '4.9.0', postinstall = """\
    chmod +x bin/spotbugs
"""}

❯ cat /Users/iilyak/Library/Caches/mise/aqua-registry/pkgs/iilyak/spotbugs/spotbugs/registry.yaml
packages:
  - name: spotbugs/spotbugs
    type: github_release
    repo_owner: spotbugs
    repo_name: spotbugs
    description: A tool for static analysis to look for bugs in Java code.
    asset: spotbugs-{{trimV .Version}}.tgz
    format: tar.gz

However the chmod command will fail because it runs in the current directory. I think the mise should pass MISE_TOOL_DIR (or some other suitably named) environment when it runs postinstall hook defined for the tool. The MISE_TOOL_VERSION would be nice as well.

[jdx]

any reason spotbugs can't set the executable bit in their tarball?

I'm also open to adding an env var around this but it definitely seems like something that should be fixed upstream too

[iilyak]

I was able to temporary workaround this issue. But it is bad. Ideally this should go into aqua registry spec. Some yaml field to update permission for a file matching the pattern. Requiring the user of the tool to do it manually every time is bad UX. I realize that it might be hard to convince aquaproj to change the spec.

"aqua:iilyak/spotbugs/spotbugs" = { version = '4.9.0', postinstall = """\
    mise where aqua:cloudant/spotbugs/spotbugs | xargs -I {} chmod +x {}/spotbugs/bin/spotbugs
"""}

[jdx]

this isn't a common problem, I don't think I've seen where we've needed to do this for a tool in the past