[CrashX]

Here is another way .. It seems that speedfan has the abilty to log temperature information to a file .. you can set that up via configuration ..

We can now use some command line ulity to parse that log file and retrieve temperature information .. We pass that commandline as a parameter to xbmc advancedsettings "cputempcommand". This is currently allowed only for linux but can be easily enabled for windows.

[TheQuestor]

Speedfan, Everest, GPU-Z all have the ability to log to a file. That was what I was lookin at using when I started asking about this, but got so busy with other things I totally spaced it off Glad to see others picking up the torch and moving forward.

[CapnBry]

Here is another way .. It seems that speedfan has the abilty to log temperature information to a file .. you can set that up via configuration ..

We can now use some command line ulity to parse that log file and retrieve temperature information .. We pass that commandline as a parameter to xbmc advancedsettings "cputempcommand". This is currently allowed only for linux but can be easily enabled for windows.

I'm pretty sure it is enabled on windows too. I'm still unsure why anyone would want to put an intermediate step when speedfan has the SMA though.

EDIT: Booooo it isn't enabled in win32. Haha if I remove the ifdef from it, it works with a 50k speedfan reader, but a console window opens on top of XBMC for a split second.

[CrashX]

CapnBry:-

We need to find a generic way to solve this problem without writing custom code in xbmc.

Wiso:-

Can you please remove if define for CGUIInfoManager::GetSystemHeatInfo(int info) so that it works on windows as well:

Code:

#if defined(_LINUX)
    m_cpuTemp = g_cpuInfo.getTemperature();
    m_gpuTemp = GetGPUTemperature();
#endif

[WiSo]

Thanks for the code CapnBry but you're right I don't like this kind of custom third party code.

@CrashX: I can enable it if you confirm it to be working. I'm to lazy to test it

[CapnBry]

Thanks for the code CapnBry but you're right I don't like this kind of custom third party code.

@CrashX: I can enable it if you confirm it to be working. I'm to lazy to test it

I said above, it does work but throws up a quick console window. However it's not supposed to work (because it calls _popen which the API docs say "If used in a Windows program, the _popen function returns an invalid file pointer that causes the program to stop responding indefinitely").

If you really think it is useful I can write a class that returns the output of a console-redirected program in a CStdString, which then can be used in CCPUInfo::getTemperature().

[WiSo]

Not really. It would be ok if it would work out of the box but I don't think we should put effort in it.
But I'm open for suggestions (and code) if this "feature" is really wanted

[CrashX]

Wiso:-

Since popen isn't suppose to work on windows why not write our own though which works on all OS...

[WiSo]

As said I don't see it as a nice solution and won't put effort in it.
However if you and CapnBry put something nice together I wouldn't deny implementing it.