denon icon indicating copy to clipboard operation
denon copied to clipboard

error: Uncaught (in promise) Error: not implemented p.kill("SIGKILL")

Open vsmahesh opened this issue 4 years ago • 2 comments

I am getting the following exception in the console

[*] [daem] restarting due to changes... error: Uncaught (in promise) Error: not implemented p.kill("SIGKILL"); ^ at deno:core/01_core.js:106:46 at unwrapOpResult (deno:core/01_core.js:126:13) at Object.opSync (deno:core/01_core.js:140:12) at opKill (deno:runtime/js/40_process.js:18:10) at Process.kill (deno:runtime/js/40_process.js:92:7) at Daemon.killAll (https://deno.land/x/[email protected]/src/daemon.ts:92:11) at Daemon.reload (https://deno.land/x/[email protected]/src/daemon.ts:35:10) at Daemon.iterate (https://deno.land/x/[email protected]/src/daemon.ts:181:22) at iterate.next () at https://deno.land/x/[email protected]/denon.ts:167:20.

  • OS: windows 10
  • Deno version: deno 1.14.1 (release, x86_64-pc-windows-msvc) v8 9.4.146.15 typescript 4.4.2
  • Denon version: v2.4.9

vsmahesh avatar Oct 10 '21 13:10 vsmahesh

I got an installation error also related to "SIGKILL"

$ deno install -qAf --unstable https://deno.land/x/denon/denon.ts
error: TS2345 [ERROR]: Argument of type 'string' is not assignable to parameter of type 'number'.
        p.kill("SIGKILL");
               ~~~~~~~~~
    at https://deno.land/x/[email protected]/src/daemon.ts:92:16

TS2345 [ERROR]: Argument of type 'string' is not assignable to parameter of type 'number'.
        p.kill("SIGKILL");
               ~~~~~~~~~
    at https://deno.land/x/[email protected]/src/daemon.ts:96:16

TS2724 [ERROR]: 'Deno' has no exported member named 'Signal'. Did you mean 'signal'?
      const signs: Deno.Signal[] = [
                        ~~~~~~
    at https://deno.land/x/[email protected]/src/daemon.ts:149:25

Found 3 errors.

did deno upgrade and tried again - that worked 👍

ghost avatar Oct 23 '21 20:10 ghost

denon 2.4.9 is incompatible with deno 1.16, this issue can be closed.

nnmrts avatar Nov 23 '21 06:11 nnmrts