Changes on MPlayer

[Salvador Lopez]

I use 0.8 version of QuickRip, i don't know if exist any newer version, if this post is outdated, i'm sorry.<br>
In some version of mplayer the syntax for selecting a track on DVD has passed from -dvd <track> to dvd://<track>.
I use Fedora Core 1 and i have this problem. QuickRip doesn't not report any error, but if i change these options on dvd.py quickrip runs correctly.<br>
In line 298:<br>
cmd = "".join(["mplayer dvd://"+str(self.track['id']), " -vop cropdetect -nosound -vo null -frames 10 -sstep ", str(sstep), " 2>/dev/null"])<br>
and rest of -dvd occurence:<br>
"dvd://"+str(self.track['id']), .....<br>
Your program is incredible useful, very thanks for it!
<br>
I user mplayer 1.0pre2-3.3.2
<br>
Regards

[Anonymous]

Could you post a diff to whatever makes it work for you?

-bmfrosty

[Salvador Lopez]

Of course, the changes and some debug messages:

298,299c298,300
<             cmd = "".join(["mplayer -dvd ", str(self.track['id']), " -vop cropdetect -nosound -vo null -frames 10 -sstep ", str(sstep), " 2>/dev/null"])
<             text = os.popen(cmd).readlines()
---
>             cmd = "".join(["mplayer dvd://"+str(self.track['id']), " -vop cropdetect -nosound -vo null -frames 10 -sstep ", str(sstep), " 2>/dev/null"])
>             print "Linea de comando:"+str(cmd)
>         text = os.popen(cmd).readlines()
331c332,333
<
---
>         print "Lenguaje seleccionado: "+str(self.aLanguage)
>         print "Passes: "+str(self.config['passes'])
333c335
<                 all_pass = [str(self.config['mencoder']), "-dvd", str(self.track['id']), "-alang", self.aLanguage, \
---
>                 all_pass = [str(self.config['mencoder']), "dvd://"+str(self.track['id']), "-alang", self.aLanguage, \
338,339c340,341
<                     all_pass.insert(3, "-aspect")
<                     all_pass.insert(4, self.aro[self.config['aspectratio']])
---
>                     all_pass.insert(2, "-aspect")
>                     all_pass.insert(3, self.aro[self.config['aspectratio']])
342,343c344,345
<                     all_pass.insert(3, "-slang")
<                     all_pass.insert(4, self.sLanguage)
---
>                     all_pass.insert(2, "-slang")
>                     all_pass.insert(3, self.sLanguage)
345,352c347,354
< #                 # Uncomment to test command
< #                 string = ""
< #                 for bit in all_pass:
< #                     bit = "".join([" ", bit])
< #                     string = "".join([string, bit])
< #                 print all_pass
< #                 print string
< #               sys.exit(1)
---
>                 # Uncomment to test command
>                 #string = ""
>                 #for bit in all_pass:
>                 #    bit = "".join([" ", bit])
>                 #    string = "".join([string, bit])
>                 #print all_pass
>                 #print string
>                 #sys.exit(1)
357c359
<                 audio_pass = [str(self.config['mencoder']), "-dvd", str(self.track['id']), "-alang", self.aLanguage, \
---
>                 audio_pass = [str(self.config['mencoder']), "dvd://"+str(self.track['id']), "-alang", self.aLanguage, \
361c363
<                 video_pass = [str(self.config['mencoder']), "-dvd", str(self.track['id']), "-alang", self.aLanguage, \
---
>                 video_pass = [str(self.config['mencoder']), "dvd://"+str(self.track['id']), "-alang", self.aLanguage, \
366,367c368,369
<                     video_pass.insert(3, "-aspect")
<                     video_pass.insert(4, self.aro[self.config['aspectratio']])
---
>                     video_pass.insert(2, "-aspect")
>                     video_pass.insert(3, self.aro[self.config['aspectratio']])
370,371c372,373
<                     video_pass.insert(3, "-slang")
<                     video_pass.insert(4, self.sLanguage)
---
>                     video_pass.insert(2, "-slang")
>                     video_pass.insert(3, self.sLanguage)
394c396
<                 audio_pass = [str(self.config['mencoder']), "-dvd", str(self.track['id']), "-alang", self.aLanguage, \
---
>                 audio_pass = [str(self.config['mencoder']), "dvd://"+str(self.track['id']), "-alang", self.aLanguage, \
398c400
<                 video_pass1 = [str(self.config['mencoder']), "-dvd", str(self.track['id']), "-alang", self.aLanguage, \
---
>                 video_pass1 = [str(self.config['mencoder']), "dvd://"+str(self.track['id']), "-alang", self.aLanguage, \
402c404
<                 video_pass2 = [str(self.config['mencoder']), "-dvd", str(self.track['id']), "-alang", self.aLanguage, \
---
>                 video_pass2 = [str(self.config['mencoder']), "dvd://"+str(self.track['id']), "-alang", self.aLanguage, \
407,410c409,412
<                     video_pass1.insert(3, "-aspect")
<                     video_pass1.insert(4, self.aro[self.config['aspectratio']])
<                     video_pass2.insert(3, "-aspect")
<                     video_pass2.insert(4, self.aro[self.config['aspectratio']])
---
>                     video_pass1.insert(2, "-aspect")
>                     video_pass1.insert(3, self.aro[self.config['aspectratio']])
>                     video_pass2.insert(2, "-aspect")
>                     video_pass2.insert(3, self.aro[self.config['aspectratio']])
413,416c415,418
<                     video_pass1.insert(3, "-slang")
<                     video_pass1.insert(4, self.sLanguage)
<                     video_pass2.insert(3, "-slang")
<                     video_pass2.insert(4, self.sLanguage)
---
>                     video_pass1.insert(2, "-slang")
>                     video_pass1.insert(3, self.sLanguage)
>                     video_pass2.insert(2, "-slang")
>                     video_pass2.insert(3, self.sLanguage)
439a442
>     print "Comando >> " + str(command)

this, works for me.

Regards.