| Thursday, March 10th, 2011, 00:15 UTC | ||
| [00:15:41] | iamlindoro: | Heh, the internet content search has been broken since the mythsystem rewrite |
| [00:17:38] | iamlindoro: | someone forgot to test that one, I guess |
| [00:19:24] | iamlindoro: | wagnerrp, I have no idea how to fix this, looks like someone added some verbose so it was likely a known issue that never got resolved |
| [00:23:04] | iamlindoro: | wagnerrp, I see what the issue is |
| [00:23:08] | iamlindoro: | the search term isn't getting quoted |
| [00:23:22] | iamlindoro: | So a one-word search works, but nothing else will |
| [00:23:36] | sphery: | dblain: This is what I have for it: http://misc.thirdcontact.com/MythTV/mythtv-ad . . . es_api.patch . Just have to figure out how to use that version for the check, instead (but figure that's a job for another day). Please let me know if you have any concerns or want it done differently. Thanks. |
| [00:23:42] | iamlindoro: | Think I have the fix, same issue occered elsewhere after the merge |
| [00:57:02] | dblain: | sphery: the only thing I question is: Q_CLASSINFO( "defaultProp", "TimeZoneID" ); |
| [00:57:41] | sphery: | heh, wasn't sure what exactly that was about, so it's probably wrong |
| [00:57:47] | dblain: | this will make it so the TimeZoneID value is used for the text node in the rendered xml |
| [00:59:00] | dblain: | so the xml won't actually have a <TimeZoneID> element. |
| [00:59:31] | dblain: | I added that feature for items like Description in the programinfo |
| [01:00:01] | dblain: | Other than removing that one line, the rest looks good. |
| [01:01:59] | sphery: | hmmm... with it in, I was still getting: http://pastebin.com/dNmLd6TB (when requesting directly using firefox), but since it's wrong, I'll remove it |
| [01:02:37] | dblain: | As for how to call it from another myth program... I haven't create a client proxy class yet. Didn't think anyone wanted to use these services from within mythtv. |
| [01:03:49] | dblain: | Might be a bug on my part... It should of had "America/New_York" as the text node of the TimeZoneInfo element |
| [01:04:28] | sphery: | I was at least considering it. We currently have a bit of an ugly hack in the backend that creates a temporary monitor connection just to ask for time zone, before it creates a real connection to the backend, and I was hoping to remove that. |
| [01:05:33] | sphery: | When I wrote the initial check, I didn't have that hack--and it caused problems with infinite message recursion, so And uin figured out the cause and fixed it, but I figured if we don't use the Myth proto stuff, it would allow us to clean up the extra connection code. |
| [01:05:46] | dblain: | ah, the existing mythxmlclient.cpp in libmythupnp could be used to add the GetTimeZone method. |
| [01:06:29] | dblain: | It's a manual change at this point, but we already have a GetConnectionInfo implementation there. |
| [01:07:00] | sphery: | you mean add it there after also adding the service stuff from the patch? |
| [01:07:10] | sphery: | or instead |
| [01:07:38] | dblain: | If developers think it's a good idea to have a client proxy for these services, I'll look into implementing deserialize class and a way to generate the proxies |
| [01:07:54] | dblain: | adding it there as well as the service. |
| [01:08:20] | dblain: | the mythxmlclient class is used to make a SOAP call to the backend |
| [01:11:23] | dblain: | mythxmlclient doesn't belong in libmythupnp, but its been there for a while so adding another method to it isn't the end of the world. |
| [01:11:23] | sphery: | ah, got it... thanks for bringing me up to speed--it's starting to make sense to me, now :) |
| [01:11:48] | dblain: | np |
| [01:12:15] | dekarl1 (dekarl1!~dekarl@e180141186.adsl.alicedsl.de) has quit (Ping timeout: 240 seconds) | |
| [01:13:52] | Mousey (Mousey!~wtfisme@sea02-v600-nat.marchex.com) has quit (Quit: Leaving) | |
| [01:14:21] | dblain: | I know it's been hard for me to justify some of the design decissions I've been making because all the libraries are shared between the backend and frontend. My designs are for seperating functionallity, and it makes it look more complicated than it needs to be because technically the frontend can use the same methods as the backend. |
| [01:14:58] | dblain: | In my ideal world, the frontend would be in a seperate source tree with all the UI code, and a protocol proxy library to allow communication to the backend. |
| [01:16:28] | ** dblain will stop rambiling now! ** | |
| [01:17:24] | sphery: | heh, that approach sounds good to me. it's just step 2 that we have to figure out, then we can profit! |
| [01:17:46] | dekarl (dekarl!~dekarl@e180139141.adsl.alicedsl.de) has joined #mythtv | |
| [01:21:49] | dblain: | I may take some time seeing what I can come up with for a client proxy layer. It is something that will definitly take longer that 10 min :-D |
| [01:22:22] | dblain: | For performance sake, how does everyone feel about a code generator that is compiled and run at build time? |
| [01:30:18] | wagnerrp: | write code that... writes code? |
| [01:30:49] | davide (davide!~gigem@cpe-76-187-29-95.tx.res.rr.com) has quit (Remote host closed the connection) | |
| [01:32:05] | dblain: | :) yes. The deserializing of the received data is time consuming using introspection. (serializing is as well). I could write a tool that generates the serialization code based on the Data Classes. |
| [01:32:34] | gigem_ (gigem_!~gigem@cpe-76-187-29-95.tx.res.rr.com) has joined #mythtv | |
| [01:32:34] | gigem_ (gigem_!~gigem@cpe-76-187-29-95.tx.res.rr.com) has quit (Changing host) | |
| [01:32:34] | gigem_ (gigem_!~gigem@mythtv/developer/gigem) has joined #mythtv | |
| [01:33:02] | jcarlos (jcarlos!~quassel@85.137.96.30.dyn.user.ono.com) has quit (Quit: No Ping reply in 180 seconds.) | |
| [01:33:05] | dblain: | Right now I could do the same thing that I did for the serialization (and it's probably where I'll start). But I always thought compile time code generation would be the best answer to performance concerns. |
| [01:33:27] | jcarlos (jcarlos!~quassel@85.137.96.30.dyn.user.ono.com) has joined #mythtv | |
| [01:33:59] | sphery: | if I end up doing the mythdataserver with protobuf, I think that's basically how it would work (so /I/ am not opposed to having code generators for data serialization) |
| [01:38:25] | wagnerrp: | dblain: ive actually got something set up like that in the python bindings, except it rewrites itself on first run, rather than compile time |
| [01:38:45] | Dave123 (Dave123!~dave@cpe-66-66-127-3.rochester.res.rr.com) has quit (Quit: Leaving) | |
| [01:53:01] | jcarlos (jcarlos!~quassel@85.137.96.30.dyn.user.ono.com) has quit (Read error: Operation timed out) | |
| [01:55:05] | jcarlos (jcarlos!~quassel@85.137.96.30.dyn.user.ono.com) has joined #mythtv | |
| [01:58:11] | jya (jya!~jya@mythtv/developer/jya) has quit (Quit: jya) | |
| [01:59:46] | gigem (gigem!~david@mythtv/developer/gigem) has quit (Remote host closed the connection) | |
| [02:00:15] | gigem (gigem!~david@host103.16.intrusion.com) has joined #mythtv | |
| [02:00:15] | gigem (gigem!~david@host103.16.intrusion.com) has quit (Changing host) | |
| [02:00:15] | gigem (gigem!~david@mythtv/developer/gigem) has joined #mythtv | |
| [02:00:42] | jcarlos (jcarlos!~quassel@85.137.96.30.dyn.user.ono.com) has quit (Quit: No Ping reply in 180 seconds.) | |
| [02:01:15] | jcarlos (jcarlos!~quassel@85.137.96.30.dyn.user.ono.com) has joined #mythtv | |
| [02:26:47] | kormoc is now known as kormoc_afk | |
| [02:46:09] | MaverickTech (MaverickTech!~MaverickT@111.86.233.220.static.exetel.com.au) has joined #mythtv | |
| [02:46:26] | MavT (MavT!~MaverickT@111.86.233.220.static.exetel.com.au) has quit (Ping timeout: 240 seconds) | |
| [02:49:03] | jya (jya!~jya@mythtv/developer/jya) has joined #mythtv | |
| [03:04:43] | wagnerrp: | Captain_Murdoch: as opposed to what it currently does, serve up the backend status page? |
| [03:05:13] | Captain_Murdoch: | I think these URLs are /html/blah |
| [03:05:26] | wagnerrp: | or are you talking about a /Guide/index.html, /Dvr/index.html, ... |
| [03:06:21] | Dave123 (Dave123!~dave@cpe-66-66-127-3.rochester.res.rr.com) has joined #mythtv | |
| [03:06:45] | Captain_Murdoch: | was just thinking of adding a /html/index.html that has links to /Guide/wsdl /Dvr/wsdl etc. eventually it might make sense to remove the backend status webpage from / and make you use /GetStatusHTML then we can serve the index.html if you just specify / |
| [03:07:35] | Captain_Murdoch: | I haven't compiled the patch yet though, but I thought by reading it that it was serving up files under /html/index.html,not /index.html |
| [03:07:59] | dblain: | correct... /html/ |
| [03:08:41] | dblain: | '/' still returns the status page |
| [03:08:52] | Captain_Murdoch: | dblain, what do you think about getting rid of the / serving up the same as /GetStatusHTML and making the http static file server be the fallback server, so if a request wasn't serviced by any other extension, we'd look in the html dir for an actual file. |
| [03:09:11] | Captain_Murdoch: | ie, so / would serve up shardir/html/index.html |
| [03:09:46] | Captain_Murdoch: | that avoids having to put /html in front of any static file request |
| [03:09:58] | dblain: | I can do that. |
| [03:10:35] | Captain_Murdoch: | if we have a default index.html page that has a link to /GetStatusHTML, then I don't see any issues getting rid of / giving the status page as well. |
| [03:12:25] | dblain: | So if I understand correctly. I should have both HttpServerExtensions listen on '/', and if no file is provided the status page is displayed (so we won't support a default index.html static file) |
| [03:12:40] | Captain_Murdoch: | RE: adding the default to index.html, it would be nice to be able to say http://localhost:8544/Setup/ instead of having to append the index.html on the end. |
| [03:13:55] | dblain: | Do you want / to always display the status page, or do you want it to load the index.html which will redirect to the status page? |
| [03:13:59] | Captain_Murdoch: | what you describe would provide backwards compatibility, but I wonder if we could break that and just have users give /GetStatusHTML if they want the status page. if we provide a default index.html file with links to the status page, wsdl, etc. |
| [03:14:46] | Captain_Murdoch: | I say we just have a link to the status page, no need to redirect. |
| [03:14:50] | dblain: | If you're willing to take the heat on breaking the status page :) I can make it so /xml still works to return the status xml |
| [03:15:06] | dblain: | which I would think most people would script against |
| [03:15:35] | dblain: | or /GetStatusXML... |
| [03:15:43] | sphery: | FWIW, most users don't know that the status page exists--they seem to think it's part of MythWeb |
| [03:15:46] | Captain_Murdoch: | yeah, they should be using that instead of / |
| [03:15:49] | wagnerrp: | well it would hardly be breaking it |
| [03:15:56] | wagnerrp: | all you would have to do is adjust the link |
| [03:16:03] | dblain: | true. |
| [03:16:09] | Captain_Murdoch: | mythweb is easy to fix if it doesn't already use the full url |
| [03:16:13] | sphery: | so with MythWeb changed to use it, few would likely even notice |
| [03:16:21] | wagnerrp: | there are only a couple programs i know of that pull from it |
| [03:17:02] | wagnerrp: | all non-essential stuff that breaking would just cause a nuisance to users |
| [03:17:38] | wagnerrp: | gnome and windows status widgets, the command line status program |
| [03:17:43] | Captain_Murdoch: | I say we should serve up a index.html page on / and make users provide a specific URL to get to the status html or xml. |
| [03:18:01] | wagnerrp: | and them there are several more scripts that for unknown reasons decided to pull off the command line status program |
| [03:18:03] | Captain_Murdoch: | things like that should have been using the xml version anyway rather than parsing html. |
| [03:18:16] | Captain_Murdoch: | screen-scraping mythbackend. :) |
| [03:18:52] | dblain: | would /status and /status/xml be okay? (that way the base url for status pages/xml would be '/status' |
| [03:19:21] | Captain_Murdoch: | sure, if we're breaking other mythxml stuff, I don't see why we can't change the status page as well. |
| [03:19:31] | sphery: | Captain_Murdoch: I agree on both counts. I like the idea of "moving" the status page, and have been mentioning that those scripts should be using the XML interface or bindings for a long time. :) |
| [03:19:55] | sphery: | seems that the authors weren't interested in my opinion, though |
| [03:20:16] | Captain_Murdoch: | so the static file extension is the fallback, anything else has a specific prefix and if no known prefixes are matched, it falls through and checks for an actual file. |
| [03:20:18] | wagnerrp: | i think they just havent touched those scripts in years |
| [03:20:18] | dblain: | BTW: every extension we add with it's own base url '/guide', '/myth', ... means we can't have a directory in the html folder with the same name! |
| [03:20:37] | Captain_Murdoch: | dblain, are they case sensitive? |
| [03:21:12] | dblain: | yes, I believe I coded them as case sensitve. |
| [03:21:26] | Captain_Murdoch: | perhaps a naming convention, services are CamelCase, dirs are lowercase |
| [03:21:32] | Captain_Murdoch: | just in case |
| [03:21:49] | wagnerrp: | yes, they are case sensitive |
| [03:21:56] | wagnerrp: | (just tested) |
| [03:22:05] | dblain: | that works. I believe all services start with a uppercase now. |
| [03:22:41] | dblain: | It would most likely fall though if a method wasn't found, but figured I'd put that potential naming conflict out there now. |
| [03:22:52] | Captain_Murdoch: | ok, so in this case, setup webpage could be /setup using APIs under things like /Blah |
| [03:23:11] | dblain: | sounds good. |
| [03:23:24] | dblain: | give me a few minutes and I'll have the changes done for the status page. |
| [03:24:45] | ** Captain_Murdoch starts a trunk compile so he can play with it and try to get a default index.html page in tonight. ** | |
| [03:25:28] | dblain: | I'll also add the support to default to index.html if only a path is specified while I'm in here. |
| [03:26:22] | kormoc_afk is now known as kormoc | |
| [03:29:40] | Captain_Murdoch: | great. thanks. |
| [03:30:27] | Captain_Murdoch: | will that require /setup/ or will it check if /setup is a dir and serve up /setup/index.html in that case? |
| [03:31:19] | dblain: | I'm using QFileInfo().isDir, then adding a "index.html" so I think /setup will work ok. |
| [03:31:38] | dblain: | I'll know more in a few minutes |
| [03:32:04] | dblain: | having an order issue with the extension... since '/' matches all requests :( |
| [03:36:52] | dblain: | Anyone have an issue with me embedding the static html extension as part of the main httpserver? when the httpServer delegates the incomming request to all registered extensions, it will call the htmlextension directly if not processed by one of them? (so it's less of an extension and would be more of an integral part) |
| [03:39:29] | Captain_Murdoch: | fine by me, you probably know better than all of us in that area. :) |
| [03:39:59] | Captain_Murdoch: | it's a simple bit of code, integrating it doesn't seem like much of an issue. |
| [03:40:06] | dblain: | it means the frontend gets a static html server as well! |
| [03:40:15] | dblain: | not sure if that's a good thing or bad. |
| [03:40:42] | dblain: | could be an issue with the shared html path being REALLY shared. |
| [03:42:27] | Captain_Murdoch: | could come in handy down the road. |
| [03:43:02] | Captain_Murdoch: | could make it /backend-setup instead of /setup in case we want a frontend setup page eventually. |
| [03:44:19] | Captain_Murdoch: | shouldn't be much in the html path unless we put it there. since this is the shared dir, we're not even serving up files from the user's home dir, we're serving up files accessible to all/most users. |
| [03:44:42] | dblain: | I don't think we need to make a decission yet. Lets see how we use the html functionality. It may be a simple setting down the road to have unique html paths. |
| [03:44:59] | dblain: | true |
| [03:49:00] | Captain_Murdoch: | anybody know if mythweb uses the html or xml status page. I assume html, but I know it does it's own css somehow. |
| [03:54:41] | wagnerrp: | yesh, its the html one, wrapped in css and header/footer |
| [04:06:29] | dblain: | Captain_Murdoch: looks like your going to need the /Status/ |
| [04:06:45] | Captain_Murdoch: | ok. /Status/GetStatusHTML ? |
| [04:07:06] | dblain: | when I add /index.html to /Status, future calls fail because they don't have the /Status |
| [04:07:15] | dblain: | That works too |
| [04:07:34] | dblain: | The valid Status url's are: |
| [04:08:01] | dblain: | ' /Status returns html |
| [04:08:15] | dblain: | that's wrong! |
| [04:08:36] | dblain: | ... /Status/ |
| [04:08:36] | jya (jya!~jya@mythtv/developer/jya) has quit (Quit: jya) | |
| [04:08:54] | dblain: | ... /Status/GetStatusHTML |
| [04:09:02] | dblain: | those two return html |
| [04:09:13] | dblain: | ... /Status/GetStatus |
| [04:09:20] | dblain: | ... /Status/xml |
| [04:09:24] | dblain: | both return xml |
| [04:15:31] | dblain: | Changes comitted. let me know if you have any issues. |
| [04:18:43] | Captain_Murdoch: | ok, I'll update and test and get the html dir in. about ready with that, just need to test with your latest mods. |
| [05:11:06] | jya (jya!~jya@60-242-40-141.static.tpgi.com.au) has joined #mythtv | |
| [05:11:06] | jya (jya!~jya@60-242-40-141.static.tpgi.com.au) has quit (Changing host) | |
| [05:11:06] | jya (jya!~jya@mythtv/developer/jya) has joined #mythtv | |
| [05:11:42] | cesman_ is now known as cesman | |
| [05:12:04] | cesman (cesman!~cecil@pool-71-254-162-41.lsanca.fios.verizon.net) has quit (Changing host) | |
| [05:12:04] | cesman (cesman!~cecil@pdpc/supporter/professional/cesman) has joined #mythtv | |
| [05:21:44] | MavT (MavT!~MaverickT@111.86.233.220.static.exetel.com.au) has joined #mythtv | |
| [05:23:09] | Beirdo: | new branch for the ffmpeg sync (based on the same ffmpeg version, but off current master) will be pushed soon |
| [05:23:35] | Beirdo: | just doing the test compile right now |
| [05:24:19] | Beirdo: | if anyone else wants to test it before it gets synced into master, let's give it 2 weeks, maybe? |
| [05:24:39] | MaverickTech (MaverickTech!~MaverickT@111.86.233.220.static.exetel.com.au) has quit (Ping timeout: 246 seconds) | |
| [05:25:46] | iamlindoro: | wonder if jannau has any info on where Alex Strange is with adding multithread support for other codecs beyond VP3 |
| [05:26:25] | iamlindoro: | ie, whether there might be anything pending which might be worthy of a tiny bump/short hold... I haven't seen much of note on ffmpeg-devel lately worth waiting, though |
| [05:26:33] | Beirdo: | yeah, agreed |
| [05:27:34] | Beirdo: | well, we'll try to find out over the next few weeks, I guess. If we hear nothing, I'd suggest we move forward, and maybe plan on another when we have more things to merge :) |
| [05:30:08] | Beirdo: | ok, done. not current master, it was as of a couple hours back, but a lot closer. |
| [05:33:39] | Captain_Murdoch: | markk, should I need to reconfigure or anything after your latest commit? I'm getting the following during compile: http://pastebin.com/hAkwptEv |
| [05:36:14] | Captain_Murdoch: | and oddly, after re-pulling to pull in nigels latest commit, it actually compiled this time. blame it on Qt I guess. |
| [05:37:14] | Captain_Murdoch: | or maybe not... I should have waited for the compile. I thought it passed the file, but it didn't. so it is broken. |
| [05:37:35] | iamlindoro: | hmm... I am pretty sure I pulled in all those changes and it compiled okay here |
| [05:39:30] | iamlindoro: | but I may or may not have distcleaned... can't remember |
| [05:40:42] | Captain_Murdoch: | an #include <QPen> fixes it here, but I'm not sure where he'd want that done (or why I'd need it but not others). qt 4.5.2 here. |
| [05:41:11] | iamlindoro: | 4.7.0 here |
| [05:42:50] | iamlindoro: | I'm sure you're right, though, it wouldn't be the first time I had randomly managed to convince a build to work when there was a real problem at hand |
| [05:42:58] | iamlindoro: | I should probably learn to shut my mouth in those instances :) |
| [05:43:24] | Captain_Murdoch: | I don't see includes for other things like QBrush though, so it must be getting picked up from elsewhere. probably a Qt version difference issue. |
| [05:48:56] | natanojl (natanojl!~jonatan@c83-252-237-63.bredband.comhem.se) has joined #mythtv | |
| [05:56:56] | Captain_Murdoch: | markk, for now, in my tree, I dropped a #include <QPen> into mythrender_opengl.h since the issue also popped up in 2 other files that include that header. |
| [06:16:57] | jmartens (jmartens!~jmartens@s5597ca60.adsl.wanadoo.nl) has joined #mythtv | |
| [06:26:46] | jmartens (jmartens!~jmartens@s5597ca60.adsl.wanadoo.nl) has quit (Quit: Leaving.) | |
| [06:33:44] | ** Captain_Murdoch notices his commit log had a typo on the backend webserver port. ** | |
| [06:33:49] | Captain_Murdoch: | oh well... |
| [06:45:12] | jmartens (jmartens!~jmartens@109.232.42.33) has joined #mythtv | |
| [06:53:02] | SteveGoodey (SteveGoodey!~steve@host86-185-50-163.range86-185.btcentralplus.com) has joined #mythtv | |
| [07:08:01] | markk: | Captain_Murdoch: so just the one extra include needed? |
| [07:15:44] | SteveGoodey (SteveGoodey!~steve@host86-185-50-163.range86-185.btcentralplus.com) has quit (Remote host closed the connection) | |
| [07:25:02] | martin__ (martin__!~quassel@static-88.131.29.2.addr.tdcsong.se) has joined #mythtv | |
| [08:01:27] | GreyFoxx (GreyFoxx!~greg@mythtv/developer/GreyFoxx) has quit (Ping timeout: 246 seconds) | |
| [08:01:34] | GreyFoxx (GreyFoxx!~greg@out.of.phaze.org) has joined #mythtv | |
| [08:08:58] | hansmuc (hansmuc!~hansmuc@ppp-188-174-11-120.dynamic.mnet-online.de) has joined #mythtv | |
| [08:09:52] | hansmuc (hansmuc!~hansmuc@ppp-188-174-11-120.dynamic.mnet-online.de) has left #mythtv () | |
| [08:11:49] | leprechau (leprechau!leprechau@c-68-52-207-200.hsd1.tn.comcast.net) has quit (Ping timeout: 252 seconds) | |
| [08:16:22] | natanojl (natanojl!~jonatan@c83-252-237-63.bredband.comhem.se) has quit (Ping timeout: 250 seconds) | |
| [08:51:50] | Goga777 (Goga777!~Goga777@shpd-78-36-179-115.static.vologda.ru) has joined #mythtv | |
| [09:08:45] | Goga777 (Goga777!~Goga777@shpd-78-36-179-115.static.vologda.ru) has quit (Read error: Connection reset by peer) | |
| [09:25:14] | MavT (MavT!~MaverickT@111.86.233.220.static.exetel.com.au) has quit (Ping timeout: 250 seconds) | |
| [09:58:19] | MaverickTech (MaverickTech!~MaverickT@111.86.233.220.static.exetel.com.au) has joined #mythtv | |
| [10:10:32] | jamesba (jamesba!~jamesba@gateb.kw.bbc.co.uk) has joined #mythtv | |
| [11:05:02] | Guest1508 (Guest1508!~mike@c-24-21-63-118.hsd1.or.comcast.net) has quit (Remote host closed the connection) | |
| [11:05:54] | mike|2 (mike|2!~mike@c-24-21-63-118.hsd1.or.comcast.net) has joined #mythtv | |
| [11:30:30] | voyo (voyo!~voyo@213-134-188-198.home.aster.pl) has joined #mythtv | |
| [11:47:20] | jya (jya!~jya@mythtv/developer/jya) has quit (Quit: jya) | |
| [12:27:31] | tris (tris!~tristan@173-164-188-122-SFBA.hfc.comcastbusiness.net) has quit (Ping timeout: 260 seconds) | |
| [12:41:18] | tris (tris!~tristan@173-164-188-122-SFBA.hfc.comcastbusiness.net) has joined #mythtv | |
| [12:59:15] | davide_ (davide_!~david@host103.16.intrusion.com) has joined #mythtv | |
| [12:59:15] | davide_ (davide_!~david@host103.16.intrusion.com) has quit (Changing host) | |
| [12:59:15] | davide_ (davide_!~david@mythtv/developer/gigem) has joined #mythtv | |
| [13:00:04] | gigem (gigem!~david@mythtv/developer/gigem) has quit (Ping timeout: 240 seconds) | |
| [13:02:49] | andreax1 (andreax1!~andreaz@p57B9540D.dip.t-dialin.net) has quit (Quit: Leaving.) | |
| [13:16:04] | j-rod|afk is now known as j-rod | |
| [14:04:24] | MaverickTech (MaverickTech!~MaverickT@111.86.233.220.static.exetel.com.au) has quit (Ping timeout: 252 seconds) | |
| [14:26:15] | MaverickTech (MaverickTech!~MaverickT@111.86.233.220.static.exetel.com.au) has joined #mythtv | |
| [14:42:12] | Captain_Murdoch: | markk, yeah, that was all I needed here for some reason. |
| [14:43:23] | Captain_Murdoch: | Beirdo, thanks for that api change. I meant to look at that and got interrupted |
| [14:46:19] | markk: | Captain_Murdoch: btw, I think I saw a reference the other day to the theme downloader not working. It consistently doesn't work for me on remote frontends. Always says it fails on the first attempt and then succeeds if I try again. repeatable on ubuntu, windoze and osx |
| [14:47:18] | Captain_Murdoch: | yeah, I replied to one guy on-list about it but never heard back from him with logs. if you can get me logs with "-v network,file,extra" from both the FE and MBE then I'll take a look. |
| [14:58:55] | martin__ (martin__!~quassel@static-88.131.29.2.addr.tdcsong.se) has quit (Remote host closed the connection) | |
| [15:14:23] | simonckenyon|2 (simonckenyon|2!~simoncken@195.7.61.12) has joined #mythtv | |
| [15:14:26] | simonckenyon (simonckenyon!~simoncken@195.7.61.12) has quit (Read error: Connection reset by peer) | |
| [15:22:53] | Kunalagon (Kunalagon!~Kunalagon@212.200.241.12) has joined #mythtv | |
| [15:41:33] | jamesba: | Hi all, I was wondering: from inside the code for a plugin's main screen what would I call to cause the plugin to close and go back to the menus (the equivalent of what happens when the user presses escape)? |
| [15:42:37] | stuartm: | Close(); |
| [15:43:12] | iamlindoro: | Well there you go with your tricky naming again |
| [15:43:29] | stuartm: | yeah, I'm terrible at it ;) |
| [15:48:02] | iamlindoro: | markk: Captain_Murdoch: Something with the creation of the theme subdir in the temp SG, perhaps? |
| [15:49:46] | iamlindoro: | nah, that wouldn't explain it being intermittent. Weird. |
| [15:52:10] | kwmonroe (kwmonroe!~kwmonroe@174-149-146-167.pools.spcsdns.net) has joined #mythtv | |
| [16:05:16] | kwmonroe (kwmonroe!~kwmonroe@174-149-146-167.pools.spcsdns.net) has quit (Quit: Ex-Chat) | |
| [16:07:33] | Kunalagon (Kunalagon!~Kunalagon@212.200.241.12) has quit (Quit: Leaving.) | |
| [16:08:24] | Captain_Murdoch: | iamlindoro, if it's a remote vs local FE issue, then it may be something in the code that detects a shared Temp SG dir. I think I short-circuit if the FE sees the file locally after it tells the MBE to download the file. I can't recall right now, I'll have to doublecheck the code. |
| [16:08:25] | jamesba: | stuartm: just a plain function call then? or is it a member of some class? |
| [16:09:20] | iamlindoro: | jamesba: Inherited from MythScreenType IIRC |
| [16:09:49] | iamlindoro: | ie, all mythui windows should be declared as a public MythScreenType and then Close(); should work on its own |
| [16:10:00] | iamlindoro: | Captain_Murdoch: ah, ok |
| [16:12:48] | jamesba: | ok, thanks |
| [16:14:08] | Captain_Murdoch: | iamlindoro, I can probably test that by starting my FE up with a different localhostname and .mythtv dir |
| [16:26:34] | stuartm: | Captain_Murdoch: if we didn't keep things like passwords and other potentially sensitive stuff in the settings table I'd love to just dump the entire thing into the submission, I think knowing which settings were actually changed from defaults would be really interesting – either in re-thinking the defaults or eliminating unused settings |
| [16:28:23] | stuartm: | certainly it would be helpful to be able to gather additional data from time to time to answer one-off questions such as "Does anyone still use that feature?" |
| [16:30:40] | hansmuc (hansmuc!~hansmuc@ppp-188-174-11-120.dynamic.mnet-online.de) has joined #mythtv | |
| [16:30:43] | iamlindoro: | Just so long as we recognize that people still using various features is not always a valid reason for keeping them :) |
| [16:30:59] | jamesba: | perhaps restructure so as not to store passwords in that table? |
| [16:31:43] | iamlindoro: | We'd just basically be creating a second, identical settings table for passwords if we did |
| [16:31:54] | martin___ (martin___!~quassel@h-39-23.A155.priv.bahnhof.se) has joined #mythtv | |
| [16:32:13] | iamlindoro: | It's be easier just to where "where value not" for the 2–3 password settings |
| [16:32:24] | iamlindoro: | er just to do |
| [16:34:04] | stuartm: | jamesba: ultimately that's what we'll be doing anyway, but those changes will be part of the multi-user framework which will take some time |
| [16:35:17] | stuartm: | and when we do move the passwords, we'll start hashing them |
| [16:37:31] | wagnerrp: | stuartm: not mythtv passwords, stuff like schedules direct account passwords |
| [16:38:49] | stuartm: | I thought those were in the external configs? They are for xmltv at least |
| [16:39:46] | wagnerrp: | nope, videosource.userid and videosource.password |
| [16:40:02] | wagnerrp: | although thats in videosource, and not settings, so... |
| [16:56:00] | Captain_Murdoch: | might be nice to have a way to have the server tell the client about additional data to collect. ie, a year from now, we decide we'd like to know who has what in TABLEABC.COLXYZ. we can't do that except for users who are updating to current fixes or master. if the server itself could tell the client "here's a list of values I'd like" then that might be handy. we'd have to do sanity checking on the client as a precaution, but |
| [16:56:01] | Captain_Murdoch: | I can see that ability being handy. |
| [16:59:39] | andreax (andreax!~andreaz@p57B9540D.dip.t-dialin.net) has joined #mythtv | |
| [17:08:04] | wagnerrp: | stuarta: opengl on my laptop seems to be broken at the moment after a recent bout of updates, so i cant prove my point |
| [17:08:26] | wagnerrp: | but back when it was working, the rage128 in my laptop had no trouble handling the opengl painter at 1280x720 |
| [17:08:42] | wagnerrp: | the renderer is different, but the UI itself worked just fine |
| [17:09:09] | wagnerrp: | meaning its not legacy users we have to worry about, but framebuffer users |
| [17:09:22] | wagnerrp: | people still trying to output from their PVR-350s and such |
| [17:09:41] | wagnerrp: | and all of them will be picked up by their playback profile information |
| [17:11:33] | jmartens (jmartens!~jmartens@109.232.42.33) has quit (Quit: Leaving.) | |
| [17:11:58] | ** Captain_Murdoch realizes if he wouldn't type 10–20 words here and there separated by minutes of interruption then he wouldn't say things like 'handy' two sentences in a row. ** | |
| [17:27:58] | voyo (voyo!~voyo@213-134-188-198.home.aster.pl) has left #mythtv () | |
| [17:31:41] | natanojl (natanojl!~jonatan@c83-252-237-63.bredband.comhem.se) has joined #mythtv | |
| [17:32:45] | kormoc is now known as kormoc_afk | |
| [17:35:26] | kormoc_afk is now known as kormoc | |
| [17:38:18] | stuartm: | Captain_Murdoch: I do that all the time |
| [17:39:43] | jmartens (jmartens!~jmartens@s5597ca60.adsl.wanadoo.nl) has joined #mythtv | |
| [17:41:39] | stuartm: | I've even done it in the same sentence |
| [17:43:10] | Captain_Murdoch: | thanks, /me doesn't feel as bad now, now I don't feel as bad :) |
| [17:52:18] | kwmonroe (kwmonroe!~kwmonroe@32.97.110.58) has joined #mythtv | |
| [17:54:15] | jamesba (jamesba!~jamesba@gateb.kw.bbc.co.uk) has quit (Remote host closed the connection) | |
| [17:58:08] | davide_ (davide_!~david@mythtv/developer/gigem) has quit (Remote host closed the connection) | |
| [17:58:33] | davide_ (davide_!~david@host103.16.intrusion.com) has joined #mythtv | |
| [17:58:33] | davide_ (davide_!~david@host103.16.intrusion.com) has quit (Changing host) | |
| [17:58:33] | davide_ (davide_!~david@mythtv/developer/gigem) has joined #mythtv | |
| [17:59:24] | justpaul (justpaul!alexdelarg@poizon.epicshells.com) has quit (Remote host closed the connection) | |
| [17:59:30] | justpaul (justpaul!alexdelarg@poizon.epicshells.com) has joined #mythtv | |
| [18:08:35] | hansmuc (hansmuc!~hansmuc@ppp-188-174-11-120.dynamic.mnet-online.de) has left #mythtv () | |
| [18:08:56] | justpaul (justpaul!alexdelarg@poizon.epicshells.com) has quit (Read error: Connection reset by peer) | |
| [18:09:32] | justpaul (justpaul!~justpaul@poizon.epicshells.com) has joined #mythtv | |
| [18:17:52] | Anssi (Anssi!hannulaa@mandriva/developer/anssi) has quit (Remote host closed the connection) | |
| [18:18:07] | MythBuild: | build #690 of master-linux-64bit is complete: Failure [failed compile core] Build details are at http://code.mythtv.org/buildbot/builders/mast . . . t/builds/690 blamelist: Robert McNamara <rmcnamara@mythtv.org > |
| [18:18:16] | iamlindoro: | doh.. hmm |
| [18:20:26] | martin___ (martin___!~quassel@h-39-23.A155.priv.bahnhof.se) has quit (Ping timeout: 260 seconds) | |
| [18:21:51] | MythBuild: | build #641 of master-linux-32bit is complete: Failure [failed compile core] Build details are at http://code.mythtv.org/buildbot/builders/mast . . . t/builds/641 blamelist: Robert McNamara <rmcnamara@mythtv.org > |
| [18:24:30] | iamlindoro: | yeah yeah ;) |
| [18:24:50] | Beirdo: | heh |
| [18:25:04] | Beirdo: | it is nice to get relatively timely notice though |
| [18:26:57] | iamlindoro: | Already reverted the breakage, next fix will still be ok but "feels" weird (it's just a method to construct a season/episode number) for being in libmyth instead of libmythmetadata |
| [18:28:02] | iamlindoro: | at least this way I won't have to touch any .pro files (and one can only assume break Mac OS X ;) ) |
| [18:30:27] | MythBuild: | build #691 of master-linux-64bit is complete: Success [build successful] Build details are at http://code.mythtv.org/buildbot/builders/mast . . . t/builds/691 |
| [18:32:11] | Beirdo: | heh |
| [18:32:25] | Beirdo: | we so need to rework our libraries again some decade |
| [18:32:58] | Beirdo: | it just gets trickier with every added platform |
| [18:33:42] | Beirdo: | I find it odd that 32 bit and 64 bit have different warnings for the plugins |
| [18:33:53] | dblain: | I think that having more platforms keeps us honest :) |
| [18:33:56] | Beirdo: | he last 2 in the 64-bit don't show up in 32-bit |
| [18:34:23] | Beirdo: | but that's all in stuff that's being worked on :) |
| [18:34:48] | Beirdo: | yeah, more platforms can be a good thing, but it does make things more painful at first |
| [18:34:59] | iamlindoro: | dblain: I'm inclined to agree-- and in a lot of cases forces us to use Qt functionality (as we should) rather than some home-grown stuff we construct because we don't know it's there |
| [18:35:03] | dblain: | yes, no denying that! |
| [18:36:25] | dblain: | iamlindoro: I know I've had to rethink some things I wanted to do because I was thinking of the "windows way"... Qt does offer help, but sometimes you need to change your thinking/design. |
| [18:36:27] | Beirdo: | now that we have the new windows build script in... I need to look at hooking that into buildbot as well. And maybe an OSX cross-compile too. It would be nice to have native and cross-compiled both in there |
| [18:36:53] | Beirdo: | QStrings not being thread-safe is a big bumre. |
| [18:36:55] | Beirdo: | bummer. |
| [18:37:31] | MythBuild: | build #642 of master-linux-32bit is complete: Success [build successful] Build details are at http://code.mythtv.org/buildbot/builders/mast . . . t/builds/642 |
| [18:37:33] | Beirdo: | we have in the doxygen docs instructions on how to do a deep copy though |
| [18:37:40] | dblain: | I don't have an issue with QStrings threading model. Not may objectes are truely Thread Safe. |
| [18:37:42] | iamlindoro: | Woo hoo! I win the buildbot |
| [18:38:04] | Beirdo: | occasionally, the QStrings have bitten our butts hard |
| [18:38:12] | dblain: | true. |
| [18:38:27] | Beirdo: | mostly because we've been subconciously expecting them to be threadsae |
| [18:38:45] | Beirdo: | what is with my typing today, jeez |
| [18:41:57] | ** iamlindoro crosses his fingers ** | |
| [18:42:01] | iamlindoro: | Second try's a charm? |
| [18:46:48] | ** dblain is about to commit a change, but doesn't want to interrupt iamlindoro's build :) ** | |
| [18:46:57] | iamlindoro: | Does it kill the build? |
| [18:47:02] | iamlindoro: | I thought it just waited |
| [18:47:09] | Beirdo: | it will queue it |
| [18:47:23] | dblain: | It was a joke! |
| [18:47:32] | Beirdo: | if your change is in within 2min of his change, then they get built as one build |
| [18:47:35] | Beirdo: | k :) |
| [18:47:51] | iamlindoro: | heh |
| [18:47:57] | stuartm: | it doesn't kill the build, and if say 3 commits go in while it's waiting for a build to finish it just schedules one build, not 3 |
| [18:47:58] | dblain: | just added a |
| [18:47:59] | iamlindoro: | I was genuinely curious |
| [18:48:04] | Beirdo: | hard to tell sometimes online :) |
| [18:48:06] | dblain: | 'version' method on all services. |
| [18:48:22] | iamlindoro: | still, at least core completed successfully, so yay me |
| [18:48:35] | iamlindoro: | You know it's a bad day when you are thrilled that your committed change actually compiles |
| [18:48:39] | Beirdo: | hehe |
| [18:48:45] | Beirdo: | I've had those days |
| [18:49:26] | iamlindoro: | K, hooray, plugins good too, I'm going for my bike ride |
| [18:49:40] | Beirdo: | have a safe one |
| [19:28:55] | wagnerrp: | danielk22: yeah, smolt already pulls all that data out of sysfs |
| [20:24:10] | andreax1 (andreax1!~andreaz@p57B95D53.dip.t-dialin.net) has joined #mythtv | |
| [20:24:21] | stuartm: | hmm, my backend just deadlocked for the second time this week |
| [20:24:57] | Beirdo: | ouchie. |
| [20:25:07] | Beirdo: | any idea which threads were involved yet? |
| [20:25:51] | andreax (andreax!~andreaz@p57B9540D.dip.t-dialin.net) has quit (Ping timeout: 260 seconds) | |
| [20:26:01] | sphery: | stuartm: You might know (or some other EIT expert)... Is http://code.mythtv.org/trac/ticket/9592#comment:1 just a case of "user needs to rescan"? |
| [20:27:00] | stuartm: | Beirdo: no, both times it was supposed to be recording and so to avoid missing the programmes entirely I just restarted rather than attempting to debug |
| [20:27:33] | Beirdo: | gotcha |
| [20:27:40] | stuartm: | both times it seems to have occurred as a recording was due to start, nothing gets recorded and the backend won't respond to the frontend |
| [20:27:48] | Beirdo: | yuck |
| [20:28:04] | Beirdo: | if I can be of any assistance on tracking it down, let me know |
| [20:29:05] | stuartm: | sphery: stuarta, jannau or danielk22 might be able to answer that, they all know a little more about the dvb code than I do |
| [20:48:23] | stuartm: | sphery: fwiw it doesn't sound like a problem that rescanning will fix, it's a broken provider who is including EIT on pid 0x11 but failing to indicate the presence of that service in the SDT |
| [20:50:27] | stuartm: | in the past we'd try to encourage users to report such problems to their broadcasters since they are often transmitting non-compliant streams by accident, not design |
| [20:53:07] | sphery: | Ah, ok. I don't know my DVB specs at all, so didn't know if it was something the broadcaster changed that would have been picked up with a rescan or what. |
| [20:56:18] | stuartm: | we check that table at runtime |
| [20:57:20] | stuartm: | also during the scan, but the problem being that we're trying to use EIT for those channels but the SDT is telling us that it's not available |
| [21:01:53] | iamlindoro: | dblain: If the service versions are something we expect to increment on adding new services, I wonder if there might be any wisdom to putting it in mythversion.h so that all the "bumped" values for the (non-bindings) code are in one place? |
| [21:02:25] | iamlindoro: | Let us know what you want from us re: the version numbering when something new gets added |
| [21:03:10] | danielk22: | stuartm: for the deadlocked mythbackend assuming "ulimit -c unlimited" in the startup script.. "killall -s 6 mythbackend" |
| [21:03:33] | danielk22: | stuartm: that will give you a core so you can produce a backtrace at your leisure.. |
| [21:05:30] | danielk22: | stuartm: I also fixed at least two potential deadlocks in the mythtv-rec2 branch, on in the recorders and one in the scheduler.. I need to move the dbcheck.cpp change to trunk so people (incl myself) can more easily switch back and forth. |
| [21:06:15] | wagnerrp: | iamlindoro, dblain: on the other hand, if the different services are exposing a page of service definitions, and the responses are all markup data that wont have the ordering issues that the protocol stringlists have |
| [21:06:21] | wagnerrp: | is the version numbering really needed |
| [21:06:41] | wagnerrp: | or can you just require that clients check against the exposed interfaces before polling information |
| [21:13:49] | Captain_Murdoch: | Beirdo, RE: that mythcommflag patch. for the race condition, we could just 'schedule' the ~realtime jobs a minute or two in the future using the schedruntime field. |
| [21:14:34] | dblain: | iamlindoro: the service version is specific to each service. I would expect it bumped when a new method or a change in parameters is made. |
| [21:15:06] | Beirdo: | Captain_Murdoch: yeah, the other option is if the file is zero bytes, delay like 10s and try again |
| [21:15:36] | Beirdo: | I would expect that by the time mythcommflag is started, we'd have most likely recorded at least 1 byte |
| [21:16:02] | iamlindoro: | dblain: I figured that's what you had in mind. Will do so but probably worth noting somewhere? It would be easy to miss if you didn't know any better |
| [21:16:03] | Beirdo: | but we shall see. If it needs tweaking, then we can tweak it :) |
| [21:16:07] | dblain: | iamlindoro: Also, each data contract has it's own version attribute that should be bumped when the class is changed. |
| [21:21:27] | Captain_Murdoch: | Beirdo, yeah, fine to wait. with the jobqueue work that wagnerrp is doing, it could mean jobs run much sooner since there will be a true job scheduler. |
| [21:21:33] | dblain: | wagnerrp: I figured the version is useful as an easy check for method signature/availablity without parsing any service description xml. But I agree that the returned data should be more forgiving since it's fully described (well at least xml & json are, who knows what other formats may be supported in the future) |
| [21:24:28] | wagnerrp: | Captain_Murdoch: i was actually intending to run the jobqueue scheduler threadless |
| [21:24:40] | wagnerrp: | pick up events off the start and end of recordings, to run automatic jobs |
| [21:25:19] | wagnerrp: | although now that i think about it, there would have to be something to run jobs scheduled for a certain time |
| [21:25:27] | stuartm: | that assumes that all jobs will be triggered by recordings? |
| [21:25:36] | dblain: | iamlindoro: true, maybe a simple comment at the top of the service classes stressing when to bump. Overall, the web service approach is faily forgiving... it will use sain |
| [21:25:52] | wagnerrp: | stuartm: no, but it does assume all write access to the jobqueue tables would be through the jobqueue code |
| [21:26:03] | dblain: | iamlindoro: saine defaults for any parameter not passed. |
| [21:26:08] | wagnerrp: | meaning if a user wanted to run a job from the frontend |
| [21:26:25] | wagnerrp: | it would send a command to the master backend with the relevant data, and the master backend would then schedule the job |
| [21:27:09] | stuartm: | k |
| [21:27:25] | Beirdo: | sounds cool |
| [21:28:12] | stuartm: | wagnerrp: I'm all for taking away direct access to tables for obvious reasons |
| [21:29:47] | wagnerrp: | i could just have it so that jobs marked to be run along side a recording get delayed by a minute, rather than started immediately |
| [21:30:09] | stuartm: | have there been any recent audio related regressions? I'm noticing a slight stutter/static type issue, especially with HD recordings |
| [21:30:44] | stuartm: | "ALSA, Error: Unable to sufficiently increase ALSA hardware buffer size – underruns are likely" << Although they've never been an issue before ... |
| [21:31:09] | Beirdo: | that stuff did get tweaked around a bit lately |
| [21:31:31] | wagnerrp: | however i do not intend to have any special handling of either the commflagger or transcoder |
| [21:33:32] | stuartm: | there's been a high volume of commits lately and I've not had the time to keep up with them all |
| [21:34:08] | Chutt (Chutt!~ijr@cpe-76-190-194-229.neo.res.rr.com) has quit (Ping timeout: 246 seconds) | |
| [21:34:32] | stuartm: | regressions are annoying :( |
| [21:38:13] | natanojl (natanojl!~jonatan@c83-252-237-63.bredband.comhem.se) has quit (Read error: Operation timed out) | |
| [21:47:07] | Kunalagon (Kunalagon!~Kunalagon@212.200.241.12) has joined #mythtv | |
| [21:53:30] | martin_ (martin_!~quassel@h-39-23.A155.priv.bahnhof.se) has joined #mythtv | |
| [22:06:24] | jmartens (jmartens!~jmartens@s5597ca60.adsl.wanadoo.nl) has quit (Quit: Leaving.) | |
| [22:11:36] | Kunalagon (Kunalagon!~Kunalagon@212.200.241.12) has quit (Quit: Leaving.) | |
| [22:41:11] | stuarta: | there's a buglet |
| [22:41:52] | stuarta: | after scanning for channels i leave mythtv setup and it tells me my starting channels are invalid |
| [22:42:36] | stuarta: | and asks me if i want to fix is. Default is "yes please", other option is "no, i know what i'm doing" |
| [22:42:47] | stuarta: | it's impossible to select no |
| [22:46:14] | iamlindoro: | if (!UserActuallyKnowsWhatHeIsDoing()) { m_noButton->Disable(); } |
| [22:46:16] | iamlindoro: | ;) |
| [22:46:33] | stuarta: | :-p |
| [22:46:44] | iamlindoro: | hehe, I had to say it no matter who said so :) |
| [22:46:50] | iamlindoro: | The opportunity was just too good |
| [22:46:53] | stuarta: | meanie |
| [22:46:57] | iamlindoro: | awww |
| [22:47:06] | ** iamlindoro hugs it out with stuarta ** | |
| [22:47:16] | stuarta: | i'll settle for a beer |
| [22:47:22] | ** stuarta beers iamlindoro ** | |
| [22:47:28] | ** iamlindoro reciprocates ** | |
| [22:54:29] | j-rod is now known as j-rod|afk | |
| [23:08:21] | abqjp (abqjp!~abqjp@67-0-78-175.albq.qwest.net) has joined #mythtv | |
| [23:19:36] | martin_ (martin_!~quassel@h-39-23.A155.priv.bahnhof.se) has quit (Remote host closed the connection) | |
| [23:37:23] | MavT (MavT!~MaverickT@111.86.233.220.static.exetel.com.au) has joined #mythtv | |
| [23:41:06] | MaverickTech (MaverickTech!~MaverickT@111.86.233.220.static.exetel.com.au) has quit (Ping timeout: 240 seconds) | |
IRC Logs collected by
BeirdoBot.
Please use the above link to report any bugs.