search

zaaack / node-systray

A cross-platform systray library for nodejs.
https://zaaack.github.io/node-systray/
MIT License
84 stars 26 forks source link

trayBin: fallback to self install? #7

Open cryptix opened 6 years ago

cryptix commented 6 years ago

I'm having problems with the way the helper binaries are stuffed under $HOME/.cache. It has problems on uncommon platforms and hides an external dependency.

Supporting other architectures, like arm(64), will also lead to an explosion of files in the trayBin folder and increase the size of the npm package much more.

Wouldn't it make more sense to clearly state the external dependeny and make individual packages for systray-portable? As a gopher myself I could help with that.

A more nodejs/self-contained approach would be native cxx module but that is not my arena of expertise.

zaaack commented 6 years ago

  1. You can specific custom copy directory via copyDir

  2. You are right, individual packages are better but its a little tedious for the benefits because I haven't add new features for a long time, especially currently I have no project is using this package. But I can review PRs and publish new packages if I have time, so prs are still welcome!

  3. I have think about this before, but go is not good to commuticate with node, I have tried rs-systray but it doesn't support macOS.