[Bug 4750] New: Review request: deepin-music - Deepin Music Player

https://bugzilla.rpmfusion.org/show_bug.cgi?id=4750 Bug ID: 4750 Summary: Review request: deepin-music - Deepin Music Player Product: Package Reviews Version: Current Hardware: x86_64 OS: GNU/Linux Status: NEW Severity: enhancement Priority: P1 Component: Review Request Assignee: rpmfusion-package-review(a)rpmfusion.org Reporter: sztsian(a)gmail.com CC: rpmfusion-package-review(a)rpmfusion.org, sensor.wen(a)gmail.com Blocks: 2 namespace: free Spec URL: https://zsun.fedorapeople.org/pub/pkgs/rpmfusion/deepin-music.spec SRPM URL: https://zsun.fedorapeople.org/pub/pkgs/rpmfusion/deepin-music-3.1.7.2-2.f... Description: Deepin Music Player Fedora Account System Username: zsun RPMFusion FAS Username: zsun * Why this package is not eligible to be included in Fedora: This depends on some codec that are not accepted in Fedora. * This is my second RPM Fusion package. (The other is bug 4749 which is also just filed) * I am a current Fedora Packager * RPMlint: $ rpmlint *.rpm deepin-music.src: I: enchant-dictionary-not-found zh_CN deepin-music.src: W: spelling-error %description -l en_US tweakful -> tweak deepin-music.src: W: spelling-error %description -l en_US gstreamer -> streamer, g streamer, steamer deepin-music.src: W: spelling-error %description -l en_US quanpin -> piquant deepin-music.x86_64: W: spelling-error %description -l en_US tweakful -> tweak deepin-music.x86_64: W: spelling-error %description -l en_US gstreamer -> streamer, g streamer, steamer deepin-music.x86_64: W: spelling-error %description -l en_US quanpin -> piquant For these above, they are from upstream and I believe they are expected. deepin-music.x86_64: E: zero-length /usr/share/dbus-1/services/com.deepin.dde.music.service Upstream provided an empty file. https://github.com/linuxdeepin/deepin-music/blob/master/music-player/data... deepin-music.x86_64: W: no-manual-page-for-binary deepin-music deepin-music.x86_64: E: invalid-desktopfile /usr/share/applications/deepin-music.desktop file contains group "Next Shortcut Group", but groups extending the format should start with "X-" deepin-music.x86_64: E: invalid-desktopfile /usr/share/applications/deepin-music.desktop file contains group "PlayPause Shortcut Group", but groups extending the format should start with "X-" deepin-music.x86_64: E: invalid-desktopfile /usr/share/applications/deepin-music.desktop file contains group "Previous Shortcut Group", but groups extending the format should start with "X-" For desktop file related. This music player is originally part of Deepin Desktop Environment(Short as DDE), and above are written under DDE's way. So I believe I shouldn't modify them. deepin-music-debuginfo.x86_64: E: useless-provides debuginfo(build-id) For debuginfo(build-id), all packages built in rawhide contains this, so I assume this won't hurt. deepin-music-debugsource.x86_64: W: no-documentation deepin-music-devel.x86_64: W: only-non-binary-in-usr-lib deepin-music-devel.x86_64: W: no-documentation 5 packages and 0 specfiles checked; 5 errors, 10 warnings. Referenced Bugs: https://bugzilla.rpmfusion.org/show_bug.cgi?id=2 [Bug 2] Tracker: New packages awaiting review -- You are receiving this mail because: You are on the CC list for the bug. You are the assignee for the bug.