New MythTV add-on using libcmyth

InoD · 2012-05-06 22:26

There is something else I'd like to look into too. (oh, what have I gotten myself into Wink

)
When resuming from standby I get some strange behaviour from the PVR addon. The recordings list is not updated, I sometimes cannot play any recordings and while playing another video I get notifications frequently that recordings are finished or deleted, which is obviously not the case. Not sure if this is related to the mythtv addon specifically or to the PVR system as a whole.
Does anyone recognize this?

tsp42 · 2012-05-06 22:55

I suspect that it is because the connection(socket) to the backend is broken when resuming from standby. Currently there is no code to try reestablishing the connection. The libcmyth code that has to be updated. We/I need to add a new function to update the cmyth_conn_t struct.

Any help will of course be highly appreciated. As you may have discovered it can be quite time consuming to fix all these bugs.

InoD · 2012-05-06 23:07

Ok, I'll have a look at how this is implemented best. Will probably take some time though....

bilbonvidia · 2012-05-09 15:18

(2012-05-06 21:08)tsp42 Wrote: bilbonvidia: Does it lockup when the program changes like fiveisalive is seeing or when you start xbmc?

It is when changing channel. I am currently running on the last commit before timeshift. Let me know if you want me to upgrade again to try anything testing wise.

bas.t · 2012-05-10 08:15

EDIT: This is not relevant anymore. I think commit 91bfc4eaed took care of this.

Hi,
I'm trying to compile but I get errors:

Code:

CPP     xbmc/guilib/GUIListItemLayout.o

fileOps.h: In constructor ‘fileOps2::fileOps2(MythConnection&)’:

fileOps.h:53: warning: ‘fileOps2::m_con’ will be initialized after

fileOps.h:52: warning:   ‘boost::filesystem::path fileOps2::m_localBasePath’

fileOps.cpp:16: warning:   when initialized here

fileOps.h:54: warning: ‘fileOps2::m_sg_strings’ will be initialized after

fileOps.cpp:17: warning:   base ‘PLATFORM::CThread’

fileOps.cpp:16: warning:   when initialized here

fileOps.cpp:21: error: ‘struct boost::filesystem::path’ has no member named ‘c_str’

fileOps.cpp: In member function ‘CStdString fileOps2::getChannelIconPath(CStdString)’:

fileOps.cpp:42: error: ‘struct std::basic_string<char, std::char_traits<char>, std::allocator<char> >’ has no member named ‘string’

fileOps.cpp: In member function ‘CStdString fileOps2::getPreviewIconPath(CStdString)’:

fileOps.cpp:63: error: ‘struct std::basic_string<char, std::char_traits<char>, std::allocator<char> >’ has no member named ‘string’

fileOps.cpp: In member function ‘CStdString fileOps2::getArtworkPath(CStdString, FILE_OPTIONS)’:

fileOps.cpp:95: error: ‘struct std::basic_string<char, std::char_traits<char>, std::allocator<char> >’ has no member named ‘string’

fileOps.cpp: In member function ‘void fileOps2::cleanCache()’:

fileOps.cpp:142: error: ‘struct std::basic_string<char, std::char_traits<char>, std::allocator<char> >’ has no member named ‘string’

fileOps.cpp:151: warning: comparison between signed and unsigned integer expressions

fileOps.cpp:150: warning: unused variable ‘mlm’

fileOps.cpp: In member function ‘bool fileOps2::writeFile(boost::filesystem::path, MythFile&)’:

fileOps.cpp:240: error: ‘struct boost::filesystem::basic_path<std::basic_string<char, std::char_traits<char>, std::allocator<char> >, boost::filesystem::path_traits>’ has no member named ‘c_str’

fileOps.cpp:246: error: ‘struct boost::filesystem::basic_path<std::basic_string<char, std::char_traits<char>, std::allocator<char> >, boost::filesystem::path_traits>’ has no member named ‘c_str’

fileOps.cpp:294: error: ‘struct boost::filesystem::path’ has no member named ‘c_str’

make[1]: *** [fileOps.o] Fout 1

make[1]: Map '/usr/src/setup/xbmc/xbmc/pvrclients/mythtv-cmyth' wordt verlaten

make: *** [pvrclients] Fout 2

I am compiling on Debian Squeeze.
Opdenkamps repo compiles just fine.
What can I do to prevent these errors?

**dteirney** · 2012-05-10 11:05

@bas.t you need to upgrade libboost to 1.46 or later.

Powderking · 2012-05-10 12:44

I'm still following your build and still use it as my frontend.
I wanted to say again thank you for your work!
It's great to see it developping Big Grin

I was amazed when I saw that preview and channel pics work now when the backend is on a different machine. And the categories are wonderful too Big Grin

Even though I have three little issues:

The categories aren't always perfect. E.g. I have a movies category where I have lots of recordings. But none of them is a movie. I understand that you sort them by program title only. But mythfrontend does it somehow different. I have a much better categroization there.
Will you be able to change that someday or is this a limitation of the cmyth library?

