Skip to content
This repository has been archived by the owner on May 13, 2020. It is now read-only.

Reliance upon HOME environment variable #8

Open
mparry opened this issue Nov 1, 2017 · 0 comments
Open

Reliance upon HOME environment variable #8

mparry opened this issue Nov 1, 2017 · 0 comments

Comments

@mparry
Copy link
Contributor

mparry commented Nov 1, 2017

There are a couple of places (src/exec.ts and src/resources.ts) that rely upon the existence of a HOME environment variable. On Windows this will not exist by default. I think you should probably also be trying USERPROFILE. For example, I got it working by editing those files as follows (respectively):

function helmHome() {
    return process.env["HELM_HOME"]
         || filepath.join(process.env["HOME"] || process.env["USERPROFILE"], '.helm');
}
let home = shell.env["HOME"] || shell.env["USERPROFILE"];

I've now added a PR (see below) that makes the same change.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant