Bug 2611 - mplayer2, an advanced general-purpose video player, can be installed alongside mplayer
Summary: mplayer2, an advanced general-purpose video player, can be installed alongsid...
Status: RESOLVED OLD
Alias: None
Product: Mageia
Classification: Unclassified
Component: New RPM package request (show other bugs)
Version: Cauldron
Hardware: All Linux
Priority: Normal enhancement
Target Milestone: ---
Assignee: Mageia Bug Squad
QA Contact:
URL: http://www.mplayer2.org/
Whiteboard: X
Keywords: PATCH
: 3273 7492 (view as bug list)
Depends on:
Blocks:
 
Reported: 2011-09-03 23:28 CEST by setlec seta
Modified: 2016-05-02 13:30 CEST (History)
10 users (show)

See Also:
Source RPM: mplayer2
CVE:
Status comment:


Attachments
specfile to build mplayer2 as mplayer2, to be installed alongside mplayer (1.09 KB, text/x-rpm-spec)
2011-10-20 22:11 CEST, Christian Lohmaier
Details
simple patch needed to also install the manpages as mplayer2 (713 bytes, patch)
2011-10-20 22:12 CEST, Christian Lohmaier
Details | Diff
spec file to build current (as of now) git master of mplayer2 (1.47 KB, text/x-rpm-spec)
2013-07-10 12:47 CEST, Christian Lohmaier
Details
patch to install manpages as "mplayer2" (873 bytes, patch)
2013-07-10 12:50 CEST, Christian Lohmaier
Details | Diff

Description setlec seta 2011-09-03 23:28:08 CEST
Description of problem:

mplayer2 is an advanced general-purpose video player. A fork of the original MPlayer project, it contains significant further development and supports a number of features not available in other Unix players.

http://www.mplayer2.org/comparison.html <== this website page shows the differences between mplayer with its fork, mplayer2.


Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.
2.
3.
Comment 1 D Morgan 2011-09-03 23:35:41 CEST
can both be installed on the same machine ?

CC: (none) => dmorganec

Comment 2 setlec seta 2011-09-04 00:44:45 CEST
honestly i preferred to remove mplayer 1 from my linux box in order to avoid any issue....
Comment 3 Philippe Didier 2011-09-04 02:07:22 CEST
If not both of them I would prefer to keep mplayer1 for this reason (written on the website) : 

"Dropped features
MEncoder is no longer available

The MEncoder codebase was thoroughly rotten and has been deleted. A different solution to provide some encoding functionality will be added in a future version."



Mplayer2 is strictly a videoplayer (better, certainly) but no more a swissknife as mplayer1 used by other apps  ! uninstalling mplayer1 implies you no longer use 2mandvd nor acidrip which need mencoder

CC: (none) => philippedidier

Comment 4 setlec seta 2011-09-04 06:40:48 CEST
it's a temporary features until a new api or solution is implemented mplayer2 is now a video player only. you can still use gstreamer enconding capabilities or ffmpeg api. I only use mplayer as video player so i don't care much about MEnconder.
Comment 5 Sander Lepik 2011-09-04 09:55:36 CEST
I use acidrip so i pretty much care and probably many other users do as well. If mplayer2 can't replace the current one fully then we should wait until it does.

CC: (none) => sander.lepik

Comment 6 Olivier FAURAX 2011-09-04 13:42:25 CEST
From the official website : "Currently mplayer2 uses the same binary and directory names as MPlayer"

So mplayer and mplayer2 can't cohabit.
As mplayer2 doesn't provide mencoder (used by many apps), this blocks the inclusion in Mageia.

The only solution is that mplayer2 be renamed to make the cohabitation possible.

CC: (none) => olivier

Comment 7 Christian Lohmaier 2011-10-20 18:50:31 CEST
dod anyone check whether it acutally is impossible to just keep using the mencoder package?

have the existing mplayer package build "mplayer1" and "mencoder" packages, and make mplayer2 the new mplayer (or add "Conflicts (or even Obsolets): mplayer" and keep it as mplayer2)

CC: (none) => lohmaier+mageia

Comment 8 Christian Lohmaier 2011-10-20 22:11:19 CEST
Created attachment 984 [details]
specfile to build mplayer2 as mplayer2, to be installed alongside mplayer

it's pretty easy to make the two not conflict with each other.

The attached specfile builds mplayer2 as mplayer2, so it can be installed alongside mplayer.

Only the installname is changed, it will still use mplayer's configuration files.

I didn't bother setting up all the buildrequires right and deal with tainted vs non-tainted and with the target-cpu (mplayer as well as mplayer2 default to optimizing for the build host's CPU, so a nogo for a RPM meant for distribution, but fine for personal use) this is left as an exercise for the actual packager.

To build it, just install mplayer's builddependencies, or look at configure's output and install the missing bits
Comment 9 Christian Lohmaier 2011-10-20 22:12:22 CEST
Created attachment 985 [details]
simple patch needed to also install the manpages as mplayer2

as with the spec itself, only the installname is changed, not the contents of the manpage.
Comment 10 Christian Lohmaier 2011-10-20 22:22:57 CEST
add --confdir=/etc/mplayer/ to the configure line, otherwise it will look for the default config in /usr/etc/mplayer/mplayer.conf (same for systemwide codecs.conf)
Comment 11 Manuel Hiebel 2011-11-06 13:53:19 CET
*** Bug 3273 has been marked as a duplicate of this bug. ***

