| Monday, January 30th, 2012, 15:28 UTC | ||
| [15:28:59] | Captain_Murdoch: | stuartm, are you going to remove metallurgy from the myththemes repo? I'm wondering what people (and the authors) about moving Childish and Mythbuntu to the MythTV-Themes/Childish and MythTV-Themes/Mythbuntu on github. we can setup groups under there to allow people who want to maintain those to commit. |
| [15:32:36] | stuartm: | Captain_Murdoch: yes, I'll remove it, I'd forgotten it was there |
| [15:33:15] | Captain_Murdoch: | ok. for some reason I thought you did that when you asked me to remove it from the download site. I just noticed it was still there so I figured I'd check. |
| [15:38:25] | sphery: | Captain_Murdoch: I'm all for moving childish and mythbuntu to there... It solves 2 problems--allows us to get authors/maintainers of the theme permissions to push and makes it less likely for users to think they need to install myththemes repo (especially if we have a good README at MythTV-Themes) |
| [15:39:22] | Captain_Murdoch: | sphery, thanks, I figured you'd be on board. I'd like to get them out of our repo as well. |
| [15:39:25] | sphery: | all that would be left would be finding people to actually be maintainers... :) |
| [15:41:59] | stuarta: | hahaha |
| [15:42:38] | jpabq_ (jpabq_!~jpabq@mythtv/developer/jpabq) has quit (Ping timeout: 252 seconds) | |
| [15:52:09] | kwmonroe (kwmonroe!~kwmonroe@32.97.110.58) has quit (Quit: Coyote finally caught me) | |
| [15:55:09] | kwmonroe (kwmonroe!~kwmonroe@32.97.110.58) has joined #mythtv | |
| [16:06:37] | sraue (sraue!~stephan@xbmc/staff/sraue) has quit (Ping timeout: 245 seconds) | |
| [16:06:38] | stichnot (stichnot!chatzilla@nat/intel/x-nzijczywbflrzkqz) has quit (Ping timeout: 240 seconds) | |
| [16:12:06] | kurre2 (kurre2!~tomimo@xdsl-83-150-88-111.nebulazone.fi) has quit (Read error: Connection reset by peer) | |
| [16:13:30] | Beirdo: | stuartm: yeah, shouldn't be a problem to change the version of cppcheck. :) I did miss it earlier |
| [16:13:59] | tomimo (tomimo!~kurre@xdsl-83-150-88-111.nebulazone.fi) has quit (Ping timeout: 252 seconds) | |
| [16:14:08] | Beirdo: | and yah, we can run a script in there, I have one to report the mythtv version, but we can tweak that to add more details, or run another script too |
| [16:18:41] | Goga777 (Goga777!~Goga777@128-71-150-153.broadband.corbina.ru) has joined #mythtv | |
| [16:21:31] | sraue (sraue!~stephan@xbmc/staff/sraue) has joined #mythtv | |
| [16:26:51] | dekarl-too (dekarl-too!51c8c614@gateway/web/freenode/ip.81.200.198.20) has joined #mythtv | |
| [16:35:29] | stuartm: | Beirdo: version info for gcc, qt would be useful when a build fails |
| [16:36:01] | jcarlos (jcarlos!~quassel@85.137.99.76.dyn.user.ono.com) has quit (Ping timeout: 276 seconds) | |
| [16:36:51] | superm1: | sphery: actually once you move those are you just going to drop myththemes all together? They're all that's left right? |
| [16:37:47] | sphery: | superm1: yeah, that's all that's left, so, AFAIK, we'd just drop myththemes repo and users would use the Theme Chooser (as they should be doing, anyway--assuming they have Internet access to the MythTV box) |
| [16:38:06] | superm1: | sounds good to me |
| [16:38:19] | Seeker`: | this ringbuffer code is certainly fun to try to follow |
| [16:38:44] | ** stuarta chuckles ** | |
| [16:39:42] | dekarl-too: | http://www.mythtv.org/wiki/User:Dekarl/Burndown_DVB <- my DVB notes for the stuarts, is that helpful that way? |
| [16:39:55] | stuartm: | Seeker`: that's easy by comparison to some other areas |
| [16:40:01] | Beirdo: | stuartm: seems reasonable. We should do that right up front at the beginning, I guess |
| [16:42:03] | Seeker`: | stuartm: im getting there. dont have enough logs to be able to work out why a m2ts file isbeung opened with RemoteFile though |
| [16:44:04] | stuartm: | I thought we streamed everything via the backend these days |
| [16:45:11] | stuartm: | hah! found it, malformed ADD events |
| [16:45:18] | Seeker`: | the attachment to #10294 shows what i am trying to work out |
| [16:50:08] | Jordack (Jordack!~jordack@h69-131-44-221.plmomi.dedicated.static.tds.net) has quit () | |
| [16:50:50] | jcarlos (jcarlos!~quassel@85.137.99.76.dyn.user.ono.com) has joined #mythtv | |
| [16:51:06] | sphery: | stuartm / Seeker` : we have code that chooses to use a local path, first, if available, for recordings. markk extended that to work for MythVideo (non-recordings) playback, too, specifically because of the performance difference involved when using myth proto for streaming ISOs |
| [16:53:54] | Seeker`: | this isnt quite streaming isos though |
| [16:55:02] | Seeker`: | it is a series of files, and there is a failure to start reading from the next file once one has finished until buffers are empty enough to produce stuttering |
| [16:57:08] | Seeker`: | the performance is fine mid-file, so if i / someone else can work out what causes the 2 second lag it would work perfectly |
| [17:02:45] | Beirdo: | OK, off to work I go |
| [17:08:27] | jcarlos (jcarlos!~quassel@85.137.99.76.dyn.user.ono.com) has quit (Quit: No Ping reply in 180 seconds.) | |
| [17:08:43] | jcarlos (jcarlos!~quassel@85.137.99.76.dyn.user.ono.com) has joined #mythtv | |
| [17:15:19] | zombor_ (zombor_!~zombor_@kohana/developer/zombor) has joined #mythtv | |
| [17:18:07] | zombor (zombor!~zombor_@kohana/developer/zombor) has quit (Read error: Operation timed out) | |
| [17:20:02] | Seeker`: | could my problem . e caused by the lack of file_eof_mythiowrapper? |
| [17:22:35] | zombor_ is now known as zombor | |
| [17:25:20] | zombor (zombor!~zombor_@kohana/developer/zombor) has quit (Read error: Connection reset by peer) | |
| [17:25:20] | zombor_ (zombor_!~zombor_@kohana/developer/zombor) has joined #mythtv | |
| [17:27:08] | zombor_ is now known as zombor | |
| [17:27:11] | jpabq_ (jpabq_!~jpabq@mythtv/developer/jpabq) has joined #mythtv | |
| [17:28:20] | stichnot (stichnot!~chatzilla@192.55.55.39) has joined #mythtv | |
| [17:32:46] | tomimo (tomimo!~kurre@xdsl-83-150-88-111.nebulazone.fi) has joined #mythtv | |
| [17:33:08] | kurre2 (kurre2!~tomimo@xdsl-83-150-88-111.nebulazone.fi) has joined #mythtv | |
| [17:35:51] | stuartm: | jpabq: fixed |
| [17:55:46] | Goga777 (Goga777!~Goga777@128-71-150-153.broadband.corbina.ru) has quit (Remote host closed the connection) | |
| [18:04:28] | davide_ (davide_!~david@mythtv/developer/gigem) has quit (Remote host closed the connection) | |
| [18:04:51] | davide_ (davide_!~david@host70.16.intrusion.com) has joined #mythtv | |
| [18:04:52] | davide_ (davide_!~david@host70.16.intrusion.com) has quit (Changing host) | |
| [18:04:52] | davide_ (davide_!~david@mythtv/developer/gigem) has joined #mythtv | |
| [18:10:13] | dekarl-too (dekarl-too!51c8c614@gateway/web/freenode/ip.81.200.198.20) has quit (Quit: Page closed) | |
| [18:11:09] | zombor_ (zombor_!~zombor_@kohana/developer/zombor) has joined #mythtv | |
| [18:13:14] | wagnerrp: | markk: does RAOP work over ipv6? or does it require v4? |
| [18:13:44] | davide_: | stuartm: Is 70ca0bf0 applicable to fixes-0.24? I think I've seen those same issues where things don't get updated in PBB until I exit and re-enter. |
| [18:14:08] | zombor (zombor!~zombor_@kohana/developer/zombor) has quit (Ping timeout: 260 seconds) | |
| [18:14:44] | Seeker`: | been doing some digging . in bluray.cpp the code has a check whether the ampunt of data returned is less than the amoumt requested; should something be done with this check to signal that the next file needs to be read from |
| [18:14:47] | zombor_ (zombor_!~zombor_@kohana/developer/zombor) has quit (Read error: Connection reset by peer) | |
| [18:15:13] | zombor (zombor!~zombor_@kohana/developer/zombor) has joined #mythtv | |
| [18:15:14] | Seeker`: | i cant confirm that is an indicator of the oroblem just yet though |
| [18:16:10] | Seeker`: | i mean bluray.c |
| [18:26:19] | zombor_ (zombor_!~zombor_@kohana/developer/zombor) has joined #mythtv | |
| [18:26:31] | Captain_Murdoch: | stuartm, did you see the idea I mentioned a while back about allowing a theme to specify an initial fallback theme. so you could make for instance Terra-Blue with overridden images but with fallback to Terra before falling back to default-wide. |
| [18:27:37] | Captain_Murdoch: | the theme downloader could handle dependencies, so if you install Terra-Blue, it auto-installs Terra as a dependency. |
| [18:28:33] | Captain_Murdoch: | from the mythui side, it just inserts a new dir into the searchpath. |
| [18:28:53] | joki (joki!~joki@p54864CC5.dip.t-dialin.net) has quit (Ping timeout: 255 seconds) | |
| [18:29:00] | joki- (joki-!~joki@p54862623.dip.t-dialin.net) has joined #mythtv | |
| [18:29:06] | Captain_Murdoch: | would allow for 'tweaks' to themes. |
| [18:29:10] | joki- is now known as joki | |
| [18:29:50] | zombor (zombor!~zombor_@kohana/developer/zombor) has quit (Ping timeout: 252 seconds) | |
| [18:32:12] | Beirdo: | MythBuild: force build cppcheck-master now |
| [18:32:13] | MythBuild: | build forced [ETA 9m18s] |
| [18:32:13] | MythBuild: | I'll give a shout when the build finishes |
| [18:34:13] | MythBuild: | Hey! build cppcheck-master #538 is complete: Success [build successful] |
| [18:34:13] | MythBuild: | Build details are at http://code.mythtv.org/buildbot/builders/cppc . . . r/builds/538 |
| [18:34:50] | Beirdo: | ummm |
| [18:35:25] | Beirdo: | stuartm: I think some cppcheck config tweakage will be in order for 1.52 :) |
| [18:36:36] | Beirdo: | let me try that without HAVE_RULES=yes |
| [18:36:52] | Beirdo: | I don't recall which way it was build last time. One moment |
| [18:37:11] | Beirdo: | but 3263 results (up from 179 or so...) |
| [18:37:24] | Beirdo: | MythBuild: force build cppcheck-master now |
| [18:37:29] | MythBuild: | build forced [ETA 5m39s] |
| [18:37:29] | MythBuild: | I'll give a shout when the build finishes |
| [18:37:46] | kormoc (kormoc!~kormoc@mythtv/developer/kormoc) has joined #mythtv | |
| [18:39:09] | MythBuild: | Hey! build cppcheck-master #539 is complete: Success [build successful] |
| [18:39:09] | MythBuild: | Build details are at http://code.mythtv.org/buildbot/builders/cppc . . . r/builds/539 |
| [18:39:39] | Beirdo: | no difference |
| [18:44:17] | Beirdo: | gonna recompile 1.50 for the moment, don't have much time to mess with it |
| [18:45:01] | Beirdo: | MythBuild: force build cppcheck-master now |
| [18:45:02] | MythBuild: | build forced [ETA 3m39s] |
| [18:45:02] | MythBuild: | I'll give a shout when the build finishes |
| [18:45:08] | MythBuild: | build #540 of cppcheck-master is complete: Failure [failed shell shell_1] Build details are at http://code.mythtv.org/buildbot/builders/cppc . . . r/builds/540 |
| [18:45:14] | Beirdo: | what? |
| [18:45:43] | Beirdo: | OK, we must not have had 1.50 stock |
| [18:45:44] | Beirdo: | hmm |
| [18:45:52] | zombor_ (zombor_!~zombor_@kohana/developer/zombor) has quit (Read error: Connection reset by peer) | |
| [18:46:15] | Jordack (Jordack!~jordack@h69-131-44-221.plmomi.dedicated.static.tds.net) has joined #mythtv | |
| [18:46:18] | zombor (zombor!~zombor_@kohana/developer/zombor) has joined #mythtv | |
| [18:46:30] | stuartm: | should have been a simple update :( |
| [18:46:40] | Beirdo: | yeah |
| [18:46:54] | stuartm: | davide_: yes, it applies to fixes, I will be backporting shortly |
| [18:46:59] | Beirdo: | unfortunatley, I shoulda copied aside the previous binary. :) |
| [18:47:19] | Beirdo: | luckily, my git pull indicated the previous sha1 |
| [18:48:23] | Beirdo: | MythBuild: force build cppcheck-master now |
| [18:48:24] | MythBuild: | build forced [ETA 3m39s] |
| [18:48:24] | MythBuild: | I'll give a shout when the build finishes |
| [18:48:47] | Beirdo: | I'll go back to tweaking it tonight, do you wanna just run the tip of master? |
| [18:49:05] | stuartm: | sure, it's rarely broken |
| [18:49:25] | Beirdo: | OK. What I just launched now SHOULD be restored to what it was before |
| [18:49:34] | Beirdo: | I'll try the tip tonight |
| [18:49:43] | Beirdo: | or if I get a spare moment a bit later :) |
| [18:53:22] | Beirdo: | Ok, it's not failing now at least. Should finish soon |
| [18:54:57] | Seeker`: | hmm, so when enabling BD debugging, there seems to be an almost constant stream of data appearing in bluray.c, which means that it is the file buffer which is running empty |
| [18:56:39] | Seeker`: | from the logs, it sends a request for data, gets 0 bytes in response and then waits for 2 seconds; I'm guessing this is the socket time out? |
| [18:56:52] | Beirdo: | taking its time because someone launched a build at the same time :) |
| [18:57:00] | stuartm: | hmm, subtitle colours are reversed on a DVD ripped by MakeMKV, outline is white and fill black – mplayer gets it right |
| [18:57:29] | stuartm: | Beirdo: how thoughtless! |
| [18:57:32] | MythBuild: | build #541 of cppcheck-master is complete: Success [build successful] Build details are at http://code.mythtv.org/buildbot/builders/cppc . . . r/builds/541 |
| [18:57:48] | Beirdo: | OK, back to 178 reported |
| [18:58:02] | Beirdo: | that will do for now, I'll play again tonight |
| [18:58:21] | Beirdo: | meeting time! |
| [18:58:32] | stuartm: | heh, mythmusic wipes out the work done in the last few days, it was down to 124 from 179 this morning |
| [19:01:47] | Seeker`: | who would know about RingBuffer code? |
| [19:19:53] | markk: | wagnerrp: there's a comment in the code about ipv6 – I should have mentioned in the commit that it may need fixing for ipv6. |
| [19:20:30] | markk: | Seeker': have you confirmed the data from the libbluray is uninterrupted? |
| [19:20:55] | Seeker`: | markk: I enabled debugging, and watched it with tail -f |
| [19:22:04] | markk: | Seeker'; btw, not that it helps, it's an issue that's been around since the bluray code was written. a fix would be very much appreciated:) |
| [19:22:28] | markk: | wagnerrp: I assume it's not working for you with ipv6? |
| [19:24:10] | wagnerrp: | no, i dont have anything to try it with |
| [19:24:32] | wagnerrp: | im just wondering in regards to fixing other things (specifically upnp) that dont work with ipv6 |
| [19:25:45] | markk: | ah – ok. raop may not work – though it should just be a straight endian fix if it isn't working ( I think) |
| [19:27:42] | markk: | stuartm: subtitle colours on a ripped dvd are guesswork – sometimes you get it right, sometimes you get it wrong. you don't have access to the original palette. |
| [19:28:54] | Seeker`: | markk: I submitted bug #10294, the attached log shows the pause. I don't fully understand the code, but can't the backend send an EOF message / close the connection? |
| [19:51:19] | davide_: | "QSqlDatabasePrivate::removeDatabase: connection 'DataDirectCon' is still in use, all queries will cease to work." I just started getting this error when running "mythbackend --testsched >/dev/null". Does anyone know what's going on? |
| [20:09:58] | Seeker`: | markk: any idea where I can find the code that streams the content to the socket on the backend? |
| [20:15:49] | wagnerrp: | Seeker`: talking about from frontend to backend? mythtv/programs/mythbackend/mainserver.cpp |
| [20:16:00] | wagnerrp: | backend to frontend* |
| [20:23:19] | Seeker`: | markk: wagnerrp should fasttimeout be set for blurays? |
| [20:23:40] | wagnerrp: | no idea on that one |
| [20:24:17] | danielk22 (danielk22!~danielk@96.57.9.142) has joined #mythtv | |
| [20:25:48] | danielk22 (danielk22!~danielk@96.57.9.142) has quit (Client Quit) | |
| [20:26:12] | danielk22 (danielk22!~danielk@96.57.9.142) has joined #mythtv | |
| [20:26:48] | Seeker`: | wagnerrp: can you have a look at ringbuffer.cpp line 853/854, and compare that to www.mythtv.org/wiki/QUERY_FILETRANSFER_(Myth_Protocol)#SET_TIMEOUT |
| [20:30:03] | rsiebert_ (rsiebert_!~quassel@e179130255.adsl.alicedsl.de) has joined #mythtv | |
| [20:32:02] | wagnerrp: | im not sure what im supposed to be looking at |
| [20:32:08] | wagnerrp: | that line is for livetv sessions, not blurays |
| [20:33:26] | Seeker`: | wagnerrp: ony place I can see that sets SET_TIMEOUT, and I think it should be false for livetv anyway |
| [20:33:27] | stuartm: | same reasoning though, fast transitiond |
| [20:33:32] | Captain_Murdoch: | Seeker`, I don't think you're hitting the timeout. you get back a '0 bytes read' response to the last read, then it waits 2 seconds before sending the DONE, then it opens the new file right away. I think the 2-second delay is the BD code not pro-actively opening the next file when it hits EOF on the current file. I don't know if that pro-active opening is something that happens in the BD ringbuffer or not. |
| [20:34:25] | Captain_Murdoch: | I don't see any FE->BE query taking more than short time in ms. |
| [20:52:13] | danielk22: | Is the gcc memory usage really much better in 4.7? http://gcc.gnu.org/bugzilla/show_bug.cgi?id=36687 |
| [20:53:03] | natanojl (natanojl!~jonatan@c83-252-237-63.bredband.comhem.se) has joined #mythtv | |
| [20:55:46] | map7_ (map7_!~map7@ppp118-209-93-114.lns20.mel4.internode.on.net) has quit (Ping timeout: 252 seconds) | |
| [20:56:28] | wagnerrp: | i honestly cant say ive seen that problem since stopping compiling on my old 1GB+0GB swap frontend |
| [20:57:03] | wagnerrp: | and that was only when using -j2 |
| [21:01:34] | danielk22: | wagnerrp: We refactored a few mythtv files to lower their memory usage, so you shouldn't have been seeing any issues. |
| [21:09:03] | danielk22: | stuartm: Yikes! mythmusic really drove up the cppcheck warnings! |
| [21:10:21] | Beirdo: | that's what we get for ignoring it :) |
| [21:11:05] | Beirdo: | heh. You shoulda seen it when I briefly tried cppcheck 1.52 |
| [21:11:24] | danielk22: | Beirdo: How long is cppcheck taking to run these days? Fast enough that it be run on every commit? |
| [21:11:27] | Beirdo: | I think we have some command-line tweakage, it spiked up to over 3000 |
| [21:11:53] | danielk22: | Beirdo: yah, I ran 1.52 too.. I think that's just new error types that it's finding.. |
| [21:12:13] | Beirdo: | about 9 minutes |
| [21:12:27] | Beirdo: | well, most of the errors were that it couldn't find the include files |
| [21:12:41] | Beirdo: | so likely some minor attention required |
| [21:13:05] | Beirdo: | we could run it more often, but every commit? |
| [21:13:14] | Seeker`: | Captain_Murdoch: yeah, sounds about right. I have absolutely no idea how to fix that. |
| [21:13:40] | Beirdo: | the problem is, it runs on the same machine as the 64bit test compile, so it ends up being a lot slower if in parallel |
| [21:14:57] | Beirdo: | it's every 2h right now |
| [21:15:08] | Beirdo: | (only when changes made) |
| [21:16:09] | skd5aner: | xris: scrolling through backlog – if you are curious, the release notes track which libs have been added / removed – I can't say it's 100% accurate, but I try to keep track as best as I can (it helps when devs explicitly call it out in the commit note, otherwise I track changes to .config to catch the rest) |
| [21:16:26] | skd5aner: | xris: http://www.mythtv.org/wiki/Release_Notes_-_0. . . . site_Changes |
| [21:17:11] | Beirdo: | danielk22: another thing to track down is the signed/unsigned warnings from the compile. I'll take a look tonight if you guys haven't got to it :) |
| [21:18:04] | Beirdo: | it's nice to see the cleanliness of the code improve |
| [21:18:38] | Beirdo: | danielk22: Oh, and if you want a new run sooner you can always do: |
| [21:18:47] | Beirdo: | MythBuild: force build cppcheck-master now |
| [21:18:48] | MythBuild: | build forced [ETA 8m24s] |
| [21:18:48] | MythBuild: | I'll give a shout when the build finishes |
| [21:18:51] | Beirdo: | :) |
| [21:19:45] | Beirdo: | which, after a massive number of fixes, might just be what the doctor ordered |
| [21:22:15] | skd5aner: | MythBuild: force build maserati now |
| [21:22:15] | MythBuild: | no such builder 'maserati' |
| [21:22:20] | skd5aner: | too bad |
| [21:22:33] | Beirdo: | now now |
| [21:22:40] | Beirdo: | don't break the buildbot :) |
| [21:22:55] | skd5aner: | MythBuild: force build Me A Sandwich Now |
| [21:22:55] | MythBuild: | no such builder 'Me' |
| [21:23:03] | ** skd5aner had to do it ** | |
| [21:24:34] | danielk22: | Beirdo: Ah, I didn't know that was possible. Very cool. |
| [21:24:57] | Beirdo: | yeah, so far it's enabled :) |
| [21:25:12] | danielk22: | Beirdo: Yeah, I plan to look at the compiler warnings too... The cppchecks were just something I could do on a 20 hour flight.. |
| [21:25:28] | Beirdo: | ahh, yeah |
| [21:25:39] | Beirdo: | good way to pass the time too :) |
| [21:26:10] | Beirdo: | the cool thing is the rsync at the end is super-fast now |
| [21:26:43] | Beirdo: | (more noticable on the doxygen rsync) |
| [21:27:44] | MythBuild: | Hey! build cppcheck-master #544 is complete: Success [build successful] |
| [21:27:44] | MythBuild: | Build details are at http://code.mythtv.org/buildbot/builders/cppc . . . r/builds/544 |
| [21:28:14] | Beirdo: | nice. Down to 81 |
| [21:28:19] | danielk22: | Pretty fast.. and down below a hundred again :) |
| [21:28:35] | skd5aner: | 20 hour flight... geeze |
| [21:29:15] | skd5aner: | can it get much longer than that? |
| [21:29:35] | Beirdo: | not by much |
| [21:29:44] | danielk22: | skd5aner: not much.. this was 11 timezones.. |
| [21:30:10] | skd5aner: | I think 12–14 was the longest for me |
| [21:30:13] | Beirdo: | that'd be about NYC-Australia or so, no? |
| [21:30:40] | Beirdo: | or would that be a touch longer.. hmm |
| [21:31:11] | sphery: | dblain: Out of curiosity, is it possible for us to affect the serialization format for a standard Qt type? Specifically, I'd like to include milliseconds when serializing the QDateTime for logs (but it won't make sense to do so for most other QDateTimes). Thought there might be an approach for specifying that in the datacontract or something. It's not critical--and I can always do it with a wrapper class, but just wanted to make sure I used ... |
| [21:31:17] | sphery: | ... the best approach. |
| [21:32:14] | sphery: | danielk22: Assuming you're not on that flight, now, have you seen http://www.gossamer-threads.com/lists/mythtv/users/500648#500648 ? Seems like that may be a typo? |
| [21:32:42] | danielk22: | Beirdo: Other way around the planet JFK->HYD.. But on second thought I think JFK to AMZ would be a fair bit longer. You'd need to go 18hr hours to JP, 5hr to AU then 6hr to NZ.. |
| [21:32:45] | sphery: | oh, and that can wait if you're busy... |
| [21:33:18] | danielk22: | sphery: yeah, looks like a reversal of intent.. |
| [21:33:30] | sphery: | if you'd like I can flip it for you |
| [21:33:50] | danielk22: | I'll do it. np. |
| [21:34:08] | sphery: | thx |
| [21:34:33] | Beirdo: | ahh |
| [21:34:38] | Seeker`: | Captain_Murdoch: do you think a valid solution would be to keep two files open, and just swap pointers? |
| [21:40:29] | stuartm: | skd5aner: iirc it was something like that when I flew to Australia, a two hour stopover in Kuala Lumpur though |
| [21:40:44] | stuartm: | definitely grim |
| [21:42:38] | zombor (zombor!~zombor_@kohana/developer/zombor) has quit (Ping timeout: 240 seconds) | |
| [21:42:48] | stuartm: | googling it's between 22h 30m – 23h 30m for Heathrow to Melbourne |
| [21:43:23] | Beirdo: | yeah. |
| [21:43:49] | Beirdo: | my aunt used to do Toronto->Kota Kinabalu far too often |
| [21:44:08] | Beirdo: | so it turned out a similar nearly all-day flight situation |
| [21:50:22] | Captain_Murdoch: | Seeker`, I think the issue is that we don't know what the next file is until it's asked for. I don't know that part of the code well enough, but if it does any buffering, I'd expect it to ask for the next file ahead of time when it sees the current file is at EOF. |
| [21:52:41] | Seeker`: | Captain_Murdoch: the code in bluray.c does |
| [21:53:12] | Seeker`: | (know what file is next, that is) |
| [21:54:46] | Captain_Murdoch: | which line range are you looking at? |
| [21:54:50] | Seeker`: | (see ~ line 1173 in bluray.c – that is how you get / open the new file) |
| [21:55:13] | Seeker`: | 1195–1203 does the actual opening |
| [21:57:50] | stuartm: | danielk22: so the pbb update bug turned out to be unrelated to disabling the reload during playback, it just exposed a bug which had been in that code since day one |
| [21:58:31] | stuartm: | the fix couldn't have been simpler, entirely disproportionate to the time I spend tracking it down :/ |
| [21:58:54] | Beirdo: | isn't that usually the case? |
| [21:59:34] | stuartm: | often enough, yeah :) |
| [22:00:47] | Seeker`: | Captain_Murdoch: I think the code in bluray.c could be made to handle opening the new file if it were possible to pass up the 'buffer has reached EOF' message back up |
| [22:04:03] | Captain_Murdoch: | Seeker`, I wonder if your 2 second delay is in BDRingBuffer::WaitForPlayer(). There's a loop in there that times out after 2 seconds. That method is called whenever the BD_EVENT_END_OF_TITLE event is sent. |
| [22:06:16] | jya (jya!~jyavenard@120.148.99.54) has joined #mythtv | |
| [22:06:16] | jya (jya!~jyavenard@mythtv/developer/jya) has joined #mythtv | |
| [22:06:16] | jya (jya!~jyavenard@120.148.99.54) has quit (Changing host) | |
| [22:06:26] | Seeker`: | Captain_Murdoch: I don't think it uses a BDRingBuffer |
| [22:09:08] | Captain_Murdoch: | probably not it anyway, this isn't the end of a title, just a clip right |
| [22:09:16] | Seeker`: | correct |
| [22:09:29] | Seeker`: | I think it uses a FileBuffer, which wraps a RemoteFile |
| [22:11:33] | natanojl (natanojl!~jonatan@c83-252-237-63.bredband.comhem.se) has quit (Ping timeout: 260 seconds) | |
| [22:14:50] | danielk22: | stuartm: So that was the whole of it? I thought that commit would just be step 1 of 20 :) |
| [22:16:37] | stuartm: | danielk22: that was it, at least it fixed the issue for me and I couldn't see any other issues which were directly related |
| [22:19:34] | stuartm: | the change of events precipitated by the invalid date was a bit longer and more complicated than described in the commit message |
| [22:19:58] | danielk22: | Cool! Honestly, I love 1 line fixes. |
| [22:21:22] | xavierh (xavierh!~chatzilla@cpc1-swin3-0-0-cust274.3-1.cable.virginmedia.com) has joined #mythtv | |
| [22:28:27] | stuartm: | we probably want to ignore the CdDecoder warnings for now, Paul is asking Lawrence to update his libcdio patch in the hope of getting it in for 0.25 (it's sorely needed), changes to CdDecoder might cause conflicts |
| [22:28:42] | Seeker`: | Captain_Murdoch: looks like it uses a FileRingBuffer, wrapping a RemoteFile, which bluray.c uses by using file_mythiowrapper.c |
| [22:28:51] | stuartm: | s/might/will/ |
| [22:29:09] | MythBuild: | build #545 of master-vista-mingw-32bit is complete: Exception [exception git] Build details are at http://code.mythtv.org/buildbot/builders/mast . . . t/builds/545 blamelist: Stuart Morgan <smorgan@mythtv.org >, John Poet <jpoet@mythtv.org >, Daniel Kristjansson <danielk@cuymedia.net > |
| [22:29:24] | Captain_Murdoch: | Seeker`, yeah, I was thinking we bypassed BDRB in this case. mythiowrapper is my code/hack to support remote DVD/BD. |
| [22:29:55] | stuartm: | MythBuild: force build master-vista-mingw-32bit now Stupid thing |
| [22:29:56] | danielk22: | stuartm: Can you put those in the suppression list? |
| [22:30:00] | MythBuild: | The build has been queued, I'll give a shout when it starts |
| [22:30:07] | stuartm: | sure |
| [22:30:34] | stuartm: | MythBuild: cancel build master-vista-mingw-32bit |
| [22:30:46] | ** stuartm is guessing at syntax ** | |
| [22:32:39] | EagleIJoe (EagleIJoe!~rockhound@d030112.adsl.hansenet.de) has joined #mythtv | |
| [22:33:25] | EagleIJoe (EagleIJoe!~rockhound@d030112.adsl.hansenet.de) has left #mythtv () | |
| [22:33:47] | stuartm: | danielk22: turns out it was already suppressed but recent changes had changed the line number |
| [22:35:00] | EagleIJoe (EagleIJoe!~rockhound@d030112.adsl.hansenet.de) has joined #mythtv | |
| [22:35:21] | EagleIJoe (EagleIJoe!~rockhound@d030112.adsl.hansenet.de) has quit (Client Quit) | |
| [22:35:43] | Seeker`: | Captain_Murdoch: My idea was to create a secondStream pointer, and doing 1 block of read in to that stream from the next title, then when bluray.c actually hits EoF, it swaps secondStream to the main stream |
| [22:36:29] | Beirdo: | hmm |
| [22:36:45] | Seeker`: | that way all of the file setup is done, and there is some data there waiting |
| [22:37:07] | Beirdo: | MythBuild: help |
| [22:37:07] | MythBuild: | Get help on what? (try 'help <foo>', or 'commands' for a command list) |
| [22:37:14] | Beirdo: | MythBuild: commands |
| [22:37:14] | MythBuild: | buildbot commands: commands, dance, destroy, excited, force, hello, help, last, list, mute, notify, source, status, stop, unmute, version, watch |
| [22:37:21] | Captain_Murdoch: | I think we've kept that file pretty much stock until now. pre-opening the next clip might be the simplest way around the issue though since the code doesn't assume there's any kind of buffering in the file_* methods. |
| [22:37:27] | Beirdo: | MythBuild: help force |
| [22:37:28] | MythBuild: | Usage: force build [--branch=branch] [--revision=revision] <which> <reason> – Force a build |
| [22:37:35] | Beirdo: | I don't see a cancel. |
| [22:37:37] | Seeker`: | Captain_Murdoch: it doesn't afaik |
| [22:37:38] | Beirdo: | that's odd |
| [22:37:43] | Beirdo: | ooohhh |
| [22:37:47] | Beirdo: | MythBuild: help stop |
| [22:37:47] | MythBuild: | Usage: stop build <which> <reason> – Stop a running build |
| [22:37:47] | stuartm (stuartm!~stuartm@mythtv/developer/stuartm) has quit (Remote host closed the connection) | |
| [22:37:51] | Beirdo: | ahhh |
| [22:38:10] | Beirdo: | MythBuild: stop build master-vista-mingw-32bit now |
| [22:38:10] | MythBuild: | build 546 interrupted |
| [22:38:15] | Beirdo: | there we go |
| [22:38:18] | MythBuild: | build #546 of master-vista-mingw-32bit is complete: Exception [exception shell configure] Build details are at http://code.mythtv.org/buildbot/builders/mast . . . t/builds/546 blamelist: Stuart Morgan <smorgan@mythtv.org > |
| [22:38:28] | MythBuild: | build forced [ETA 1h23m41s] |
| [22:38:28] | MythBuild: | I'll give a shout when the build finishes |
| [22:38:59] | Beirdo: | heh, it woulda started anyways, but that works |
| [22:39:21] | MythBuild: | build #547 of master-vista-mingw-32bit is complete: Exception [exception git] Build details are at http://code.mythtv.org/buildbot/builders/mast . . . t/builds/547 |
| [22:39:24] | danielk22: | Seeker`: I don't think you really need to get super complicated.. Just doing a read from the next file 30 seconds before you need it and tossing the data is sufficient. That will put it in the OS cache and will allow it to be reread very quickly when it is actually needed. |
| [22:39:41] | Beirdo: | I need to smack windows around again, it seems |
| [22:40:04] | Beirdo: | and it's starting again. |
| [22:40:28] | MythBuild: | build #548 of master-vista-mingw-32bit is complete: Exception [exception git] Build details are at http://code.mythtv.org/buildbot/builders/mast . . . t/builds/548 blamelist: Paul Harrison <pharrison@mythtv.org >, Mark Kendall <mkendall@mythtv.org > |
| [22:40:45] | Beirdo: | OK, time to kick it. |
| [22:41:23] | clever: | maybe something didnt fully exit and is holding the file/dir open |
| [22:43:00] | Beirdo: | It's windows, it needs to be reminded how to behave sometimes |
| [22:43:30] | Beirdo: | MythBuild: force build master-vista-mingw-32bit now, bunghole. |
| [22:43:30] | MythBuild: | build forced [ETA 1h23m41s] |
| [22:43:30] | MythBuild: | I'll give a shout when the build finishes |
| [22:43:46] | clever: | lol |
| [22:44:08] | Seeker`: | danielk22: is that really sufficient? because it is reading over the network |
| [22:46:38] | danielk22: | Seeker`: The buffering in mythtv should be able to deal with network latencies. I'm assuming it's the spin up of a new file reader that's slowing you down too much; most of that spin up is filling up the buffers. |
| [22:46:46] | stuartm (stuartm!~stuartm@cpc1-derb9-0-0-cust441.8-3.cable.virginmedia.com) has joined #mythtv | |
| [22:46:47] | stuartm (stuartm!~stuartm@cpc1-derb9-0-0-cust441.8-3.cable.virginmedia.com) has quit (Changing host) | |
| [22:46:47] | stuartm (stuartm!~stuartm@mythtv/developer/stuartm) has joined #mythtv | |
| [22:47:11] | Beirdo: | heya, stuartm ... the command you were looking for seems to be "stop build". |
| [22:47:12] | Seeker`: | bah, it only takes 0.2 seconds between requesting the new file and getting data back |
| [22:47:18] | Seeker`: | so that won't fix it |
| [22:47:18] | danielk22: | Seeker`: In any case it should be easy to test. |
| [22:47:23] | Beirdo: | I kicked the slave in the head, and restarted the build |
| [22:48:24] | Captain_Murdoch: | Seeker`, I think having the next clip open would be fairly non-invasive if handled as part of _open_m2ts() , _change_angle(), and _close_playlist(), and bd_close() |
| [22:49:44] | Seeker`: | Captain_Murdoch: i'm not sure if it will fix the problem, thinking about it |
| [22:50:40] | Captain_Murdoch: | Seeker`, if you have the next file already open (and hence it's buffers already full), then switchover should be immediate since it's all local reads. |
| [22:50:56] | Seeker`: | do the buffers autmatically fill when you open the file? |
| [22:51:10] | Seeker`: | or is there an easy way to say 'fill this buffer'? |
| [22:52:04] | stuartm: | Beirdo: cool, there didn't seem much point in queuing a second build, especially when that one is so slow |
| [22:57:29] | Captain_Murdoch: | Seeker`, if we start it with the readahead thread running it should start reading for us. |
| [22:58:01] | Seeker`: | Captain_Murdoch: cool, I'll start having a look then |
| [22:59:11] | ** stuartm just noticed that he suppressed the cddecoder stuff moments after Paul completely removed it ** | |
| [22:59:36] | Beirdo: | nice timing :) |
| [23:00:51] | stuartm: | I knew he was planning to commit the patch but not that he did it two minutes before I hit 'return' |
| [23:02:20] | Captain_Murdoch: | Seeker`, could add a nextfp pointer in BD_STREAM and check that whever we call _close_m2ts to see if we need to also call a _close_next_m2ts() if one is open. you might want to run it by iamlindoro since he is the main person for that area of the code. we've tried to keep the changes to a minimum except for the remote support I believe. |
| [23:03:41] | davide_: | Beirdo: Did you see my earlier post about seeing "QSqlDatabasePrivate::removeDatabase: connection 'DataDirectCon' is still in use, all queries will cease to work." when running "mythbackend --testsched?" It appears the DataDirectCon is being closed twice, once in Scheduler::~SCheduler() and again in MDBManager::CloseDatabases(). Is there any harm from removing the close from ~Scheduler()? |
| [23:03:59] | Seeker`: | Captain_Murdoch: not sure you'll always need to call close_next though, will you? |
| [23:04:07] | MythBuild: | build #122 of master-osx-snow-leopard is complete: Failure [failed compile_2] Build details are at http://code.mythtv.org/buildbot/builders/mast . . . d/builds/122 blamelist: Paul Harrison <pharrison@mythtv.org >, Mark Kendall <mkendall@mythtv.org > |
| [23:04:34] | Seeker`: | because in the middle of the movie you'd call open(file), read(file)...read(file), close(file), open(file2) etc. |
| [23:04:50] | Seeker`: | so it'd only be when you reach the end of the disk that you would need to close the next one |
| [23:05:03] | Beirdo: | davide_: not sure. I think that danielk22 may have a better grip on the current database shutdown code, he's tweaked it a couple times since I looked at it last. |
| [23:05:07] | Seeker`: | s/disk/title |
| [23:05:32] | Beirdo: | if that's during shutdown, there's likely no harm in disabling one or the other |
| [23:05:55] | Beirdo: | but I'm not sure at this point, I'd have to go brush up on the latest code |
| [23:08:34] | davide_: | danielk22: see bit about DataDirectCon above. |
| [23:09:21] | dblain: | sphery: regarding custom formatting of specific properties in the serialization code... It does not currently support it. The best we could do is add ClassInfo to the datacontract which we could extract and use during serialization. |
| [23:09:55] | davide_: | Beirdo: I tried it and didn't see any errors, but then, I didn't do anything exotic either. |
| [23:10:02] | dblain: | It wouldn't be too difficult to implement, but I'd want to think about the best way to structure the classinfo data to allow for the most flexability |
| [23:10:50] | danielk22: | davide_: The teardown in ~Scheduler prolly isn't needed anymore. But let me take a look. |
| [23:11:21] | sphery: | dblain: If you prefer, I could just create a class with a QDateTime member and just have a toString() that uses the format I want... Would be simple enough and should work, right? |
| [23:11:54] | Beirdo: | thanks, danielk22. :) I really need to refresh my brain on the changes there so I'm not so clueless :) |
| [23:12:32] | davide_: | danielk22: somebody (you?) removed the call to CloseSchedCon() that used to be there. I suspect it's the same issue. |
| [23:12:45] | danielk22: | davide_: Strangely I'm not seeing the error message you are. |
| [23:13:01] | dblain: | sphery: That would work in the short term, but having a way to give rendering hints (metadata about the properties) would be a cleaner long term solution |
| [23:13:07] | Beirdo: | could well be a race condition related to processor speed or the like |
| [23:13:23] | Beirdo: | multithreaded fun |
| [23:13:43] | danielk22: | davide_: Basically, the connections are all torn down in the thread epilog now, so explicitly closing the connections isn't necessary. |
| [23:14:01] | sphery: | dblain: ok, I'm just using seconds resolution for initial development, but may extend it... Thanks for the info. |
| [23:14:09] | danielk22: | That avoid all the issues we saw with closing connections in a different thread than the one that they were created in.. |
| [23:14:16] | danielk22: | s/avoid/avoids/ |
| [23:14:31] | Beirdo: | which plagued us quite a bit |
| [23:14:53] | davide_: | danielk22: FWIW, I just started seeing it. Ooh, now that I think of it, Debian/testing recently did a Qt update. I bet that's when I started seeing the new warning. |
| [23:22:57] | Jordack (Jordack!~jordack@h69-131-44-221.plmomi.dedicated.static.tds.net) has quit () | |
| [23:23:00] | danielk22: | davide_: Can you send a log? It should actually be harmless to call CloseDDCon() before CloseDatabases(). |
| [23:27:15] | davide_: | danielk22: It's in the mail. |
| [23:28:49] | danielk22: | davide_: got it |
| [23:30:18] | rsiebert_ (rsiebert_!~quassel@e179130255.adsl.alicedsl.de) has quit (Ping timeout: 272 seconds) | |
| [23:32:36] | Seeker`: | Captain_Murdoch: bah, I cant test this properly; my dev machine isn't wired to my backend :( |
| [23:41:11] | Seeker` (Seeker`!~cjo20@unaffiliated/seeker) has quit (Ping timeout: 252 seconds) | |
| [23:47:16] | Seeker` (Seeker`!~cjo20@host86-160-233-91.range86-160.btcentralplus.com) has joined #mythtv | |
| [23:47:16] | Seeker` (Seeker`!~cjo20@host86-160-233-91.range86-160.btcentralplus.com) has quit (Changing host) | |
| [23:47:16] | Seeker` (Seeker`!~cjo20@unaffiliated/seeker) has joined #mythtv | |
| [23:50:54] | Seeker` (Seeker`!~cjo20@unaffiliated/seeker) has quit (Read error: Operation timed out) | |
| [23:52:17] | Seeker` (Seeker`!~cjo20@host86-160-233-91.range86-160.btcentralplus.com) has joined #mythtv | |
| [23:52:18] | Seeker` (Seeker`!~cjo20@host86-160-233-91.range86-160.btcentralplus.com) has quit (Changing host) | |
| [23:52:18] | Seeker` (Seeker`!~cjo20@unaffiliated/seeker) has joined #mythtv | |
| [23:59:50] | wagnerrp: | danielk22: yeah, this would have been similar time frame to you opening that report, 2008–2009, building 0.21 |
| Tuesday, January 31st, 2012 | ||
| [00:01:20] | wagnerrp: | sphery: would it be better to transform it to epoch time instead? |
| [00:01:55] | wagnerrp: | easier to manipulate that way |
| [00:04:19] | MythBuild: | Hey! build master-vista-mingw-32bit #549 is complete: Success [build successful] |
| [00:04:19] | MythBuild: | Build details are at http://code.mythtv.org/buildbot/builders/mast . . . t/builds/549 |
| [00:07:06] | MythBuild: | build #125 of master-osx-snow-leopard is complete: Success [build successful] Build details are at http://code.mythtv.org/buildbot/builders/mast . . . d/builds/125 |
| [00:09:28] | zombor (zombor!~zombor_@kohana/developer/zombor) has joined #mythtv | |
| [00:11:01] | xavierh (xavierh!~chatzilla@cpc1-swin3-0-0-cust274.3-1.cable.virginmedia.com) has quit (Ping timeout: 276 seconds) | |
| [00:26:21] | zCougar (zCougar!~cougar@2001:67c:32c:600:fc5b:69b5:7872:dd22) has quit (Ping timeout: 252 seconds) | |
| [00:38:10] | zCougar (zCougar!~cougar@2001:67c:32c:600:395f:ee0f:136b:4591) has joined #mythtv | |
| [00:40:54] | Captain_Murdoch: | Seeker`, yeah, you'd only need to cal the _close_next_m2ts when you call bd_close or in a few other places that _close_m2ts is called like _change_angle and _close_playlist (off the top of my head) |
| [00:43:52] | Captain_Murdoch: | Seeker`, I think that if you "mythavtest myth://THEBDSG@localhost/path/to/my/bd/BDMV" then it will stream the bluray over the protocol even if it's local if you have a local BE on your dev box and access to the BDMV dir |
| [00:47:02] | Seeker`: | Captain_Murdoch: just rerouted a few cat 5 cables across the floor for now |
| [00:47:26] | Captain_Murdoch: | ah, physical connectivity, can't help you there. :) |
| [00:48:11] | Seeker`: | I've got it to the point where it has loaded the clip it needs next. then it segfaults :P |
| [00:48:48] | Captain_Murdoch: | care to pastebin the patch? |
| [00:49:00] | Seeker`: | give me a sec |
| [00:53:50] | Seeker`: | bah, doesn't work |
| [00:53:54] | Seeker`: | fixed the segfault though |
| [00:54:09] | jpabq_ (jpabq_!~jpabq@mythtv/developer/jpabq) has quit (Quit: jpabq_) | |
| [00:56:53] | stichnot (stichnot!~chatzilla@192.55.55.39) has quit (Ping timeout: 248 seconds) | |
| [00:57:22] | jcarlos (jcarlos!~quassel@85.137.99.76.dyn.user.ono.com) has quit (Read error: Connection reset by peer) | |
| [00:57:28] | jcarlos (jcarlos!~quassel@85.137.99.76.dyn.user.ono.com) has joined #mythtv | |
| [01:05:17] | Seeker`: | Captain_Murdoch: right, got it preloading and swapping now, but it doesn't fix it. |
| [01:05:54] | Seeker`: | Captain_Murdoch: I think the preload might need to be done in a different thread to actually see any benefit |
| [01:09:10] | Seeker`: | Captain_Murdoch: might improve it marginally actually. Still symptomatic |
| [01:11:47] | Captain_Murdoch: | yeah, could be the opening of the 'next' next file is causing the delay now since you already have the 'next' file open. |
| [01:13:32] | Seeker`: | Captain_Murdoch: bah, it doesn't fix it :( |
| [01:14:04] | Seeker`: | I stopped it preloading the file if it finds a preloaded match (i.e. every other swap should be fixed), but it still stutters on each change |
| [01:19:56] | Seeker`: | Captain_Murdoch: http://paste.ubuntu.com/823309 |
| [01:29:38] | Captain_Murdoch: | hard to understand context with only that short part. can you paste the whole diff? |
| [01:29:53] | Seeker`: | that is the whole diff :P |
| [01:30:05] | Seeker`: | I've not added stuff to do with closing the files yet |
| [01:32:03] | Seeker`: | or do you mean paste the whole file? |
| [01:35:45] | Captain_Murdoch: | no, I know that's part of _open_m2ts, I just am not seeing how reads are coming from the right place if you're just overriding the value of st inside _open_m2ts. outside open_m2ts, reads will still come from the caller's 'st' which is bd->st0. do you actually see it printing your debug lines alternating in your logs? |
| [01:36:51] | Seeker`: | Captain_Murdoch: I use memcpy to copy the struct data over |
| [01:37:19] | Captain_Murdoch: | ah, doh! I saw that and glossed over it. |
| [01:37:26] | Seeker`: | :) |
| [01:39:51] | ** Captain_Murdoch shouldn't try to eat pizza, type, and think all at the same time. ** | |
| [01:40:58] | Seeker`: | I find doing two things at once pretty difficult :P |
| [01:43:34] | kormoc (kormoc!~kormoc@mythtv/developer/kormoc) has quit (Quit: kormoc) | |
| [01:44:53] | Captain_Murdoch: | can you paste the output log showing your cjo log messages so I can see them in context with the other "BD_DEBUG(DBG_STREAM" messages |
| [01:45:59] | Seeker`: | they're fairly few and far between compared to the other messges |
| [01:47:08] | Captain_Murdoch: | the ringbuffer ones? |
| [01:48:22] | Seeker`: | the BD_DEBUG ones look like "bluray.c:401: Reading unit [6144 bytes] at 2967552... (0x155cf80)" |
| [01:49:50] | Captain_Murdoch: | ok, nevermind. |
| [01:59:57] | Seeker`: | I dont think this approach will work. Next thing to look at is not creating a new ringbuffer each time, just swapping the file it is reading from in to the buffer once it hits an EoF |
| [02:04:51] | davide_ (davide_!~david@mythtv/developer/gigem) has quit (Remote host closed the connection) | |
| [02:05:13] | davide_ (davide_!~david@host70.16.intrusion.com) has joined #mythtv | |
| [02:05:14] | davide_ (davide_!~david@host70.16.intrusion.com) has quit (Changing host) | |
| [02:05:14] | davide_ (davide_!~david@mythtv/developer/gigem) has joined #mythtv | |
| [02:18:40] | Seeker`: | Captain_Murdoch: This is gonna be a big rewrite I think; Going to try to implement an EoF detection using FileRingBuffer::GetReadPosition( and FileRingBuffer::GetRealFileSize(), change the code that looks for running off the end of the current file to look for when the two of those are equal, and change open_m2ts to call FileRingBuffer::OpenFile() instead of creating a new FileRingBuffer for the new file |
| [02:19:40] | wagnerrp: | Seeker`: is dealing with multiple files really that common? |
| [02:19:56] | wagnerrp: | is this some segmented disk that allows multiple different cuts of a film? |
| [02:20:16] | Captain_Murdoch: | Seeker`, we have ReOpen() support in the remote file proto now, I forgot I added that. that could be useful here possibly. |
| [02:20:58] | Captain_Murdoch: | actually, now I recall why. I haven't tested it with reads, only writes. I think I have it disabled for reads because of caching. |
| [02:21:29] | Captain_Murdoch: | it was added for the HTTP Live Streaming changes since that writes lots of tiny files out using a ringbuffer. |
| [02:23:11] | Seeker`: | wagnerrp: AIUI, makemkvcon is a relatively common way of backing up whole bluray discs, which leaves you with a load of .m2ts files |
| [02:23:47] | wagnerrp: | seems odd, normally a single video is a single m2ts on the disk itself |
| [02:24:04] | wagnerrp: | at least in my experience |
| [02:24:39] | Seeker`: | I don't think I've got any blurays which are a single m2ts |
| [02:25:02] | Seeker`: | Disney blurays are particularly bad for it. I think it might be to do with how they deal with different language versions of the films |
| [02:25:19] | kormoc (kormoc!~kormoc@mythtv/developer/kormoc) has joined #mythtv | |
| [02:25:25] | danielk22 (danielk22!~danielk@96.57.9.142) has quit (*.net *.split) | |
| [02:25:26] | mag0o (mag0o!20001@slackhost.lynchmv.com) has quit (*.net *.split) | |
| [02:25:26] | foobum (foobum!~foobum@78-105-15-213.zone3.bethere.co.uk) has quit (*.net *.split) | |
| [02:25:26] | mrand (mrand!~mrand@ubuntu/member/mrand) has quit (*.net *.split) | |
| [02:25:28] | Slasher` (Slasher`!~Slasher@188.165.164.15) has quit (*.net *.split) | |
| [02:25:32] | mag0o (mag0o!20001@slackhost.lynchmv.com) has joined #mythtv | |
| [02:25:58] | Slasher` (Slasher`!~Slasher@188.165.164.15) has joined #mythtv | |
| [02:26:00] | danielk22 (danielk22!~danielk@96.57.9.142) has joined #mythtv | |
| [02:26:04] | foobum (foobum!~foobum@78-105-15-213.zone3.bethere.co.uk) has joined #mythtv | |
| [02:26:18] | mrand (mrand!~mrand@ubuntu/member/mrand) has joined #mythtv | |
| [02:26:46] | Seeker`: | wagnerrp: although I am slightly tempted to see about re-ripping them all as .mkv files |
| [02:27:11] | Seeker`: | as I doubt there will be BDJ support in the near future |
| [02:43:28] | pheld (pheld!idkvii@109-109-76-195.bb.cust.telefiber.no) has quit (Quit: Leaving.) | |
| [02:44:00] | voyo (voyo!~voyo@213-134-188-198.home.aster.pl) has joined #mythtv | |
| [03:36:06] | voyo (voyo!~voyo@213-134-188-198.home.aster.pl) has quit (Read error: Connection reset by peer) | |
| [03:38:15] | kormoc (kormoc!~kormoc@mythtv/developer/kormoc) has left #mythtv () | |
| [03:43:20] | stichnot (stichnot!~chatzilla@192.55.54.38) has joined #mythtv | |
| [03:54:17] | kormoc (kormoc!~kormoc@mythtv/developer/kormoc) has joined #mythtv | |
| [03:58:16] | Beirdo: | OK, let's try this again |
| [03:58:25] | Beirdo: | MythBuild: force build cppcheck-master now |
| [03:58:25] | MythBuild: | build forced [ETA 9m31s] |
| [03:58:25] | MythBuild: | I'll give a shout when the build finishes |
| [04:07:29] | MythBuild: | Hey! build cppcheck-master #548 is complete: Success [build successful] |
| [04:07:29] | MythBuild: | Build details are at http://code.mythtv.org/buildbot/builders/cppc . . . r/builds/548 |
| [04:11:36] | Beirdo: | hwhwhw |
| [04:11:43] | Beirdo: | 3210 results |
| [04:11:51] | Beirdo: | OK, what changed. |
| [04:27:19] | Memphis (Memphis!Memphisau@124-170-87-30.dyn.iinet.net.au) has joined #mythtv | |
| [04:34:41] | zombor (zombor!~zombor_@kohana/developer/zombor) has quit (Remote host closed the connection) | |
| [04:38:42] | Beirdo: | MythBuild: force build cppcheck-master now |
| [04:38:43] | MythBuild: | build forced [ETA 9m17s] |
| [04:38:43] | MythBuild: | I'll give a shout when the build finishes |
| [04:38:48] | Beirdo: | Let's try this again |
| [04:47:27] | MythBuild: | Hey! build cppcheck-master #549 is complete: Success [build successful] |
| [04:47:28] | MythBuild: | Build details are at http://code.mythtv.org/buildbot/builders/cppc . . . r/builds/549 |
| [04:48:06] | Beirdo: | down to 248 |
| [04:56:20] | Beirdo: | OK, I think we are at a decent spot, but stuartm might wanna take a look at the config and see if there's something else obvious, especially for the lzo stuff |
| [04:58:51] | netw1z (netw1z!~overseas_@cpe-72-225-175-91.nyc.res.rr.com) has quit (Read error: Connection reset by peer) | |
| [05:05:04] | netw1z (netw1z!~overseas_@cpe-72-225-175-91.nyc.res.rr.com) has joined #mythtv | |
| [05:43:55] | cattelan is now known as cattelan_away | |
| [06:03:54] | superm1 (superm1!u4318@ubuntu/member/superm1) has quit (Read error: Operation timed out) | |
| [06:11:28] | superm1 (superm1!u4318@ubuntu/member/superm1) has joined #mythtv | |
| [06:25:57] | stichnot (stichnot!~chatzilla@192.55.54.38) has quit (Ping timeout: 248 seconds) | |
| [08:11:32] | gregL (gregL!~greg@cpe-74-76-125-87.nycap.res.rr.com) has quit (Read error: Connection reset by peer) | |
| [08:16:40] | pheld (pheld!~heldal@cl-5.osl-01.no.sixxs.net) has joined #mythtv | |
| [08:28:23] | gregL (gregL!~greg@cpe-74-76-125-87.nycap.res.rr.com) has joined #mythtv | |
| [08:29:46] | Memphisau (Memphisau!~Memphisau@124-170-98-144.dyn.iinet.net.au) has joined #mythtv | |
| [08:33:53] | Memphis (Memphis!Memphisau@124-170-87-30.dyn.iinet.net.au) has quit (Ping timeout: 252 seconds) | |
| [08:34:01] | Memphis (Memphis!Memphisau@124-171-91-148.dyn.iinet.net.au) has joined #mythtv | |
| [08:34:46] | Memphisau (Memphisau!~Memphisau@124-170-98-144.dyn.iinet.net.au) has quit (Ping timeout: 276 seconds) | |
| [09:40:35] | ** stuarta hunts around for libcdio ** | |
| [09:45:05] | ** stuarta ahahs! ** | |
| [09:50:50] | eharris (eharris!~eharris@99-179-7-82.lightspeed.austtx.sbcglobal.net) has quit (Ping timeout: 245 seconds) | |
| [10:00:02] | Peitolm: | stuarta: have you seen any reports of issues where a MythFrontend.app compiled on one machine fails to work on another (assume the same xcode, and same 10.X) |
| [10:00:37] | stuarta: | nope, but then i haven't been looking :) |
| [10:00:42] | stuarta: | in a meeting, biab |
| [10:17:57] | stuarta: | oh honestly, what moron coded libcdio and libcddb |
| [10:18:19] | stuarta: | build libcdio. "new enough libcddb not found" |
| [10:18:51] | stuarta: | clean out libcdio so it can be rebuilt with libcddb |
| [10:19:06] | stuarta: | build libcddb. "new enough libcdio not found" |
| [10:19:31] | stuarta: | chicken n egg! |
| [10:24:59] | eharris (eharris!~eharris@99-179-7-82.lightspeed.austtx.sbcglobal.net) has joined #mythtv | |
| [10:47:05] | kormoc (kormoc!~kormoc@mythtv/developer/kormoc) has quit (Ping timeout: 252 seconds) | |
| [10:55:34] | CaCtus491 (CaCtus491!~Kent@123-243-197-152.static.tpgi.com.au) has joined #mythtv | |
| [11:05:03] | mike|2 (mike|2!~mike@c-76-115-119-121.hsd1.or.comcast.net) has quit (Remote host closed the connection) | |
| [11:05:39] | kwtm (kwtm!~kwtm1@adsl-75-61-99-82.dsl.pltn13.sbcglobal.net) has joined #mythtv | |
| [11:05:53] | mike|2 (mike|2!~mike@c-76-115-119-121.hsd1.or.comcast.net) has joined #mythtv | |
| [11:21:52] | stuarta: | third time lucky |
| [11:22:54] | ** stuarta ponders how to script install libcddb, install libcdio, uninstall libcddb, rebuild and reinstall libcddb ** | |
| [11:23:05] | kwtm: | good place to ask user questions? mythtv-users seems to be dead ... |
| [11:23:08] | stuarta: | in osx-packager |
| [11:23:10] | stuarta: | kwtm: no |
| [11:23:21] | stuarta: | ask in -users, and somebody will respond when they wake up |
| [11:29:44] | stuartm: | force build cppcheck-master now |
| [11:29:50] | stuartm: | MythBuild: force build cppcheck-master now |
| [11:29:51] | MythBuild: | build forced [ETA 8m55s] |
| [11:29:51] | MythBuild: | I'll give a shout when the build finishes |
| [11:30:46] | CaCtus491: | Is that a build from scratch, and if so, on what hardware? |
| [11:38:48] | MythBuild: | Hey! build cppcheck-master #553 is complete: Success [build successful] |
| [11:38:48] | MythBuild: | Build details are at http://code.mythtv.org/buildbot/builders/cppc . . . r/builds/553 |
| [11:39:41] | stuarta: | CaCtus491: you asking about osx? |
| [11:40:30] | CaCtus491: | whatever buildbot was just building for |
| [11:40:37] | CaCtus491: | (<10 mins seems fast!) |
| [11:40:43] | stuarta: | it's cppcheck |
| [11:40:52] | stuarta: | not a full mythtv build |
| [11:46:12] | stuartm: | it's not even a build, just running cppcheck against the code |
| [11:46:29] | stuarta: | that said, the majority of the builds complete in <10m |
| [11:46:41] | stuarta: | ccache is a wonderful thing |
| [11:47:12] | stuarta: | most of your questions can be answered by poking around on the buildbot pages |
| [11:48:01] | stuarta: | that "memory leak: cl" error gets on my nerves |
| [11:48:25] | CaCtus491: | Thanks, I've been meaning to set somethign similar up at work for ages |
| [11:48:32] | stuarta: | buildbot.net |
| [11:48:50] | stuarta: | it's pretty easy to get going |
| [11:50:32] | CaCtus491: | thanks |
| [11:50:35] | zombor (zombor!~zombor_@kohana/developer/zombor) has joined #mythtv | |
| [11:51:18] | CaCtus491: | ...I thought that I'd start to idle in here again, since the mythtv-dev list stays pretty quiet these days to hear about the latest developments :) |
| [11:51:43] | CaCtus491: | Definately keen on the new json api stuff making it's way in. |
| [11:52:18] | danielk22: | Beirdo: So most of those cppcheck warnings in 1.52 were missing includes? |
| [11:54:09] | zombor (zombor!~zombor_@kohana/developer/zombor) has quit (Remote host closed the connection) | |
| [11:59:32] | stuartm: | danielk22: I think some were also caused by the lack of a platform arg, but I didn't see the list so that's just speculation |
| [12:00:28] | stuartm: | the missing include stuff would be interesting though, it usually indicates where we're relying on indirect inclusion through another header which is not 'best practice' |
| [12:06:18] | danielk22: | stuartm: Yep. I know we do that. Practically every time I'm working with a header I see instances of that. Still we should probably fix that separately. Right now about 1 in 15 of the things I see in the report cause actual misbehavior in the program. So we probably have about 9 bugs of significance left in that list. |
| [12:07:19] | danielk22: | If we add the header stuff we'll still have that many, but have 3000 nice to haves added to the list, further obscuring those 9 nuggets... |
| [12:08:05] | stuartm: | I wasn't suggesting that we re-enable them |
| [12:10:16] | stuartm: | not even after we've fixed every other reported issue, 3000 is too many, but I might write a script to generate a new report for the include stuff so they can be dealt with separately |
| [12:13:55] | danielk22: | Yeah, that could be helpful. |
| [12:26:26] | Seeker`: | how hard would it be to create two new menu items, one of which only displays tv series, the other only |
| [12:26:33] | Seeker`: | displaying movies |
| [12:29:10] | danielk22: | MythBuild: force build cppcheck-master now |
| [12:29:10] | MythBuild: | build forced [ETA 9m04s] |
| [12:29:10] | MythBuild: | I'll give a shout when the build finishes |
| [12:38:18] | MythBuild: | Hey! build cppcheck-master #555 is complete: Success [build successful] |
| [12:38:18] | MythBuild: | Build details are at http://code.mythtv.org/buildbot/builders/cppc . . . r/builds/555 |
| [12:39:54] | MythBuild: | build #135 of master-osx-snow-leopard is complete: Failure [failed compile_2] Build details are at http://code.mythtv.org/buildbot/builders/mast . . . d/builds/135 blamelist: Daniel Kristjansson <danielk@cuymedia.net > |
| [12:53:41] | Yancho (Yancho!~mpulis@unaffiliated/yancho) has quit (Read error: Operation timed out) | |
| [13:02:52] | voyo (voyo!~voyo@213-134-188-198.home.aster.pl) has joined #mythtv | |
| [13:04:52] | davide_ (davide_!~david@mythtv/developer/gigem) has quit (Remote host closed the connection) | |
| [13:05:16] | davide_ (davide_!~david@host70.16.intrusion.com) has joined #mythtv | |
| [13:05:17] | davide_ (davide_!~david@host70.16.intrusion.com) has quit (Changing host) | |
| [13:05:17] | davide_ (davide_!~david@mythtv/developer/gigem) has joined #mythtv | |
| [13:17:14] | MythBuild: | build #137 of master-osx-snow-leopard is complete: Success [build successful] Build details are at http://code.mythtv.org/buildbot/builders/mast . . . d/builds/137 |
| [13:17:29] | jeff999 (jeff999!~jeff@124-171-111-66.dyn.iinet.net.au) has quit (Quit: Leaving) | |
| [13:19:01] | danielk22: | stuartm: Those event casts don't need to be dynamic_casts.... |
| [13:19:17] | kwtm (kwtm!~kwtm1@adsl-75-61-99-82.dsl.pltn13.sbcglobal.net) has quit (Ping timeout: 248 seconds) | |
| [13:20:03] | stuartm: | no, technically not, but we've used dynamic casts everywhere else for events so it's more about consistency |
| [13:20:46] | stuartm: | I suppose in hindsight that's lousy reasoning |
| [13:21:34] | MythBuild: | build #2934 of master-linux-64bit is complete: Failure [failed compile core] Build details are at http://code.mythtv.org/buildbot/builders/mast . . . /builds/2934 blamelist: Stuart Morgan <smorgan@mythtv.org > |
| [13:21:51] | stuartm: | I'll change them all to static casts at a later date |
| [13:22:36] | Yancho (Yancho!~mpulis@c171-244.i02-3.onvol.net) has joined #mythtv | |
| [13:22:37] | Yancho (Yancho!~mpulis@c171-244.i02-3.onvol.net) has quit (Changing host) | |
| [13:22:37] | Yancho (Yancho!~mpulis@unaffiliated/yancho) has joined #mythtv | |
| [13:22:40] | stuartm: | ugh, some lines have gone astray from that patch |
| [13:27:43] | MythBuild: | build #1701 of master-linux-ppc is complete: Failure [failed compile core] Build details are at http://code.mythtv.org/buildbot/builders/mast . . . /builds/1701 blamelist: Stuart Morgan <smorgan@mythtv.org > |
| [13:29:32] | MythBuild: | build #1720 of master-freebsd-64bit is complete: Failure [failed compile core] Build details are at http://code.mythtv.org/buildbot/builders/mast . . . /builds/1720 blamelist: Stuart Morgan <smorgan@mythtv.org > |
| [13:31:37] | MythBuild: | build #1457 of master-debian-stable-64bit is complete: Failure [failed compile core] Build details are at http://code.mythtv.org/buildbot/builders/mast . . . /builds/1457 blamelist: Stuart Morgan <smorgan@mythtv.org > |
| [13:31:58] | MythBuild: | build #2683 of master-linux-32bit is complete: Failure [failed compile core] Build details are at http://code.mythtv.org/buildbot/builders/mast . . . /builds/2683 blamelist: Stuart Morgan <smorgan@mythtv.org > |
| [13:34:15] | MythBuild: | build #2935 of master-linux-64bit is complete: Success [build successful] Build details are at http://code.mythtv.org/buildbot/builders/mast . . . /builds/2935 |
| [13:38:41] | MythBuild: | build #1721 of master-freebsd-64bit is complete: Success [build successful] Build details are at http://code.mythtv.org/buildbot/builders/mast . . . /builds/1721 |
| [13:40:05] | MythBuild: | build #139 of master-osx-snow-leopard is complete: Failure [failed compile] Build details are at http://code.mythtv.org/buildbot/builders/mast . . . d/builds/139 blamelist: Stuart Morgan <smorgan@mythtv.org > |
| [13:41:57] | MythBuild: | build #2684 of master-linux-32bit is complete: Success [build successful] Build details are at http://code.mythtv.org/buildbot/builders/mast . . . /builds/2684 |
| [13:43:53] | Dave123 (Dave123!~dave@cpe-74-74-200-106.rochester.res.rr.com) has joined #mythtv | |
| [13:45:47] | MythBuild: | build #1458 of master-debian-stable-64bit is complete: Success [build successful] Build details are at http://code.mythtv.org/buildbot/builders/mast . . . /builds/1458 |
| [13:52:28] | MythBuild: | build #140 of master-osx-snow-leopard is complete: Success [build successful] Build details are at http://code.mythtv.org/buildbot/builders/mast . . . d/builds/140 |
| [13:56:04] | stichnot (stichnot!~chatzilla@192.55.55.41) has joined #mythtv | |
| [14:01:08] | stuartm: | 1 and 7 are false positives, reporting them upstream |
| [14:05:26] | MythBuild: | build #1703 of master-linux-ppc is complete: Success [build successful] Build details are at http://code.mythtv.org/buildbot/builders/mast . . . /builds/1703 |
| [14:20:48] | zombor (zombor!~zombor_@kohana/developer/zombor) has joined #mythtv | |
| [14:40:14] | MythBuild: | build #556 of master-vista-mingw-32bit is complete: Failure [failed compile] Build details are at http://code.mythtv.org/buildbot/builders/mast . . . t/builds/556 blamelist: Stuart Morgan <smorgan@mythtv.org >, Daniel Kristjansson <danielk@cuymedia.net > |
| [14:46:20] | stuartm: | what's the mingw build doing that causes it to take so long? |
| [14:55:45] | voyo: | hi. I had some issue with LNB/Diseqc setup. I couldnt lock transporder on second dish until I set switch type to DiSEqC to 4ports type – but "problem" is – I have switch with only 2 ports. By debuging I noticed that wrong DiSEqC commands were sent (comparing to what was set by 'scan' utlity). Not sure if this is a bug, or something wrong with my LNB hardware... (however Im preety sure that before (older |
| [14:55:47] | voyo: | mythtv) it was working OK with 2port switch.) |
| [15:07:58] | stuartm: | voyo: selecting 2 port switch didn't work? |
| [15:09:10] | stuartm: | if that's the case then you should open a ticket – http://svn.mythtv.org/trac/wiki/TicketHowTo |
| [15:11:25] | voyo: | stuartm: exatcly, I tried all kind of switches (tone,voltage, etc ). only when I choosed DiSEqC with 4 ports – it worked. |
| [15:12:12] | voyo: | but also DiSEqC command sent is different than this one from 'scan' util. but works ;) |
| [15:12:35] | stuartm: | odd, a quick looks suggests 2 and 4 port switches are handled the same |
| [15:13:25] | stuartm: | you'll have to open a ticket, those who know that code aren't around and without a ticket it will just be lost/forgotten |
| [15:13:47] | stuartm: | which version of mythtv btw? |
| [15:14:06] | voyo: | ok,I will. I wanted to assure if this is not my fault, with my hardware or so |
| [15:14:22] | voyo: | currently mythtv-0.24.1 |
| [15:16:06] | stuartm: | voyo: as I say those who know that code aren't around to say whether it's a hardware issue or not, I've no experience or any knowledge of DiSEqC personally since it's not really required in the UK |
| [15:17:31] | rsiebert_ (rsiebert_!~quassel@e179131162.adsl.alicedsl.de) has joined #mythtv | |
| [15:18:36] | Jordack (Jordack!~jordack@h69-131-44-221.plmomi.dedicated.static.tds.net) has joined #mythtv | |
| [15:18:43] | kth1 (kth1!~kth@dyndsl-080-228-178-011.ewe-ip-backbone.de) has joined #mythtv | |
| [15:18:51] | voyo: | stuartm: ok. I'll test with trunk yet, and will open ticket. DiSEqC is usualy not required anywhere, its just a way to have more ;) |
| [15:20:38] | stuartm: | well in the UK there isn't more worth having, unless you have a good command of French/German/Dutch/Polish etc |
| [15:20:57] | cesman (cesman!~cesman@pdpc/supporter/professional/cesman) has quit (Quit: Konversation terminated!) | |
| [15:21:01] | rsiebert (rsiebert!~quassel@e179130255.adsl.alicedsl.de) has quit (Ping timeout: 276 seconds) | |
| [15:26:03] | voyo: | ye, probably you are right. to be honest Im watching only few channels from second dish, and rarely. but its nice hobby anyway ;) |
IRC Logs collected by
BeirdoBot.
Please use the above link to report any bugs.