Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 528808 - app-misc/anki USE=sound should depend on || ( media-video/mplayer media-video/mplayer2 )
Summary: app-misc/anki USE=sound should depend on || ( media-video/mplayer media-video...
Status: RESOLVED UPSTREAM
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal
Assignee: Thomas Kahle (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-11-10 11:56 UTC by Alexander Berntsen (RETIRED)
Modified: 2014-11-11 09:18 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Alexander Berntsen (RETIRED) gentoo-dev 2014-11-10 11:56:55 UTC
Please make anki use mplayer2 if it is installed, instead of pulling the rather large dependency mplayer (which is no longer developed either).
Comment 1 Jeroen Roovers (RETIRED) gentoo-dev 2014-11-10 12:29:44 UTC
Wouldn't that additionally require changing what executable is called? Also, this looks like an optional run-time dependency that shouldn't be pulled in through a USE flag in the first place.
Comment 2 Thomas Kahle (RETIRED) gentoo-dev 2014-11-10 12:43:25 UTC
From the doc:

"Anki uses a program called mplayer in order to support sounds and videos. A wide variety of file formats are supported, but not all of these formats will work on AnkiWeb and the mobile clients. MP3 audio and MP4 video seems to be the most universally supported."

Are you sure that mplayer2 is supported?  From looking at the source, it seems that anki simply calls the 'mplayer' executable using a system call.  That will fail if you only have mplayer2.  Even if we patch that accordingly, it is not clear what will happen regarding integration.

It would be great if you could take this upstream, that is, talk to anki developers about mplayer2 support.
Comment 3 Alexander Berntsen (RETIRED) gentoo-dev 2014-11-10 16:00:47 UTC
mplayer2 is not "supported". You would need to patch it. The integration sucks (I looked at the source). I could use alias mplayer=mplayer2 and everything would be cool beans as far as anki is concerned.

They could add support for mplayer2 by making their command = ("mplayer", "mplayer2", [args]) and trying both of them. But the program doesn't even run from the git source -- it fails with Python syntax errors with both Python2 and Python3, so I shrugged and moved on with my day.

I couldn't find any bug tracking service associated with anki either.
Comment 4 Thomas Kahle (RETIRED) gentoo-dev 2014-11-11 09:18:45 UTC
(In reply to Alexander Berntsen from comment #3)
> mplayer2 is not "supported". You would need to patch it. The integration
> sucks (I looked at the source). I could use alias mplayer=mplayer2 and
> everything would be cool beans as far as anki is concerned.
>
> They could add support for mplayer2 by making their command = ("mplayer",
> "mplayer2", [args]) and trying both of them. 

Well, a pull request on github is one way to raise things.  E-mail is another.

> But the program doesn't even
> run from the git source -- it fails with Python syntax errors with both
> Python2 and Python3, so I shrugged and moved on with my day.

It does for me with no problems.  Did you run tools/build_ui.sh?

> I couldn't find any bug tracking service associated with anki either.

I agree they are not the most open project out there.