search

ioBroker / ioBroker.zwave

Legacy Z-Wave integration using OpenZWave
Other
24 stars 7 forks source link

Suppress permission denied warnings #74

Closed cburghardt closed 4 years ago

cburghardt commented 4 years ago

There are some confusing permission denied warnings because the tmp directory is used. This change suppresses them. I also removed the patch install check as I don't see the necessity.

AlCalzone commented 4 years ago

I'll wait until this is green, then merge and publish the next version

AlCalzone commented 4 years ago

Travis complains about unexpected redirects. This is because execSync defaults to /bin/sh, which does not support the redirects. Please update line 13:

- return(cp.execSync(script).toString());
+ return cp.execSync(script, { shell: '/bin/bash' }).toString();
cburghardt commented 4 years ago

Also improves #6

AlCalzone commented 4 years ago

Also improves #6

I don't see how ;)

Do you want to do anything else or should I merge?

cburghardt commented 4 years ago

Let's merge it, it is definitely already an improvement

AlCalzone commented 4 years ago

All released as v1.6.0