/srv/irclogs.ubuntu.com/2018/10/10/#ubuntu-devel.txt

tsimonq2@pilot in01:05
=== udevbot_ changed the topic of #ubuntu-devel to: Archive: Feature Freeze (Cosmic Cuttlefish) | 18.04 released | Devel of Ubuntu (not support) | Build failures: http://qa.ubuntuwire.com/ftbfs/ | #ubuntu for support and discussion of Trusty-Bionic | If you can't send messages here, authenticate to nickserv first | Patch Pilots: tsimonq2
tsimonq2This is more of an experiment for now, but I'm following what cyphermox set out in his email: https://lists.ubuntu.com/archives/ubuntu-devel/2018-September/040501.html01:05
tsimonq2Maybe the bot can eventually be updated to distinguish +1 vanguards vs Patch Pilots but I'll also be looking at the sponsorship queue.01:06
tsimonq2Ah, I see not much SRU processing lately but it's understandable given the state of the development cycle...01:08
sarnoldwoah a patch pilot01:08
sarnoldthis takes me back :)01:08
tsimonq2Yeah, hehe01:09
tsimonq2sarnold: If you're around, how's this apparmor profile look? bug 178810201:21
ubottubug 1788102 in ntpsec (Ubuntu) "ntpsec's ntpd fails to write ntp.drift file because of apparmor" [Undecided,Confirmed] https://launchpad.net/bugs/178810201:21
tsimonq2I don't personally know enough about apparmor so your opinion on it either way would be welcome.01:21
sarnoldtsimonq2: lgtm, richard's usually on the ball01:21
tsimonq2Cool.01:22
tsimonq2sarnold: Uploaded to the queue, thanks!01:25
sarnoldtsimonq2: woot, thanks :)01:25
=== giraffe is now known as Guest86558
tsimonq2@pilot out02:05
=== udevbot_ changed the topic of #ubuntu-devel to: Archive: Feature Freeze (Cosmic Cuttlefish) | 18.04 released | Devel of Ubuntu (not support) | Build failures: http://qa.ubuntuwire.com/ftbfs/ | #ubuntu for support and discussion of Trusty-Bionic | If you can't send messages here, authenticate to nickserv first | Patch Pilots:
tsimonq2https://lists.ubuntu.com/archives/ubuntu-release/2018-October/004611.html02:05
=== zhongjun2_ is now known as zhongjun2
cyphermoxtsimonq2: I did ask how we could update the bot code to distinguish (or replace, since patch pilot is largely a dead thing)04:15
tsimonq2cyphermox: I don't even know where the code is for it. :/04:15
=== lan3y is now known as Laney
=== _hc is now known as M_hc_[m]
=== M_hc_[m] is now known as M_hc[m]
coreycbdoko: i know python3.7 will be the default in the next ubuntu release, but out of curiosity will it also have python3.6 available?12:18
xnoxcoreycb, no, the plan is python3.7 only12:21
xnoxcoreycb, for 19.04 that is.12:21
coreycbxnox: ok thanks, and yes 19.04 is what I was curious about12:22
coreycbxnox: just formulating an email upstream12:22
xnoxcoreycb, does that have some implications? as in, why are you asking? =)12:22
coreycbxnox: well i'm going to try to enable py37 tests upstream. there might be some push back because not not all tests are passing yet from recent py36 enablement.12:24
coreycbxnox: looks fairly trivial to enable, but a decent amount of busy work across many projects. getting fixes may be another story. hopefully i can convince them it needs to be enabled and get community focus on fixes soon.12:25
=== cpaelzer_ is now known as cpaelzer
rbasakjuliank: thank you for the version regression report. I wonder if we could perhaps add this to the pending sru report? Or should it be separate?12:51
juliankrbasak: I don't think it's something to consider in the SRU reporting. It also applies to security updates, and for those, there sometimes are valid reasons to temporarily have the new release in a stable release first12:52
juliankbut not sure12:54
xnoxwell, a new archive report to be run and published on ~ubuntu-archive12:55
rbasakI just wondered about when someone will look.12:55
xnoxjuliank, you do know ubuntu-archive-tools repository?12:56
rbasakIf left to the AAs then I suppose they have their own separate reports to check for all the other wrong things too12:56
juliankxnox: yes12:56
xnoxadd it there =)12:56
juliankI think optimally it should also query security thing and check if any CVEs are fixed in older releases, but not in newer.12:56
juliankxnox: I will soon12:57
juliankrbasak: I'm not sure if there's anything actionable for the SRU team except for not releasing an SRU until it has a higher version number in later releases12:57
rbasakjuliank: it might be useful to block releasing to updates (from proposed).12:58
juliankMaybe britney could check that automatically12:58
rbasakI know I've accepted from the queue into proposed before with the caveat of "not in updates until fixed in <future>"12:58
rbasakThere is talk of having britney do the release to updates, for all the extra britney checking too.12:59
xnoxit's not actually something we'd want to block on.12:59
xnoxit is legit for something to be good in one release, and stuck in proposed in another, as long as we sort it out eventually.13:00
xnoxa separate report for now, would be good enough.13:00
rbasakxnox: I'm talking about something different - when the fix isn't in a future release (not even proposed), but the SRU is uploaded to a previous release.13:00
juliankthe problem if the versioning in an SRU is wrong though, and it's in proposed, is that you can't go back to an older version number AFAIUI13:01
xnoxrbasak, which is nothing to do with this report.... cause it acts on what's accepted, not what's proposed.13:01
julianki.e. I upload -ubuntu0.1 to bionic, then -ubuntu0.16.04.1 to xenial13:01
juliankI'd have to upload -ubuntu0.0.16.04.1 to xenial to fix the mess I created13:02
rbasakOh, I see.13:02
juliankxnox: It does show you if the new release has a proposed one though13:02
julianki.e. those would be greyed out a bit13:02
juliankin html13:02
xnoxbut like it doesn't look at the queue13:02
juliankyeah13:03
tsimonq2rbasak: Britney doing the release to updates> Oooh, I'm curious about that. Autopkgtests and all? Or just a seven day aging period plus bug tags being in order? Is there discussion about this somewhere that I could read up on? :)13:05
rbasaktsimonq2: AFAIK, it's just something people have mentioned should happen. I don't know that anyone's working on it, and so I don't believe there are any other details.13:07
xnoxtsimonq2, it is running, and does report adt regressions, but i don't think we actually use "unblock" to release sru13:08
tsimonq2That would be cool.13:08
rbasakHmm. On the lines of something being in development -proposed, I have a case where something is in development unapproved but with an SRU ready to go.13:16
rbasakI presume that's OK as well, but it'd be nice to see it in cosmic-proposed since then it's "guaranteed" to land.13:17
rbasakI wonder if block-proposed and an accept is appropriate?13:17
rbasakOr easier if a release team member could review systemd unapproved in Cosmic please?13:18
rbasakxnox: do you have a git tree or similar for systemd anywhere? Trying to match up pieces of your delta with the changelog.13:31
rbasakxnox: on that point, given you don't mention the patch names in the changelog, it'd be nice if you used Bug-Ubuntu dep3 headers to help match things up.13:31
xnoxrbasak, no, we don't use dep3 headers, we use git-buildpackage quilt to manage them13:36
xnoxthe git is at13:36
xnoxrbasak, https://code.launchpad.net/~ubuntu-core-dev/+git/systemd13:37
xnoxand it is indeed broken by change by commit, and changelog is git-dch managed too13:37
rbasakxnox: thanks! Just as I've finished breaking down your upload manually :-/13:37
rbasakI'm not sure that's a reason to not have dep3 headers though13:38
xnox(the no-dep3 thing kind of comes from debian.... cause their patchset is large, and then we add things ontop and interweb upstream cherrypicks)13:38
rbasakYou can add dep3 to patches that you add though, surely?13:38
xnoxrbasak, well there are topics, so upstream cherrypicks are first, then debian topic debian-specific-patches and then debian topic UBUNTU- prefix is ubuntu-specific stuff13:39
rbasakeg. debian/patches/core-move-enforcement-of-the-start-limit-into-per-unit-type-code-again.patch has one, just with no Bug-Ubuntu.13:39
xnoxbut gbp pq import/export doesn't process those13:39
xnoxand they don't end up in the changelog13:39
xnoxLP: # -> do13:39
xnoxcause gbp pq, tries to ensure that patches are `git am`ble & `git format-patch`able & dpkg `quilt-applicable`13:40
xnoxrbasak, if gbp pq gains dep3 support, that would be nice.13:40
rbasakYes but AIUI none of that prevents you from adding Bug-Ubuntu dep3 headers13:40
rbasakPut it in the commit message and it'll end up in the quilt patch, no?13:41
xnoxi'd still have to put in LP: #, as well13:41
rbasakYes you would.13:41
rbasakAs you would when adding dep3 headers and not using git.13:41
xnoxhave you heard of DRY?! =)13:41
rbasakAll I'm saying is it makes it difficult to review.13:41
xnoxLP: # urls are clickable on all ubuntu terminals13:41
rbasakdep3 solves the problem, allowing reviewers to match things up.13:42
xnoxdo you not use gnome-terminal?13:42
rbasakAn LP reference in the changelog doesn't tell me what the corresponding quilt patch is.13:42
xnoxi'm not sure how dep3 helps here13:42
rbasakSome people put the quilt patch name in the changelog entries. You do not, which is fine, but then I'd like dep3 to help me match things up.13:42
xnoxyeah cause that is only visible in the git repo13:42
rbasakWhich I didn't have before, and Vcs-Git doesn't point to it.13:43
rbasakIn any case, AIUI, uploads are expected to follow convention and stand on their own without VCS.13:43
xnoxsure13:43
sforsheeLocutusOfBorg: it turns out that the upstream vboxvideo does not have dependencies on vboxguest like I thought, so we can just disable it13:44
sforsheeI've sent patches to start using the dkms drivers again - https://lists.ubuntu.com/archives/kernel-team/2018-October/095882.html13:44
sforsheeOdd_Bloke: ^13:44
xnoxcause like our ./debian/git-cherry-pick script which we use for cherrypicking things from upstream, and injecting it in the right place in the quilt stacks doesn't autogenerate stanzas either13:45
xnoxas we use plain `git cherry-pick` there13:45
* xnox ponders if that can grow "better" referencing integration13:45
rbasakI don't think it's OK to blame your tooling. As we established, it's trivial for you to add dep3 in your commit messages like everyone else does.13:45
rbasak(AFAIK, everyone else using dep3 currently writes them by hand)13:46
xnoxrbasak, the only patch with dep3 i see, was actually contributed to the upload and was not done by me. and doesn't actually follow systemd packaging conventions.13:48
xnoxbut i tend to take contributions as is, if they apply correctly and work correctly, cause i value their contibutions to systemd packaging.13:49
xnoxthus dep3 header in that one patch is exception, in systemd, rather than the norm.13:49
rbasakxnox: my request still stands. Please use dep3 headers when uploading SRUs, since it makes reviewing easier.13:52
LocutusOfBorgsforshee, did you take vboxvideo from which package?13:56
LocutusOfBorgI added a kernel 4.18 patch in 5.2.1813:56
xnoxhttps://git.launchpad.net/~ubuntu-core-dev/+git/systemd/commit/?id=6a8b3e99b74704b66a5f9422a828f58f532ac3af https://git.launchpad.net/~ubuntu-core-dev/+git/systemd/commit/?id=cc8f39042c5b47bc853b7450f3fe80467dd5897c https://git.launchpad.net/~ubuntu-core-dev/+git/systemd/commit/?id=fd832930ef280c9a4a9dda2440d5a46a6fdb623213:59
Odd_Blokesforshee: \o/14:01
Odd_Blokesforshee: When might I expect to see that land?14:02
sforsheeLocutusOfBorg: using the upstream vboxvideo still, as in bionic14:05
sforsheeOdd_Bloke: can't say for sure, but we'll have to upload it very soon to be in the release14:06
sforsheeLocutusOfBorg: I updated the drivers from 5.2.18-dfsg-2 in those patches14:07
LocutusOfBorgack14:11
Odd_Blokesforshee: Cool, thanks!14:15
rbasakxnox: the AutomountResult enum in src/core/automount.h doesn't form part of any ABI does it?14:56
rbasak(or API even)14:56
rbasakxnox: same question with {busname,mount,path,service,socket,swap,timer,unit} - it's all the same pattern.14:57
xnoxrbasak, internally they are enums/states et.al. externally they are all encoded/decoded/shown as a single string.14:58
rbasakOK, thanks. As long as the numbers are internal :)14:59
xnoxrbasak, as per their abi stability guides, they are free to remove/add/change those strings. but it is guaranteed to remain a string.14:59
rbasakThanks. Handy for you to know the answer without me having to dig :)14:59
xnoxthe abi is like `is-failed` which knows which strings are "bad" and which ones are "good"14:59
xnoxfor every unit type.14:59
xnoxit's meant to be informational15:00
cyphermoxtsimonq2: thanks for looking at slideshow :)15:09
tsimonq2cyphermox: No problem. :)15:10
dgadomskimwhudson: hi, could you please also release the fix for bug #1749289 to xenial?15:40
ubottubug 1749289 in ubiquity (Ubuntu Xenial) "Installer stops after pressing Cancel on Select a language screen during OEM install" [Undecided,Confirmed] https://launchpad.net/bugs/174928915:40
=== slangasek is now known as vorlon
=== vorlon is now known as slangasek
=== slangasek is now known as vorlon
ogravorlon, identity crisis ?15:50
vorlonstuck settings in the client that I'm trying to shake loose15:50
rbasakWould it be acceptable in Ubuntu for a package to dynamically add an extra binary package not listed in debian/control through debian/rules? The reason is that I want to keep in sync with Debian but build an extra split out binary package to stay in universe.15:55
rbasak(in Debian this wouldn't happen so I don't think there's an issue there)15:55
tsimonq2Why is it not possible to list it in debian/control?15:56
rbasakBecause it wouldn't be appropriate to build the binary package on Debian.15:56
sarnoldwouldn't taht just turn into a delta in debian/rules?15:56
rbasakYes it would.15:56
rbasakBut we do that already in other places, using dpkg-vendor15:57
tsimonq2Ohh, I catch your drift.15:57
rbasakMeans that Debian and Ubuntu build from the same source, which is convenient. Saves doing merges etc.15:57
tsimonq2That personally sounds doable but I guess I'd defer to an archive admin. :)15:57
sarnoldI dislike it, but not enough to be annoying about it. :)15:58
tsimonq2^15:58
rbasakISTR someone saying that Debian disallowed this. But it might be OK for Ubuntu. The only reason to do this is caused by Ubuntu's main/universe distinction.15:58
rbasakI think the alternatives is for sarnold to +1 our MIR from a security perspective, or to keep doing package merges forever.15:59
sarnoldheh, which one? :)15:59
rbasakbug 178152915:59
ubottubug 1781529 in mecab (Ubuntu) "[MIR] mecab" [High,New] https://launchpad.net/bugs/178152915:59
tsimonq2As someone once said, "Policy is your friend. Trust the Policy. Love the Policy. Obey the Policy." so I guess I'd be curious to say if Policy has a specific rule around it. :)15:59
sarnoldaha, dunno if I can get there today or not. I didn't sleep well last night :/15:59
rbasakI'll finish writing my reply. My question was prompted in writing a reply :)15:59
sarnoldhaha :)15:59
tsimonq2rbasak: Oh, I just recalled that dbgsym packages aren't mentioned in d/control.16:00
tsimonq2So while they're "special" packages, it should mean that it's permissible under policy.16:01
rbasakReply written in the bug.16:01
rbasakvorlon or infinity: ^ please could you tell me if this is acceptable? See bug 1781529 comment 7.16:01
ubottubug 1781529 in mecab (Ubuntu) "[MIR] mecab" [High,New] https://launchpad.net/bugs/178152916:01
tsimonq2infinity: ^ (since, if I recall correctly, he only gets pings if the message starts with his name.)16:02
sarnoldrbasak: very nice reply, thanks :)16:03
rbasaksarnold: thanks. Over to you I guess? If you want to nack the MIR on security grounds, that's fine - we'll find some way, and if we can't, we'll either continue merging or continue without mecab support.16:06
vorlonrbasak: binary packages produced by a source package are expected to be listed in the debian/control shipped in the source package16:40
rbasakvorlon: "expected", or "must"? I'm aware of the expectedness; that's why I'm asking :)16:41
vorlonrbasak: must16:41
rbasakOK, thanks16:41
vorlonit's bad to have packages listed in debian/control that you never build, but that's less bad than building packages you don't list in debian/control; the latter breaks cross-references in the indices16:42
rbasakI could list it in debian/control and then never build it, but then I'd be foisting that on Debian, which seems inappropriate.16:43
xnoxvorlon, nvidia-* packages currently shadow each other ;-)16:48
vorlonxnox: yes; which is permitted for the same reason it's permitted for gccs, you shouldn't have to reupload the old package in order for a new package to take over its binaries16:49
=== dannf` is now known as dannf
vorlonmdeslaur: why did woff2 have a no-change rebuild in bionic-security? should we copy it forward to cosmic? (cf juliank's post)17:25
juliankvorlon: he did upload 1.0.2-1build1 to cosmic a few hours or so ago17:27
vorlonah, well then17:27
vorlonstill wondering what the no-change rebuild was for :)17:27
juliankYeah, it would have been useful to have "for ..." in the changelog17:28
sarnoldiirc it was so that we could do a security update on webkit2gtk https://launchpad.net/ubuntu/+source/webkit2gtk/+changelog17:30
jbichavorlon: we moved woff2 to main17:34
vorlonah17:34
vorlonmoving to main does not require a rebuild, only a pocket copy17:35
jbichaok, good to know. We don't do it very often :)17:35
mdeslaurvorlon: if you only do a pocket copy, you break ubuntu-support-status17:36
vorlonoh?17:36
vorlonok17:36
mdeslaursince the installed package on people's systems still comes from universe17:36
jbichavorlon: if you have time, brotli binaries need to be moved to main in xenial too LP: #179507717:37
ubottuLaunchpad bug 1795077 in brotli (Ubuntu Xenial) "Backport brotli 1.0.3 to Ubuntu 16.04 LTS" [Medium,Fix committed] https://launchpad.net/bugs/179507717:37
jbichaand woff2 (SRU wasn't accepted yet) LP: #179509417:37
ubottuLaunchpad bug 1795094 in woff2 (Ubuntu Xenial) "Backport woff2 to Ubuntu 16.04 LTS" [Medium,In progress] https://launchpad.net/bugs/179509417:37
sarnoldjbicha: do we really? I thought webkit2gtk added new toolchain dependencies that meant we can't support it in trusty any longer?17:38
jbichasarnold: we never announced that webkit2gtk/xenial was unsupported so maybe…17:38
jbichalatest webkit requires gcc6 and no one's volunteered to backport that to xenial yet17:41
jbichahttps://trac.webkit.org/wiki/WebKitGTK/GCCRequirement17:42
mdeslaurit's a bit more complicated than that...we can't just backport gcc617:43
mdeslaurthe stdlibc++ library on xenial is provided by gcc517:43
jbichaI admit I don't know what that library means17:44
rbasakcoreycb: would you prefer me to wait for neutron in bug 1795424 before reviewing? Or are you happy with releasing bits at a time?17:50
ubottubug 1795424 in nova (Ubuntu Bionic) "[SRU] queens stable releases" [High,Fix committed] https://launchpad.net/bugs/179542417:50
rbasakcoreycb: or, given they all get tested together, maybe it's better to wait in the general case?17:51
coreycbrbasak: thanks for taking a look. i'd really like to include neutron as it has some critical bug fixes.17:52
coreycbrbasak: it's all regression tested, i think we can mark the remaining neutron bug as verification-done since it passed regression testing.17:53
rbasakcoreycb: there's still two more days left to age for neutron17:53
coreycbrbasak: ah, right. well i think you can hold off and we can get those all released together fri or mon.17:55
rbasakOK, thanks.17:55
coreycbrbasak: thanks17:55
rbasakcoreycb: it'll be Mon. We generally don't release on Fridays.17:55
coreycbrbasak: ok makes sense17:55
rbasakjuliank: I appreciate the impact statement in bug 1796081 :)17:56
ubottubug 1796081 in dpkg (Ubuntu Bionic) "dpkg frontend locking" [Undecided,In progress] https://launchpad.net/bugs/179608117:56
=== vorlon is now known as slangasek
=== slangasek is now known as vorlon
vorlonjbicha: brotli overrides done18:03
jbichathanks18:04
mwhudsondgadomski: oh i didn't realize it affected xenial too20:51
mwhudsondgadomski: is it in git for xenial yet?20:51
xnoxlibtool: compile:  mpicxx -DHAVE_CONFIG_H -I. -I../.. -Wdate-time -D_FORTIFY_SOURCE=2 -I/usr/include -I../.. -g -O2 -DNDEBUG -DZDOT_RETURN -c _hrr_c0_66.cc -o _hrr_c0_66.o21:18
xnoxvirtual memory exhausted: Cannot allocate memory21:18
xnoxbut builds fine in debian.... compiler bug? PIE or some such making it explode?21:18
xnoxmwhudson, vorlon, doko - any suggestions on where to go from here?21:19
mwhudson!?21:19
mwhudsonxnox: package, architecture, etc?21:19
xnoxbagel, all arches21:19
xnoxcosmic21:19
xnoxdid --no-parallel build in xnox/nonvirt ppa and also fails on all arches21:20
mwhudsonhuh21:20
vorlonyeah, virtual memory exhausted would seem to imply you're out of address space rather than out of physical memory21:20
mwhudsonsounds compiler bug-like to me21:21
vorlonand the compiler in question is... mpicxx?21:21
mwhudsonwhat is mpicxx? mpi as in the multi process thing?21:21
mwhudsonyes, seems so21:21
xnoxyeah, openmpi implementation c++ compiler....21:21
mwhudsonxnox: tried locally?21:22
mwhudsonit's a different version of libmpich-dev, 3.3~b2-6  vs 3.3~b2-7build121:24
xnoxmwhudson, well can't remember now. rerunning build in unstable chroot; then will rerun build in our chroot; then will see if libmpich-dev makes a difference.21:24
mwhudsonchanges between the versions are trival so it would only be if mpicxx built with gcc-8 is broken or some such fun21:25
=== vorlon is now known as slangasek
=== slangasek is now known as vorlon

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!