CC: (none) => lda.thedevil.dark

Comment 12 Marja Van Waes 2011-12-30 21:57:57 CET
Setting version to cauldron, because package requests will usually be done for cauldron first and are then expected to be available in the first stable release that follows.

Keywords: (none) => PATCH
CC: (none) => marja11
Version: 1 => Cauldron
Summary: replace mplayer with mplayer2 (fork of the first mplayer) => mplayer2, an advanced general-purpose video player, can be installed alongside mplayer
Source RPM: (none) => mplayer2

Helge Hielscher 2012-01-02 07:10:37 CET

CC: (none) => hhielscher

Marja Van Waes 2012-02-17 16:39:37 CET

Whiteboard: (none) => X

Comment 13 Manuel Hiebel 2012-09-16 14:43:19 CEST
*** Bug 7492 has been marked as a duplicate of this bug. ***

CC: (none) => xpris

Comment 14 AngryPenguin 2013-07-06 14:56:45 CEST
What about mplayer2 pack for Mageia?
Please make rpm mplayer2 without conflict mplayer.
Comment 15 Christian Lohmaier 2013-07-10 12:47:47 CEST
Created attachment 4201 [details]
spec file to build current (as of now) git master of mplayer2

as with the previous version, this spec is mean to be compiled on the target-computer, as it doesn't use the runtime-cpu detection, but optimizes for the host-cpu. It also doesn't list the requries for vdpau, quicktime and similar. If you want full set of features, you should also install buildrequires of mplayer package.

Attachment 984 is obsolete: 0 => 1

Comment 16 Christian Lohmaier 2013-07-10 12:50:18 CEST
Created attachment 4202 [details]
patch to install manpages as "mplayer2"

note that while it installs as mplayer2, no string replacement has been done, so manpages, etc will continue to call it mplayer in most places, but it can be installed alongside mplayer package without problem.

Attachment 985 is obsolete: 0 => 1

Comment 17 Olivier FAURAX 2013-08-08 17:45:02 CEST
I tried to test the build of a RPM for mplayer2, but I didn't find the sources....
How can I get the sources at the correct revision from their website?
Is it possible to ask the mplayer2 team for a simple tar.gz release?
Comment 18 Christian Lohmaier 2013-08-08 18:50:10 CEST
The spec contains the pointer to the source:
Source0:	http://git.mplayer2.org/mplayer2/snapshot/mplayer2-6c87a98.tar.bz2


so do a "wget http://git.mplayer2.org/mplayer2/snapshot/mplayer2-6c87a98.tar.bz" or download with your browser or curl, or whatever... and put it into 
~/rpmbuild/SOURCES/

It is unlikely that mplayer2 team will provide official release tarballs themselves. They're basically in the same situation that mplayer was critisized for: "You don't do releases, thus distributions have outdated versions." So the solution for distros/user is to build from current sources themselves.

BTW: If you're like me and like your audio to have priority over video when switching playback-speed for example, you might want to revert http://git.mplayer2.org/mplayer2/commit/?id=213a224eec05cbd86b06abbdd843f2e252658b37

I'll also look into http://mpv.io/ - another fork (based on mplayer2 this time) - that for example did not include that change.
Comment 19 Olivier FAURAX 2013-08-08 19:50:40 CEST
Ok, sorry, I should have looked.

But the spec in this ticket doesn't work properly.
The %prep section is broken (should be: %setup -q -n %{name}-%{gitver}).
The build breaks on the manpages generation ("rst2man not found").

If there is another fork (with a DIFFERENT name, thanks guys!), let's wait to know on which we should really spend packaging time.
Comment 20 Christian Lohmaier 2013-08-08 23:45:24 CEST
well, apparently they updated their webfrontend to include the version info - so yes, you're right, setup should now include the version.

And getting the buildrequires right: that's why I didn't bother becoming a packager :-)
And finding out where rst2man comes from is not a problem (urpmf for example) 

I have quite a bit of stuff installed from other projects, so getting the requirements 100% correct was of no priority for me.

So Stick a "BuildRequires: python-docutils" in there and try again :-)

Quoting myself:
"I didn't bother setting up all the buildrequires right and deal with tainted vs non-tainted and with the target-cpu (mplayer as well as mplayer2 default to optimizing for the build host's CPU, so a nogo for a RPM meant for distribution, but fine for personal use) this is left as an exercise for the actual packager."

So if you want to actually package mplayer2, you need to configure it for using runtime cpu detection at least.

The spec was initially meant as a proof that they can both be installed alongside with very little effort. 

problem with a package with another name is that it has another name. Users don't know about it. you could do stuff and have it provide "mplayer2" and make mplayer2 only a virtual thing - but again that's something for the real packagers to decide. mplayer2 didn't receive new commits since april, while mpv still receives commits regularily.
Comment 21 José Jorge 2016-05-02 12:25:51 CEST
I think we can close this bug, now that mpv aka ex-mplayer2 is in cauldron and in 5 backports.

Status: NEW => RESOLVED
CC: (none) => lists.jjorge
Resolution: (none) => OLD

Comment 22 Christian Lohmaier 2016-05-02 13:30:32 CEST
yes, kill it, mpv is even better.

Note You need to log in before you can comment on or make changes to this bug.