Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 549214 - media-video/mpv - dvd use flag appears not to work
Summary: media-video/mpv - dvd use flag appears not to work
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: AMD64 Linux
: Normal major (vote)
Assignee: Ben de Groot (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-05-11 22:18 UTC by faminebadger
Modified: 2015-05-24 04:46 UTC (History)
6 users (show)

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


Attachments
changes required for >=mpv-0.8.3 ebuilds (mpv-9999.ebuild.patch,1.31 KB, patch)
2015-05-23 17:26 UTC, Nikoli
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description faminebadger 2015-05-11 22:18:52 UTC
Running mpv dvd:// from a prompt results in:

   cplayer: Playing: dvd://
    stream: No protocol handler found to open URL dvd://
    stream: The protocol is either unsupported, or was disabled at compile-time.
   cplayer: 
   cplayer: 
   cplayer: Exiting... (Errors when loading file)

However, the dvd use flag is enabled.

$ eix -I mpv
[I] media-video/mpv
     Available versions:  0.3.10 0.8.3 (~)0.9.1 **9999 {+X +alsa bluray bs2b (+)cdio +cli (-)doc-pdf drm dvb +dvd dvdnav egl +enca encode +iconv jack (-)joystick jpeg ladspa lcms +libass libav libcaca libguess libmpv lirc lua luajit (+)mpg123 (-)openal +opengl oss portaudio +postproc pulseaudio pvr +quvi -radio raspberry-pi rubberband samba sdl selinux +shm v4l vaapi vcd vdpau vf-dlopen wayland xinerama +xscreensaver (+)xv}
     Installed versions:  0.9.1(20:42:20 04/05/15)(X alsa cli dvd enca encode iconv jpeg lcms libass lua luajit openal opengl pulseaudio vaapi vdpau xscreensaver xv -bluray -bs2b -cdio -doc-pdf -drm -dvb -dvdnav -egl -jack -ladspa -libav -libcaca -libguess -libmpv -oss -pvr -raspberry-pi -rubberband -samba -sdl -selinux -v4l -vf-dlopen -wayland -xinerama)
     Homepage:            http://mpv.io/
     Description:         Media player based on MPlayer and mplayer2
$

This appears contradictory.  Is the use flag actually doing anything?


Reproducible: Always

Steps to Reproduce:
1. mpv dvd://
2.
3.
Actual Results:  
stream: No protocol handler found to open URL dvd://menu
stream: The protocol is either unsupported, or was disabled at compile-time.


Expected Results:  
Play the DVD.


DVD is fine, and is accessible to the system (dd if=/dev/sr0 works fine).

Judging by the error message, the ebuild is not including dvd support properly.
Comment 1 Karl-Johan Karlsson 2015-05-12 20:57:40 UTC
Looking at the ebuild for media-video/mpv-0.9.1, it says this:

        dvd? (
                >=media-libs/libdvdread-4.1.3
                dvdnav? ( >=media-libs/libdvdnav-4.2.0 )
        )

"man mpv" has this to say about dvdread:

   Play a DVD using the old libdvdread code. This is what MPlayer and older mpv
   versions use for dvd://. Use is discouraged. It's provided only for compati-
   bility and for transition.

So, does "mpv dvdread://" work? Does "mpv dvd://" and "mpv dvd://menu" work if you enable USE=dvdnav?

The ebuild has IUSE="+dvd dvdnav". Since upstream apparently prefers dvdnav, maybe that should default to on as well?
Comment 2 josef.95 2015-05-14 12:50:01 UTC
Only for a little info, feedback :)
I use
[ebuild   R   ] media-video/mpv-0.9.1  USE="X alsa cli dvd enca encode iconv jack jpeg libass lua luajit opengl vdpau xinerama xscreensaver xv -bluray -bs2b -cdio -doc-pdf -drm -dvb -dvdnav -egl -ladspa -lcms -libav -libcaca -libguess -libmpv -openal -oss -pulseaudio -pvr -raspberry-pi -rubberband -samba -sdl (-selinux) -v4l -vaapi -vf-dlopen -wayland"


(In reply to Karl-Johan Karlsson from comment #1)
> Looking at the ebuild for media-video/mpv-0.9.1, it says this:
> 
>         dvd? (
>                 >=media-libs/libdvdread-4.1.3
>                 dvdnav? ( >=media-libs/libdvdnav-4.2.0 )
>         )
> 
> "man mpv" has this to say about dvdread:
> 
>    Play a DVD using the old libdvdread code. This is what MPlayer and older
> mpv
>    versions use for dvd://. Use is discouraged. It's provided only for
> compati-
>    bility and for transition.
> 
> So, does "mpv dvdread://" work? Does "mpv dvd://" and "mpv dvd://menu" work
> if you enable USE=dvdnav?
> 
> The ebuild has IUSE="+dvd dvdnav". Since upstream apparently prefers dvdnav,
> maybe that should default to on as well?

Yes, "mpv dvdread://" woks here fine.
And rebuild with USE=dvd dvdnav
"mpv dvd://" and "mpv dvd://menu"
works fine here.
Thank you :)
Comment 3 Nikoli 2015-05-14 14:18:59 UTC
Indeed we need to either enable dvdnav by default or merge this flag with 'dvd'.
Comment 4 Nikoli 2015-05-23 17:26:44 UTC
Created attachment 403826 [details, diff]
changes required for >=mpv-0.8.3 ebuilds

Since commit https://github.com/mpv-player/mpv/commit/8ce73ebbe0167a7a6fd3555a9fe24b4f8f3202e2 dvd:// defaults to dvdnav, so having separate 'dvdnav' USE flag is pointless.
Comment 5 Ben de Groot (RETIRED) gentoo-dev 2015-05-24 04:46:32 UTC
+*mpv-0.9.2-r1 (24 May 2015)
+*mpv-0.8.3-r1 (24 May 2015)
+
+  24 May 2015; Ben de Groot <yngwin@gentoo.org> +mpv-0.8.3-r1.ebuild,
+  +mpv-0.9.2-r1.ebuild, -mpv-0.9.1.ebuild, -mpv-0.9.2.ebuild, mpv-9999.ebuild:
+  merge dvdnav useflag into dvd (bug #549214)