SuperNG6 / Docker-qBittorrent-Enhanced-Edition

Docker-qBittorrent-Enhanced-Edition
https://hub.docker.com/r/superng6/qbittorrentee
754 stars 77 forks source link

Fix typo #76

Closed ssfdust closed 3 weeks ago

ssfdust commented 3 weeks ago

/download和/config写反了

SuperNG6 commented 3 weeks ago

感谢PR,我在公司,暂时没有测试环境;PR 是否解决了这个问题?有进行测试吗

SuperNG6 commented 3 weeks ago

如果还未解决,我准备回退到 S6V2,V3 真的是复杂了很多,对于大部分容器跑的应用来说意义不是太大 https://github.com/SuperNG6/docker-baseimage-alpine

ssfdust commented 3 weeks ago

本地测试通过

  1. 具备chown权限,第一次初始化启动,正常设置权限

    ...
    Permssion of configuration directory is set properly.
    Permssion of download directory is not set properly, try to set now...
    ...
  2. 具备chown权限,第二次权限可用情况启动

    ...
    [qbittorrentee] | ───────────────────────────────────────
    [qbittorrentee] | Permssion of configuration directory is set properly.
    [qbittorrentee] | Permssion of download directory is set properly.
    [qbittorrentee] | [custom-init] No custom files found, skipping...
    [qbittorrentee] | [ls.io-init] done.
    ...
  3. 无chown权限,但是权限为可写的情况,输出与2相同

  4. 无chown权限,但是权限不可写的情况

    ...
    chown: changing ownership of '/config': Operation not permitted
    Permssion of download directory is not set properly, try to set now...
    chown: changing ownership of '/downloads': Operation not permitted
    s6-rc: warning: unable to start service qbittorrentee-pre: command exited 1
    ...
ssfdust commented 3 weeks ago

不过v3确实比v2复杂太多了,将来回退的话我也能理解,现在这个问题应该确实是解决了

SuperNG6 commented 3 weeks ago

已合并,并构建镜像。
现在的想法就是,这个镜像大部分的时候都是自动同步一下上游的二进制,如无必要不做大更改了,稳定为先