gmg137 / netease-cloud-music-gtk

Linux 平台下基于 Rust + GTK 开发的网易云音乐播放器
GNU General Public License v3.0
1.5k stars 89 forks source link

fix set_position and add seeked() signal for `mpris-player` #251

Closed mokurin000 closed 7 months ago

mokurin000 commented 8 months ago

….git as upstream stopped maintaining

do not send PropertiesChanged on Position update

StyleContext::add_provider_for_display is deprecated

fix #112

gmg137 commented 8 months ago

mpris-player 已经停止维护了,其主页推荐的:https://github.com/pop-os/dbus-settings-bindings/tree/main/mpris2 也很久没更新了。mpris-player 的依赖版本都非常低,再在这个库上修补感觉意义不大。另外这种直接将依赖整体引入项目的做法也是不推荐的,最好还是能向上游贡献代码,然后在这里整体迁移过去。

mokurin000 commented 8 months ago

mpris-player 已经停止维护了,其主页推荐的:https://github.com/pop-os/dbus-settings-bindings/tree/main/mpris2 也很久没更新了。mpris-player 的依赖版本都非常低,再在这个库上修补感觉意义不大。另外这种直接将依赖整体引入项目的做法也是不推荐的,最好还是能向上游贡献代码,然后在这里整体迁移过去。

但是上游已经停止维护了,两年前和一年前的PR都不理不睬 作者自己却开了个 WIP migrate to zbus

mokurin000 commented 8 months ago

mpris-player 已经停止维护了,其主页推荐的:https://github.com/pop-os/dbus-settings-bindings/tree/main/mpris2 也很久没更新了。mpris-player 的依赖版本都非常低,再在这个库上修补感觉意义不大。另外这种直接将依赖整体引入项目的做法也是不推荐的,最好还是能向上游贡献代码,然后在这里整体迁移过去。

或者可以考虑迁移到 https://github.com/Mange/mpris-rs 看起来是仅客户端的库

mpris2-zbus 虽然九个月没有更新,但它的实现没有问题 只不过它没有提供阻塞实现,虽然去拉PR的工作量并不大

另外有个基于 zbus 的很新的 https://github.com/SeaDve/mpris-server

mokurin000 commented 8 months ago

@gmg137 接受使用 mpris-player 的新fork吗 我看了一下,glib和dbus迁移到最新版都没有什么问题

gmg137 commented 8 months ago

@gmg137 接受使用 mpris-player 的新fork吗 我看了一下,glib和dbus迁移到最新版都没有什么问题

这样最好了,只要改下依赖就行了,代码基本不用动。如果可以建议提交到 crates.io。

mokurin000 commented 8 months ago

@gmg137 接受使用 mpris-player 的新fork吗 我看了一下,glib和dbus迁移到最新版都没有什么问题

这样最好了,只要改下依赖就行了,代码基本不用动。如果可以建议提交到 crates.io。

改好了喵

gmg137 commented 8 months ago

请先解决冲突

mokurin000 commented 8 months ago

请先解决冲突

Screenshot_20231023-095300_Kiwi Browser

mokurin000 commented 8 months ago

请先解决冲突

github bug?我回家手动merge你main

gmg137 commented 8 months ago

我这里显示有冲突无法合并,等有时间我再手动合并看看

截图 2023-10-23 10-21-24