vlc-1.1.2: please enable reading bookmarks from saved playlists

Bug #620290 reported by Doug McMahon
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
VLC media player
Fix Released
Wishlist
Unassigned
vlc (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

Binary package hint: vlc

If a rebuild of vlc is going to happen in Maverick it would be nice if reading EXTVLCOPT info from playlists was enabled (allowed
Has been so in the 1.2 vlc and is a simple 1 line patch that could easily be used in 1.1.2
I believe this was the 1.2 commit - works fine when applied to 1.1.2 though the line #'s may be different ( manually edited here

diff --git a/src/libvlc-module.c b/src/libvlc-module.c
index de04ee2..e786ba7 100644
--- a/src/libvlc-module.c
+++ b/src/libvlc-module.c
@@ -1855,6 +1855,7 @@ vlc_module_begin ()

         add_string( "bookmarks", NULL, NULL,
                      BOOKMARKS_TEXT, BOOKMARKS_LONGTEXT, true )
    + change_safe ()

         set_section( N_( "Default devices") , NULL )

ProblemType: Bug
DistroRelease: Ubuntu 10.10
Package: vlc 1.1.2-1+exp1ubuntu1
ProcVersionSignature: Ubuntu 2.6.35-15.21-generic 2.6.35.1
Uname: Linux 2.6.35-15-generic i686
NonfreeKernelModules: nvidia
Architecture: i386
Date: Thu Aug 19 02:39:06 2010
InstallationMedia: Ubuntu 10.10 "Maverick Meerkat" - Alpha i386 (20100705)
ProcEnviron:
 PATH=(custom, user)
 LANG=en_US.UTF-8
 SHELL=/bin/bash
SourcePackage: vlc

Revision history for this message
Doug McMahon (mc3man) wrote :
Revision history for this message
Rémi Denis-Courmont (rdenis) wrote :

There is no way to enable or disable EXTVLCOPT parsing. I trust the Debian/Ubuntu Multimedia maintainers have not purposefully removed EXTVLCOPT support. And in any case, your patch does not match your bug description.

Changed in vlc (Ubuntu):
status: New → Invalid
Doug McMahon (mc3man)
summary: - vlc-1.1.2: please enable reading EXTVLCOPT from playlists
+ vlc-1.1.2: please enable reading bookmarks from saved playlists
Revision history for this message
Doug McMahon (mc3man) wrote :

This has nothing to do with dm and it's not my patch - it's yours, and jb's suggestion
http://forum.videolan.org/viewtopic.php?f=13&t=79335

If the description was wrong you could have simply changed it, i'm sure you know what this concerns...

If you don't want to enable, fine - matters little here, others have been asking how to do short of building there own vlc.

Changed in vlc (Ubuntu):
status: Invalid → New
Changed in vlc:
status: New → Fix Committed
milestone: none → 1.1.4
importance: Undecided → Wishlist
Changed in vlc (Ubuntu):
status: New → Confirmed
tags: added: patch-accepted-upstream
Revision history for this message
rusivi2 (rusivi2-deactivatedaccount) wrote :

Thank you for reporting this bug.

When you update to the newest version of VLC does this still happen?

Changed in vlc (Ubuntu):
status: Confirmed → Incomplete
Revision history for this message
Rémi Denis-Courmont (rdenis) wrote :

assumed fixed in VLC 1.1.4 in Meerkat

Changed in vlc:
status: Fix Committed → Fix Released
Changed in vlc (Ubuntu):
status: Incomplete → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.