| Monday, December 17th, 2018, 01:18 UTC | ||
| [01:18:43] | gregl (gregl!~greg@cpe-24-194-253-7.nycap.res.rr.com) has joined #mythtv | |
| [02:07:14] | Hydroponx (Hydroponx!~hydr@66-191-154-200.dhcp.mtgm.al.charter.com) has quit (Ping timeout: 244 seconds) | |
| [02:11:34] | Hydr0p0nX (Hydr0p0nX!~hydr@66-191-154-200.dhcp.mtgm.al.charter.com) has joined #mythtv | |
| [02:12:45] | gigem: | hampton: Mine? I don't remember anything. |
| [02:17:14] | gigem: | peterbennett: I didn't realize that. Those options can be very dangerous. Probably not for the changes being discussed but there have been others in the past and probably will be again where that could cause serious problems. That's one reason I'd prefer the frontend to switch to only using services, api calls where we could in theory more easily support multiple versions. |
| [07:08:32] | Steve-Goodey (Steve-Goodey!~steve@2a00:23c5:7da3:4501:7a84:3cff:fedf:a99) has joined #mythtv | |
| [07:08:33] | Mode for #mythtv by ChanServ!ChanServ@services. : +v Steve-Goodey | |
| [08:18:14] | RokLobsta (RokLobsta!~Dirkka@119-18-13-51.cust.aussiebb.net) has joined #mythtv | |
| [08:46:44] | willcooke (willcooke!~willcooke@ubuntu/member/willcooke) has joined #mythtv | |
| [09:03:21] | Merlin83b (Merlin83b!~Daniel@2a00:1ee0:3:1337:8185:5334:a44b:f76f) has joined #mythtv | |
| [09:39:31] | Steve-Goodey (Steve-Goodey!~steve@2a00:23c5:7da3:4501:7a84:3cff:fedf:a99) has quit (Quit: Konversation terminated!) | |
| [09:44:58] | stuarta: | we need to fix the blank branch information in the ubuntu builds. it's making it's way into the smolt data now |
| [10:28:22] | Tobbe5178534 (Tobbe5178534!~asdf@2001:2002:51eb:d24e:945c:d547:5be1:12f7) has joined #mythtv | |
| [10:30:21] | Tobbe5178 (Tobbe5178!~asdf@2001:2002:51eb:d24e:f8ba:60cf:3cc8:d78f) has quit (Ping timeout: 260 seconds) | |
| [10:44:17] | stuarta: | i suspect it's because it's built from a tarball and not a git checkout |
| [11:00:23] | stuarta: | well i can confirm, if the .git metadata is missing then version.h gets built without a branch defined |
| [11:02:03] | PaddyTEST (PaddyTEST!~PaddyF@p5B0B7740.dip0.t-ipconnect.de) has quit (Remote host closed the connection) | |
| [11:29:54] | stuarta: | but, if the repo is exported with git archive, then EXPORTED_VERSION contains sufficient data to correctly set branch and version |
| [11:34:37] | stuarta: | need tgm to wake up and let me know some implementation details |
| [11:39:02] | SteveGoodey (SteveGoodey!~steve@2a00:23c5:7da3:4501:7a24:afff:fe9d:c233) has joined #mythtv | |
| [11:39:04] | Mode for #mythtv by ChanServ!ChanServ@services. : +v SteveGoodey | |
| [12:05:22] | stuarta: | SteveGoodey: do you use the ubuntu packages? |
| [12:05:41] | SteveGoodey: | stuarta: Yup. |
| [12:07:56] | SteveGoodey: | I've just sent a reply to your mythtv-users mail about the version problem. I think it started about the time tgm 4883 moved the packaging process from his own server onto launchpad. |
| [12:08:24] | stuarta: | agreed, i suspect it's because it's not using git archive to create the tarball it uses |
| [12:09:01] | stuarta: | done some investigation and when using git-archive, it properly populates EXPORTED_VERSION, which then gets rolled into the build |
| [12:11:08] | stuarta: | although a tweak may be needed to version.sh to strip leading origin from branch name |
| [12:11:18] | stuarta: | (on my test, branch got set to origin/fixes/29) |
| [12:43:52] | Warped_ (Warped_!~Warped@2600:1702:b40:851f:71b5:c49d:9758:4366) has joined #mythtv | |
| [12:46:16] | Warped (Warped!~Warped@unaffiliated/warped) has quit (Ping timeout: 264 seconds) | |
| [12:54:53] | Warped_ (Warped_!~Warped@2600:1702:b40:851f:71b5:c49d:9758:4366) has quit (Quit: ChatZilla 0.9.92-rdmsoft [XULRunner 35.0.1/20150122214805]) | |
| [12:56:43] | Warped (Warped!~Warped@unaffiliated/warped) has joined #mythtv | |
| [13:07:49] | blahdodo (blahdodo!~blahdodo@69.172.190.157) has quit (Quit: Bye bye) | |
| [13:08:08] | blahdodo (blahdodo!~blahdodo@69.172.190.157) has joined #mythtv | |
| [13:10:42] | stuarta: | and time to redeliver a whole bunch of github webhooks :-/ |
| [13:17:27] | Steve-Goodey (Steve-Goodey!~steve@2a00:23c5:7da3:4501:7a84:3cff:fedf:a99) has joined #mythtv | |
| [13:17:28] | Mode for #mythtv by ChanServ!ChanServ@services. : +v Steve-Goodey | |
| [13:52:32] | stuartm (stuartm!~stuartm@mythtv/developer/stuartm) has quit (Quit: Gone) | |
| [13:53:11] | stuartm (stuartm!~stuartm@mythtv/developer/stuartm) has joined #mythtv | |
| [13:53:11] | Mode for #mythtv by ChanServ!ChanServ@services. : +v stuartm | |
| [15:36:07] | natanojl (natanojl!~jonatan@mythtv/developer/natanojl) has joined #mythtv | |
| [15:36:07] | Mode for #mythtv by ChanServ!ChanServ@services. : +v natanojl | |
| [16:51:18] | tgm4883 (tgm4883!uid23806@ubuntu/member/tgm4883) has joined #mythtv | |
| [17:42:41] | Merlin83b (Merlin83b!~Daniel@2a00:1ee0:3:1337:8185:5334:a44b:f76f) has quit (Quit: Leaving) | |
| [17:42:56] | gigem: | stuarta: Does that email mean Mark accepted your invitation? |
| [17:44:28] | stuarta: | seems to have, he's just committed a fix for one of the shaders |
| [17:44:39] | stuarta: | and now another |
| [17:46:12] | gigem: | EXcellent. |
| [17:48:09] | stuarta: | i've let him know master is bugfix only right now, in prep for cutting the branch |
| [17:48:27] | ** stuarta insults new buildbot ** | |
| [17:50:44] | tgm4883: | stuarta: supposedly the version stuff should be working |
| [17:51:00] | stuarta: | tgm4883: seems not to be :( |
| [17:51:04] | tgm4883: | yea |
| [17:51:07] | stuarta: | are you using git archive in the build? |
| [17:51:36] | ** tgm4883 shrugs ** | |
| [17:51:42] | tgm4883: | it's all done via a LP recipe now |
| [17:51:54] | tgm4883: | I can check how that gets created |
| [17:52:02] | stuarta: | well what does that do? i had trouble finding the actual recipe |
| [17:52:03] | tgm4883: | Here's the recipe for v29 https://code.launchpad.net/~mythbuntu/+recipe/mythtv-daily-29 |
| [17:52:45] | stuarta: | i saw that, but it doesn't tell me what it's actually doing as part of those steps |
| [17:53:05] | tgm4883: | yea I know |
| [17:53:06] | stuarta: | `lp:~mythbuntu-dev/mythtv/+git/mythtv fixes/29` that's clearly building mythtv |
| [17:53:08] | tgm4883: | I'm asking LP now |
| [17:53:54] | stuarta: | tgm4883: okay thanks, from the digging i did, we need to run `git archive` to create the tarball with the proper version stuff populated into EXPORTED_VERSION |
| [17:54:24] | stuarta: | if it just creates a tarball (dropping the .git dir) then it breaks the way we are seeing |
| [17:56:51] | tgm4883: | asking in #launchpad now |
| [17:58:12] | tgm4883: | I looked at it a bit as well and I think we could just not run the version.sh stuff and populate the version via the recipe but fixing this would be cleaner |
| [18:00:50] | tgm4883: | stuarta: 9:59 AM <cjwatson> Ah yeah, dpkg-buildpackage from the recipe probably excludes .git |
| [18:01:43] | stuarta: | where does dpkg-buildpackage get it's tarball from? |
| [18:03:34] | stuarta: | tgm4883: i'm popping over there, i've got ~20mins |
| [18:05:10] | stuarta: | tgm4883: another question then, where is the packaging bits dkpg-buildpackage is using? |
| [18:05:50] | ** stuarta notes a `sourcehook` item in the dpkg-buildpackage man page ** | |
| [18:06:40] | stuarta: | is it the stuff in https://github.com/MythTV/packaging ? |
| [18:12:17] | stuarta: | tgm4883: i think i spy a simple fix |
| [18:12:27] | stuarta: | https://github.com/MythTV/packaging/blob/mast . . . htv.make#L52 |
| [18:12:34] | stuarta: | assuming that's what is used |
| [18:12:42] | stuarta: | swap it out for a git archive |
| [18:14:25] | stuarta: | hmmm... https://github.com/MythTV/packaging/blob/mast . . . tv.make#L140 |
| [18:17:31] | stuarta: | wait, the debian/rules file is meant to be copying in debian/DESCRIBE which should contain the right info |
| [18:18:09] | stuarta: | tgm4883: the other option here -> https://github.com/MythTV/packaging/blob/mast . . . n/rules#L100 |
| [18:18:24] | tgm4883: | sorry was in a meeting |
| [18:18:45] | tgm4883: | so yea, we're pulling in https://github.com/MythTV/packaging |
| [18:18:48] | stuarta: | if we do `cp debian/DESCRIBE mythtv/EXPORTED_VERSION` then the version.sh script will do the right thing |
| [18:19:02] | tgm4883: | you'll see in the recipe that we're nesting 3 repos inside of the mythtv repo |
| [18:19:12] | stuarta: | tgm4883: okay, so *ideally* we use git-archive to create the tarball |
| [18:19:15] | tgm4883: | packaging, mythweb and the mythbuntu theme |
| [18:19:37] | stuarta: | that's fine |
| [18:20:57] | tgm4883: | stuarta: so you think that's the best option then, the cp debian/DESCRIBE mythtv/EXPORTED_VERSION` |
| [18:21:20] | stuarta: | it's a workaround |
| [18:21:57] | stuarta: | tgm4883: if there is a step which takes the pristine checkout, and turns it into a tarball, then that should use `git archive` to create it |
| [18:22:14] | stuarta: | which automatically populates mythtv/EXPORTED_VERSION with the correct data from git |
| [18:22:33] | stuarta: | which version.sh then correctly parses |
| [18:22:52] | stuarta: | into mythtv/VERSION and mythtv/libs/libmythbase/version.h |
| [18:24:43] | stuarta: | i have to run off now, so if you can have a think about how to do that, it would be appreciated |
| [18:29:03] | willcooke (willcooke!~willcooke@ubuntu/member/willcooke) has quit (Quit: Do your hobbies) | |
| [20:16:44] | jasabella (jasabella!jasabella@nxdoma.in) has left #mythtv () | |
| [20:38:31] | gigem: | hampton: In light of peterbennett's revelation that the android frontend ignores protocol and schema mismatches, go ahead and commit your changes. We'll just have to keep an eye on such changes and decide what to do when a truly incompatible one is made. |
| [20:40:08] | hampton: | lol. np. |
| [21:05:19] | nephyrin (nephyrin!~neph@2601:600:817f:f46a:e41d:6f5c:e3be:b8ad) has quit (Ping timeout: 250 seconds) | |
| [21:09:45] | nephyrin (nephyrin!~neph@2601:600:817f:f46a:e41d:6f5c:e3be:b8ad) has joined #mythtv | |
| [21:42:07] | Steve-Goodey (Steve-Goodey!~steve@2a00:23c5:7da3:4501:7a84:3cff:fedf:a99) has quit (Quit: Konversation terminated!) | |
| [22:10:58] | ** stuarta insults new buildbot again ** | |
| [22:11:42] | stuarta: | sigh, still struggling to get a config which works for the triggered builds and allows forcing of builds |
| [22:19:09] | nephyrin (nephyrin!~neph@2601:600:817f:f46a:e41d:6f5c:e3be:b8ad) has quit (Ping timeout: 252 seconds) | |
| [22:19:45] | nephyrin (nephyrin!~neph@2601:600:817f:f46a:e41d:6f5c:e3be:b8ad) has joined #mythtv | |
| [22:21:38] | stuarta: | tgm4883: lets go with copying debian/DESCRIBE to mythtv/EXPORTED_VERSION |
| [22:22:36] | stuarta: | tgm4883: mainly because it looks `mythtv.make` is expected to be in the tarball |
| [22:26:19] | SteveGoodey (SteveGoodey!~steve@2a00:23c5:7da3:4501:7a24:afff:fe9d:c233) has quit (Quit: Konversation terminated!) | |
| [22:28:46] | stuarta: | hampton: good spot on noticing freebsd12 is out |
| [22:29:05] | stuarta: | i can shoot and replace my freebsd10 buildslave |
| [22:29:50] | hampton: | rhel80 beta is out too. I installed it this afternoon, but I'm having trouble figuring out how to configure it for updates. |
| [22:30:04] | stuarta: | heh, i have that somewhere too |
| [22:38:02] | ** stuarta downloads freebsd12 iso ** | |
| [22:49:34] | stuarta: | dang, still have to fix the trac patches for mysql support to get them to work on f28 |
| [22:50:44] | hampton: | Any luck on figuring out the magic incantation to get cppchek to run? |
| [22:51:39] | stuarta: | cppcheck is triggered by the buildbot, i think you mean coverity? |
| [22:52:16] | stuarta: | i've backed out the changes it was working on for the buildbot, so that it will at least do scheduled & triggered builds |
| [22:52:24] | stuarta: | which includes cppcheck & doxygen |
| [22:53:01] | hampton: | ok. I wanted to try and knock off a few of the top items in cppcheck before the release. |
| [22:53:47] | stuarta: | cppcheck will run again 3:15 AM GMT |
| [22:54:01] | stuarta: | so in about 4hr20m |
| [22:54:08] | hampton: | thanks |
| [22:54:17] | ** stuarta crosses fingers ** | |
| [23:30:38] | Tobbe5178534 (Tobbe5178534!~asdf@2001:2002:51eb:d24e:945c:d547:5be1:12f7) has quit (Read error: Connection reset by peer) | |
| [23:47:41] | natanojl (natanojl!~jonatan@mythtv/developer/natanojl) has quit (Ping timeout: 244 seconds) | |
IRC Logs collected by
BeirdoBot.
Please use the above link to report any bugs.