MythLogBot@irc.freenode.net :: #mythtv

Daily chat history

Current users (40):

Anssi, brfransen, ChanServ, davic, dym, ghoti, gigem, gregbert, gregl, hampton, ijc, jpabq_, jpharvey, jya, knowledgejunkie, kwizart, libsci, mad_enz, markspieth, MitchCapper, mkbloke, MythBuild, MythLogBot, MythNotifyBot, M_kveef, Panic, peper03, peterbennett, poptix, pppingme, sdrik, sphery_, stuarta, taylorr, tgm4883, tonsofpcs, tris-, Warped, xris, _charly_
Tuesday, October 16th, 2018, 00:35 UTC
[00:35:38] taylorr (taylorr!~taylorr@unaffiliated/elmojo) has joined #mythtv
[00:44:05] mad_enz (mad_enz!~mad_enz@2607:fea8:bdf:e784:c0bf:490c:4a79:4e61) has quit (Ping timeout: 250 seconds)
[00:57:36] mad_enz (mad_enz!~mad_enz@CPE00508b114c3e-CMac202e117380.cpe.net.cable.rogers.com) has joined #mythtv
[01:04:30] markspieth (markspieth!~markspiet@mythtv/developer/markspieth) has quit (Ping timeout: 244 seconds)
[01:08:31] davic (davic!~davic@unaffiliated/davic) has quit (Ping timeout: 252 seconds)
[01:08:42] davic (davic!~davic@unaffiliated/davic) has joined #mythtv
[03:03:04] SteveGoodey (SteveGoodey!~steve@host81-158-30-102.range81-158.btcentralplus.com) has quit (Quit: Konversation terminated!)
[05:13:11] jpabq_ (jpabq_!~quassel@mythtv/developer/jpabq) has joined #mythtv
[05:13:12] Mode for #mythtv by ChanServ!ChanServ@services. : +v jpabq_
[05:15:31] jpabq (jpabq!~quassel@mythtv/developer/jpabq) has quit (Ping timeout: 246 seconds)
[05:38:38] Warped (Warped!~Warped@unaffiliated/warped) has quit (Quit: ChatZilla 0.9.92-rdmsoft [XULRunner 35.0.1/20150122214805])
[06:09:53] Steve-Goodey (Steve-Goodey!~steve@2a00:23c5:7da3:4501:7a84:3cff:fedf:a99) has joined #mythtv
[06:09:54] Mode for #mythtv by ChanServ!ChanServ@services. : +v Steve-Goodey
[07:52:49] Steve-Goodey (Steve-Goodey!~steve@2a00:23c5:7da3:4501:7a84:3cff:fedf:a99) has quit (Quit: Konversation terminated!)
[07:55:00] SteveGoodey (SteveGoodey!~steve@host81-158-30-102.range81-158.btcentralplus.com) has joined #mythtv
[07:55:00] Mode for #mythtv by ChanServ!ChanServ@services. : +v SteveGoodey
[08:01:49] willcooke (willcooke!~willcooke@ubuntu/member/willcooke) has joined #mythtv
[09:09:09] ghoti (ghoti!ghoti@dev-160.experiencepoint.com) has quit (Ping timeout: 268 seconds)
[10:33:11] willcooke (willcooke!~willcooke@ubuntu/member/willcooke) has quit (Remote host closed the connection)
[10:40:22] markspieth (markspieth!~markspiet@mythtv/developer/markspieth) has joined #mythtv
[10:40:22] Mode for #mythtv by ChanServ!ChanServ@services. : +v markspieth
[10:51:44] Steve-Goodey (Steve-Goodey!~steve@2a00:23c5:7da3:4501:7a84:3cff:fedf:a99) has joined #mythtv
[10:51:44] Mode for #mythtv by ChanServ!ChanServ@services. : +v Steve-Goodey
[10:59:40] willcooke (willcooke!~willcooke@ubuntu/member/willcooke) has joined #mythtv
[11:26:40] willcooke (willcooke!~willcooke@ubuntu/member/willcooke) has quit (Ping timeout: 252 seconds)
[11:28:11] willcooke (willcooke!~willcooke@ubuntu/member/willcooke) has joined #mythtv
[11:51:58] willcooke (willcooke!~willcooke@ubuntu/member/willcooke) has quit (Ping timeout: 252 seconds)
[12:10:11] willcooke (willcooke!~willcooke@ubuntu/member/willcooke) has joined #mythtv
[12:11:11] ooshlablu (ooshlablu!~ooshlablu@2601:18d:4600:5f31:808:ec5f:1cc8:d6c) has quit (Remote host closed the connection)
[12:48:10] stuarta: #13328 is still annoying me
[12:48:10] ** MythLogBot https://code.mythtv.org/trac/ticket/13328 **
[13:09:45] kwizart: stuarta, I don't think SYSROOT is an accurate name usually it should be called DESTDIR
[13:10:34] kwizart: but I don't understand why this patch would change anything https://code.mythtv.org/trac/attachment/ticke . . . f8_fix.patch
[13:13:25] ooshlablu (ooshlablu!~ooshlablu@96.70.215.161) has joined #mythtv
[13:17:24] kwizart: as I understand SYSROOT should be empty in our case as it's probably better used for cross-compilation
[13:18:28] kwizart: but then we need to have a DESTDIR variable to install libraries and binaries into our dedicated path
[13:19:45] willcooke (willcooke!~willcooke@ubuntu/member/willcooke) has quit (Ping timeout: 252 seconds)
[13:20:00] kwizart: the RPM Fusion package currently use that on install:
[13:20:01] kwizart: make install INSTALL_ROOT=%{buildroot}
[13:20:17] kwizart: whereas it's usually make install DESTDIR=%{buildroot}
[13:24:29] kwizart: btw about non-nvidia vdpau cases (nvidia tegra) https://paste.fedoraproject.org/paste/u85eR4svWgTk-oohs7cUYQ
[13:27:06] willcooke (willcooke!willcooke@ubuntu/member/willcooke) has joined #mythtv
[13:56:34] stuarta: kwizart: interesting thanks. i have the src.rpm so i've been doing test builds and can observe any changes
[13:59:12] stuarta: peterbennett: any comment on the nvidia tegra vdpau change ^^^ from kwizart ???
[14:02:27] kwizart: actually I wonder if the NVIDIA vdpau workaround for the "desktop" gpu is still needed, but at least I expect the bug would be different enought on a mesa or grate-driver
[14:03:25] kwizart: I haven't managed to test mythtv on this builder that been said, specially as it would only be capable to output using xv
[14:03:59] kwizart: I meant, testing mythtv on this arm machine (tegra20 paz00 or trimslice)
[14:05:33] stuarta: hmm, i think you might be onto something here. we should be correctly setting DEPLIBS, but the rpmfusion spec file then mangles DEPLIB
[14:06:10] stuarta: well actually it can't in this instance, because we changed DEPLIBS so the sed regexp doesn't matter
[14:06:11] peterbennett: kwizart: What problem is that patch trying to fix?
[14:06:13] stuarta: *match
[14:06:21] stuarta: https://github.com/rpmfusion/mythtv/blob/master/mythtv.spec#L958
[14:08:26] kwizart: peterbennett, the tegra vdpau backend is reported as "NVIDIA Tegra VDPAU backend" and would match the string reported as "the desktop gpu" which are reported as "NVIDIA VDPAU Driver Shared Library"
[14:09:03] kwizart: peterbennett, it's unlikely that the tegra vdpau backend needs the same workaround as normal desktop
[14:09:27] ** stuarta suspects he has found the culprit and fires off a test build **
[14:11:05] kwizart: peterbennett, now maybe the whole workaround need to be dropped instead
[14:11:57] stuarta: anyone remember *what* we are working around?
[14:12:21] kwizart: https://bugs.freedesktop.org/show_bug.cgi?id=80561
[14:12:50] peterbennett: kwizart: The thing that says "// Work around MESA bug 80561" ?
[14:13:12] kwizart: a mesa bug from 2014, yes
[14:13:14] stuarta: peterbennett: the link kwizart just posted
[14:14:16] peterbennett: So that says a fix was pushed upstream, so maybe we don't need the check any more
[14:15:20] stuarta: isn't it possible to pull the version info out of vdpau to be able to identify buggy versions?
[14:22:10] peterbennett: I don't know, I have not done anything with VDPAU to date.
[15:23:25] Warped (Warped!~Warped@unaffiliated/warped) has joined #mythtv
[16:20:56] stuarta: getting somewhere with that ridiculous library link error in the plugins
[17:30:51] natanojl (natanojl!~jonatan@mythtv/developer/natanojl) has joined #mythtv
[17:30:51] Mode for #mythtv by ChanServ!ChanServ@services. : +v natanojl
[17:41:55] amessina (amessina!~amessina@unaffiliated/amessina) has joined #mythtv
[17:54:45] hampton: peterbennett: I've got a couple of changes I'd like to make to FFmpeg to cleanup nullptr warnings. Do I need to double commit them somewhere?
[17:54:56] hampton: https://github.com/linuxdude42/mythtv/commit/ . . . ece71d8675b9
[18:04:38] peterbennett: hampton: Preferable we don't want to make changes to ffmpeg that may cause merge problems later when merging in the next ffmpeg release
[18:07:19] peterbennett: hampton: If a change fo ffmpeg is necessary, make it in the MythTV/FFmpeg repository, first in master branch, then cherry pick to the release/4.0 branch. Then you can copy the source file into MythTV master branch.
[18:09:18] peterbennett: hampton: The code in MythTV/mythtv/mythtv/external/FFmpeg (master branch) is an identical copy of MythTV/FFmpeg release/4.0 branch.
[18:10:03] hampton: ok
[18:11:15] hampton: This change (and one more elsewhere) will allow me to have the compiler enforce the use of the C++11 nullptr value.
[18:12:37] peterbennett: Are you talking about forcing nullptr in the FFmpeg directory itself or in MythTV code where FFmpeg headers are included?
[18:13:22] peterbennett: Are you changing header files or c files?
[18:13:30] hampton: Enforcing nulllptr in MythTV. The changes are all to header files used by MythTV.
[18:14:11] peterbennett: I see, so there are a couple of cases where NULL or 0 is used for a pointer in FFmpeg header files?
[18:14:45] hampton: Yes. The earlier link points to the proposed changes.
[18:15:00] amessina (amessina!~amessina@unaffiliated/amessina) has quit (Quit: Konversation terminated!)
[18:15:00] peterbennett: How about #define NULL nullptr ? (I always like to do things the easy way).
[18:16:24] hampton: That would probably work at the start of those four header files. I'll give it a shot.
[18:17:24] peterbennett: Probably should have #undef NULL at the end of the header file in that case
[18:17:40] peterbennett: so that other uses of NULL do not slip through.
[18:18:09] hampton: And I'll need to undef it at the start of the file too, since its already defined to be 0.
[18:19:47] peterbennett: How about putting the #define before #include of the header file?
[18:19:50] natanojl (natanojl!~jonatan@mythtv/developer/natanojl) has quit (Ping timeout: 268 seconds)
[18:19:55] peterbennett: and #undef after
[18:20:14] peterbennett: That way the ffmpeg files do not need to be changed.
[18:20:25] hampton: For the first two files I think doing all the defines/undefines will require a larger change set. Its probably amortized for the third file.
[18:20:57] hampton: I can check. I will have to see how many files include the FFMpeg headers, and where NULL is defined.
[18:21:20] peterbennett: Probably not included from too many places
[18:21:20] hampton: I'll do some checking and get back to you.
[18:26:40] hampton: A quick grep says its just in the h264 parser. I suspect that's not all though, as one of the changes was in avassert.h which is probably included by other include files whose names I didn't search for. I'll try reverting my changes and see how many places actually throw errors in a test compile.
[18:29:10] peterbennett: There are probably not many places in the system where ffmpeg includes are used, particularly the ffmpeg internal includes.
[18:32:06] peterbennett: We should really only use the approved ffmpeg includes, which are those that are deployed in the /usr/include/mythtv/libav* directories. If we ever want to get away from our customized FFmpeg we would have to do that.
[18:42:02] hampton: Makes sense.
[19:15:09] SteveGoodey: lol
[19:53:37] willcooke (willcooke!willcooke@ubuntu/member/willcooke) has quit (Remote host closed the connection)
[20:32:33] Steve-Goodey (Steve-Goodey!~steve@2a00:23c5:7da3:4501:7a84:3cff:fedf:a99) has quit (Quit: Konversation terminated!)
[20:36:25] hampton: peterbennett: I can redefine NULL once in H264Parser.h, and then I don't need any of the changes to the FFMpeg headers. Thanks for the pushback. :-)
[20:38:20] ooshlablu (ooshlablu!~ooshlablu@96.70.215.161) has quit (Remote host closed the connection)
[20:50:53] peterbennett: hampton: good, thanks
[21:06:21] natanojl (natanojl!~jonatan@mythtv/developer/natanojl) has joined #mythtv
[21:06:22] Mode for #mythtv by ChanServ!ChanServ@services. : +v natanojl
[21:17:19] markspieth: hampton: does -DNULL=nullptr work in CFLAGS?
[21:27:39] peterbennett: stuarta: What about CUDA, aka NVDEC aka NVCUVID ? It seems that this is the replacement for VDPAU ? VDPAU is dead long live NVDEC ?
[21:30:16] hampton: markspieth: Probably not. I had to put the undef/define immediately before the include that caused problems. I initially tried it earlier in that file and something redefnied it on me.
[21:30:45] MythBuild: build #1072 of master-ubuntu-lts-64bit is complete: Failure [failed compile core] Build details are at http://code.mythtv.org/buildbot/builders/mast . . . /builds/1072 blamelist: John Poet <jpoet@mythtv.org >
[21:32:25] markspieth: Also the usage for SYSROOT is for system includes and libs (empty on linux builds) and MYTHROOT which contains staging for any build libraries. That was the intention so that plugins could be built without haivng installed mythlib dev first.
[21:36:28] MythBuild: build #920 of master-debian-testing-64bit is complete: Failure [failed compile core] Build details are at http://code.mythtv.org/buildbot/builders/mast . . . t/builds/920 blamelist: John Poet <jpoet@mythtv.org >
[21:36:56] MythBuild: build #1090 of master-debian-jessie-64bit is complete: Failure [failed compile core] Build details are at http://code.mythtv.org/buildbot/builders/mast . . . /builds/1090 blamelist: John Poet <jpoet@mythtv.org >
[21:39:46] MythBuild: build #300 of master-f27–64bit is complete: Failure [failed compile core] Build details are at http://code.mythtv.org/buildbot/builders/mast . . . t/builds/300 blamelist: John Poet <jpoet@mythtv.org >
[21:41:06] MythBuild: build #2660 of master-fedora-32bit is complete: Failure [failed compile core] Build details are at http://code.mythtv.org/buildbot/builders/mast . . . /builds/2660 blamelist: John Poet <jpoet@mythtv.org >
[21:41:07] MythBuild: build #788 of master-archlinux-64bit is complete: Failure [failed compile core] Build details are at http://code.mythtv.org/buildbot/builders/mast . . . t/builds/788 blamelist: John Poet <jpoet@mythtv.org >
[21:44:07] MythBuild: build #1052 of master-debian-jessie-32bit is complete: Failure [failed compile core] Build details are at http://code.mythtv.org/buildbot/builders/mast . . . /builds/1052 blamelist: John Poet <jpoet@mythtv.org >
[21:47:47] MythBuild: build #102 of master-f28–64bit is complete: Failure [failed compile core] Build details are at http://code.mythtv.org/buildbot/builders/mast . . . t/builds/102 blamelist: John Poet <jpoet@mythtv.org >
[21:50:16] SteveGoodey (SteveGoodey!~steve@host81-158-30-102.range81-158.btcentralplus.com) has quit (Quit: Konversation terminated!)
[21:54:45] jheizer_ (jheizer_!~jheizer@c-73-51-93-177.hsd1.il.comcast.net) has quit (Quit: Leaving)
[21:56:16] MythBuild: build #1460 of master-centos7–64bit is complete: Failure [failed compile core] Build details are at http://code.mythtv.org/buildbot/builders/mast . . . /builds/1460 blamelist: John Poet <jpoet@mythtv.org >
[21:56:56] jheizer_ (jheizer_!~jheizer@c-73-51-93-177.hsd1.il.comcast.net) has joined #mythtv
[21:56:56] Mode for #mythtv by ChanServ!ChanServ@services. : +v jheizer_
[21:57:07] MythBuild: build #222 of master-fedora-rawhide-64bit is complete: Failure [failed compile core] Build details are at http://code.mythtv.org/buildbot/builders/mast . . . t/builds/222 blamelist: John Poet <jpoet@mythtv.org >
[21:58:54] jheizer_ (jheizer_!~jheizer@c-73-51-93-177.hsd1.il.comcast.net) has quit (Client Quit)
[22:04:14] jheizer_ (jheizer_!~jheizer@c-73-51-93-177.hsd1.il.comcast.net) has joined #mythtv
[22:04:14] Mode for #mythtv by ChanServ!ChanServ@services. : +v jheizer_
[22:04:41] jheizer_ (jheizer_!~jheizer@c-73-51-93-177.hsd1.il.comcast.net) has quit (Client Quit)
[22:37:25] natanojl (natanojl!~jonatan@mythtv/developer/natanojl) has quit (Ping timeout: 244 seconds)
[23:16:14] MythBuild: build #103 of master-f28–64bit is complete: Success [build successful] Build details are at http://code.mythtv.org/buildbot/builders/mast . . . t/builds/103
[23:23:49] MythBuild: build #223 of master-fedora-rawhide-64bit is complete: Success [build successful] Build details are at http://code.mythtv.org/buildbot/builders/mast . . . t/builds/223
[23:24:58] MythBuild: build #190 of master-fedora-aarch64 is complete: Failure [failed compile core] Build details are at http://code.mythtv.org/buildbot/builders/mast . . . 4/builds/190 blamelist: John Poet <jpoet@mythtv.org >
[23:26:56] MythBuild: build #789 of master-archlinux-64bit is complete: Success [build successful] Build details are at http://code.mythtv.org/buildbot/builders/mast . . . t/builds/789
[23:29:25] MythBuild: build #2661 of master-fedora-32bit is complete: Success [build successful] Build details are at http://code.mythtv.org/buildbot/builders/mast . . . /builds/2661
[23:42:45] ghoti (ghoti!ghoti@dev-160.experiencepoint.com) has joined #mythtv

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