Remove dependency on deprecated module 'os-homedir'

[EightArmCode]

No description provided.

[demurgos]

Hi,
As mentioned in my other message, the goal would be to remove it from the next branch (not master).

I think the only change should be around this line:

spawn-wrap/src/lib/context.ts

Line 173 in 74e6528

return path.join(osHomedir(), DEFAULT_SHIM_ROOT_NAME);

(and remove imports/dependency)

[EightArmCode]

I see. When do you anticipate this update will be publishable to NPM?

[demurgos]

The next branch is temporarily published as demurgos-spawn-wrap (it is used as part of the c8 update).
The c8 update is the main blocker to merge the various forks. There is still a bit of work around the Node API, coverage branches and CLI handling. I'll try to work on this during the week-end. It will definitely be finished by the end of September (hopefully sooner). This fork will then be merged and published on npm as spawn-wrap (the goal is not to have a long-lived fork).

[EightArmCode]

alright sounds cool. I'll try to get to it today.

[EightArmCode]

So sorry I got busy and then have been gone on holiday. Going to create a PR on the next branch now...

[EightArmCode]

Alright - I've deleted my fork and re-forked/cloned and it appears that you've already merged my changes. Am I getting that right? The file you mentioned above doesn't seem to exist anymore, and I see my own "git blame" entry in the current version. Do you still need me to do anything with this?

[demurgos]

I don't believe I merged it (the next branch still uses os-homedir).

I prefer to keep the PR open at the moment. I opened many PRs to spawn-wrap and other libraries used for code coverage, but the maintainers seem to be busy.

[demurgos]

The PRs are starting to get merged to spawn-wrap. I'll make sure that this change is included.

[demurgos]

Fixed in the main spawn-wrap lib.