XBMC Community Forum
[WINDOWS] External Player Support Thread - Printable Version

+- XBMC Community Forum (http://forum.xbmc.org)
+-- Forum: Help and Support (/forumdisplay.php?fid=33)
+--- Forum: XBMC General Help and Support (/forumdisplay.php?fid=111)
+---- Forum: Windows support (/forumdisplay.php?fid=59)
+---- Thread: [WINDOWS] External Player Support Thread (/showthread.php?tid=43511)



- jondr13 - 2009-04-18 06:17

tiben20 Wrote:for virtual drive [Image: opt1240027043e.JPG]
for the external player you create the xml file in your userdata directory like you said and you insert the defaultplayer settings like this
Code:
]<advancedsettings>
  <video>
    <defaultplayer>externalplayer</defaultplayer>
  </video>

<externalplayer>
  <filename>C:\mplayer\mplayer.exe.....</filename>
  <forceontop>false</forceontop>
  <hidexbmc>true</hidexbmc>
  <hideconsole>false</hideconsole>
  <hidecursor>false</hidecursor>
</externalplayer>
</advancedsettings>

and about the auto mount you want it would be easier to do it with daemon tools than virtual drive

Got it.

When you mount your file to play in XMBC, are you doing this while in XBMC?....or are you doing it outside of XBMC??
I want to be able to look through my movie collection in XBMC, select a movie, it automatically mount my ISO file to Virtual Clone Drive and then open up my external player......all within XBMC.

The screen shot you provided for VCD is not within XBMC.

Again, thanks for the help


- digitalhigh - 2009-04-19 20:38

tiben20 Wrote:i just did a work around that work with only one extension i added after
Code:
// only dvdplayer can handle these normally
  if (url.GetFileType().Equals("sdp")
  ||  url.GetFileType().Equals("asf"))
  {
    vecCores.push_back(EPC_DVDPLAYER);
  }
in the PlayerCoreFactory.cpp
Code:
CURL externalPlayerUrl(g_advancedSettings.m_externalPlayerFilename);
  

    if (url.GetFileType().Equals("wmv") && externalPlayerUrl.GetFileType().Equals("exe"))
      {
        vecCores.push_back(EPC_EXTPLAYER);
      }
Im not really an expert in c++ but this is doing the job.
Its looking the extension of the file and its looking if the external player is an .exe extension too. Just in case you dont have an external setted


Can this be set up to do this with MKV files? The only reason I don't use externalplayer is the inconvenience of it. If it could finally be trained to respond automagically to certain filetypes, I'd be all over it.


look at this - tiben20 - 2009-04-20 03:26

http://xbmc.org/ticket/6383this is something i done 3 days ago

to have custom external player settings for every file extension.


- ashlar - 2009-04-20 12:52

tiben20 Wrote:http://xbmc.org/ticket/6383this is something i done 3 days ago

to have custom external player settings for every file extension.
I'm curious as if developers will/would be willing to commit this to SVN.

In the past I've got the impression that they were against this, but I wasn't clear if the reason was lack of resources to dedicate to its development or more "philosophical" reasons.

Is your patch crossplatform?


- spiff - 2009-04-20 13:04

in it current state it definitely won't be merged. there is absolutely no reason to limit this to external players.

i'll discuss with the author on trac as to why that is, when i get around to it.


- ashlar - 2009-04-20 18:54

spiff Wrote:in it current state it definitely won't be merged. there is absolutely no reason to limit this to external players.
I'm not sure I get what you are meaning here. Unless... you mean that for certain extensions one should be allowed to configure for regular DVDplayer? In that case I wholeheartedly agree. DVDplayer ISO support is second to none, for instance (in my experience, in my opinion, etc. Wink).


- spiff - 2009-04-20 18:56

yes, there is no reason to limit the configuration file to externalplayers. i have always wanted this functionality in general to rid the code of a nasty function. instead we can configure using an xml file, set a default and override where we see fit.

so tiben20, not a slamming of your stuff, just think it can be more


- moby-uk - 2009-04-20 23:20

I had, I think, the same thoughts; replacing the entire PlayerCoreFactory with something XML-driven. However, I didn't expect that much of a change to be welcomed since it largely benefits the externalplayer which I thought was viewed a bit like a ginger step-child [somewhat rightly so, no-one wants to see XBMC as a launcher].


- spiff - 2009-04-20 23:24

well, if other parts can benefit...

and my personal view in general; if people want to use xbmc as a launcher, why should we limit their option to do so? while i wouldn't put any effort into it myself, you guys have done all the work and everyone benefits..


- moby-uk - 2009-04-20 23:35

fair enough.

i guess we should take the implementation discussion to trac.


- digitalhigh - 2009-04-21 00:07

+1


about this - tiben20 - 2009-04-21 08:53

The only reason why i did this patch. its only because im having some playback problem with wmv file. And i hate having to use the sub menu to select a different player everytime i play a wmv file.
And just for telling everyone its the first time i was coding in c++. So dont expect too much from me to develop this patch very far.


- ashlar - 2009-04-21 10:02

What spiff was suggesting should be simple I guess (I'm not a programmer, so I might be totally wrong). Just allow for DVDplayer to be one of the players that can be chosen. Well, now that I think of it... there's no executable to call directly, maybe it's not so simple.


- moby-uk - 2009-04-21 18:51

tiben20 Wrote:The only reason why i did this patch. its only because im having some playback problem with wmv file. And i hate having to use the sub menu to select a different player everytime i play a wmv file.
And just for telling everyone its the first time i was coding in c++. So dont expect too much from me to develop this patch very far.

I'm happy to do some work on this, it's obviously not going to be in 9.04 though (just to manage expectations).

spiff: I'll pm you if you're happy to discuss...


- zag - 2009-04-22 09:39

What would be even better is to allow conditions like filename so external players could be launched for MMS:// streams or HTTP:// streams.