search

Zeal-L / BiliBili-Manga-Downloader

一个好用的哔哩哔哩漫画下载器，拥有图形界面，支持关键词搜索漫画和二维码登入，黑科技下载未解锁章节，多线程下载，多种保存格式，本地漫画管理，一键检查更新！
GNU Affero General Public License v3.0
943 stars 51 forks source link

增加对 Linux 的支持 #76

Closed Nikaidou-Shinku closed 1 year ago

Nikaidou-Shinku commented 1 year ago
Zeal-L commented 1 year ago

好的，感谢贡献，我先测试一下功能

Zeal-L commented 1 year ago

@Nikaidou-Shinku

Nikaidou-Shinku commented 1 year ago

@Nikaidou-Shinku

  • 测试后发现你基本没考虑其他适配情况，例如各种打开文件夹以及文件的操作绑定，这些用的都是win api，在linux下只会报错
  • 不过还是先合并了
  • 我已经在 commit 8fb04b6 中完善了你的PR，可以参考学习一下
  • 总之对于你的贡献，我还是要表示感谢 ♥

这个 pr 最初只是为了满足我自己使用需要所进行的修改，所以我并没有进行很完善的测试，十分抱歉。

使用 webbrowser module 会在没有浏览器的环境下报错，我不知道这里有没有更好的选择，我不太熟悉 Python 的生态。

最后感谢你对跨平台支持的关注，欢迎继续讨论如何提升这一块的代码质量！

Zeal-L commented 1 year ago

这个 pr 最初只是为了满足我自己使用需要所进行的修改，所以我并没有进行很完善的测试，十分抱歉。

好的！没有关系！

使用 webbrowser module 会在没有浏览器的环境下报错，我不知道这里有没有更好的选择，我不太熟悉 Python 的生态。

最后感谢你对跨平台支持的关注，欢迎继续讨论如何提升这一块的代码质量！

在没有浏览器的环境下报错这问题我也发现了，所以目前暂时都在 develop 分支。 想要打开文件夹并且选中文件，在linux上好像很难找到一种自带通用的优雅的实现方式，我也还在研究中。有更多想法欢迎继续 PR