diyhue / diyHue

Main diyHue software repo
https://diyhue.org/
Other
1.52k stars 274 forks source link

Remove coap-client binaries #887

Closed 7FM closed 1 year ago

7FM commented 1 year ago

Currently untested besides building the docker image.

mariusmotea commented 1 year ago

On what platform did you build the image? Maybe I can test on the others.

7FM commented 1 year ago

On what platform did you build the image? Maybe I can test on the others.

Only on my laptop (x86_64). This would be great :)

7FM commented 1 year ago

The CI seems to build the new docker images too, so at least building succeeds.

7FM commented 1 year ago

Hm, when hitting the tradfri pair button with the new docker image, the following error message appears in the log: EMRG coaps URI scheme not supported in this version of libcoap.

7FM commented 1 year ago

Apparently, libcoap2-bin is built without TLS support: TLS Library: None. I also tried libcoap-1-0-bin this version, however, does not support the -u option.

7FM commented 1 year ago

Got it, the package does provide coap-client-gnutls which should work!

mariusmotea commented 1 year ago

Hi. Is this ready to be merged?

7FM commented 1 year ago

Hi. Is this ready to be merged?

Yes

mariusmotea commented 1 year ago

Thanks for all your efforts .