Closed northsea4 closed 6 months ago
kjtsune
commented
6 months ago 感谢 PR。 暂不考虑 .gitignore。配置文件冲突有解决方案，见：https://github.com/kjtsune/embyToLocalPlayer/pull/40#issuecomment-2026430965 可以关闭重新PR IINA的。下字幕有耗时，sub_file 可能为 None，以下更合理些。
if is_iina and sub_file and not mount_disk_mode:
srt = save_sub_file(url=sub_file)
cmd.append(f'--sub-files={srt}')
northsea4
commented
6 months ago 感谢 PR。 暂不考虑 .gitignore。配置文件冲突有解决方案，见：#40 (comment) 可以关闭重新PR IINA的。下字幕有耗时，sub_file 可能为 None，以下更合理些。
if is_iina and sub_file and not mount_disk_mode: srt = save_sub_file(url=sub_file) cmd.append(f'--sub-files={srt}')
IINA这个处理，还是麻烦你进行修改吧😅。我也没有太详细地阅读代码，只是大概测试出那个参数可行。
至于.gitignore，比如这样：我fork了这个仓库后，clone到本地进行开发，大概率我会修改ini配置文件，还有运行测试。那至少以下3个文件或目录就会被git检测到变动：
在进行git提交时，我需要做一些操作后才能把这几个文件或目录排除在外，因为不应该把它们提交。
如果有.gitignore，则不需要操心这些。
当然，开发习惯也是因人而异的，我只是出于个人经验说一下自己的想法。
另外也可以考虑添加一下开源协议。
kjtsune
commented
6 months ago 我现在没有 macOS 环境，不过应该也没什么问题了。
~PyCharm 没你说的那些问题~
了解了一下， .gitignore 有全局配置，通用的忽略放全局合理些。
.tmp， log.txt ，改名后的 .ini ，会放 .gitignore。
IINA外挂字幕
经过一番测试，发现
--sub-files=本地字幕可以加载成功。补充环境信息:
添加.gitignore
理由：使用
.gitignore更方便开发和协作。除了常见的忽略规则外，以下文件或目录也被忽略:
其中
embyToLocalPlayer_config.ini文件配置信息很可能是因人而异，所以忽略并更改为一个示例文件，即embyToLocalPlayer_config.samle.ini。 引入.gitignore后，您打包方面可能需要做一些调整，烦请跟进处理一下，另外可以考虑使用github actions进行自动构建处理。