/srv/irclogs.ubuntu.com/2016/03/29/#ubuntu-devel.txt

mwhudsonhm, running do-release-upgrade under a screen i created worked fine00:04
mwhudsonhm the next question is how do i do lts to lts testing of a package change i want to make?00:36
slangasekmwhudson: bypass do-release-upgrade and just use apt-get dist-upgrade01:01
mwhudsonoh ok01:01
slangaseknacc: so your rebuild.3 list includes php-hamcrest, which I know I just did a non-no-change upload of...01:01
mwhudsonthat's not supported but actually works in practice?01:01
stgrabermwhudson: depends on the lts-to-lts in question, sometimes, as was the case with multiarch, the source apt isn't capable of understanding some package relationships in the target series, in which case the upgrade will blow up01:04
stgrabermwhudson: that's why we don't support dist-upgrade officially01:04
mwhudsonok01:04
stgrabermwhudson: do-release-upgrade downloads an upgrader tarball which contains the apt and dpkg of the target series01:04
mwhudsonthat shouldn't be a problem trusty->xenial?01:04
stgraberyeah, I'm not aware of any big archive change which would be a problem01:05
stgraberhowever if you do find an apt bug, then you'll be stuck :)01:05
=== chuck_ is now known as zu
=== zu is now known as zul
mwhudsonoh no i might break a container!01:32
mwhudsonthis happened to an upgrade again: http://paste.ubuntu.com/15548112/01:32
mwhudsonis there some idle timeout somewhere?01:32
stgrabercould be logind or something killing your session01:33
sarnoldno details of any sort printed? how annoying01:34
mwhudsoni also get this when i log back in: http://paste.ubuntu.com/15548150/01:36
mwhudsonshould i file a bug for this?01:36
mwhudson(and if so, which project?)01:36
cyphermoxmwhudson: there's not enough to know there, you'd want to look at /var/log/dist-upgrade if it mentions anything to clue you in which package has an issue, otherwise it might be ubuntu-release-upgrader01:38
cyphermoxthe ssh session seems to die too early to be because of stuff being upgraded, though01:39
mwhudsonthere's nothing much odd looking in /var/log/dist-upgrade01:40
naccslangasek: argh, sorry! just forgot to update the list01:51
naccslangasek: that's the only one that should be cross-listed01:51
=== juliank is now known as Guest77434
=== juliank_ is now known as juliank
mwhudsonwell that odd session killing behaviour doesn't happen when you just run dist-upgrade02:22
mwhudsonso i guess that's a ubuntu-release-upgrader bug02:22
cyphermoxerr, we don't do stuff to the session in u-r-u02:25
cyphermoxor at least, not that I can think of would make sense, or that I've noticed in the code so far02:25
cyphermoxI also already did an upgrade successfully not that long ago, a few days maybe02:25
mwhudsonhmm02:28
mwhudsonit could also point to a problem in lxd i guess?02:28
mwhudsoncyphermox: did you do your upgrade in lxd or a real system?02:29
cyphermoxfile a bug against ubuntu-release-upgrader, and make sure the logs are included (whatever is in /var/lib/dist-upgrade, as well as the apt-clone tarball if possible)02:29
cyphermoxit was a VM, but not lxd02:29
cyphermoxI may be wrong about the upgrade being successful. that would be a good data point for the desktop upgrade issues too02:30
mwhudsonah i've killed the lxd02:30
mwhudsonwill reproduce in a moment02:31
* mwhudson gets ready to use his new superpowers for the first time02:41
* sarnold stands back02:41
mwhudsonno regular expressions are involved02:43
mwhudsonfiled https://bugs.launchpad.net/ubuntu/+source/ubuntu-release-upgrader/+bug/1563168 fwiw03:13
ubottuLaunchpad bug 1563168 in ubuntu-release-upgrader (Ubuntu) "14.04 -> 16.04 upgrade in lxd container dies most of the way through" [Undecided,New]03:13
cpaelzergood morning06:25
dholbachgood morning06:43
mwhudsonmy first upload to ubuntu appears to have migrated fine without exploding the universe, i guess that's good :)07:06
infinitymwhudson: Alternately, whatever you screwed up is something our infrastructure doesn't catch. ;)07:06
mwhudsoninfinity: yeah, probably07:06
mwhudsonhaha07:07
mwhudsoni was thinking of making some joke along the lines of "given that there is a lxd upload every 10 minutes or so we'll find out soon"07:08
mwhudsonand it turns out that lxd was published in -proposed *3 minutes ago*07:08
infinity:)07:09
infinitymwhudson: If lxd is your testcase, I'm assuming the implication is that you've finished the go-default migration and all should be good?07:10
infinitylxd built everywhere, so that's a good sign.07:10
mwhudsoninfinity: yeah, that's all done, it was just some tweaking today07:11
mwhudsonand a new patch from IBM07:11
infinitymwhudson: Well, congrats on getting your upload on.  Sorry I wasn't there to give another +1, but clearly I wasn't needed. :P07:12
mwhudsonthanks!07:12
cpaelzerjamespage: that dpdk upload of last wednesday still doesn't show up but also isn't in update_excuses07:59
infinitycpaelzer: It's sitting in the unapproved queue, awaiting review.08:00
infinity(long weekends hurt)08:00
cpaelzerinfinity: oh thanks, at least I found it now08:00
cpaelzerinfinity: is there anything I can do to help?08:01
infinityOther than "not upload 26k diffs"?08:01
cpaelzerinfinity: yeah other than packaging bleeding edge stuff that feels like it could take 25k patches every day :-/08:02
cpaelzerjust trying to get things stable and working til release08:02
infinityUnderstood.  If no one beats me to it tonight, I'll review it in the morning.08:02
cpaelzerinfinity: thanks, and double thanks for identifying where it currently resides08:03
xnoxinfinity, lovely, but what is that?08:26
infinityxnox: systemctl status08:27
xnoxhahahahaha08:28
xnoxFailed to initialize libzfs08:30
xnoxfun08:30
=== marcusto_ is now known as marcustomlinson
cjwatsonjdstrand: Yep, thanks, will sort out ASAP.  Feeling very stupid about that one, I swear I tested but obviously not quite the final thing I uploaded.  I'll also add a syntax check to the build process so that sort of thing can't happen again.09:14
martinst-slangasek ping09:28
=== hikiko is now known as hikiko|ln
Mirvok I managed to find out that new glibc is the cause for bug #1560528. so I'll leave the bug to be against glibc while commenting out the test on 32-bit archs in order to get forward with new Qt upload.10:45
ubottubug 1560528 in qtbase-opensource-src (Ubuntu) "tst_LargeFile::mapOffsetOverflow started failing on 32-bit xenial" [Undecided,Incomplete] https://launchpad.net/bugs/156052810:45
=== hikiko|ln is now known as hikiko
ogra_smoser, pingaling ... seems cloud-init messes up a lot more on snappy (like creating a duplicated /e/n/i config now)12:06
ogra_(snappy creates that itself on first boot from snappy config)12:07
ogra_smoser, bug 1563296 for you12:15
ubottubug 1563296 in cloud-init (Ubuntu) "cloud-init 0.7.7~bzr1189-0ubuntu1 adds duplicated network config in /etc/network/interfaces.d/ in snappy leaving the boot completely network-less " [Undecided,New] https://launchpad.net/bugs/156329612:15
dokotumbleweed, are you asking for a FFe for pypy 5.0?12:22
=== inaddy_ is now known as inaddy
spineaucyphermox: hello13:47
spineaucyphermox: I commented on your tpm2-tools bug, https://bugs.launchpad.net/ubuntu/+bug/1561834/13:47
ubottuLaunchpad bug 1561834 in Ubuntu "[FFE] [needs-packaging] tpm2-tss and tpm2-tools" [Wishlist,Confirmed]13:47
spineaucyphermox: I've started to use the TSS tpmtest tool13:48
spineaucyphermox: you also could consider adding it to one of the existing binaries13:49
spineaucyphermox: it's not super verbose about the errors when it fails but it can certainly help developers/testers13:50
cyphermoxok13:51
spineaucyphermox: I looked at it this morning and upstream commits quite often to this tool, so I guess it's useful at least for them :)13:53
spineaucyphermox: sorry to jump quite late in the review though13:53
LocutusOfBorghi folks, can anybody please retry https://launchpad.net/ubuntu/+source/libvirt/1.3.1-1ubuntu8/+build/9336238 https://launchpad.net/ubuntu/+source/libvirt/1.3.1-1ubuntu8/+build/9336239 https://launchpad.net/ubuntu/+source/libvirt/1.3.1-1ubuntu8/+build/933624313:54
LocutusOfBorgthey should build now and make libvirt migrate13:55
LocutusOfBorg@core-devs ^^13:56
udevbotError: "core-devs" is not a valid command.13:56
caribouLocutusOfBorg: ok, will do13:57
LocutusOfBorgthanks13:58
LocutusOfBorghow is it possible?14:00
LocutusOfBorgdamn14:00
LocutusOfBorguniverse-main?14:00
LocutusOfBorgthanks, but LP: 1532198 needs to be sorted out before14:04
ubottuLaunchpad bug 1532198 in zfs-linux (Ubuntu) "[MIR] zfs-linux" [High,Incomplete] https://launchpad.net/bugs/153219814:04
slangasekmartinst-: hi14:06
=== kickinz1|eod is now known as kickinz1
SkuggenAnyone here familiar with dbconfig-common? I've been looking into LP: 1563274, but I can't find a simple solution14:24
ubottuLaunchpad bug 1563274 in mysql-5.7 (Ubuntu) "dbconfig-mysql sets blank port in config" [Undecided,New] https://launchpad.net/bugs/156327414:24
LocutusOfBorginfinity, did you get the time to dig into lp: 1562480?14:36
ubottuLaunchpad bug 1562480 in glibc (Ubuntu) "fp-compiler not installable on powerpc since glibc 2.23" [Undecided,New] https://launchpad.net/bugs/156248014:36
LocutusOfBorgI'm trying to read the code, but without real hw is impossible I think14:37
cyphermoxLaney: hey15:05
=== shuduo-afk is now known as shuduo
infinityslangasek, stgraber, kees, mdeslaur: Tee Bee?16:07
mdeslaurinfinity: ah! yes16:07
=== marcusto_ is now known as marcustomlinson
=== kickinz1 is now known as kickinz1|eod
=== Spads_ is now known as Spads
=== ubott2 is now known as ubottu
=== mfisch is now known as Guest28049
=== adam_g` is now known as adam_g
=== FourDollars_ is now known as FourDollars
=== attente_ is now known as attente
=== tlyu_ is now known as tlyu
=== lfaraone_ is now known as lfaraone
=== Pici` is now known as Pici
smosercan someone please confirm ? i'm17:23
smoserin xenial, nothing from udev will write /etc/udev/rules.d/70-persistent-net.rules17:25
smoserpreviously it would get written with a header like:17:26
smoser# This file was automatically generated by the /lib/udev/write_net_rules17:26
smoser# program, run by the persistent-net-generator.rules rules file.17:26
smoserbut '/lib/udev/write_net_rules' is gone now. as is 'persistent-net-generator.rules'17:26
cjwatsonjdstrand: do my debmirror fixes supersede RT#89946, or is that an independent problem?17:37
jdstrandlet me check. I think it is a different problem17:37
sarnoldsmoser: pit ti sent this around a while ago https://lists.ubuntu.com/archives/ubuntu-devel/2015-May/038761.html17:40
sarnoldsmoser: I'm not sure it was 100% done, but it looks like ifnames uses /lib/udev/rules.d/80-net-setup-link.rules instead17:41
jdstrandcjwatson: rt is still an issue17:41
smosersarnold, thank you17:42
tumbleweeddoko: do you want me to? :)17:51
tumbleweedI've spent the last two weeks fighting with it17:51
tumbleweeddoko: do you have access to a big endian porterbox with lots of RAM? I want to find if a patchset I'm proposing makes a significant difference on big endian. And it'll take me another couple of days on debian's porterboxes17:53
Pharaoh_Atemnacc: so, I've got a port of libvirt-php that is *supposed* to work on php7, which I'm getting some folks at my workplace to help complete so that I can submit it back upstream17:56
Pharaoh_Atemnacc: it's available here in the php7 branch: https://gitlab.com/Conan_Kudo/libvirt-php717:57
Pharaoh_Atemnacc: do you know what the latest date it can be, before we'd have to make a decision on dropping a package?17:58
Pharaoh_Atembecause I'm trying to get the timelines worked out so that I can get everything done and submitted upstream in time17:58
naccPharaoh_Atem: depending on the importance, we might be able to leave it as uninstallable18:00
naccPharaoh_Atem: it's good, in some sense, that it's in universe18:00
Pharaoh_Atembecause there's not much in the way of guarantees there?18:01
naccwell, if it was in main, that wouldn't be an option18:03
sarnoldthere's always kicking it out of main, too :)18:03
naccPharaoh_Atem: what's your tentative eta?18:03
Pharaoh_Atemnacc: I'm trying to pull together the finishing work within the next week or two, and try to get it reviewed and upstream a week afterward18:04
Pharaoh_AtemI'd like to pull the timelines in more, but it's getting really hard to get resources18:04
Pharaoh_Atemfrom what Remi said, the current tree should be buildable, but I have no idea how well it works because the test coverage is abysmal18:05
naccPharaoh_Atem: yeah so that'd be outside of getting into 16.04 if it's not available upstream yet, but let's sync up again tmrw (i'm trying to get as much of the list that can be done completed)18:07
Pharaoh_Atemokay18:07
Pharaoh_Atemnacc: do you know what the cutoff date would be to get it into 16.04?18:08
Pharaoh_AtemI can try to get things prioritized to make it in, or at least provide a squashed patchset to apply while it sits in review18:08
nacchttps://wiki.ubuntu.com/XenialXerus/ReleaseSchedule18:12
Pharaoh_Atemso would the April 14th date be the correct one?18:12
Pharaoh_Atemls18:13
Pharaoh_Atem>_>18:13
naccfinal freeze implies bugfixes only, so really it should be sooner than that, afaqict18:14
naccbut since it's in universe ...18:14
naccand it's not seeded, is it?18:14
Pharaoh_Atemhow would I know if it's seeded?18:15
naccit's not18:15
nacc`seeded-in-ubuntu libvirt-php`18:15
Pharaoh_Atemah18:15
Pharaoh_Atemyeah, it's not18:15
elbrusSkuggen: /me received the bug about dbconfig-common, thanks18:33
elbrusI will need to think about it18:34
elbrusand indeed, probably not a trivial solution18:34
=== juliank_ is now known as juliank
slangaseknacc: somewhat curious why all of these php-horde packages that are being changed now only to adjust test deps didn't show up as regressions when stuff was migrating earlier18:42
slangaseknacc: e.g. maybe things that only have test deps on php5 should be left to the end?18:42
slangasekor I guess these also have runtime deps for which they need no-change rebuilds, hmm18:42
naccslangasek: right, they really need no-change rebuilds, but then their test will forcibly use php518:43
slangasekgotcha18:43
nacc(afaict)18:43
slangaseknacc: sorry, ignore my vain attempt to reduce the number of uploads I'll be sponsoring today ;)18:43
naccslangasek: :) yeah it's a bit ugly; i'm going to go and try to clean up excuses again now and respond to the bugs from over the weekend18:44
elbrusSkuggen: do you know if /etc/mysql/debian.cnf is already fixed in mysql-server-5.7? Because dbconfig-common takes its defaults from there...18:44
SkuggenHang on, let me take a look at it.18:45
elbrusSkuggen: ignore my last comment18:45
Pici2518:45
naccslangasek: fyi, the php-pear failures (only on amd64) might be transient autopkgtest issues?18:46
Skuggenelbrus: Ah :D18:46
naccslangasek: can you check and possibly resubmit those?18:46
SkuggenIf you want to look at it, though, it's here: https://anonscm.debian.org/cgit/pkg-mysql/mysql-5.7.git/tree/debian/mysql-server-5.7.postinst#n16018:46
slangaseknacc: let me get through this sponsorship queue first and I'll look18:46
naccslangasek: sounds good, thanks18:47
elbrusSkuggen: so indeed debian-sys-maint logs in via the socket so doesn't have this issue18:47
Skuggenelbrus: We did actually find a bug in the 5.7 postinst when investigating the dbconfig issue; The debian-sys-maint user doesn't have access to grant other users access, which will cause installation of redmine/phpmyadmin to fail if mysql-server-5.7 is already installed18:47
SkuggenYeah, it shouldn't be affected by this issue18:47
elbrusSkuggen: you are coordinating/working as Debian mysql team as well, right?18:48
SkuggenYeah18:48
* elbrus is more a Debian guy than an Ubuntu guy18:48
slangasekneat, did not know syncpackage had a -V option18:48
naccslangasek: ah that's handy!18:49
slangaseknacc: especially since php-horde-mapi 1.0.8-1 is now available18:49
infinityslangasek: syncpackage has a -V option.18:49
SkuggenThe packaging I linked to is actually primarily for Debian, but we just apply a couple of changes for Ubuntu, like removing the libmecab-dev build-dep18:49
nacci was just looking for that and had given up for php-horde-mapi bug18:49
elbrusSkuggen: mariadb-server is/was setting the password to the empty string.. is that also going to fail?18:49
naccslangasek: yeah, i should see if i can patch requestsync to take one too :)18:49
slangasekinfinity: that's super neat!  now tell me something I don't know18:50
Skuggenelbrus: Both mariadb and mysql have changed to set root access through unix socket only if the password supplied is empty18:50
SkuggenEr, nevermind, that only affects the root user during installation of the server18:50
SkuggenBut no, string options can be blank18:50
elbrusSkuggen: and dbconfig-common during any database operation18:51
infinityslangasek: Emus can sprint at 30mph.18:51
elbrusbecause that runs as root18:51
slangasekinfinity: actual thought process: "darn, there's a newer version of php-horde-mapi that invalidates this sync request.  But wait, a sync is just an LP API call, and LP has imported the older versions too... I wonder if there's an option..."18:51
elbrusand takes the password from /etc/mysql/debian.cnf18:51
Skuggenelbrus: Ah, right, so if dbconfig installs the server it will set an empty root password?18:52
elbrus(as agreed with Otto Kel...)18:52
infinityslangasek: But yes, syncpackage is just (yet another) wrapper around copy-package, so if it didn't take a -V, it would have been trivial to add one.18:52
elbrusSkuggen: eh... dbconfig doesn't install the server18:52
infinityslangasek: I feel like half our tools are friendly wrappers around copyPackage(), which doesn't say much for the friendliness of copy-package(1) itself. :P18:53
Skuggenelbrus: I'm a bit confused. Does dbconfig use the root user or debian-sys-maint (which is the one in debian.cnf)?18:53
elbrusit just uses the credentials in /etc/mysql/debian.cnf to log into the local server to create the database and users18:53
slangasekinfinity: syncpackage does a copy-package plus other things (bug closures)18:53
Skuggenelbrus: Then it's using debian-sys-maint, and that's fine18:53
elbrusSkuggen: debian-sys-maint if the server is mysql-server and root if it is mariadb server18:53
infinityslangasek: Fair.  Other wrappers are also a bit more interesting (like sru-release).18:54
elbrusdbconfig just takes it from /etc/mysql/debian.cnf, so I don't care what is there18:54
SkuggenI'm not 100% sure about mariadb, but that should be fine for mysql18:54
infinityslangasek: Of course, then there's copy-proposed-kernel, which other than filtering *-signed, is pretty much JUST a wrapper so we don't have to remember copy-package args.18:54
infinityslangasek: But whatever works. :P18:54
elbrusI talked to the maintainer of mariadb at FOSDEM and we agreed that this was the best18:54
elbrusthe autopkgtests of dbconfig-common also test both mariadb and mysql18:55
Skuggenelbrus: Yeah, using debian.cnf will work fine. Mysql still uses that in the same way in the maintenance scripts. Only difference in 5.7 is that we removed a deprecated option from it (basedir)18:55
elbrusdo you know why the previous test used mysql-server-5.7 and the latest test mysql-server-5.6?18:55
SkuggenWhich latest test?18:55
elbrusautopkgtest of dbconfig-common18:56
infinityelbrus: If mysql-5.7 is still in -proposed (I assume it is), the default is to test against the release pocket.18:56
slangaseknacc: I beat you to php-horde-kolab-storage already (it was blocking migrations over the weekend)18:56
elbrusthat is how I already knew that it was broken, but the latest test appeared to be fixed18:56
naccslangasek: ah ok, thanks!18:56
elbrusinfinity: but it tests once against proposed to check?18:56
Skuggenelbrus: http://people.canonical.com/~ubuntu-archive/proposed-migration/update_excuses.html#mysql-5.7 is what I've been looking at18:57
slangaseknacc: I've noticed a few times that your patches are out-of-date wrt the archive; are you using pull-lp-source to make sure you always get the latest version when you start working?  Or is it just a delay between when you grab the package and when you submit the bug?18:57
SkuggenUploading of 5.7 to proposed would trigger it to test against it18:57
infinityelbrus: When a dep is uploaded, we test rdeps against it, and that's what we record as the current state for that pair of packages.18:57
elbrusSkuggen: than I am sooooo glad that I have those tests... I would have been a pain if we discovered afterwards18:57
infinityelbrus: So, what you're seeing in http://autopkgtest.ubuntu.com/packages/d/dbconfig-common/xenial/amd64/ is, essentially, a statement that "updating ucf doesn't break dbconfig-common, but updating mysql does".18:58
Skuggenelbrus: Same here. For instance I know this will also break phpmyadmin, but that didn't show up there :)18:58
naccslangasek: i think it's just the latter, i'll try to be more diligent about checking that going forward18:58
elbrusthat is because phpmyadmin uses dbconfig-common18:58
elbrusor something else?18:58
SkuggenIt can also use mysql as a backend, in which case it'll break with the same blank port option error18:59
infinityelbrus: So, the outcome of that is that we'd let ucf migrate, but we won't let mysql migrate until both mysql and dbconfig-common get along.18:59
infinityelbrus: If that makes sense?18:59
elbrussure18:59
elbrusso that puts presure on the solution as I expect that Ubuntu 16.04 wants to ship with 5.719:00
infinityIndeed, that's the goal, though time is certainly running out on realizing it.19:00
Skuggenelbrus: There is a potential stop-gap solution we could do on the mysql end if this is too big a task19:00
* elbrus is also involved with php7.0 transition... 19:00
* elbrus would like it that these things happen earlier in the cycle19:01
SkuggenYeah :|19:01
=== Guest28049 is now known as mfisch
SkuggenWe could patch the mysql client to allow the empty port option as it was in 5.6, but that really shouldn't be a lasting solution19:01
=== mfisch is now known as Guest82454
elbrusSkuggen: agree, but it would buy us time to find a proper solution19:02
elbruswithout rushing19:02
SkuggenYeah. I'm going to test it tomorrow.19:02
infinityelbrus: It seems that everyone's best work happens right after you announce a freeze; the obvious solution is for me to open a new cycle with "we will be frozen for the next six months, no major changes allowed" and watch the uploads roll in.19:02
=== Guest82454 is now known as mfisch
slangaseknacc: php-horde-javascriptminify-jsmin was also done already19:03
elbrus:)19:03
Skuggeninfinity: But you'd have to keep adding freeze cycles as people get wise to it :)19:04
Skuggenelbrus: I just wanted to get some feedback on whether this was simple to fix in dbconfig and I was just too confused by it to spot it :)19:04
* elbrus always tries to not rush things in, but indeed we also did it for lazarus (which now fails on powerpc because of glibc update)19:04
infinitySkuggen: Yup, it's like the alarm clock rewing arms race.19:04
infinitys/rewing/rewind/19:04
infinityelbrus: Yeah, I was looking into the fpc/ppc thing over the weekend a bit.  Still not solved, but I got... Somewhere.  I think.  Ish.19:05
elbrusSkuggen: I don't think it is trivial as it requires careful thinking about corner cases19:05
elbrusinfinity: I heard from ginggs19:06
SkuggenOk. I'll test the client patch tomorrow then. We really need to get these test issues cleared up :)19:06
SkuggenThis should fix the redmine failures as well19:06
elbrusSkuggen: redmine is using dbconfig-common as well...19:06
SkuggenYeah19:06
SkuggenBoth redmine and phpmyadmin get database config files from dbconfig, so the issue propagates to them19:07
elbrusindeed19:07
elbrusbut redmine probably got tested seperately because it does not require mysql (it can work with other dbs)19:07
SkuggenI'll post an update to the bug once I've tested the patch19:11
slangaseknacc: I think I'm keeping up with all of the patches you're attaching, but you might want to make sure when you're adding a patch to a bug for an already-uploaded bug that you reopen the bug19:46
naccslangasek: similarly, the php-crypt-gpg failure with ubuntu2 is a infra issue (internal timeout)19:47
naccslangasek: yep, duly notied19:47
slangaseknacc: yeah, the php-pear failures are marked 'tmpfail' on ci.u.c, unfortunately these just show up as 'regression' on update_excuses and there's no autoretry19:52
slangasek(retried now)19:52
naccslangasek: ah ok, thanks!19:53
naccslangasek: it seems like php-fdomdocument should go through, as the version of phpdox actually in the archive now did pass its tests?19:54
naccsame for php-fxsl19:54
slangaseknacc: usual process, we retry those tests now that the new version is in xenial so that we confirm that the cross-configuration works and there really aren't any regressions.  I'll retry those now19:55
naccslangasek: ok, thanks19:55
naccslangasek: make sense, i'm just not allowed to resubmit19:55
slangasekyep19:56
slangaseknacc: have you started your Ubuntu developer application yet?19:56
naccslangasek: no, that's on my todo for this week19:56
slangasek;)19:56
slangaseknacc: oh and php-apigen FTBFS again19:56
naccslangasek: i believe it needs the patch from LP: #156309819:57
ubottuLaunchpad bug 1563098 in php-apigen (Ubuntu) "Update d/rules UPSTREAM regex" [Undecided,Fix committed] https://launchpad.net/bugs/156309819:57
naccdoes it still fail with that?19:57
slangaseknacc: pretty sure that's the one I just uploaded19:58
slangasekyeah it is19:58
slangasekheh, https://launchpadlibrarian.net/250345806/buildlog_ubuntu-xenial-amd64.php-apigen_4.1.2-1ubuntu1_BUILDING.txt.gz19:58
slangaseknacc: did you build-test this, or blindly trust my regexp? ;)20:01
slangaseknacc: I think my regexp is right but that it's not properly escaped for makefile20:01
naccslangasek: blind trust :) my own fault will fix it locally and update20:01
naccslangasek: given that the build has failed, can i submit an update to the existing version?20:02
naccslangasek: or does it need to be an ubuntu2?20:02
slangaseknacc: must be ubuntu2. once the source is accepted by Launchpad, the version is burnt20:02
naccok, thanks20:03
naccslangasek: is there any reason the regex couldn't just be: 's/.*(//;s/-.*).*//' ?20:05
=== jgrimm is now known as jgrimm-afk
slangaseknacc: it's less generally correct for Debian versioning; the Debian part of the version number is always the part after the /last/ -20:27
naccslangasek: so, sed 's/.*(//;s/-[^-]*).*//'20:31
nacc?20:31
martinst-slangasek ping20:32
slangaseknacc: try and see, I think20:32
infinitynacc: Are you trying to split debian versions?20:46
infinityecho 2:1.2.3-5-09-0ubuntu1 | sed -e 's/\([0-9]*\):\(.*\)-\(.*\)/Epoch: \1, Upstream: \2, Debian: \3/'20:46
infinitySomething like that works.20:46
infinityThough less friendly when all the components aren't there...20:47
slangasekinfinity: he just needs the upstream version, and is trying to adjust the existing Debian packages' buggy regexp20:58
slangasekinfinity: and he's parsing dpkg-parsechangelog output in one step20:58
cjwatsonjdstrand: Thanks for checking20:58
Saviqslangasek, hey, I've a feeling new cmake will cause us a lot of trouble: bug #156354820:58
ubottubug 1563548 in cmake (Ubuntu) "CMake 3.5's pkg-config support broken" [Undecided,New] https://launchpad.net/bugs/156354820:58
Saviqmterry, FYI ↑20:59
slangasekSaviq: <self censoring>21:01
infinityslangasek: Right.  Well, my above crap regex lays out the components, but obviously fails if some aren't present.  Needs to be a bit more complex to adjust for the fact that all but the middle one are optional.21:01
slangasekWHO put cmake 3.5.0 into the archive past the freeze?21:01
infinityslangasek: I find it's easier (or, easier to read), if one just does it in three steps instead of one scary one.21:01
LocutusOfBorgslangasek, LP: 153426321:03
ubottuLaunchpad bug 1534263 in cmake (Ubuntu) "[FFe] [merge request] Import cmake-3.5 series to Ubuntu Xenial 16.04LTS" [High,Fix released] https://launchpad.net/bugs/153426321:03
Saviqok so maybe not *that* much headache, but at least unity8 doesn't build, and I can find a few more projects relying on those variables that are suddenly empty in cmake 3.5 for no good reason21:06
dokotumbleweed, well, it's in main, but as a b-d, so with the archive-reorg on the horizon, it still could be demoted21:06
Saviqthey should be there as far as I can read https://cmake.org/cmake/help/v3.5/module/FindPkgConfig.html21:06
slangasekSaviq: cross-build for unity? or native build?21:07
Saviqslangasek, native unity8 build21:07
slangasekLocutusOfBorg: ^^ please figure out why Saviq's build isn't working21:07
SaviqLocutusOfBorg, we rely on the _INCLUDEDIR variable to find some shared .h files, that var (and LIBDIR and VERSION) are now empty21:08
slangasekLocutusOfBorg, Saviq: I give this until my end of day to get sorted, and then I am uploading a revert of the cmake merge21:09
LocutusOfBorgslangasek, probably that path cross stuff in ubuntu2 might be the clue21:09
SaviqLocutusOfBorg, that's a diff between 3.2 and 3.5's cache for pkg_check_modules(GLIB2 glib-2.0) http://pastebin.ubuntu.com/15555106/21:09
LocutusOfBorgSaviq, can you please try with the shiny just uploaded cmake 3.5ubuntu2?21:10
SaviqLocutusOfBorg, sure, trying21:10
mwhudsonslangasek, infinity, nacc: if you are doing things to debian versions with regexes, consider just given up and writing perl instead21:10
slangasektwitch no21:11
mwhudsonDpkg::Version is more readable than a regex...21:11
SaviqLocutusOfBorg, FWIW this is native build, not cross, so I'd be surprised21:13
LocutusOfBorgSaviq, I saw your bug report as soon as you opened it, and I liked it already in the 1534263 bug21:14
LocutusOfBorgSaviq, building cmake on my wily, lets see21:15
LocutusOfBorgSaviq, I have not confirmed21:19
LocutusOfBorgslangasek, (please read)21:19
LocutusOfBorgbut your issue is the following code21:19
LocutusOfBorginclude(FindPkgConfig)21:20
LocutusOfBorgif you read that FindPkgConfig file21:20
LocutusOfBorgif(EXISTS "/etc/debian_version")21:20
LocutusOfBorg  include(${CMAKE_ROOT}/Modules/MultiArchCross.cmake OPTIONAL RESULT_VARIABLE _INCLUDED_MULTIARCH_TOOLCHAIN_FILE)21:20
LocutusOfBorgendif()21:20
Saviqah so maybe it is the missing bit after all21:20
Saviqthat'd be good21:20
LocutusOfBorgso probably this is defined in many other places, and somewhere it is blowing up stuff21:20
LocutusOfBorgit might seem not related, but cmake does rely on it in other places21:21
slangasekok21:21
* LocutusOfBorg tried to remove it once ago, when he did some merge work21:21
LocutusOfBorganyhow, 90% of the build completed, let me test21:21
* LocutusOfBorg was traveling on train, otherwise he could have restored the block tag quickly21:21
* Saviq sad nobody tried a cross-build when uploading new cmake :/21:27
Saviqwe're really not treating our cross story with the attention it deserves21:27
LocutusOfBorgSaviq, some autopkgtestsuite might be good21:28
LocutusOfBorganyway, I have been to busy and sick today to help as I wished21:29
LocutusOfBorgand I did my contribution a little bit late21:29
LocutusOfBorgI hope to fix everything in the next few hours, fever permitting21:29
slangasekSaviq: sorry, I was aware of the request to include a new cmake in xenial and that the cross-support might not have been handled, but I had not seen the FFe request or that pitti had granted it21:33
naccinfinity: yeah, what slangasek said; i wonder if there should just be some DEBHELPER option for doing it for you, but I've noted down your example; I think we've got the buggy regexs fixed for now21:38
SaviqLocutusOfBorg, afraid it doesn't help21:39
LocutusOfBorgSaviq, I saw21:46
LocutusOfBorgdebugging21:52
LocutusOfBorgin file FindPkgConfig.cmake:111 the variables are correctly filled21:53
=== jgrimm-afk is now known as jgrimm
LocutusOfBorgSaviq, I got it22:17
LocutusOfBorgprobably22:17
LocutusOfBorgSaviq, please test22:23
LocutusOfBorgin FindPkgConfig.cmake22:23
LocutusOfBorg-          _pkgconfig_set("${_pkg_check_modules_pkg}_${variable}" "${${_pkg_check_modules_pkg}_${variable}}")22:23
LocutusOfBorg+          _pkgconfig_set("${_pkg_check_prefix}_${variable}" "${${_pkg_check_prefix}_${variable}}")22:23
=== AndyTechGuy_ is now known as AndyTechGuy
naccslangasek: i'm not able to reproduce the php-horde-http failure from autopkgtest, and the error doesn't make a lot of sense to me, as i only changed the running to be php-cli (so it'd be the php7 runner) ... could that be a transient issue?22:31
naccslangasek: php-horde-mapi is a real failure and i'm trying to figure out how best to deal with it, might be a bit convoluted :/22:33
slangaseknacc: perhaps it's because php-horde-exception was uploaded in parallel, which means php-horde-http is being tested against the old version of php-horde-exception, which was still set up for php5?22:34
slangaseknacc: so not "transient", but ordering; can be given a straight retry once php-horde-exception is resolved, which is bound up with php-horde-mapi that you just mentioned22:35
naccslangasek: yep, that does look likely, thanks for clarifying22:35
LocutusOfBorgslangasek, can you please sponsor the debdiff?22:35
slangasekLocutusOfBorg: link?22:36
LocutusOfBorghttps://launchpadlibrarian.net/250356853/debdiff22:36
LocutusOfBorgjust attached to the bug22:36
LocutusOfBorgupstream messed up the parameters of the macro22:36
LocutusOfBorgthis patch is *so much* conservative, I keep both variables in the CMakeCache.txt file22:37
LocutusOfBorgeven if one seems rather useless22:37
LocutusOfBorgyou can also drop that line, I honestly think it was just a typo22:37
naccslangasek: ok, so I need some guidance here: there are two packages, src:php-seclib and src:php-phpseclib, v1.0 and v2.0 of the same library. Debian has repackaged the latter so they are coinstallable, because the former is backwards compatible to PHP5 (which they still support). Upstream seclib is not updating 1.0 to PHP7 compliance because upstream 1.0 is BC to PHP4 :) There was a point in the Debian22:38
naccpackage where they had broken BC and we probably woudl have passed the php-horde-mapi tests, but it broke BC and they reverted in Debian.22:39
naccslangasek: i see two ways forward: 1) we diverge from debian's php-seclib and put the php7 patch back in22:39
nacc2) we migrate the dependencies on php-seclib to php-phpseclib and remove php-seclib22:39
naccbut the debian packaging for php-phpseclib allows for both to be installed, so the php files in php-phpseclib get put in /usr/share/php/phpseclib ... which in turn makes them not in the usual place for callers22:40
slangasekLocutusOfBorg: uploading.  You've tested that this fixes Saviq's problem?22:40
LocutusOfBorgI'm updating the bug report with V=122:41
naccslangasek: so either way we'd diverge from debian, and i'm not sure which is better; debian's chagnelog (in git) also mentions not all users will be php-seclib 2.0 ready, although I'm not 100% on what the means22:41
LocutusOfBorgslangasek, that was the testcase I used to dig into the issue22:42
slangaseknacc: it sounds to me like re-patching php-seclib would be the quickest fix and require the least amount of chasing loose ends22:42
LocutusOfBorgso, yes, I confirm the variables are correctly filled on a xenial amd64 pbuilder clean chroot22:42
LocutusOfBorgand it seems a stupid upstream typo I'm going to report22:43
naccslangasek: yeah, let me see if we can do that easily22:43
LocutusOfBorgdamn damn damn22:45
LocutusOfBorgmy holy glories about contributing to cmake codebase vanished22:45
LocutusOfBorghttps://anonscm.debian.org/cgit/pkg-cmake/cmake.git/commit/?id=b71943bca4ed372568050d460960764ca2b3d4aa22:45
LocutusOfBorgtwo hours ago, version 3.5.1 landed in Debian22:45
LocutusOfBorg*exactly* the same fix22:45
sarnoldawww22:45
sarnoldbetter luck next bug :)22:45
naccslangasek: would you be ok with a sync to a version that's not current in Debian (1.0.1-3 vs. 1.0.1-4; we currently have 1.0.1-1) or would you rather we take 1.0.1-4 and revert the revert? I'm not sure which is clearer22:45
LocutusOfBorgsarnold, I already had one bug in kernel codebase, already fixed but not merged because of 4.6 window not yet open blah22:46
sarnoldLocutusOfBorg: \o/22:46
LocutusOfBorgpatch sent, but somebody was quicker for a few days22:46
sarnoldaww22:46
LocutusOfBorgand it happened with glib on sparc22:46
LocutusOfBorghappy to see people faster than me :D22:46
sarnoldthe other user found it first? hehe22:47
LocutusOfBorgI reported the bug on glibc it was fixed on 2.23 already, but debian and ubuntu were a little bit behind the latest release22:47
LocutusOfBorgand sparc not a release arch anymore, so nobody noticed it22:47
LocutusOfBorgslangasek, honestly I think because of the fixes in 3.5.1 (little changes, trivial to check, no new features and so on), 3.5.1 might be a better candidate for xenial22:49
LocutusOfBorgbut maybe I'll propose a debdiff if no new bugs are opened22:49
infinityIf we're committed to 3.5, 3.5.1 is likely better, yes.22:49
infinity(Saying that without looking, mind you)22:49
LocutusOfBorgI think we are committed to 3.5, if things don't break, or if things breaks and we can fix them quickly :)22:50
Saviqslangasek, LocutusOfBorg, confirmed fixes23:03
rharperslangasek: jgrimm: I generated a new debdiff for squid3 (https://bugs.launchpad.net/ubuntu/+source/squid3/+bug/1473691); this includes the dist-upgrade fix I identified on Monday;  it would be good to get this uploaded into proposed (to replace the current one) so we can further test squid in proposed with dist-upgrade and other scenarios.23:04
ubottuLaunchpad bug 1473691 in squid3 (Ubuntu) "[FFe] squid: Update to latest upstream release (3.5)" [Wishlist,Fix committed]23:04
LocutusOfBorg<3 Saviq23:05
LocutusOfBorgthanks23:05
LocutusOfBorgslangasek, if you want to sponsor again https://bugs.launchpad.net/ubuntu/+source/cmake/+bug/156358023:05
ubottuLaunchpad bug 1563580 in cmake (Ubuntu) "please merge cmake from debian" [Undecided,New]23:05
LocutusOfBorgbut you have to dget -u http://incoming.debian.org/debian-buildd/pool/main/c/cmake/cmake_3.5.1-1.dsc23:06
LocutusOfBorgor from my ppa :)23:06
SaviqLocutusOfBorg, FWIW glib-2.0_INCLUDEDIR is incorrect according to https://cmake.org/cmake/help/v3.5/module/FindPkgConfig.html23:07
jgrimmrharper, awesome.  thanks for looking at that. lingered longer than I'd wanted.23:07
Saviqprefix should always be there23:07
LocutusOfBorgSaviq, the new bug dropped it23:09
sarnoldrharper: oy, 21M debdiff?23:09
LocutusOfBorgI agree, and this is why I opened a new bug with a new debdiff23:09
LocutusOfBorgand no patch, since there is an upstream fix23:09
rharpersarnold: big jump23:09
jgrimmrharper, did debian have a bug there?23:09
rharpersarnold: 3.3 to 3.5.12; long overdue of course23:09
SaviqLocutusOfBorg, ah, ack23:10
rharperjgrimm: I didn't attempt to recreate on debian;23:10
sarnoldman that's gonna make users happy, they've been grumbly about the ages of the squid packages for a while :) thanks23:10
rharpersarnold: yea, and squidguard works again!23:10
sarnold\o/23:10
rharpersarnold: rabask did almost all of the work23:10
* doko grumbles why this took three years ...23:11
rharperI helped track down the dist-upgrade from trusty (with configured squid-deb-proxy/squidguard) setup that kickinz1|eod documented thoroughly in the bug23:11
sarnolddoko: presumably because it was held off slightly too long the first time around, 2.5 years ago.. hehe :)23:11
rharperrbasak is out for a bit and jgrimm doesn't want to wait any longer23:11
Saviqslangasek, LocutusOfBorg, thanks a bunch!23:11
LocutusOfBorgthanks to you23:12
LocutusOfBorgI hope to see cmake 3.5 in ubuntu23:12
LocutusOfBorgI already failed for libpng1.623:12
LocutusOfBorgI'm leaving23:13
LocutusOfBorgbye! if there are any troubles, please drop a note on bug reports23:14
LocutusOfBorgand I'll followup23:14
LocutusOfBorgtomorrow I'll have irc blocked at @company23:14
* LocutusOfBorg takes the remaining 4 hours of rest23:14
dokoLocutusOfBorg, https://launchpad.net/ubuntu/+source/cmake/3.5.0-1ubuntu3/+build/941430623:14
sladenssh -D23:14
dokogvingback ..23:15
LocutusOfBorgdoko, makes no sense with the patch23:15
LocutusOfBorgdoko, probably uploading 3.5.1 from the bug 1563580 is better23:15
ubottubug 1563580 in cmake (Ubuntu) "please merge cmake from debian" [High,New] https://launchpad.net/bugs/156358023:15
LocutusOfBorgbecause the patch introduces an useless variable in cache, and maybe it can trigger an output of some warnings23:16
dokoLocutusOfBorg, let's migrate this to the release pocket first23:16
LocutusOfBorgdoko, sure23:16
LocutusOfBorgactually the upstream commit has a patch for two files23:17
LocutusOfBorgone regarding ncurses23:17
LocutusOfBorgso the failure might be because of an incomplete patch too23:17
LocutusOfBorgmy builds seems fine https://launchpad.net/~costamagnagianfranco/+archive/ubuntu/locutusofborg-ppa/+sourcepub/6241790/+listing-archive-extra23:18
slangaseknacc: syncing to 1.0.1-3 sounds good to me23:18
naccslangasek: ok, bug filed (LP: #1563579)23:19
ubottuLaunchpad bug 1563579 in phpseclib (Ubuntu) "Please sync phpseclib 1.0.1-3 from Debian unstable" [Undecided,New] https://launchpad.net/bugs/156357923:19

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