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 mxpv#270
  • Loading branch information
jtagcat committed Jan 2, 2022
1 parent 142b0e0 commit 97685e8
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 1 deletion.
2 changes: 2 additions & 0 deletions Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -7,5 +7,7 @@ RUN wget -O /usr/bin/youtube-dl https://github.com/ytdl-org/youtube-dl/releases/
apk --no-cache add ca-certificates python ffmpeg tzdata
COPY podsync /podsync

# (/app/)config.toml is relative, depends on /app working directory
WORKDIR /app
ENTRYPOINT ["/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 97685e8

Please sign in to comment.