| Wednesday, November 30th, 2011, 00:00 UTC | ||
| [00:00:22] | Beirdo: | oh no, slightly different error, my bad |
| [00:00:48] | Beirdo: | undefined reference to V4LRecorder::SetOption(QString const&, QString const&) |
| [00:01:05] | iamlindoro: | Beirdo: why is windows even building v4lrecorder? |
| [00:01:25] | Beirdo: | it may not be |
| [00:01:36] | kormoc (kormoc!~kormoc@mythtv/developer/kormoc) has joined #mythtv | |
| [00:02:06] | Beirdo: | but mythplayer.h includes NuppelVideoRecorder.h which includes V4LRecorder now instead of DTVRecorder |
| [00:02:49] | Beirdo: | https://github.com/MythTV/mythtv/commit/fb5c5 . . . eoRecorder.h |
| [00:03:00] | Beirdo: | that's the offending... |
| [00:03:09] | Beirdo: | oh, instead of RecorderBase, my bad |
| [00:05:25] | Beirdo: | I think if I move the function itself into the .cpp, it might just fix this mess. |
| [00:07:23] | Beirdo: | if it doesn't, we can revert that and let the author fix it later when he has time ;) |
| [00:08:36] | Beirdo: | revert that meaning the potential fix, not the change in that commit message |
| [00:22:52] | zombor (zombor!~zombor_@kohana/developer/zombor) has quit (Remote host closed the connection) | |
| [00:47:46] | pheld (pheld!~heldal@cl-5.osl-01.no.sixxs.net) has quit (Quit: Leaving.) | |
| [01:03:41] | zombor (zombor!~zombor_@kohana/developer/zombor) has joined #mythtv | |
| [01:33:07] | natanojl (natanojl!~jonatan@c83-252-237-63.bredband.comhem.se) has quit (Ping timeout: 244 seconds) | |
| [01:49:13] | jpabq_ (jpabq_!~jpabq@mythtv/developer/jpabq) has quit (Quit: jpabq_) | |
| [01:59:31] | MythBuild: | build #330 of master-vista-mingw-32bit is complete: Success [build successful] Build details are at http://code.mythtv.org/buildbot/builders/mast . . . t/builds/330 |
| [02:05:52] | davide (davide!~david@host70.16.intrusion.com) has quit (Remote host closed the connection) | |
| [02:06:14] | davide (davide!~david@host70.16.intrusion.com) has joined #mythtv | |
| [02:43:35] | zombor (zombor!~zombor_@kohana/developer/zombor) has quit (Remote host closed the connection) | |
| [02:51:27] | justinh (justinh!~justin@cpc10-salf5-2-0-cust50.10-2.cable.virginmedia.com) has quit (Ping timeout: 248 seconds) | |
| [03:04:24] | zombor (zombor!~zombor_@kohana/developer/zombor) has joined #mythtv | |
| [03:15:40] | Beirdo^2 (Beirdo^2!~Beirdo@mythtv/developer/beirdo) has joined #mythtv | |
| [03:31:18] | Beirdo^2 (Beirdo^2!~Beirdo@mythtv/developer/beirdo) has quit (Quit: Bye) | |
| [03:37:12] | justinh (justinh!~justin@cpc10-salf5-2-0-cust50.10-2.cable.virginmedia.com) has joined #mythtv | |
| [03:55:36] | moodboom (moodboom!~moodboom@pdpc/supporter/active/moodboom) has quit (Remote host closed the connection) | |
| [03:58:10] | zombor (zombor!~zombor_@kohana/developer/zombor) has quit (Remote host closed the connection) | |
| [04:23:12] | jflatt (jflatt!46adfdf8@gateway/web/freenode/ip.70.173.253.248) has joined #mythtv | |
| [04:24:39] | jflatt: | wagnerrp MythTV/packaging commit ed4b9a2a2e Ebuild version bump is messed up |
| [04:26:05] | ** iamlindoro presumes there is a much more technical explanation forthcoming ** | |
| [04:26:51] | jflatt: | Just verifications failed, extraneous ebuilds in the Manifest, etc |
| [04:27:30] | iamlindoro: | a pastebin would probably be more prodcutive |
| [04:28:44] | wagnerrp: | i know, i saw the ebuild |
| [04:28:55] | wagnerrp: | erm... github comment |
| [04:29:33] | wagnerrp: | i think ebuild got confused with several additional tarballs i had downloaded |
| [04:29:40] | wagnerrp: | rather than just ones linked to existing ebuilds |
| [04:30:18] | jflatt: | ahh I see the github comments now, sorry for the bumrush |
| [04:31:55] | jflatt (jflatt!46adfdf8@gateway/web/freenode/ip.70.173.253.248) has quit () | |
| [05:01:35] | MaverickTech (MaverickTech!~MaverickT@111.86.233.220.static.exetel.com.au) has quit (Ping timeout: 244 seconds) | |
| [05:01:54] | dlblog (dlblog!~dlblog@c-76-127-227-175.hsd1.ma.comcast.net) has quit (Read error: Connection reset by peer) | |
| [05:06:13] | dlblog (dlblog!~dlblog@c-76-127-227-175.hsd1.ma.comcast.net) has joined #mythtv | |
| [05:12:42] | MaverickTech (MaverickTech!~MaverickT@111.86.233.220.static.exetel.com.au) has joined #mythtv | |
| [05:24:30] | kormoc (kormoc!~kormoc@mythtv/developer/kormoc) has quit (Quit: kormoc) | |
| [05:49:50] | jmartens (jmartens!~jmartens@s5597ca60.adsl.wanadoo.nl) has joined #mythtv | |
| [05:54:58] | NightMonkey (NightMonkey!~NightMonk@pdpc/supporter/professional/nightmonkey) has quit (Quit: Body blow! Body blow!) | |
| [06:00:14] | jeff999 (jeff999!~jeff@58-6-93-73.dyn.iinet.net.au) has joined #mythtv | |
| [06:14:54] | jmartens (jmartens!~jmartens@s5597ca60.adsl.wanadoo.nl) has quit (Quit: Leaving.) | |
| [06:32:54] | jmartens (jmartens!~jmartens@109.232.42.33) has joined #mythtv | |
| [07:04:19] | JEDIDIAH__ (JEDIDIAH__!~jedi@cpe-76-185-75-39.tx.res.rr.com) has quit (Read error: Operation timed out) | |
| [07:19:36] | JEDIDIAH__ (JEDIDIAH__!~jedi@cpe-76-185-75-39.tx.res.rr.com) has joined #mythtv | |
| [07:37:43] | dserban_ (dserban_!~dserban@S0106001346beb5f3.ok.shawcable.net) has joined #mythtv | |
| [07:40:37] | nutron (nutron!~dserban@S0106001346beb5f3.ok.shawcable.net) has quit (Ping timeout: 240 seconds) | |
| [07:57:44] | dekarl: | danielk22: looks like you can end up in a loop if subclasses of MPEGDescriptor dont overload toString, e.g. http://paste.ubuntu.com/754562/ likely caused by missing toString here https://github.com/MythTV/mythtv/blob/master/ . . . ptors.h#L162 |
| [08:26:37] | pheld (pheld!~heldal@cl-5.osl-01.no.sixxs.net) has joined #mythtv | |
| [09:14:34] | jarle (jarle!~jarle@70.84-234-133.customer.lyse.net) has joined #mythtv | |
| [10:03:04] | jmartens (jmartens!~jmartens@109.232.42.33) has quit (Read error: Connection reset by peer) | |
| [10:03:14] | jmartens (jmartens!~jmartens@109.232.42.33) has joined #mythtv | |
| [10:06:59] | MaverickTech (MaverickTech!~MaverickT@111.86.233.220.static.exetel.com.au) has quit (Ping timeout: 260 seconds) | |
| [10:07:34] | jmartens (jmartens!~jmartens@109.232.42.33) has quit (Ping timeout: 245 seconds) | |
| [10:11:05] | jmartens (jmartens!~jmartens@109.232.42.33) has joined #mythtv | |
| [10:19:45] | jarle (jarle!~jarle@70.84-234-133.customer.lyse.net) has quit (Remote host closed the connection) | |
| [10:21:43] | jarle (jarle!~jarle@70.84-234-133.customer.lyse.net) has joined #mythtv | |
| [10:26:08] | MaverickTech (MaverickTech!~MaverickT@111.86.233.220.static.exetel.com.au) has joined #mythtv | |
| [10:32:40] | jarle (jarle!~jarle@70.84-234-133.customer.lyse.net) has quit (Remote host closed the connection) | |
| [10:43:25] | MaverickTech (MaverickTech!~MaverickT@111.86.233.220.static.exetel.com.au) has quit (Ping timeout: 240 seconds) | |
| [10:44:39] | willcooke (willcooke!~will@canonical/willcooke) has joined #mythtv | |
| [10:54:38] | MaverickTech (MaverickTech!~MaverickT@111.86.233.220.static.exetel.com.au) has joined #mythtv | |
| [11:05:02] | Guest60755 (Guest60755!~mike@c-76-115-119-121.hsd1.or.comcast.net) has quit (Remote host closed the connection) | |
| [11:05:57] | mike|2 (mike|2!~mike@c-76-115-119-121.hsd1.or.comcast.net) has joined #mythtv | |
| [11:42:02] | tris (tris!tristan@2001:1868:a00a::4) has quit (Quit: Leaving) | |
| [11:49:39] | tris (tris!tristan@2001:1868:a00a::4) has joined #mythtv | |
| [11:51:05] | zombor (zombor!~zombor_@kohana/developer/zombor) has joined #mythtv | |
| [12:04:03] | zombor (zombor!~zombor_@kohana/developer/zombor) has quit (Remote host closed the connection) | |
| [12:05:15] | jeff999 (jeff999!~jeff@58-6-93-73.dyn.iinet.net.au) has quit (Ping timeout: 244 seconds) | |
| [12:42:46] | jarle (jarle!~jarle@70.84-234-133.customer.lyse.net) has joined #mythtv | |
| [12:45:03] | jeff999 (jeff999!~jeff@58-6-93-73.dyn.iinet.net.au) has joined #mythtv | |
| [12:57:36] | sraue (sraue!~stephan@xbmc/staff/sraue) has quit (Quit: Ex-Chat) | |
| [13:04:30] | davide (davide!~david@host70.16.intrusion.com) has quit (Remote host closed the connection) | |
| [13:04:50] | davide (davide!~david@host70.16.intrusion.com) has joined #mythtv | |
| [13:05:51] | willcooke (willcooke!~will@canonical/willcooke) has quit (Ping timeout: 248 seconds) | |
| [13:10:03] | sraue (sraue!~stephan@xbmc/staff/sraue) has joined #mythtv | |
| [13:29:25] | jeff999 (jeff999!~jeff@58-6-93-73.dyn.iinet.net.au) has quit (Ping timeout: 240 seconds) | |
| [13:40:05] | willcooke (willcooke!~will@nat/canonical/x-tibhavduheahtplg) has joined #mythtv | |
| [13:40:05] | willcooke (willcooke!~will@canonical/willcooke) has joined #mythtv | |
| [13:40:05] | willcooke (willcooke!~will@nat/canonical/x-tibhavduheahtplg) has quit (Changing host) | |
| [14:04:17] | j-rod|afk is now known as j-rod | |
| [14:09:35] | danielk22: | dekarl: thanks, I |
| [14:09:52] | danielk22: | ll look at that bt shortly. |
| [14:22:07] | zombor (zombor!~zombor_@kohana/developer/zombor) has joined #mythtv | |
| [14:31:18] | danielk22: | Beirdo: Why was windows complaining about the NVR SetOption ? It looks like it was valid code.. |
| [14:56:27] | danielk22: | dekarl: I've fixed the immediate cause, but I'll see if I can't rework the macro to catch this type of error at compile time.. |
| [15:38:48] | guest___ (guest___!bcc2232f@gateway/web/freenode/ip.188.194.35.47) has joined #mythtv | |
| [15:39:36] | guest___: | Hi, is there already a feature freeze date? |
| [15:40:15] | wagnerrp: | no |
| [15:45:12] | guest___: | hmm....when will it be (days or months)? |
| [15:46:28] | guest___: | this year or next year ;) |
| [15:46:38] | wagnerrp: | the feature freeze or the release? |
| [15:46:57] | guest___: | the feature freeze |
| [15:47:09] | wagnerrp: | potentially this year, probably not |
| [15:47:29] | wagnerrp: | there are generally several weeks notice to get things cleaned up and in before a freeze |
| [15:47:51] | wagnerrp: | and then 4–6 weeks of freeze before release |
| [15:49:04] | guest___: | ok thx |
| [15:49:59] | guest___ (guest___!bcc2232f@gateway/web/freenode/ip.188.194.35.47) has quit () | |
| [16:00:47] | willcooke (willcooke!~will@canonical/willcooke) has joined #mythtv | |
| [16:01:35] | brfransen (brfransen!~brfransen@64.179.142.146) has quit (Quit: brfransen) | |
| [16:02:28] | saintdev (saintdev!~saint@c-24-9-201-236.hsd1.co.comcast.net) has joined #mythtv | |
| [16:02:28] | saintdev (saintdev!~saint@c-24-9-201-236.hsd1.co.comcast.net) has quit (Changing host) | |
| [16:02:28] | saintdev (saintdev!~saint@unaffiliated/saintdev) has joined #mythtv | |
| [16:09:37] | willcooke (willcooke!~will@canonical/willcooke) has quit (Ping timeout: 252 seconds) | |
| [16:18:25] | jmartens (jmartens!~jmartens@109.232.42.33) has quit (Ping timeout: 252 seconds) | |
| [16:19:11] | jpabq_ (jpabq_!~jpabq@mythtv/developer/jpabq) has joined #mythtv | |
| [16:31:15] | zCougar (zCougar!~cougar@2001:67c:32c:600:20dd:9004:9d9c:4f54) has quit (Ping timeout: 244 seconds) | |
| [16:32:44] | aloril (aloril!~aloril@dsl-tkubrasgw3-fe7ef900-153.dhcp.inet.fi) has quit (Ping timeout: 252 seconds) | |
| [16:42:49] | yang (yang!yang@freenode/sponsor/cacert.assurer.yang) has joined #mythtv | |
| [16:42:53] | yang: | Hello ! |
| [16:43:02] | yang: | Excuse me my newbie question |
| [16:43:08] | zCougar (zCougar!~cougar@2001:67c:32c:600:7581:6725:9fd3:ddb1) has joined #mythtv | |
| [16:43:09] | yang: | another thing I want to figure out is, am I able to have my own IPTV solution, like can I get IPTV feeds from some public service (for free) and use my internet bandwidth to do it |
| [16:43:15] | wagnerrp: | it can be answered in #mythtv-users |
| [16:43:20] | wagnerrp: | this is the development channel |
| [16:43:20] | yang: | ok |
| [16:45:51] | aloril (aloril!~aloril@dsl-tkubrasgw3-fe7ef900-153.dhcp.inet.fi) has joined #mythtv | |
| [16:55:44] | jmartens (jmartens!~jmartens@s5597ca60.adsl.wanadoo.nl) has joined #mythtv | |
| [17:00:50] | zombor_ (zombor_!~zombor_@kohana/developer/zombor) has joined #mythtv | |
| [17:02:27] | reynaldo (reynaldo!~rverdejo@pc-59-218-86-200.cm.vtr.net) has joined #mythtv | |
| [17:03:11] | zombor (zombor!~zombor_@kohana/developer/zombor) has quit (Ping timeout: 248 seconds) | |
| [17:14:19] | willcooke (willcooke!~will@canonical/willcooke) has joined #mythtv | |
| [17:17:24] | willcooke (willcooke!~will@canonical/willcooke) has quit (Client Quit) | |
| [17:19:04] | willcooke (willcooke!~will@canonical/willcooke) has joined #mythtv | |
| [17:32:08] | brfransen (brfransen!~brfransen@64.179.142.146) has joined #mythtv | |
| [17:38:12] | danielk22: | hmm, I committed the ceton code with Ronald as the author, but my name as the committer doesn't appear in the commit message, what do I need to do to get that to appear next time? |
| [17:39:24] | iamlindoro: | I think it might just be a limitation of our post-commit hook-- think Github shows the commit/author attribution properly |
| [17:41:06] | wagnerrp: | it was an intentional limitation, the alternative being to maintain a lookup table between github emails and trac accounts |
| [17:42:26] | danielk22: | wagnerrp: is it possible to just show either the github email or trac account? i.e. so if someone sees problems due to the commit they know to contact me.. |
| [17:44:15] | wagnerrp: | i originally set it up that way, but was decided to change it to how it is now |
| [17:44:22] | wagnerrp: | i dont recall the details of the discussion off hand |
| [17:44:29] | wagnerrp: | it was way back in february or so |
| [17:48:32] | wagnerrp: | there wouldnt be much to putting it back |
| [17:48:40] | wagnerrp: | i think i left the original lines just commented out |
| [17:58:17] | yang (yang!yang@freenode/sponsor/cacert.assurer.yang) has left #mythtv () | |
| [17:59:58] | kth (kth!~kth@unaffiliated/kth) has joined #mythtv | |
| [18:03:56] | zombor_ (zombor_!~zombor_@kohana/developer/zombor) has quit (Remote host closed the connection) | |
| [18:04:48] | davide (davide!~david@host70.16.intrusion.com) has quit (Remote host closed the connection) | |
| [18:05:13] | davide (davide!~david@host70.16.intrusion.com) has joined #mythtv | |
| [18:11:37] | natanojl (natanojl!~jonatan@c83-252-237-63.bredband.comhem.se) has joined #mythtv | |
| [18:21:49] | kormoc (kormoc!~kormoc@mythtv/developer/kormoc) has joined #mythtv | |
| [18:25:10] | iamlindoro: | danielk22: So there are no ceton-provided headers necessary for the recorder? Just wondering as I wonder whether the device would work under Mac OS X – if not directly, then at least when bridged and shared from another machine |
| [18:28:42] | MythBuild: | build #333 of master-vista-mingw-32bit is complete: Failure [failed compile] Build details are at http://code.mythtv.org/buildbot/builders/mast . . . t/builds/333 blamelist: Ronald Frazier <ron@ronfrazier.net > |
| [18:29:14] | willcooke (willcooke!~will@canonical/willcooke) has quit (Quit: We will learn more of his wisdom later) | |
| [18:41:46] | danielk22: | iamlindoro: No ceton headers required, I believe it would work on OSX when bridged from another machine using "RTP" as the card number. |
| [18:43:04] | danielk22: | From the MythBuild error it looks like it won't work on windows though.. |
| [18:43:46] | wagnerrp: | does the backend work at all on the windows build? |
| [18:44:34] | wagnerrp: | ive seen claims that you can use a HDHomeRun with it, but i expected some of the file access routines wouldnt be compatible |
| [18:52:07] | danielk22: | wagnerrp: The HDHomeRun should work in theory, but I don't know if it does in practice. I did test it on OSX years ago. |
| [19:00:58] | danielk22: | Does anyone know if it possible to create an extra fork of a repo on github without creating a new github account? Just hitting the fork button when you already have a fork brings you to your existing fork.. |
| [19:02:30] | kth (kth!~kth@unaffiliated/kth) has quit (Quit: Leaving.) | |
| [19:03:39] | wagnerrp: | awesome! hardcore forking action! |
| [19:05:26] | wagnerrp: | you could do it manually, but i dont believe it lets you create two forks off the same repo on a single account |
| [19:05:28] | rsiebert (rsiebert!~quassel@g226060014.adsl.alicedsl.de) has joined #mythtv | |
| [19:05:38] | wagnerrp: | it doesnt let you fork a repo in your own account either |
| [19:08:09] | kth (kth!~kth@unaffiliated/kth) has joined #mythtv | |
| [19:08:31] | rsiebert_ (rsiebert_!~quassel@g226061017.adsl.alicedsl.de) has quit (Ping timeout: 248 seconds) | |
| [19:09:59] | wagnerrp: | danielk22: any reason you couldnt simply have two separate branches on your one fork? |
| [19:12:43] | Beirdo: | danielk22: Not 100% sure, but it liked it when I moved the actual implementation to the CPP file rather than the h |
| [19:16:09] | zombor (zombor!~zombor_@kohana/developer/zombor) has joined #mythtv | |
| [19:26:07] | danielk22: | wagnerrp: The existing for is 'retired' but I don't want to nuke it. I had updated it's trunk with "safe" changeset only when we were having backend stability problems with the logging last summer. Now I want to do some experimentation, so I want to make another fork but github doesn't let me using their web interface. |
| [19:26:39] | wagnerrp: | yeah, you can clone one on the command line, and push it back to github |
| [19:26:47] | danielk22: | wagnerrp: My hope was that there is some command line invcation that I can use to create a 2nd fork. |
| [19:26:50] | wagnerrp: | but it wont have that fancy linkage in the github webpage |
| [19:27:32] | danielk22: | so do a clone and push it to where? |
| [19:27:53] | wagnerrp: | http://alt.adrianshort.co.uk/blog/2011/11/08/ . . . github-repo/ |
| [19:28:06] | danielk22: | wagnerrp: thanks! |
| [19:32:11] | Beirdo: | you don't need multiple forks, you can just pull another branch to your fork, and have two branches on your fork. |
| [19:33:20] | Beirdo: | but whatever works, I guess |
| [19:33:47] | neufeld (neufeld!~user@ottawa-hs-206-191-28-140.s-ip.magma.ca) has joined #mythtv | |
| [19:36:19] | neufeld: | So, seem to have found a mythtranscode bug in fixes/0.24. Somehow there are duplicate pointers in vFrame during the MPEG2fixup destructor, so objects get deleted more than once, corrupting the malloc space. I've got a test case and compiler up if anybody wants to suggest a patch. mpeg2fix.cpp:300 |
| [19:36:29] | Chutt: | guess i should renew mythtv.com =) |
| [19:36:52] | neufeld: | Cycle time on the test case is about 7 minutes executing. |
| [19:37:08] | neufeld: | I'm going to try a simple patch, skipping deletes on pointers previously seen. |
| [19:38:22] | Beirdo: | Chutt: yeah, that might be a good idea :) |
| [19:39:08] | Beirdo: | neufeld: that code has been redone already in master, but no reason we can't put a bandaid on it in 0.24 fixes |
| [19:39:46] | Beirdo: | we'd need to make sure that the new code has that covered too, of course :) |
| [19:40:10] | neufeld: | Beirdo: OK, let me pull the master and see if it fixes things for me. If so, I'll compile that for my mythtv box and consider it good enough. |
| [19:40:13] | Jordack (Jordack!~jordack@h69-131-44-221.plmomi.dedicated.static.tds.net) has joined #mythtv | |
| [19:40:37] | Beirdo: | well, if you have a fix, we can put it in anyways for the people who aren't ready to upgrade |
| [19:41:14] | ** Beirdo slaps the windows slave. ** | |
| [19:41:22] | neufeld: | We'll see, I'm running my change now. At worst, all it should do is introduce a memory leak. I'm running it under valgrind, so I'll know if that's the case. |
| [19:42:14] | neufeld: | If I compiled mythtranscode off master, could I drop it into a fixes/0.24 setup and expect it to run? No database mismatches? |
| [19:43:39] | Beirdo: | oooh. guru meditation state |
| [19:43:46] | danielk22: | Beirdo: Sure it is possible with branches, but it would just be more complicated and hence I'd not have as much confidence in it as a clean fork. |
| [19:43:54] | Beirdo: | neufeld: not likely |
| [19:44:34] | Beirdo: | danielk22: more complicated? Not so sure about that, but if it fits your needs better and gives you more confidence, go for it :) |
| [19:45:12] | neufeld: | Beirdo: OK, thanks. Well, let's see if my bandaid fixes it. If so, I'll use that. |
| [19:47:50] | kth (kth!~kth@unaffiliated/kth) has quit (Read error: Connection reset by peer) | |
| [19:47:53] | neufeld: | Hrmph. So, no more malloc failure, but the program ABENDs anyway with "expectedPTS != expectedDTS+ptsIncrement". |
| [19:48:22] | Beirdo: | it could be a particularly bad input file |
| [19:49:32] | neufeld: | Ought not to be. It renders fine around that point (the first cutpoint in the broadcast). I'll move the cut point by a keyframe and see what happens. |
| [19:50:13] | Beirdo: | Yeah, that code was (and is) fairly fragile |
| [19:52:09] | neufeld: | And the cut wasn't what I thought it was, it was two cuts with something between them. Chances are tweaking the cutpoint fixed the whole thing. I'll know in 7 minutes. |
| [19:53:24] | Beirdo: | OK, windows buildslave VM is back up |
| [19:54:02] | Beirdo: | you should see the massive spew VirtualBox gives when it's shutting down |
| [20:00:33] | neufeld: | Yeah, that fixed it. So, fragile code. I somehow created two side-by-side cut regions instead of merging them as usually happens, and broke the transcoder. Thank you for the help. |
| [20:02:17] | Beirdo: | No problem |
| [20:03:38] | neufeld (neufeld!~user@ottawa-hs-206-191-28-140.s-ip.magma.ca) has quit (Quit: ERC Version 5.3 (IRC client for Emacs)) | |
| [21:04:35] | danielk22: | iamlindoro: I was just updating the utc patch... and it got me to thinking about the services API.. we should probably send any times on that interface out in UTC so clients don't depend on us converting those to the local time of the server once we convert mythbackend to use UTC internally.. |
| [21:07:20] | iamlindoro: | danielk22: I'm of two minds about it-- I see the point from a technical perspective, though it's nice to simply translate the dates I'm getting from it to strings ;) |
| [21:07:29] | iamlindoro: | All told, though, I think you're right |
| [21:07:47] | iamlindoro: | Could probably be fixed at a single point in the serializers? |
| [21:09:24] | danielk22: | iamlindoro: Except for a few places we store times as localtime and tagged as localtime so just adding a .toUTC() at the right point in the serializers should fix it. |
| [21:09:50] | danielk22: | iamlindoro: I hear you on the convenience, but that's what got us in trouble in the first place :) |
| [21:10:32] | danielk22: | + it would be nifty to open up a tablet in CA and see times on my NY box localized. :) |
| [21:10:41] | iamlindoro: | danielk22: Yeah, I agree with you, it's the right way to go |
| [21:12:04] | kormoc (kormoc!~kormoc@mythtv/developer/kormoc) has quit (Quit: kormoc) | |
| [21:17:43] | iamlindoro: | and less disruptive over the next couple releases too |
| [21:24:01] | danielk22: | yeah, it's best to make changes to the services protocol now while it's still tagged as unstable and we don't have a lot of external parties depending on it yet. |
| [21:24:23] | iamlindoro: | danielk22: untested but compiles: http://pastebin.com/vgEBS8Gr |
| [21:38:30] | dekarl: | danielk22: thanks for the fix |
| [21:41:38] | dekarl: | iamlindoro: might be a good time to consider adding explicit Z at the end for parsers to know its utc |
| [21:42:21] | iamlindoro: | dekarl: it's Qt's rendering of ISODate, I'd prefer to just use their format rather than building our own |
| [21:43:34] | iamlindoro: | but I dunno, I guess if it's required for there to be a Z I could be convinced-- it just feels so much cleaner to just do Qt::ISODate and let Qt handle it |
| [21:43:35] | dekarl: | I see, just looking at http://www.w3.org/TR/2001/REC-xmlschema-2-20010502/#dateTime and though it would be nice to be explicit |
| [21:44:51] | iamlindoro: | heh, guess I should put a break: at the end of those cases... |
| [21:44:57] | danielk22: | iamlindoro: dekarl: "Z" is the official way to indicate Zulu time, but I don't know the level of support for that in Qt. |
| [21:45:30] | iamlindoro: | danielk22: dekarl: Let's see if Qt has a constant for it |
| [21:45:51] | iamlindoro: | other than the missing break; and telling it to use Qt::ISODate, the above patch works |
| [21:46:21] | iamlindoro: | http://stackoverflow.com/questions/3350124/pr . . . ts-qdatetime |
| [21:46:48] | iamlindoro: | (and https://bugreports.qt.nokia.com//browse/QTBUG-9698) |
| [21:47:18] | iamlindoro: | Looks like fixed in most current Qt? |
| [21:47:57] | iamlindoro: | Hmm, actually, maybe they haven't fixed it |
| [21:48:39] | iamlindoro: | Nope, sorry, last comment in that bug suggests they pulled in a merge request to fix the missing Z in March |
| [21:49:00] | iamlindoro: | So my inclination is to say we keep on using Qt::ISODate and the format will fix itself in most distros over the next few months |
| [21:49:31] | danielk22: | iamlindoro: Heh, less work, sounds good to me. :) |
| [21:54:00] | Anduin (Anduin!~awithers@pdpc/supporter/professional/anduin) has quit (Ping timeout: 244 seconds) | |
| [21:54:01] | dekarl: | Do I read the ticket right, the fix went into the branch for QT 4.7? And it'lll likely come out as 4.7.4 in 3–4 months? If the datetimes are implicit UTC until then it should be no issue. Just document what it is :D |
| [21:54:49] | Anduin (Anduin!~awithers@pdpc/supporter/professional/anduin) has joined #mythtv | |
| [21:55:26] | iamlindoro: | dekarl: exactly |
| [21:59:40] | Jordack (Jordack!~jordack@h69-131-44-221.plmomi.dedicated.static.tds.net) has quit (Quit: I'll start being nicer if you start being smarter.) | |
| [21:59:59] | sphery: | danielk22: re: the commit, most of us have been trying to remember to use -s/--signoff to get an explicit Signed-off-by line added to the commit for exactly the reason you mention (so people will know who to contact) |
| [22:01:03] | sphery: | so it will look something like http://www.gossamer-threads.com/lists/mythtv/ . . . 96432#496432 in the email |
| [22:02:36] | davide: | Beirdo: i just noticed a misfeature/regression in master caused by the logging changes. for mythbackend --printsched/--testsched, i don't the output from Scheduler::PrintList/Rec() should have the time/level prepended, at least when sent to the console. it's not really log information in those use cases. |
| [22:03:11] | Beirdo: | hmm, OK, fair enough |
| [22:03:16] | Beirdo: | we should fix that |
| [22:03:52] | Beirdo: | It is being logged though |
| [22:04:00] | Beirdo: | rather than being console output... |
| [22:04:08] | Beirdo: | is anything processing that output? |
| [22:04:40] | Beirdo: | we can certainly change it back to console output if there's a good reason for it :) |
| [22:04:59] | iamlindoro: | dblain: Can you see the above conversation re: Services API datetimes converting to UTC? I agree with Daniel and have a working patch ( http://pastebin.com/iWst7CnQ ) but would like your approval |
| [22:05:27] | davide: | i don't know of anything that parses it, but i wouldn't be surprised if some user does. |
| [22:05:32] | dekarl: | stuartm: btw, I'll be looking into extending the xmltv configuration api with the string types soonish (I'd like to have automatic setup ready on our side pre christmas) with types for "filename/directory/cachefile/cachedirectory" so some sensible defaults can be set. Basically just minimal extensions to get ready for this http://wiki.xmltv.org/index.php/NoLineupProposal#interactions |
| [22:05:32] | Beirdo: | K. |
| [22:05:50] | Beirdo: | well, if ya feel strongly about it, that's good enough of a reason anyways. :) |
| [22:06:22] | Beirdo: | although, it might be nice to have it in the logfiles... Whichever, I guess |
| [22:06:52] | danielk22: | Beirdo: davide: "LOG(VB_STDIO, logLevel," functions like "cout<<" but synced with the logging. followed by "LOG(VB_STDIO|VB_FLUSH, logLevel," to function like a cout<<flush; |
| [22:07:02] | sphery: | davide: would be nice to get that info returned as a proper machine-readable document using Services API, then just have the --printsched (maybe even moved to mythutil) request it from services then format it for console/user--or, even better, just have python/perl scripts do the formatting for users (since users can change those) |
| [22:07:14] | Beirdo: | ah yes, that's right |
| [22:07:18] | davide: | yeah, that's why i said at least for the console. sending as regular log when writing to a file is probably ok. |
| [22:07:22] | sphery: | (not requesting you do it, but if you like the idea, I can put it on my todo) |
| [22:07:23] | Beirdo: | thatnks for the reminder, davide :) |
| [22:08:18] | Beirdo: | so VB_STDIO | VB_WHATEVER should log it, and put it on the console? |
| [22:08:34] | davide: | sphery: that'd be ok for --printsched, but not for --testsched. |
| [22:08:48] | iamlindoro: | Abbreviated program output from the services API in UTC in case anyone cares to check it: http://pastebin.com/MeLrpJGa |
| [22:09:16] | sphery: | cool, I'll add it to my list |
| [22:10:37] | danielk22: | VB_STDIO sends it to the console. It's not intended to be used with anything other than perhaps VB_FLUSH.. It won't add the time, it's just a way to output to the console and never get a log message crammed into the middle of the text.. |
| [22:11:13] | dekarl: | danielk22: if you got use for DVB Dynamic PMT samples I can put an extract of such a recording somewhere http://paste.ubuntu.com/755436/ |
| [22:11:29] | Beirdo: | danielk22: OK, but it will still go to the logfile? |
| [22:12:48] | danielk22: | Beirdo: nope |
| [22:13:14] | Beirdo: | lame |
| [22:13:32] | Beirdo: | fair enough. There are some times we'd want taht |
| [22:14:10] | danielk22: | Beirdo: normally you wouldn't want something like a "00.0% complete" "00.1% complete", etc in the log :) |
| [22:14:19] | Beirdo: | but I'd prefer to see anything spewed to the console to actually be logged. Makes for much easier remote debugging if we don't have to tell lusers to capture the output as well as the logs |
| [22:14:41] | Beirdo: | danielk22: true, but for something like the printsched... |
| [22:16:12] | Beirdo: | and I would want periodic % updates in the log, just not every 0.01% or something fun like that :) Every 5–10% maybe :) |
| [22:16:38] | Beirdo: | there is a case for it, for sure. |
| [22:16:46] | davide: | i agree with Beirdo on this particular logging. when debuggin user issues, i usally like to have them run --testsched, but in some cases, i really need them to run with the appropriate -v schedule on their master backend. in those cases, the PrintRec() stuff needs to go somewhere. |
| [22:18:38] | danielk22: | davide: Beirdo: then we probably need to extend the logging.. right now it is limited to 2000 characters per log message, so we can't just put the message on single log line with a as the first char. |
| [22:19:43] | Beirdo: | when do you need more than 2k on a single line? I'd venture to say very rarely, and if so, maybe you should split that message up a bit |
| [22:19:55] | Beirdo: | that's 2 full screens on a 80x24 terminal |
| [22:19:57] | dekarl: | iamlindoro: I don't spot a difference to local time ;) But noticed the Synopsis being in the main element (the wiki examples have that, too) and Stars being 0, is Fringe really rated so bad? Or is it not distinguishing between not rated and rated really bad. |
| [22:20:29] | iamlindoro: | dekarl: If there's no rating provided, which generally only exists for movies in Schedules Direct, it's 0 |
| [22:20:32] | Beirdo: | err, just over one screen (can't multiply) |
| [22:20:58] | jpabq__ (jpabq__!~jpabq@mythtv/developer/jpabq) has joined #mythtv | |
| [22:21:17] | Beirdo: | I guess it could get bumped to 4k |
| [22:21:33] | jpabq_ (jpabq_!~jpabq@mythtv/developer/jpabq) has quit (Ping timeout: 244 seconds) | |
| [22:22:17] | davide: | Beirdo: i think danielk22 means to accumlate the entire output into a single string with embedded \n's and log the whole thing as one "line" with a leading \n. |
| [22:22:36] | Beirdo: | but my question stands: why? |
| [22:22:51] | Beirdo: | that breaks any and all log processor software you will find out there |
| [22:23:14] | iamlindoro: | dekarl: For some reason the XML serializer renders anything in <Description> tags as plain text-- dblain would have to explain that-- but it's rendered properly in JSON, which is what I use, so I haven't pursued it |
| [22:23:17] | Beirdo: | 2k is a LOT of text for a single message :) |
| [22:23:31] | Beirdo: | I just don't get it, but I'm probably missing something |
| [22:24:00] | danielk22: | Beirdo: As a hack to get rid of the time-stamping. We're thinking of the logging for debugging not for log processing software. |
| [22:24:16] | davide: | i hadn't gotten to that! :) i was going to point out the the kernel recently (i think) added the concept of a continuation message to printk(). with that, the first call gets the log prefix stuff printed, but none of the continuation lines do. |
| [22:25:12] | Beirdo: | maybe not for you, but for me, the timestamping is a crucial part of debugging a multi-threaded application |
| [22:25:29] | Beirdo: | but yeah, I guess. |
| [22:25:44] | j-rod is now known as j-rod|afk | |
| [22:26:20] | Beirdo: | and if you really want to strip it off, it's easy to do with tr... if the format of every line is constant. which it isn't if you embed \n |
| [22:26:29] | Beirdo: | tr or awk... or sed... |
| [22:28:05] | Beirdo: | pretty sure people have (or might want to) use logwatch for things like notifying them on failure of a recording |
| [22:29:16] | danielk22: | Beirdo: sure.. I'm not saying that isn't useful! |
| [22:29:57] | danielk22: | Ideally we'll send the printsched output to the console without timestamps on each line and send it to the log normally. |
| [22:30:05] | dekarl: | iamlindoro: ok I see. So the xml isnt really used, yet. The ProgramFlags (and other bitfields) split into explicit attributes or elements would be useful for simple XSLT renderings. Use case would be stuff like "print a list of all HDTV movies with just wget and a stylesheet" |
| [22:30:17] | Beirdo: | yeah, that would be the ideal :) |
| [22:30:52] | iamlindoro: | dekarl: XML can be used by anyone who wants to use it-- JSON is just way easier to use |
| [22:30:57] | danielk22: | I was really just saying the hack of putting multiple lines in one LOG() line wouldn't work :P |
| [22:30:57] | dekarl: | I can write some stylesheets as example if you think its useful |
| [22:31:19] | iamlindoro: | dekarl: I really don't have any interest in breaking out the bit masks, personally |
| [22:31:25] | dekarl: | ok |
| [22:31:27] | Beirdo: | ahh :) Heh, yeah. Gotcha. I need more coffee |
| [22:32:06] | iamlindoro: | dekarl: I struggled with how to render them when I added them, and decided that ultimately it's designed for programmatic use, not end-user interaction, so the bit mask made more sense to me |
| [22:32:41] | iamlindoro: | dekarl: Plus, there were already existing bitmask/int fields so it was consistent with what the original author added |
| [22:32:54] | Beirdo: | jya: you created a "utc" branch? ;) |
| [22:33:55] | danielk22: | Beirdo: you really need that coffee :] |
| [22:34:48] | Beirdo: | hahah, I missed it when you branched it earlier, my bad. |
| [22:34:57] | Beirdo: | heh, OK, fetching caffeine |
| [22:35:03] | zombor (zombor!~zombor_@kohana/developer/zombor) has quit (Remote host closed the connection) | |
| [22:35:07] | dekarl: | iamlindoro: I see your point. It is just something that I had to fiddle around at work lately (xslt and bitmasks) and its easiest to just break it out into explicit nodes instead. |
| [22:36:08] | iamlindoro: | dekarl: The opposite side of that coin is, for example, I've written a Cocoa wrapper for all the APIs in Services…. and parsing a bit mask is way easier than looping through a bunch of keys/values and comparing strings |
| [22:36:20] | sphery: | FWIW, the embedded newlines are the reason we have to use such a long/complex post-processing command to simplify file logging to console logging (see perl command at http://www.mythtv.org/wiki/Logging#File_Logging ) rather than a simple: cut -d " " -f 1,2,3,9- /path/to/log |
| [22:40:05] | jmartens (jmartens!~jmartens@s5597ca60.adsl.wanadoo.nl) has quit (Quit: Leaving.) | |
| [22:40:41] | sphery: | dekarl / iamlindoro: not sure what you're talking about, specifically (AudioProps, VideoProps, etc.?), but IMHO, ideally we wouldn't give defined-in-mythtv-source-code or defined-in-mythtv-database-schema bit masks or defines in the output--primarily because that means we end up with the same broken approach of clients having to know what those values mean/interpreting data themselves (and getting broken when the masks/defines change) |
| [22:41:38] | danielk22: | sphery: I think we talked about that here a while back, IIRC stripping out newlines for the file log was uncontroversial so long as the console log wasn't affected. Maybe better would be to put each line on it's own with a timestamp in the file log, but I remember that being discussed. |
| [22:41:53] | dekarl: | iamlindoro: just saying that boolean or existance/absence of childs is native XML and works better with native XML toolchains |
| [22:42:12] | iamlindoro: | dekarl: That isn't compatible with what the XML serializer does |
| [22:42:24] | sphery: | danielk22: yeah, your idea of allowing different console formatting versus log formatting would work well |
| [22:42:26] | iamlindoro: | attributes aren't supported by the serializer, and I'm not sure how they could be |
| [22:42:49] | dekarl: | can it dynamically leave out elements? |
| [22:42:49] | iamlindoro: | (at least, I presume you mean having attributes which are bools) |
| [22:43:13] | iamlindoro: | ? |
| [22:43:36] | iamlindoro: | The code decides what gets passed into the serializer-- if you don't want something to appear in the output, don't pass it to the serializer |
| [22:43:42] | dekarl: | or elements which are boolean (true/false) or present/absent |
| [22:44:29] | iamlindoro: | That doesn't have anything to do with the serialized-- if you don't want to it be present, you simply don't include it |
| [22:44:56] | iamlindoro: | dekarl: How would you want to see the props represented in xml? |
| [22:44:57] | dekarl: | sphery: yes, I was making up the use case of filtering recordings by "it is HDTV" from the VideoProps |
| [22:48:02] | dekarl: | iamlindoro: From what I understand the two supported variants are a) <theelementname/> being present/absent and b) <theelementname>[true|false]</theelementname>. likely prefering a) but b) is good too if thats easier. |
| [22:48:35] | dekarl: | the decision between attribute and child element is another topic. If the code doesn't to attributes its an easy decision :) |
| [22:48:43] | dekarl: | s/to/do/ |
| [22:49:05] | iamlindoro: | dekarl: So yes, the serializer could do that-- it's just way less efficient in my personal opinion-- way easier to parse one value and interpret its meaning, IMO |
| [22:49:17] | iamlindoro: | I do agree it's less friendly |
| [22:49:44] | iamlindoro: | That said, depending on how much of a hit it was, I could live with doing both |
| [22:57:13] | sphery: | again, I like giving <hdtv /> and <widescreen /> or however, separately, rather than relying on in-db-schema set values that could change at any point and break clients (clients should just display, not interpret, data) |
| [22:59:00] | dekarl: | hmm, you would provide the xml, I'd provide the xslt, now we just need to find someone who wants to use it. I'll put an example of something together. (unwatched HD recordings ordered by rating or similar) |
| [23:18:13] | dblain: | iamlindoro: I need to head out for a few hours but wanted to give you my feedback. |
| [23:19:09] | dblain: | I'm fine with using UTC (that patch looks good as well). What about method parameters? We accept dates as params to a lot of methods. |
| [23:19:14] | iamlindoro: | dekarl: The XML output really doesn't really interest me, honestly-- I have a harder time envisioning being an end-user consuming the XML output versus an app author using JSON, which is lighter weight |
| [23:20:12] | dblain: | iamlindoro, dekarl: I choose to use elements insteam of attributes for xml rendering sue to simplicity. It could be changed or an alternate serializer added to favor attributes over elements. |
| [23:20:50] | iamlindoro: | dblain: yeah, you have a point.. I suppose there will be symmetry once myth uses UTC internally… but not sure what to do now |
| [23:21:37] | dblain: | also, currently, the Node Text of the element is always the value even if a node has children. I figured the default text for a program is it's description. That can be changed by updating the CLASSINFO in the datacontract if we don't like that behavior. |
| [23:23:05] | dblain: | I took a shot at designing the datacontracts/serializers before I had a need for them, so they may not be the best or most efficent design and you're free to change/standardize/optimise them all you want. :) |
| [23:24:32] | ** dblain shakes his head at all the typo's in the above comments :( ** | |
| [23:26:51] | dekarl: | iamlindoro: I'll look into providing an example early next week. |
| [23:27:34] | iamlindoro: | dekarl: I think you'll probably need to submit patches for what you want, I am just really short of time lately and working on XML output/output I'm not going to use is going to fall pretty low on the list |
| [23:28:01] | iamlindoro: | with a compelling example I'll definitely think about it, but I just have a ton that I really *want* to do right now |
| [23:28:31] | dekarl: | iamlindoro: sure, I don't expect you to go out of your way at all... just hinting at what I've been doing lately at work and sharing the experience |
| [23:30:03] | iamlindoro: | dblain: I think dekarl was asking about this code: http://pastebin.com/n4hdKS4S Which keeps tags called "description" plaintext in xml output |
| [23:30:16] | iamlindoro: | I figured there was a good reason for that, but couldn't figure it out |
| [23:34:11] | dekarl is now known as dekarl_zZz | |
| [23:34:37] | Seeker` (Seeker`!~cjo20@gateway/shell/ubottu/x-ngaruhwwgdjmhxau) has quit (Ping timeout: 258 seconds) | |
| [23:54:23] | natanojl (natanojl!~jonatan@c83-252-237-63.bredband.comhem.se) has quit (Ping timeout: 244 seconds) | |
IRC Logs collected by
BeirdoBot.
Please use the above link to report any bugs.