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

slangasekdoko: did you give back cmake on arm64 again?00:04
slangasekbecause it seems to be failing repeatedly, and being given back00:05
slangasek(before I ever get to look at the build log)00:05
dokoslangasek, it was on the same buildd. will stop giving it back00:06
slangasekdoko: so there's a buildd-specific problem?00:07
dokoslangasek, I don't know, but usually I retry arm64 builds before I report issues00:08
slangasekdoko: heh, ok00:08
dokoso yes, maybe I should record arm64 ftbfs before I give these back00:09
slangasekdoko: and I see uploads for openjdk-7 dropping out of main, thanks :)00:09
dokoyeah, 6 already removed, 7 in universe, and hopefully removed as well00:10
naccslangasek: building and then will test a new version of mythtv with dpkg-maintscript-helper additions, and then will be done for the day (so your inbox can rest easy :)00:10
=== kitterma is now known as ScottK
slangaseknacc: heh; I just sponsored the php-horde-crypt change, but I also just noticed that php-horde-crypt 2.7.0-1ubuntu1 passes its tests again together with php-horde-http 2.1.5-3ubuntu200:15
naccslangasek: ah ok ... so maybe timing again; I don't think it should do any harm, and it will mean if we do have a network blip, it will skip & not fail, i think00:16
slangaseknacc: none of these are network blips, it's the deliberate network filtering policy on the autopkgtest runners00:18
naccslangasek: oh i see, horde-http being a the newere version fixed the issue properly, got it00:19
slangaseknacc: yeah, seems so00:19
tewardstupid obvious question, but, does landscape-client in Xenial not work the way it should with Landscape?  I only ask because in trying to set up an actual Xenial server, following my workflow of configuring it to Landscape, it chokes during server ISO setup...00:21
slangaseknacc: hmmm which arch was having the weird kernel timeout related bugs from one of the php test suites?00:39
dokoinfinity, please have a look at https://bugs.launchpad.net/ubuntu/+source/qtbase-opensource-src/+bug/156052800:41
ubottuLaunchpad bug 1560528 in qtbase-opensource-src (Ubuntu) "tst_LargeFile::mapOffsetOverflow started failing on 32-bit xenial" [Undecided,Incomplete]00:41
dokoslangasek, failed again, this time on a different buildd: https://launchpad.net/ubuntu/+source/cmake/3.5.0-1ubuntu300:43
slangasekdoko: yep; trying to figure out if this is related to another bug we saw with a php package's testsuite on one of the armen architectures00:43
slangasekdoko: that was php-imagick on armhf; but I guess that's also running on a 64-bit kernel, so could be the same issue00:44
slangasekhttps://objectstorage.prodstack4-5.canonical.com/v1/AUTH_77e2ada1e7a84929a74ba3b87153c0ac/autopkgtest-xenial/xenial/armhf/p/php-imagick/20160323_112011@/log.gz00:44
slangasekhmmmm completely different kernel version, though00:45
slangasek(3.13.0-45 vs. 4.2.0-30)00:46
=== zul_ is now known as zul
=== JanC_ is now known as JanC
=== juliank is now known as Guest82876
=== juliank_ is now known as juliank
naccslangasek: i think it was armhf w/ imagick?02:14
naccslangasek: also, i tihnk php-horde's failed amd64 test, php-horde-data's i386 failed test are transient; php-horde-mapi may need to be manually kicked now that seclib is in02:17
naccslangasek: and oddly horde http failed again ... will investigate in the am02:18
stgraberroaksoax: package renames AND copyright updates, you really like to make my life difficult...02:40
stgraberI'm sure the diff would actually be readable without all that stuff...02:40
sarnoldwhere does the '3819' come from in the ulimit -a output? it's also in e.g. systemctl show fwupd  or systemctl show sshd02:48
sarnoldthe usual grep -rl 3819 /etc /usr /lib  didn't do the trick02:48
slangaseknacc: php-horde-mapi retriggered02:48
slangaseksarnold: "3819"? for what limit?02:50
sarnoldslangasek: both pending signals and max user processes02:50
slangaseksarnold: those numbers are calculated dynamically based on available system RAM IIRC02:51
sarnoldinteresting02:52
sarnold# for u in $(systemctl list-unit-files  | awk '{print $1}') ; do systemctl show  $u | grep 3819 ; done | wc -l02:52
sarnold80002:52
sarnoldhehe02:52
slangaseknacc: php-horde-http failed again because it was still picking up the old -exception-, didn't wait long enough03:09
slangaseknacc: oh, rather, php-horde-exception is still blocked by php-horde-mapi, so *definitely* didn't wait long enough03:10
slangaseknacc: php-horde-http still fails, and this time I don't know why because it was using the new php-horde-exception04:22
pittiGood morning05:29
stgrabergood morning pitti05:30
stgraberpitti: the lxc adt failure on ppc64el is an actual regression, I just sent a branch upstream to fix it and cherry-picked the fix already. It's in the queue now if you have a minute to review.05:32
pittistgraber: accepted, thanks05:34
pittistgraber: wow, was the mirror default dropped accidentally, or was this somehow able to get along without ports.u.c. all this time?05:35
stgraberpitti: thanks, going to go get some sleep now :)05:35
stgraberpitti: the Debian maintainer sent a branch to set a default MIRROR so that debootstrap would be happy in Debian05:35
stgraberpitti: but he apparently didn't know that we don't have all arches on archive.ubuntu.com :)05:35
pittistgraber: squid3> btw, you know debian/pkgname.maintscript ?05:36
stgraberpitti: (and I apparently suck at review and didn't notice it)05:36
stgraberpitti: I sure do and squid3 is a mess. I just uploaded it because I accepted a buggy one earlier and felt bad :)05:36
pittiheh, thanks05:36
stgraberalso squid3 is cdbs... not that it matters much in this case, but it makes it just that much more painful to deal with05:37
pitticoreycb: can you please look at http://people.canonical.com/~ubuntu-archive/proposed-migration/update_excuses.html#swift ? your swift upload has been trapped by the regression for 3 weeks05:54
pittiyofel: FYI, baloo-kf5 segfaults during its tests on 32 bit (i386 and armhf); so it's been stuck in -proposed for 3 weeks already05:57
alkisgbdrung, bdrung_work, good morning, about xul-ext-adblock-plus, I see that you uploaded version 2.7.2+dfsg-1~ubuntu16.04.1~ppa1 a while ago, did you managed to get it to work with the extension signing etc?06:07
alkisgI managed to get the presigned .xpi to work with this command:06:07
alkisg# wget https://update.adblockplus.org/latest/adblockplusfirefox.xpi -O /usr/lib/firefox/browser/extensions/{d10d0bf8-f5b5-c8b4-a8b2-2b9879e08c5d}.xpi06:07
alkisgIf there are no alternatives, maybe xul-ext-adblock-plus could just package the signed blob?06:08
alkisgOr is there some way that I'm unaware of, to get the 2.7.2+dfsg-1~ubuntu16.04.1~ppa1 to work for all users?06:09
Unit193https://sources.debian.net/src/firefox/45.0.1-1/debian/patches/prefs/Don-t-auto-disable-extensions-in-system-directories.patch/ :P06:10
alkisgUnit193, bdrung has already proposed a patch there: https://bugs.launchpad.net/ubuntu/+source/firefox/+bug/153248406:14
ubottuLaunchpad bug 1532484 in firefox (Ubuntu) "Don't warn about unsigned extension installed via Debian packages" [High,Confirmed]06:14
alkisg....but chrisccoulson replied,  "This isn't something that we're going to be changing in Ubuntu", dunno why06:15
Unit193alkisg: Heh, was mostly kidding there, sorry that wasn't more clear.06:17
mwhudsoncan i get an Opinion06:28
infinitymwhudson: Grilled cheese sandwiches are best if you don't add anything other than the cheese.06:29
Unit193infinity: I like bread with mine.06:30
RAOFWe have a new candidate for History's Greatest Monster: infinity!06:30
mwhudsoni've been asked to sponsor a merge from debian, and it disables a feature in debian by not having the libxxx-dev package in build-depends06:30
infinityNot uncommon.06:30
mwhudsonwhich seems a little smelly, it's fine for the buildds i guess but it means a developer building the package on her system might get different functionality06:30
infinityGenerally to keep things out of main (or to avoid filing an MIR :P)06:31
mwhudsoni guess the lesson here is "autoconf sucks"?06:31
mwhudsoninfinity: i think this is actually not the case here :-)06:31
Unit193mwhudson: Look at irssi.06:31
infinitymwhudson: Correctly, it should drop the build-dep *and* use --disable-feature, but people rarely bother with the latter.06:31
mwhudsoninfinity: it's to avoid an ffe i think :)06:31
mwhudson(it's tgt, there's stuff from you on the bug)06:32
mwhudsonUnit193: do i have to? :-)06:32
infinityIs there?06:32
Unit193mwhudson: No.06:32
mwhudsoninfinity: ok, noted06:32
infinityI'm asked to comment on bugs often.06:32
infinityRefresh my memory?06:32
mwhudsoninfinity: https://launchpad.net/bugs/155570006:32
ubottuLaunchpad bug 1555700 in tgt (Ubuntu) "Please merge tgt 1.0.63 from Debian (unstable)" [Undecided,New]06:32
infinityWas this the libaio thing?06:32
mwhudsoninfinity: you didn't comment, but irc logs were pasted06:32
mwhudsonunless they were excellent forgeries of logs :-p06:33
mwhudsonyeah, the aio thing06:33
infinitymwhudson: Anyhow, yeah, dropping the build-dep is a common way to achieve that.  Strictly speaking, you're right, and one would also either use Build-Conflicts or (more sanely) pass --disable-thing to configure, but Debian and Ubuntu have moved to a "there's no way we can conflict with everything correctly and specifying every single configure option is a pain" stance, combined with "oh, look, source uploads and clean build chroots", so...06:35
infinitymwhudson: Basically, yes, "the user's build might differ" sucks a bit, but also who cares.06:35
mwhudsoninfinity: ack06:35
Unit193d/control needs an easier way to have added build-deps in derivs, or 'build-recommends' to replace the silly dephere | something in main thing. :/06:39
mwhudsonthere must be some punctuation that doesn't have meaning to dpkg left, right?06:40
infinityWe could build everything in ubuntu in an ubuntu build-profile, and you could use "Build-Depends: foo, bar <!ubuntu>, baz"06:41
infinityBut really, the effort there would be convincing Debian maintainers to take your patch.06:41
alkisgIn LTSP we've implemented directives like X-Ubuntu-Depends: etc to use the same packaging in both Debian and Ubuntu06:42
Unit193infinity: What if I'm the Debian maintainer?  ;)  Problem being, something Ubuntu isn't OK with either.06:44
infinityUnit193: Sure, if you're the Debian maintainer, it's a bit easier, but if you're the Debian maintainer, you also don't have a hard time tracking your own merges, I suspect. ;)06:45
infinityThe real issue with Debian deltas are all the "lost" merges from drive-by bugfixes.06:45
Unit193infinity: Can't upload to Debian or Ubuntu, so moot on the sponsors queue.  And indeed. :/06:45
Unit193infinity: BTW, I'm right in thinking Ubuntu has the same stance as Debian on GPL/OpenSSL linkage?06:47
infinityUnit193: Yes.06:47
Unit193Figured, dang.06:48
infinityIt's usually not hard to get upstream to agree to a license exception.06:48
infinityEspecially if they wrote their application to use openssl. :P06:48
Unit193They did and would like it, but they won't as they can't contact all contributors.06:49
infinityPersonally, I'm of the opinion that if the work is 100% original code and was obviously a derived work from day 1 (ie: it always linked to OpenSSL), the exception is implied, but that's not a particularly sane legal argument.  And falls down hard as soon as the project pulls in any GPL code from elsewhere, or if the openssl-using bits were written later, or...06:51
Unit193So I just end up building it too.  LibreSSL to save the day? /s06:53
infinityUnit193: LibreSSL would have the same issue, it's an OpenSSL fork.06:54
Unit193Anywho, I'll stop bugging you unless you feel 'uploady'.06:54
infinityI'm feeling more sleepy than uploady.06:55
Unit1933am here.06:56
mwhudsonoh hell why does trying to build things in my trusty schroot unmount my home directory :(07:08
dholbachgood morning07:09
infinitymwhudson: Unmount it in your host, you mean, or fail to mount it in the schroot?07:14
mwhudsoninfinity: in my host07:14
infinitymwhudson: !07:14
infinitymwhudson: That's special.07:14
mwhudsoninfinity: adt-run did it once too!07:14
mwhudsonbut yes07:15
* infinity resists the urge to get involved and sleeps instead.07:15
mwhudsoni have the # Mount a large scratch space for the build, so we don't use up07:15
mwhudson# space on an LVM snapshot of the chroot itself.07:15
mwhudson stuff in the schroot profile07:15
mwhudsonwhich given i don't use lvm is probably a touch pointless07:16
mwhudsoninfinity: go to bed07:16
infinitymwhudson: If you use ephemeral chroots with a tmpfs for the overlay (like I do), then the scratch is useful.07:16
mwhudsoni don't think i do that07:17
mwhudsonunion-type=overlayfs07:17
mwhudsonanyway, go to bed :)07:17
infinitymwhudson: I use union-type=overlays combined with:07:17
infinityschroot         4.9G     0  4.9G   0% /var/lib/schroot/union/overlay07:17
infinitymwhudson: Which makes build-deps install basically instantly.07:18
mwhudsonhm07:18
infinityBut then I build in a scratch, cause disk speed at build time is less important.07:18
infinity# Mount a large scratch space for the build, so we don't use up07:18
infinity# space on an LVM snapshot of the chroot itself.07:18
infinity/var/lib/sbuild/build/buildnonerw,bind0007:18
infinity/home/homenonerw,bind0007:18
mwhudsoninfinity: do you have encypted home dir?07:18
infinityNein.07:18
infinityFull disk.07:18
mwhudsonluks or something?07:19
infinityOr something, yes.07:19
infinity(oh, and lest that "schroot" confuse you, it's just this:)07:20
infinityschroot  /var/lib/schroot/union/overlay  tmpfs  defaults,size=25%  0 007:20
infinityBasically, gets you best of both worlds.  tmpfs for build-dep installation (especially when also backed by a proxy of some sort) means that step happens in seconds, then the build happens on a real ext4 filesystem in the scratch dir and can take all the space it wants (and not suffer weird bugs due to being on overlayfs)07:21
mwhudsonyeah i have squid-deb-proxy installed and omg i'm so glad i did that07:22
infinityhome mounted for sbuild profiles is entirely optional, but I'm lazy and reuse the same "custom" profile for sbuild and schroot.07:22
bdrung_workalkisg, no update on the signing issue.07:50
mwhudsonwell that was all very exciting07:50
mwhudsonthere is a bug about this, apparently fixed in xenial07:50
mwhudson(i'm still on wily)07:51
mwhudsongrabbing the schroot packaged from xenial didn't fix the problem though07:51
=== kickinz1|eod is now known as kickinz1
alkisgbdrung_work: are you interested in packaging the signed .xpi? Or should I do that in some PPA of my own, until some other method is found?07:51
mwhudsonhttps://bugs.launchpad.net/ubuntu/+source/schroot/+bug/143055707:52
ubottuLaunchpad bug 1430557 in schroot (Debian) "sbuild / schroot unmounted encrypted home directory" [Unknown,Confirmed]07:52
alkisgchrisccoulson, is it possible to get a reason why bdrung_work's patch for signed extensions can't be accepted?07:52
bdrung_workalkisg, i am against packaging signed xpi file as they are not the preferred form for modifications07:53
alkisgbdrung_work, ok, thanks, I'll override the xul-adblock-plus package from my ppa07:53
bdrung_workalkisg, as alternative, you can still disable the signed xpi check in about:config07:55
alkisgbdrung_work: I tried with lockPref and it's not working for all users that way07:55
alkisgFor single users, it does work07:55
alkisg*per user07:55
alkisgI don't want to have to tell to all the 5-10 year old students how to go to about:config...07:56
bdrung_workvalid point07:56
alkisgI also asked chrisccoulson for a reason behind his refusal in your bug report, but no reply there either07:56
alkisgDebian did accept a similar patch...07:56
jamespagemorning all07:58
Unit193alkisg: And you can't just override it in something like  /usr/lib/firefox/browser/defaults/preferences/unit193.js  ?08:01
alkisgUnit193: for some reason, the usual method for overriding it doesn't work... I tried: lockPref("xpinstall.signatures.required",false);08:03
alkisgIt did disable it in about:config, yet it was not actually working08:03
alkisgMaybe the firefox devs are thinking that this is a very significant to allow it to be overriden by the sysadmin, they want it per user? dunno.08:04
alkisg*a very significant setting08:04
Unit193Actually meant like Debian did, but again not tested that I just ship signed. :P08:07
alkisgDebian patches firefox (iceweasel) afaik08:07
alkisgbdrung also proposed a patch for firefox, but it's not getting accepted...08:08
alkisgOK thank you guys, I'll package the adblock plus blob in a PPA08:08
Unit193(It's actually just 'firefox' now.)08:10
alkisgAh, nice!08:13
=== Zic is now known as Guest49910
Saviqpitti, hey, did you recover over Easter?09:49
Saviqpitti, have a question about britney - we've removed the only test from qtmir and qtmir-gles (because it was a "does-it-build" test, not suitable for britney, especially in the new trigger-based britney)09:52
Saviqnow britney will complain that this is a regression because there's no tests at all09:52
Skuggeninfinity: Patched mysql-5.7 to work around the dbconfig issue. Seems to work fine, so once rbasak is back tomorrow we can do another upload, I think09:53
Skuggenelbrus: ^09:53
pittiSaviq: I did recover, thanks! I hope you had some nice holidays too09:55
Saviqpitti, actually now looking at http://autopkgtest.ubuntu.com/packages/q/qtmir/ maybe it's ok with it after all...09:56
pittiSaviq: qtmir does not have a Testsuite: any more, so why would britney try to run a test for it?09:57
pittiSaviq: do you have the excuses URL of that silo?09:57
Saviqpitti, seems to be running here https://requests.ci-train.ubuntu.com/static/britney/vivid/landing-066/excuses.html09:58
pittiSaviq: oh, is that the silo that drops the test?09:58
Saviqpitti, no, the test is dropped in trunk already09:59
pittiSaviq: it's "always failed", so it won't block anything09:59
Saviqhttps://bazaar.launchpad.net/~mir-team/qtmir/trunk/revision/46109:59
pittiSaviq: and gsettings-qt should run against the qtmir in the archive/overlay, not in the silo10:00
Saviqpitti, ok let's see how it fares, I think it was less happy on xenial, but there's no excuses for that yet10:00
pittibut if that still has the tests, they will just run the old one, and if it's the new package without tests it wouldn't be triggered any more10:00
Saviqpitti, ok, I'll let you know if I have trouble after all, thanks10:00
pittiSaviq: ack, thanks10:00
=== Guest49910 is now known as Zic
=== Zic is now known as Guest54678
=== Guest54678 is now known as Zic
=== marcusto_ is now known as marcustomlinson_
=== marcustomlinson_ is now known as marcustomlinson
Saviqpitti, yeah, it's unhappy on xenial for some reason https://requests.ci-train.ubuntu.com/static/britney/xenial/landing-066/excuses.html10:53
Saviqpitti, what's also weird is that it's not running tests for unity8 for some reason10:54
Saviqand don't ask me what's "-ginkgocadx"10:54
=== Zic is now known as Guest6725
=== Zic is now known as Guest71894
=== Guest71894 is now known as Zic
pittiSaviq: ah, qtmir in landing-066 xenial still has Xs-Testsuite: autopkgtest12:10
pittiSaviq: so apparently this somehow came back12:10
pittior wasn't removed properly12:10
dokomwhudson, docker.io ftbfs on s390x12:15
Saviqpitti, oh.... debian/control... forgot to remove in there indeed, will have a follow-up MP12:24
=== arges_ is now known as arges
Saviqpitti, any idea why unity8 tests don't run there? it's in the silo...12:24
Saviqhuh... it doesn't have Xs-Testsuite: autopkgtest ¿?12:26
pittiSaviq: unity8 lost its Testsuite: header12:27
Saviqwaaat12:27
pitti(XS- prefix is obsolete, btw)12:27
Saviqwonder if it ever had it...12:27
pittiand normally it is being added automatically12:27
pittiif there's a d/t/control12:27
pittinope, no debian/tests/ in unity812:28
Saviqbut it's there... https://bazaar.launchpad.net/~unity-team/unity8/trunk/files/head:/debian/tests/12:28
Saviqpitti, why do you say it has no d/t/control?12:31
Saviqhttps://launchpadlibrarian.net/250392626/unity8_8.12+16.04.20160330-0ubuntu1.diff.gz lists it (granted, no XS-Testsuite: autopkgtest - since it was added automagically as you say)12:31
pittiSaviq: erk, sorry, was mis-looking at qtmir12:31
Saviqmaybe it's not auto-added any more and we need it explicit now?12:31
pittihttp://ppa.launchpad.net/ci-train-ppa-service/landing-066/ubuntu/dists/xenial/main/source/Sources.xz does not have the header, though12:31
pittior maybe it's only added for v3 sources, not sure how dpkg handles that12:32
Saviqok, so we need that header explicit now, /me will add (and drop from qtmir)12:32
pittibut xenial's unity8 has a Testsuite: header, the PPA doesn't12:32
Saviqthat explains things, thanks pitti12:32
pittiSaviq: oooh!12:33
pittiSaviq: there's no relevant diff in the source12:33
pittiSaviq: but robru recently changed to building sources not in a schroot12:34
Saviqprobably12:34
pittiand I bet he's using the old trusty dpkg-dev to build the sources, that doesn't add Testsuite: yet12:34
pittibefore we'd have used xenial's dpkg-dev to build sources12:34
Saviqyup12:34
pittithat explains why the header isn't auto-added any more12:35
xavigarciapitti: Hi there!12:38
pittihello xavigarcia12:38
xavigarciapitti: I'd like to ask you a question about upower12:38
xavigarciapitti: I'd need to change the configuration ONLY for the phone... to use percentages and poweroff the phone12:38
pittixavigarcia: that's a matter of presentation12:39
xavigarciapitti: do you know if there's any other way to overwrite the config file other than using dpkg-divert?12:39
pittiupower tells you percentages and absolute limits12:39
pittiand it does not "power off" the phone12:39
pittioh, for critical action you mean12:40
pittisorry, I thought you meant the hold reboot/shutdown API12:40
xavigarciapitti: I did some tests last week, and it did it... yeah... it's for the critical action12:40
pittixavigarcia: it's a conf file, so you can just change the values12:40
xavigarciapitti: this is the MR I wanted to land: https://code.launchpad.net/~xavi-garcia-mena/ubuntu/vivid/upower/percentages-power-off/+merge/28993512:41
* ogra_ would do it from a livecd-rootfs hook script (as i said in the other channel)12:41
pittidoes HybridSleep really work on a phone?12:41
pittiand hibernate?12:41
pittiI would have thought we disable that, and let it fall back to power off12:42
xavigarciapitti: I neede to change it to PowerOff12:42
xavigarciapitti: that's the main change in the config12:42
ogra_hibernate doesnt (not enough swap)12:42
=== _salem is now known as salem_
ogra_according to the comment that should automatuically fall back to PowerOff without you doing anything12:44
pittiso I'd think that it's easiest to keep such config changes central in the image build scripts (livecd-rootfs), otherwise just change the file and put that into the overlay12:45
ogra_yeah12:45
xavigarciapitti ogra_: that was my initial intention12:46
xavigarciapitti, ogra_: that's why I already had the MR, but Pat was asking to do that in a separated package12:46
xavigarciapitti, ogra_: but as upower is already managing the config file I don't see any other way to do that than using dpkg-divert12:48
ogra_xavigarcia, just use sed12:48
pittino, don't divert a conffile, this is going to blow up12:48
xavigarciaogra_, pitti: well, that's another option, but not the best, as the config file is still maintained by upower :/12:49
ogra_thats fine on the phone, really12:49
xavigarciaogra_, pitti: I think I will retry the overlay again12:49
ogra_we dont support apt upgrades12:49
pittiwell, you never dist-upgrade that anyway12:49
xavigarciaogra_, pitti: oh, ok... thanks for the reminder!12:50
ogra_sed -i 's/^UsePercentageForPolicy=.*/^UsePercentageForPolicy=true/' /path/to/UPower.conf12:50
ogra_just put something like that in a livecd-rootfs hook script12:50
=== francisco is now known as Guest87927
xavigarciaogra_ we could maybe add that to the ubuntu-phone-session package...12:51
xavigarciasorry, ubuntu-touch-session12:51
ogra_nah, just do it at build time12:51
ogra_no need to run that every session startup and to slow down the start12:52
xavigarciaogra_ but in fact...if we apply it at build time... what would be the difference to adding it to the config file?12:52
ogra_bzr branch lp:livecd-rootfs ...12:52
ogra_add your snippet under: live-build/ubuntu-touch/hooks/12:53
ogra_and creat an MP from that12:53
ogra_the difference is that it will also apply if you switch to a new upower version without you having to hack the package12:53
xavigarciaogra_: I see.... thanks for the pointers!12:54
xavigarciaogra_: Could you please take a look to the following MP? https://code.launchpad.net/~xavi-garcia-mena/livecd-rootfs/bug-1317860-upower-poweroff-phone-only/+merge/29045413:13
xavigarciaogra_: I've assigned it to you... the config change was already reviewed last week in the following MP: https://code.launchpad.net/~xavi-garcia-mena/ubuntu/vivid/upower/percentages-power-off/+merge/28993513:13
xavigarciaogra_: it was rejected after Pat asking to do it in a different way... but it was previously approved13:14
ogra_xavigarcia, commanted ... please give it to sil2100 to merge it in the overlay livecd-rootfs though13:18
ogra_*commented even13:18
xavigarciaogra_: cool... thanks!13:19
ogra_i'm happy to merge it in xenial once it landed in teh overlay (for consistency)13:19
xavigarciasil2100: ^13:19
xavigarciasil2100: please see ogra's messages and my MP link13:20
pittiSaviq: FYI, I just deployed the fix for bug 154491713:29
ubottubug 1544917 in Auto Package Testing "Retry says "does not have any test results" on reported Regressions" [Low,Fix released] https://launchpad.net/bugs/154491713:29
Saviqpitti, ack, thanks :)13:35
sil2100o/13:52
sil2100xavigarcia: on it13:54
xavigarciasil2100: great, thanks!13:55
caribouslangasek: just saw LP: #1563027 after doing the work on this this morning14:10
ubottuLaunchpad bug 1563027 in vsftpd (Ubuntu) "vsftpd 3.0.3-3ubuntu1 fails to start on installation" [High,Confirmed] https://launchpad.net/bugs/156302714:10
slangasekcaribou: well, it's all yours :)14:42
caribouslangasek: just wondering if I should just revert the value or wait to hear from the DM first14:42
slangasekcaribou: IMHO there's no need to revert, this is pretty much the only change in -proposed and it can just sit there until resolved14:44
caribouslangasek: thought so; I'll wait to hear about the DM then14:44
rharperslangasek: I'm testing wily squid3 upgrade to xenial with the existing proposed package;  I'm trying to confirm if a wily -> xenial upgrade also trips the same logic from debian and if the /etc/init.d/squid3 service file exists (where it didn't for trusty -> xenial upgrade);14:53
naccha! as brief as it might be, excuses is nearly php clean right now :)15:04
naccslangasek: thanks for all your help, as usual!15:05
tewardis the ubuntu-server metapackage new in Xenial?15:05
naccslangasek: just sent another sync request which will clear out the remaining one that should be fixed15:05
tewardI... just answered my question15:05
naccteward: looks like it acc'g to rmadison15:05
tewardnacc: i have a question as to why lxc is installed as part of it by default15:06
teward:/15:06
naccteward: it's on the server seed15:06
ogra_is it ?15:10
* ogra_ thought it was added to -standard15:10
nacclxc (from lxc) is seeded in:15:10
nacc  ubuntu-server: daily15:10
ogra_i wonder why (given we (will) have it in desktop by default as well)15:11
ogra_standard would seem more logical15:11
slangasekstandard is inherited by all flavors; this is an Ubuntu Server+Desktop product decision15:12
* teward scratches his head15:13
dokoopenjdk-7 demoted15:13
ogra_slangasek, ah, but that means not all ubuntu features will work in all flavours15:14
tewardogra_: i'm going to stop the !crosspost that i appear to be doing, and going to stick to here specifically, did I miss an email chain where this was discussed?15:14
* teward might have, receiving too many emails a day15:15
ogra_not sure if there were mails15:15
ogra_theer were definitely several announcements that it will come in 16.04 throughout the development cycle15:15
ogra_and at UOS15:15
tewardmmm, well i've been so engrossed in nginx lately and my own studies I guess the mails slipped by me15:16
slangasekogra_: if you expect this to land in standard, you'd better start a discussion with the flavor teams via ubuntu-devel15:16
tewardit seems odd, though, that container support would be needed - but I think this needs to be discussed more, because if it's included on every server, and they never use it, why should it be in the ubuntu-server seed *shrugs*15:16
tewardmy own opinion though :)15:17
tewardand by 'they' i mean any user of the server installer and server flavor of Ubuntu15:17
ogra_slangasek, i dont expect it anywhere ... it was just my assumption that we call it a default ubuntu feature nowadays (iirc it is also the base of supporting snappy packages which i imagine flavours would like)15:17
tewardoh that explains it :)15:17
* teward facedesks15:17
ogra_but i trust the persons that made the decision that they thought about it ;)15:18
ogra_teward, why is that ...15:18
tewardogra_: well, afaict it's not in -standard, only -server.  the head-scratching left behind makes me think that if it should be in -standard but isn't, then there's something missing15:18
tewardogra_: because i'm old fashioned :)15:18
* ogra_ will surelöy prefer to run servers as snappy packages over debs at any time15:18
* teward comes from the world of old-fashioned "Start with absolutely basic, add what's necessary"15:19
ogra_yeah15:19
tewardogra_: indeed, but once I saw you state that it's the base of snappy packages, that answered the question15:19
tewardas i said, "oh that explains it :)"15:19
ogra_right15:19
* teward isn't well versed in Snappy yet15:19
slangasekrharper: so, followed up to the squid3 bug with more explanation.  I think you overlooked that trusty was upstart-only, there was no init script shipped in the package in trusty; /this/ is why the upgrade path behaves differently between trusty and wily15:27
rharperslangasek: ok;  I'm trying to figure out if this is something we introduced or are just incompatible with the postinst from debian;  it sounds like we need to do something different since debian didn't deal with upstart service scripts?15:28
slangasekrharper: you need to do what I outlined in the bug15:29
rharperslangasek: perfect, thanks15:29
naccslangasek: question on mythtv: currently, there are no conffile listed for mythweb; is that expected?15:44
tewardogra_: I think we need to have lxc defaults changed15:47
tewardogra_: lxcbr0 on a default install is given 10.0.3.1, but this conflicts in my own network - I have many maps in the 10.* IP space around, and I bet there will be IP range conflicts like this in other deployments15:48
teward(default install of Server)15:48
ogra_teward, file a bug :)15:48
tewardwould if LP would respond to me trying to reaoch it15:49
ogra_(i'm not in any way an lxc guy ... better talk to stgraber )15:49
cyphermoxteward: there might be conflicts in any possible range one might pick15:49
* teward blames Comcast15:49
tewardcyphermox: true, but then this brings up other questions: if it's likely to cause IP conflicts and routing conflicts, why is it in the standard images again?15:50
tewardi know ogra stated Snappy, but it brings up more questions then15:50
slangaseknacc: I have no idea; what do you see in mythweb that you think should be a conffile15:50
slangasek?15:50
tewards/then/than answers/15:50
naccslangasek: the one you wanted me to move, /etc/php5/apache2/conf.d/20-mythweb.ini15:50
cyphermoxthat isn't my call. I'm just saying that no matter what app, no matter what IP range you'll pick, you'll always run the risk of conflicts since it's all an arbitrary thing in a finite pool of resources15:51
slangaseknacc: guess I should look ;)15:52
naccslangasek: for reference, LP: #156218415:52
ubottuLaunchpad bug 1562184 in mythtv (Ubuntu) "Update to PHP7.0 dependencies" [Undecided,Incomplete] https://launchpad.net/bugs/156218415:52
tewardguess i'll bring up my concerns at the server team meeting then15:52
teward'cause that's where it's incorporated by default15:52
stgrabertwlxcbr0 is disappearing in a few days15:53
naccteward: --^15:53
stgraberwe're just waiting for juju to be done switching to lxdbr0 and then we can have lxd stop pulling lxcbr0 into the cloud and server images15:53
tewardah15:53
tewardstgraber: okay, that makes sense, because the ISO i tested last week with regards to the keyboard layout selection thing not working didn't have this. the iso I have from yesterday's batch (20160329) had it, and it actually broke net routing when the thing first came up (same subnet ranges due to overlap, so routing blew up)15:54
tewardI have a workaround temporarily in my installation, but eh15:55
stgraberhmm, lxcbr0 has been around by default in all images since wily (for server and cloud), so if you didn't have it before, that was a bug :)15:55
tewardstgraber: i never tested wily :P15:55
* teward shrugs15:55
slangaseknacc: $ dpkg-deb -R ubuntu/pool/multiverse/m/mythtv/mythweb_0.28.0+fixes.20160325.2520617-0ubuntu2_all.deb mythweb15:56
slangasek$ cat mythweb/DEBIAN/conffiles15:56
slangasek/etc/default/mythweb15:56
slangasek/etc/php5/apache2/conf.d/20-mythweb.ini15:56
slangasek$15:56
tewardin any case, i'm not a fan of routing blowign up in my face on a default setup15:56
tewardfortunately it's a VM that I can recreate, assign to different subnets, etc. but15:56
slangaseknacc: so, that looks like a conffile to me; not sure where you were looking, but DEBIAN/conffiles is authoritative (and with a modern package, is autopopulated by debhelper based on "is this file in /etc Y/N?")15:56
naccslangasek: weird, apt doesn't show it; but does in my rebuilt one (which was buggy, fixing still) that included dpkg-maintscript-helper15:57
slangaseknacc: I'm not aware of any option to apt to show conffiles15:58
naccslangasek: it does by default, or did for my package15:58
naccwhich is why i was confused :)15:58
slangasekwhen running what command?15:58
naccapt-cache show ...15:58
slangaseknacc: ok, not sure why that ever showed you conffiles, I've never known it to do so and it doesn't here :)15:59
naccslangasek: for refrence, http://paste.ubuntu.com/15560435/15:59
nacci wonder if it's something that dpkg-maintscript-helper does16:00
cyphermoxslangasek: when the package is installed?16:01
nacccyphermox: yeah, i had the older version installed, though, and it doesn't show any16:01
nacccyphermox: testing the upgrade path16:02
nacchence my confusion :)16:02
=== tvoss is now known as tvoss|dinner
cyphermox(that's my guess, I see conffiles for n-m, but only for the specific version installed, not others in cache)16:02
naccslangasek: cyphermox: well, either way, it's fine, i think i see what i did wrong with my change. But was curious16:04
=== kickinz1 is now known as kickinz1|eod
rharperslangasek: looking at 'removing etc/init.d/squid3 on upgrade';   IIUC, the new package scripts that get called are newpkg.preinst and newpkg.postinst;  the preinst script is called with 'upgrade' but postinst is called with 'configure' ; should I attempt to remove the old service script in preinst then (when $1 is upgrade)? or just test for existence in postinst and remove there during configure?   the ideal spot seem16:11
rharpers to be (previous version of squid's postrm upgrade call; which could remove old files) but that doesn't seem right since the current package doesn't do that;16:11
robrupitti: yes, train is trusty. let me know if you need me to backport dpkg-dev or something.16:29
infinityrharper: Or, use dpkg-maintscript-helper?16:30
slangasekrharper: you should call dpkg-maintscript-helper with the appropriate args, unconditionally16:34
slangasekcyphermox: my test case for this was 'apt-cache show base-files', which certainly should be installed ;)16:35
cyphermoxhrm, fun ;)16:35
cyphermoxah, maybe there's something there16:36
cyphermoxn-m show Status: install ok installed16:36
cyphermoxbase-files doesn't16:36
cyphermoxthat said, I don't think that's the cause of my upgrade bugs ;)16:37
slangasekcyphermox: right, which means somehow your apt-cache show is merging contents from /var/lib/dpkg/status (which is also where the Conffiles are), maybe that's what you're seeing for a package that's installed but is *not* available at that version in the cache?16:37
cyphermoxI see three versions of n-m in cache, one of which is the one installed16:38
naccit might be explainable, but IMO violates the principle of least surprise :)16:39
naccas the version in /var/lib/dpkg/status, here (of base-files) is the one installed16:39
naccand /var/lib/dpkg/status does list hte conffiles entries16:39
naccwhile apt-cache show does not16:40
naccdunno how it decides :/16:40
slangasekcyphermox: well, yes it's in the cache but I'm guessing 'apt-cache policy' shows it as locally installed and not available from any source16:40
cyphermoxah, maybe yeah16:41
slangaseknacc: I notice a few of the tasks on LP: #1544352 are stuck as fix committed; at least one of these, jquery-goodies, seems to be a case of the upload going missing.  Can you double-check these and set them back to a non-fix-committed state for me if I need to reupload?17:48
ubottuLaunchpad bug 1544352 in libfpdf-tpl-php (Ubuntu) "[PHP7] After bootstrapping, these PHP packages can be rebuilt" [Undecided,Fix committed] https://launchpad.net/bugs/154435217:48
naccslangasek: will do17:49
slangaseknacc: and sorry, on mythweb I guess I should have explained more about dpkg-maintscript-helper.  While this is the maintainer script interface, you should never put this in the maintscripts directly (because DRY); this ought to be in debian/mythweb.maintscript instead, which debhelper (via dh_installdeb(1)) will pick up and include17:51
slangaseknacc: do you want to redo, or should I go ahead and fix it up on my side?  the latter is probably faster overall17:51
naccslangasek: ah interesting; i was just going off `man dpkg-maintscript-helper` and a few other exampels in the archive already17:51
naccslangasek: should we modify that manpage? as it explicitly says to do what i did :)17:52
slangaseknacc: yeah, the 'SEE ALSO' at the bottom of that manpage is unfairly subtle17:52
naccheh17:52
slangasekand yes, +1 for better documentation17:52
naccslangasek: if you can fix it up, that's fine with me -- the change is logically the same to me17:52
slangasekI'm sure your maintainer script snippets are technically correct, they're just unmaintainable ;P17:52
naccright17:53
slangaseknacc: btw, mythtv is stuck in -proposed for un-php reasons (mysql-5.7).  maybe that's something that could use a bit of help?17:54
naccslangasek: i think it's a known thing that rbasak is working on17:54
naccbut let me check17:54
slangasekok17:54
naccyeah it's the mysql 5.7 transition stuff, which is blocked right now while they verify the upgrade path17:55
slangaseknacc: ah, also your patch is against the xenial version of mythtv, not the xenial-proposed... will fix17:56
naccslangasek: argh, sorry! i think it should be pretty similar17:57
rharperinfinity: slangasek: thanks!  should prior-version include the XubuntuY part of the previous versions or just the source version ?18:04
infinityrharper: The manpage has a whole section on how to select prior-version correctly.18:05
rharperhrm18:05
* rharper rereads18:05
infinityCOMMON PARAMETERS18:05
infinity       prior-version18:05
infinity(three paragraphs follow)18:05
infinityrharper: TLDR; you probably want it to be 3.5.12-1ubuntu4~18:07
infinityrharper: Except... Wait.  Which conffile are you removing?18:08
rharperinfinity: I've read those now, 3 times;  it's not clear to me yet;18:08
rharperinfinity: /etc/init.d/squid318:08
infinityrharper: Cause stgraber already did that in the last upload.18:08
infinityhttp://launchpadlibrarian.net/250372995/squid3_3.5.12-1ubuntu2_3.5.12-1ubuntu3.diff.gz18:08
rharperinfinity: ok, let me look at 1ubuntu3 then;  and see what else i need to do as slangasek help figure out for fixing upgrade paths18:10
sladenkirkland: kudos for the press coverage18:12
ricotzinfinity, hi, would you have a moment to sync a package? https://bugs.launchpad.net/ubuntu/+source/plank/+bug/156377818:17
ubottuLaunchpad bug 1563778 in plank (Ubuntu) "FFe: Sync plank 0.11.1-1 (universe) from Debian unstable (main)" [Undecided,New]18:17
infinityricotz: Iz done.18:20
ricotzinfinity, thank you!18:22
=== ben64 is now known as Guest83800
elbrusSkuggen: thanks (I don't think you are "working around the dbconfig issue", but rather "give the world some time to adjust to the new mysql behavior")18:45
dokoslangasek, your mythtv upload has a .orig left. do you want to re-upload?19:54
slangasekdoko: I certainly don't /want/ to ;)19:54
dokook, I'll do it19:54
mwhudsondoko: i know about docker.io/s390x, arguing with ibm about it20:06
Skuggenelbrus: Yeah, I agree that going straight from silently accepting the behavior to throwing errors is a bit sudden :)20:06
elbrusyup, I read your upstream bug report20:06
slangasekmwhudson: is xnox in the loop on said arguing? :)20:06
SkuggenAt least for options like port where an empty value will work20:07
mwhudsonslangasek: no, but i can add him i guess20:07
mwhudsonarguing is sort of the wrong word20:08
mwhudsoncommiserating20:08
mwhudson(the whole situation is just painful)20:08
slangasekheh20:08
mwhudsonhere is the pain: https://go-review.googlesource.com/#/c/20949/220:11
slangasekahaha fun20:12
* doko looks at updating lua ...20:13
slangasekdoko: because... ?20:20
infinityslangasek: I can haz verbal +1 on merging lintian so it stops whining about my packages living in the future?20:25
slangasekinfinity: is that about Standards-Version?20:25
infinityslangasek: Standards-Version and uscan/watch version, in my case.20:25
dokoinfinity, slangasek: so which of the NEW packages would be still appropriate? trying to look at the go uploads from December20:26
slangasekinfinity: weak verbal +1, would still want a queue review, though I guess lintian isn't release-critical20:26
infinityslangasek: Will stage in my staging PPA to make sure the testsuite doesn't completely blow up like it does on every single merge. :P20:26
slangasekdoko: per standing policy, any of them are "appropriate", some of them are higher priority than others.  I have dibs on tpm2-* though20:27
infinitydoko: The go ones are totally appropriate, if they pass review.20:27
infinityslangasek: Ooo.  How long until archive reorg? :)20:32
infinityslangasek: lintian could be a sync if it was done. :P20:32
* infinity will merge it with a lower-than-debian version so we can sync over when we get there.20:33
xnoxmwhudson, is that what makes docker.io FTBFS in xenial now? =)20:48
mwhudsonxnox: yes20:48
xnoxmwhudson, enjoy! i'm glad i don't support packages in universe. And slangasek will back me up on that =)20:49
* xnox goes back to hacking curtin, MAAS and all things nice20:49
infinitydoko: If you have a moment https://bugs.launchpad.net/ubuntu/+source/libdata-alias-perl/+bug/156408220:49
ubottuLaunchpad bug 1564082 in libdata-alias-perl (Ubuntu) "[MIR] libdata-alias-perl" [Undecided,New]20:49
cjwatsoninfinity: by not-very-much-coincidence I was just starting work on the LP changes to allow tweaking the ogre model20:50
infinitycjwatson: I assume we're all agreed on what that model is.20:50
cjwatsonAIUI it should allow supported to b-d on unsupported but free must still only b-d on free.20:51
infinitycjwatson: Bingo.  I figured we'd both reach that obvious conclusion without external influence anyway.20:51
cjwatsonso {main, universe} -> main universe, {restricted, multiverse} -> main restricted universe multiverse20:52
cjwatsonand it'll be a flag on distroseries so that we can quickly flip the behaviour back if it breaks20:52
infinityxnox: Oh.  I should have told you that I already tried -fno-pie -no-pie on s390x and it didn't help.20:52
cjwatson(do analyse as much as possible beforehand though ...)20:52
infinityxnox: (for glib2.0)20:53
xnoxinfinity, yeah, it doesn't look like pie issue. pie things don't hang usually, just misbehave in wtf ways.20:54
xnoxunless it's ftw ways, then you know it's an endian issue =)20:54
infinityxnox: PS, try not to use the archive as test-build infra. :P20:54
infinityOh.20:55
xnoxinfinity, i have enough test-build infra =) but yeah, retry build was unnecessary20:55
infinityI can't read.20:55
infinityThat was doko doing that.20:55
xnoxinfinity, unless we get a notes field and/or "disable retry button"20:55
infinitydoko: I already tried -fno-pie -no-pie for glib2.0, that's not the issue, sadly.20:55
dokoinfinity, done20:55
xnoxpeople will keep stabbing things20:55
pittirobru: the "auto-add Testsuite: field" does seem to cause some trouble indeed; I guess backport vs. fixing packages is between you and sil2100, but if it's not too hard to run this on xenial that might be easier20:56
infinitydoko: And thanks on the MIR.20:56
dokoxnox, infinity: I'm tempted to not run the tests on s390x, they succeed on a local machine20:56
dokobut having the glib2.0 for the test rebuild would be nice20:57
infinitydoko: They do?  They sure didn't work for me when I was testing...20:57
infinitydoko: Is your local machine up to date?20:57
xnoxdoko, what's your libc and kernel?20:57
dokoinfinity, at least with -no-pie20:57
xnoxis that devac02?20:57
mwhudsonxnox: yeah, i'll get this sorted one way or another20:57
dokoyes20:57
robrupitti: i do intend to update to xenial shortly after its release as we currently rely on many backports and I'd like to get rid of that. Can you file a bug explaining the issue in more detail?20:57
mwhudsonxnox: my main concern is getting the same thing into xenial as will go into 1.620:57
xnoxdoko, devac02 may have be running a hacked up kernel from apw20:57
mwhudsoner 1.7 i mean20:57
mwhudsonalthough even then we can cope20:58
robrupitti: not sure how urgent this is, if you can wait until after xenial is released or if we should fix it sooner.20:58
infinityxnox: If it works on that machine with an up-to-date glibc, that would be a good datapoint.20:58
infinityxnox: Then we can go kernel bisecting.20:58
xnox2.23-0ubuntu220:59
xnox4.4.0-16-generic20:59
* infinity raises his brow.20:59
infinityIs that the archive's -16, or something else?21:00
infinitycat /proc/version_signature21:00
xnoxUbuntu 4.4.0-16.32-generic 4.4.621:00
infinityorly.21:00
xnoxhm... i wonder what libc is inside the doko's chroot though21:01
infinityOkay, let me update a buildd to that and smack the retry button.21:01
infinityOh, check his chroot first. :P21:01
dokoxnox, all current from proposed21:01
infinityOkay, lemme doctor up a buildd with the new kernel.21:01
xnoxalso 2.23-0ubuntu221:01
xnoxinfinity, yeah that's a thought21:02
pittirobru: done, bug 156408421:03
ubottubug 1564084 in Bileto "Does not automatically add "Testsuite:" header any more" [Undecided,New] https://launchpad.net/bugs/156408421:03
pittiSaviq: ^21:03
robrupitti: thanks21:03
infinityxnox: Alright, building with new kernel.21:05
infinityxnox: OTOH, I see nothing in the -16 upload that would relate to this.21:08
infinityxnox: Can I get some SSH love to the machine where it works?21:09
dokotumbleweed, https://bugs.launchpad.net/ubuntu/+source/pypy/+bug/156408821:09
ubottuLaunchpad bug 1564088 in pypy (Ubuntu) "FFe: update to pypy 5" [Undecided,New]21:09
=== salem_ is now known as _salem
naccPharaoh_Atem: can you sync up with ondrej and remi and find out what is to be done with php-horde-mongo? php-mongo is deprecated with php7, i afaict21:16
tumbleweeddoko: I'm still deciding about another patch (slow test builds are slow)21:20
mwhudsonthe way packages disappear from the +source page while they're in the process of migrating from -proposed to -release is pretty odd21:22
infinitymwhudson: Hit the publishing history, it'll show it's pending publication.21:28
infinitymwhudson: But yes, that "pending" should be on the previous page, many of us agree.21:28
mwhudsoninfinity: yeah i've discovered that trick21:29
mwhudsonhowever i've already done my time in the launchpad slave pits, not going to try to fix this one :)21:29
infinityHeh.21:29
sarnoldbetter luck with the next new core-dev :)21:31
infinitysarnold: Better a core-dev unwilling to look at LP bugs than one who enjoys LP bugs so much that he defects.21:32
infinitycjwatson: Traitor.21:32
sarnoldlol21:32
dokomwhudson, you didn't yet hear about the archive slave pits ...21:35
mwhudsondoko: i am learning every day!21:35
slangasekxnox: ahhh I see, so you fix build failures on s390x in universe when it suits you, and then when it regresses and you don't want to deal with it, it's mwhudson's problem ;)21:38
mwhudsondocker broke because of a golang upload i did, i think you'll have a hard time blaming this one on xnox21:39
naccPharaoh_Atem: i think i need to use a pkg-php-tools-override and specify php-mongodb as the php7 compliant version21:40
naccPharaoh_Atem: actuall, nm, it's not clear that's going to work, as php-mongodb is quite a bit different21:51
naccslangasek: so depending on the above --^ (and i've also filed a ticket with upstream horde), php-horde-mongo is going to be uninstallable for a bit21:57
simon_ghi22:02
simon_gi'm not sure if it's a proper channel, but I got a rather surprising error while compilling kernel on the newest (beta) ubuntu22:03
naccsimon_g: depends, what kind of error? compiling the ubuntu kernel? or mainline?22:06
simon_ghi, nacc. http://wklej.org/id/2193203/ here is a link, kernel was a standard one from the kernel.org, with no extra patches or in fact any modifications that i'd think of (apart from compiling ext2/3/4 as a part of kernel, not as a module)22:07
simon_gi did sudo make-dpkg clean && fakeroot make-kpkg --initrd kernel_image -j1022:08
simon_git was a while since I was doing stuff on linux, but as far as i remember (and as far as the stuff i've found over the internet say) that was a valid way of compiling kernel (with .deb packages)22:09
simon_gi'll try it with the 4.5 kernel now22:10
sarnoldsimon_g: apt-get install libssl-dev22:10
Pharaoh_Atemnacc: php-mongodb is dead, afaik22:11
Pharaoh_Atemafaik, it was replaced with the new php-mongo22:11
Saviqpitti, thanks, about our previous conversation - I've now got gsettings-qt, qtmir and qtmir-gles stuck in proposed because of the leftover Xs-Testsuite header http://people.canonical.com/~ubuntu-archive/proposed-migration/xenial/update_excuses.html#gsettings-qt - how'd you say we deal with that?22:11
Pharaoh_Atemnacc: but I'll talk to Remi about the horde package, for sure22:11
Saviqpitti, already have an MP up for it https://code.launchpad.net/~saviq/qtmir/drop-testsuite-header/+merge/290468 but that won't unblock gsettings-qt anyway since it's trying to run the previous qtmir tests22:12
naccPharaoh_Atem: other way around22:12
simon_gsarnold, yeap, that's what i thought i needed. but why the heck should i need it?22:13
Pharaoh_Atemnacc: oh, was it?22:13
* Pharaoh_Atem gets them mixed up all the time :/22:13
naccPharaoh_Atem: yeah, 16.04 has no php-mongo, has php-mongodb22:13
simon_geven on the 4.5 with standard settings (nothing has been changed) i still get that error22:13
sarnoldsimon_g: you may be able to turn off something in the kernel configuration file to no longer build those tools22:13
simon_gthanks, sarnold, it's compiling.22:20
sarnold\o/22:21
cjwatsoninfinity: :-P22:27
infinitycjwatson: :)22:27
mwhudsoninfinity: so dh_golang is just looking at build-depends to generate built-using22:29
mwhudsonso i think it would be appropriate to do some special casing to report the golang-X.Y-go version too22:30
mwhudsonjust not 100% sure how to do that22:30
infinitymwhudson: Oh, really?  It's that brain-dead?22:33
infinitymwhudson: So, yeah, I'd special-case "golang-go" to readlink /usr/bin/go | dpkg -S | reverse-lookup.22:34
infinityWell, I guess it must already be doing the reverse lookup bit, unless it's REALLY brain-dead.22:34
infinitySince Built-Using is meant to reference sources, not binaries.22:35
mwhudsoninfinity: https://anonscm.debian.org/cgit/collab-maint/dh-golang.git/tree/script/dh_golang#n8822:35
infinitySo, just readlink | dpkg -S -> existing bits.22:35
mwhudsonis there Dpkg::foo for dpkg -S or should i just shell out?22:35
infinitylibdpkg-perl likely has the right bits in it somewhere.  I don't use it much.22:36
infinitymwhudson: Actually, maybe not.  libdpkg-perl is more about manipulating packages, not the package database.22:37
infinitymwhudson: So forking is probably the right way.22:37
mwhudsonok22:39
mwhudsoninfinity: http://paste.ubuntu.com/15563663/ seems to do the job, any ideas for proper testing?22:48
mwhudsoninfinity: http://paste.ubuntu.com/15563673/22:51
infinitymwhudson: Surely, that belongs in the if{}22:55
infinitymwhudson: (In case someone builds a package with dh-golang that doesn't actually use golang because derpy derp)22:56
infinitymwhudson: Also should only trip is "golang-go" is in the build-deps, I'd think.22:56
infinitys/is/if/22:56
mwhudsoninfinity: dh-golang invokes go xxx a bunch of times, if there's nothing there things are going to go bad23:05
mwhudsonbut hmm23:05
mwhudsonmaybe if golang-defaults turns up in the list of source packages23:06
infinitymwhudson: It does?23:12
infinitymwhudson: It only depends on perl, so that's surprising.23:13
infinitymwhudson: Wow, it really does.  So, in that case, dh-golang is broken for your new world order anyway. :/23:14
infinitymwhudson: Since it won't understand a package that build-deps on golang-1.6 and calls the compiler directly.23:14
infinitymwhudson: But the quick fix for now while we assume the world all uses defaults would be to just keep what you have but move it back into the if block, IMO.23:15
mwhudsoninfinity: http://paste.ubuntu.com/15563799/ ?23:15
infinitymwhudson: And conditionalise it on that dpkg-query actually returning true.23:16
mwhudsoninfinity: for that sort of thing, i've just been putting /usr/lib/go-1.6/bin on $PATH in rules23:16
mwhudsonwhich is possibly a bit dumb23:16
infinitymwhudson: Ahh, I like that iteration better (not your rules file, but dh-golang fix).23:17
mwhudsoni guess that's not strictly strictly correct, you could build-depend on golang-src in which case there might not be a /usr/bin/go at all23:17
mwhudsonbut there's only so much stupidity it's worth tolerating23:18
infinitymwhudson: The PATH hack could concievably be the "right" way to do it in the Go world, but you should maybe codify that.  Or even get dh-golang to help somehow.23:18
infinitymwhudson: Well, conditionalize the dpkg-query.23:18
infinitymwhudson: If they build-dep on golang-src and there's no /usr/bin/go, then that search will fail.23:19
infinitymwhudson: And perhaps eat stderr on that.  dpkg-query is noisy when it hates you. :P23:19
mwhudsoninfinity: what's perl for telling if the command succeeded?23:20
mwhudsoni think if i 2>/dev/null i'll endup appending just " " so it kinda almost "works" in that case23:21
mwhudsonoh $?23:22
infinitymwhudson: http://paste.ubuntu.com/15563848/23:24
infinitymwhudson: That's how I'd do it, likely.23:24
infinity(base)adconrad@nosferatu:~/golang/dh-golang-1.12$ perl foo.pl23:25
infinityfoo bar baz coreutils23:25
infinityHuh, though I'm getting an extra \n out of that.23:26
naccslangasek: fyi, the php-horde-spellchecker failure seems odd: "E: Failed to fetch http://ftpmaster.internal/ubuntu/pool/main/g/groff/groff-base_1.22.3-7_amd64.deb  Temporary failure resolving 'ftpmaster.internal'"23:27
mwhudsoninfinity: http://paste.ubuntu.com/15563863/23:28
mwhudsoninfinity: extra newlines matter <= 0 in this context i think :)23:28
naccslangasek: it built successfully for me with and without -proposed, so I think it might have been a trasnient issue23:31
infinitymwhudson: Possibly not, but easily fixed with a chomp too.23:32
infinityhttp://paste.ubuntu.com/15563889/23:32
mwhudsoninfinity: is a string that is just "\n" truthy in perl?23:32
infinitymwhudson: Strings should never be evaluated for truthiness.23:33
infinitymwhudson: Perl has different operators for strings and values.23:33
mwhudsonisn't that what the if statement is doing? or is perl being even odder than i expect?23:33
infinitymwhudson: eq/ne/etc are literal string comparisons.23:33
infinityWait, which if?23:33
cjwatsontruthiness of strings is dangerous to use in perl because "0" is false, for instance23:34
cjwatsonas it happens "\n" is true in a boolean context, but ...23:34
mwhudsonoh `` returns undef if the command fails23:34
infinityMy "if" is testing if the `` exited cleanly and, thus, set the variable.23:34
infinityThe later if is doing a string comparison with ''23:34
tsimonq2bug 1547395 is the cause of the FTBFS for libquvi, what needs to happen now? Desktop team ack? Just curious...23:34
ubottubug 1547395 in luasocket (Ubuntu) "MIR: new dependencies for libquvi-scripts / libquvi" [Undecided,New] https://launchpad.net/bugs/154739523:34
mwhudsonyeah ok23:34
cjwatsoninfinity: your if in http://paste.ubuntu.com/15563889/ is testing string truthiness though23:35
cjwatsonor at least may be23:35
infinitycjwatson: Shouldn't be.23:35
infinitycjwatson: Oh.  I see.  Yeah, it's not in the failure case, but might be in the success case.23:36
cjwatsonRight.23:36
infinityMy perl is rusty.23:36
cjwatsonI'd do "my $golang_go_dep = `...`; if ($? == 0) { ... }"23:36
infinityLuckily, a package name will probably always be "true", but ick.  Remind me how to express that properly?23:36
infinityFair enough.23:36
infinitymwhudson: ^-- Listen to the irishman.23:36
mwhudsonblarghl i had that version at one point :)23:37
infinitymwhudson: Plus chomp, cause chomp is never wrong.23:37
infinityFirst thing I learned about string manip in perl. :P23:37
cjwatsonor if you want to be fancy then I think it's possible to glue autodie into backtick handling23:37
mwhudsoni have no desire to be fancy at all23:37
infinityFanciness is overrated.23:37
mwhudsonbeyond what is necessary when in perl-mode23:38
cjwatsonbut that probably has rather more effects on the program as a whole23:38
mwhudsoni don't think it would be appropriate here23:38
* mwhudson rebuilds lxd for the 20th time23:38
infinitycjwatson: Wait.  Are you sure I'm testing the string?23:39
mwhudsonhttp://paste.ubuntu.com/15563916/23:39
infinitycjwatson: Replacing the `` with `echo 0` and `echo 1` seem to both succeed and add to the list.23:39
cjwatsontry s/echo/printf/g23:39
cjwatson"0" is false, "0\n" is true23:39
infinitycjwatson: Shouldn't the () around the expression be testing the return of the assignment rather than the content?23:39
cjwatsonthanks Larry23:39
infinityHahaha.23:40
infinityHah.23:40
infinityHah hah hah.23:40
infinitycjwatson: I concede.  And also, HAH.23:40
mwhudsonoh right, it's the shell's `` that strips a trailing newline if present23:40
cjwatsonthe return value of an assignment expression is the assigned thing23:40
cjwatsonbasically23:40
cjwatsonif I were writing perl stuff from scratch these days I'd probably start with Modern::Perl and autodie23:41
infinitycjwatson: So there's no way to express the shell shortcut of "if assign = expr", you have to go longhand with the return?  Oh well.23:41
cjwatsonand using largely python-style exception handling23:41
cjwatsoninfinity: well, you can, it's just that the assigned thing in this case isn't very boolean-friendly23:41
cjwatsonyou could say "if (length(my $foo = ...))"23:42
infinitymwhudson: That looks plausibly correct, given above.23:42
mwhudsonit also seems to work23:42
cjwatson(iirc)23:42
infinitycjwatson: Right, fair.  length() would be a cheat in cases where success also implies I set something non-null.  Which works here.23:43
cjwatsonyou can also write it as "chomp(my $golang_go_dep = ...)" I think23:43
infinitycjwatson: I guess I'm looking for the shell shorthand where I'm not actually testing the assignment at all, but the rv of the ``23:43
infinitycjwatson: Which is probably more of a bug in POSIX sh than a lack of feature in Perl.23:43
cjwatsoninfinity: well, but you are here, it's just that the test on the rv has different semantics23:43
cjwatsonit's still a test on the rv though23:43
infinitycjwatson: Yeah, you could chomp on either line, but chomping where he did keeps line length down. ;)23:44
mwhudsonchomp returns the number of characters chomped i think?23:44
infinityOh, it does.23:44
infinityYeah.23:44
infinityKeep the chomp where it is.23:44
infinityIt's lovely.23:44
infinityAnd chompy.23:44
infinityOn nom.23:44
infinitymwhudson: Ship it before we bikeshed more.23:44
infinityI need a shower.23:44
mwhudsonanyway i need lunch23:44
infinityAnd to run out.23:45
mwhudsonyeah ok23:45
infinityEvery time I do something Perly, I feel guilty about all the Perl I've forgotten.23:45
infinityHas the same experience futzing with sbuild a few days ago.23:45
infinitys/Has/Had/23:45
mwhudsoninfinity: [ubuntu/xenial-proposed] dh-golang 1.12ubuntu1 (Waiting for approval)23:46
infinitymwhudson: Danke.23:47
infinitymwhudson: Are you uploady for that in Debian too?  Should be fixed there as well if you've done the defaults thing.23:47
mwhudsoninfinity: no, and that hasn't happened yet23:47
mwhudsonhopefully soon23:47
mwhudsoni've sent a mail to debian-newmaint and then a whole lot of nothing has happen afaict23:47
infinitymwhudson: Also, that diff had another diff inside.23:48
infinitymwhudson: debdiff before upload. ;)23:48
mwhudsonoh right, native package23:48
mwhudsoninfinity: you'll reject?23:49
infinitymwhudson: Already have.23:49
infinityYou should have angry email.23:49
mwhudsonnope23:49
infinityWell, you will some day. :P23:49
mwhudsonbut i assume it'll arrive in due course23:49
sarnoldfunny that the "AOL guy" didn't go with "You've got angry email"23:49
infinityAnyhow, it's rejected.23:49
mwhudsonuploaded again23:51
tsimonq2cyphermox: you around? (sorry for the naked ping)23:52
tsimonq2ahh, looks like he's AFK23:53
infinitymwhudson: Ta.23:53
mwhudsoninfinity: ah got the rejection mail now, i like your explanation23:54
infinitymwhudson: Given that we kinda want to start tracking built-using properly, we might want to do no-change rebuilds of all golang things once this lands, but we can worry about that another time.23:54
infinitymwhudson: I might wait for the inevitable next lxd upload first, see how things look, then perhaps rebuild the world.23:55
mwhudsonok23:55
mwhudsonwell rebuild all things with binaries that are in main, at least :-)23:56
mwhudsonanyway, really need lunch now23:56
mwhudsonbiab23:56
naccslangasek: aiui, phpmyadmin is ok to leave as-is, as all the php5 dependencies are from alternatives?23:59

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