-
Notifications
You must be signed in to change notification settings - Fork 18
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Trying to install the Neolink Addon and get this error #10
Comments
Seems that the url changed, will have a look on Saturday and let you know. |
Will need to switch the version |
Also while searching for your repo after not starring it before I found this new fork that could be useful especially with Home Assistant |
Yes also found that, will switch to this |
(Came from the linked issue) Looking at your dockerfile is there any reason you don't use something like FROM quantumentangledandy/neolink:latest To start off the image. Then I think all you'll need is your run.sh This image has multi arch docker support so it should work for all your arch |
Could be possible yes, I had a reolink cam for about a week and created that addon. Never looked to deep. Will think about switching to Andy’s fork and check if it is possible to directly use his image. |
@QuantumEntangledAndy if I try to use your docker image I end up with:
if I try to use your binaries (instead of thirtythreefourty) I end up with:
|
you can find both approaches on the repo in branches andy-docker and andy-binary |
Ok, both are rather quick to fix I've detailed them in #11 and made a PR against the |
@stoniwankenobii can you please try 0.0.4 and reply if it works for you - we switched to the fork of Andy. @QuantumEntangledAndy thanks for your quick help; your docker image is working on my amd64 host. |
@QuantumEntangledAndy can you confirm, that your docker image runs with all the arch that my container was running before: AARCH64, AMD64, ARMHF, ARMV7, I386. And does your multi arch support gives support to even more archs? |
Installed and got it up and running. Will get the camera added tomorrow. |
We build it for linux/amd64,linux/arm/v7,linux/arm64,linux/386 so the 32/64bit x86 and the 32/64 bit arms (hardware floats) HF and v7 are the same as far as I'm aware |
@stoniwankenobii please be aware: if you want to use the newest features of andys fork you ne to rebuild my HA addon so that it forces to use the newest image provided by Andy. |
@stoniwankenobii I updated to 0.0.5 to support mqtt mode aswell, find details here: #12 |
Just thought I'd point out that the mqtt is in a bit of a odd state at the moment. I recently switched to a green thread model (helped reduce cpu among other things) and I haven't fully finished upgrading the mqtt to use it yet |
Failed to install add-on
The command '/bin/bash -o pipefail -c wget https://nightly.link/thirtythreeforty/neolink/workflows/build/master/release-arm64-buster.zip' returned a non-zero code:8
I have had the addon installed before not sure if that could cause an issue as well.
The text was updated successfully, but these errors were encountered: