| Wednesday, October 20th, 2021, 00:40 UTC | ||
| [00:40:29] | peterbennett (peterbennett!~peter@mythtv/developer/peterbennett) has quit (Quit: Leaving.) | |
| [01:19:58] | mad_enz (mad_enz!~mad_enz@cpe00508b114c3e-cm788df7e75290.cpe.net.fido.ca) has quit (Remote host closed the connection) | |
| [01:36:32] | mad_enz (mad_enz!~mad_enz@cpe00508b114c3e-cm788df7e75290.cpe.net.fido.ca) has joined #mythtv | |
| [02:13:28] | zbot (zbot!~limnoria@2601:647:4802:4c3:222:4dff:fe51:6728) has quit (Remote host closed the connection) | |
| [02:16:25] | zbot (zbot!~limnoria@2601:647:4802:4c3:222:4dff:fe51:6728) has joined #mythtv | |
| [02:44:08] | hampton: | 1 and 2 seems like they could be fixed by pushing the memory allocation later in the code by 15 or so lines. |
| [02:45:14] | hampton (hampton!~david@mythtv/developer/hampton) has left #mythtv () | |
| [02:45:26] | hampton (hampton!~david@mythtv/developer/hampton) has joined #mythtv | |
| [02:45:26] | Mode for #mythtv by ChanServ!ChanServ@services.libera.chat : +v hampton | |
| [02:48:02] | hampton: | 6 seems like a bug in cppcheck. m_crid is defined as 'std::array<const uint8_t*,8>', so m_crid[n] is a 'const uint8_t*', and m_crid[n][x] is an 'uint8_t'. That's definitely not a pointer. Possibly rewriting that one line as two might eliminate the warning. |
| [02:48:36] | hampton: | Not sure about 3. |
| [05:08:08] | Steve-Goodey (Steve-Goodey!~steve@2a00:23c5:7d83:6501:6ea1:20d:5872:d872) has joined #mythtv | |
| [06:56:16] | gregl_ (gregl_!~gregl@cpe-24-194-235-95.nycap.res.rr.com) has joined #mythtv | |
| [07:09:50] | gregl (gregl!~gregl@cpe-24-194-235-95.nycap.res.rr.com) has quit (Ping timeout: 260 seconds) | |
| [07:09:50] | stuarta[m] (stuarta[m]!~stuartama@2001:470:69fc:105::fff2) has quit (Ping timeout: 260 seconds) | |
| [07:15:07] | stuarta[m] (stuarta[m]!~stuartama@2001:470:69fc:105::fff2) has joined #mythtv | |
| [07:23:37] | Steve-Goodey (Steve-Goodey!~steve@2a00:23c5:7d83:6501:6ea1:20d:5872:d872) has quit (Quit: Konversation terminated!) | |
| [07:52:54] | stuarta: | hampton: that's why i found them interesting.... and curious |
| [08:49:36] | Steve-Goodey (Steve-Goodey!~steve@2a00:23c5:7d83:6501:6ea1:20d:5872:d872) has joined #mythtv | |
| [09:06:48] | DevMythNotifyBot (DevMythNotifyBot!~nodebot@cust32-dsl91-135-4.idnet.net) has quit (Remote host closed the connection) | |
| [09:06:55] | DevMythNotifyBot (DevMythNotifyBot!~nodebot@cust32-dsl91-135-4.idnet.net) has joined #mythtv | |
| [10:02:10] | SteveGoodey (SteveGoodey!~steve@2a00:23c5:7d83:6501:fa68:6631:5ed6:d871) has joined #mythtv | |
| [10:24:06] | Steve-Goodey (Steve-Goodey!~steve@2a00:23c5:7d83:6501:6ea1:20d:5872:d872) has quit (Quit: Konversation terminated!) | |
| [10:34:00] | SteveGoodey (SteveGoodey!~steve@2a00:23c5:7d83:6501:fa68:6631:5ed6:d871) has quit (Quit: Konversation terminated!) | |
| [10:58:05] | paul-h (paul-h!~paul@pres-25-b2-v4wan-161467-cust2666.vm29.cable.virginm.net) has joined #mythtv | |
| [10:58:05] | paul-h (paul-h!~paul@mythtv/developer/paul-h) has joined #mythtv | |
| [10:58:05] | paul-h (paul-h!~paul@pres-25-b2-v4wan-161467-cust2666.vm29.cable.virginm.net) has quit (Changing host) | |
| [10:58:05] | Mode for #mythtv by ChanServ!ChanServ@services.libera.chat : +v paul-h | |
| [10:58:58] | paul-h: | Just noticed all the Ubuntu master builds failed last night :( |
| [11:00:03] | paul-h: | configure complaining about missing soundtouch 1.8.0 or later |
| [11:00:59] | stuarta: | soundtouch is now a dep, and the dev package a builddep |
| [11:01:23] | stuarta: | add those and it'll go back to building |
| [11:01:46] | paul-h: | OK – was that recent change I must of missed that |
| [11:02:07] | paul-h: | Don't we use our own copy though |
| [11:02:14] | stuarta: | https://github.com/MythTV/mythtv/commit/fe572 . . . fddb86f07a0e |
| [11:02:29] | stuarta: | our own copy was *way* behind |
| [11:02:38] | stuarta: | https://github.com/MythTV/mythtv/issues/401 |
| [11:07:44] | stuarta: | paul-h: package is called libsoundtouch-dev on ununtu |
| [11:12:21] | paul-h: | Yeah I'll add it later |
| [11:13:21] | paul-h: | Strange it's only just started failing unless I missed the earlier emails |
| [11:15:08] | stuarta: | change was only 6 days ago |
| [11:32:22] | DevMythNotifyBot (DevMythNotifyBot!~nodebot@cust32-dsl91-135-4.idnet.net) has quit (Read error: Connection reset by peer) | |
| [11:32:35] | DevMythNotifyBot (DevMythNotifyBot!~nodebot@cust32-dsl91-135-4.idnet.net) has joined #mythtv | |
| [11:34:27] | hampton: | I'm very happy with the change to external soundtouch. I used to always get a volume jump when I changed the playback speed. That's gone now with the external library. :) |
| [11:36:25] | Steve-Goodey (Steve-Goodey!~steve@2a00:23c5:7d83:6501:6ea1:20d:5872:d872) has joined #mythtv | |
| [11:37:34] | stuarta: | \o/ |
| [11:39:03] | paul-h: | Is it only used by the frontend? just wondering where to put the runtime dependency |
| [11:40:49] | hampton: | Its used from libs/libmyth/audio. |
| [12:00:03] | paul-h: | Hopefully that's fixed it. It will be at least an hour before we find out |
| [12:00:53] | paul-h: | ${shlibs:Depends} should in theory pull in the correct runtime library so just added the build time dependency |
| [12:01:58] | paul-h: | Also added jammy to the master (32) builds |
| [12:04:31] | paul-h: | Looks like Jammy will be a LTS release so we should start to think about releasing v32 in time for it? |
| [12:06:54] | paul-h: | Ideally we would release 32.0 early and plan to get 32.1 in the LTS release with any bug fixes found already fixed rather than the usual shambles we have :) |
| [12:14:52] | paul-h (paul-h!~paul@mythtv/developer/paul-h) has quit (Quit: Konversation terminated!) | |
| [12:46:45] | stuarta: | i wonder if we are now reaching a point where we should merge devel/http back into master |
| [12:47:23] | stuarta: | i'm fairly sure the v2 api is now feature complete compared to the old one |
| [12:47:37] | stuarta: | the web app is unfinished, but that could continue to be worked on in master |
| [12:48:23] | stuarta: | we are now left with the SSDP/UPnP separation work |
| [12:49:20] | stuarta: | as well as needing to work out how to make the backend startup without a DB even configured, but enough of the API up to configure it, and move onto other bits of the setup |
| [13:19:59] | peterbennett (peterbennett!~peter@mythtv/developer/peterbennett) has joined #mythtv | |
| [13:19:59] | Mode for #mythtv by ChanServ!ChanServ@services.libera.chat : +v peterbennett | |
| [13:34:01] | peterbennett (peterbennett!~peter@mythtv/developer/peterbennett) has quit (Remote host closed the connection) | |
| [13:34:45] | peterbennett (peterbennett!~peter@mythtv/developer/peterbennett) has joined #mythtv | |
| [13:34:45] | Mode for #mythtv by ChanServ!ChanServ@services.libera.chat : +v peterbennett | |
| [13:55:36] | peterbennett (peterbennett!~peter@mythtv/developer/peterbennett) has quit (Remote host closed the connection) | |
| [13:56:19] | peterbennett (peterbennett!~peter@mythtv/developer/peterbennett) has joined #mythtv | |
| [13:56:19] | Mode for #mythtv by ChanServ!ChanServ@services.libera.chat : +v peterbennett | |
| [14:06:07] | peterbennett (peterbennett!~peter@mythtv/developer/peterbennett) has quit (Quit: Leaving.) | |
| [14:21:31] | warpme__ (warpme__!uid391875@id-391875.helmsley.irccloud.com) has quit (Ping timeout: 252 seconds) | |
| [14:22:39] | warpme__ (warpme__!uid391875@helmsley.irccloud.com) has joined #mythtv | |
| [14:37:18] | gregl_ (gregl_!~gregl@cpe-24-194-235-95.nycap.res.rr.com) has quit (Remote host closed the connection) | |
| [14:40:13] | peterbennett (peterbennett!~peter@mythtv/developer/peterbennett) has joined #mythtv | |
| [14:40:13] | Mode for #mythtv by ChanServ!ChanServ@services.libera.chat : +v peterbennett | |
| [14:44:22] | DevMythNotifyBot: | COMMIT: billmeek pushed 1 commit(s) to branch devel/http: https://github.com/MythTV/mythtv/compare/6208 . . . b27d1c4640fa |
| [15:04:15] | peterbennett: | New Status/GetBackendStatus method will give the same info as Status/xml but using tags for all values instead of attributes |
| [15:06:08] | stuarta: | peterbennett: nice, looking forward to it! |
| [15:07:15] | peterbennett: | The choice is whether to use the existing method FillStatusXML and then extract the values from the dom document to populate the GetBackendStatus output, or make the new method stand alone so we can evenetulayyy delete the old method FillStatusXML |
| [15:07:44] | peterbennett: | s/evenetulayyy/eventually/ |
| [15:08:31] | stuarta: | peterbennett: is it possible to have 1 method which populates the data, the serializers will do the conversion for the new code, and FillStatusXML would have to convert? or do we just duplicate for now? |
| [15:08:38] | peterbennett: | The existing GetStatusHTML uses the dom document and extracts the data for the html reult |
| [15:10:25] | peterbennett: | I am leaning towards the new method populate the data into the new structure and then we could convert the FillXML and HTML at some point. |
| [15:12:10] | peterbennett: | The new structure at least would have standard names and Get mothods unlike the dom document which requires obscure calls to get the data out. |
| [15:14:30] | stuarta: | yes that was my thinking too |
| [15:16:41] | peterbennett: | There is a bug/restriction in the design – If you have a structure that is optional inside a response, it gets populated with default vales when not used |
| [15:17:16] | peterbennett: | For example GetEncoderList populats the encoder with a default recording in progress even if nothing is recording. |
| [15:17:17] | stuarta: | the old or new? |
| [15:17:41] | peterbennett: | I think both |
| [15:17:51] | stuarta: | well at least it's consistent :-/ |
| [15:17:56] | peterbennett: | Both old and new |
| [15:19:15] | peterbennett: | The thing is – the old nonstandard status also returns encoder list and tje program recording bit more sensibly it does not return a program recording if there is none |
| [15:19:36] | peterbennett: | s/bit/but/ |
| [15:20:52] | stuarta: | that code can get away with it because it hand cranks the xml |
| [15:21:20] | peterbennett: | It happens because of the implementation of the name macro that does a "new" if it is called to access a component that is nullptr. |
| [15:25:09] | peterbennett: | Anyway I will create the new method and it will have some extra bits in the output and use the standard encoder list, so there may be extra unexpected things in the output like blank programs |
| [15:25:39] | stuarta: | pretty sure that's totally valid in json |
| [15:25:49] | stuarta: | eg. stuff: {} |
| [15:26:20] | peterbennett: | But the blan programs come with all the blank subfields also |
| [15:26:59] | stuarta: | hmmm, we may need to work out how to suppress that |
| [15:29:30] | peterbennett: | how do I get curl to give json back? |
| [15:30:48] | peterbennett: | Ah got it -H "Accept: application/json" |
| [15:32:33] | peterbennett: | We get stuff like "Recording": {"StartTime": null, "EndTime": null, "Title": null, "SubTitle": null, "Category": null, |
| [15:34:42] | stuarta: | if you pipe it through "jq ." it'll colourize and/or complain if it's malformed |
| [15:34:50] | peterbennett: | That is from the GetEncoderList |
| [15:34:50] | stuarta: | (you may need to install the package) |
| [15:35:59] | peterbennett: | ah yes that looks nicer |
| [15:36:03] | stuarta: | :-) |
| [15:36:12] | stuarta: | it's part of my essential toolkit |
| [15:36:29] | stuarta: | along with json_reformat |
| [15:37:04] | peterbennett: | Nothing malformed, just a few hundred spurious null fields |
| [15:37:11] | stuarta: | progress :) |
| [15:37:59] | peterbennett: | what is json_reformat ? |
| [15:39:13] | stuarta: | it's a tool for reformatting json to make it more readable, the main use i have for it is syntax checking. jq is the day to day goto |
| [15:39:14] | peterbennett: | i have json_pp and json_xs |
| [15:40:02] | peterbennett: | json_pp deos the same as jq |
| [15:40:12] | stuarta: | i'd expect that |
| [15:40:33] | stuarta: | jq is useful if you wish to pull specific bits of data from the json |
| [15:40:48] | peterbennett: | json_pp uses 2 spaces for indent where jq uses 1 |
| [15:43:38] | stuarta: | `jq .` is shorthand for everything. `jq .object1.subobject2` pulls object1 subobject2 out of the json |
| [15:44:03] | stuarta: | there is a whole web page on jq filtering if you want to explore the topic |
| [15:44:56] | stuarta: | so compare `/Myth/GetHostName | jq .` with `/Myth/GetHostName | jq .HostName` |
| [15:45:00] | peterbennett: | I saw the man page It is very chatty :) |
| [15:45:13] | stuarta: | that's a simple example to illustrate ^^^ |
| [15:45:55] | stuarta: | with complex json responses it can be a lifesaver to find what you need |
| [15:59:48] | gregl (gregl!~gregl@cpe-24-194-235-95.nycap.res.rr.com) has joined #mythtv | |
| [16:05:11] | peterbennett (peterbennett!~peter@mythtv/developer/peterbennett) has quit (Ping timeout: 264 seconds) | |
| [16:20:20] | peterbennett (peterbennett!~peter@mythtv/developer/peterbennett) has joined #mythtv | |
| [16:20:20] | Mode for #mythtv by ChanServ!ChanServ@services.libera.chat : +v peterbennett | |
| [17:21:13] | peterbennett (peterbennett!~peter@mythtv/developer/peterbennett) has quit (Quit: Leaving.) | |
| [17:58:44] | peterbennett (peterbennett!~peter@mythtv/developer/peterbennett) has joined #mythtv | |
| [17:58:44] | Mode for #mythtv by ChanServ!ChanServ@services.libera.chat : +v peterbennett | |
| [18:10:29] | peterbennett (peterbennett!~peter@mythtv/developer/peterbennett) has quit (Quit: Leaving.) | |
| [19:27:07] | peterbennett (peterbennett!~peter@mythtv/developer/peterbennett) has joined #mythtv | |
| [19:27:07] | Mode for #mythtv by ChanServ!ChanServ@services.libera.chat : +v peterbennett | |
| [20:18:46] | peterbennett (peterbennett!~peter@mythtv/developer/peterbennett) has quit (Quit: Leaving.) | |
| [20:22:23] | Steve-Goodey (Steve-Goodey!~steve@2a00:23c5:7d83:6501:6ea1:20d:5872:d872) has quit (Quit: Konversation terminated!) | |
| [20:28:37] | m6wiq (m6wiq!~m6wiq@51.148.147.13) has joined #mythtv | |
| [20:29:10] | m6wiq (m6wiq!~m6wiq@51.148.147.13) has quit (Client Quit) | |
| [20:41:43] | peterbennett (peterbennett!~peter@mythtv/developer/peterbennett) has joined #mythtv | |
| [20:41:43] | Mode for #mythtv by ChanServ!ChanServ@services.libera.chat : +v peterbennett | |
| [21:57:35] | peterbennett (peterbennett!~peter@mythtv/developer/peterbennett) has quit (Quit: Leaving.) | |
| [22:50:04] | warpme__ (warpme__!uid391875@helmsley.irccloud.com) has quit (Quit: Connection closed for inactivity) | |
| [23:41:19] | peterbennett (peterbennett!~peter@mythtv/developer/peterbennett) has joined #mythtv | |
| [23:41:19] | Mode for #mythtv by ChanServ!ChanServ@services.libera.chat : +v peterbennett | |
IRC Logs collected by
BeirdoBot.
Please use the above link to report any bugs.