----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/100965/#review3654 -----------------------------------------------------------
Ship it! Just ship it, amarok.desktop has been in <...>/kde4/ for ages. - Kevin On March 29, 2011, 10:52 a.m., Aurélien Gâteau wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/100965/ > ----------------------------------------------------------- > > (Updated March 29, 2011, 10:52 a.m.) > > > Review request for Amarok. > > > Summary > ------- > > Prepending "kde4" to the desktop file name is necessary for Ubuntu Sound Menu > (and maybe others) to find Amarok desktop file because it is installed in > $prefix/share/applications/kde4, not in $prefix/share/applications. > > See > http://standards.freedesktop.org/menu-spec/1.0/go01.html#term-desktop-file-id > for more info. > > > Diffs > ----- > > src/dbus/mpris2/Mpris2DBusHandler.cpp bd69a05 > > Diff: http://git.reviewboard.kde.org/r/100965/diff > > > Testing > ------- > > > Thanks, > > Aurélien > >
_______________________________________________ Amarok-devel mailing list Amarok-devel@kde.org https://mail.kde.org/mailman/listinfo/amarok-devel