/srv/irclogs.ubuntu.com/2018/02/15/#ubuntu-devel.txt

xeviousnacc: phpdox 0.10.1 introduced namespaced PHPUnit classes. The latest tagged version is 0.11.0.00:03
xeviousI've got a loose definition of calling it a day.00:04
xeviousThere's a 0.10.1 package in sid: https://packages.debian.org/source/sid/phpdox00:05
naccxevious: i've uploaded 0.11 already00:05
naccxevious: it's in bionnic-proopsed00:05
xevioushah00:05
naccxevious: they are wedged (i just checked, thanks for the poke) on php-phpdocumentor-reflection00:06
nacclooking at it now00:06
naccxevious: looks like it needs an update too00:08
naccxevious: heh, 1.1.0 packaged, 3.0.0 released upstream00:09
naccsometimes Debian00:09
* nacc shakes fist00:09
xeviousI wonder how strongly people would object to letting PHP project .deb packages contain a vendor folder that's created by running `composer install` during `debuild`.00:11
naccwell, i mean we use composer00:11
naccthe problem is the composer.json in 1.1.0 is so solld00:12
nacc8old00:12
naccbah, old00:12
xeviousI mean instead of handling dependencies via dpkg dependencies.00:12
naccxevious: so you mean shipping all your dependencies in the deb?00:12
nacci think that would be expressly rejected00:12
xeviousI think so, too.00:12
nacci think debian/ubuntu would rather drop the php packages00:13
naccnow that cmoposer exists :)00:13
xeviousJust having packages for the PHP interpreter and extensions would be a-ok with me.00:13
naccyeah, that seems lilke a 'better' future00:14
naccbut i'm not sure we can do that for 18.0400:14
naccmaybe 20.04 :)00:14
xeviousYeah, that's definitely too much for 18.04.00:14
naccand reallly, i'm hoping debian will pick up some of this 'slack'00:15
xeviousDebian's PHP ecosystem is heavily tied to PECL/PEAR, which hardly anyone in the PHP community is using any more.00:15
nacce.g., start dropping horde00:15
xeviousYeah, Horde's got to go.00:16
xeviousI hope there's some activity on this front soon: https://wiki.php.net/rfc/deprecate-pear-include-composer00:17
naccxevious: yeah that would make sense to me00:18
naccso much of PECL is cruft00:18
xeviousI agree with deprecating PEAR in that RFC, but I'm not sure about outright including Composer in the PHP core. I think it's doing a good job, but don't want to reject the possibility of something better coming along.00:20
naccyeah, i'm not sure why it needs to be 'in' the core00:20
naccbut i'm not reallly a php user or developer00:20
naccjust stuck in this swamp :)00:20
xeviousMy job is all over the place. A pretty even mix of PHP, Python, JS, C, C++, C#. Sprinkle in a bit of Bash, Ruby, Assembly, ASL/DSL, and packaging for almost all platforms.00:27
xeviousI think you're spot-on with calling it a swamp. Getting rid of the PHP applications and focusing on the interpreter and extensions would make it much easier to maintain.00:29
xeviousIt'd be interesting to monitor how often the PHP application packages are actually used.00:30
cjwatsonThere are probably some hot spots, like Icinga00:30
cjwatsonI imagine popcon output in Debian would be helpful00:30
nacccjwatson: yeah that's a good point00:32
xeviousSo, that brings me back around to my first suggestion... Most modern PHP applications manage their dependencies with Composer: have their packages put their source in /usr/lib/packagename (with a complete ./vendor hierarchy) and create /usr/bin symlinks for any `bin` entries in the composer.json.00:35
xeviousIt'd lead to duplicate files, but it would allow different PHP applications to depend on different versions of a library.00:36
naccxevious: yeah, basically what snaps do for applications00:36
xeviousExactly00:36
xeviousIt'd be a small amount of bloat (bytes are cheap these days) in exchange for drastically simplifying the packaging process for PHP on Debian/Ubuntu.00:38
xeviousnacc: Have you heard back about mcrypt?00:46
naccxevious: not yet00:46
xeviousIf PHP upstream is abandoning it, it seems silly to go through the effort of packaging the PECL version.00:47
xeviousIs there a proper venue for polling Ubuntu Server users about things such as "Can we remove Horde? Does anyone actually care?"00:49
blahdeblahhorde == burn it with fire :-)00:49
sarnoldmcrypt deserves the same inferno, right?00:52
naccxevious: i'd start with ubuntu-server@lists.ubuntu00:56
Unit193sarnold: Except, I thought I needed that for something, but for the life of me can't remember what!00:57
xevioussarnold: Yeah. Projects still using mcrypt need to update and get with the times.00:59
=== BrAsS_mOnKeY is now known as william
=== sergiusens_ is now known as sergiusens
cpaelzerhmm - all x86 builders seem to be locked in "cleaning"09:12
cpaelzeris there another maintenance change going on or do the just need some help to get out of that state?09:13
cjwatsonprobably just the latter.  poking09:16
cpaelzerI see them turn idle and picking up jobs again, thanks cjwatson09:18
seb128could somebody here trigger an Ubuntu/bionic iso build?10:02
seb128the daily one built a bit before some seed changes we want to try were in place10:02
Laneyok10:04
seb128Laney, thx!10:05
tseliotginggs: I have just uploaded 390.25-0ubuntu1~ppa1.4, and the upgrading issues should be gone. Please test it, if you can10:45
ahasenackhi, I need a component-mismatch fix for http://people.canonical.com/~ubuntu-archive/proposed-migration/update_excuses.html#sssd11:07
ahasenackpython-sss is in main, that's the py2 version11:07
ahasenackpython3-sss is in universe11:07
ahasenackthese python3?-sss packages are produced by the sssd source itself11:07
tarzeauhow to get software mentioned in "Software" ?13:29
* tarzeau would like to suggest something for "Audio Creation & Editing": klystrack, protracker, schismtracker, milkytracker13:30
tarzeauand how come, in Games, "Mr Boom" has a "No screenshot provided"?13:31
tarzeauand the "Fonts" section is half-screen empty...13:32
tarzeauEducation & Science, Astronomy: is missing saods913:33
julianktarzeau: saods9 does not provide any appstream data13:34
julianksame for everything else you mention probably13:35
tarzeauhow, and who is one supposed to add appstream data?13:35
jbichahttps://www.freedesktop.org/software/appstream/docs/13:35
jbichavery few fonts provide AppStream metadata13:35
tarzeauis that in the package? or external?13:35
juliankSummary: AppStream data is an XML file to be provided by upstream to be shipped in packages.13:36
* tarzeau reads up on given url13:36
juliankin /usr/share/metainfo/%{id}.metainfo.xml,13:36
juliankwhere %{id} is a unique identifier13:36
sladenthought it used to also fetch from  http://screenshots.debian.net/  or is that out of date13:36
jbichatarzeau: also https://wiki.debian.org/AppStream/Guidelines13:36
tarzeausladen: mentioned things have screenshots.d.n13:36
juliankfor example, /usr/share/metainfo/org.gnome.Calendar.appdata.xml13:36
tarzeauthanks for the links, /me will try to improve13:36
juliankscreenshots have to be provided alongside the appstream metadata13:37
juliankunless someone patched something into gnome-software to do something with screenshots13:37
tarzeaujbicha: does debian use the appstream meta data, anywhere?13:37
jbichayou can add AppStream metadata downstream in the packaging but forward upstream if you can (I did this for Cantarell)13:37
julianksure13:37
jbichasome fonts don't really have an active upstream though13:38
juliankand yes, Debian uses AppStream13:38
tarzeauwhere?13:38
julianktarzeau: All the stuff in http://ftp.de.debian.org/debian/dists/unstable/main/dep11/ comes from it13:38
jbichatarzeau: Debian GNOME installs the GNOME Software app by default. That app only works with AppStream13:38
jbichathere is also the Discover app for KDE users13:38
tarzeauoh, i've never used the gnome software app on debian (or any kind of gnome software at all, actually)13:38
tarzeaujbicha: never heard of that, but will try once i visit a kde user13:39
jbichawhat desktop do you prefer?13:39
tarzeaujbicha: amiwm, or wmaker+gnustep software13:39
tarzeaunot exactly a desktop, but with gworkspace you have a nice file manager13:40
jbichaoldschool :)13:40
tarzeaubut fast! even with remote x on slow lines13:40
tarzeauthe few hundred users @work use mate, unity, gnome mainly. and used to use kde but it got far less than a few years ago13:41
tarzeausome use tiling wms, ratpoison, fvwm2 still, but very rare13:41
jbichaif you like terminals, there's appstreamcli I guess13:41
tarzeauand enlightenment13:41
tarzeaudesktop linux users are so rare13:42
juliankGNOME, GNOME, GNOME is the only thing I use13:43
juliankthough nobody really knows why13:43
juliankI mean, I basically don't use much except the shell and the terminal13:43
tarzeaui've had headaches when i saw the menus of gedit lately13:43
tarzeauthe hamburger menu, and menu placements etc13:44
juliankgedit looks awesome13:44
tarzeaui prefer any editor in terminal over it13:44
juliankbut since it does not support mixed space/tab indentation, it's a toy13:44
juliankjust like vs code, atom, gnome-builder, and their friends13:45
juliankor well, a bad joke, rather13:46
juliankhence I use geany13:46
juliankUnfortunately it does not have a header bar with client side decoration yet, but a menu bar and a tool bar13:47
juliankwith colored toolbar icons13:47
juliankterrible13:47
juliankapp has menu => throw away13:48
tarzeauwhat a pity must be in package. :(13:52
tarzeauand its creation is like work people already have done (like debian/* copyright, homepage etc)13:52
tarzeauthere's no debian2appstream script or something?13:52
* tarzeau found appstream-generator13:56
jbichatarzeau: ideally, the appstream metadata is upstream (it is for official GNOME apps)14:11
jbichasee also https://appstream.debian.org/sid/main/ or http://appstream.ubuntu.com/14:12
ginggstseliot: thanks, i'll try downgrading + upgrading - no problems with previous version and cuda 9.114:14
tseliotginggs: great, thanks14:19
tarzeaujbicha: i see. but the "Software" store, should not be GNOME only, well the gtk version, or the maybe qt version discover for kde (which I didn't have a look at yet)14:24
tarzeaubut right, anyone can provide such appstreams... preferably upstream, i got that14:24
jbichatarzeau: GNOME Software's .desktop doesn't set OnlyShowIn/NotShowIn; you can use it even if you don't use GNOME Shell14:26
julianktarzeau: appstream is not a gnome thing - it's a cross distribution development effort that was started by distributions, for distributions, over half a decade ago.14:26
juliankMostly by Fedora, SUSE, and Debian, IIRC14:27
jbichaLocutusOfBorg: you're going to have a problem with tracker's autopkgtests - upstream decided to add failing tests in the 2.0.x series after 2.0.1 :(14:28
juliankthere also are more than those two store apps14:29
roaksoax /win 314:46
xeviousIt'd be convenient if the first line of the autopkgtest logs was the date & time that the job started.15:21
LocutusOfBorgjbicha, I know, maybe we can disable such tests?15:32
LocutusOfBorgI'm wondering what is the best thing to do here15:32
LocutusOfBorgnew tests, failed probably even before... maybe a versioned hint is good15:32
LocutusOfBorgsince they fail in Debian too, and in fedora too15:32
LocutusOfBorg(I spent some time over the fedora/upstream bugs today)15:33
jbichathe tracker tests worked fine until upstream added broken tests15:37
jbichaI complained at tracker, but maybe it would help if someone else complained15:37
LocutusOfBorgjbicha, I know, maybe we can disable such tests?15:40
LocutusOfBorgI'm wondering what is the best thing to do here, new tests, failed probably even before... maybe a versioned hint is good15:40
LocutusOfBorgsince they fail in Debian too, and in fedora too (I spent some time over the fedora/upstream bugs today)15:40
LocutusOfBorgalso freeipa is a sad thing15:40
jbichawhat do you mean "failed probably even before"? the bad tests were added in 2.0.215:40
LocutusOfBorgjbicha, I mean, they arent related to the transition, and probably the problem is also in release15:43
jbichait is not related to the libunistring transition, but bionic only has 2.0.1 so you still have the problem of newly introduced bad tests15:47
LocutusOfBorgso, relaxing the tests is fine I would say15:53
jbichathat would make the tracker autopkgtest useless, right?15:58
LocutusOfBorgyep :/16:01
jbichado you want to open a tracker bug upstream to complain about the troubles they are causing us?16:01
juliankFailing tests should be optionalllllllllllll16:08
juliankWell, it should say ignored failure or something, everything else is just bad16:08
xeviousnacc: https://gist.github.com/iammattcoleman/88013cb5f92105b15a66ee2ada442a16#file-2018-02-15_101023_php-defaults_issues-md16:29
naccxevious: thanks16:30
nacci think mcrypt has more than that16:30
naccbased up on reverse-depends php-mcrypt16:30
xeviousI just based that on the autopkgtest logs.16:31
naccah ok16:31
naccyeah16:31
xeviousThe ones with "Declaration of DOMNodeComparator::assertEquals() must be compatible with ObjectComparator::assertEquals()" were almost all run on PHPUnit 5.4.6. Can you retrigger those with PHPUnit 6?16:32
xeviousMaybe all, actually.16:32
naccxevious: yeah i should be able to16:32
naccsorry, 'm digging into the horde stack right now16:32
naccand am on a call16:32
xeviousSure I understand.16:32
naccbut yeah i can use that16:33
* Laney has a race condition with juliank's race condition post :-)16:50
juliankLaney ...16:51
juliankI feel like I really should go build apt update --strict16:51
LaneyI wrote a draft saying "this is a race condition"16:51
Laneyand then I saved it to test the patch I'm attaching16:51
Laneyand there your post is :P16:51
juliankLaney: I wrote all this on IRC yesterday already, should have posted it to the ML then, but ML required some setup16:52
juliankI receive emails to ubuntu.com on one account, but can only send them from another :D16:53
juliankSo I re-subscribed with my canonical.com account, so I can send emails from there too16:53
juliankLaney: I think it's best I introduce Acquire::TransientErrorsImportant, and then scripts can pass Acquire::TransientErrorsImportant=true or something16:56
juliankor a counter16:56
Laneyit already has a bad exit status no?16:57
juliankIt exits 016:57
juliankWell the update exits 016:57
juliankthe install eatmydata exits 1 I guess16:57
juliankOne shows "W:", the other "E:"16:57
juliankyes. for the same error type16:57
Laneymmm16:59
juliankLaney: There's a wait+retry in the script so that would have caught it17:00
juliankupdate || { sleep 10: update}17:00
juliankbasically17:00
juliankum, ";", not ":"17:00
Laneyyeah we have those in a few places17:01
Laneyit's annoying having to sprinkle it everywhere17:01
juliankWell and it also does not work.17:01
juliankProbably because DonKult fixed a few bugs17:01
juliankand now more errors are transient than before or something17:02
juliankWell, any error from connect() is transient IIRC17:02
juliankand after connect some HTTP errors are transient17:03
juliankand transient errors only cause warnings, as they are "temporary"17:03
juliank:D17:03
juliankI should (1) add an option to control stuff (2) SRU it everywhere17:04
xeviousnacc: Would you like me to work on 'Class 'PHPUnit_Framework_TestCase' not found'? Can you start the tests for php-net-ldap2 2.2.0-3ubuntu1, php-parser 3.1.3-1, php-text-captcha 1.0.2-4ubuntu1, and phpdox 0.11.0-0ubuntu1?17:05
naccxevious: php-net-ldap2 and php-text-captcha retriggered17:07
naccxevious: php-parser and phpdox i'm trying to unstick so that they can migrate through then it's easier to retrigger17:07
naccxevious: that would be fine17:09
naccxevious: php-mockery needs an update from upstream but that in turn is failnig here, i'm debugging that too17:09
xeviousOk. Is there a place I can see which packages are stuck like php-parse and phpdox? phpdox is listed as 'Valid candidate' on the 'excuses...' page.17:11
naccxevious: http://people.canonical.com/~ubuntu-archive/proposed-migration/update_output.txt17:12
naccxevious: if it's valid and not migrating that page will explain why17:12
xeviousGreat. Thanks.17:13
naccin this case because of php-phpdocumentor-reflection17:13
naccwhich i believe depends on a specific rev of php-parser17:13
naccxevious: oh that's why i was down that rathole yesterday17:15
naccphp-phpdocumenter-reflection needs an updated php-mockery17:15
naccwhich in turn fails to pass its tests by default :)17:15
ginggstseliot: downgrading was somewhat messy, but after cleaning up i enabled your ppa and everything upgraded smoothly :)17:17
tseliotginggs: that's good. Thanks for testing17:18
ginggsyw!17:18
naccxevious: ok, i see how to fix the mockery failure17:23
naccxevious: just need to do it correctly :)17:23
xevious'Correctly' seems like a solid plan.17:26
naccxevious: ah is see mockery added a helpers file that needs to be sourced before running the tests17:27
* nacc tries with an updated bootstrap.php17:27
naccwoo i think it is passing17:29
naccthat should unblock a few more (and fix the regression with php-defaults -> php-mockery)17:30
naccxevious: also php-crypt-chap has been removed from bionic17:31
naccxevious: see LP: #174974517:31
ubottuLaunchpad bug 1749745 in gosa (Ubuntu) "php7.2 has removed the mcrypt module" [Undecided,Incomplete] https://launchpad.net/bugs/174974517:31
xeviousWasn't that a dependency of one of the Horde packages?17:31
naccxevious: reverse-recommends only17:32
tarzeaujuliank: ah okay, completely missed it.17:33
xeviousYay, no more mcrypt!17:34
tarzeaujbicha: so it's only the debian/fonts-cantarell.metainfo.xml that is doing the appstream thing?17:36
jbichatarzeau: yes, but that was upstreamed in Cantarell 0.100 (in Debian experimental)17:37
tarzeaujbicha: i'm always in contact with my packaged software, upstream (where possible)17:37
tarzeauso my plan would be, add it where it's useful, and provide it upstream for inclusion in future releases17:38
jbichait's a bit tricky to test appstream metadata. If you upload a package that installs it, it will show up in Debian or Ubuntu's GNOME Software about a day later17:38
tarzeaujbicha: that's great :)17:38
tarzeaui'll try it for fonts first, then games, then multimedia apps, later scientific software17:39
naccslangasek: i've got a package at 0.9.5 -- that then went to 1.0. There is also a 1.0.0~alpha1. uscan seems the 1.0.0-alpha1 and wants to use that and not the 1.0. Is it appropriate to mangle the uversion to suffix a .0 to two-digit versions so they sort properly?17:39
nacc(upstream versions for the above)17:39
* tarzeau votes for epoch!17:39
nacctarzeau: wrt to my question? not sure how that would help? I guess it would force 1:1.0 to sort after 0.9.5 ?17:42
nacctarzeau: aiui, i don't think an epoch bump is neeeded here, 1.0 should sort after 0.9.5 (per the manual)17:50
naccthe problem is uscan doesn't compare them properly, i think17:51
naccwith my manual mangle, it sees 1.0 as 1.0.0 and correctly updates to ti17:51
nacc*it17:51
tarzeauah17:55
nacctarzeau: that's my understanding right now, at least...17:56
naccxevious: and that pulls in a new php-hamcrest :)18:11
naccxevious: uploding php-hamcrest18:16
naccxevious: waiting on a response to the above before i upload php-mockery18:16
xeviousphp-mail-mime's log is unclear about what the actual errors are. It just says which tests failed, not why.18:20
naccxevious: let me look18:21
naccxevious: yeh that's becuse they are using pear not phpunit directly :/18:22
naccit *might* need the php-pear from proposed18:22
naccxevious: i can retry it locallyw ith propsed18:22
naccxevious: do you have autopkgtest setup locally?18:23
xeviousI don't. I was just going to ask how I can get my local machine configured so I can actually start helping with the packages.18:24
naccxevious: 1) have lxd setup locally18:25
naccxevious: 2) install autopkgtest18:25
naccxevious: 3) autopkgtest-build-lxd ubuntu-daily:bionic18:26
naccxevious: 4) autopkgtest -U -s php-mail-mime -- autopkgtest-virt-lxd autopkgtest/ubuntu/bionic/amd6418:27
naccxevious: if you want to test with proposed, you puass --apt-pocket=proposed18:28
naccxevious: you can also pass it a soruce package dsc rather than the name, fi you have local stuff18:28
xeviouseasy-peasy18:28
xeviousThanks18:28
naccxevious: you can also specify specific packages in the apt-pocket line, iirc, which is what LP is doing18:29
naccand what changing the triggers changes18:29
naccinfinity: would you have any opinion on the d/watch change i mentioned above? (I can repost it if that's easier)18:30
naccxevious: php-mail-mime also fails with proposed enabled18:33
naccxevious: debugging it18:33
xeviousnacc: I'm running 16.04 on this machine and didn't get an autopkgtest-build-lxd, autopkgtest, or autopkgtest-virt-lxd commands from installing the autopkgtest package. It installed adt-build-lxd, adt-run, and adt-virt-lxd, but adapting your command 4) didn't work.18:55
xeviousadt-run -U -s php-mail-mime -- adt-virt-lxd autopkgtest/ubuntu/bionic/amd6418:56
naccxevious: s/autopkgtest/adt/ in what i wrote18:56
naccit just got renamed18:56
xeviousThere isn't a command just named adt and adt-run is complaining about those parameters.18:57
xeviousnacc: adt-run: error: adt/ubuntu/bionic/amd64: unsupported action argument18:59
xeviousThat was the output when I ran: adt-run -U -s php-mail-mime -- adt-virt-lxd adt/ubuntu/bionic/amd6418:59
naccxevious: when you ran the build-lxd, it succeeded, right?19:13
naccxevious: if so, then check then anme (`lxc image list`)19:14
naccxevious: oh and adt might need three dashes19:14
naccnot two19:14
xeviousYes it succeeded. The image is named adt/ubuntu/bionic/amd6419:14
xeviousTrying three dashes19:15
xeviousThat did it19:15
naccxevious: sorry, i forgot about that19:17
xeviousSeems like my container can't lookup hostnames. Can I pass it nameserver info or just tell it to use the host system's /etc/resolv.conf?19:19
naccxevious: have you used lxd before on your system?19:22
naccxevious: it might need soe network config if not19:22
slangaseknacc: yes, no problem with version mangling to 1.0.0.19:45
naccslangasek: ok, thanks19:49
xeviousnacc: php-mail-mime passed on my system.19:50
xevious(without proposed enabled)19:50
naccxevious: hrm, but with php-defaults from proposed? that's what is broken19:50
xeviousRerunning it19:51
naccxevious: thanks, i definitely saw lall the failure siwth all proposed (which is my usual first test)19:51
naccas it's easier to type then figuring out the exact packages to test (locally)19:52
xeviousYeah, I just added '--apt-pocket=proposed' for this run.19:52
naccxevious: yep19:53
xeviousFor the projects that need to be updated for namespaced PHPUnit classes, most of them have already fixed that upstream. Can we just update to a new upstream version, or do we have to patch the packaged version?19:54
naccxevious: depends? :)19:54
naccif uscan can find it, and uupdate works, and the package builds and tests successfully, go for it19:54
naccbut sometimes, that ends up making its own nest of dependency migrations19:55
xeviousYeah, gotta watch out for backwards incompatible changes, for sure.19:55
xeviousIf all the PHP library dependencies were packaged with the PHP applications, then several versions could happily coexist on a system.19:57
xeviousI really need to post about that on the ubuntu-server mailing list.19:57
xeviousnacc: Running the test with proposed enabled reproduced the log from the excuses... page.19:58
naccxevious: yep19:58
xeviousnacc: Now that I've got autopkgtest/adt set up on my system, how do I use it to test local changes to a package?19:59
naccxevious: you know how to build source pacakges?19:59
xeviousYes20:00
naccxevious: so e.g., `pull-lp-source  ... ` etc20:00
naccmake changes, etc. dpkg-buildpackage -S -nc -d20:00
naccpass the resulting dsc to adt20:00
nacc(after modifying the changelog)20:00
xeviousI'm very comfortable with everything but the LaunchPad aspect of it. I've either worked with repos from internal source control or just modifying packages downloaded with `apt-get source`.20:01
naccack20:02
naccso don't worry about lp for now :)20:02
nacc(pull-lp-source is a way to do `apt-get source` without having to have sources defined)20:02
naccand to pull down source from any release20:02
naccyou can file bugs against the srcpkg, and attach debdiffs (`debdiff existing.dsc new.dsc`) and i can sponsor them for you20:02
xeviousThat sounds relevant, since I've got LXC set up on a 16.04.3 system right now.20:02
xeviousI'll read up on pull-lp-source and dig up my LP credentials.20:03
xeviousI'm fairly certain my GPG keys are long gone. :\20:03
naccxevious: pull-lp-source doesn't require auth, fwiw20:03
naccjust the bug filing will20:03
xeviousRight.20:03
xeviousI'm going to go find some food to munch on.20:05
xeviousbbiaf20:06
naccxevious: sounds good, i'm uploading php-mockery now and i filed a bug to remove php-phpdocumentor-reflection20:07
naccxevious: php-mail-mime is using count() in Mail/mimePart.php line 31420:12
naccxevious: changing it to be if (is_array($this->subparts) && .. ) and it passes20:14
naccxevious: i'll upload that fix20:14
naccxevious: oh i think it's fixed in 1.10.2 upstream20:17
nacci'll update to that20:17
nacc(safer in this case)20:17
naccxevious: ah it's in debian building right now (1.10.2-0.120:51
xeviousnacc: php-doctrine-cache 1.7.1 should resolve the issues. Can you rerun its tests with the new package?20:51
xeviousWoops20:52
xeviousI forgot to refresh.20:52
xeviousnacc: Are you still focused on Horde?21:10
infinitynacc: Your debian/watch woes can be solved with a rewrite of -alpha to ~alpha (and similar for -beta) if you know that's how upstream names things.21:53
xeviouswatch woes?21:53
infinitynacc: It's not that uscan is sorting "incorrectly", just that it uses a dpkg version sort, and upstream doesn't.21:53
xeviousI'm looking at php-guzzlehttp-promises and the archive it's pulling down doesn't include the tests folder.21:54
xevious...just because they chose to upload a tarball without the tests folder for that release, apparently.21:55
xeviousUsing the GH API tarball URL also produces an archive without the 'tests' directory. Anyone have any idea what's going on here?22:00
naccinfinity: it does that already (afaict)22:00
naccinfinity: but ack on the 'incorrectly'22:01
naccxevious: i can look22:01
xeviousnacc: Is there a setting on GH that makes it not include a 'tests' directory in archives?22:01
naccxevious: not that i know of, but possibly it's controlled by a file in the repo22:02
xeviousIt sure is22:03
xevious.gitattributes22:03
naccxevious: yeah22:03
naccxevious: does that imply guzzlehttp-promises also fails without -proposed?22:03
infinitynacc: Oh!  I focussed on the ~ and missed the 1.0 versus 1.0.0.  Derp.22:03
naccinfinity: yeah, it's that bit22:03
xeviousnacc: I was trying to just update it from 1.1.0 to 1.3.1, since they've already switched to namespaced PHPUnit classes.22:04
infinitynacc: Yeah, you could either mangle to add another .0 or (which seems more sane) add a version ignore for the 1.0.0~* ones.22:04
xevious1.3.0 should work, though.22:04
naccinfinity: ack, thanks22:04
naccsigh, libc6 in bionic is < libc6 in artful-security?22:04
naccthat seems less than ideal?22:05
nacc2.26-0ubuntu2 vs 2.26-0ubuntu2.122:05
naccand does that imply a security fix raced with the bionic open?22:05
naccxevious: you might compare the current package's results on artful22:08
naccxevious: so the version in the archive now has a tests/ dir22:09
xeviousEverything prior to 1.3.1 did, because there wasn't a .gitattributes file in the repo preventing tests from being exported.22:10
naccxevious: oh i see22:10
naccxevious: yeah, so you cn bump to just 1.3.0 if you want22:10
xeviousIs it reasonable to submit a "Fix Debian/Ubuntu packaging" PR that removes 'tests' from .gitattributes to the upstream repo, or should I adapt the package to upstream's changes?22:11
xeviousI could change the get-orig-source target in rules to use `git archive` instead of uscan.22:12
naccxevious: you could do that, but then you'd also need to change it to git cloen first22:12
naccthat's not great22:12
naccas you'd need to make git a build-depends22:12
xeviousA 1 line PR to upstream to include tests in the archive is the simpler solution from the packaging perspective. However, I'm hesitant to make Debian/Ubuntu-specific requests in an upstream project.22:14
naccxevious: i've done it before :)22:14
naccxevious: do they explain upstream why they changed it?22:14
xeviousTo "improve packaging" https://github.com/guzzle/promises/commit/e278912eaaa94079b9277d5dbe23fc0a22d308bf22:15
naccxevious: phpdox just migrated22:16
xeviousIt wouldn't need a 'git clone' in the build process. Updating the get-orig-source target to use `git archive` would still produce a tarball.22:16
naccxevious: how? it's run from debian/rules?22:16
naccxevious: where youdon't have a git repo22:17
xeviousWell that plan's out the window22:19
xeviousGitHub doesn't allow archiving.22:20
naccxevious: so 1.3.0 didn't work?22:20
xeviousnacc: You can pass `--remote=` to `git acrhive` to create a tarball of a remote repository.22:20
naccxevious: oh interesting22:20
naccxevious: i didn't realize that, sorry22:20
xeviousIt would still require adding git as a build dependency, which isn't great.22:21
naccyeah22:21
xeviousI'll just open a PR to make the tests directory export again. We'll see what they say.22:21
naccxevious: fwiw, i think php-horde-core is more than just a retry -- you can see that some did run against the version in b-p and still failed22:23
naccxevious: so you're still working on guzzlehttp-promises? mockery should migrate in a bit and i'll retrigger it then22:28
xeviousI'm just going to open a PR with the change and move onto the next PHPUnit-related one.22:29
xeviousAlternatively, I could just make the package not run the tests.22:32
xeviousThat doesn't seem great.22:32
naccit has been done to a few packages now22:32
xeviousWell, if there's precedent...22:32
xeviousWhat do you recommend?22:32
naccxevious: so, aiui, the issue is that the upstream isn't packaging the tests, so the dep8 tests fail? doe sit still build?22:34
naccxevious: you can remove the dep8 tests if they really aren't available anymore, i think, and i will need to send a hint MP to ignore the regression (removing tests is considred a regression too)22:34
xeviousIt doesn't build because there's a patch for bootstrap.php in the tests directory.22:35
naccxevious: right i had to fix soethig similar recently22:35
naccin a different package22:35
naccxevious: but i mean the dep8 tests no longer make sense right?22:35
naccxevious: since the tests are no longer shipped22:35
xeviousRight22:35
xeviousI can either remove the tests from the packaging process or I can open a PR with upstream in hopes that they quickly tag a new version.22:36
naccxevious: you can do both :)22:36
naccxevious: which is probably ideal ;)22:37
xeviousOk22:37
xeviousI'll do both22:37
xeviousnacc: https://github.com/guzzle/promises/pull/8722:41
naccxevious: yeah, i can understand the upstream theory, you don't need the tests to have a functional package22:41
naccbut i've not seen other upstreams do that22:41
naccand the extra space is ... minimal22:42
xeviousYeah, I understand their reasoning, However, I can think of a number of cases where you'd want to use an archive download rather than git, and then be able to run the tests.22:43
jbichacjwatson: if you're around, can you kick the ppc64el builders?22:43
naccxevious: yep22:43
xeviousnacc: It looks like php-guzzlehttp-promises is inherited from Debian. How does cooperating with them work? Should I make the change on LaunchPad and we incorporate it into Bionic, then upstream it to Debian? Or, should I work with Debian from the start and pull it into Bionic when it's in their repo?22:46
naccxevious: either is ok, the latter tends to take longer22:46
naccxevious: so the simpleset answer is make the change in ubuntu, and we can send stuff up to debian (there is a submittodebian helper)22:46
naccafter we get ubuntu working :)22:46
xeviousPerfect22:46
naccxevious: fixed php-horde-core22:46
nacctesting it again and then i'll upload22:47
xeviousRight on!22:47
naccthat might be what's needed throughout22:47
nacci *think* possibly phpunit 6 dropped support for autoloading a boostrap.php file?22:47
naccthat seems to fix a lot of packages (passing the option to phpunit)22:47
sarnoldslangasek,bdmurray, I have a suspicion Something Is Wrong: a friend showed me 'apt upgrade' https://bpaste.net/show/a0ccc1e394d7 and 'apt dist-upgrade' https://bpaste.net/show/aa8d1ff48634 results that looked like they worried him, and I can reproduce something very similar: http://paste.ubuntu.com/p/WTD9rYCcFq/ (please ignore firefox..)22:54
naccyep somoene elsehwere also reported the removal of unity-scope* from their system22:54
sarnoldnacc: oh, hooray, did you recall where? or bug number?22:55
nacclooking in my logs22:55
naccsarnold: #ubuntu-discuss, no bug filed22:55
bdmurraysarnold: what release?22:56
sarnoldbdmurray: I believe xenial all around22:56
naccchecking on that there but they did say xenial at one point22:56
naccwhich seems 'even worse' :)22:56
sarnoldhttps://irclogs.ubuntu.com/2018/02/15/%23ubuntu-discuss.html#t20:1722:57
naccsarnold: thanks22:57
naccuggggh23:00
nacccompiz-core migrated without a unity rebuild?23:00
naccunity: depends on compiz-core-abiversion-20151010,23:00
naccProvides: compiz-core-abiversion-2017063023:01
nacclatter from compiz-core in updates23:01
bdmurrayif its already in -updates we'll need an archive admin23:01
naccit looks to be in both right now23:02
nacchttps://launchpad.net/ubuntu/+source/compiz23:02
naccupdates and proposed, i mean23:02
naccbdmurray: i'm not 100% on my analysis but that abi mismatch seems worrisome with no changes to unity23:02
sarnoldis this time for the !regressions batsignal or whatever it is?23:07
naccheh23:07
naccsarnold: it's being discussed in #ubuntu-release23:08
sarnoldnacc: <323:08
naccsarnold: phasing stopped23:08
nacc(presumably all affected were in the last 4 hours)23:08
naccxevious: fyi php-horde-core uploaded23:10
xeviousGreat. Looking forward to seeing a bunch more green...23:12
naccxevious: bbiab23:14
tsimonq2Can a Core Dev please approve this bug nomination? 174680723:17
tsimonq2bug 1746807 even23:17
ubottubug 1746807 in base-installer (Ubuntu) "18.04 daily installer fails missing kernel" [Critical,Confirmed] https://launchpad.net/bugs/174680723:17
rbasaktsimonq2: why does it need nomination for Bionic?23:17
tsimonq2rbasak: It's a Bionic-specific bug, I'll nominate Bionic out of habit I guess (for future ref should I need to revisit it at any point).23:18
tsimonq2s/Bionic out/Bionic if I have access to out/23:19
tsimonq2rbasak: I guess there's no *need* to but it's good for bookkeeping I guess23:20
rbasaktsimonq2: it'll affect >= Bionic until it is fixed, no?23:22
rbasakThat's the same as any other bug, and we don't usually nominate the development release for that kind of thing.23:22
tsimonq2OK23:22
rbasakThere was a time when people were adding a development-release-specific bug task for release bug tracking purposes23:23
rbasakI'm not sure if we still do that23:23
rbasakI'm not sure it's a good idea in general, because it collides with tracking SRUs when a series is relaesed.23:23
rbasakBut I don't think anyone has actually tried to define a policy on this since Kate left.23:24
tsimonq2OK23:24
rbasakinfinity, slangasek: ^ opinion?23:25
slangasekrbasak: we do have per-team reports on bugs targeted to the current development release; I gather Server Team isn't using this? http://reqorts.qa.ubuntu.com/reports/rls-mgr/rls-bb-tracking-bug-tasks.html#ubuntu-server23:26
xnoxrbasak, for foundations it does make a different if something is "whenever" vs "whenever, but to fix for bionic"23:27
rbasakslangasek: we're aware of the link, but we never controlled what was actually on that list, or how to get bugs out of the list, partially (IMHO) due to a lack of an Ubuntu-wide policy on it.23:27
rbasakSo (again IMHO) it never seemed to be a particularly useful way of tracking anything23:28
xnoxrbasak, well, the report is fixed, in what it is generating into accepted / incoming / rejected pages.23:28
rbasakxnox: who accepts/rejects though, and what if the server team disagrees with such a nomination?23:28
slangasekrbasak: there is a tag to decline a bug from that list as not-committed-by-team despite being targeted to the release; I forget the exact name, bdmurray would remember23:28
bdmurraynotfixing23:29
slangasekit might be rls-$series-declined23:29
slangasekok that23:29
xnoxrbasak, people who what it request for a bug to be considered add rls-LL-incoming; if $team accepts it, the $team removes the tag and targets to series, and potentially to a monthly milestone (but we moved on to creating trello cards for these)23:30
xnoxrbasak, or drop incoming tag and replace by notfixing.23:30
xnoxrbasak, as $server team is meant to keep track of the bugs, to packages, which the team is subscribed to, for rls-incoming bugs.23:30
xnoxat least foundations does it for foundations-bugs subscribed packages.23:31
slangasek"is meant to" - well, that's the intent of the report, but we obviously don't dictate how other teams use it23:31
* xnox wonders if we due to scope we need automation for this23:31
* xnox wonders if we do, due to scope we need automation for this23:31
* xnox wonders if we do, due to scope. Hence we need automation for this23:31
* bdmurray wonders about xnox23:31
xeviousnacc: Should I add `override_dh_auto_test` to rules, or should I delete the `test` rule from the upstream project's Makefile?23:31
rbasakWe use Trello too23:32
xnoxbdmurray, i had a volleyball training, my fingers are a bit shaky, and I'm misstyping a lot.23:32
slangasekcan we automate the scope, if the guards come with him23:32
xnoxrbasak, for us bugs are public; trello is private.23:32
rbasakOur Trello is public I think23:32
rbasakBut it's really a ~canonical-server Trello, rather than an ~ubuntu-server Trello.23:33
xnoxrbasak, yeah, most teams have it like that. It's just the PDX based managers who have it private.... kernel team / foundations team / security (?!) / is (?!)23:33
tsimonq2rbasak: You guys have your plans to take over the world there? :P23:33
tsimonq2s/guys/guys and gals/ if you will23:33
* tsimonq2 steps away from that debate real quick23:34
slangasekxnox: we were keeping the managers' locations private23:34
rbasakPrivate customer work happens on a different board23:34
xnoxrbasak, yeah, so rls-incoming is "canonical-foundations is commiting to things" vs "non rls-tracked bugs ubuntu-foundations may volunteer to do it with no sla"23:34
rbasakxnox: for packages to which you subscribe, presumably23:34
xnoxrbasak, typically rls-LL-incoming bugs we get; are escalations from either community or other ~canonical-foo or ~ubuntu-foo teams. We try to assess criticality, and impact. As we get flodded with bugs =/ and we use that for people to bubble things up to us; we review incoming bugs at every weekly meeting.23:35
xnoxwe hardly are a zero-boogz-found team =/23:36
xnoxaka inbox-zero23:36
rbasakThat sounds reasonable except I'm not convinced about the need to peg it to a particular release23:37
xnoxwell, we used to have foundations STS handle stable releases incoming bugs a bit ;-)23:38
xnoxand reports currently track things as "accepted" when targetted to a release, instead of an 'rls-LL-accepted' tag.23:38
xnoxi guess that is simply how the reports got implemented.23:38
rbasaktsimonq2: so I suppose that means that you need to tag your bug rls-bb-incoming23:40
rbasaktsimonq2: rather than asking for a nomination to be accepted directly23:40
rbasaktsimonq2: since Foundation is subscribed to that package (presumably)23:40
tsimonq2rbasak: aha, ok23:40
xnoxthat =) at least for packages that are subscribed by ~foundations-bugs team23:41
tsimonq2rbasak: I'm trying to see if it's something I can figure out on my own, but obviously if I can't do that, I'll pass to Foundations23:41
xnoxrbasak, rls-xx-incoming sometimes become hot in the run up to point releases... vs rls-bb-incoming in the run up to feature freeze / milestones....23:41
xeviousnacc: adt-run [18:42:14]: @@@@@@@@@@@@@@@@@@@@ summary23:46
xevious*                    SKIP no tests in this package23:46
xeviousOn to the next one...23:46
naccxevious: i'm back now23:55
naccxevious: nice, i'm guessing you need sponsorship for that? file a bug against hte srcpkg and upload the debdiff to it23:56
naccxevious: and subscribe me to it23:57
cjwatsonjbicha: done23:59

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