The next issue is when I enter this movies category (in fact any with a large number of recordings) it takes minutes until the recordings are listed. I read something about that it downloads all the covers. Has this to do with it?

And the last thing is that I like to have the newest recordings on the top. But When I sort one category the others aren't affected. Would it be possible to implement a menu entry or something that sets everything to be sorted be date?

Thank you again very much for your great work!

lunarok · 2012-05-11 11:03

Hi,

I read with attention your work, but I cannot use your git repo for compiling as I'm using MythTV 0.25. And this is going to be common as it's the version include in Ubuntu 12.04, so the version of Mythbuntu too.

But Daniel Teirney has patched the cmyth for support of MythTV 0.25 (https://github.com/dteirney/xbmc/tree/myth-0.25)
And seems it has been pull to the main branch of XBMC git (https://github.com/xbmc/xbmc cmyth 19 hours ago [cmyth] Fix memory corruption caused by incorrect size allocation for… [David Teirney])

Is it possible to merge your lib directory from the xbmc git so we will have the new version of cmyth and support of 0.25 ?
As your addon is full cmyth based, must be enough to get xbmc-pvr working with myth 0.25

Am I wrong ?

Jimmer · 2012-05-11 12:15

(2012-05-11 11:03)lunarok Wrote: Hi,

I read with attention your work, but I cannot use your git repo for compiling as I'm using MythTV 0.25. And this is going to be common as it's the version include in Ubuntu 12.04, so the version of Mythbuntu too.

But Daniel Teirney has patched the cmyth for support of MythTV 0.25 (https://github.com/dteirney/xbmc/tree/myth-0.25)
And seems it has been pull to the main branch of XBMC git (https://github.com/xbmc/xbmc cmyth 19 hours ago [cmyth] Fix memory corruption caused by incorrect size allocation for… [David Teirney])

Is it possible to merge your lib directory from the xbmc git so we will have the new version of cmyth and support of 0.25 ?
As your addon is full cmyth based, must be enough to get xbmc-pvr working with myth 0.25

Am I wrong ?

five has managed it here:

http://forum.xbmc.org/showthread.php?tid...pid1098065

it needs a file patching, and this was done a little while ago, but it may still be working. Probably better to wait a few days for it to get merged into Dushmaniac, and then again here. It'll happen soon, I'm sure

fiveisalive · 2012-05-12 17:49

(2012-05-11 12:15)Jimmer Wrote:
(2012-05-11 11:03)lunarok Wrote: Hi,

I read with attention your work, but I cannot use your git repo for compiling as I'm using MythTV 0.25. And this is going to be common as it's the version include in Ubuntu 12.04, so the version of Mythbuntu too.

But Daniel Teirney has patched the cmyth for support of MythTV 0.25 (https://github.com/dteirney/xbmc/tree/myth-0.25)
And seems it has been pull to the main branch of XBMC git (https://github.com/xbmc/xbmc cmyth 19 hours ago [cmyth] Fix memory corruption caused by incorrect size allocation for… [David Teirney])

Is it possible to merge your lib directory from the xbmc git so we will have the new version of cmyth and support of 0.25 ?
As your addon is full cmyth based, must be enough to get xbmc-pvr working with myth 0.25

Am I wrong ?

five has managed it here:

http://forum.xbmc.org/showthread.php?tid...pid1098065

it needs a file patching, and this was done a little while ago, but it may still be working. Probably better to wait a few days for it to get merged into Dushmaniac, and then again here. It'll happen soon, I'm sure

I tested patching tsp's cmyth branch with the myth-0.25 patch and I have it "working" in the sense that it loads the EPG and channel info. I tested this on a separate machine with no video card, using the "dummy tuner" module built into the mythbackend, so I was unable to test Live TV or recording playback, but at least the EPG/schedule and channel info loads correctly, which is generally a good sign.

InoD Junior Member Posts: 38 Joined: Apr 2012 Reputation: 1	2012-05-06 23:07 Post: #723 Ok, I'll have a look at how this is implemented best. Will probably take some time though....
	find quote

bilbonvidia Fan Posts: 433 Joined: Jun 2011 Reputation: 1 Location: Britain	2012-05-09 15:18 Post: #724 (2012-05-06 21:08)tsp42 Wrote: bilbonvidia: Does it lockup when the program changes like fiveisalive is seeing or when you start xbmc? It is when changing channel. I am currently running on the last commit before timeshift. Let me know if you want me to upgrade again to try anything testing wise.
	find quote

dteirney Team-XBMC Developer Posts: 820 Joined: Jul 2007 Reputation: 6 Location: New Zealand	2012-05-10 11:05 Post: #726 @bas.t you need to upgrade libboost to 1.46 or later. Use MythTV for recording TV? Try the integrated MythTV support in XBMC Media Center. Now with commercial skip support built-in and integration with the Movie database!
	find quote