! Please note that this is a snapshot of our old Bugzilla server, which is read only since May 29, 2020. Please go to gitlab.xfce.org for our new server !
VIDEO_TS vs video_ts
Status:
RESOLVED: FIXED
Product:
Thunar-volman
Component:
General

Comments

Description Jani Monoses 2007-03-21 14:35:37 CET
User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8.0.4) Gecko/20060508 Firefox/1.5.0.4
Build Identifier: 

Thunar would not recognize DVD's which have VIDEO_TS on them. The code checks for video_ts, I am not sure if both upper and lower case are valid but at least for the 2 DVDs I tried it did not work.
BTW does the DVD drive need to be mounted in order to be played or is it specifric to totem? gxine can play it by just using the dvd:/ MRL, as with audio CD's it does not need mounting

Reproducible: Always
Comment 1 Tino Keitel 2007-08-10 07:56:18 CEST
Created attachment 1306 
check for VIDEO_TS in addition to video_ts to detect DVDs

The video_ts case seems to be with DVDs mounted as ISO9660. However, video DVDs should be mounted as UDF, and in this case, the directory structure uses capital letters.
Comment 2 Jack Kelly 2007-09-21 14:11:48 CEST
Your patch will leak memory because you don't g_free(path_video_ts2); before returning at around line 277 or in the cleanup at around line 282.
Comment 3 Jack Kelly 2007-09-21 16:20:40 CEST
Created attachment 1366 
Patch to make thunar-volman consider VIDEO_TS on dvds - no leaking memory
Comment 4 Don Mahurin 2007-10-09 15:25:20 CEST
*** Bug 3596 has been marked as a duplicate of this bug. ***
Comment 5 Benedikt Meurer editbugs 2007-10-09 19:10:43 CEST
Fixed with revision 3318.

2007-10-09	Benedikt Meurer <benny@xfce.org>

	* thunar-volman/tvm-block-device.c: Properly ignore case looking
	  for VCD, VIDEO_TS and DCIM folders. Bug #3038.

Bug #3038

Reported by:
Jani Monoses
Reported on: 2007-03-21
Last modified on: 2009-11-09
Duplicates (1):
  • 3596 DVD not detected for autoplay

People

Assignee:
Jannis Pohlmann
CC List:
4 users

Version

Version:
unspecified

Attachments

Additional information