FS#12466 - mplayer.desktop should be “mplayer” instead of “gmplayer”

Attached to Project: Arch Linux
Opened by changed user account to dsohler (Dirk.Sohler) - Monday, 15 December 2008, 19:09 GMT
Last edited by Roman Kyrylych (Romashka) - Monday, 26 January 2009, 10:53 GMT
Task Type Feature Request
Category Packages: Extra
Status Closed
Assigned To Hugo Doria (hdoria)
Architecture All
Severity Low
Priority Normal
Reported Version None
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 2
Private No

Details

Description:
If i install MPlayer, or it will be updated via Pacman, it will be „gmplayer“, when i open a video by using a program, that supports the mime type definitions (like PCManFM). If i look at /usr/share/applications/mplayer.desktop, i see “gmplayer %F” for “TryExec” and “Exec=”. But the name shows MPlayer.

I expect “mplayer” starting (video window only) instead of “gmplayer” (video window and GUI) when setting the MIME aware application to “MPlayer”. (I always need to change this by hand after MPlayer update …)

Additional info:
* Package: extra/mplayer 1.0rc2-9
* Version shown by MPlayer: MPlayer 1.0rc2-4.3.2 (C) 2000-2007 MPlayer Team
* Default configuration with unaltered /usr/share/applications/mplayer.desktop

Steps to reproduce:
* Install or update MPlayer
* Set MIME aware application to MPlayer
* Load a video file with this application

Suggestion/solution:
* Create two *.desktop-Files: One loading MPlayer and one loading GMPlayer.
This task depends upon

Closed by  Roman Kyrylych (Romashka)
Monday, 26 January 2009, 10:53 GMT
Reason for closing:  Upstream
Additional comments about closing:  Dirk Sohler requested closure with "not an Arch bug :)"
Comment by Hugo Doria (hdoria) - Friday, 19 December 2008, 15:45 GMT
I think that gmplayer should be the one in the menus. Most of the people use it. Also, its just a GUI. IMHO, we do not need to add another desktop file. The one we have is fine.
Comment by changed user account to dsohler (Dirk.Sohler) - Friday, 19 December 2008, 20:38 GMT
Then the name in the *.desktop file should be “GMPlayer” instead of “MPlayer”, because if i set the application to “MPlayer” i expect starting “MPlayer” nad not “GMPlayer”
Comment by Giorgio Lando (patroclo7) - Tuesday, 23 December 2008, 11:02 GMT
I find it very annoying too. gmplayer is not mplayer, it is a different executable, so they need two *.desktop files.
Comment by Hugo Doria (hdoria) - Thursday, 22 January 2009, 11:06 GMT
Fixed in mplayer 28347-1.
Comment by changed user account to dsohler (Dirk.Sohler) - Thursday, 22 January 2009, 19:44 GMT
you guys fixed nothing!

i updated and guess what? my edited .desktop-file was CHANGED TO GMPLAYER again!
Comment by Aaron Griffin (phrakture) - Thursday, 22 January 2009, 19:45 GMT
Aren't we just shipping upstream desktop files? Why not post this bug report on the mplayer bug tracker?
Comment by changed user account to dsohler (Dirk.Sohler) - Thursday, 22 January 2009, 22:18 GMT
I don’t know what should be shipped, but after updating to MPlayer 28347-1 the “mplayer.desktop” file contained “gmplayer” again. Why it is not possible to ship “gmplayer.desktop” containing “gmplayer” and “mplayer.desktop” containing “mplayer” instead of only shipping “mplayer.desktop” containing “gmplayer”?

I had enough! I just added /usr/share/applications/mplayer.desktop to the NoUpdate section in /etc/pacman.conf
Comment by Aaron Griffin (phrakture) - Thursday, 22 January 2009, 22:28 GMT
With my comment, I was intending to say this:
With as many of our packages as possible we try to ship what the upstream developers intended. If they intend "gmplayer" to be in the desktop file, then that is their prerogative.

It would make more sense to yell at the upstream devs about this. I think this is done intentionally. I suspect that it is similar to the way vim works - gvim has a desktop file, vim does not.

But yes, NoUpdate is the obvious solution here.

EDIT: Wow, english fail. than->then there->their
Comment by Hugo Doria (hdoria) - Friday, 23 January 2009, 13:41 GMT
Humm..

I agree with Aaron.

I will keep the upstream .desktop file in the next update. Also, i will report a bug about this in the mplayer project.
Comment by changed user account to dsohler (Dirk.Sohler) - Friday, 23 January 2009, 15:45 GMT
So it’s not because you modified the original files too much, it’s because the original MPlayer files are messed up and the maintainers of most of the Linux distributions fix this already and providing two packages? (Ubuntu ships “mplayer” and “mplayer-nogui” and Debian iirc does not ship GMPlayer at all.)

So your (i would say “our” *g*) package is less tweaked as the packages in other distributions? (what i totally prefer over overtweaked packages!)

Well, in this case this is more like a MPlayer bug, instead an Arch bug ;)
Comment by Dirk (dsohler) - Friday, 23 January 2009, 15:50 GMT
for notification because of user account change

Loading...