ironicbadger / pms-wiki

The aim is to share knowledge and information about building an open-source media server.
https://perfectmediaserver.com
Other
405 stars 55 forks source link

Please go ahead and fix your Dockerfile and requirements.txt #110

Closed evrial closed 7 months ago

evrial commented 7 months ago

This is pathetic

RealOrangeOne commented 7 months ago

What exactly is broken about them?

If you're going to be rude, at least have the decency to give some detail as to why it's "pathetic"

evrial commented 7 months ago
docker compose up --build
[+] Building 0.3s (2/2) FINISHED                                                                                                                          docker:default
 => [pms-wiki internal] load build definition from Dockerfile                                                                                                       0.0s
 => => transferring dockerfile: 237B                                                                                                                                0.0s
 => ERROR [pms-wiki internal] load metadata for git.ktz.me/alexktz/mkdocs-material-insiders:latest                                                                  0.1s
------
 > [pms-wiki internal] load metadata for git.ktz.me/alexktz/mkdocs-material-insiders:latest:
------
WARN[0000] current commit information was not captured by the build  error="failed to read current commit information with git rev-parse --is-inside-work-tree"
failed to solve: git.ktz.me/alexktz/mkdocs-material-insiders:latest: failed to do request: Head "https://git.ktz.me/v2/alexktz/mkdocs-material-insiders/manifests/latest": dial tcp: lookup git.ktz.me on 1.1.1.1:53: no such host
~/pms-wiki$ mkdocs serve
INFO    -  DeprecationWarning: 'materialx.emoji.twemoji' is deprecated.
           Material emoji logic has been officially moved into mkdocs-material
           version 9.4. Please use Material's 'material.extensions.emoji.twemoji'
           instead of 'materialx.emoji.twemoji' in your 'mkdocs.yml' file.
       markdown_extensions:
         - pymdownx.emoji:
             emoji_index: !!python/name:material.extensions.emoji.twemoji
             emoji_generator: !!python/name:material.extensions.emoji.to_svg
       ```

       'mkdocs_material_extensions' is deprecated and will no longer be
       supported moving forward. This is the last release.

         File "/home/dietpi/.local/lib/python3.11/site-packages/materialx/emoji.py", line 118, in twemoji
           return _patch_index(options)
         File "/home/dietpi/.local/lib/python3.11/site-packages/materialx/emoji.py", line 68, in _deprecated_func
           warnings.warn(

WARNING - Material emoji logic has been officially moved into mkdocs-material version 9.4. Please use Material's 'material.extensions.emoji.twemoji' instead of 'materialx.emoji.twemoji' in your 'mkdocs.yml' file.

       ```
       markdown_extensions:
         - pymdownx.emoji:
             emoji_index: !!python/name:material.extensions.emoji.twemoji
             emoji_generator: !!python/name:material.extensions.emoji.to_svg
       ```

       'mkdocs_material_extensions' is deprecated and will no longer be
       supported moving forward. This is the last release.

WARNING - Config value 'plugins': Plugin 'material/privacy' option 'links_attr_map': Unrecognised configuration name: links_attr_map WARNING - Config value 'plugins': Plugin 'material/social' option 'cards_layout': Unrecognised configuration name: cards_layout INFO - Building documentation... WARNING - Unrecognised configuration name: ironicbadger INFO - Cleaning site directory INFO - Downloading external file: https://unpkg.com/mermaid@10.7.0/dist/mermaid.min.js INFO - The following pages exist in the docs directory, but are not included in the "nav" configuration:

Aborted with a BuildError!

RealOrangeOne commented 7 months ago

This uses the insiders version of mkdocs-material, which is private. So you won't be able to build the container sadly. I suspect the reason your build is failing is because there's something in the configuration which depends (or at least assumes) it's running on the insiders build. Sure, they're annoying, and make running a full local build difficult (or impossible perhaps without some TLC), but I wouldn't call them "pathetic".

evrial commented 7 months ago

In first case the DNS has no A record, in second that's not my business, it is what it is 🤷‍♀️

ironicbadger commented 7 months ago

It's an insiders build, as orange says. This is not a public image, go have a word with upstream if that is a problem to you.

Second, that is a private git server running in my house behind Tailscale, so you won't be able to access it (by design).

evrial commented 7 months ago

Ok thanks, as I expected.