| -queuebot:#ubuntu-release- New binary: patiencediff [ppc64el] (eoan-proposed/universe) [0.1.0-1] (no packageset) | 05:12 | |
| -queuebot:#ubuntu-release- New binary: patiencediff [amd64] (eoan-proposed/universe) [0.1.0-1] (no packageset) | 05:12 | |
| -queuebot:#ubuntu-release- New binary: patiencediff [i386] (eoan-proposed/universe) [0.1.0-1] (no packageset) | 05:13 | |
| -queuebot:#ubuntu-release- New binary: patiencediff [armhf] (eoan-proposed/universe) [0.1.0-1] (no packageset) | 05:14 | |
| -queuebot:#ubuntu-release- New binary: patiencediff [arm64] (eoan-proposed/universe) [0.1.0-1] (no packageset) | 05:15 | |
| -queuebot:#ubuntu-release- New binary: patiencediff [s390x] (eoan-proposed/universe) [0.1.0-1] (no packageset) | 05:15 | |
| Laney | xnox: 🤘 | 08:31 |
|---|---|---|
| Laney | also git diff --word-diff is the best for reviewing autopkgtest-cloud config changes | 08:34 |
| -queuebot:#ubuntu-release- New: accepted cc-tool [amd64] (eoan-proposed) [0.26.20190527-1] | 08:34 | |
| -queuebot:#ubuntu-release- New: accepted cc-tool [armhf] (eoan-proposed) [0.26.20190527-1] | 08:34 | |
| -queuebot:#ubuntu-release- New: accepted cc-tool [ppc64el] (eoan-proposed) [0.26.20190527-1] | 08:34 | |
| -queuebot:#ubuntu-release- New: accepted golang-github-joyent-gosign [amd64] (eoan-proposed) [0.0~git20161114.9abcee2-1] | 08:34 | |
| -queuebot:#ubuntu-release- New: accepted patiencediff [armhf] (eoan-proposed) [0.1.0-1] | 08:34 | |
| -queuebot:#ubuntu-release- New: accepted patiencediff [s390x] (eoan-proposed) [0.1.0-1] | 08:34 | |
| -queuebot:#ubuntu-release- New: accepted cc-tool [arm64] (eoan-proposed) [0.26.20190527-1] | 08:34 | |
| -queuebot:#ubuntu-release- New: accepted cc-tool [s390x] (eoan-proposed) [0.26.20190527-1] | 08:34 | |
| -queuebot:#ubuntu-release- New: accepted patiencediff [i386] (eoan-proposed) [0.1.0-1] | 08:34 | |
| -queuebot:#ubuntu-release- New: accepted cc-tool [i386] (eoan-proposed) [0.26.20190527-1] | 08:34 | |
| -queuebot:#ubuntu-release- New: accepted patiencediff [arm64] (eoan-proposed) [0.1.0-1] | 08:34 | |
| -queuebot:#ubuntu-release- New: accepted patiencediff [amd64] (eoan-proposed) [0.1.0-1] | 08:35 | |
| -queuebot:#ubuntu-release- New: accepted swaybg [amd64] (eoan-proposed) [1.0-1] | 08:35 | |
| -queuebot:#ubuntu-release- New: accepted swaybg [armhf] (eoan-proposed) [1.0-1] | 08:35 | |
| -queuebot:#ubuntu-release- New: accepted swaybg [ppc64el] (eoan-proposed) [1.0-1] | 08:35 | |
| -queuebot:#ubuntu-release- New: accepted patiencediff [ppc64el] (eoan-proposed) [0.1.0-1] | 08:35 | |
| -queuebot:#ubuntu-release- New: accepted swaybg [i386] (eoan-proposed) [1.0-1] | 08:35 | |
| -queuebot:#ubuntu-release- New: accepted swaybg [arm64] (eoan-proposed) [1.0-1] | 08:35 | |
| -queuebot:#ubuntu-release- New: accepted swaybg [s390x] (eoan-proposed) [1.0-1] | 08:35 | |
| -queuebot:#ubuntu-release- New: rejected lsp-plugins [source] (eoan-proposed) [1.1.9-0ubuntu1] | 09:01 | |
| -queuebot:#ubuntu-release- New: accepted u-boot [arm64] (eoan-proposed) [2019.04+dfsg-2ubuntu1] | 09:02 | |
| -queuebot:#ubuntu-release- New source: lsp-plugins (eoan-proposed/primary) [1.1.9-0ubuntu1] | 09:02 | |
| -queuebot:#ubuntu-release- New: accepted lsp-plugins [source] (eoan-proposed) [1.1.9-0ubuntu1] | 09:03 | |
| -queuebot:#ubuntu-release- New binary: lsp-plugins [amd64] (eoan-proposed/universe) [1.1.9-0ubuntu1] (no packageset) | 09:19 | |
| -queuebot:#ubuntu-release- New binary: lsp-plugins [i386] (eoan-proposed/universe) [1.1.9-0ubuntu1] (no packageset) | 09:26 | |
| -queuebot:#ubuntu-release- Unapproved: lvm2 (cosmic-proposed/main) [2.02.176-4.1ubuntu3 => 2.02.176-4.1ubuntu3.18.10.1] (core) | 10:09 | |
| -queuebot:#ubuntu-release- Unapproved: lvm2 (xenial-proposed/main) [2.02.133-1ubuntu10 => 2.02.133-1ubuntu10.1] (core) | 10:10 | |
| -queuebot:#ubuntu-release- Unapproved: lvm2 (disco-proposed/main) [2.02.176-4.1ubuntu4 => 2.02.176-4.1ubuntu4.1] (core) | 10:10 | |
| -queuebot:#ubuntu-release- Unapproved: neutron (bionic-proposed/main) [2:12.0.5-0ubuntu5 => 2:12.0.6-0ubuntu1] (openstack, ubuntu-server) | 10:11 | |
| -queuebot:#ubuntu-release- Unapproved: neutron (disco-proposed/main) [2:14.0.0-0ubuntu1 => 2:14.0.0-0ubuntu1.1] (openstack, ubuntu-server) | 10:11 | |
| -queuebot:#ubuntu-release- Unapproved: neutron (cosmic-proposed/main) [2:13.0.2-0ubuntu3.2 => 2:13.0.2-0ubuntu3.3] (openstack, ubuntu-server) | 10:11 | |
| jamespage | if there is an sru team member around the 2:12.0.6-0ubuntu1 for neutron ^^ supercedes the existing one in the UNAPPROVED queue | 10:16 |
| xnox | Laney: 🎉🎉🎉 and triggered linux-* retries in bionic and disco now. Hopefully it gets better. | 10:23 |
| Laney | heh | 10:24 |
| Laney | I'd have probably done one at first ... | 10:24 |
| rbalint | bdmurray, RAOF, please accept the gce-compute-image-packages srus if you have time today | 10:32 |
| xnox | ah, well =) | 10:35 |
| RAOF | rbalint: I'll do them in my shift if bdmurray doesn't get to them first 😀 | 10:39 |
| === cpaelzer__ is now known as cpaelzer | ||
| -queuebot:#ubuntu-release- New source: grubzfs-testsuite (eoan-proposed/primary) [0.1] | 10:50 | |
| RAOF | It seems that the recent Qt5 security release to Bionic has included a regression in Wayland support? | 11:10 |
| RAOF | Alan Griffiths has more details; I'll get him to file a bug. | 11:11 |
| mdeslaur | RAOF: ok, please let me know the bug number and I'll take a look | 11:25 |
| alan_g | I'm just trying to figure out where to file it... | 11:35 |
| mdeslaur | alan_g: https://bugs.launchpad.net/ubuntu/+source/qtbase-opensource-src/+filebug | 11:37 |
| alan_g | Sorry for the delay. There's a factor I don't yet understand making it hard to document a reproducer. | 12:02 |
| mdeslaur | np | 12:03 |
| -queuebot:#ubuntu-release- Unapproved: newlib (bionic-proposed/universe) [2.4.0.20160527-3build1 => 2.4.0.20160527-3ubuntu0.1] (no packageset) | 12:34 | |
| ddstreet | tkamppeter are you able to verify the systemd in b/c-proposed for lp #1754671 | 12:37 |
| ubot5 | Launchpad bug 1754671 in network-manager (Ubuntu Bionic) "Full-tunnel VPN DNS leakage regression" [High,In progress] https://launchpad.net/bugs/1754671 | 12:37 |
| -queuebot:#ubuntu-release- Unapproved: cinder (cosmic-proposed/main) [2:13.0.3-0ubuntu1 => 2:13.0.5-0ubuntu1] (openstack, ubuntu-server) | 13:11 | |
| -queuebot:#ubuntu-release- Unapproved: heat (cosmic-proposed/main) [1:11.0.0-0ubuntu2 => 1:11.0.2-0ubuntu1] (openstack, ubuntu-server) | 13:22 | |
| -queuebot:#ubuntu-release- Unapproved: nova (cosmic-proposed/main) [2:18.1.0-0ubuntu3 => 2:18.2.0-0ubuntu1] (openstack, ubuntu-server) | 13:26 | |
| -queuebot:#ubuntu-release- Unapproved: horizon (cosmic-proposed/main) [3:14.0.2-0ubuntu4 => 3:14.0.3-0ubuntu1] (openstack, ubuntu-server) | 13:36 | |
| Eickmeyer | sil2100: Thank you so much!!!! | 14:20 |
| Eickmeyer | sil2100: Now I see that it's binNEW... what now? | 14:25 |
| === cpaelzer__ is now known as cpaelzer | ||
| sil2100 | Eickmeyer: ah! On it! | 14:39 |
| sil2100 | Forgot about it ;) | 14:39 |
| Eickmeyer | sil2100: Ha! No worries. | 14:39 |
| Eickmeyer | Well, no worries now that you're on it. :) | 14:39 |
| sil2100 | Eickmeyer: oh, so it FTBFS on all the other arches? | 14:40 |
| sil2100 | Eickmeyer: should we do a follow up upload limiting the list of arches to amd64 and i386? | 14:40 |
| sil2100 | Or should it actually build and be usable everywhere? | 14:40 |
| Eickmeyer | sil2100: It should build and be usable everywhere per the developer, at least in the bug report conversation we had yesterday. :S | 14:41 |
| Eickmeyer | That said, I'm not sure I'm willing to support every arch on the planet. | 14:41 |
| sil2100 | uh | 14:42 |
| Eickmeyer | sil2100: I stand corrected. He built on multiple systems/kernels, not architectures. Should be amd64/i386 only. | 14:43 |
| Eickmeyer | I had to go back and read the conversation. | 14:43 |
| Eickmeyer | sil2100: He did build on armv7, raspberry pi. | 14:44 |
| Eickmeyer | https://github.com/sadko4u/lsp-plugins/issues/50#issuecomment-498409847 | 14:44 |
| gitbot | sadko4u issue 50 in lsp-plugins "Tests fail on debian/ubuntu build " [Hotfix, Pending For Release, Feature Request, Closed] | 14:44 |
| sil2100 | Eickmeyer: could you find out what should be the state here? Like, I'd like the package to correspond to the requirements and needs of upstream | 14:46 |
| sil2100 | e.g. if upstream says it's amd64/i386 only, then let's modify the packaging as such - otherwise we'd need upstream to help us get it building on the respective architectures ;/ | 14:46 |
| Eickmeyer | sil2100: I'll file a new bug report/question and see what the response is. | 14:48 |
| Eickmeyer | sil2100: Bug report filed, waiting for an answer. Upstream has been very responsive. | 14:51 |
| sil2100 | Eickmeyer: thanks! Let's leave the binaries in there for now - anyway, we're 'almost there' now ;) | 14:57 |
| Eickmeyer | sil2100: You've been super encouraging through this. Thank you so much! | 15:01 |
| infinity | This doesn't look like an "upstream needs to help" situation. | 15:02 |
| infinity | It just shouldn't be calling compilers with -m32. | 15:02 |
| infinity | Also, the part where it seems to detect the target arch as "i586" on non-amd64 is a bit lolz. | 15:04 |
| Eickmeyer | sil2100: ^ | 15:05 |
| sil2100 | Well, I didn't look at the source of the problem so I wouldn't know - I leave it up to Eickmeyer and/or upstream to take care of that | 15:07 |
| sil2100 | (ETOOMANYTHINGS, sorry about that o/) | 15:08 |
| infinity | Well, the source of this problem is debian/rules | 15:08 |
| acheronuk | BUILD_PROFILE=i586 | 15:08 |
| acheronuk | in rules | 15:08 |
| infinity | acheronuk: Jinx. :) | 15:08 |
| acheronuk | if not amd64 | 15:08 |
| infinity | Upstream also defines aarch64 (for arm64) and armv7a (for armhf). | 15:09 |
| sil2100 | Sure, but I didn't see the project's Makefile's to know what impact it is | 15:09 |
| infinity | So at least those two would be trivial to enable in rules. | 15:09 |
| sil2100 | s/it is/it has/ | 15:09 |
| sil2100 | Anyway | 15:09 |
| * Eickmeyer is looking now | 15:09 | |
| sil2100 | Eickmeyer: thanks o/ | 15:10 |
| sil2100 | Eickmeyer: in case you get something that seems to be working, please push it to the vcs and give me a sign | 15:11 |
| Eickmeyer | sil2100: Will do. | 15:11 |
| sil2100 | Eickmeyer: but before that happens do some test-builds on a PPA with more than just amd64/i386 enabled | 15:11 |
| sil2100 | That's something I could have noticed before, since the autobuild PPA seems to only have those two arches | 15:12 |
| infinity | That said, all this BUILD_PROFILE stuff looks completely bogus. Tearing it all out would work better (as an upstream recommendation). | 15:12 |
| sil2100 | Eickmeyer: ^ maybe you could give that a try in a PPA? ;) | 15:12 |
| Eickmeyer | infinity: Yeah, that was stuff left over from the KXStudio repo (lots of poor packaging happens there, hence my personal undertaking of making it "unnecessary" for Studio users). I should've noticed it, sorry. | 15:13 |
| infinity | To be clear, I don't mean "removing it from debian/rules will have the desired result", I mean "the entire concept in the upstream makefiles is ridiculous because it's setting compiler defaults that would be correct if the developer didn't try to micromanage". | 15:14 |
| Eickmeyer | sil2100: That's my plan at the moment. Might have to drop everything to get my son ready for school. | 15:14 |
| infinity | Like, seriously, explicitly setting "-m elf_x86_64"? | 15:14 |
| Eickmeyer | infinity: It does appear to be made that way. :/ | 15:16 |
| infinity | Eickmeyer: I think for debian/ubuntu, I'd be inclined to add a "generic" BUILD_PROFILE to scripts/make/configure.mk that doesn't do any of this silliness, and just specify THAT one in debian/rules. | 15:16 |
| infinity | Eickmeyer: Then compiler defaults will win and DTRT, regardless of arch. | 15:17 |
| infinity | Eickmeyer: You already patch out the only braindead use of LD_PATH, so it's just CC_ARCH and LD_ARCH getting in your way here. | 15:18 |
| Eickmeyer | infinity: Right. As soon as I can figure out a way to patch out this stuff, then I'll get that done. In the meantime, though, I have to get my son out the door for school. Will report back. | 15:19 |
| Eickmeyer | infinity, sil2100: upstream only supports i386, arm64, aarch64, and armv7-ar. Does this affect anything? | 16:01 |
| infinity | Eickmeyer: Define "supports". | 16:01 |
| Eickmeyer | https://github.com/sadko4u/lsp-plugins/blob/04cc45a4aee3bd3a46ecbd0a9160d0d87f8f8a32/README.txt#L30-L47 | 16:01 |
| infinity | Eickmeyer: Upstream only has build targets for those arches, but as we've established, those build targets are bunk. | 16:01 |
| Eickmeyer | infinity: Then I guess I'm lost. Should I patch out the entire CC_ARCH and LD_ARCH variables? | 16:02 |
| infinity | Eickmeyer: So, IMO, if you suddenly built on other arches, they'd just also become "experimental" support by upstream's definition. No big. | 16:02 |
| infinity | Eickmeyer: Patching out hundreds of lines is much more effort than just adding a target that sets those variables blank, and using it. | 16:03 |
| infinity | Eickmeyer: ie: right before the x86_64 target, create a "generic" one. Then in debian/rules, always export BUILD_PROFILE=generic. DOne. | 16:03 |
| infinity | There's literally zero reason to build with -m64/-m32 or -m elf_foo on any sane Debian/Ubuntu toolchain. We already pick the right things for ABI compat with our ports. | 16:04 |
| Eickmeyer | infinity: Okay, That makes sense. | 16:08 |
| infinity | Ahh, balls. | 16:08 |
| infinity | No, wait, I found one last place where BUILD_PROFILE is actually legit used. | 16:08 |
| Eickmeyer | Standing by. | 16:08 |
| infinity | https://github.com/sadko4u/lsp-plugins/blob/04cc45a4aee3bd3a46ecbd0a9160d0d87f8f8a32/src/dsp/Makefile | 16:08 |
| infinity | So, we *do* want a generic, but it should only be selected as a fallback. | 16:09 |
| infinity | You want to beef up your debian/rules selection to select i586 for i386, x86_64 for amd64, armv7a for armhf, aarch64 for arm64, and generic as the fallback. | 16:09 |
| infinity | Then it should all DTRT. | 16:09 |
| infinity | (THose targets are still "wrong", but they're wrong in harmless ways, so whatever) | 16:10 |
| Eickmeyer | infinity: That thought had crossed my mind. I'll go ahead and do it. | 16:10 |
| infinity | Eickmeyer: And, actually, those vars are set empty before the test, so you don't actually need to patch in a "generic" BUILD_PROFILE is you don't want to. | 16:15 |
| infinity | Just setting BUILD_PROFILE to generic (or any garbage) for arches that don't have an explicit profile definition should DTRT. | 16:15 |
| infinity | So all you need to fix is debian/rules. | 16:16 |
| Eickmeyer | infinity: That can be done. | 16:16 |
| infinity | Eickmeyer: And, IMO, if it still FTBFS or otherwise sucks on arches upstream doesn't define, that's fine, IMO. Just a best effort attempt to set those 4 profiles correctly and *try* to build generically on everything else is fine. | 16:17 |
| Eickmeyer | infinity: Works for me. | 16:17 |
| Eickmeyer | infinity, sil2100: Made the changes, pushed, building in my personal PPA (to avoid enabling other architectures in the Studio autobuilds PPA). | 16:43 |
| Eickmeyer | infinity, sil2100: Successful builds for those architectures. We're good to go. | 16:57 |
| infinity | Eickmeyer: Some nitpicking about the changes in your PPA before you push, but the intent is there. :) | 17:05 |
| Eickmeyer | infinity: I haven't pushed the tags, if that's what you mean. | 17:06 |
| infinity | Eickmeyer: The major one is that you want DEB_HOST_ARCH, not DEB_HOST_ARCH_CPU | 17:06 |
| infinity | Eickmeyer: Also, "armhl" should be "armhf" | 17:07 |
| Eickmeyer | heh, silly me and my typo. | 17:07 |
| infinity | Testing for DEB_HOST_ARCH_CPU means most of those were just building "generic" instead. | 17:07 |
| infinity | Since CPU is x86_64, i386 (sometimes i686), armv7a, etc. | 17:08 |
| Eickmeyer | Uh, interesting. Still successfully built. Makes me question if those lines are even necessary, though you did discover the reason, so I trust you. | 17:09 |
| infinity | Eickmeyer: Sure, it'll build, but for amd64, you'd skip building the optimised bits. | 17:09 |
| infinity | Same for arm64 and armhf. | 17:09 |
| Eickmeyer | Ok. | 17:10 |
| infinity | Or... | 17:12 |
| infinity | Maybe what's actually happening is you broke it so well that it fixed it. :) | 17:12 |
| infinity | In that because none of those match, BUILD_PROFILE isn't being set, and the upstream makefile is selecting the right one based on uname(1) :P | 17:13 |
| infinity | Which only works on Ubuntu buildds, because we force uname to match the target build arch. | 17:13 |
| infinity | It would fail miserably in Debian. | 17:13 |
| infinity | Or local builds. | 17:13 |
| Eickmeyer | I guess I know how to break things. :D | 17:14 |
| infinity | Yep, that's definitely what's happened. | 17:14 |
| infinity | Entertaining, though. | 17:15 |
| Eickmeyer | Indeed! lol | 17:15 |
| infinity | No real point in using filter() there either. | 17:17 |
| infinity | Just ifeq ($(DEB_HOST_ARCH),arm64) is enough. | 17:18 |
| Eickmeyer | infinity: ack | 17:21 |
| infinity | Eickmeyer: It also seems to be missing an "include /usr/share/dpkg/architecture.mk" before that ifeq mess. | 17:24 |
| infinity | Eickmeyer: Which sort of makes it all a no-op... | 17:24 |
| infinity | Unless dh(1) does something fancy there on re-entry. | 17:24 |
| infinity | But not that I know of. | 17:25 |
| Eickmeyer | infinity: I'm learning a lot today. | 17:25 |
| -queuebot:#ubuntu-release- Unapproved: accepted gce-compute-image-packages [source] (disco-proposed) [20190522-0ubuntu1~19.04.0] | 17:25 | |
| infinity | Eickmeyer: That looks like it might actually do what you expect now. | 17:29 |
| Eickmeyer | infinity: Cool. I'll be watching the build. :) | 17:31 |
| -queuebot:#ubuntu-release- Unapproved: accepted gce-compute-image-packages [source] (cosmic-proposed) [20190522-0ubuntu1~18.10.0] | 17:36 | |
| -queuebot:#ubuntu-release- Unapproved: accepted gce-compute-image-packages [source] (bionic-proposed) [20190522-0ubuntu1~18.04.0] | 17:42 | |
| -queuebot:#ubuntu-release- Unapproved: accepted gce-compute-image-packages [source] (xenial-proposed) [20190522-0ubuntu1~16.04.0] | 17:45 | |
| -queuebot:#ubuntu-release- Unapproved: accepted exim4 [source] (disco-proposed) [4.92-4ubuntu1.1] | 18:02 | |
| -queuebot:#ubuntu-release- Unapproved: accepted ktouch [source] (disco-proposed) [4:18.12.3-0ubuntu1.1] | 18:05 | |
| Eickmeyer | infinity, sil2100: Looks like that actually gave some decent builds. | 18:26 |
| -queuebot:#ubuntu-release- Unapproved: accepted hw-detect [source] (disco-proposed) [1.117ubuntu6.19.04.1] | 18:26 | |
| infinity | Eickmeyer: \o/ | 18:33 |
| Eickmeyer | infinity: Do you want to reject sil2100's upload of lsp-plugins? Then, if you want to, you can reupload and approve. | 18:33 |
| Eickmeyer | Unless sil2100 wants to do it. :) | 18:34 |
| infinity | Eickmeyer: Gimme a .dsc somewhere to sponsor for you? | 18:34 |
| infinity | I'm curious about the testsuite override. What/how does it break? | 18:35 |
| -queuebot:#ubuntu-release- New: rejected lsp-plugins [amd64] (eoan-proposed) [1.1.9-0ubuntu1] | 18:35 | |
| -queuebot:#ubuntu-release- New: rejected lsp-plugins [i386] (eoan-proposed) [1.1.9-0ubuntu1] | 18:35 | |
| Eickmeyer | infinity: https://code.launchpad.net/~eeickmeyer/+archive/ubuntu/ppa/+packages | 18:35 |
| infinity | Eickmeyer: Well, yes, I saw that, but that lacks a correct changelog entry and version. | 18:36 |
| Eickmeyer | Ah, yes. Standby... | 18:36 |
| * Eickmeyer has to move to a system that'll actually do the build manually, not this Celeron he's currently typing on | 18:36 | |
| infinity | They still make Celerons? | 18:37 |
| Eickmeyer | Dual-core at that! | 18:37 |
| infinity | Oh, the test override is commented out. Maybe you should delete that cruft. :P | 18:39 |
| Eickmeyer | infinity: I thought I had.... weird. | 18:40 |
| infinity | ... | 18:41 |
| infinity | Also, you end up building twice... | 18:41 |
| Eickmeyer | ... | 18:42 |
| infinity | And I suspect what you install is the binaries you built in the second pass, since I don't see it building them somewhere else. | 18:42 |
| * Eickmeyer shakes fist at upstream | 18:42 | |
| infinity | Oh, wait. | 18:42 |
| Eickmeyer | That's exactly what vorlon and I were trying to avoid. | 18:43 |
| infinity | It builds the build in .build and the test in .test | 18:43 |
| infinity | I think. | 18:43 |
| -queuebot:#ubuntu-release- Unapproved: accepted hw-detect [source] (cosmic-proposed) [1.117ubuntu6.18.10.1] | 18:43 | |
| infinity | So hard to decipher with the non-verbose build. | 18:43 |
| Eickmeyer | Yes. That was upstream's doing. | 18:43 |
| infinity | Okay. | 18:43 |
| infinity | Gross, but okay. | 18:43 |
| Eickmeyer | Indeed. | 18:44 |
| infinity | OOI, why are the test binaries "not suitable" to ship? | 18:44 |
| cjwatson | Policy says to override upstream and make the build verbose | 18:44 |
| cjwatson | Hopefully there's a V=1 switch or similar | 18:44 |
| cjwatson | (https://www.debian.org/doc/debian-policy/ch-source.html#main-building-script-debian-rules) | 18:44 |
| Eickmeyer | I honestly don't know. The "make test" doesn't do what one would expect. | 18:45 |
| -queuebot:#ubuntu-release- Unapproved: accepted hw-detect [source] (bionic-proposed) [1.117ubuntu6.18.04.1] | 18:45 | |
| infinity | Ahh, -DLSP_TESTING -DLSP_TRACE | 18:46 |
| infinity | It's using that to include extra trace and breakpoint vomit. | 18:46 |
| infinity | Gross, but whatever. | 18:46 |
| Eickmeyer | So should I keep that as-is? | 18:47 |
| infinity | Yeah, it's fine. | 18:47 |
| infinity | Ignore me. | 18:47 |
| Eickmeyer | heh, no worries. | 18:47 |
| infinity | Making the upstream Makefiles verbose would be lovely, but unfortunately, that's not practical either. | 18:47 |
| infinity | Cause he literally just prefaces everything with @ | 18:48 |
| Eickmeyer | Shoddy makefiles. | 18:48 |
| infinity | It'd be a nightmare to replace them all with a macro in a distro patch. | 18:48 |
| vorlon | Eickmeyer: so, that double-build (test one build, ship one build) is exactly what I said we should avoid in the package. If this is really the status quo, the dh_auto_test override should come back. | 18:52 |
| Eickmeyer | vorlon, infinity: In that case, I'll revert the change I just made. | 18:53 |
| Eickmeyer | IMO, upstream doesn't understand the meaning of "test suite". | 18:53 |
| -queuebot:#ubuntu-release- Unapproved: accepted libguestfs [source] (disco-proposed) [1:1.40.2-1ubuntu1.1] | 19:00 | |
| Eickmeyer | infinity: uploading to my (other) ppa. | 19:07 |
| Eickmeyer | infinity: I have a dsc file for you at https://launchpad.net/~eeickmeyer/+archive/ubuntu/ppa2/+packages | 19:15 |
| infinity | Eickmeyer: ACK, will look when this TB meeting lets out. | 19:19 |
| -queuebot:#ubuntu-release- Unapproved: accepted libguestfs [source] (cosmic-proposed) [1:1.38.4-1ubuntu2.1] | 19:29 | |
| -queuebot:#ubuntu-release- Unapproved: accepted libguestfs [source] (bionic-proposed) [1:1.36.13-1ubuntu3.3] | 19:32 | |
| vorlon | E: librados2: symbols-file-contains-current-version-with-debian-revision on symbol | 19:50 |
| vorlon | _ZN5boost6spirit2qi4ruleIN9__gnu_cxx17__normal_iteratorIPKcNSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEEEEEF11MonCapGrantvENS0_11unused_typeESG_SG_E6defineIN4mpl_5bool_ILb1EEENS_5proto7exprns_4exprINSM_6tagns_3tag11shift_rightENSM_7argsns_5list2IRKNSO_ISR_NST_IRKNSO_ISR_NST_IRKNSO_ISR_NST_IRKNSO_ISR_NST_IRKNSO_ISR_NST_IRKNSO_ISR_NST_IRKNSO_ISR_NST_IRKNSO_ISR_NST_IRKNSO_INSQ_6negateENSS_5list1I | 19:50 |
| vorlon | RNS2_ISD_SG_SG_SG_SG_EEEELl1EEERKNSO_ISU_NSV_IRKNSO_ISR_NST_IRKNSO_INSQ_8terminalENSS_4termINS0_11terminal_exINS0_3tag3litENS_6fusion6vectorIJRA6_S5_EEEEEEELl0EEESX_EELl2EEEEELl1EEEEELl2EEERKNSO_IS12_NS13_INS14_IS16_NS18_IJRA8_S5_EEEEEEELl0EEEEELl2EEERKNSO_INSQ_10bitwise_orENST_IRKNSO_IS12_NS13_INS14_IS16_NS18_IJcEEEEEEELl0EEESX_EELl2EEEEELl2EEERKNSO_IS12_NS13_INS14_INS15_4attrENS18_IJSC_EEEEEEELl0E | 19:50 |
| vorlon | EEEELl2EEERNS2_ISD_FSC_vESG_SG_SG_EEEELl2EEES2Q_EELl2EEERKNSO_IS12_NS13_INS14_IS2K_NS18_IJSt3mapISC_16StringConstraintSt4lessISC_ESaISt4pairIKSC_S37_EEEEEEEEEELl0EEEEELl2EEERKNSO_IS12_NS13_INS14_IS2K_NS18_IJiEEEEEEELl0EEEEELl2EEERKNSO_ISU_NSV_IRKNSO_ISR_NST_IRKNSO_ISR_NST_IRKNSO_ISR_NST_ISX_S20_EELl2EEESX_EELl2EEES2X_EELl2EEEEELl1EEEEELl2EEEEEvRSH_RKT0_SL_@Base and 1 others | 19:50 |
| vorlon | :P | 19:50 |
| infinity | SPAM | 19:50 |
| Eickmeyer | !paste | 19:50 |
| ubot5 | For posting multi-line texts into the channel, please use https://paste.ubuntu.com | To post !screenshots use https://imgur.com/ !pastebinit to paste directly from command line | Make sure you give us the URL for your paste - see also the channel topic. | 19:50 |
| infinity | Eickmeyer: Erm, you wanted -0ubuntu2 there, not ubuntu1. | 19:51 |
| infinity | Eickmeyer: ubuntu1 is already in the archive. | 19:51 |
| Eickmeyer | infinity: d'oh! | 19:51 |
| vorlon | I mean, yes, but pastebinning would ruin the effect | 19:51 |
| Eickmeyer | will fix. | 19:51 |
| vorlon | for those playing along at home, c++filt also chokes on that symbol name | 19:51 |
| infinity | Eickmeyer: Make sure the upload includes the original ubuntu1 changelog entry, and then ubuntu2 with the fixes. | 19:52 |
| Eickmeyer | infinity: ack | 19:52 |
| infinity | Eickmeyer: Debdiff against "pull-lp-source -d lsp-plugins" to make sure it's sane. | 19:52 |
| vorlon | (chokes gracefully - it refuses to demangle it. I was actually curious to see if c++filt would spiral out of control) | 19:52 |
| infinity | vorlon: Didn't I hear doko talking about some new and wonderful utopia where templated symbols will eff right off and we can live happily ever after? | 19:53 |
| infinity | But yeah, that one's pretty amazing. | 19:53 |
| vorlon | infinity: which do you bet is larger, the symbol name or the implementation? | 19:53 |
| infinity | The worst part is how much of it I can demangle in my head without a tool. | 19:54 |
| infinity | This is not knowlege I should have, and I resent it being in there. | 19:54 |
| vorlon | haha | 19:56 |
| Eickmeyer | infinity: File lists identical (after any substitutions), so sanity is ensured? | 20:10 |
| Eickmeyer | FYI, my next package is a cakewalk compared to this one. | 20:13 |
| infinity | Eickmeyer: I meant debdiffing the sources. :) | 20:15 |
| infinity | Eickmeyer: To make sure the changelog looked sane, etc. | 20:16 |
| Eickmeyer | infinity: Oh. That showed only my changes/changelog addition. | 20:16 |
| infinity | Eickmeyer: Kay. Gimme? | 20:16 |
| Eickmeyer | Sure. | 20:16 |
| -queuebot:#ubuntu-release- Unapproved: kexec-tools (bionic-proposed/main) [1:2.0.16-1ubuntu1 => 1:2.0.16-1ubuntu1.1] (core) | 20:18 | |
| Eickmeyer | Wait a minute... | 20:19 |
| -queuebot:#ubuntu-release- Unapproved: rejected mesa [source] (disco-proposed) [19.0.2-1ubuntu1.1] | 20:21 | |
| -queuebot:#ubuntu-release- Unapproved: kexec-tools (cosmic-proposed/main) [1:2.0.16-1ubuntu3 => 1:2.0.16-1ubuntu3.1] (core) | 20:21 | |
| Eickmeyer | infinity: This is what it spat-out, but maybe I'm doing something wrong. The only source I can find is the tarball. https://paste.ubuntu.com/p/yN4t9hhxr5/ | 20:22 |
| Eickmeyer | I have the feeling I should update to eoan. | 20:22 |
| -queuebot:#ubuntu-release- Unapproved: accepted lvm2 [source] (disco-proposed) [2.02.176-4.1ubuntu4.1] | 20:24 | |
| Eickmeyer | infinity: Anyhow, updated .dsc file | 20:24 |
| infinity | Eickmeyer: The diff against the archive version is https://paste.ubuntu.com/p/B8K7jbqZMd/ | 20:28 |
| infinity | Eickmeyer: Which LGTM. | 20:28 |
| Eickmeyer | infinity: Cool. Sorry about my incompetence on that one. Still learning. | 20:29 |
| infinity | Eickmeyer: Uploaded. | 20:30 |
| Eickmeyer | infinity: Thanks! Are we going to have to wait for a second AA to approve? | 20:30 |
| apw | that is for eoan, so no | 20:31 |
| infinity | binNEW will still need a review, but I can do that when they pop up. | 20:32 |
| infinity | I suppose I should have reviewed the binaries from the previous upload in case they were obviously busted. :P | 20:32 |
| infinity | Oh well. | 20:32 |
| Eickmeyer | heh. Thanks for your help today. I have two other packages in the sponsorship queue, so we can get that straightened out as soon as a MOTU can look at them. | 20:33 |
| infinity | Eickmeyer: Ahh, thanks to include/dsp/types.h it won't build on anything but the 4 arches you enabled, but that's fine. | 20:35 |
| Eickmeyer | infinity: Ha! I was just typing to mention that. | 20:36 |
| infinity | I mean, it's kinda ridiculous, not fine, but I don't expect you to port it to PPC and S/390 either. | 20:36 |
| Eickmeyer | Gotcha. Honestly, it would be rediculous for someone running audio plugins on those architectures anyhow (including ARM imo, but crazier things have happened). | 20:37 |
| infinity | Ridiculous on an s390x mainframe, maybe (though, if they can be used for command-line processing, maybe not, it's crazy fast) | 20:38 |
| infinity | But not at all weird to do multimedia stuff on PPC. | 20:38 |
| infinity | Either way, carefactor low, if I crusaded to make every upstream write portable code, I wouldn't have time for basic bodily functions. | 20:38 |
| cjwatson | Might be quite fun to try to get LP appservers working on a mainframe :) | 20:40 |
| infinity | cjwatson: Honestly, if we could get a couple more of them, I'd be down with moving our entire infra to s390x. | 20:40 |
| infinity | cjwatson: But it's not like we're likely to pay for that, and IBM seems shy about sending us spares. | 20:40 |
| infinity | Can't imagine why. | 20:40 |
| cjwatson | (More realistically the database, but that's probably boring because PostgreSQL) | 20:40 |
| cjwatson | IronStack | 20:41 |
| infinity | Plus, you get a free Thinkpad with every mainframe, so that's a super good deal. | 20:42 |
| * cjwatson orders the reinforced floor | 20:42 | |
| infinity | Well, you also get a free POWER8/9 (the storage subsystem). | 20:42 |
| infinity | It's three computers for the price of one! | 20:42 |
| infinity | Well, for the price of dozens. | 20:42 |
| infinity | But still. | 20:42 |
| -queuebot:#ubuntu-release- Unapproved: accepted lvm2 [source] (cosmic-proposed) [2.02.176-4.1ubuntu3.18.10.1] | 20:46 | |
| -queuebot:#ubuntu-release- New binary: lsp-plugins [amd64] (eoan-proposed/universe) [1.1.9-0ubuntu2] (no packageset) | 20:52 | |
| -queuebot:#ubuntu-release- New binary: lsp-plugins [i386] (eoan-proposed/universe) [1.1.9-0ubuntu2] (no packageset) | 20:54 | |
| -queuebot:#ubuntu-release- Unapproved: accepted neutron [source] (disco-proposed) [2:14.0.0-0ubuntu1.1] | 20:56 | |
| -queuebot:#ubuntu-release- New binary: lsp-plugins [arm64] (eoan-proposed/universe) [1.1.9-0ubuntu2] (no packageset) | 21:08 | |
| -queuebot:#ubuntu-release- New binary: lsp-plugins [armhf] (eoan-proposed/universe) [1.1.9-0ubuntu2] (no packageset) | 21:26 | |
| -queuebot:#ubuntu-release- Unapproved: accepted neutron [source] (cosmic-proposed) [2:13.0.2-0ubuntu3.3] | 22:24 | |
| -queuebot:#ubuntu-release- Unapproved: rejected neutron [source] (bionic-proposed) [2:12.0.6-0ubuntu1] | 22:29 | |
| -queuebot:#ubuntu-release- Unapproved: accepted neutron [source] (bionic-proposed) [2:12.0.6-0ubuntu1] | 22:42 | |
| bdmurray | connor_k: should bug 1828615 also be tested with the 4.15 kernel? | 23:12 |
| ubot5 | bug 1828615 in v4l2loopback (Ubuntu Bionic) "v4l2loopback 10.0-1ubuntu1 ADT test failure with linux 5.0.0-14-generic" [Undecided,Fix committed] https://launchpad.net/bugs/1828615 | 23:12 |
| connor_k | bdmurray, I tested it with 4.15.0-49 and 5.0.0-14 | 23:13 |
| bdmurray | connor_k: I don't see that in comment #5 | 23:15 |
| connor_k | bdmurray, sorry, you're right. Those tests were for before proposed, let me verify on 4.15 real quick, I'll upload that as well | 23:19 |
| bdmurray | connor_k: thanks, I'm here for a bit yet so just ping me | 23:19 |
| connor_k | bdmurray, just added the test for 4.15 to comment #6: https://bugs.launchpad.net/ubuntu/+source/v4l2loopback/+bug/1828615/comments/6 | 23:28 |
| ubot5 | Launchpad bug 1828615 in v4l2loopback (Ubuntu Bionic) "v4l2loopback 10.0-1ubuntu1 ADT test failure with linux 5.0.0-14-generic" [Undecided,Fix committed] | 23:28 |
| vorlon | jamespage: hi, when you write that ceph's packaging is re-synced with "upstream", what upstream is that? It's not Debian AFAICS and there are various binary NEW packages which I'm trying to understand | 23:51 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!