重写配置模块

[glyh]

Closes https://github.com/Yuukiy/JavSP/issues/353 （删除了这项功能和大部分需要手动传入arg的功能）

• 重构配置文件，现在使用YAML保存配置文件 e096d83

• 除了-c,--config以外的其他命令行参数，都被改为与配置文件的命名一致的传入方法。 e096d83

  比如需要重写扫描的目录可以这样传入参数：

  poetry run javsp -- --oscanner.input_directory '/some/directory'

• 删除了所有环境变量，现在环境变量的传入方法如下。 e096d83

  env JAVSP_SCANNER.INPUT_DIRECTORY='/some/directory' poetry run javsp

• 为了辅助用户迁移，提供了迁移工具：/tools/config_migration.py
• 由于类型注释，Python版本被bump到了3.10

[glyh]

因为提供了迁移工具，用户应该不会遇到太多使用上的问题。

[qicfan]

发现个问题，media_servers这个配置项好像搞丢了，现在生成的nfo都叫movie.nfo，没办法兼容更多媒体服务器。

[glyh]

那个选项被我删掉了，因为之前有人提了issue说这个设置会报错。可以给出具体说明不同的配置应该输出什么样的文件格式吗？我重新实现一下。

[glyh]

我可以添加一个新的选项basename_pattern，让用户自己设置生成的nfo文件名的格式。

[qicfan]

movie.nfo可以被jellyfin或者emby识别。 {basename}.nfo可以被几乎所有媒体服务器识别。

报错可能是因为多CD影片或者文件名过长导致的 多CD影片报错可能是因为识别CD数量之类的有问题，不过这个是小众需求不常见，可以不用理会； 文件名过长这个概率比较大，现在应该已经解决了才对。

[qicfan]

我可以添加一个新的选项basename_pattern，让用户自己设置生成的nfo文件名的格式。

我觉得不用哈，就把以前那个逻辑还原回来就行啦，媒体服务器识别的格式就这两种。

[glyh]

原来的逻辑我本地运行会报错

[qicfan]

原来的逻辑我本地运行会报错

有示例文件名么，我测试下，我刚好要用这个功能，可以顺便PR，到时候再测试下（我先不加多CD兼容）

[glyh]

额，就是随便找了个文件都会报错。

https://github.com/Yuukiy/JavSP/issues/353

但我还是建议不要把原来的代码改回来，因为感觉旧代码没有类型注释可读性很差，我这里的LSP还一直在抱怨没法进行类型推导。

[glyh]

欢迎PR

[qicfan]

直接重新一个逻辑，不搬了，哈哈

[glyh]

建议API设计成这样：

[qicfan]

忘了提了，这个不仅仅影响nfo，包括poster和fanart都会影响 universal: movie.nfo, poster.jpg, fanart.jpg jellyfin: {basename}.nfo, {basename}-poster.jpg, {basename}-fanart.jpg

[qicfan]

以前API设计是合理的，还沿用即可。

[glyh]

hmm，我PR加一下这几个功能吧。主要是我希望他们尽可能generic，这样后面不需要为了各种各样的需求再改。

[qicfan]

OK，那你直接改吧，以前的cdx这部分可以先不要了，逻辑就还是这个逻辑，有错误解决即可，可能以前这部分分支太多了，容易错。

if 'universal' in cfg.NamingRule.media_servers:
    movie.nfo_file = os.path.join(save_dir, 'movie.nfo')
    movie.fanart_file = os.path.join(save_dir, 'fanart.jpg')
    movie.poster_file = os.path.join(save_dir, 'poster.jpg')
else:
    movie.nfo_file = os.path.join(save_dir, f'{basename}.nfo')
    movie.fanart_file = os.path.join(save_dir, f'{basename}-fanart.jpg')
    movie.poster_file = os.path.join(save_dir, f'{basename}-poster.jpg')

[glyh]

Done https://github.com/Yuukiy/JavSP/commit/d103767e870fe16ada6bfaf7774658fbc1777711 你可能需要重新运行migration脚本

[qicfan]

OK，关机啦，明儿试试。 dean @. ---- 回复的原邮件 ---- 发件人Yihang @.>发送日期2024年9月27日 @.>@.>, @.>主题Re: [Yuukiy/JavSP] 重写配置模块 (PR #378) Done d103767 — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you commented.Message ID: @.>

[qicfan]

[glyh]

我测试的时候没用javdb，可能是那个client哪里挂了 我看下

[qicfan]

OK