Beirdobot, irc.libera.chat

Monday, December 13th, 2021, 00:44 UTC

[00:44:56]

ulmus-scott (ulmus-scott!~ulmus-sco@user/ulmus-scott) has quit (Ping timeout: 256 seconds)

[00:46:03]

peterbennett (peterbennett!~peter@mythtv/developer/peterbennett) has joined #mythtv

[00:46:03]

Mode for #mythtv by ChanServ!ChanServ@services.libera.chat : +v peterbennett

[00:56:48]

ulmus-scott (ulmus-scott!~ulmus-sco@user/ulmus-scott) has joined #mythtv

[01:00:53]

peterbennett:

ulmus-scott: Re ffmpeg stripping. A lot of the history of why and what for ffmpeg is in the heads of people no longer around.

[01:01:40]

peterbennett:

ulmus-scott: I think we avoid stripping all of the components, leaving that up to the packagers if they want to

[01:02:34]

Steve-Goodey (Steve-Goodey!~steve@2a00:23c5:7d83:6501:6ea1:20d:5872:d872) has joined #mythtv

[01:04:13]

peterbennett:

The packagers may package the debug data separately.

[01:04:32]

ulmus-scott:

OK. I'll leave the FFmpeg stripping disabled.

[01:04:53]

ulmus-scott:

None of the MythTV code is stripped.

[01:21:23]

amessina (amessina!~amessina@user/amessina) has quit (Read error: Connection reset by peer)

[01:35:35]

peterbennett (peterbennett!~peter@mythtv/developer/peterbennett) has quit (Quit: Leaving.)

[01:58:56]

ulmus-scott:

My earlier comment about what FFmpeg tries to strip was incorrect: FFmpeg still strips many files but it must have not needed to recompile them. (I think they are x86 assembly files.)

[02:19:30]

Steve-Goodey (Steve-Goodey!~steve@2a00:23c5:7d83:6501:6ea1:20d:5872:d872) has quit (Quit: Konversation terminated!)

[03:05:27]

ulmus-scott (ulmus-scott!~ulmus-sco@user/ulmus-scott) has quit (Ping timeout: 256 seconds)

[03:54:21]

ulmus-scott (ulmus-scott!~ulmus-sco@user/ulmus-scott) has joined #mythtv

[06:02:49]

ulmus-scott (ulmus-scott!~ulmus-sco@user/ulmus-scott) has quit (Ping timeout: 256 seconds)

[08:08:41]

Steve-Goodey (Steve-Goodey!~steve@2a00:23c5:7d83:6501:6ea1:20d:5872:d872) has joined #mythtv

[10:59:48]

Steve-Goodey (Steve-Goodey!~steve@2a00:23c5:7d83:6501:6ea1:20d:5872:d872) has quit (Quit: Konversation terminated!)

[11:11:58]

paul-h (paul-h!~paul@mythtv/developer/paul-h) has joined #mythtv

[11:11:59]

Mode for #mythtv by ChanServ!ChanServ@services.libera.chat : +v paul-h

[11:19:02]

stuarta:

paul-h: regarding your question on Angular (v2), yes it's a framework for responsive web design. the material icon font(s) are core to angular, materialize is another implementation of that for use with the svelte based frontend webapp (i'm going to replace that with an Angular one for consistency)

[11:42:40]

stuarta:

also the look and feel is added as css etc as normal

[11:43:53]

stuarta:

the framework handles all the async comms and updating components with data as it become available (aka "loading..." -> "loaded web page")

[11:44:14]

stuarta:

and allows you to build stuff up out of nice reusable components.

[11:44:36]

stuarta:

currently working on a new version of the status page, which is a good example of what can be done

[11:57:47]

paul-h:

stuarta: I was playing with it over the weekend and did manage to get the demo pages to display something :)

[11:58:58]

paul-h:

getting it to work on Ubuntu was a challenge because there are no up to date packages for the stuff that is required

[11:59:10]

stuarta:

npm install :-p

[11:59:33]

paul-h:

that's what I ended up doing

[11:59:45]

stuarta:

it's the *only* way with these frameworks

[12:00:45]

stuarta:

btw. i've made some improvements that make it much easier to develop in

[12:01:05]

stuarta:

i just haven't pushed them yet. i will look todo that tonight

[12:02:25]

paul-h:

Whatever you do don't install ng-common on Ubuntu thinking it is what you need it isn't

[12:04:08]

paul-h:

How are you testing it are you just running 'ng serve' in the correct directory or deploying it each time?

[12:06:01]

paul-h:

While I have got something to render it's not finding the icon or the services api correctly and the text is too big so I think it's not finding the css or something

[12:07:34]

paul-h:

I'm not running the server on my backend server so not surprising it's not finding the services API correctly

[12:09:43]

paul-h:

I'm just doing everything from my desktop pc by changing into the mythtv/mythtv/html/backend directory and running ng serve which is probably not the correct way to do it?

[12:10:53]

stuarta:

that's the problem i've just fixed to allow development

[12:11:26]

stuarta:

until I push the changes you'll need to do `npm run build` and a make install in the html directory

[12:11:46]

