VLC 0.9.2 plays all video files instead of DVD menu

Reinhard Tartler : You were the last one to apply a fix on VLC package, can you review the branch that I linked to this bug report?

The attached branch adds a patch in the debian/patches folder that changes %U to %f in share/applications/vlc.desktop . This way, VLC opens /media/cdrom0 instead of file:///media/cdrom0 . Playing video files individually still works perfectly, as much as DVD/SVCD/Audio CDs with my patch.

This bug was fixed in the package vlc - 0.9.3-0ubuntu1

---------------
vlc (0.9.3-0ubuntu1) intrepid; urgency=low

  [ Reinhard Tartler ]
  * new upstream bugfix only Releases LP: #274721

  Relevant fixes from NEWS file:

    Various bugfixes:
   * Fixed DTS channel order on 5.1 systems
   * Fixed pausing behavior for subtitles and for Audio-CD
   * Multiple subtitles and podcast fixes
   * Various crashes fixed in PS, SSA, mkv, xspf, freetype
   * Fixed update system bugs
   * Other bug fixes (dvd language selection, subtitle colours, HTTP keep-alive...)

  Qt4 interface:
   * Added Faster/Slower icons to the controller panel
   * Fixed lost playlist columns when switching the playlist view
   * Added needed options to Simple preferences (to avoid NVIDIA drivers issues)
   * Fullscreen controller: added time label, remembering of last position
   * Fixed drag'n drop behaviour on the playlist
   * Multiple other fixes (Enter hotkey in preferences, Skins selection...)

  [ Saïvann Carignan ]
  * debian/patches/301_DVD_media.diff: Change %U to %f
    in VLC .desktop file . (LP: #275043)

 -- Reinhard Tartler <email address hidden> Sat, 27 Sep 2008 09:30:04 +0200

18:11:16 < xtophe> siretart: btw, have you noted the comment of courmisch in the trac ticket associated to LP#275043
18:13:16 < siretart> xtophe: sorry no. I'll reopen the bug

courmisch didn't want to mention a potential solution but rejected the actual one. However, even if the actual solution in ubuntu is considered as a workaround, it still removes this bug from intrepid until it get fixed upstream also for http URLs. I was going to re-open the LP bug once it get fixed upstream.

This bug has been fixed upstream. That means it will be possible to drop current 301_DVD_media patch when next VLC bugfix release will be released and uploaded to jaunty.

http://git.videolan.org/?p=vlc.git;a=commitdiff;h=8a900ab6231a1ad4b1ac033cbd31a518b6bdb723

This commit is not enough to fix the bug, other commits are necessary so I suggest to wait for next VLC release rather than searching which commits are needed.

debian/patches/301_DVD_media.diff causes Nautilus to start multiple instances of VLC when opening multiple files (e.g. bug #333572). Hardly something anyone is expecting to happen.

Could the exec line be at least changed to %F?

Reinhard Tartler : I pushed a branch to remove my last patch since it is not needed anymore, the bug is fixed upstream and already fixed in jaunty. Can you review my branch?

Pekka Vuorela : The exec line is already using %f. %f will probably be replaced back to %U. If you try it, does that fix bug #333572 ?

Saïvann Carignan : I tried manually editing the .desktop file to using %F (note the capital letter) and %U. Both work fine regarding #333572.

Pekka Vuorela : Thanks for your confirmation. VLC will switch back to %U in karmic so both this bug and your bug will be closed.

This bug was fixed in the package vlc - 1.0.0~rc2-1ubuntu1

---------------
vlc (1.0.0~rc2-1ubuntu1) karmic; urgency=low

  * New upstream version.
  * tighten build-dependencies on libschroedinger-dev, fixes FTBFS.
  * Following bugs have an associated and now closed upstream ticket in
    Launchpad: LP: #281295, #73449, #282567, #117640, #172938, #260918,
    #273685, #275043, #277834, #282394, #290753, #294136, #298925, #304265,
    #309728, #311742, #314038

vlc (1.0.0~rc2-1) experimental; urgency=low

  * New Upstream Release Candidate
    + Closes: #527010, #491441, #508618, #522824
  * Delete patches which were picked from upstream
  * Refresh patches
  * Remove vlc-plugin-esd and vlc-plugin-arts as they have been removed
    by upstream
  * SONAME change for libvlccore
  * Add new symbols for libvlc2
  * Fix static libs compile
  * Use vlc-wrapper manpage from upstream
  * vlc.install: add drawable and screen was renamed x11_screen
  * vlc-nox.install: Add and remove modules added/removed upstream
  * Point out in NEWS the modules renamed or splited
  * Add links to upstream NEWS and co in vlc-nox
  * Version the depedency of libvlcore2 on vlc-data
  * Remove unnecessary "Section:" in debian/control

vlc (0.9.9a-3) unstable; urgency=low

  * Correct typo in 0.9.9a-2 changelog entry
  * Disable more optimization with DEB_BUILD_OPTIONS=noopt
  * Cache the configure test results as we're running configure several
    times
  * Fix building as root
  * Fix the clean target
  * Remove unexistant config options
  * Better check commad line
  * Use all the procs on i386 and amd64
  * Build a version of libvlccore without altivec (Closes: #523035)
  * Fix typo (thanks to Salvatore Bonaccorso)(Closes: #528044)
  * Reword the command line to get full logs in bug/presubj (Closes: #527012)
  * Remove duplicate "extended Settings" entry in context menu
   (Closes: #526603) - thanks to Matt Kraai

 -- Reinhard Tartler <email address hidden> Fri, 05 Jun 2009 12:28:15 +0200