MythLogBot@irc.freenode.net :: #mythtv

Daily chat history

Current users (67):

coling, ElmerFudd, MythLogBot, Steve_Goodey, tonsofpcs, aberrios_, aloril, Anssi, caelor_, Captain_Murdoch, clever, Cougar, dblain, dekarl, eee-blt, fetzerch, gregL, GreyFoxx, jams, jarryd, jnylen, jst, jwhite, jya, kormoc, moparsthbest, MythBuild, peper03, purserj, rsiebert, Sharky112065, sheedy-away, sphery, stuarta, tris, unforgiven512, wagnerrp, Warped, wseltzer1, xris, poptix, zentec, stuartm, _charly_, jpharvey, sraue, kurre2, robink, adamw, jpabq, joki, amessina_, nephyrin`, rhpot199`, Chutt, J-e-f-f-A, superm1, jheizer, Casper0082, toeb, jarle, enyc_, Gibby, emm386_, XDS2010, knightr__, cesman
Thursday, September 4th, 2014, 00:06 UTC
[00:06:48] knightr_ (knightr_!~Nicolas@69-165-170-178.dsl.teksavvy.com) has joined #mythtv
[00:09:05] jya__ (jya__!~jyavenard@mythtv/developer/jya) has joined #mythtv
[00:09:29] jya__ (jya__!~jyavenard@mythtv/developer/jya) has quit (Client Quit)
[00:09:46] cecil__ (cecil__!~cesman@pool-173-60-115-40.lsanca.fios.verizon.net) has joined #mythtv
[00:10:16] jya (jya!~jya@mythtv/developer/jya) has left #mythtv ()
[00:11:43] emm386_ (emm386_!~user@classg.net) has joined #mythtv
[00:11:49] zentec (zentec!~zentec@71.82.202.47) has joined #mythtv
[00:12:10] joki (joki!~joki@p54860B64.dip0.t-ipconnect.de) has quit (Ping timeout: 255 seconds)
[00:12:11] Tobbe5178 (Tobbe5178!~asdf@h104n2-sv-a13.ias.bredband.telia.com) has quit (Ping timeout: 243 seconds)
[00:12:11] jya_ (jya_!~jyavenard@mythtv/developer/jya) has quit (Ping timeout: 243 seconds)
[00:12:11] wagnerrp (wagnerrp!~wagnerrp_@mythtv/developer/wagnerrp) has quit (Ping timeout: 255 seconds)
[00:12:11] J-e-f-f-A (J-e-f-f-A!~J-e-f-f-A@24-177-48-184.dhcp.oxfr.ma.charter.com) has quit (Ping timeout: 255 seconds)
[00:12:11] knightr (knightr!~Nicolas@mythtv/developer/knightr) has quit (Ping timeout: 255 seconds)
[00:12:11] zentec_ (zentec_!~zentec@71.82.202.47) has quit (Ping timeout: 255 seconds)
[00:12:11] emm386 (emm386!~user@pdpc/supporter/active/emm386) has quit (Ping timeout: 255 seconds)
[00:12:12] joki (joki!~joki@p54860B64.dip0.t-ipconnect.de) has joined #mythtv
[00:12:12] cecil_ (cecil_!~cesman@pool-173-60-115-40.lsanca.fios.verizon.net) has quit (Ping timeout: 255 seconds)
[00:12:30] J-e-f-f-A (J-e-f-f-A!~J-e-f-f-A@24-177-48-184.dhcp.oxfr.ma.charter.com) has joined #mythtv
[00:12:32] jya (jya!~jya@mythtv/developer/jya) has joined #mythtv
[00:12:40] wagnerrp (wagnerrp!~wagnerrp_@mythtv/developer/wagnerrp) has joined #mythtv
[00:13:57] andreaz (andreaz!~andre_000@p5792344E.dip0.t-ipconnect.de) has quit (Read error: Connection reset by peer)
[00:20:14] sheedy-away is now known as sheedy
[00:23:00] warpme (warpme!~piotro@89-79-250-31.dynamic.chello.pl) has quit (Ping timeout: 252 seconds)
[00:52:12] joki (joki!~joki@p54860B64.dip0.t-ipconnect.de) has quit (Ping timeout: 255 seconds)
[00:52:49] joki (joki!~joki@p54860B64.dip0.t-ipconnect.de) has joined #mythtv
[01:04:53] Seeker (Seeker!~cjo20@unaffiliated/seeker) has quit (Ping timeout: 240 seconds)
[01:16:41] Seeker (Seeker!~cjo20@host86-164-169-53.range86-164.btcentralplus.com) has joined #mythtv
[01:16:41] Seeker (Seeker!~cjo20@host86-164-169-53.range86-164.btcentralplus.com) has quit (Changing host)
[01:16:41] Seeker (Seeker!~cjo20@unaffiliated/seeker) has joined #mythtv
[01:18:04] J-e-f-f-A (J-e-f-f-A!~J-e-f-f-A@24-177-48-184.dhcp.oxfr.ma.charter.com) has quit (Quit: Oh No!!!! ;-))
[01:27:41] J-e-f-f-A (J-e-f-f-A!~J-e-f-f-A@24-177-48-184.dhcp.oxfr.ma.charter.com) has joined #mythtv
[01:36:57] jheizer (jheizer!~jheizer@73.51.93.177) has quit (Quit: Leaving)
[02:36:20] sheedy (sheedy!sheedy@gateway/shell/kde/x-lrvlwhybulwtnavx) has quit (Ping timeout: 260 seconds)
[02:41:13] sheedy (sheedy!sheedy@gateway/shell/kde/x-qiaxtnyywwekakxv) has joined #mythtv
[02:52:02] sheedy is now known as sheedy-away
[02:54:36] XDS2010____ is now known as XDS2010
[02:56:02] peper03 (peper03!~peper03@mythtv/developer/peper03) has quit (Ping timeout: 276 seconds)
[02:57:06] peper03 (peper03!~peper03@mythtv/developer/peper03) has joined #mythtv
[03:10:02] fetzerch (fetzerch!~quassel@unaffiliated/fetzerch) has quit (Ping timeout: 250 seconds)
[03:11:13] fetzerch (fetzerch!~quassel@unaffiliated/fetzerch) has joined #mythtv
[03:27:26] SteveGoodey (SteveGoodey!~steve@host109-155-184-37.range109-155.btcentralplus.com) has joined #mythtv
[03:56:12] SteveGoodey (SteveGoodey!~steve@host109-155-184-37.range109-155.btcentralplus.com) has quit (Quit: Konversation terminated!)
[03:58:12] jheizer (jheizer!~jheizer@73.51.93.177) has joined #mythtv
[04:26:07] ElmerFudd (ElmerFudd!~le@87-55-166-130-static.dk.customer.tdc.net) has quit (Ping timeout: 245 seconds)
[04:40:14] arescorpio (arescorpio!~arescorpi@120-239-16-190.fibertel.com.ar) has quit (Excess Flood)
[04:53:58] ElmerFudd (ElmerFudd!~le@87-55-166-130-static.dk.customer.tdc.net) has joined #mythtv
[05:34:49] jnylen: Could anyone add http://xmltv.se (http://xmltv.xmltv.se) to Germany, Austria and Netherlands aswell?
[05:39:45] dekarl: in the wiki? happy to add germany, austria and switzerland ;)
[05:47:15] jnylen: You could add "Some channels" to NL, DE, AU and SWZ
[05:52:32] dekarl: And german speaking italy :)
[05:53:21] dekarl: not sure about Luxembourg and Liechtenstein, data on wikipedia is sparse
[05:54:22] dekarl: looks like Liechtenstein watches the neighbouring countries TV and started one local station to be able to participate in the Eurovision Song Contest only...
[05:59:05] SteveGoodey (SteveGoodey!~steve@host109-155-184-37.range109-155.btcentralplus.com) has joined #mythtv
[06:08:50] SteveGoodey (SteveGoodey!~steve@host109-155-184-37.range109-155.btcentralplus.com) has quit (Quit: Konversation terminated!)
[06:16:09] esperegu: does mythfrontend run on the banana pi? Or is there any effort to get that going?
[06:17:05] dekarl: esperegu: why wouldn't it run if it has enough memory and CPU grunt for software decoding?
[06:17:41] esperegu: dekarl: donno. its like the raspberry pi but with a dual core, 1gb mem and a gb ethernet
[06:18:10] dekarl: and a VPU that does not provide a VDPAU/VAAPI API so its all software decode
[06:18:37] esperegu: dekarl: I read that the raspberry pi is not supported and therefor maybe there was no support/packages yet
[06:20:26] dekarl: esperegu: we don't support anything based on that definition (only providing source code to packagers)
[06:21:05] dekarl: the raspberry pi can also run the frontend in principle but its super limited due to lack of memory and the lack of decent CPU makes the missing support for the VPU a killer
[06:23:11] dekarl: according to http://forum.lemaker.org/2872-1-1-precompiled . . . _debian.html there is an experimental VDPAU driver for the banaba pi's VPU. looking forward to hear results from tests
[06:24:09] esperegu: dekarl: then this might be hopefull for the banana? https://github.com/linux-sunxi/libvdpau-sunxi
[06:24:18] esperegu: aha. thats probably the same then
[06:40:45] FabriceMG (FabriceMG!~Thunderbi@217.112.59.207) has joined #mythtv
[07:00:56] stichnot (stichnot!stichnot@mythtv/developer/stichnot) has quit (Ping timeout: 268 seconds)
[07:03:24] XDS2010 (XDS2010!sid1218@gateway/web/irccloud.com/x-jlnnaskdadxuvgmb) has quit (Ping timeout: 260 seconds)
[07:13:25] XDS2010 (XDS2010!sid1218@gateway/web/irccloud.com/x-bfznmxuiyknmetem) has joined #mythtv
[08:13:14] Merlin83b (Merlin83b!~Daniel@2a00:1ee0:3:1337:45f3:471f:f8b0:11e6) has joined #mythtv
[08:43:37] joki (joki!~joki@p54860B64.dip0.t-ipconnect.de) has quit (Ping timeout: 245 seconds)
[08:48:55] joki (joki!~joki@p548626C0.dip0.t-ipconnect.de) has joined #mythtv
[10:07:22] len_ (len_!~quassel@75-168-45-25.mpls.qwest.net) has quit (Read error: Connection reset by peer)
[10:24:22] stuarta: round 2 of vm perf tuning done
[10:26:56] stuarta: jya: were you planning to commit the fix you've been sent which resolves the unset SRC_PATH_BARE ??
[10:27:10] stuarta: on f20 qt5 builds.
[10:46:06] ** stuarta needs to explore the cross platform virt for an arm builder **
[11:05:21] jya: stuarta: will do asap
[11:05:42] stuarta: jya: thanks very much
[11:05:47] stuarta: it's appreciated
[11:14:03] jya: not done yet :)
[11:48:40] amessina_ (amessina_!~amessina@2001:470:c1dc:7779:d6be:d9ff:fe8d:7c1e) has joined #mythtv
[11:50:18] peper03_ (peper03_!~peper03@mythtv/developer/peper03) has joined #mythtv
[11:50:58] peper03 (peper03!~peper03@mythtv/developer/peper03) has quit (Ping timeout: 260 seconds)
[11:50:59] gigem (gigem!~david@mythtv/developer/gigem) has quit (Ping timeout: 260 seconds)
[11:50:59] aloril (aloril!~aloril@dsl-tkubrasgw2-54faa3-2.dhcp.inet.fi) has quit (Ping timeout: 260 seconds)
[11:50:59] amessina (amessina!~amessina@2001:470:c1dc:7779:d6be:d9ff:fe8d:7c1e) has quit (Ping timeout: 260 seconds)
[11:51:01] peper03_ is now known as peper03
[11:51:10] gigem (gigem!~david@pool-71-170-165-247.dllstx.fios.verizon.net) has joined #mythtv
[11:51:11] gigem (gigem!~david@pool-71-170-165-247.dllstx.fios.verizon.net) has quit (Changing host)
[11:51:11] gigem (gigem!~david@mythtv/developer/gigem) has joined #mythtv
[11:54:39] aloril (aloril!~aloril@dsl-tkubrasgw2-54faa3-2.dhcp.inet.fi) has joined #mythtv
[12:13:38] rsiebert (rsiebert!~quassel@g225156229.adsl.alicedsl.de) has joined #mythtv
[12:16:27] rsiebert_ (rsiebert_!~quassel@g225046199.adsl.alicedsl.de) has quit (Ping timeout: 246 seconds)
[12:19:41] jya: #12261 “Scientific method helped me”. I’m not even sure the guy is joking...
[12:19:41] ** MythLogBot http://code.mythtv.org/trac/ticket/12261 **
[12:20:36] jya: the hostname is only ever resolved for the point of knowing how to created the QTcpSocket.. So wtf is he on about…
[12:21:42] stuarta: and type in the url? what is he smoking?
[13:23:40] sheedy-away is now known as sheedy
[13:33:29] stuartm: jya: there still is an issue I've yet to track down with the services API where it's treating an IP address as a hostname and falling to do a lookup
[13:38:00] stuartm: "2014-09–04 14:37:47.724929 E No address defined for host: 192.168.159.2"
[13:38:13] stuartm: ok, seems related to album artwork
[13:42:07] stuartm: hmm, seems like it might be a combination of things, 1) the assumption that the 'host' in myth:// urls is a hostname (it's not) and the fact that some code somewhere is supplying an IP address in myth urls
[13:44:15] stuartm: if you assume that the 'host' in myth:// urls is a hostname then it should also accept IP addresses, but that then breaks the storage group code which assumes 'hostname' is the unique identifier in the database
[13:54:55] stuartm: what a pretty mess
[14:02:27] esperegu (esperegu!~quassel@ip-213-124-221-127.ip.prioritytelecom.net) has quit (Read error: No route to host)
[14:03:12] esperegu (esperegu!~quassel@ip-213-124-221-127.ip.prioritytelecom.net) has joined #mythtv
[14:06:06] stichnot (stichnot!~stichnot@mythtv/developer/stichnot) has joined #mythtv
[14:06:15] knightr__ (knightr__!~Nicolas@69-165-170-178.dsl.teksavvy.com) has joined #mythtv
[14:08:52] Jordack (Jordack!~Jordack@75-151-31-172-Michigan.hfc.comcastbusiness.net) has joined #mythtv
[14:12:10] stuartm: this patch should fix the issue I'm seeing (untested) – http://pastebin.com/52hv4V9G
[14:13:15] stuartm: specifically the second half, the first half just removes another invalid IP for 'hostname' replacement
[14:14:24] stuartm: basically the storage group code expects a hostname, if you give it an IP it doesn't know what to do with it and can't currently reverse it back to a valid identifier
[14:16:49] stuartm: basically https://github.com/MythTV/mythtv/commit/520ea17e is wrong
[14:17:01] knightr_ (knightr_!~Nicolas@69-165-170-178.dsl.teksavvy.com) has quit (*.net *.split)
[14:17:58] stuartm: hmm, actually it's not, I apologise, it was just replacing broken code with similarly broken code
[14:18:13] stuarta: easier to fix
[14:21:14] stichnot (stichnot!~stichnot@mythtv/developer/stichnot) has quit (Ping timeout: 268 seconds)
[14:22:27] stuartm: stuarta: not so sure, there is probably code which has evolved to expect the 'hostname' portion of a myth:// url to be an IP or an actual hostname
[14:23:29] stuartm: we should have renamed hostname internally to something else years ago instead of letting more and more code build upon it, and letting the confusion continue
[14:23:41] stuarta: needs a broom through the whole lot
[14:24:06] stuartm: it definitely needs a coordinated, cooperative effort to fix
[14:26:51] stuartm: 1) rename all host/hostname columns in the database to mythdeviceid (deviceid) or something 2) Fixup all the sql 3) Rename all the variables 4) Add guards to prevent an IP being inserted or compared against 'deviceid' 5) Fix all the code which breaks as a result
[14:27:42] stuartm: 5 would require some changes of it's own, small scale refactors
[14:29:49] stuartm: we'd probably want to create a new device table, listing all know 'devices*', their unique ids, their 'friendly' names so people can change names without breaking configs, and importantly the IP address (updated on startup) of the device
[14:32:35] stuartm: it'll never happen though, we've not managed to organise a group activity like that for years :(
[14:33:28] knightr_ (knightr_!~Nicolas@69-165-170-178.dsl.teksavvy.com) has joined #mythtv
[14:34:05] knightr_ (knightr_!~Nicolas@69-165-170-178.dsl.teksavvy.com) has quit (Ping timeout: 244 seconds)
[14:42:46] stichnot (stichnot!~stichnot@216.239.45.89) has joined #mythtv
[14:42:46] stichnot (stichnot!~stichnot@216.239.45.89) has quit (Changing host)
[14:42:46] stichnot (stichnot!~stichnot@mythtv/developer/stichnot) has joined #mythtv
[14:58:56] stuartm: in fact ignore every I said about ip vs hostname in myth:// urls, decided I'm wrong about that (or at least not 100% right)
[14:59:10] stuartm: got a better fix for the immediate mythmusic issue
[15:25:04] sheedy is now known as sheedy-away
[15:58:32] FabriceMG (FabriceMG!~Thunderbi@217.112.59.207) has quit (Quit: FabriceMG)
[16:01:14] sphery (sphery!~mdean@mythtv/developer/sphery) has quit (Quit: leaving)
[16:02:04] sphery (sphery!~mdean@mythtv/developer/sphery) has joined #mythtv
[16:21:05] SteveGoodey (SteveGoodey!~steve@host109-155-184-37.range109-155.btcentralplus.com) has joined #mythtv
[16:29:29] Steve-Goodey (Steve-Goodey!~steve@host109-155-184-37.range109-155.btcentralplus.com) has joined #mythtv
[16:30:07] SteveGoodey (SteveGoodey!~steve@host109-155-184-37.range109-155.btcentralplus.com) has quit (Ping timeout: 268 seconds)
[16:40:20] Steve-Goodey (Steve-Goodey!~steve@host109-155-184-37.range109-155.btcentralplus.com) has quit (Ping timeout: 255 seconds)
[16:40:25] Steve_Goodey (Steve_Goodey!~steve@host86-132-99-123.range86-132.btcentralplus.com) has joined #mythtv
[16:43:21] dekarl1 (dekarl1!~dekarl@p4FCEEE0C.dip0.t-ipconnect.de) has joined #mythtv
[16:43:30] dekarl (dekarl!~dekarl@p4FE843DC.dip0.t-ipconnect.de) has quit (Ping timeout: 246 seconds)
[16:49:06] dekarl1 (dekarl1!~dekarl@p4FCEEE0C.dip0.t-ipconnect.de) has quit (Ping timeout: 246 seconds)
[16:49:07] dekarl (dekarl!~dekarl@p4FE84A98.dip0.t-ipconnect.de) has joined #mythtv
[16:59:48] dekarl: stuartm isn't that (dnshostname, ip, mythtvsystemname mixup) a good reason to move everything to helper functions like IsThisME(something) and IsThisTheMasterBackend(something)?
[17:03:45] stuartm: dekarl: that would help, although the current IsThisHost() stuff needs some additional work, there's nothing to stop it wasting it's time trying to look up settings in the database with an IP address
[17:05:28] stuartm: and there's still a use-case that would be confused by that setup, moving a frontend or backend to another machine (host), e.g. splitting up a combined fe/be into two devices without losing the fe or be config
[17:17:42] Merlin83b (Merlin83b!~Daniel@2a00:1ee0:3:1337:45f3:471f:f8b0:11e6) has quit (Quit: Leaving)
[18:50:02] Steve-Goodey (Steve-Goodey!~steve@host86-132-99-123.range86-132.btcentralplus.com) has joined #mythtv
[19:32:02] adamw (adamw!~adamw@redhat/adamw) has quit (Read error: Connection reset by peer)
[19:32:25] adamw (adamw!~adamw@redhat/adamw) has joined #mythtv
[19:37:48] adamw (adamw!~adamw@redhat/adamw) has quit (Read error: Connection reset by peer)
[19:38:27] adamw (adamw!~adamw@redhat/adamw) has joined #mythtv
[19:57:49] len_ (len_!~quassel@75-168-45-25.mpls.qwest.net) has joined #mythtv
[20:20:31] len_ (len_!~quassel@75-168-45-25.mpls.qwest.net) has quit (Remote host closed the connection)
[20:50:28] Steve_Goodey (Steve_Goodey!~steve@host86-132-99-123.range86-132.btcentralplus.com) has quit (Quit: Konversation terminated!)
[21:02:44] dekarl: meh... the recording rule is a actor search without inetref, the program has a proper tmbd.py_xyz inetref, but the recording get a tvdb_py interef via automatic lookup... http://www.themoviedb.org/movie/27579 -> http://thetvdb.com/?tab=series&id=80389&lid=14 oh well, at least its not showing naked people
[21:06:10] dekarl: Really need to figure out how to test / fix these cases :(
[21:12:51] esperegu (esperegu!~quassel@ip-213-124-221-127.ip.prioritytelecom.net) has quit (Remote host closed the connection)
[21:14:55] Steve-Goodey (Steve-Goodey!~steve@host86-132-99-123.range86-132.btcentralplus.com) has quit (Quit: Konversation terminated!)
[21:16:15] stuartm: dekarl: was the rule setup before the programme data contained the inetref?
[21:17:14] stuartm: err, nevermind, misread what you wrote
[21:20:11] Jordack (Jordack!~Jordack@75-151-31-172-Michigan.hfc.comcastbusiness.net) has quit ()
[21:22:21] dekarl: oh and the program's category_type=movie and subtitle is empty
[21:33:56] stuartm: dekarl: yeah, I think it falls back to using tvdb if it can't find a match through tmdb which it obviously should be doing when all signs point to it being a film
[21:34:13] stuartm: but that code is a maze
[21:58:29] andreaz (andreaz!~andre_000@p5792344E.dip0.t-ipconnect.de) has joined #mythtv
[22:01:39] jya: stuartm: I probably only have added to the confusion of hostname/deviceid
[22:01:58] jya: especially as it was never working with IPv6 so I added IPv6 handling (and resolution of such address)
[22:03:03] stuartm: we'll get it sorted out
[22:05:14] stuartm: thinking the quickest solution to the device <> IP problem is the one I floated earlier where on startup each device writes it's current IP into a table, lookups then just become a query instead of resolving from the "hostname which isn't a hostname"
[22:07:26] stuartm: another variation is that the master backend maintains the list instead of the db, which it's already doing for frontends, then frontends just as the master for the IP of a slave
[22:12:51] stuartm: good news is that with the fixes I've already committed fetching album artwork via the services API is faster and reliable, no more logs full of errors and randomly failing downloads
[22:14:03] jya: stuartm: do you think the genmythurl putting the IP fix you posted earlier should be backported to 0.27?
[22:15:56] jya: it should still work fine I think
[22:16:28] jya: I still need to test my changes to the channel number detection code...
[22:16:58] jya: I received the mac pro (8 cores, 32GB ECC, AMD D500).. Gosh that things fly
[22:17:29] jya: takes me 6 minutes to compile the entire firefox package (vs 19 minutes on my new MBA) et 12’ on my earlier 3.4GHz I7
[22:20:12] stuartm: jya: the bit in genmythurl doesn't do anything but print a warning as a way to prevent recurrences of the problem, I may cherry-pick some of the other changes though
[22:21:33] stuartm: will wait a week to see what, if anything breaks as a result of those commits
[22:21:53] jya: oh I didn’t see that you pushed some changes
[22:22:17] jya: FWIW, that whole IP address vs hostname: is what caused the biggest amount of unforeseen regressions in 0.27.2
[22:22:37] jya: there are dozen of corner cases that didn’t behave like the rest… I thought I had tested it a fair bit.
[22:22:43] jya: wasn’t enough obviously
[22:23:21] jya: https://github.com/MythTV/mythtv/commit/2e486 . . . 0d8d59c6R444 are you sure about that one?
[22:24:34] jya: I don’t think that’s right
[22:24:51] stuartm: jya: all apart from the QFile:exists()/RemoteFile::exists() change which I reverted in a later commit
[22:25:16] stuartm: oh and the "url.host() == gCoreContext->GetMasterServerIP()" bit
[22:25:19] jya: you now create a storage group as if it was the current machine ; but the url here may very well not be
[22:25:41] stuartm: both those were wrong, got a bit turned around
[22:26:34] stuartm: if (isLocal(url) || gCoreContext->IsThisBackend(host))
[22:26:40] jya: what was wrong? before or after ?
[22:26:44] stuartm: context is important
[22:26:47] jya: ah yes ok..
[22:26:51] jya: you’re right
[22:27:10] jya: but gCoreContext->GetHostname should be the same as qurl.host() there no?
[22:27:40] stuartm: that earlier commit was when I was going for a more minimal fix that only addressed the bits which were causing immediate problems, in the last commit I just went for a total fix
[22:28:02] stuartm: jya: no, it can be (and was) an IP address
[22:28:28] jya: even after your change to GetMythUrl ?
[22:28:29] stuartm: IsThisBackend() will compare against both hostname and IP
[22:28:46] jya: yes, I added that changes to fix the IPv6 issues
[22:29:27] stuartm: and mythmusic (and other places) were passing through IP addresses in myth:// urls
[22:30:21] jya: your changes look all reasonable then now…
[22:30:34] jya: I would backport that to devel/027candidates
[22:30:51] stuartm: so in that case it splits up the URL, sticks an IP into 'host' then feeds it to StorageGroup() which will lookup stuff in the database with "WHERE hostname=<ip>" which would obviously fail
[22:30:59] jya: yes
[22:31:43] jya: that probably will fix the issue of the bug where the guy used scientific method
[22:40:42] stuartm: https://code.mythtv.org/cgit/mythtv/commit/?i . . . 5280c3c2ee8e is the big one
[22:45:03] jya: do you see “IP address instead of hostname” much now ?
[22:46:10] jya: there was no need for that: if (!addr.isNull() && addr.protocol() == QAbstractSocket::IPv6Protocol) a null address would return false
[22:46:49] stuartm: yeah, I know, I was being overly pedantic
[22:48:41] stuartm: I haven't seen any instances of the warning in the logs so far, but it's still early, I've tested the bits I frequently use and I only have a combined fe/be which limits the areas of code which are being hit
[22:48:45] jya: problem though… maybe it would have been a good idea to change GetMasterHostName into something less confusing… and not replace all the “ip” with “hostname”
[22:49:04] stuartm: although I have AlwaysStreamFiles enabled
[22:49:53] stuartm: jya: I was thinking either MythDeviceID or MythHostID
[22:49:55] jya: if it’s not a “hostname” but a device id
[22:50:10] jya: what would be the difference between the two?
[22:50:19] jya: ah soorry
[22:50:22] jya: I read “and"
[22:50:31] jya: DeviceID is less confusing
[22:51:00] jya: and in that big commit, String uri = gCoreContext->GenMythURL(hostname, port, basename, could have been String uri = gCoreContext->GenMythURL(device, port, basename,
[22:51:11] jya: or ServerID
[22:51:26] jya: seeing that those are always created for connecting to a backend (either a slave or a master)
[22:53:47] stuartm: for Master/slave it would always be a Server, but I was thinking about keeping the name consistent, frontends also have a DeviceID and combined frontends and backends share the same ID at which point ServerID could become confusing again
[22:54:37] jya: what about making it even more clear: BackendID vs FrontendID. in the situation where it’s a combined BE/FE they will be the same
[22:54:46] jya: but logically the use is very different
[22:55:25] jya: and looking at the code would remove all ambiguities
[22:56:43] stuartm: yeah, could work, although what would GetHostname() become? Just GetID()? returning BackendID on a backend and FrontendID on a frontend?
[22:56:49] sheedy-away is now known as sheedy
[22:57:07] stuartm: thinking of common code here
[22:57:33] jya: GetHostname would be GetDeviceID; and if it’s a backend it returns BackendID or if frontend FrontendID
[22:58:17] stuartm: seems like we've got a plan then
[22:59:08] jya: I think that would remove all ambiguities, and cater in the case should we ever want to split FE and BE setup on a single machine
[22:59:34] stuartm: the bulk of the work will still be transitioning over the database tables and all sql queries, can't just do a search/replace on those unfortunately
[23:00:20] jya: don’t really have to modify the database
[23:00:32] stuartm: IMHO renaming the 'hostname' columns has to be done eventually
[23:00:33] jya: I would be fine with that, so long as the API to use it is clear
[23:00:52] stuartm: although it's a less immediate concern
[23:01:05] andreaz (andreaz!~andre_000@p5792344E.dip0.t-ipconnect.de) has quit (Read error: Connection reset by peer)
[23:01:06] jya: indeed
[23:27:19] sheedy is now known as sheedy-away
[23:46:27] stichnot (stichnot!~stichnot@mythtv/developer/stichnot) has quit (Ping timeout: 245 seconds)
[23:46:43] cecil__ is now known as cesman
[23:47:11] cesman (cesman!~cesman@pool-173-60-115-40.lsanca.fios.verizon.net) has quit (Changing host)
[23:47:11] cesman (cesman!~cesman@pdpc/supporter/professional/cesman) has joined #mythtv
[23:56:52] Seeker (Seeker!~cjo20@unaffiliated/seeker) has quit (Ping timeout: 252 seconds)

IRC Logs collected by BeirdoBot.
Please use the above link to report any bugs.