Skip to content

Commit

Permalink
readd WORKDIR /app
Browse files Browse the repository at this point in the history
`config.toml` uses relative path, from the current directory, `$PWD` by default is `/`.

commit f337f6c
removed `WORKDIR /app`, and `/config.toml` was used instead of `/app/config.toml`.
The change was breaking, unexpected and undocumented.
As users already mount `/app/config.toml`, revert to using it.

Revert 16315c0
(what was workaround for breaking change,
though there is no need to break anything here)

Closes #270

+ requested: podsync located again at /app/podsync, instead of /podsync
  • Loading branch information
jtagcat committed Jan 2, 2022
1 parent 142b0e0 commit 34c708d
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 3 deletions.
7 changes: 5 additions & 2 deletions Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -2,10 +2,13 @@
# See docs for details: https://goreleaser.com/customization/docker/

FROM alpine:3.10
WORKDIR /app

RUN wget -O /usr/bin/youtube-dl https://github.com/ytdl-org/youtube-dl/releases/latest/download/youtube-dl && \
chmod +x /usr/bin/youtube-dl && \
apk --no-cache add ca-certificates python ffmpeg tzdata
COPY podsync /podsync
COPY podsync /app/podsync


ENTRYPOINT ["/podsync"]
ENTRYPOINT ["/app/podsync"]
CMD ["--no-banner"]
2 changes: 1 addition & 1 deletion docker-compose.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,4 +9,4 @@ services:
- 80:80
volumes:
- ./data:/data/
- ./config.toml:/config.toml
- ./config.toml:/app/config.toml

0 comments on commit 34c708d

Please sign in to comment.