search

owntracks / docker-recorder

Docker image for OwnTracks Recorder
151 stars 68 forks source link

Update config.mk #74

Closed ippocratis closed 9 months ago

ippocratis commented 9 months ago

The config.mk should reflect https://github.com/owntracks/recorder/commit/25cb0c3fa42a4aeac2f5d67abe382e726fe6b175 so that the Dockerfile is buildable https://github.com/owntracks/docker-recorder/issues/73#issuecomment-1917752943

jpmens commented 9 months ago

Merged, thank you!

You beat me to it: it was on my list for when we (very soon now) build new Recorder Debian packages and the docker image.

Appreciated. :-)

jpmens commented 9 months ago

@ippocratis have you tested this? Is the file correctly installed to /usr/share/owntracks/recorder/timezone16.bin ?

ippocratis commented 9 months ago

@ippocratis have you tested this? Is the file correctly installed to /usr/share/owntracks/recorder/timezone16.bin ?

Will the docker filepath be the same?

Does not look like it is there

docker exec -it owntracks-otrecorder ls /usr/share/

apk man udhcpc ca-certificates misc

And

docker exec -it owntracks-otrecorder find / -type f -iname '*timezone*' 2>/dev/null

Returns nothing

jpmens commented 9 months ago

It can't really be, hence the question. Dockerfile now has

RUN make install DESTDIR=/app

so the path should be somewhere there, but ignore for now: we will probably be building images tomorrow and I will check.

Thanks.

jpmens commented 9 months ago

Docker image for Recorder 0.9.7 built and is available on Docker Hub.