/srv/irclogs.ubuntu.com/2015/06/16/#ubuntu-devel.txt

=== salem_ is now known as _salem
slangasekinfinity: shim itself build-depends on gnu-efi, so if gnu-efi is missing on arm64, pesign isn't terribly relevant00:45
infinityslangasek: Well, unless you want to sign grub yourself.  But yes, that does put a kink in the shim->grub->kernel way we do things.00:50
slangasekoh right, we don't actually need shim on arm64 because no signing regime00:51
psusieh?  I thought that "windows certified" arm hardware *required* signing00:51
slangaseksure it does00:51
slangasekwith a key that Microsoft will not use to sign anything from third parties00:51
infinitypsusi: None of which is a target for us currently.00:52
=== mfisch is now known as Guest68660
pittiGood morning04:14
robert_ancellRAOF, have you ever removed a package from the archive? I want to clean out all the old vala packages. Wondering if there's anyone I should check with first05:01
RAOFrobert_ancell: You're not an archive admin, right? You *can't* remove a package from the archive :).05:01
pittirobert_ancell: there's two things to check: (1) all reverse dependencies must be gone, and (2) the package should also be removed in Debian, or we should blacklist it05:01
StevenKRemoving packages from the archive was one of my favourite things.05:01
pittithese checks can be done by anyone05:01
RAOFBut the way to do so is to file a bug against all the various packages and subscribe ubuntu-archive.05:01
pittiStevenK: ~ubuntu-cruft-busters! \o/05:02
RAOFAlso that.05:02
robert_ancellRAOF, ah, I was reading https://wiki.ubuntu.com/ArchiveAdministration and thinking I just needed core perms05:02
robert_ancellRAOF, pitti, OK I have the bugs and I've chased down the reverse-deps so I'll sub ~ubuntu-archive05:02
RAOFrobert_ancell: I'm pretty sure that needs LP privs. (Same as you can't accept things into foo-proposed).05:02
StevenKpitti: Represent!05:04
Unit193pitti: Upgraded to vivid just so I could get systemd user units (yes I'm aware upstart had them.) :P05:05
infinitypitti: Didn't that all start from the two of us sitting at a table in Montreal, whining about duplication in the archive?05:05
infinityAlthough, the cruft-busters team is much newer than our spec. :P05:05
pittiinfinity: I can easily whine about it on any table in the world05:06
infinitypitti: You're so versatile.05:06
infinitypitti: This must be why you have a fan club.05:06
robert_ancellmterry is the last thing blocking Vala 0.16 (via vala-dep-scanner which looks like it too should be removed from the archive)05:06
pittirobert_ancell: sorry, we can't possibly remove mterry from wily05:07
infinityWait, vala-0.16 is still in the archive?05:07
infinityIt's been removed from Debian since last year.05:07
robert_ancellinfinity, yes, I was surprised too05:08
pitti$ reverse-depends -b src:vala-0.1605:08
infinitypitti: If we remove him, he'll just MIR himself back in.05:08
robert_ancellBug 1465486, bug 1465487 and bug 1465488 for Vala 0.18, 0.20, 0.2405:08
ubottubug 1465486 in vala-0.18 (Ubuntu) "Remove from archive" [Medium,Triaged] https://launchpad.net/bugs/146548605:08
ubottubug 1465487 in vala-0.20 (Ubuntu) "Remove from archive" [Medium,Triaged] https://launchpad.net/bugs/146548705:08
ubottubug 1465488 in vala-0.24 (Ubuntu) "Remove from archive " [Medium,Triaged] https://launchpad.net/bugs/146548805:08
robert_ancellI figure we don't need six versions of Vala...05:09
infinityI suspect you're right.05:09
StevenKMaybe it's trying to make Python jealous05:09
infinityDoes vala-dep-scanner have any rdeps?05:09
pittivala-dep-scanner has zero reverse deps05:09
infinityJinx.05:09
pittiso maybe we shuld just drop that along?05:09
pittiit's never been in Debian either05:09
robert_ancellv-d-s looks like an experiment that went nowhere05:09
infinityYep, agreed.  Punt it, if someone likes it, they can resurrect it with a newer vala.05:10
* pitti sharpens the knife05:10
robert_ancellSee bug 1465484 and bug 118964205:10
ubottubug 1465484 in vala-0.16 (Ubuntu) "Remove from archive" [Medium,Triaged] https://launchpad.net/bugs/146548405:10
ubottubug 1189642 in vala-dep-scanner (Ubuntu) "Update vala-dep-scanner for vala-0.28 or remove from archive" [Medium,Triaged] https://launchpad.net/bugs/118964205:10
infinitypitti: Are you all over this?  I was just about to start the abuse, but you're welcome to have the honours.  I know how much you love deleting things. :P05:10
pittiinfinity: we can share the love -- we have four bugs, nicely splits in half :)05:11
* pitti is on 0.16 ATM05:11
infinitypitti: No, no.  Go ahead.  I'm trying to make Mirv happy.05:12
infinityWhich is, apparently, really difficult.05:12
pitti-16: history05:12
* robert_ancell gets the marshmallows05:13
pitti-18 is gone as well05:14
pittirobert_ancell: -20 has rdeps, I followed up to bug 146548705:14
ubottubug 1465487 in vala-0.20 (Ubuntu) "Remove from archive" [Medium,Triaged] https://launchpad.net/bugs/146548705:14
pittierr05:15
pittisorry, it seems UDD or so is out of date05:15
robert_ancellpitti, I've updated all those, not sure if they've flowed through yet05:15
pittiall gone05:17
pittiremoving 4 compilers on one day? it's too good!05:17
StevenKpitti: Up next, gcc?05:17
pittiand python 205:17
sarnoldand how many awks do we really need?05:18
robert_ancellThanks pitti!05:19
StevenKpitti: Python 2 isn't a compiler, why is it on that list? :-P05:19
StevenKpypy on the other hand ...05:19
robert_ancellI did a few packages to help move python2 off the image...05:19
robert_ancellbye all05:20
=== Malsasa_ is now known as Malsasa
LocutusOfBorg1good morning07:26
LocutusOfBorg1does anybody know why gdb-arm-none-eabi is built for debian but not for ubuntu?07:26
LocutusOfBorg1I mean, where the hell does the gdb source come from in debian buildds?07:27
cjwatsongdb-arm-none-eabi build-depends on gdb-source, which is a binary package built by the gdb source package.07:29
cjwatsonThe gdb-arm-none-eabi build failure in Ubuntu just looks like it's because the patches it carries are out of sync with Ubuntu's gdb.07:29
dholbachgood morning07:29
LocutusOfBorg1thanks cjwatson :)07:33
LocutusOfBorg1hi dholbach 107:34
dholbachhey LocutusOfBorg107:35
tjaaltonis wily "safe" to use yet? :)07:57
seb128tjaalton, no issue here or that I saw others report07:59
seb128so I would say "yes"07:59
tjaaltoncool07:59
tjaaltonwill switch my laptop to it then07:59
seb128:-)07:59
tjaalton@pilot in08:12
=== udevbot changed the topic of #ubuntu-devel to: Archive: wily open | Devel of Ubuntu (not support or app devel) | build failures -> http://qa.ubuntuwire.com/ftbfs/ | #ubuntu for support and discussion of lucid -> utopic | #ubuntu-app-devel for app development on Ubuntu http://wiki.ubuntu.com/UbuntuDevelopment | See #ubuntu-bugs for http://bit.ly/lv8soi | Patch Pilots: tjaalton
tjaaltonugh08:12
seb128bdmurray, hey, is that a known issue that e.u.c "channels" category stopped listing channels?08:31
seb128it only has "all channels" option08:31
seb128same for "rootfs" or "device image"08:31
zygahey08:45
* zyga filed a bug about those python packages that use PEP440 incompatible versions08:46
zygahttps://bugs.launchpad.net/ubuntu/+source/apturl/+bug/146554908:46
ubottuUbuntu bug 1465549 in PlainBox (Toolkit) "Plainbox tests are broken by PEP 440 incompatible Debian/Ubuntu packages" [Undecided,Confirmed]08:46
zygaI can fix all three easily but I don't have any rights to upload them08:46
zygawhat should I do? attach debdiffs to the bug?08:47
=== vrruiz_ is now known as rvr
zygajuliank: can the fixed package be uploaded to vivid-updates?09:13
juliankOh, I'm online.09:14
davmor2juliank: or are you?09:14
juliankzyga: Well, vivid needs the commit cherry-picked, that'd solve the issue. There are some other commits that should be cherry picked as well, though.09:16
juliankThe others are the same I'll propose for a stable update in Debian. They fix crashes on LFS .deb files, and a memory leak.09:17
zygajuliank: can you do that? I can only prepare debdiffs / patches but I have no upload rights09:17
juliankI have no upload rights either, but mvo has.09:17
* juliank should work on getting upload rights09:17
* zyga feels the same09:18
zygamvo: how can I get upload rigths for command-not-found?09:18
juliankApply for PPU09:19
mvojuliank, zyga: what needs sponsoring?09:19
mvozyga: yeah, what juliank said09:19
zygajuliank: thanks, I'll check what that is (personal package uploads?)09:19
juliankmvo: We'll need to cherry-pick some patches from python-apt 1.0 beta to the vivid version (and the one in wily, or just push the 1.0 beta there)09:20
zygamvo: I'd like to see a few PEP440 violations fixed, juliank was thinking about python-apt09:20
cjwatsonper-package upload permissions.  Ubuntu has a bad acronym habit.09:20
zygais there a spec for PPU, I see lots of wikis for people (that I know) that have applied09:21
juliankNot sure how urgent this is, we could try to get a python-apt 0.9.3.12 uploaded to Debian stable and then merge that release + the versioning fix into vivid-updates09:22
zygajuliank: one by istself is not urgent but those versions do break tests in our team so we'd like to fix that over time09:22
juliankLet me prepare the list of other patches for python-apt (aka create a branch), so we now what we're dealing with. (It'd be useless to do two uploads)09:24
=== jincreator1 is now known as jincreator
zygajuliank: thanks09:27
juliankmvo: I pushed three branches into the python-apt repository (they might still need some rebasing though). debian/jessie contains fixes for crashes and the memory leak, debian/jessie-pu fixes 2 multi-arch and 1 .dsc parsing issue (all small, but not sure whether we'll get that into Debian jessie). And finally, ubuntu/vivid, based on debian/jessie-pu, that also contains the setup.py versioning fix.09:44
juliankIf 0.9.3.12 would get all fixes from debian/jessie-pu, then the diff for vivid-updates would be http://paste.debian.net/231237/09:44
* juliank probably should also strip ~ubuntu... in setup.py, but that's not needed for stable updates anyway09:46
mvojuliank: neat!09:46
juliankThe LFS patches for tarfile do not work entirely correctly, because the size members are not actually long long yet, but they still catch too large allocations instead of crashing09:46
juliankOK, the version number should probably be different, not sure about Ubuntu stable release update versioning09:47
juliank(in the paste)09:48
infinity0.9.3.12ubuntu1 is sane versioning for an ubuntu update to a native package.09:48
juliankOK, great09:48
infinityShould be vivid instead of vivid-updates, but that's easy enough to fix.09:49
juliankOK09:49
juliankit was just to give an overview anyway, not final yet.09:49
juliankI'd use my email address for an Ubuntu update, anyway.09:50
julianks/email/Ubuntu email/09:50
rbasakIs there any easy way for me to detect in a maintainer script whether dpkg considers certain conffiles to have been modified?10:02
rbasakI know I could just scan for known md5s or something. Is there an easier way?10:03
juliankBack soon10:07
rbasakpitti: can you tell me how systemd integrates with dh_installinit please? If I use dh_installinit with --error-handler, will the error handler get called if it's actually a systemd unit that fails to start? I can't see how that would happen, nor where else I could put --error-handler.10:12
=== mvo__ is now known as mvo
pittirbasak: AFAIK there is no special integration -- this is simply expanded to invoke-rc.d .. || error_handler10:26
pittirbasak: I have never used it myself, but according to the manpage and the template you just define a myerror() { ... } in your postinst and then dh_installinit --error-handler=myerror; this will then generate invoke-rc.d myservice start || myerror10:27
rbasakpitti: I'm confused because the invoke-rc.d ... start call is wrapped in if [ -x "/etc/init.d/#SCRIPT#" ] || [ -e "/etc/init/#SCRIPT#.conf" ]10:29
rbasakpitti: so if there weren't init.d or upstart scripts, then it wouldn't get called? So I wondered if systemd services are started from elsewhere, but I couldn't see where.10:30
pittirbasak: ah, so you have a package which only ships a systemd unit, but no corresponding init script?10:30
rbasakpitti: in this case I think there is an init script. I was wondering about the safe thing to do in the general case.10:30
infinityrbasak: Given that Debian policy requires an init script, that shouldn't matter.10:30
rbasakOr if that meant I was missing something.10:30
rbasakAh, OK. Thanks.10:30
infinityrbasak: Though, that may not always be true.10:30
pittirbasak: the "main" service from a package should always be accompanied by an init script, by Debian policy10:30
pittirbasak: for some "auxilliary" units, that's why dh_systemd_start exists10:30
rbasakI will write an error handler then. Thank you!10:31
pittirbasak: e. g. for timers, sockets, or if the units are more fine-grained than the init script10:31
juliankmvo_: I asked for permission to update jessie's python-apt in https://bugs.debian.org/788928. After 0.9.3.12 gets accepted, we can merge it into vivid.10:54
ubottuDebian bug 788928 in release.debian.org "jessie-pu: package python-apt/0.9.3.12" [Normal,Open]10:54
juliank(I'll rebase the vivid branch in that case, and prepare the upload, you just need to sponsor it)10:55
juliankThis way, everyone should be happy :)10:56
* juliank really needs to work on https://wiki.ubuntu.com/JulianAndresKlode/DeveloperApplication-PPU2 so he can upload himself10:58
juliankBTW, IIRC, It's called DeveloperApplication-PPU2 instead of DeveloperApplication-PPU due to a wiki issue (I think I could not rename it)10:58
mvo_juliank: awesome, thanks a bunch11:01
juliankzyga: mvo_: I'd like to talk about command-not-found. Debian's still at 0.2.38, and Ubuntu has seen tons of ubuntu... releases on top of 0.3. A sane new upstream release would be appreciated, in case I need to update it. (I also have or had a much faster (but slightly limited) version in C somewhere I might push instead, and PackageKit has its own handler, although it's not installed yet)11:04
juliankI uploaded that thing 5 years ago, so it will probably be hard for me to merge the new release, as I have a bunch of changes in there.11:05
* juliank needs to find his bzr branch from 200911:05
zygajuliank: so there are a few options that we can explore11:06
zygajuliank: first of all, a reboot of the current ubuntu package, it has lots of issues and I wasn't a very effective upstream11:07
zygajuliank: but that was largely caused by my lack of effective way to release it to ubuntu11:07
zygajuliank: a C version would be appreciated (speed and ctrl+c support)11:08
juliankI need to see if I can find it and bring it up to the current standards.11:08
juliankOr just restart, it's not going to take longer than a week.11:09
tjaalton@pilot out11:09
=== udevbot changed the topic of #ubuntu-devel to: Archive: wily open | Devel of Ubuntu (not support or app devel) | build failures -> http://qa.ubuntuwire.com/ftbfs/ | #ubuntu for support and discussion of lucid -> utopic | #ubuntu-app-devel for app development on Ubuntu http://wiki.ubuntu.com/UbuntuDevelopment | See #ubuntu-bugs for http://bit.ly/lv8soi | Patch Pilots:
zygajuliank: I'm breaking for lunch, let's talk about this later11:11
juliankOK, I also need to do university work11:11
=== MacSlow is now known as MacSlow|lunch
=== mzanetti is now known as mzanetti|run
=== pgraner-dr is now known as pgraner
caribouDoes someone have a minute to help out with a merge (rsyslog) ?12:37
caribouI need to remove a dependency to liblogging-stdlog since it is a Universe package12:38
cariboubut the new version introduce a slew of unit tests where some of those tests rely on that package12:39
caribouour previous version did not implement unit tests at all12:39
cariboushould I just strip out each test that rely on this dependancy ?12:40
caribouI also have other FTBS caused by some of the tests race conditions (according to upstream's statements)12:41
rbasakI don't see liblogging-stdlog12:42
rbasakIs it new? Should it stay in universe or should it be pulled into main?12:42
caribourbasak: yes, it's new in the 8.0 stream12:43
rbasakAh it's liblogging-stdlog012:43
=== mzanetti|run is now known as mzanetti
caribourbasak: sorry, I cnp the --enable statement12:44
caribourbasak: maybe, I just sorta followed what had been done before for other dependancies not into main12:45
=== _salem is now known as salem_
rbasakI've read up on what liblogging is. Sounds like it's very much a client thing, so presumably rsyslog is using it to run tests only, and it doesn't form any part of the function of rsyslog itself.12:46
rbasakSo I guess it doesn't make sense to pull it into main unless it's necessary.12:46
juliankmvo_: Can you sponsor a python-apt sync for me (syncpackage -s juliank python-apt)? I'd like to see python-apt 1.0.0~beta2 in wily now, the fix from 0.9.4ubuntu1 is contained in it as well (well, I rewrote it, but achieved the same in the end). The earlier we sync, the better.12:46
caribourbasak: mmnormalize, kafka, rsyslog-mongodb are also dropped because of Universe dependancies12:46
mvo_sure12:46
rbasakcaribou: I'd say drop the tests from build time. If possible, you can add dep8 tests since those can depend on stuff in universe.12:47
caribourbasak: well, it is now enabled by default so I suppose it does use it somehow12:47
caribourbasak: all tests or only those specific to that lib ?12:47
rbasakcaribou: in that case I'd like to understand the consequences of not building with it, since that might have a bearing on whether we want it in main.12:47
caribourbasak: ok, I'll dig a bit deeper into this12:48
rbasakcaribou: preferably specific to that lib. If that's really awkward then maybe better to remove all and have dep8 do the full test instead? I'm not really sure about this though, would prefer a +1 from someone else.12:48
mvo_juliank: done12:49
mvo_juliank: lots of great fixes/features!12:49
juliankmvo_: Yeah, thanks. I still need to work out the new Files API for ~beta3 (or ~rc1, depending on how I feel)12:50
caribourbasak: here is a short discussion b/w the debian maintainer & upstream about what that library is meant to be : http://lists.adiscon.net/pipermail/rsyslog/2014-February/036448.html12:54
mvo_juliank: :)12:55
zygare12:57
=== anthonyf is now known as Guest89967
* zyga added another patch to https://bugs.launchpad.net/ubuntu/+source/apturl/+bug/146554912:58
ubottuUbuntu bug 1465549 in PlainBox (Toolkit) "Plainbox tests are broken by PEP 440 incompatible Debian/Ubuntu packages" [Undecided,Confirmed]12:58
zygajuliank: about command-not-found, I'd like to iterate on the python version12:58
zygajuliank: to fix some annoyances12:58
zygajuliank: and get the UI better12:58
zygajuliank: and then work on the C rewrite12:58
=== MacSlow|lunch is now known as MacSlow
zygajuliank: (c/rust/go/$shiny)12:58
zygajuliank: to improve performance12:58
zygajuliank: but I'd like to keep the python API as strangely enough, some things call it12:59
juliankzyga: The patch looks wrong, is c a valid Python alternative to ~rc?12:59
zygayes12:59
zygajuliank: it's part of PEP48612:59
zygajuliank: no, wait, wrong pep :)13:00
juliank44013:00
zygajuliank: a - alpha, b - beta, c - candidate (rc only for python itself)13:00
zygajuliank: (there's another pep, I think, that defines this, apart from 440)13:00
zygaone sec13:00
zyga38613:00
juliankPEP440 only defines a b and rc13:00
zygathose x86 names :P13:00
juliankOh no, it says Installation tools SHOULD interpret c versions as being equivalent to rc versions (that is, c1 indicates the same version as rc1 ).13:01
zygathat's more what I remember13:01
zygahttps://www.python.org/dev/peps/pep-0386/#id1813:01
zygarc and c are equivalent13:01
zygaand c is more recommended13:01
zygaAFAIR13:01
juliank386 is superseded, I would not refer to it.13:01
zygathanks, I need to read 440 then13:02
zygaI should patch python-versiontools to support this13:02
zygaoh, and I'm always happy to see packages accepted in debian :-) woot13:02
zygajuliank: do you think I should keep 'rc' as rc?13:03
zygajuliank: I can regenerate the pathc13:03
caribourbasak: reading more about it, looks like liblogging-stdlog will be more and more in use with rsyslog; might be a better choice to MIR the lib13:03
zygapatch13:03
juliankzyga: I (would) do that. The function I use for python-apt does it. It even maps ubuntu to +ubuntu, so that information is not lost either13:04
zygajuliank: perhaps I should move your function to python-versiontools and start patching all of those to just import it, right now that's just three packages (that I see) but if the same code needs to be copy-pasted around it doesn't look too good13:05
rbasakcaribou: sure, if you think that's appropriate. I can't find much in rsyslog 8.9.0-3 that would be lost if built without liblogging, but maybe in the future.13:06
caribourbasak: looks to me like rsyslog upstream is also the author of that library so I wouldn't be surprised if he was to make more usage of it. I'll ping him13:07
juliankzyga: Something like that might be a good idea (I suggested to move it to dh-python, but its maintainer was not that happy about the idea). The ~exp thing is a bit python-apt specific, though, but the rest is generic.13:07
juliankzyga: Also, normalized versioning now uses 1.0.alpha1 instead of 1.0a1 (with an added dot before the a), AFAICT13:09
juliankNot sure if a or alpha is standard13:10
juliankprobably a13:10
zygajuliank: .alpha1?13:10
juliankso 1.0.a113:10
zygaI think a13:10
zygayes, that's more like it13:10
juliankOops, I might be wrong13:10
zyga1.0a113:10
juliankIt's far too complicated13:10
zygait's not much different from pep386 -- I'm still reading 44013:10
zygait's more liberal from what I remember13:11
zygabut not incompatible with 38613:11
juliankThe dev releases have ".dev", but alpha and beta have "a" and "b" without a dot13:11
zyga440 adds different version types13:11
juliankin the normalized version13:11
juliankbut 1.0.a1 is accepted to13:11
julianktoo13:11
zygayes, .devN and .postN are different13:11
juliankjust like 1.1-a113:11
juliankIt's crazy13:11
zygawell, I think that -a1 should be different13:12
zygaat least if it's debian's -a113:12
zygaI would strip those out enitrely13:12
zygaas this fact (debian version) is almost never relevant13:12
zygain fact, in all the patches that was my intent13:12
zygathough those are debian native packages with weird non-pythonic conventions13:12
zyga(e.g broken setup.py that only works via debian/rules)13:12
juliankWell, in native packages we don't really have the issue with -. But I prefer keeping, for example, the Ubuntu part, and map it to a local label, by prepending a + to it13:13
zygayeah, it seems that local version is "debian version" in generic temrs13:13
zygaterms13:13
zygaSource distributions using a local version identifier SHOULD provide the python.integrator extension metadata (as defined in PEP 459 ).13:14
zygaI need to catch up on PEPs13:14
juliankSummary is good https://www.python.org/dev/peps/pep-0440/#summary-of-permitted-suffixes-and-relative-ordering13:15
zygajuliank: I agree it's complicated but it seems that the goal was to capture everything sensible that is done in reality and just standardize that13:15
juliankDebian version numbers are much more simple and capture more of the domain13:15
zygajuliank: I don't know, I'm too corrputed by exposure to both over years :-)13:16
zygajuliank: go ahead explain ~ : + and - vs _ to a novice ;)13:16
juliankI really need to work on university stuff now, I have to hand in things tomorrow.13:16
zygajuliank: good luck, thanks for the help!13:17
juliankNo problem. I'll now hangout with python-ldap and see how that works....13:18
zygajuliank: is that your university stuff?13:18
juliankPart of it13:18
juliankLearning ldap13:19
=== pete-woods1 is now known as pete-woods
juliankzyga: One thing I do today is for a lecture about distributed systems. Part of it this week is LDAP.13:20
zygajuliank: when does it start? ;-)13:20
juliankdistributed systems is a fun lecture, much like networks I had last year.13:20
* zyga makes silly distributed systems what-time-is-it jokes13:20
juliankzyga: Well, that's just the homework.13:21
juliankI also need to do university work work stuff sometime, though13:21
* juliank works 10 hours / week at university. Could use something else 10 hours a week instead ...13:22
zygajuliank: working at the university or remotely?13:22
* zyga likes his university days13:22
juliankzyga: I'm helping students with programming an Android app 4 hours a week at the university (aka sit around and wait for a question, last week I worked on python-apt during work time), and the rest I'm at home and don't have much work to do.13:23
juliankIt's a boring job :)13:24
juliankGives me some time to work on open source13:24
zygajuliank: perhaps, I cannot relate to that -- though I think some questions may be eye-opening :)13:24
julianks/open source/free software/13:24
juliankI'm happy that I can solve the LDAP exercise in Python (although only Python 2, unfortunately) and don't have to use Java.13:25
juliankOh, there's a python3-ldap3 package.13:26
juliankBecause two threes are better than one?13:26
zygajuliank: because apis and packages perhaps13:26
juliankAh entirely different software.13:27
* zyga updates his RTM phone to vivid, \o/13:27
bdmurrayseb128: The channels don't display if you are not logged in.13:46
kamal@pilot in13:59
=== udevbot changed the topic of #ubuntu-devel to: Archive: wily open | Devel of Ubuntu (not support or app devel) | build failures -> http://qa.ubuntuwire.com/ftbfs/ | #ubuntu for support and discussion of lucid -> utopic | #ubuntu-app-devel for app development on Ubuntu http://wiki.ubuntu.com/UbuntuDevelopment | See #ubuntu-bugs for http://bit.ly/lv8soi | Patch Pilots: kamal
LocutusOfBorg1mterry, seems that unity-scope-click is blocking libjsoncpp migration14:04
mterryLocutusOfBorg1, https://code.launchpad.net/~mterry/unity-scope-click/missing-iostream/+merge/26203514:04
LocutusOfBorg1holy sh*t you rock man14:05
LocutusOfBorg1I was downloading the source14:05
LocutusOfBorg1thanks!14:05
LocutusOfBorg1dholbach, do you mind syncing libserial from debian? :)14:05
mterryLocutusOfBorg1, :)14:06
LocutusOfBorg1or kamal if you are patch piloting ;)14:06
kamalLocutusOfBorg1, I'm on patch pilot duty, but I don't have upload rights (afaik!), so afraid I can't actually do so14:08
dholbachkamal, just for the kernel packages, right?14:08
kamaldholbach, correct (I only have upload rights for kernel)14:08
dholbachok... the review/pilot duties are then a special case as discussed with the kernel team ages ago :)14:09
kamaldholbach, yup, I'm just here because my calendar says I'm supposed to be here :-)14:10
dholbachLocutusOfBorg1, looking at it now14:11
caribourbasak: turns out that the one test is easily fixed with conditionals so I can still use it.14:15
caribourbasak: I sent an email upstream to know is future plans14:15
dholbachLocutusOfBorg1, done14:16
dholbachLocutusOfBorg1, I won't have time to look into other sponsoring items today though14:17
LocutusOfBorg1no problem dholbach, thanks!14:18
dholbachanytime14:18
=== Guest68660 is now known as mfisch
=== mfisch is now known as Guest81226
rbasakcaribou: great. Thanks!14:33
caribourbasak: wow, Rainer has already replied14:33
juliankmvo__: Oh I see, python-apt failed the autopkgtest because of deprecation warnings in the test you added for checking that the deprecated argument works :( - They're somehow shown by default during autopkgtest14:36
juliankPython3 has assertWarns, but Python2 does not14:37
juliankI'm fixing it14:38
=== davidcalle_ is now known as davidcalle
juliankmvo_: Fixed in debian/sid14:41
juliankIt now uses catch_warnings to catch the warning and record it, and then asserts that the use of the md5 parameter caused a DeprecationWarning14:42
juliank:)14:42
juliankMy editor misdetected the indentation , I'll fix that too14:44
juliankI can either push a ~beta3 to unstable and we resync, or we just push out an ~beta2ubuntu1 release for now14:49
juliankWe should make our testing more strict to also fail on stderr during package build, I believe the buildds showed the warning too14:50
juliankNo they did not. Strange.14:51
juliankThat is, we could use warnings.simplefilter('error') or something in test_all14:54
juliankOh well, an error filter does not allow me to use catch_warnings.14:55
juliankand it does not raise any exception if used in a catch_warnings block14:55
juliank...14:55
=== mfisch is now known as Guest25676
zygajuliank: for python2 you can use unittest214:58
zygajuliank: it has assertWarns and the zoo of new features14:58
juliankWell, I used catch_warnings manually. Better than adding a build-time dependency.14:58
zygajuliank: yep, it's typically something upstreams should adopt14:59
zygajuliank: for command not found in debian, I'm scared of building the hints database15:01
=== Guest25676 is now known as mfisch
zygajuliank: for ubuntu I did it manually on my machine after downloading 50GBs of the archive15:02
juliankzyga: Well in Debian, we download Contents files and build from that with a command in the package.15:02
zygajuliank: and I'd like that process to be separate from the upstream hacking on the tool itself15:02
zygakgunn: oh15:02
zygajuliank: oh15:02
zygajuliank: contents is insufficient15:02
juliankzyga: That's true, but it's all we get for now.15:02
zygajuliank: you need postinst scripts and metadata (symlink, mode) that is AFAIR not there15:02
juliankzyga: ftpmaster did not want the hints thing packaged15:02
zygajuliank: ah, ok15:02
zygajuliank: what?15:02
zygajuliank: why not15:02
juliankWell, I split it into its own package, which probably was the main factor (so we could have shared the main c-n-f package), but even in the same package, I'm not sure they would have liked it, because it gets outdated to quickly.15:03
zygajuliank: I somewhat agree with them15:04
zygajuliank: ideally, it would be a bit of meta-data that one can harvest easily15:04
juliankSo, only contents files for now, until we have the proper metadata in the archive.15:04
zygajuliank: and declarative15:04
juliankThat's WiP15:04
zygajuliank: not just random insanity like guessing from postinst scripts15:04
juliankzyga: Adding metadata is ximion's show15:05
zygaximion: tell me about it, I'm very interested in this15:05
ximionmetadata what? metadata where? ;-)15:05
* ximion reads backlog15:05
zygaximion: declarative list of executables per package15:06
LaneyX-Metadata-Lover: ximion15:06
zygaximion: and handling stuff like vim using alternatives15:06
zygaximion: and divertions15:06
zygaximion: so that common (but hard) cases are supported15:06
zygajuliank: one other thing that is problematic is knowing the right package to install15:06
zygajuliank: often a -bin or -common package is the one that has the executable15:06
zygajuliank: but the foo (not foo-bin) should be installed for good upgrades15:07
zygajuliank: I'd love to have this at the meta-data layer as well15:07
zygajuliank: and lastly, I want "gcc", there are 1234 versions of gcc avaialble and the hint should be smart there, the "smarts" should be declarative15:07
zyga(coffee)15:07
juliankWell, it should simply install the default gcc, obviously. Which is in the gcc package15:08
zygajuliank: or mabe pentium-builder15:08
zygajuliank: or maybe the package names are not a hint at all (gcc is a symlink or something like alternative)15:09
zygajuliank: obviously is not good, it has to be an algorithm :)15:09
zygajuliank: or if it's manual, it has to be a packaging-level declarative hint15:09
ximionzyga: about gcc I agree with juliank. About binary-fetching, that is in-scope for DEP-11/AppStream, and could be added to it at a later stage (we need to see how much overhead it would be to add this information to every metadata)15:09
zygaximion: I read dep 11 a while ago, how close is that to reality?15:10
juliankI do *not* want pentium-builder if I ask for gcc. The algorithm should take the command - package name distance into account when finding a package15:10
zygaximion: (I actually had some of that in one of my first packages but my sponsor asked to remove the meta-data)15:10
juliankdistance to package names is always a good idea.15:11
zygajuliank: that's google's business15:11
zygajuliank: it's hard problem15:11
zygajuliank: and gcc is the bad example as some packages are widely different15:11
ximionzyga: at Debian: very close, I am running the data generator there, and I hope I can soon switch it to automatic mode. The remaining bits are archive integration then (pull the data into the apt repo) and Apt integration, where all the groundwork has already been done by mvo's GSoC student a while ago, AFAIR15:11
zygajuliank: and command and package names will actually worsen the result (suggesting other packages)15:11
* zyga has gone that way 15:11
ximionat Ubuntu, I have no idea yet15:11
zygaximion: wait, I'm lost one one thing, can I use the new provides syntax today or not?15:12
juliankzyga: You just need to calculate the Levenshtein distances between the command and the package names and choose the smallest as the best option15:12
zygaximion: and do I understand that it's still needed to patch all the packages to declare this correctly15:12
zygajuliank: it's crap, I tried that15:12
zygajuliank: really15:12
zygajuliank: I spent a year just on this part of c-n-f15:12
zygajuliank: and it's utter crap in the end15:12
zygajuliank: you need way more context to produce hints that are not silly15:13
zygajuliank: and you want to still have fast lookup15:13
juliankWhat? *If you know* that the file is in a few packages, choosing the one with the closest distance is the best idea.15:13
zygajuliank: (though I'm sure there are structures that answer this quickly)15:14
zygajuliank: I see what you mean but the problem is that it's not good in general15:14
ximionzyga: no, it's not yet usable to the full extend. It's enough though to power GNOME-Software, if you do a little bit of manual work15:14
zygajuliank: gcc is just one example15:14
zygaximion: ok, so I can use it today?15:15
juliankIt may fail on some cases, but in most it should work just fine.15:15
zygaximion: and then this gets aggregated by some archive15:15
ximionzyga: no, a lot of data can be extracted automatically - but for really rich metadata, you need to drop a little bit of XML into /usr/share/appdata15:15
zygaximion: (cnf should also be offline, if it is online we can just ask google each time)15:15
juliankawk is a silly example, because both mawk and gawk have the same distance15:15
zygajuliank: I think this is highly subjective but the value is in _useful_ hints, not theory, IMHO non-declarative or weighted declarative approaches for that are not worth the trouble as the advice is just poor and the extra cost is large15:16
zygaximion: ok, for the trivial cases dep-11 is not an improvement15:16
juliankIt's extracted from packages and XML in packages, and put into a YAML (?) index that APT will download. You/It then parse the index and create a database15:16
juliankThe difficult cases need to be declared manually then15:17
ximionzyga: https://github.com/ximion/dep11 - careful though, the code has some very awkward bits which I intend to improve soon. Reason is its past as integral DAK component and GSoC work15:17
zygaximion: it is an improvement for cases like vim, that are actually quite complex and rely on alternatives15:17
zygaximion: and that naive harvesters cannot process15:17
zygaximion: the harvester in cnf is one level above naive15:17
zygaximion: so dep11 has to be better in practice if it can be adopted as a data source15:18
ximionzyga: for the immediate use, I'd stick to c-n-f - it does a reasonably good job :) (don't have enough backlog to fully understand the issues you have with it)15:18
* zyga looks at the link15:18
juliankzyga: Try taking popcon into account as well, that might help. Algorithms. Get a few drunk Google search people to help you.15:18
zygaximion: the issue is that 1) harvesting requires the full archive 2) it's a heruistic as many "interesting" packages use scripts to actually create executables and simple scans don't reveal those15:18
* juliank is not sure if you can drink and develop search algorithms15:18
zygajuliank: is popcon data available offline15:18
ximionzyga: the python-dep11 stuff just generates the metadata. To access it, you can use "appstream-index search <blah>", or whatever the tool hughsie wrote for appstream-glib is called15:19
juliankzyga: You'd obviously fetch it once.15:19
zygajuliank: I strongly oppose online searches as this basically turns cnf into a different product/service (which is not bad, just different)15:19
juliankand from time to time15:19
zygaximion: I see, so ...15:19
juliankthat is, when apt-get update runs15:19
zygaximion: so to get a replacement dataset for cnf15:19
zygaximion: I'd still have to do this locally and then regenerate a big -data package15:20
zygaximion: (well not big but non-trivial)15:20
zygaximion: and that would work for debian and ubuntu down the line later15:20
zygaximion: I'd preferrably have a python or ctypes python api though I can probably make that (gi should work)15:20
juliankWell no, APT get will download the DEP-11 metadata. You'd parse it in a post-download hook and generate a cache locally on the client (or use the appstream library)15:21
julianks/APT get/APT/15:21
zygaximion: do you accept pep-8 clenup patches?15:21
ximionzyga: I accept all reasonable patches15:21
ximion....and PEP-8 cleanup is highly welcomed!15:21
ximionneed to fire it at the code again soon anyway, after the dust has settled15:22
juliankWe need a newer pep8 version in the archives15:22
* ximion plans to rip out some really bad design issues and dak-isms to make it nicer to use by 3rd-parties outside Debian15:22
juliankzyga: barry: Any idea how to translate 1.0~ubuntu1 to PEP440 version numbers? Not really possible, right?15:23
juliank1.0 and the 1 just chosen as an example15:23
zygajuliank: hmm, perhaps to 1.0c115:23
zygajuliank: but yeah, .preN doesn't exist, I think15:23
zygaximion: as a good sanity check, make the package pip-installable on something totally non-debian15:24
juliankWhat I am saying is: I miss a local label for sort of pre-versions15:24
ximionzyga: and APT will later know about DEP-11 and will download it on-demand. We can then read the data directly, or use the cache which will be automatically updated (Xapian is fast as hell \o/)15:24
zygajuliank: I'd just ignore it TBH15:24
zygajuliank: most of the time it's not relevant15:24
* zyga hatex xapian 15:24
zygaslow as hell to update15:25
barry1.0rc1 i think15:25
juliankximion: Downloading code is in.15:25
zygabrb15:25
juliankin DonKult's branch15:25
juliankFor arbitrary indices15:25
juliankin an archive15:25
ximionzyga: then you haven't met the appstream-index tool ;-)15:26
ximionjuliank: I know, mvo told me :)15:26
* juliank does not like xapian either, because C++15:27
juliankwhich is not good for minimal solutions.15:27
juliankBut otherwise, it's good.15:27
barryat least there are rumors that xapian is now py3 compatible15:27
juliankbarry: Is Python 3 all you can think about?15:28
juliank;)15:28
zygaximion: I think that xapian should warrant the move to online queries15:28
zygaximion: but if you look at other OSes xapian is very bad in comparison :/15:28
zygabarry: yeah, I heard that too15:28
barryjuliank: what else is there in life?15:28
zygabut it's still slow and bloated15:29
juliankbarry: Music15:29
zygafor what it does15:29
barryjuliank: my bass is py3 compatible15:29
juliankzyga: Well, you could parse the indices yourself and generate your own database.15:29
juliankzyga: It'll be cool and awesome.15:30
zygajuliank: I still think that offline package generated by something like the repo itself (replace repo with something on the server side) is better15:31
ximionzyga: if you think of apt-xapian-index, it might be slow, but appstream-index beats most other tools in performance and memory usage (it's also 99% C code). Richard's appstream-glib uses the XML/YAML directly without cache, implementing a custom XML parser which does some pretty crazy things to be really, really fast15:31
zygajuliank: as downloadin a small update vs computing this over and over on memory-constrained devices (and slow devices) feels like the wrong tradeoff15:31
juliankThere's no large difference between a package an an index.15:31
juliankWith such an index, computation is not an issue15:32
zygaximion: I haven't played with a-i, I'll check it out15:32
ximionso, no matter how you access AppStream/DEP-11 data, it will not really take long ;-)15:32
zygaximion: cnf is slow15:32
zygaximion: I'd like to give hints in 30ms15:32
zygaximion: that would warrant rewrite to C15:32
zygaximion: (rust/go/$shiny)15:32
juliankcnf is slow. TheC version takes < 10 ms IIRC15:32
zygaximion: and proper databases15:32
zygajuliank: yeah, python start-up is just slow for that15:33
ximionyes indeed, C would be a much better fit for cnf15:33
zygajuliank: no doubt about that15:33
zygaI think one of the oldest versions of CNF were written in C15:33
zygabut I didn't publish that15:33
zygathen I rewrote it to vala15:33
juliankIt's more crazy on HDDs and uncached.15:33
zygabut one of the bits were missing a .vapi and I didn't know how to make the lacking parts15:33
zygathe last thing I did is a dbus service15:33
zygawhere it would spawn and sit around15:34
juliankOh god, no dbus15:34
zygathat was actually neat15:34
zygabut yeash15:34
zygaand then it was impossible to shutdown dbus services reliably15:34
zygaon idle15:34
juliankJust a simple hash db like now, and a C frontend.15:34
zyganow with systemd it is, but that was long ago15:34
zygajuliank: some things (spell checks) are cpu and db heavy15:34
zygajuliank: I think the current DB is not good for that15:34
zygajuliank: or the hash db has to have misspellings but that's crazy15:35
juliankWell, then you need either xapian or sqlite315:35
zygajuliank: a hybrid db would be better (hybrid == different DBs)15:35
juliankthose can match related words15:35
zygajuliank: I wasn't aware sqlite has that feature, do you have any hints?15:36
ximioncan KyotoCabinet match similar phrases?15:36
zyga(currently cnf just generates more queries015:36
juliankKyotoCabinet might be nice too15:37
zygajuliank: looking at the api it doesn't seem to15:37
zygajuliank: unless you want to just walk the keyspace and run conversions15:37
juliankWell, it's a B+ Tree15:38
zygayeah15:38
zygaI did a project a decade ago15:38
zygathat has some value15:38
zygabut it does require an offline processed database index15:38
zygathough15:38
zygaone small observation15:38
zygaeven if each debian package has 10 executables15:39
juliankI think sqlite3's full-text-search engine has a feature to match misspelled words, but I'm not sure15:39
zygathat is still _very little_ data to compute15:39
juliankBut I think it's overkill and the current version works well enough.15:39
zygaand I think just runing a very fast routine on _everything_ is faster than anything else fancy15:39
juliankPeople won't run command-not-found on phones15:39
zygaa list of just the keys (executable names)15:39
zygathat list is probably around a meg or two15:40
zygajuliank: try raspberry pi15:40
zygajuliank: slow CPU, slow IO15:40
juliankI'm basically happy with the state I have in Debian's 0.2.38.15:40
* zyga hasn't used cnf on debian yet15:40
juliankzyga: Why would you want to run stuff there in an interactive terminal?15:41
zygajuliank: I do all the time :)15:41
juliankYou should just deploy your finalized container there....15:41
* juliank had to mention containers once in his lifetime15:41
zygajuliank: I use it for tinkering and on-device hacking, different use case, it's not a server15:41
zygaheh :D15:41
juliankI think Vala is a good language, but it's a bit overly complex for something as simple as command-not-found.15:43
juliankNo need for object orientation and friends.15:43
juliankzyga: The hard-core version is to create a domain-specific library in C++ and then create C++ code from the archive and compile the entire database into the executable.15:44
juliankThat would be insanely fast.15:44
zygajuliank: with my love for C++ that's not likely, I'll probably use C for the fast version15:45
zygajuliank: for the database, not sure15:45
juliankWe don't want something exotic.15:45
juliankIt should be priority: standard or important or required15:46
zygajuliank: I agree15:46
zygajuliank: I think just running the exact same stuff in C will be great15:46
juliankIt will.15:47
juliankThere are not many alternatives anyway. You could switch to berkeley db, sqlite3 or xapian.15:47
jrwrenzyga: i remove xapian. :)  Who needs it?15:47
juliankNothing is going to improve a lot.15:47
zygaseparate problem, first let's revive the upstream code15:48
zygaand dedupe forks and whatever15:48
zygaso that there's one code to work on15:48
juliankzyga: BTW, The thing with the spelling corrections is: The DBs use stemming algorithms for it. That won't work well on packages.15:49
zygajuliank: ah, yeah, package names can be hostile15:49
juliankor well, command names.15:49
juliankzyga: Just take the 0.3ubuntu15.1 release and call it 0.3.1? Then move the data out of the package, so mvo or a bot can update it automatically15:51
zygajuliank: yep, that's a good first step15:51
juliankThat is, have a command-not-found source package and a command-not-found-data package.15:51
julianksource package.15:51
zygajuliank: oh, that is true today15:51
zygajuliank: but those are binary packages15:51
zygajuliank: I think what you are asking for are source packages15:51
zygajuliank: that would make sense15:52
juliankRight, yes15:52
juliankThen you have sane releases of code.15:52
zygajuliank: one thing I don't know how to do is where to keep the git tree, I'm used to svn based python workflow in debian15:52
zygaand I love git15:52
juliankAnd can just version the data in a sane way, such as 15.10.2015040315:52
zygabut I don't know how to get a collab maint git repo in debain15:52
zygadebian15:52
juliankzyga: Launchpad also has limited git support now.15:53
zygajuliank: I know that very well (using it)15:53
juliankzyga: But collab-maint is easy. You just need to join the group in alioth15:53
barryzyga: it's pretty easy15:53
barryzyga: well, alioth is15:53
zygagive me a sec15:53
juliankAnd then you ssh to alioth and run a command15:53
zygaok, I've sshd to alioth15:53
zyganow what?15:54
juliankhttps://wiki.debian.org/CollaborativeMaintenance15:54
juliankzyga: cd to /git15:54
barryzyga: if it's dpmt: https://wiki.debian.org/Python/GitPackaging15:55
juliankNo wait.15:55
juliankThere was a shell script somewhere15:55
juliankhttps://wiki.debian.org/Alioth/Git15:56
kamalzyga, juliank: https://wiki.debian.org/Alioth/Git15:56
kamalyeah.  that.  :-)15:56
zygadaughter emergency15:56
juliankzyga: Easiest way: Run gbp create-remote-repo from your source tree15:56
zygacrying because jurrasic world is not good for 6 year olds15:56
zygajuliank, barry, kamal: thanks, I'll give that all a try in a sec15:57
zygaas soon as I can :)15:57
juliankOh cool, my 0.2.38-1 package has proper patches.15:59
juliankzyga: My idea would be to have the code + update from Contents file thing upstream, and installing update-command-not-found on !ubuntu (depending on command-not-found-data in Ubuntu instead). This way, we can have one command-not-found package synced between Debian and Ubuntu.16:04
juliankNot even have to merge it.16:04
juliankUbuntu just provides the additional command-not-found-data source and binary package and everyone is happy.16:05
juliank(Maybe also ship update-command-not-found in Ubuntu for third-party repositories)16:05
=== salem_ is now known as _salem
zygajuliank: that sounds good to me (still not here yet)17:37
=== _salem is now known as salem_
=== salem_ is now known as _salem
mhall119cjwatson: pitti: mdeslaur: and any other dual Ubuntu/Debian developers, I've been asked for somebody from the Ubuntu side to comment on https://lists.debian.org/debian-derivatives/2015/06/msg00021.html about how we manage the same process (porting patches back to Debian)18:58
mdeslaurmhall119: I use submittodebian, I don't have anything further to contribute to that discussion beyond that18:59
mhall119mdeslaur: is that an ubuntu-specific tool?19:00
mdeslaurmhall119: yeah, it's in the ubuntu-dev-tools package19:01
mhall119mdeslaur: you might just mention that it exists and what it does, maybe it's something rasbian can use19:01
* mdeslaur googles "rasbian"19:02
mhall119raspbian actually19:02
* mdeslaur googles "raspberry pi"19:02
mhall119it's Debian for Raspberry pi19:02
mhall119what rock have you been under?19:03
dobeya securely reinforced one, i hope19:04
mdeslaurmhall119: I'm old. When I was in school, our boards had 6809's on them.19:05
mhall119when I was in school, our boards were black and required chalk :-P19:05
mdeslaurhehe19:06
zygamhall119: ah, the best kind19:06
mhall119zyga: meh, the I/O throughput left a lot to be desired19:06
zygamhall119: I'll take that over whiteboards any day19:06
sarnoldthat "new chalk smell" tho19:06
=== _salem is now known as salem_
mhall119some classrooms even had dual-board technology: http://aarco.com/05B.gif19:08
sarnoldbanked memory! sweet!19:08
mdeslaurwow, fancy :)19:08
mdeslauryou went to one of those expensive schools by the looks of it :)19:09
mhall119only problem was that whenever you tried to delete something, you invariable ended up with data all over your hands19:10
jrwreni/o throughput on rpi isn't that great either ;)19:11
mhall119jrwren: so I've heard :)19:12
=== salem_ is now known as _salem
=== _salem is now known as salem_
=== anthonyf is now known as Guest41199
=== athairus is now known as afkthairus
kamal@pilot out20:39
=== udevbot changed the topic of #ubuntu-devel to: Archive: wily open | Devel of Ubuntu (not support or app devel) | build failures -> http://qa.ubuntuwire.com/ftbfs/ | #ubuntu for support and discussion of lucid -> utopic | #ubuntu-app-devel for app development on Ubuntu http://wiki.ubuntu.com/UbuntuDevelopment | See #ubuntu-bugs for http://bit.ly/lv8soi | Patch Pilots:
=== afkthairus is now known as athairus
=== tvoss is now known as tvoss|test
=== tvoss|test is now known as tvoss
juliankGuys, anyone still here (barry?? I'm not sure what's happening: Our Python extension is producing a DeprecationWarning. I want to catch that in the unittests (did not check previously). Now, on the build daemons, the warning is not issued (those the new assert fails), but on autopkgtests it is (previously failed because warning was printed)20:59
juliankWhat's the best way out this? Pass -Wall to our test script in both enviroments?20:59
juliankI mean, I'm not entirely sure why the DeprecationWarning is issued on autopkgtest, but not on the build servers. That does not make sense to me21:01
* juliank just got the report from the daily PPA build, that's why he's asking now21:02
infinityjuliank: The warning is so issues on the build daemons.21:08
juliankso issues?21:08
infinityjuliank: The only difference here is that you can spew whatever you want to stderr in a build log and we don't care, but autopkgtest's default is to fail on stderr.21:08
ogra_some ?21:08
ogra_:)21:08
infinitys/issues/issued/21:08
infinity/build/buildd/python-apt-1.0.0~beta2/tests/test_paths.py:46: DeprecationWarning: Using the md5 keyword is deprecated, please use 'hash' instead21:09
infinity  md5="abcdef")21:09
infinityI see that in your build log.21:09
infinityThe same warning that's killing adt.21:09
juliankinfinity: No, it does not show at all. Python disables it by default, but the autopkgtest server seems to enable it.21:09
juliankinfinity: Well, strange. On the PPA, the build failed, because no warning was issues: https://launchpadlibrarian.net/209249914/buildlog_ubuntu-vivid-amd64.python-apt_1.0.0~beta2%2B866~ubuntu15.04.1_BUILDING.txt.gz21:09
infinityjuliank: I'm staring at a build log right now with the warning in it...21:09
juliankThis assertion checks that a warning was issued.21:09
infinityhttps://launchpadlibrarian.net/209219612/buildlog_ubuntu-wily-amd64.python-apt_1.0.0~beta2_BUILDING.txt.gz21:09
juliankRight. That was the old version. We did not catch the warning there. We catch now, and the PPA build fails.21:10
infinityjuliank: Okay, so this isn't about a difference between a buildd and adt, this is about your code changing. :)21:11
infinityjuliank: And catching warning implies not printing them, I suspect.21:11
infinityJust like any other caught exception.21:12
juliankYes, that's the point. I'm not sure why it cannot catch the warning (I assume it does not produce two warnings, I check for len(warnings) == 1)21:12
juliankif it was issued previously, it should be now21:12
juliankAnyway, I think I'll just add -Wall everywhere and it should work21:12
infinityjuliank: Okay, maybe I'm misunderstanding.  I assumed you were talking about the production adt, which was printing the same warning as the production buildd on the same version.21:12
juliankOr does someone have a better idea?21:12
infinityjuliank: And the PPA build, of course, is catching and not printing the warning, but asserting instead, which seems to be what you asked it to do.21:13
juliankinfinity: The autopkgtest on 1.0.0~beta2 failed because it wrote the warning. Now I'm trying to fix that. I catched the warning and assert that it's there. But during the build, it suddenly is not there anymore according to the PPA builder (or 2 warnings were issued inside the block, which seems unlikely).21:13
juliankI checked that precisely one warning is issued and that it has the type deprecationwarning21:14
infinityjuliank: Okay, but this has nothing to do with how buildds are calling/building it, as can be seen by the version in the archive whose output matches the adt run.21:14
juliankBut how do those buildds and adt call it to enable the warning? Maybe there's a difference to PPA builders (Debian buildds do not show the warning either)21:15
infinityjuliank: They do nothing special.  It's just dpkg-buildpackage21:15
juliankBut it's strange. On 1.0.0~beta2, the same version. The Ubuntu buildds show a DeprecationWarning and the Debian ones don't21:16
juliankLet me check how previous PPA builds went21:16
juliankThe old PPA build issued the warning too21:17
infinityjuliank: Debian buildds appear to skip your testsuite entirely because sources.list isn't readable.21:17
infinityjuliank: So, not a useful datapoint.21:17
juliankTrue21:18
juliankBut still, that commit http://anonscm.debian.org/cgit/apt/python-apt.git/commit/?id=12b0e06cb6db76e60455d13269e6ff3fedab5812 should catch precisely the warning and check for it.21:18
juliankIs there any danger in passing -Wall to python?21:19
infinityI'm not a python guy, don't know.21:20
infinityjuliank: So, if I'm reading https://docs.python.org/2/library/warnings.html right, the warning filter ignores DeprecationWarning by default.21:23
juliankYeah, but it somehow shows up on the buildds and on autopkgtest21:23
juliankThat's the reason I'm wondering21:23
infinityjuliank: Not a python guy, but I assume they mean the fancy filter, ie: the one you just added to catch your warnings.21:24
infinityMaybe not, though.  I dunno.21:24
juliankIf I run it locally, it works with -Wall (the warning is issued and catched (previous behavior as it was on buildd)). Without -Wall, the warning is not issued and not caught21:25
juliankBut now, it is not caught on the buildds, despite being printed in previous builds.21:26
juliankWhich is crazy21:26
infinityI would have assumed that's a byproduct of you importing the class and mucking with it.21:26
infinityjuliank: 28.6.4 seems relevant on that page.21:28
infinityjuliank: I think you just need a 'warnings.simplefilter("always")' before you trip your warning.21:28
infinityjuliank: But that exact code example is exactly what you want to do, so a bit of copy and paste should do it.21:28
infinityhttps://docs.python.org/2/library/warnings.html#testing-warnings21:29
juliankAh right21:29
juliankMissed that bit21:29
juliankThanks, infinity21:29
juliankFixed in git. Started an import in launchpad, and will then trigger a PPA build again.21:38
juliankIf it works, I'' do a beta3 tomorrow21:39
robert_ancellHow to I get valac-0.28 into main? Since nothing explicitly depends on it is there some override that brings it in like valac-0.26?22:06
juliankOh yes, it build!22:13
infinityrobert_ancell: You don't.22:31
infinityrobert_ancell: Not until the default vala version (defined by who builds the valac binary) changes.22:31
robert_ancellinfinity, that already appears to have changed22:31
robert_ancellinfinity, I'm getting build failures e.g. https://launchpadlibrarian.net/209188618/buildlog_ubuntu-wily-amd64.zeitgeist_0.9.14-2.2ubuntu5_BUILDING.txt.gz22:32
infinityrobert_ancell: Oh, so it did, apt-cache and I were having a small argument there.22:32
infinityrobert_ancell: So, the answer is "ask".  I'll do it now.22:32
robert_ancellinfinity, thanks!22:32
infinityrobert_ancell: Fixed after ~30m of publisher churn.22:33
=== anthonyf is now known as Guest21341

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