stuarta:

and visit your backend new http server port to see it

[12:13:11]

stuarta:

paul-h: pm

[12:31:37]

stuarta:

sheesh, the time to retire mythweb thread is getting on my wick. just test the bloody PR and then we can merge it

[12:55:00]

paul-h (paul-h!~paul@mythtv/developer/paul-h) has quit (Quit: Konversation terminated!)

[13:28:48]

Steve-Goodey (Steve-Goodey!~steve@2a00:23c5:7d83:6501:6ea1:20d:5872:d872) has joined #mythtv

[13:44:13]

jpabq_ (jpabq_!~quassel@75.161.232.75) has quit (Remote host closed the connection)

[13:55:55]

peterbennett (peterbennett!~peter@mythtv/developer/peterbennett) has joined #mythtv

[13:55:55]

Mode for #mythtv by ChanServ!ChanServ@services.libera.chat : +v peterbennett

[14:10:27]

peterbennett:

stuarta: I plan to test the mythweb PR with both php7.4 and php8 . If all ok I can merge it

[14:11:33]

stuarta:

peterbennett: go for it

[14:12:12]

stuarta:

i know i kinda inherited maintainership of it, but i just don't have the time to even look at it

[14:12:35]

peterbennett:

stuarta: I did not get any answer to my services question about version numbers and compatibility.

[14:12:57]

stuarta:

the real question is whether any client actually check that

[14:13:13]

stuarta:

i have my doubts

[14:13:29]

stuarta:

the safest path is to do a patch level increment on the versions

[14:13:35]

peterbennett:

So I think I should set the version numbers as they were before.

[14:13:40]

stuarta:

yes

[14:13:53]

stuarta:

and increment if we fix/change stuff that would api break

[14:14:10]

peterbennett:

Also the methods that do update but were GET and I changed to POST, maybe I should allow GET for backward compatibility

[14:14:40]

stuarta:

personally i'd like to see them as PUT's but again it depends on the clients "in the wild"

[14:14:47]

stuarta:

are there even any

[14:14:47]

peterbennett:

The one boolean that was wrongly an int I should change back to int.

[14:15:23]

stuarta:

tbh, the primary aim with the first pass of conversion was API equivalence with the old version

[14:16:00]

peterbennett:

Maybe just leave them and see if anybody complains? I dont know if anybody uses them. I already changed my application to use PUT

[14:16:16]

peterbennett:

sorry POST

[14:16:29]

stuarta:

i'd go with that. it's a trivial fix to support legacy stuff if really needed

[14:17:19]

peterbennett:

Sorry go with which ? POST or allow GET? Boolean or int ?

[14:17:57]

peterbennett:

The GET always allows POST as well by default

[14:18:07]

stuarta:

POST only

[14:18:22]

stuarta:

if stuff breaks it's easy to re-enable GET's

[14:18:43]

peterbennett:

ok good, maybe we should document that somewhere

[14:18:55]

peterbennett:

Not sure where

[14:21:54]

stuarta:

wiki probably

[14:23:32]

peterbennett:

ok

[14:50:16]

hampton:

The first hit I get when searching for GET vs POST says "GET requests are only used to request data (not modify)". Dump the backward compatibility for the GETs. Those old GET calls should return a "405 Method not Allowed" error code

[14:57:45]

peterbennett (peterbennett!~peter@mythtv/developer/peterbennett) has quit (Quit: Leaving.)

[15:04:58]

stuarta:

hampton: that's what the new code does

[15:07:54]

hampton:

:)

[15:13:15]

hampton:

Code question in mythhttpserver.cpp. Clazy is pointing out that the code calls 'emit ProcessTCPQueue()' twice, but ProcessTCPQueue() isn't a signal (its a slot).

[15:13:37]

stuarta:

it's both iirc

[15:15:54]

stuarta:

clazy appears correct

[15:16:18]

stuarta:

hmm, actually, it might be in another class

[15:16:55]

hampton:

Can't be. A slot is an actual function written in the code, and a signal is a function generated by moc. All that's going to do is call the ProcessTCPQueue() directly, not call it on another thread.

[15:17:21]

stuarta:

it needs todo it asynchronously

[15:17:35]

hampton:

Anyway, wasn't sure how it should be fixed, so I asked.

[15:19:10]

hampton:

I think you probably need another signal, and wire that up in the MythHTTPServer constructor to call ProcessTCPQueue().

[15:19:36]

stuarta:

i thought i had it hooked up in the lower level class.... :-/

[15:22:49]

peterbennett (peterbennett!~peter@mythtv/developer/peterbennett) has joined #mythtv

[15:22:50]

Mode for #mythtv by ChanServ!ChanServ@services.libera.chat : +v peterbennett

[15:31:23]

hampton:

I think I have a fix for the missing "atomic" symbols. Still testing. Building FFMpeg on an rpi3 is slow.

[15:50:01]

peterbennett:

hampton: Will do

[16:39:11]

SteveGoodey (SteveGoodey!~steve@2a00:23c5:7d83:6501:fa68:6631:5ed6:d871) has joined #mythtv

[16:41:11]

peterbennett:

stuarta: mythweb master with the upgraded php code seems to run fine against mythtv 31. Is there any difference? Once we have fully tested can we just merge master into fixes/31 on mythweb to support php 8?

[16:50:56]

peterbennett (peterbennett!~peter@mythtv/developer/peterbennett) has quit (Quit: Leaving.)

[16:56:26]

peterbennett (peterbennett!~peter@mythtv/developer/peterbennett) has joined #mythtv

[16:56:26]

Mode for #mythtv by ChanServ!ChanServ@services.libera.chat : +v peterbennett

[17:08:17]

peterbennett (peterbennett!~peter@mythtv/developer/peterbennett) has quit (Quit: Leaving.)

[17:40:41]

WarpedAI (WarpedAI!~Warped@user/warped) has joined #mythtv

[17:43:12]

Warped (Warped!~Warped@user/warped) has quit (Ping timeout: 265 seconds)

[17:43:13]

newbie|2 (newbie|2!~Warped@user/warped) has joined #mythtv

[17:46:17]

WarpedAI (WarpedAI!~Warped@user/warped) has quit (Ping timeout: 252 seconds)

[17:48:41]

DevMythNotifyBot:

issue reopened by kmdewaal, "configure does not set optimization flag in CXXFLAGS": https://github.com/MythTV/mythtv/issues/406

[18:25:21]

DevMythNotifyBot:

COMMIT: stuarta pushed 5 commit(s) to branch master: https://github.com/MythTV/mythtv/compare/6692 . . . 1bb9bb786ac9

[19:42:56]

peterbennett (peterbennett!~peter@mythtv/developer/peterbennett) has joined #mythtv

[19:42:56]

Mode for #mythtv by ChanServ!ChanServ@services.libera.chat : +v peterbennett

[19:54:21]

paul-h (paul-h!~paul@pres-25-b2-v4wan-161467-cust2666.vm29.cable.virginm.net) has joined #mythtv

[19:54:21]

paul-h (paul-h!~paul@mythtv/developer/paul-h) has joined #mythtv

[19:54:21]

paul-h (paul-h!~paul@pres-25-b2-v4wan-161467-cust2666.vm29.cable.virginm.net) has quit (Changing host)

[19:54:21]

Mode for #mythtv by ChanServ!ChanServ@services.libera.chat : +v paul-h

[20:39:05]

peterbennett:

Actually before this commit of the php 8.0 changes, master was just 1 commit ahead of fixes/31 – " Fix typo found by chance.". So I don't think there is any problem with them being the same.

[20:40:58]

stuarta:

peterbennett: i was going to answer your question with backport it to fixes/31

[20:41:08]

stuarta:

but you had disappeared

[20:41:16]

peterbennett:

Sorry

[20:41:30]

stuarta:

i blame your sleepy laptop :-p

[20:41:41]

peterbennett:

anyway just merge to fixes/31 should do it

[20:41:47]

stuarta:

yup

[20:42:10]

peterbennett:

I had to visit the Pharmacy, or Chemist as you may call it.

[20:42:25]

stuarta:

for me the terms are interchangable

[20:42:51]

peterbennett:

I will wait a day or two in case any problems are reported then merge into fixes/31.

[20:43:28]

peterbennett:

Just don't tell anybody in the USA you are visiting the Chemist...

[20:43:42]

stuarta:

:-p

[20:44:07]

stuarta:

just like you don't tell US Customs you are there for work, when you mean business

[20:44:40]

peterbennett:

Hmm yes I guess so. Not if you want to be let in.

[21:10:01]

peterbennett (peterbennett!~peter@mythtv/developer/peterbennett) has quit (Quit: Leaving.)

[21:19:06]

DevMythNotifyBot:

pull_request opened by ulmus-scott, "configure optimizations, other changes": https://github.com/MythTV/mythtv/pull/430

[21:22:11]

stuarta:

hah victory!

[21:25:27]

stuarta:

now to make it look less shit

[21:59:37]

DevMythNotifyBot:

COMMIT: stuarta pushed 1 commit(s) to branch master: https://github.com/MythTV/mythtv/compare/1bb9 . . . 380ddcbcb7a8

[22:02:45]

stuarta:

styling entirely knicked from previous implementations

[22:36:51]

Steve-Goodey (Steve-Goodey!~steve@2a00:23c5:7d83:6501:6ea1:20d:5872:d872) has quit (Quit: Konversation terminated!)

[22:38:58]

peterbennett (peterbennett!~peter@mythtv/developer/peterbennett) has joined #mythtv

[22:38:58]

Mode for #mythtv by ChanServ!ChanServ@services.libera.chat : +v peterbennett

[22:56:39]

peterbennett (peterbennett!~peter@mythtv/developer/peterbennett) has quit (Quit: Leaving.)

[23:10:35]

SteveGoodey (SteveGoodey!~steve@2a00:23c5:7d83:6501:fa68:6631:5ed6:d871) has quit (Quit: Konversation terminated!)

[23:32:26]

amessina (amessina!~amessina@user/amessina) has joined #mythtv

MythLogBot@irc.libera.chat :: #mythtv

Current users (26):