<div dir="ltr">I removed it because it seems to provide the same feature, I don't have a kde4 to test if this kind of freedesktop entry file works there.<div>We can just keep it, it doesn't hurt.</div><div><br></div><div>For the mimes I removed the disc corresponding ones so only the good VLC entry get returned for those mime types.</div></div><br><div class="gmail_quote"><div dir="ltr">Le mar. 23 janv. 2018 à 17:52, Jean-Baptiste Kempf <<a href="mailto:jb@videolan.org">jb@videolan.org</a>> a écrit :<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Sorry, but the removal of the SOLID support and the removal from .mimetypes mandates a bit more explanation.<br>
<br>
On Tue, 23 Jan 2018, at 17:48, Mathieu Velten wrote:<br>
> ---<br>
>  <a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a>                    | 32 --------------------------------<br>
>  share/Makefile.am               | 16 ++++------------<br>
>  share/solid/vlc-openbd.desktop  |  9 ---------<br>
>  share/solid/vlc-opencda.desktop | 10 ----------<br>
>  share/solid/vlc-opendvd.desktop | 10 ----------<br>
>  share/solid/vlc-openvcd.desktop | 10 ----------<br>
>  share/<a href="http://vlc-openbd.desktop.in" rel="noreferrer" target="_blank">vlc-openbd.desktop.in</a>     |  9 +++++++++<br>
>  share/<a href="http://vlc-opencda.desktop.in" rel="noreferrer" target="_blank">vlc-opencda.desktop.in</a>    |  9 +++++++++<br>
>  share/<a href="http://vlc-opendvd.desktop.in" rel="noreferrer" target="_blank">vlc-opendvd.desktop.in</a>    |  9 +++++++++<br>
>  share/<a href="http://vlc-openvcd.desktop.in" rel="noreferrer" target="_blank">vlc-openvcd.desktop.in</a>    |  9 +++++++++<br>
>  share/vlc.desktop.mimetypes     |  4 ----<br>
>  11 files changed, 40 insertions(+), 87 deletions(-)<br>
>  delete mode 100644 share/solid/vlc-openbd.desktop<br>
>  delete mode 100644 share/solid/vlc-opencda.desktop<br>
>  delete mode 100644 share/solid/vlc-opendvd.desktop<br>
>  delete mode 100644 share/solid/vlc-openvcd.desktop<br>
>  create mode 100644 share/<a href="http://vlc-openbd.desktop.in" rel="noreferrer" target="_blank">vlc-openbd.desktop.in</a><br>
>  create mode 100644 share/<a href="http://vlc-opencda.desktop.in" rel="noreferrer" target="_blank">vlc-opencda.desktop.in</a><br>
>  create mode 100644 share/<a href="http://vlc-opendvd.desktop.in" rel="noreferrer" target="_blank">vlc-opendvd.desktop.in</a><br>
>  create mode 100644 share/<a href="http://vlc-openvcd.desktop.in" rel="noreferrer" target="_blank">vlc-openvcd.desktop.in</a><br>
><br>
> diff --git a/<a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a> b/<a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a><br>
> index 11e8460..c28f635 100644<br>
> --- a/<a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a><br>
> +++ b/<a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a><br>
> @@ -3710,14 +3710,6 @@ AC_SUBST(QT_VERSION)<br>
>  AM_CONDITIONAL(ENABLE_QT, [test "$enable_qt" != "no"])<br>
>  AM_CONDITIONAL([HAVE_QT5_X11], [test "${have_qt5_x11}" = "yes"])<br>
><br>
> -dnl<br>
> -dnl detect kde4-config patch (used for kde solids).<br>
> -dnl<br>
> -AC_ARG_VAR([KDE4_CONFIG], [kde4-config utility])<br>
> -AS_IF([test "x$KDE4_CONFIG" = "x"], [<br>
> -  KDE4_CONFIG="kde4-config"<br>
> -])<br>
> -<br>
>  dnl<br>
>  dnl Simple test for skins2 dependency<br>
>  dnl<br>
> @@ -4157,30 +4149,6 @@ AS_IF([test "${ac_cv_c_bigendian}" = "yes"], [<br>
>  ])<br>
>  AC_SUBST(DEFS_BIGENDIAN)<br>
><br>
> -dnl<br>
> -dnl Where to install KDE solid .desktop<br>
> -dnl<br>
> -AC_ARG_WITH(kde-solid,<br>
> -  AS_HELP_STRING([--with-kde-solid=PATH],<br>
> -                 [KDE Solid actions directory (auto)]),, [<br>
> -<br>
> -if test "${SYS}" != "mingw32" -a "${SYS}" != "darwin" ; then<br>
> -  with_kde_solid="yes"<br>
> -fi<br>
> -])<br>
> -soliddatadir=""<br>
> -AS_IF([test "${with_kde_solid}" != "no"], [<br>
> -  AS_IF([test "${with_kde_solid}" = "yes"], [<br>
> -    kde4datadir="`${KDE4_CONFIG} --install data`"<br>
> -    AS_IF([test "x$kde4datadir" = "x"], [kde4datadir='${datadir}/kde4/<br>
> apps'])<br>
> -    soliddatadir="${kde4datadir}/solid/actions"<br>
> -  ], [<br>
> -    soliddatadir="${with_kde_solid}"<br>
> -  ])<br>
> -])<br>
> -AC_SUBST(soliddatadir)<br>
> -AM_CONDITIONAL(KDE_SOLID, [test "x${soliddatadir}" != "x"])<br>
> -<br>
>  dnl<br>
>  dnl Check wether we have the PROCESS_MITIGATION_IMAGE_LOAD_POLICY<br>
>  dnl It should only be available when building for win10, but some SDKs<br>
> define it unconditionnaly<br>
> diff --git a/share/Makefile.am b/share/Makefile.am<br>
> index bf81792..b81e2eb 100644<br>
> --- a/share/Makefile.am<br>
> +++ b/share/Makefile.am<br>
> @@ -5,7 +5,7 @@ EXTRA_DIST =<br>
>  desktopdir = $(datadir)/applications<br>
>  if !HAVE_WIN32<br>
>  if !HAVE_DARWIN<br>
> -desktop_DATA = vlc.desktop<br>
> +desktop_DATA = vlc.desktop vlc-openbd.desktop vlc-opendvd.desktop vlc-<br>
> openvcd.desktop vlc-opencda.desktop<br>
>  appdatadir = $(datarootdir)/metainfo<br>
>  appdata_DATA = $(appdata_in_files:.<a href="http://xml.in" rel="noreferrer" target="_blank">xml.in</a>=.xml)<br>
>  appdata_in_files = <a href="http://vlc.appdata.xml.in" rel="noreferrer" target="_blank">vlc.appdata.xml.in</a><br>
> @@ -13,9 +13,10 @@ endif<br>
>  endif<br>
><br>
>  EXTRA_DIST += <a href="http://vlc.desktop.in" rel="noreferrer" target="_blank">vlc.desktop.in</a> vlc.desktop.mimetypes<br>
> +EXTRA_DIST += <a href="http://vlc-openbd.desktop.in" rel="noreferrer" target="_blank">vlc-openbd.desktop.in</a> <a href="http://vlc-opendvd.desktop.in" rel="noreferrer" target="_blank">vlc-opendvd.desktop.in</a> vlc-<br>
> <a href="http://openvcd.desktop.in" rel="noreferrer" target="_blank">openvcd.desktop.in</a> <a href="http://vlc-opencda.desktop.in" rel="noreferrer" target="_blank">vlc-opencda.desktop.in</a><br>
>  CLEANFILES += $(desktop_DATA) $(appdata_DATA)<br>
><br>
> -vlc.desktop: <a href="http://vlc.desktop.in" rel="noreferrer" target="_blank">vlc.desktop.in</a> $(top_builddir)/config.status<br>
> +%.desktop: %.<a href="http://desktop.in" rel="noreferrer" target="_blank">desktop.in</a> $(top_builddir)/config.status<br>
>       $(AM_V_GEN)mimetypes="$$(sed 's/\s*#.*$$//g' $(srcdir)/<br>
> vlc.desktop.mimetypes | egrep -v '^$$' | tr "\n" ';')"; \<br>
>       sed \<br>
>               -e 's,\@bindir\@,$(bindir),g' \<br>
> @@ -52,8 +53,7 @@ endif<br>
>  EXTRA_DIST += \<br>
>       $(skins2_default_vlt_FILES) \<br>
>       $(DIST_icons) \<br>
> -     $(DIST_http_lua) \<br>
> -     $(DIST_solid)<br>
> +     $(DIST_http_lua)<br>
><br>
>  nobase_vlcdata_DATA =<br>
>  nobase_dist_vlcdata_DATA =<br>
> @@ -61,9 +61,6 @@ if BUILD_SKINS<br>
>  nobase_vlcdata_DATA += skins2/default.vlt<br>
>  nobase_dist_vlcdata_DATA += $(DIST_skins2)<br>
>  endif<br>
> -if KDE_SOLID<br>
> -soliddata_DATA = $(DIST_solid)<br>
> -endif<br>
><br>
>  DIST_icons = \<br>
>       vlc512x512.png<br>
> @@ -299,8 +296,3 @@ EXTRA_DIST += \<br>
>       lua/sd/icecast.lua \<br>
>       lua/sd/icast.lua \<br>
>       lua/sd/jamendo.lua<br>
> -DIST_solid = \<br>
> -     solid/vlc-openbd.desktop \<br>
> -     solid/vlc-opencda.desktop \<br>
> -     solid/vlc-opendvd.desktop \<br>
> -     solid/vlc-openvcd.desktop<br>
> diff --git a/share/solid/vlc-openbd.desktop b/share/solid/vlc-<br>
> openbd.desktop<br>
> deleted file mode 100644<br>
> index 8522d8c..0000000<br>
> --- a/share/solid/vlc-openbd.desktop<br>
> +++ /dev/null<br>
> @@ -1,9 +0,0 @@<br>
> -[Desktop Entry]<br>
> -X-KDE-Solid-Predicate=[ StorageVolume.ignored == false AND<br>
> OpticalDisc.availableContent & 'VideoBluRay' ]<br>
> -Type=Service<br>
> -Actions=open;<br>
> -<br>
> -[Desktop Action open]<br>
> -Name=Open with VLC media player<br>
> -Exec=vlc bluray://%d<br>
> -Icon=vlc<br>
> diff --git a/share/solid/vlc-opencda.desktop b/share/solid/vlc-<br>
> opencda.desktop<br>
> deleted file mode 100644<br>
> index 866e85f..0000000<br>
> --- a/share/solid/vlc-opencda.desktop<br>
> +++ /dev/null<br>
> @@ -1,10 +0,0 @@<br>
> -[Desktop Entry]<br>
> -X-KDE-Solid-Predicate=[ StorageVolume.ignored == false AND<br>
> OpticalDisc.availableContent & 'Audio' ]<br>
> -Type=Service<br>
> -Actions=open;<br>
> -<br>
> -[Desktop Action open]<br>
> -Name=Open with VLC media player<br>
> -Exec=vlc cdda://%d<br>
> -Icon=vlc<br>
> -<br>
> diff --git a/share/solid/vlc-opendvd.desktop b/share/solid/vlc-<br>
> opendvd.desktop<br>
> deleted file mode 100644<br>
> index eae868e..0000000<br>
> --- a/share/solid/vlc-opendvd.desktop<br>
> +++ /dev/null<br>
> @@ -1,10 +0,0 @@<br>
> -[Desktop Entry]<br>
> -X-KDE-Solid-Predicate=[ StorageVolume.ignored == false AND<br>
> OpticalDisc.availableContent & 'VideoDvd' ]<br>
> -Type=Service<br>
> -Actions=open;<br>
> -<br>
> -[Desktop Action open]<br>
> -Name=Open with VLC media player<br>
> -Exec=vlc dvd://%d<br>
> -Icon=vlc<br>
> -<br>
> diff --git a/share/solid/vlc-openvcd.desktop b/share/solid/vlc-<br>
> openvcd.desktop<br>
> deleted file mode 100644<br>
> index b612107..0000000<br>
> --- a/share/solid/vlc-openvcd.desktop<br>
> +++ /dev/null<br>
> @@ -1,10 +0,0 @@<br>
> -[Desktop Entry]<br>
> -X-KDE-Solid-Predicate=[ StorageVolume.ignored == false AND<br>
> OpticalDisc.availableContent & 'VideoCd|SuperVideoCd' ]<br>
> -Type=Service<br>
> -Actions=open;<br>
> -<br>
> -[Desktop Action open]<br>
> -Name=Open with VLC media player<br>
> -Exec=vlc vcd://%d<br>
> -Icon=vlc<br>
> -<br>
> diff --git a/share/<a href="http://vlc-openbd.desktop.in" rel="noreferrer" target="_blank">vlc-openbd.desktop.in</a> b/share/<a href="http://vlc-openbd.desktop.in" rel="noreferrer" target="_blank">vlc-openbd.desktop.in</a><br>
> new file mode 100644<br>
> index 0000000..d105324<br>
> --- /dev/null<br>
> +++ b/share/<a href="http://vlc-openbd.desktop.in" rel="noreferrer" target="_blank">vlc-openbd.desktop.in</a><br>
> @@ -0,0 +1,9 @@<br>
> +[Desktop Entry]<br>
> +Version=1.0<br>
> +Name=Open Blu-ray with VLC media player<br>
> +NoDisplay=true<br>
> +Exec=@bindir@/vlc --started-from-file bluray://%f @@<br>
> +Icon=vlc<br>
> +Terminal=false<br>
> +Type=Application<br>
> +MimeType=x-content/video-bluray<br>
> diff --git a/share/<a href="http://vlc-opencda.desktop.in" rel="noreferrer" target="_blank">vlc-opencda.desktop.in</a> b/share/<a href="http://vlc-opencda.desktop.in" rel="noreferrer" target="_blank">vlc-opencda.desktop.in</a><br>
> new file mode 100644<br>
> index 0000000..79bd0c5<br>
> --- /dev/null<br>
> +++ b/share/<a href="http://vlc-opencda.desktop.in" rel="noreferrer" target="_blank">vlc-opencda.desktop.in</a><br>
> @@ -0,0 +1,9 @@<br>
> +[Desktop Entry]<br>
> +Version=1.0<br>
> +Name=Open Audio CD with VLC media player<br>
> +NoDisplay=true<br>
> +Exec=@bindir@/vlc --started-from-file cdda://%f @@<br>
> +Icon=vlc<br>
> +Terminal=false<br>
> +Type=Application<br>
> +MimeType=x-content/audio-cdda<br>
> diff --git a/share/<a href="http://vlc-opendvd.desktop.in" rel="noreferrer" target="_blank">vlc-opendvd.desktop.in</a> b/share/<a href="http://vlc-opendvd.desktop.in" rel="noreferrer" target="_blank">vlc-opendvd.desktop.in</a><br>
> new file mode 100644<br>
> index 0000000..af03b9d<br>
> --- /dev/null<br>
> +++ b/share/<a href="http://vlc-opendvd.desktop.in" rel="noreferrer" target="_blank">vlc-opendvd.desktop.in</a><br>
> @@ -0,0 +1,9 @@<br>
> +[Desktop Entry]<br>
> +Version=1.0<br>
> +Name=Open DVD with VLC media player<br>
> +NoDisplay=true<br>
> +Exec=@bindir@/vlc --started-from-file dvd://%f @@<br>
> +Icon=vlc<br>
> +Terminal=false<br>
> +Type=Application<br>
> +MimeType=x-content/video-dvd<br>
> diff --git a/share/<a href="http://vlc-openvcd.desktop.in" rel="noreferrer" target="_blank">vlc-openvcd.desktop.in</a> b/share/<a href="http://vlc-openvcd.desktop.in" rel="noreferrer" target="_blank">vlc-openvcd.desktop.in</a><br>
> new file mode 100644<br>
> index 0000000..718f980<br>
> --- /dev/null<br>
> +++ b/share/<a href="http://vlc-openvcd.desktop.in" rel="noreferrer" target="_blank">vlc-openvcd.desktop.in</a><br>
> @@ -0,0 +1,9 @@<br>
> +[Desktop Entry]<br>
> +Version=1.0<br>
> +Name=Open VCD with VLC media player<br>
> +NoDisplay=true<br>
> +Exec=@bindir@/vlc --started-from-file vcd://%f @@<br>
> +Icon=vlc<br>
> +Terminal=false<br>
> +Type=Application<br>
> +MimeType=x-content/video-vcd;x-content/video-svcd<br>
> diff --git a/share/vlc.desktop.mimetypes b/share/vlc.desktop.mimetypes<br>
> index 0d866f7..d521c6c 100644<br>
> --- a/share/vlc.desktop.mimetypes<br>
> +++ b/share/vlc.desktop.mimetypes<br>
> @@ -119,10 +119,6 @@ x-scheme-handler/icyx   # Icecast<br>
><br>
>  # Linux desktop environment hooks for ISOs etc.<br>
>  application/x-cd-image<br>
> -x-content/video-vcd<br>
> -x-content/video-svcd<br>
> -x-content/video-dvd<br>
> -x-content/audio-cdda<br>
>  x-content/audio-player<br>
><br>
>  # Playlists / text/xml list with URLs<br>
> --<br>
> 2.14.3<br>
><br>
> _______________________________________________<br>
> vlc-devel mailing list<br>
> To unsubscribe or modify your subscription options:<br>
> <a href="https://mailman.videolan.org/listinfo/vlc-devel" rel="noreferrer" target="_blank">https://mailman.videolan.org/listinfo/vlc-devel</a><br>
<br>
<br>
--<br>
Jean-Baptiste Kempf -  President<br>
+33 672 704 734<br>
</blockquote></div>