Re: QuickTime version

Kevin DeKorte wrote:
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
>
> =?ISO-8859-1?Q?Giuseppe_Ghib=F2_ wrote:
> |> Can you give me an exact failure page? I tested the link above with
> |> gecko-mediaplayer, which uses the same signature as mplayerplug-in and
> |> that site seems to work for me.
> |>
> |
> | BTW, current svn? As with gecko-mediaplayer 0.6.0 I get the same
> | behaviour of before (it asks for a newer version of QuickTime).
>
> Yeah, having the broken website was helpful. I was able to take a look
> at the javascript and see what it was looking for and then match it. I
> have fixed the descriptor in mplayerplug-in and gecko-mediaplayer
>
> Changes have been committed.
>
>   

Great thanks. BTW, looking at CVS I see:

http://mplayerplug-in.cvs.sourceforge.net/mplayerplug-in/mplayerplug-in/Source/plugin-setup.cpp?r1=1.145&r2=1.146

But afaik the real QT 7.4.5 uses QuickTime Plug-in 7.4.5, so the patch 
should be:

diff -p -up mplayerplug-in/Source/plugin-setup.cpp.745 
mplayerplug-in/Source/plugin-setup.cpp
--- mplayerplug-in/Source/plugin-setup.cpp.745    2008-05-13 
23:22:20.000000000 +0200
+++ mplayerplug-in/Source/plugin-setup.cpp    2008-05-13 
23:22:38.000000000 +0200
@@ -461,7 +461,7 @@ NPError GetValue(NPPVariable variable, v
 #endif
 
 #ifdef QT
-    *((const char **) value) = "QuickTime Plug-in 6.0 / 7";
+    *((const char **) value) = "QuickTime Plug-in 7.4.5";
 #endif
 
 #ifdef RM

This one works for me on the previous page.


-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft 
Defy all challenges. Microsoft(R) Visual Studio 2008. 
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/