![]() |
|
New MythTV add-on using libcmyth - Printable Version +- XBMC Community Forum (http://forum.xbmc.org) +-- Forum: Development (/forumdisplay.php?fid=32) +--- Forum: PVR Development (/forumdisplay.php?fid=136) +--- Thread: New MythTV add-on using libcmyth (/showthread.php?tid=110694) Pages: 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61 62 63 64 65 66 67 68 69 70 71 72 73 74 75 76 77 78 79 80 81 82 83 84 85 86 87 88 89 90 91 92 93 94 95 96 97 98 99 100 101 102 103 104 105 106 107 108 109 110 111 112 113 114 115 116 117 118 119 120 121 122 123 124 125 126 127 128 129 130 131 132 133 134 135 136 137 138 139 140 141 142 143 144 145 146 147 148 149 150 151 152 153 154 155 156 157 158 159 160 161 162 163 164 165 166 167 168 169 170 171 172 173 174 175 176 177 178 179 180 181 182 183 184 185 186 187 188 189 190 191 192 193 194 195 196 197 198 199 200 201 202 203 204 205 206 207 208 209 210 |
RE: New MythTV add-on using libcmyth - InoD - 2012-05-12 20:40 Support for graceful reconnecting doesn't seem to be very straightforward, at least, not at the level I'd like to have it implemented. The cmyth library sets conn_hang in the cmyth_conn_t structure, which can be retrieved using the ConnHung function. However, if you want to initiate the reconnecting from XBMC side (MyhTV PVR Client code), then each function communicating with the library should first check the connection (ConnHung) and if hung initiate the reconnecting. That will result in a lot of code, but you will be able to create notifications. If implemented at the other end, connection.c in the library, then the connection could be reconnected right where and when it is detected to be disconnected. XBMC will not know it has reconnected, also you might run into an infinite reconnecting loop. But, I think the library should be dumb and not do such a thing unless it is requested to do so. XBMC offers some functionality to detect when waking up from standby, this could be used to force reconnecting (if you know the connection is gone it is faster to reconnect than the wait for the next command to fail), but having a reconnect mechanism will also cover the other use-cases in which the connection could have been closed. What do you think is the best way to implement this? Do you have other ideas? RE: New MythTV add-on using libcmyth - tdavis - 2012-05-13 00:09 (2012-05-12 17:49)fiveisalive Wrote:(2012-05-11 12:15)Jimmer Wrote:(2012-05-11 11:03)lunarok Wrote: Hi, I did this: Code: mkdir xbmcAnd then patch lib/cmyth/libcmyth/connection.c with Code: --- tsp2/lib/cmyth/libcmyth/connection.c 2012-05-12 15:08:00.476169895 -0700and compiled it up. Everything works - LiveTV, Recordings, EPG. RE: New MythTV add-on using libcmyth - tsp42 - 2012-05-13 21:53 I've ported the addon to the stable Eden PVR branch. You should be able to "git checkout Eden-PVR" now. Next change will be to add the MythTV 0.25 patch. (2012-05-09 15:18)bilbonvidia Wrote:It would be nice if you could provide me with a stack trace as outlined in the first post.(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. (2012-05-10 12:44)Powderking Wrote: I'm still following your build and still use it as my frontend.Currently the addon uses the genre type, duration, the MythTV programID and an optional regex to test if a recording is a movie or not. The genre type currently only works for EPG data that follows the DVB genre type convention (=cable TV). I plan to add support for mapping other genres to the ones supported by xbmc. The genres are defined in pvrclient-mythtv.cpp. Quote: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?No the addon downloads the covers in the background so it shouldn't cause any slowdowns. Quote: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?It could be done. I think it currently uses the default value for the video folders. You may want to add the request to the official XBMC PVR repository as it is not addon specific. (2012-05-12 20:40)InoD Wrote: Support for graceful reconnecting doesn't seem to be very straightforward, at least, not at the level I'd like to have it implemented.Well my suggestion would be to add the check to the MythConnection, MythEventHandler::ImpMythEventHandler, MythFile and MythRecorder class. We would also have to add a new cmyth function to reconnect an existing cmyth_conn_t structure. This would save the trouble of renewing and tracking all the cmyth pointers in the c++ code. I don't have much experience with socket programming but I suspect that the current way of detecting a hung connection in socket.c is not the best way to do it. RE: New MythTV add-on using libcmyth - fiveisalive - 2012-05-13 22:43 (2012-05-13 00:09)tdavis Wrote: I did this: Yep, that was almost exactly the same set of commands as in my post (but a bit neater than mine). Quote:Everything works - LiveTV, Recordings, EPG. Good to hear! RE: New MythTV add-on using libcmyth - fiveisalive - 2012-05-13 23:01 (2012-05-13 21:53)tsp42 Wrote: I've ported the addon to the stable Eden PVR branch. You should be able to "git checkout Eden-PVR" now. Next change will be to add the MythTV 0.25 patch. Great news! I would like to add the complete pvr patch against the original stock Eden source in the xbmc package I maintain for Fedora/RPM Fusion. Can you clue me in to which git commit I should use to create the diff/patch? Second, is the plan to backport features from master to this new branch on a semi-regular basis? The myth 0.25 patch will be critical since Fedora/RPM Fusion only ships with myth >= 0.25. Third, if I move the files for the add-on client into a separate subpackage (so only that subpackage is dependent on a backend, eg myth), would that cause a probably for xbmc itself? i.e. some people might want xbmc without pvr/myth and having a separate xbmc-{mythtv,tvheadend} packages only containg the addon parts would workaround this. Lastly, do you know what the plan is for dushmaniac's merge to upstream? I wasn't able to find on open pull request on github, or a forum post about this, but I noticed that development on the addon seems to have stopped in his branch, and there was one forum post that alluded to him preparing for the merge, but nothing from him directly about the schedule for that. RE: New MythTV add-on using libcmyth - bilbonvidia - 2012-05-14 12:08 "I've ported the addon to the stable Eden PVR branch. You should be able to "git checkout Eden-PVR" now." What does this mean exactly, I can get the stable eden pvr with this addon? Is it literally just git checkout Eden-PVR then a git pull and recompile? Thanks and sorry for all the questions. RE: New MythTV add-on using libcmyth - kburkart - 2012-05-14 16:00 (2012-05-13 00:09)tdavis Wrote:(2012-05-12 17:49)fiveisalive Wrote:(2012-05-11 12:15)Jimmer Wrote: five has managed it here: When you say compiled it up what do you mean? Do I have to compile both the dt directory and the tsp directory? I did a bootstrap, configure, make and make install of the tsp directory. I get the epg info, but when I try to change and watch a live channel, XBMC crashes and takes me to XBMCbuntu desktop. RE: New MythTV add-on using libcmyth - tsp42 - 2012-05-14 16:13 (2012-05-14 12:08)bilbonvidia Wrote: "I've ported the addon to the stable Eden PVR branch. You should be able to "git checkout Eden-PVR" now."Yes it is the mythtv addon with the Eden PVR code. It should be more stable. You have to pull before you checkout but else it is just a recompile RE: New MythTV add-on using libcmyth - bas.t - 2012-05-14 16:47 EDIT: This is not needed anymore. I think commit 91bfc4eaed took care of this. (2012-05-10 11:05)dteirney Wrote: @bas.t you need to upgrade libboost to 1.46 or later. Thanks a lot! It compiled just fine now. I'm going to test. RE: New MythTV add-on using libcmyth - bas.t - 2012-05-14 17:23 So having mythtv 0.25 support boiles down to this: (correct me if I'm wrong, please!) Code: curl -Ls --output 0.25.diff http://pastebin.com/raw.php?i=bjeCUELy #I combined the patches mentioned by fiveisalive to a new diff |