elevenbuckets / elevenbuckets/11BE_SimpleWallet

should add "PATH: process.env.PATH" while spawn a subprocess

Open
#3 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
JavaScript
Stars
0
Forks
0
PR merge metrics
No merged PRs in 30d

Description

That is, [Line 78 of Wallet.js](https://github.com/elevenbuckets/11BE_SimpleWallet/blob/master/Wallet.js#L78) should change to something like:
`env: {DISPLAY: process.env.DISPLAY, XAUTHORITY: process.env.XAUTHORITY, PATH: process.env.PATH, configDir},`

In some cases there's no `node` in the default PATH, for example, people who use `nvm`. In such case, `launchGUI()` would fail due to cannot find `node` and return error message "path not found".

Contributor guide

No contributing guide indexed for this repository

Research direction

Start at line 78 of Wallet.js, where launchGUI() spawns the subprocess. Reproduce the failure in an environment using nvm or otherwise lacking node in the default PATH. Done means launchGUI() can find node and no longer returns the “path not found” error.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript
Domain
desktop
Issue type
Bug
Difficulty
1/5
Estimated time
Under an hour
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
55/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.