/srv/irclogs.ubuntu.com/2016/01/14/#ubuntu-devel.txt

naccrbasak: one of the big transitions is from php-config5 to php-config ... and i just realized that command-not-found would also need updating for all the php5 commands00:14
naccrbasak: for those, would we be leaving in php7.0 (replacing php5) or just putting in php as appropriate?00:15
naccrbasak: also, haven't seen any invite for a hangout tmrw; even if ondrej isn't available, could we do a quick sync on some questions I have?01:09
sarnolddoes anyone why know apparmor-dbgsym doesn't appear in http://ddebs.ubuntu.com/dists/trusty/main/binary-i386/Packages ? the packages appear to exist just fine: http://ddebs.ubuntu.com/pool/main/a/apparmor/01:32
sarnold(also not in binary-amd64/, the first place I looked)01:32
sarnoldhow odd, there's apparmor-2.8.98 packages there, but no apparmor-2.8.95~... versions01:34
naccondrej: where does php-pear come from in php7? I see that in php5 it's created by the php5 src pkg. But not in PHP7 -- and so when I try to build, e.g., php-pkg-tools from your git tree, it fails to find a2~2~2~02:07
naccerr, ignore the tail of that, clearly ... fails to find php-pear02:07
naccit == sbuild02:07
naccondrej: i also noticed in my xenial VM, after install php7.0, that I can't install php-pear (it depends on php5-common and php5-cli). I saw in your e-mail that you mentioned "php-pear is not built from independent source package", which made me think it came from php7.0?02:10
naccondrej: oh! was that a typo, perhaps .. i found http://anonscm.debian.org/cgit/pkg-php/php-pear.git/02:17
naccondrej: and i'm guessing, since it's so new, it might not yet be packaged?02:17
=== TheMuso` is now known as TheMuso
=== cpaelzer is now known as cpaelzer_afk
=== cpaelzer_afk is now known as cpaelzer
pittiGood morning07:52
didrocksgood morning pitti, ça va ?07:53
pittididrocks: j'ai des muscles dolores de basketball, mais je vais bien, merci ! et toi ?07:54
didrockspitti: ça va bien :)07:55
pittiSaviq: I forced it now07:55
pittibdmurray: I saw the MP, will do, thanks!07:55
pittihallyn: will respond in the bug; this should probably get told to not restart the service on package upgrade then (dh_installinit --no-restart-on-upgrade), but I'll read the details and confirm07:57
pittihallyn: pam-cgm> ah, did the other changes land in xenial? or do I still need the PPA for testing?07:57
hallynpitti: everything should now be in xenial except the systemd change07:57
hallynnote, libvirt-guests.service is already installed in debian/rules with --no-restart-on-upgrade07:58
hallynthat's why i'm confused07:58
pittihallyn: great, thanks; I'll merge some fixes from Debian while I'm at it, and trim the patch07:58
hallynit appears libvirt-bin's restart is forcing libvirt-guest's restart.  which i assume is due to an error in how i relate them,07:58
hallynthough i swear when i did this back in october of something it worked,07:58
pittijamespage: hey, how are you?08:04
pittijamespage: can you please have a look at debian bug 810702? They have a question about our Ubuntu delta, and I'm afraid I can't answer that08:05
ubottuDebian bug 810702 in open-iscsi "Generate initiator name on install, not first boot" [Normal,Open] http://bugs.debian.org/81070208:05
=== ara is now known as Guest95629
pittihallyn: hm, I see that you now have two patches in scope.c, both of which do a chown08:57
pittihallyn: oh, nevermind, the other patch is commented out08:58
hallynthought i removed the other before my final debdiff09:00
pittihallyn: still, core-Chown-systemd-cgroup-to-users.patch looks a lot nicer and smaller, didn't that one work?09:01
hallynpitti: it did not.09:01
hallyni didn't dig very deep;  don't know if i needed to mkdir it first or of something else was going on09:02
pittihallyn: ack; anyway, I'll work on that, and also adjust the autopkgtest (as that will now fail with the reduced patch)09:02
hallynpitti: thanks09:03
pittihallyn: thanks to you! nice to get this into xenial, so that the CPU hotplugging people get happy again :)09:03
ondrejnacc: php-pear is src:php-pear now09:23
ondrejnacc: Matthieu probably didn't upload it yet to Debian unstable, I'll poke him09:24
=== henrix_ is now known as henrix
pittihallyn: oh yes -- "chown(u->cgroup_path," doesn't work, that's not the full path, just something like /user/user-1000.slice09:29
pittihallyn: with libpam-cgm installed, is it normal that this only does stuff for "freezer", not for any other cgroup?09:31
pittisessionoptionalpam_cgm.so -c freezer09:31
pittihallyn: ^ I suppose it is (that's from /etc/pam.d/common-session, sorry about the broken tabs)09:32
pittiah, changing it to "freezer,memory" does work09:32
=== cpaelzer is now known as cpaelzer_afk
pittihallyn: works nicely, I tested in a fresh VM, lxc-start works, and cgroups look good; nice work!10:35
* pitti goes to adjust the autopkgtest, will upload then10:36
=== cpaelzer_afk is now known as cpaelzer
pitticaribou-: hm, I found a crash with xenial's rsyslog; installing juju-local triggers that (reproducer is in the bug); bug 153410611:43
ubottubug 1534106 in juju-core (Ubuntu) "rsyslogd crashed with SIGSEGV with juju-local configuration" [Medium,New] https://launchpad.net/bugs/153410611:43
pitticaribou-: mind having a look?11:43
pittireproduces perfectly in a VM11:43
cariboupitti: sure, will look at it11:44
=== _salem is now known as salem_
kickinz1Is there someone working on ntp merge already?11:48
pittiodd, why isn't that on merges.u.c.12:00
pittikickinz1: anyway, Laney is TIL, so it just needs to be cleared with him12:01
* pitti would assume Laney is fine with losing that12:01
Laneyhe already emailed me and I said okay12:01
kickinz1Laney, thanks again. Just to be sure noone else is working on it here (as I already had a pb wrt previous merge work)12:04
rbasaknacc: are you happy for me to upload your logwatch merge or did you want to check with Debian/upstream on the snapshot question first?12:04
=== jincreator is now known as jincreator_
smbpitti, Darn. Sorry it seems the more I look at plumbing the more I get confused. Is it intentional that in Xenial the numbers to "dh_installinit ... -- defaults SS KK" seem to be ignored. Maybe I should just close my eyes and move ahead to units/services...12:11
pittismb: SS and KK have been history for many many years, since insserv came along :)12:12
pittismb: they should not be used any more (and are ignored), instead insserv will compute dependencies from the LSB headers in init.d scripts12:12
smbpitti, *sigh* I am too old for that sh.. :-P Just glad that I think with "Before=" I have a better chance of modelling a "if thats there then start before" condition12:16
pittismb: right, the equivalent of that is X-Start-Before:  in the LSB header12:17
smbpitti, ah ok... that was never used. looks like pure luck this ever worked12:23
ginggsLaney: would you be able to apply the patch from Debian bug #808842 to glib2.0 in experimental please?  otherwise I can apply it to Ubuntu for now12:37
ubottuDebian bug 808842 in src:glib2.0 "glib2.0: FTBFS with PCRE 8.38: regex (?(?<ab)) produces different error" [Serious,Open] http://bugs.debian.org/80884212:37
Laneyginggs: Don't diverge.12:40
LaneyI'll fix it.12:40
=== kickinz1 is now known as kickinz1|lunch
ginggsLaney: thanks!12:45
dokopitti, python3.5 tests pass, now blocked on postgresql-multicorn13:11
=== kickinz1|lunch is now known as kickinz1
pittidoko: no, multicorn has an override13:23
jamespagepitti, blimey I had to go re-read the original bug report for that one13:23
pittidoko: ah, it's the pyqt5/s390x regression, I'll look/retry13:23
jamespage(iscsi patch/debian bug)13:23
pittijamespage: yeah, that delta is pretty old, and the logic changed in Debian several times; I'm not even sure if it's still needed, but when building an image in a chroot with policy-rc. the init script wouldn't run and generate the id13:26
xnoxbarry, yeah - fedora people got FTBFS in cython too now, and came out about it on my cython devel thread =)13:27
=== sergiusens_ is now known as sergiusens
rbasakondrej: around? Can we arrange a time to sync with nacc, Pharaoh_Atem and myself please?14:10
kickinz1rbasak, di dyou get a chance to look at amavisd-new merge ?14:14
rbasakkickinz1: not yet, sorry. I intend to review caribou's merge next, and then yours. Hopefully this afternoon.14:15
kickinz1rbasak, ok I understand.14:18
pittidoko: ah, it landed now14:18
kickinz1I've splitted the ntp delta to my git repo on launchpad. If you want to look at it (there is a 'space' typo in debian/apparmor-profile, which appear at all diffs :( ).14:20
kickinz1rbasak, ^this typo would imply 2 rebase and 9 re-tags, and the new merge will be on a major version bump; tell me if you think it is better to remove it.14:24
ioanmhi, i'd like to pay back to ubuntu by becoming a dev :) I know it says anyone can contribute, but i would like to be part of the main team14:30
ioanmi'm familiar with C/C++ and Assembly14:33
ioanmwhere should i start?14:33
pittirbasak: hey Robbie, how are you?15:28
pittirbasak: not sure who/whether someone cares in the server team, but I'm currently moving xenial to postgresql-9.5 (only)15:28
pittirbasak: for an LTS we are pretty much required to ship the latest major upstream release15:29
rbasakpitti: o/15:34
rbasakpitti: postgres has been pretty painless for us. Go ahead!15:35
naccondrej: ah ok! thanks -- yes, that'll get autosync'd and I can verify it in ubuntu that way, at least, too15:35
=== smoser` is now known as smoser
rharperrbasak: on the git merge stuff, I want to import the new/debian dsc; does the state of the repo matter (should I be at reconstruct/xxx ) ?15:45
rbasakrharper: I would be at old/debian when importing new/debian. Then the history makes sense.15:46
rharperok15:46
rbasakI'm not sure how much this matters though.15:46
rbasakIt seems wrong to be anywhere else.15:47
rharperagreed15:47
Odd_Blokepitti: Do you use the armhf cloud images for autopkgtest stuff?15:49
pittiOdd_Bloke: not at the moment, no; I use the "ubuntu" template with debootstrap15:49
pittiOdd_Bloke: I was going to use them by moving to lxd, but we have some problems on arm64 which currently prevent that15:50
Odd_Blokepitti: Would you be comfortable with us dropping them for xenial?15:50
Odd_Blokepitti: (We're not really seeing any downloads for them ATM)15:50
pittiOdd_Bloke: fine with me, as long as we keep LXC/LXD images15:50
pittiOdd_Bloke: well, hang on; there is a chance that we can soon boot armhf images in Scalingstack15:51
pitti(wgrant was looking into some blockers there)15:51
Odd_Blokepitti: Right, I'm waiting to hear back from wgrant on that.15:51
Odd_Blokepitti: (But I asked that in15:51
Odd_Bloke#launchpad, just to make following the conversation harder :p)15:51
pittiOdd_Bloke: if/once that works, having images would be nice, but I think that an lxd image would suffice for that as we need to specify kernel and initrd separately anyway15:51
cjwatsonpitti: for Launchpad, our plan is to boot arm64 images with a kernel option apw added to make them emulate armhf more accurately15:54
cjwatsonpitti: mostly because that fits better into the way we do multi-architecture-capable builders15:54
pitticjwatson: ah, ok; with that appraoch I'm currently blocked by bug 1531768 :/15:54
ubottubug 1531768 in linux (Ubuntu) "arm64 kernel and multiple CPUs is unusably slow" [Medium,Confirmed] https://launchpad.net/bugs/153176815:54
Saviqpitti, hey, I'm working on adding autopkgtest runs to our Jenkaas, got a few minutes for questions?15:57
pittiSaviq: we have team meeting in 2 mins, but just ask15:58
Saviqpitti, I've used autopkgtest from trusty-backports (was it a good idea? maybe there's a better source?), any idea why artifacts would not get copied over from the testbed? I can see the tests writing to ADT_ARTIFACTS but they didn't show up in --output-dir https://unity8-jenkins.ubuntu.com/job/run-commands/376/16:02
pittiSaviq: trsuty-backports is fine (I shoudl update it, though)16:02
Saviqpitti, a different question is now how to restrict tests to a certain testbed (I'd like to put autopilot tests, meant to run on a phone, into a autopkgtest)16:03
pittiSaviq: perhaps your didn't tell your jenkins job to copy the artifacts/ subdir?16:03
SaviqI know I could just pass --testname, but maybe there's a better way, one that would work/not break britney, too?16:03
Saviqpitti, d'oh, **16:04
Saviqmost likel16:04
Saviqy16:04
pittiSaviq: why can't the autopilot tests run with britney tests? we have quite a number of packages who do that16:04
pittiSaviq: there's some autopilot-sandbox-run convenience script to set up Xvfb, dbus etc.16:05
pittiSaviq: otherwise, make the test check for an expected environment and exit with 0 and some message like "SKIP: not on an ubuntu phone"16:05
Saviqpitti, they need a phone, a full Unity8/Mir session, at least until we get virtual output from Mir16:05
Saviqpitti, ack, that works, thanks16:06
dokobarry, just gave back cython, still ftbfs16:08
barry:(16:08
Saviqpitti, ah, last thing, parallelizing builds, that's fixed in latest adt-run, right? probably just not in the trusty-backports one?16:09
pittiSaviq: should be in trusty-backports, hang on16:10
barrydoko: there's been a little more discussion on that cython thread xnox started.  maybe the cython devs will fix that bug16:10
pittiSaviq: that was in 3.17.316:10
dokobarry, xnox: reference?16:12
Saviqpitti, hmm if I didn't pass DEB_BUILD_OPTIONS="parallel=4" it went and built on a single core16:12
pittiDEB_BUILD_OPTIONS=parallel=$(grep -c ^processor /proc/cpuinfo | sed 's/^0$/1/');16:13
barrydoko: https://mail.python.org/pipermail/cython-devel/2016-January/004639.html16:13
pittiSaviq: it uses that; what target are you running this on? perhaps /proc/cpuinfo is wedged there?16:13
xnoxdoko, barry: https://github.com/python/asyncio/issues/308 https://mail.python.org/pipermail/cython-devel/2015-December/004630.html https://mail.python.org/pipermail/cython-devel/2016-January/004639.html16:14
Saviqpitti, looks fine, gave out "4"16:14
Saviqas "nproc" does16:14
pittiSaviq: oh, nproc, I learned something new!16:15
Saviq:)16:15
pittithat uses sched_getaffinity()16:15
pittiSaviq: but either way, it works locally, so I'm afraid I need some more info to reproduce; can you please file a bug for that?16:16
Saviqpitti, will do when I confirm16:16
dokoxnox, but no real feedback from the cython side?16:25
xnoxdoko, radio silence16:25
kickinz1rbasak, so I did some rework on my ubuntu delta for ntp, but now it doesn't get pushed correctly to launchpad16:27
nacckickinz1: what happens?16:28
kickinz1my current local git is: http://paste.ubuntu.com/14496958/16:28
naccrbasak: rharper: it will matter a bit, for the current branch you are on, but in theory that can be fixed later with rebases/renames16:29
kickinz1and in my launchpad tree it is like I have a duplicate master branch instead of the deltas: https://git.launchpad.net/~kick-d/ubuntu/+source/ntp/log/?h=reconstruct/4.2.6.p5%2bdfsg-3ubuntu916:29
nacckickinz1: what branches do you have locally and how are you pushing?16:29
nacckickinz1: so it looks like you pushed your local master (or reconstruct) to the remote's master16:30
kickinz1I pushed both of them.16:31
naccif you look at your local tree, it seems like tag: new/debian, tag: 1_4.2.8p4+dfsg-3, reconstruct/4.2.6.p5+dfsg-3ubuntu9, master are all 04b751c16:31
kickinz1nacc, thanks.16:32
nacckickinz1: on what branch is e.g., cfdd2ee16:32
naccyou want to push that to the remote, so that it updates ... remotely :)16:32
naccrbasak: Pharaoh_Atem: either of you around?16:34
kickinz1nacc, thanks again16:36
nacckickinz1: np, i actually find the 'pretty' log you pastebin'd much harder to parse than it needs to be ... but that's because I want the names (tags, brances) to be first in my head ... would have been a lot more obvious what was going on if that was the case (rather than a parenthized suffix)16:37
kickinz1nacc, OK, I'll rewrite my gitconfig, thanks for the tip.16:38
nacckickinz1: yeah, whatever works best for you, though, for sure. I also use git enough that I would rather call the commands how I want rather than change the defaults :)16:40
kickinz1nacc :)16:41
rbasaknacc: o/16:41
naccrbasak: heya! do you have a few minutes to chat? I know it's getting late for you16:42
Pharaoh_Atemnacc: just got in16:44
naccPharaoh_Atem: cool, let's see if rbasak is also free, and if not, maybe we can chat a bit16:46
Pharaoh_Atemnacc: cool16:54
naccondrej: how are you dealing with SWIG breakage with PHP7? Seems not yet fixed upstream, so certain src packages that use SWIG to rebuild simply won't work with PHP7.0?17:07
rbasaknacc: yes I'm free.17:09
=== kickinz1 is now known as kickinz1|eod
rbasakI don't be tomorrow evening, but today is OK-ish. I have some bits to do but am around and can rearrange to find whatever time.17:09
naccrbasak: great, thanks -- i'll set up a hangout, if that's ok with you and Pharaoh_Atem?17:09
rbasaknacc: sure. How soon? Now is fine, I just need to reboot into Chrome OS etc.17:10
rbasakPharaoh_Atem might want more notice though!17:10
rbasakNot heard from ondrej yet.17:10
naccrbasak: I got a couple responses from ondrej ... tbh, let me just ask my questions here, and we'll see how far that gets us17:10
Pharaoh_Atemrbasak: I mean, I'm here17:10
Pharaoh_Atemnacc: I can certainly do the hangout thing17:11
Pharaoh_Atemjust let me know when and I'll find somewhere to go do this17:11
naccso it looks like swig isn't getting fixed necessarily anytime soon; I have the build in my PPA to disable the PHP bindings17:11
naccbut that will break e.g., graphviz for a while17:12
rbasakOnly graphviz's PHP bindings?17:12
naccwell, any package that tries to use swig17:12
rbasakOr do we need to add a graphviz delta for that?17:12
naccgraphviz happens to be in main17:12
naccso I was considering it first :)17:12
naccrbasak: right, so i figured we'd modify graphviz ... but wanted to understand how best to do that? just disable the binary package that contains the php bindings?17:13
rbasakGood question. That would certainly work.17:13
rbasakI think we might be best getting wider consensus on this.17:14
rbasakGood job on identifying graphviz. Do you have a way of generating a complete list?17:15
naccrbasak: i think the pad is now pretty current17:15
nacci've not yet broken it down by swig/pear/pecl17:15
naccthat was going to be my next step17:15
rbasakCan we publish that and see who objects to disabling those bindings? It'll come down to either doing that or not shipping PHP 7.0 I think, so I assume it'll be OK to do that, but I want to make sure other devs are happy with that.17:15
rbasakBefore we go mass uploading those disablements.17:15
naccrbasak: yeah, where's a good place to publish it? I can clean it up a bit and put it somewhere "official"17:16
rbasaknacc: ubuntu-devel ML please.17:16
Pharaoh_AtemI guess I should be on that ml, shouldn't I?17:16
xnoxgiven the two are coninstallalbe, surely a bunch of swig based bindings can stay at 5.0...17:16
naccrbasak: also, i noticed that the package in graphviz is 'libgv-php5' ... so would we create a libgv-php? or libgv-php7.017:16
naccxnox: they won't be coinstallable :)17:16
rbasakxnox: except that swig is in main.17:16
xnoxor for example, build everything for both.17:16
rbasakAlso, I want php 5.6 gone. Not in main, not in universe.17:17
Pharaoh_Atemkill it with FIRE! :D17:17
rbasakI think we agreed on that previously, although we can always change that based on new information of course.17:17
xnoxrbasak, we should just have archive reorg and thus have everything able to build-dep on anything =) no more package splits \o/17:17
rbasakxnox: yeah, how are you progressing with that? :-)17:17
xnoxrbasak, we moved cjwatson into launchpad for exactly that reason!17:17
rbasakAnyway, it won't matter if we're losing 5.6 entirely.17:18
xnoxrbasak, when will swig be 7.0 ready?17:18
naccrbasak: ok, i'll send an e-mail by EOD17:18
naccxnox: no ETA17:18
xnox=/17:18
rbasakxnox: not imminently, if ever.17:18
naccxnox: as in no upstream progress yet in that regard17:18
rbasakxnox: https://github.com/swig/swig/issues/57117:18
rbasaknacc: thanks!17:19
rbasaknacc: I suggest that you propose to drop the bindings, point out that not doing so blocks php 7.0, and invite objections.17:19
rbasak(with a list of affected packages)17:19
naccrbasak: yep, that makes sense17:20
* xnox ponders if porting swig is easier than disable all the things17:20
rbasakPharaoh_Atem: join the ML if you like. Though posts are moderated for non-uploaders anyway, and an archive is available, so it probably makes little difference. You can participate the same either way :)17:20
rbasak(I'm sure the moderators will pass all your emails regarding this transition through)17:20
naccxnox: i was going to look at what was needed, but that was a lower priority for me in the short-term17:21
naccrbasak: ok, next question, I noticed with the move of some commands, tools, the API version is now obtained by php-config --phpapi rather than php5-config --phpapi ... and I noticed that if you run `php-config` from a Xenial prompt17:21
naccc-n-f doesn't know where to find it :)17:21
rbasakI think c-n-f is autogenerated periodically17:21
naccrbasak: so ... does c-n-f need updating manually? or is there some voodoo to make it look up new packages?17:21
rbasakI'm not really familiar with how it works.17:21
naccrbasak: yeah, i started looking at doing it manually (as we'd want to remove references to the php5 commands in the PPA) ... it didn't seem to like my changes, but it was late when i was doing that17:22
hallynpitti: yup, freezer only by default.  i suppose we could have systemd do that one and Lennart may even be ok with it - but we need pam-cgm for users who want memory cgroup etc - this makes it trivial to add that17:22
rbasakmvo knows I think.17:22
naccrbasak: also, the php-pear move is going to be annoying until debian merges it17:22
naccrbasak: that is, w/ PHP5.0, php-pear was generated by the php5 src pkg17:22
naccerr, PHP517:22
* xnox ponders if i confused php5+7 coninstallability with perl5+6 coinstallability17:23
naccwith PHP7.0, it is its own src pkg, which doesn't seem to exist anywhere yet17:23
naccxnox: i think Debian will have coinstallability, but it's kinda messy17:23
naccxnox: and they will move to 7.0 only in the next release, I think17:23
xnoxand we want 7.0 because....?17:23
* rbasak wonders why php-pear didn't autosync17:23
naccrbasak: it hasn't been pushed to debian either yet, i don't think17:24
naccrbasak: based upon ondrej's comment earlier17:24
xnoxi'd say until things in main are fixed to work with php7 we don't want it, as default, or only php. (and that includes swig)17:24
Pharaoh_Atemrbasak: I've subscribed regardless17:25
naccrbasak: but we're kind of stuck on pear fixes until that happens, it seems like17:25
xnoxusually that's the criteria for every other major upgrade.17:25
cjwatsonrbasak: $ rmadison -udebian -asource php-pear17:26
cjwatsonphp-pear   | 1:1.10.1+submodules+notgz-1 | new        | source17:26
naccright, so i think this is a workflow I didn't understand (and applies to a few other packages that are getting updated to 7.0) -- "new" means that debian is aware of a new revision, but hasn't actually packaged them yet?17:27
rbasakcjwatson: ah, thanks.17:29
rbasaknacc: ondrej can give us a copy of what he uploaded to Debian NEW if necessary, and we can upload that to Ubuntu in the meantime if necessary. We'll need an Ubuntu Archive Admin to review it if we do that, but it can be done if we're blocked on a Debian ftpmaster processing it at the Debian end.17:30
rbasakIn fact ondrej's upload is probably in git.17:30
naccrbasak: ok, there was at least one other package that i hit that with in my review17:30
naccrbasak: yes it is17:30
naccrbasak: so i was going to ask that next17:30
naccrbasak: as i go through these pacakges17:30
naccsome have been up dated in git with the master-7.0 branches17:30
xnoxnacc, new means there are brand new binaries intoroduced, and are awaiting manual review before being included. But e.g. maintainer did package and upload it. It's just being held up.17:30
cjwatsonbinaries or sources17:30
xnoxnacc, mostly legal review.17:30
=== cpaelzer is now known as cpaelzer_afk
cjwatsonwhere "brand new" == "a binary/source name not currently in the archive", not new version17:31
naccis it appropriate for me to merge them in the ubuntu git tree i'm using to build out a new dsc with an appropriate changelog (referring to the SHA1) entry?17:31
naccxnox: cjwatson: ah ok, thanks!17:31
Pharaoh_Atemrbasak: so it looks like the debian package doesn't include working fpm configs17:32
Pharaoh_Atemso I'm working on that for the tests and stuff17:32
naccrbasak: the package i was most considering that for was pkg-php-tools, which does have some changes in git that i can merge into my local tree and build out a new package from, i think ... but since it's not yet in a debian package, i wasn't sure if that made sense, or how to changelog it properly (I guess it's a one-off for testing still, so that doesn't matter too much) but basically without the right17:35
nacc phpapi-YYYYMMDD value at build time, the rebuilds don't trigger properly, etc.17:35
rbasaknacc: if it's going into Debian as version foo, you can upload to a PPA foo~ppa1. Then foo is higher, so it will supersede it when it appears.17:39
naccxnox: i think the rationale, also, was that if you needed SWIG, then you can stay on 14.04 LTS ... similar to if you needed PHP5 at all. I'm not saying it's perfect, but I think that was the reasoning. Also, PHP5.6 support ends aug. 2017, which is prompting some of the consideration of how to support that package in the LTS, I think17:42
naccrbasak: ah interesting! more nuance, thanks!17:42
xnoxnacc, and rhel will support 5.4 into 202417:46
naccxnox: not sure rhel should be anyone's model citizen :-P17:46
naccxnox: but yes, there is that17:46
xnoxi think pulling off php7 now, will make it a crap 16.04, for the two years that it matters, vs having a good 5.6 for two years.17:47
naccxnox: i'm really not trying to argue the point, sorry -- i agree with you, we should not break *anything* in main, and otherwise not do it17:47
naccxnox: do you mean crap because too much will break? or because PHP7.0 is bad?17:47
xnoxnacc, not enough critical mass. E.g. not everyone needs it, not everyone can use it, not all dependencies available.17:49
xnoxpeople who want php7.0 will be able to use e.g. 16.1017:50
naccxnox: fair points -- I think rbasak was getting quite a bit of PR-work from people that disagree (based upon twitter)17:50
xnoxbut well, it's up to rbasak and server team to decide what they want to have. but e.g. security support for 5.6 over the next 5 years, is easy, given that everyone else is stuck on 5.x series.17:51
naccrbasak: ok, another practical question, just for my own edification. in PHP5, php-json src pkg provides php-json and php5-json bin packages. Now php7.0-json is generated by php7.0 src pkg. So do we remove php-json src pkg and have php7.0-json produce a php-json bin package? Or remove php5-json bin package from the php-json package and have it refer/depend on php7.0-json for now? I guess the latter is mo17:53
naccre sensible17:53
naccxnox: yeah, i'm on that team :) trying to help figure it out as best I can17:53
rbasaknacc: we'll need to check the php-json situation. In 5 we (Debian, Ubuntu, Fedora, maybe others) had to diverge from upstream because the bundled json module wasn't Free Software. I don't know if that has changed now.17:59
rbasakAs a result we had a third party external source (which I think Remi from Fedora put together).17:59
rbasakThat built php5-json instead of src:php518:00
rbasakIt did lead to a circular build dependency that needed to be resolved manually to break the loop.18:00
naccrbasak: ah i see18:01
rbasakNow if the licensing is now resolved upstream and we can ship it, then src:php7.0 can build php7.0-json and maybe a php-json alias and we should be fine.18:02
naccrbasak: right, that makes sense; and if not, we'd need a php7.0-json or something?18:03
rbasaknacc: if you mean a src:php7.0-json if not, then yes that's right. I suspect it probably is resolved though as I presume that's why ondrej didn't create one?18:03
naccrbasak: do you have a reference I can read as to the prior licensing concern?18:04
naccrbasak: right, sorry, yes that's what i meant18:04
rbasaknacc: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=692613 and also https://bugs.launchpad.net/ubuntu/+source/php-json/+bug/1287726 is relevant.18:04
ubottuDebian bug 692613 in src:php5 "php5: non-free files in upstream tarball ("The Software shall be used for Good, not Evil")" [Serious,Fixed]18:04
ubottuLaunchpad bug 1287726 in php-json (Ubuntu) "Wrong evaluation whether json is valid or not" [Medium,Triaged]18:04
naccrbasak: right, looks like src:php7.0's php-json is under the PHP 3.01 license18:06
rbasakLovely!18:07
rbasakI'm glad they fixed that.18:07
nacchrm, i wonder if something didn't get updated, though, README.REDIST.BINS still referes to the JSON license for ext/json/json_parser, but the files in taht directory refer to the PHP 3.01 license18:08
naccrbasak: cjwatson: xnox: thanks for your patience and helpful education! learning a lot quickly...18:12
roadmrhey folks! so lp:ubuntu/checkbox is at a point where we can transition it to being a non-native package. The branch would only contain the ubuntu-specific debian/ directory. It will build a single ubuntu-specific package since all its deps are now pulled from Debian.18:19
roadmrwe're wondering if it's bad practice to "reset" the branch for xenial. The (long and colorful) history for older releases would still be kept in the per-release branches...18:21
LocutusOfBorgPici, , mbiebl_ nice talk18:30
LocutusOfBorgpitti, ^^^18:30
* LocutusOfBorg wrt debugging and dissecting systemd boot issues :)18:30
LocutusOfBorgthanks!18:30
xnoxdoko, barry - i wonder if those tests should be skipped for now, to get us going... However i think there is now more test failures, than before.18:36
xnoxit used to be just the generator on s390x.18:36
barryxnox: more failures w/3.5.1?18:37
xnoxone sec, fiddling with logs.18:37
barryxnox: don't forget, i fixed the pickling issue and doko cherry picked that18:37
xnoxbarry, ah, right. so right, wonderful =) so it's just the generator error.18:38
xnoxanybody using asyncio, with generators, on python3.5.1, on xenial, and can figure out what's wrong are welcome to complain upstream =)18:38
barrywell, "fixed" maybe :)  in any case, reverted the regression for stable releases18:39
xnoxcause asyncio changes do look cryptic, and have a bug report attached.18:39
barryxnox: i'm not asyncio expert, but i've done some projects using it.  e.g. aiosmtpd still passes all its tests18:40
barryi can't judge those asyncio changes tho18:40
xnoxbarry, well, it's not just 3.5.1 but with a co-routie too18:43
Laneyginggs: uploaded to exp, feel free to test/sync when it becomes available18:43
Laneyginggs: could be up to 8 hours though, so I'll look tomorrow if necessary18:43
* Laney waves18:44
mvorbasak: cnf unfortuntaely does need manual updating, its pretty trivial, there is a cron job collecting the update and a bzr-buildpackage hook to get it. I can do an update now18:49
smoseranyone know if in a buildd (for livefs build) i'd be able to do overlayfs mount ?18:50
xnoxsmoser, i think Odd_Bloke does do loop mount / losetup. i would think overlayfs would be allowed too.18:50
xnoxubuntu-desktop is overlayfs, but i guess on boot only.18:51
smoserright. i know that loop mounts work.18:51
smoserand even asked cjwatson for nbd to work18:51
rharpernacc: when I'm done with my logical branch, Im somewhat confused (though it may be because strongswan is special in that we never really had an old/debian that was debian's, rather we have an older ubuntu we're using as old/debian);  when I'm replaying my logical branch onto the new/debian; there's not much overlap since the new/debian is a lot different;  that is, I don't see much of any of the logical changes being18:52
rharperable to apply to a new/debian tag because they're so divergent18:52
smoserrharper, well, they may not apply, but the logical intent you need to maintain. or decide if you can drop.18:54
rharperright18:54
hallynpitti: doh, should have checked the bug earlier - when you're back and have some time, i would very much appreciate some guidance in bug 153383919:11
ubottubug 1533839 in libvirt (Ubuntu) "vms shutting down on libvirt upgrade" [Critical,Triaged] https://launchpad.net/bugs/153383919:11
hallynmbiebl_: ^ actually i suspect you can guide me there.  basically, i need help figuring what in libvirt's debian/rules, debian/libvirt-bin.preinst, or debian/libvirt-bin.libvirt-bin.service and debian/libvirt-bin.libvirt-guests.service is causing libvirt-guests.service restart on pkg upgrade19:12
naccrharper: yeah, i think if the underlying source changes a lot the rebase may lead to very different commits (in terms of the code), but the changelog should be applicable still (with the new code) -- or the changes are no longer needed19:17
rharperyeah, I'm just going to grind through a rebase and see what I end up with;19:24
rharpernacc: there are something like 50 changes I've got in my logical;  it wasn't clear to me that I could drop any of these as we've diverged quite a bit w.r.t how things are packaged (subpackage per plugin versus two packages with all plugins) and aa profiles and other things we're bringing along19:25
naccrharper: yeah, that's probably the simplest thing to do ... it's trivial to roll stuff back, now that you've got tags for the old/new stuff, etc. and you can make throwaway branches of course19:25
rharpernacc: also, wasn't quite clear to me how much to drop from the logical branch or whether it should be dropped during the rebase19:25
rharperyeah, doing this rebase on a branch19:26
naccrharper: yeah, i think you want to drop as little as possible ... or as you said, do it as late as possible19:26
rharperyeah19:26
naccrharper: so will new/ubuntu follow new/debian's system (subpackages etc.) or will you maintain old/ubuntu's  (two packages with all plugins?) (or v.v., depending on which way you meant it)?19:27
naccrharper: i guess those are the decisions you kind of have to make in the rebase/merge :)19:27
rharpernot sure yet19:28
rharperI've neen discussing this with one of our users19:28
rharperI'll probably write up an email to ubuntu-devel to discuss19:28
rharperit certainly would be nice for merging to not have to keep 40 odd plugin install files; but that's a change from current ubuntu meaning we need to handle upgrades in a sane way19:29
naccyeah, that seems tricky19:30
naccrharper: would something like that be specified in the debian/ dir somewhere? that is, if a certain conf file is here in now, when you upgrade to this version, it goes *here* now?19:31
rharperthere are two things19:31
rharpernot really19:32
naccrharper: how is that handled then? i mean, generally speaking when a program changes where it looks for config files, or the structure of its runtime loaded data? symlinks?19:33
rharperthe new "plugin-standard" and plugin-extras packages would obsolete (I think) the per-plugin packages; so on upgrade, we'd know that if you had plugin-foo, then to install plugins-standard package19:33
rharperdepends19:33
rharperin some cases if enough movement happens, then new versions of the package conflict and break old packages19:34
rharperothertimes, configs can be migrated in pre/post scripts;  I'm weak on they policy, so we'd need to read the guides for sure19:34
naccrharper: interesting -- yeah, rbasak pointed me at some docs yesterday, i'm sure it's covered in there somewhere19:36
naccrharper: i'm going to be biking downtown and then i'll be back online -- good luck, let me know if i can help at all19:44
rbasakrharper: my intent for the logical tag was to eliminate any churn in old/ubuntu. That is, make old/ubuntu exactly what was intended before in the cleanest and most minimal way. However, retain mistakes, since that helps with verification. Then the logical tag should still be identical to old/ubuntu, but "git log -p" should also be very clear and make sense.20:34
rbasakrharper: so quite heavy rebasing, but without actually changing any code.20:34
rbasakrharper: then you're in the best possible position to decide how to apply that to new/debian, and you can do it one commit at a time, and not worry about losing anything.20:35
rbasakAnd you can have confidence that there are no mistakes up to that stage, since "git diff old/ubuntu" should be empty except for meta changes.20:36
rharperrbasak: I probably need to suffer this rebase and get to the end before I'll really understand this20:38
rharperrbasak: that said, I do think strongswan is unique at this point given the large delta between ubuntu strongswan and debian strongswan w.r.t debian/*20:39
rharperas an example, I have a conflict during rebase now because new/debian has a subpackage called strongswan-charon, and we don't have that package name at all (rather we package that in a different subpackage)20:39
Pharaoh_Atemrbasak: am I doing something wrong here? I can't seem to get the php-fpm config to work: http://fpaste.org/310968/14528042/20:44
rbasakrharper: yeah sorry, the strongswan merge turned out to be even more painful than I expected, and you've ended up stuck with it. I think this git workflow shines when it comes to this complexity though. I don't know how I could manage it any other way.20:45
Pharaoh_Atemrbasak: err, php-fpm apache config20:46
rharperrbasak: no worries;  I figured if I can get this one to work then the rest will be easy (ha! tomcat is next I'm told)20:46
rbasakPharaoh_Atem: I'm not sure. This an area I don't know off the top of my head. I'd have to wade through the Apache config docs to understand, sorry.20:47
Pharaoh_Atemit's fine20:47
Pharaoh_Atemoh, geez20:47
Pharaoh_AtemI think I know why now20:47
Pharaoh_Atem:/20:47
rbasakGreat! Glad to have helped :-P20:47
Pharaoh_Atemthe socket path for php-fpm in deb/ubuntu doesn't match what the upstream doc said it would be at20:48
Pharaoh_Atemso... a tweak here should do it20:48
Pharaoh_AtemX_X20:48
Pharaoh_Atemnope20:49
Pharaoh_Atemhmm20:49
rharperrbasak: so, given the divergence, would you expect that after I've rebased old/debian onto new/debian that I'd need to apply some new patches ?  for example, in debian/control debian has different packages (plugin-standard, plugin-extras)20:52
rbasakrharper: absolutely. Rebasing old/ubuntu onto new/debian is just the start. That's the automatic way of doing it, and then you have to fix up as needed to make the delta apply correctly, drop unneeded bits, etc.20:53
rharperrbasak: Ok20:54
bdmurraymterry: I forget did you have some experience with software-center?20:54
rharperthat helps me make some sense20:54
rbasakrharper: you don't even need to rebase if it's too complicated. You could cherry-pick each one instead, fixing up as you go, and just not cherry-pick where not relevant (or write yourself instead of cherry-pick)20:54
mterrybdmurray, no not really20:54
rbasakrharper: the importance of logical is to make sure that you understand the previous delta, and do the appropriate thing for each item. Even if that's re-write, or drop.20:54
rharperrbasak: right, that makes sense20:55
rharperrbasak: so, strongswan-5.3.5 % ls debian/*.install | wc -l21:02
rharper921:02
rharper% ls debian/*.install | wc -l21:02
rharper6721:02
rharperin general, debian puts all plugins into two packages;  we've got them split out package per plugin;21:02
rharperso, using anything from debian w.r.t packaging is *hard*21:03
rharperunless, we want to go down the path of switching back to fewer packages21:03
rbasakrharper: any path of reducing our delta against Debian is a good one.21:03
rbasakrharper: I'm not sure it's worth maintaining a delta to do otherwise here, unless it's because of the main/universe split.21:03
rbasakMaybe jpds could explain why it is this way? ^^21:04
rharperrbasak: agreed, so, couple of thoughts:  1) I don't know why we split them up  2) existing ubuntu users may have subset of plugins installed so upgrading would require some care to handle not enabling all of the new plugins in the combined plugin packages21:05
rharperrbasak: but I do think it would help tremendously to drop the nearly 60 package split w.r.t simplifying delta between ubuntu and debian21:05
rbasakrharper: +1, but I guess we do need to trade off the time available with time you could spend on other packages.21:06
rharperthe other major changes are apparmor policies for packages, but those are trivial in comparision21:06
rharperrbasak: indeed, and with jgrimm asking us to look at tomcat next, we're definitely short on time21:06
rbasakrharper: also, another advantage of doing it for Xenial is that any transitional delta can be dropped at Xenial+1, so we don't need to carry it as long.21:06
rbasak(by transitional delta I mean any Ubuntu delta against Debian that we need to keep only for user upgrade paths from previous Ubuntu releases)21:07
rharperright21:07
rbasakBut if it's marked out and flagged as such, which is easier to do and keep separate with git, then it's not so bad I guess.21:08
rbasakI wouldn't compromise other things over it. It would just be nice, that's all.21:08
=== salem_ is now known as _salem
nacc_cjwatson: so as you pointed out earlier, php-pear has a source in debian that's in new ... how do I go find that source? the debian pages all redirect back to php5 as that's the src: for php-pear right now21:37
rbasaknacc_: the Debian NEW queue is private, on the basis that it hasn't been reviewed from a licensing perspective yet (that's in part what NEW is for)21:38
nacc_rbasak: ah i found it anyways21:38
rbasaknacc_: so you have to ask the uploader, or fetch it from VCS or some combination.21:38
nacc_http://anonscm.debian.org/cgit/pkg-php/php-pear.git21:39
Unit193Or mentors.21:39
nacc_rbasak: just not immediately obvious how to find a link to the above URL, but that's ok21:39
nacc_rbasak: makes sense21:39
rbasaknacc_: the standard is to link to the Vcs from the debian/control file to which you don't have access :)21:39
rbasakThe PTS gets its link from that field21:39
nacc_rbasak: ok, new question, so since pkg-php-tools depends on an appropriate php-pear (which i also need to build), how do I tie them together for sbuild/test purposes?21:40
rbasakYou can use a PPA21:40
rbasakOr sbuild has some options21:40
nacc_rbasak: ok, so if i just push php-pear first then pkg-php-tools, it will dtrt?21:40
rbasak--extra-package and --extra-repository maybe? They're new-ish options and I did it the old way.21:40
rbasaknacc_: yes, provided that the version of php-pear's binaries on which pkg-php-tools build-depends are higher than the ones in the archive, so yours are preferred.21:41
nacc_rbasak: got it21:41
nacc_rbasak: ok, new question, related (also, isn't it late for you??)21:46
nacc_rbasak: but in any case, php-pear only exists in a git-tree so far (no debian package). so there's no tarball to use for dpkg-buildpackage, etc.21:46
nacc_rbasak: do i just need to wait?21:46
rbasaknacc_: you can take ondrej's upstream branch and use "git archive" to make a tarball out of it, then use that as your orig tarball.21:53
rbasakWe would prefer to upload to Ubuntu a binary-identical tarball to what ondrej uploaded to Debian since that makes life easier for us, but in a PPA or for local testing using your own constructed tarball should be fine.21:54
rbasakYes it is late for me :)21:54
rbasakSo much coordination happens in this window though, sometimes I just extend my working hours if I'm around, and adjust some other day in lieu.21:55
nacc_rbasak: ok, don't feel any need to stay on my account! i'm sorry to hold you up21:57
rbasaknacc_: don't worry, if I had to do something else, I'd be doing it :)22:04
* rbasak does need to go now though!22:04
rbasakSee you tomorrow.22:04

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