| didrocks | hey Mirv, happy new year! | 06:48 |
|---|---|---|
| === ev changed the topic of #ubuntu-ci-eng to: Ubuntu CI Engineering Team | Vanguard: ev | Landing instructions: http://paste.ubuntu.com/6292280/ | Known issues: - | ||
| popey | didrocks: no landing task force meeting? It disappeared. | 09:30 |
| popey | (morning and HNY btw) | 09:30 |
| didrocks | popey: magically! | 09:31 |
| didrocks | popey: happy new year ;) | 09:31 |
| ogra_ | lol | 09:31 |
| didrocks | yeah, Mirv and sil2100 are off today | 09:31 |
| popey | ah | 09:31 |
| didrocks | no real need to have one, let's get people catching up emails | 09:31 |
| didrocks | and trolling for ogra_ :p | 09:31 |
| popey | haha | 09:31 |
| popey | ogra_: nexus 5 done yet? | 09:31 |
| popey | how about now? | 09:31 |
| popey | and now? | 09:31 |
| popey | etc | 09:31 |
| ogra_ | so i got calendar notifications from my phone every damned day during my vacations ... but the firs meeting after my vacations doesnt notify me :P | 09:32 |
| popey | haha | 09:32 |
| didrocks | popey: apparently, he's rewriting something in perl soon… :p | 09:32 |
| ogra_ | lol | 09:32 |
| popey | ☻ | 09:32 |
| popey | Hmm.. Ubuntu Touch on one of these puppies... http://liliputing.com/2014/01/hp-unveils-slate21-pro-android-powered-one-desktop.html? | 09:39 |
| ogra_ | ++ | 09:40 |
| asac | o/ | 09:42 |
| asac | HNY :) | 09:42 |
| ogra_ | same same :) | 09:42 |
| didrocks | hey asac, HNY too! | 09:46 |
| asac | didrocks: thanks!\ | 09:47 |
| === john-mca` is now known as john-mcaleely | ||
| rsalveti | popey: that device looks quite cool, but would be way better with ubuntu | 11:46 |
| ogra_ | i doubt HP would have liked to wait for 14.10 and a working desktop mode in the converged UI though :) | 11:55 |
| davmor2 | didrocks, ogra_: are you looking to promote a new image at all today? or can I test the current? | 12:41 |
| didrocks | davmor2: just test current, we won't promote anything I guess until we are back on shape :) | 12:42 |
| * ogra_ hasnt heard anything ... and we have no meeting today | 12:42 | |
| ogra_ | i guess its not *that* urgent to promote one today | 12:42 |
| * didrocks wonders why the launchpad cred message isn't shown | 12:42 | |
| === alan_g is now known as alan_g|lunch | ||
| === josepht changed the topic of #ubuntu-ci-eng to: Ubuntu CI Engineering Team | Vanguard: josepht | Landing instructions: http://paste.ubuntu.com/6292280/ | Known issues: - | ||
| Kaleo | wasn't there a QA website with boot startup time for Ubuntu? | 13:20 |
| ogra_ | Kaleo, for x86/amd64 there is ... | 13:21 |
| Kaleo | ogra_, where is it? I cannot find it | 13:21 |
| * ogra_ still has an open task to set something similar up for touch | 13:21 | |
| ogra_ | Kaleo, https://jenkins.qa.ubuntu.com/view/Bootspeed/ | 13:21 |
| Kaleo | ogra_, hmmm | 13:22 |
| Kaleo | ogra_, it was a very different website before | 13:22 |
| ogra_ | hmm | 13:22 |
| Kaleo | ogra_, with even rendering stats per hardware too | 13:22 |
| ogra_ | yeah, there must be bootcharts somewhere | 13:22 |
| Kaleo | http://ci.ubuntu.com/ | 13:25 |
| Kaleo | ... | 13:25 |
| Kaleo | ...:) | 13:25 |
| Kaleo | that's it :) | 13:25 |
| ogra_ | hmm, actually not the one i meant | 13:27 |
| ogra_ | mine actually had proper bootchart pngs linked | 13:28 |
| ogra_ | but i cant find it ... these bootspeed measurements arent very helpful to track down services delaying the boot | 13:28 |
| xnox | Kaleo: http://ci.ubuntu.com/bootspeed/arch/amd64/ | 13:31 |
| xnox | ogra_: ^ | 13:31 |
| ogra_ | xnox, no bootchart | 13:31 |
| ogra_ | (thats the details page of Kaleo's link above) | 13:32 |
| seb128 | ogra_, it has, click on "detailed view" and then "raw data" on the left | 13:34 |
| ogra_ | doesnt load anything for me :( | 13:35 |
| seb128 | it's slow for me too, I had to retry 3 times to get the list | 13:35 |
| xnox | ogra_: no without pain, one can download get everything from there... there is even a full tarball of logs & full-blow bootchart images available.... somewhere.... | 13:40 |
| ogra_ | all i could find looks like a self knotted bootchart impostor thingie | 13:40 |
| ogra_ | *knitted | 13:41 |
| ogra_ | (through jenkins.qa.u.c) | 13:41 |
| seb128 | ogra_, https://jenkins.qa.ubuntu.com/job/bootspeed-trusty-desktop-i386-acer-veriton-01/9/artifact/11/bootchart.png | 13:45 |
| ogra_ | yeah | 13:45 |
| ogra_ | it finally loaded here too | 13:45 |
| * ogra_ just looks at artifact 13 .... 57sec boot time ... ugh | 13:46 | |
| seb128 | seems the system stopped working though, no record since 20131108 | 13:46 |
| ogra_ | yeah | 13:47 |
| === alan_g|lunch is now known as alan_g | ||
| === alex_abreu is now known as alex-abreu | ||
| fginther | morning | 14:23 |
| didrocks | hey fginther! happy new year :) | 14:25 |
| fginther | didrocks, happy new year to you as well! | 14:26 |
| didrocks | thanks ;) | 14:26 |
| didrocks | fginther: before your meeting starts, do you know where you backup the gpg and ssh keys for ps-jenkins? | 14:26 |
| didrocks | I think we'll need them for the citrain and ev needs to put that as part of his incoming juju deploy script | 14:27 |
| fginther | didrocks, yep, I know where they are, let me dig them out for you | 14:27 |
| didrocks | fginther: yeah, if you can send them to me as well, that would help for my local testing! :) | 14:27 |
| fginther | didrocks, and get them documented | 14:27 |
| didrocks | perfect! | 14:27 |
| kgunn | didrocks: happy new year ! | 14:59 |
| kgunn | didrocks: am i correct in seeing, you guys just have some backlog to work thru before getting to mir ? | 15:00 |
| kgunn | totally understandable | 15:00 |
| didrocks | kgunn: hey! happy new year :) | 15:00 |
| didrocks | kgunn: yeah, there are some backlogs and Mirv/sil2100 start only tomorrow :) | 15:00 |
| ogra_ | hmm, nobody pulled in the session changes it seems | 15:00 |
| didrocks | kgunn: so expect that we try landing Mir on wednesday I would say, is that ok for you? | 15:00 |
| kgunn | ok...np, will be looking to make sure mir is well behaved :) | 15:00 |
| didrocks | kgunn: thanks :) | 15:01 |
| didrocks | ogra_: no, see associated comment, it was pending on mterry, I didn't track then when I was on holidays | 15:01 |
| ogra_ | hmm, i thought it was ready when i left | 15:01 |
| ogra_ | didrocks, bah, thats long obsolete | 15:03 |
| didrocks | ogra_: ah, well, it was still written :) | 15:03 |
| ogra_ | (he commented on the MP) | 15:03 |
| didrocks | (not sure why it's not showing up as red anymore) | 15:03 |
| === charles_ is now known as charles | ||
| sergiusens | fginther, hey, can we circle back with the android builder? | 16:39 |
| === gatox is now known as gatox_lunch | ||
| === alan_g is now known as alan_g|afk | ||
| fginther | sergiusens, I'll be available shortly | 16:49 |
| sergiusens | fginther, ack, thanks | 16:50 |
| fginther | sergiusens, here's the last attempt: http://s-jenkins.ubuntu-ci:8080/job/ubuntu-touch-image-builder/12/console | 17:04 |
| === alan_g|afk is now known as alan_g | ||
| sergiusens | fginther, try and install xz | 17:28 |
| fginther | sergiusens, xz is there | 17:29 |
| sergiusens | hmm | 17:30 |
| sergiusens | fginther, is this precise, trusty,..? | 17:31 |
| fginther | sergiusens, it was raring, I've since updated to saucy and am now trying a fresh build | 17:31 |
| sergiusens | fginther, ack, error looks weird to me | 17:32 |
| sergiusens | as in, not much information :-) | 17:32 |
| === ev changed the topic of #ubuntu-ci-eng to: Ubuntu CI Engineering Team | Vanguard: cihelp | Landing instructions: http://paste.ubuntu.com/6292280/ | Known issues: - | ||
| === gatox_lunch is now known as gatox | ||
| === alan_g is now known as alan_g|EOD | ||
| Kaleo | ogra_, any idea how to disable MTP on the phone? | 18:44 |
| Kaleo | ogra_, it seems to be the probable cause of my adb/ssh shell being disconnected constantly | 18:44 |
| Kaleo | just dug up https://code.launchpad.net/~pwlars/ubuntu-test-cases/mtp-is-less-broken/+merge/197942 | 18:45 |
| ogra_ | Kaleo, iirc there was a regression that cyphermox and sergiusens were about to fix | 18:52 |
| === fginther changed the topic of #ubuntu-ci-eng to: Ubuntu CI Engineering Team | Vanguard: fginther | Landing instructions: http://paste.ubuntu.com/6292280/ | Known issues: - | ||
| sergiusens | Kaleo, ogra_ constantly or just at boot? | 18:57 |
| fginther | sergiusens, still fails on saucy: http://s-jenkins.ubuntu-ci:8080/job/ubuntu-touch-image-builder/14/console | 18:57 |
| fginther | sergiusens, same error regarding vmlinuz | 18:57 |
| sergiusens | Kaleo, ogra_ there is only one disconnect left and it's when android starts and resets the usb device | 18:57 |
| ogra_ | sergiusens, what was that regression davmor2 talked about ? | 18:58 |
| sergiusens | ogra_, something about mtp not working on boot when the device was connected | 18:58 |
| ogra_ | (i dont think the disconnect on boot is a regression, it was always there) | 18:58 |
| sergiusens | although that works for me | 18:58 |
| sergiusens | but it's not related to Kaleo's comment at all | 18:58 |
| Kaleo | sergiusens, constantly when connecting via ssh over adb | 18:59 |
| Kaleo | sergiusens, less frequently with just adb | 18:59 |
| sergiusens | Kaleo, that means you change the usb settings? | 18:59 |
| davmor2 | ogra_, sergiusens: cyphermox I think was looking at that, the mtp from Friday | 18:59 |
| Kaleo | sergiusens, I don't know what that means; I'm just connecting via ssh | 19:00 |
| davmor2 | ogra_, sergiusens, Kaleo: This is when you're ssh/adb session gets kicked when mtp connects I guess which is an older issue right? | 19:00 |
| davmor2 | your even | 19:01 |
| Kaleo | davmor2, that's my guess | 19:01 |
| Kaleo | davmor2, yes | 19:01 |
| davmor2 | Kaleo: yeah it does there is a bug for that :) | 19:01 |
| sergiusens | Kaleo, davmor2 I don't know how adb can reset ssh unless you connect over rndis | 19:01 |
| Kaleo | davmor2, linkie? | 19:02 |
| davmor2 | sergiusens: I don't know about ssh but adb shell work fine if you attach after mtp has attempted to connect if you attach before you get kicked when mtp attaches | 19:03 |
| davmor2 | Kaleo: I'll try and find it | 19:03 |
| Kaleo | davmor2, thanks | 19:03 |
| sergiusens | davmor2, I know; but the definition of constantly disconnects Kaleo gave is a bit different than disconnects on boot | 19:03 |
| davmor2 | https://bugs.launchpad.net/mtp/+bug/1233613/ | 19:04 |
| ubot5 | Ubuntu bug 1249162 in android-tools (Ubuntu) "duplicate for #1233613 Devices lose adb connection after phablet-flash loop" [High,Confirmed] | 19:04 |
| sergiusens | davmor2, mtp attaching should be the cause of the last disconnect; it's actually setting the usb device from ubuntu, then when android starts have it do the same (hence the disconnect) | 19:04 |
| sergiusens | davmor2, and the mtp job doesn't touch the usb config anymore, which waas the 3 disconnect | 19:05 |
| davmor2 | sergiusens: ah fair enough | 19:06 |
| davmor2 | Kaleo: ^ sorry I didn't add your nick to the bug | 19:07 |
| davmor2 | sergiusens: that's good to know, so now in theory it shouldn't kick adb off when mtp connects right? if so I'll run a test on that once mtp is working correctly :) | 19:08 |
| sergiusens | davmor2, it shouldn't, no; the usb device is reset only when changing the android property for the usb device | 19:09 |
| Kaleo | davmor2, thank you | 19:09 |
| cjohnston | / | 19:10 |
| davmor2 | sergiusens: thanks for the info I'll run a few tests on it when I know the mtp fix has landed | 19:12 |
| dobey | is there anything special one needs to do, to get a whole new package landed on the image? | 19:14 |
| sergiusens | dobey, needs to be in the touch seed | 19:14 |
| sergiusens | https://code.launchpad.net/~ubuntu-core-dev/ubuntu-seeds/ubuntu-touch.trusty | 19:15 |
| dobey | sergiusens: right. i mean, if the package is set up for daily-release and in CI already, but not in ubuntu. | 19:17 |
| sergiusens | afaik, the daily release team can get into ubuntu | 19:18 |
| sergiusens | robru, ^^ | 19:18 |
| dobey | who all is the daily release team? | 19:19 |
| sergiusens | dobey, long read https://wiki.ubuntu.com/DailyRelease, short read https://launchpad.net/~ubuntu-unity | 19:22 |
| sergiusens | so that's basically cyphermox robru didrocks Mirv sil2100 | 19:23 |
| dobey | yeah, most of the people on it are in europe | 19:25 |
| dobey | kenvandine: !! :) | 19:26 |
| kenvandine | hey dobey | 19:27 |
| dobey | hey kenvandine. ready for the freezing tomorrow? | 19:27 |
| kenvandine | not really :) | 19:28 |
| dobey | heh | 19:28 |
| kenvandine | my plan is to not leave the house :) | 19:28 |
| kenvandine | dobey, what package do you need? | 19:28 |
| dobey | kenvandine: can you trigger a build of ubuntu-purchase-service into the daily ppa? | 19:28 |
| kenvandine | what stack is it in? | 19:28 |
| dobey | then do the rest of the daily-release work to get it into ubuntu if it builds successfully | 19:29 |
| dobey | kenvandine: click (is that a stack?) should be in the same stack that unity-scope-click is in | 19:29 |
| kenvandine | doesn't look like it's in the click stack | 19:30 |
| dobey | i put it next to unity-scope-click in the CI config, anyway | 19:30 |
| dobey | huh | 19:30 |
| kenvandine | ok, i see it now | 19:31 |
| kenvandine | but not showing in jenkins | 19:31 |
| dobey | hrmm | 19:31 |
| kenvandine | maybe it wasn't redeployed? | 19:31 |
| dobey | not sure. i saw the landing jobs for it when i looked at jenkins the other day | 19:32 |
| dobey | or maybe some other bit needs to be deployed for daily-release jobs stuff that i don't know about | 19:32 |
| dobey | who to ping about that? cihelp? | 19:32 |
| fginther | dobey, looking | 19:33 |
| kenvandine | i'm redeploying the stack | 19:34 |
| fginther | kenvandine, thanks | 19:34 |
| dobey | ah ok | 19:35 |
| fginther | dobey, I also deploy the webcred stack to finish ubuntuone-client-data. sorry for overlooking this portion of the deploy | 19:38 |
| dobey | thanks fginther | 19:38 |
| fginther | (2 deploys are needed on 2 different jenkins, I only did the first one) | 19:38 |
| kenvandine | http://q-jenkins.ubuntu-ci:8080/job/cu2d-click-package-head-1.1prepare-ubuntu-purchase-service/2/console | 19:40 |
| kenvandine | fginther, what's wrong with the version? | 19:40 |
| fginther | kenvandine, looking | 19:41 |
| kenvandine | oh | 19:41 |
| kenvandine | it isn't UNRELEASED maybe? | 19:41 |
| fginther | kenvandine, trusty should be ok | 19:43 |
| dobey | hrmm, one second | 19:45 |
| kenvandine | does it require an ubuntu revision? | 19:46 |
| * kenvandine doesn't remember what's expected there | 19:46 | |
| dobey | it might. | 19:46 |
| dobey | kenvandine: can you try to do builds of ubuntuone-client-data or ubuntuone-credentials? | 19:46 |
| dobey | if those fail on the version it's probably the broken expectation of a non-native version | 19:47 |
| dobey | so, blame didrocks :) | 19:47 |
| dobey | not sure what code to fix, in order to fix that, though | 19:48 |
| kenvandine | i started ubuntuone-client-data | 19:48 |
| kenvandine | same error | 19:48 |
| dobey | ok, so almost certainly it doesn't like native versions | 19:51 |
| dobey | hrmm | 19:51 |
| dobey | let me see if i can fix it | 19:51 |
| dobey | oh no, that's not the problem | 19:53 |
| dobey | seems it's parsing the date wrong | 19:53 |
| dobey | ah, becasue the regexp is wrong | 19:54 |
| fginther | dobey, kenvandine, looks like it expects 20131204 and not 013.12.19 | 19:54 |
| fginther | err 2013.12.19 | 19:54 |
| dobey | oh | 19:55 |
| dobey | yeah | 19:55 |
| dobey | where did i get the one with dots from | 19:57 |
| dobey | huh | 20:02 |
| dobey | robru: why was this done? https://bazaar.launchpad.net/~dobey/ubuntu-purchase-service/fix-version/revision/7 | 20:02 |
| dobey | and apparently without an MP for it? | 20:04 |
| dobey | kenvandine, fginther: https://code.launchpad.net/~dobey/ubuntu-purchase-service/fix-version/+merge/200578 | 20:17 |
| kenvandine | ugh... you reverted the packaging cleanup... | 20:19 |
| kenvandine | robru would have done that to follow our packaging standards | 20:19 |
| dobey | robru should have made a merge proposal | 20:20 |
| kenvandine | agreed | 20:20 |
| dobey | and explained why those changes make sense here | 20:20 |
| kenvandine | https://wiki.ubuntu.com/DailyRelease/InlinePackaging#Package_inlining_checklist | 20:21 |
| kenvandine | not sure why their wasn't a MP... maybe it didn't have CI setup yet? | 20:22 |
| dobey | it had ci | 20:22 |
| dobey | or it should have | 20:22 |
| dobey | at the very least it had people working on the code and reviewing branches | 20:22 |
| dobey | "jenkins isn't running on it yet" is not a valid reason to not have an MP | 20:23 |
| kenvandine | dobey, agreed | 20:25 |
| dobey | the changes to debian/control made no sense anyway. it didn't make it easier to read | 20:25 |
| kenvandine | all of the daily release packages we do that, didrocks set a standard for that | 20:25 |
| kenvandine | trailing comma | 20:25 |
| kenvandine | etc | 20:25 |
| kenvandine | sorting | 20:25 |
| kenvandine | robru, can you comment on dobey's MP? | 20:29 |
| dobey | i have no problem discussing such proposed changes, but that's what MPs are for. you don't just outright commit to the branch | 20:30 |
| dobey | remvoing debian/source and adding .bzr-builddeb were not necessary, and wrap-and-sort -a -t makes the control file less readable and more annoying to manage. the added comment about Vcs-Bar is also confusing and unnecessary | 20:32 |
| dobey | fml | 20:40 |
| dobey | qmltestrunner == pain. | 20:40 |
| dobey | qmltestrunner: could not find a Qt installation of '' | 20:42 |
| dobey | how am i even supposed to debug that? | 20:42 |
| kenvandine | does that support qtchooser? | 20:42 |
| sergiusens | dobey, QT_SELECT=qt5 probably missing | 20:42 |
| kenvandine | what sergiusens said :) | 20:42 |
| dobey | and why would i have to specify that? | 20:43 |
| dobey | i certainly don't have to when building locally | 20:43 |
| sergiusens | to support qt4 and qt5 at the same time | 20:44 |
| sergiusens | dobey, you have qt5-defaults package installed | 20:44 |
| dobey | i don't support qt4 at all | 20:44 |
| sergiusens | to support it on the same system that is | 20:44 |
| kenvandine | dobey, you could build dep on qt5-defaults | 20:45 |
| kenvandine | but i think someone was discouraging that | 20:46 |
| dobey | it's a clean environment and i build-dep on qt5base-dev; that should be enough | 20:47 |
| kenvandine | "should" | 20:48 |
| kenvandine | but without qt5-defaults, it can't find qt | 20:48 |
| dobey | well, software has been known to be buggy | 20:48 |
| kenvandine | it would be nice if it was smart about falling back | 20:49 |
| dobey | why doesn't the qt5 stuff depends on qt5-defaults? | 20:49 |
| kenvandine | because then it wouldn't work for qt4 | 20:49 |
| robru | kenvandine, dobey: sorry was at lunch. I sometimes don't do MPs for such simple packaging changes since I follow didrock's packaging guidelines strictly, they should be uncontrovercial changes. Also it's possible that there was some rush to get a package out at the time (eg, jenkins was delayed by several hours but we wanted to get the package published for an image build). | 20:50 |
| kenvandine | on the same system | 20:50 |
| dobey | robru: this package isn't in ubuntu yet. it's a brand new project | 20:50 |
| dobey | robru: and i do find those guidelines controversial :) | 20:50 |
| kenvandine | dobey, you are the first upstream to complain about them | 20:51 |
| robru | dobey, yeah, I thought we were in a hurry to get it into ubuntu at the time. | 20:51 |
| kenvandine | dobey, that is sooo "dobey" of you :) | 20:51 |
| dobey | kenvandine: i'm probably also the only one who cares enough about package quality to complain about them :) | 20:51 |
| robru | dobey, yeah, I find your standards bizarre. 'wrap-and-sort -a -t' makes it way nicer, trailing commas especially make future diffs a lot easier to read. | 20:52 |
| dobey | robru: i don't care if diffs are easier to read or not. i'm not reading diffs when i'm looking at debian/control :) | 20:52 |
| dobey | and i don't think wrap-and-sort makes it nicer | 20:52 |
| kenvandine | didrocks set strick guidance on package consistency for everything landing in daily release | 20:52 |
| robru | dobey, um... lots of people are reading diffs of debian/control? like the release team? | 20:53 |
| dobey | having 1 space at the beginning of the line for EVERY dependency item, makes it much nicer for me | 20:53 |
| dobey | robru: really? debian/control shouldn't be changing so often | 20:53 |
| kenvandine | dobey, part of this is so the cu2d system can decide if the packaging needs packaging review before publishing | 20:54 |
| robru | dobey, it may not change often, but when it does, clean diffs are nice. also one-space indents are literally unheard of, it's a lot easier to read with a nice big indent that sets it apart from the rest of the file. one-space indent is nearly invisible, hard to see where the dep list ends etc | 20:54 |
| kenvandine | and gives a packaging only diff for us to review | 20:54 |
| dobey | kenvandine: they can't both be guidelines, and be strict. it's one or the other. and i've already bugged didrocks about the split/native thing, and i'm trying to make it better by not having it, here | 20:54 |
| dobey | robru: the problem with wrap-and-sort is that the indent is not consistent across the entire file | 20:55 |
| kenvandine | what i care about is consistency... i really don't want some packages split and some native | 20:55 |
| dobey | kenvandine: i don't want any packages split | 20:55 |
| robru | dobey: it is consistent: each following line is indented to the position of the first line ;-) | 20:55 |
| dobey | kenvandine: things that include the debian/ dir upstream are native packages | 20:55 |
| kenvandine | i don't want to go back and change all of them | 20:55 |
| dobey | kenvandine: using bzr-builddeb split mode is a red herring | 20:55 |
| dobey | robru: no, it is not consistent | 20:56 |
| dobey | robru: it is indented according to the value it is filling | 20:56 |
| dobey | robru: "Build-Depends:" and "Depends:" are not the same width | 20:56 |
| robru | dobey, yes, each stanza is consistently indented according to the value it's filling ;-) | 20:56 |
| dobey | robru: which means copying a dep from one to the other is a pain | 20:56 |
| robru | dobey, it's not a pain, just run 'wrap-and-sort -a -t' when you're done, it will indent it nicely for you. or get a real editor ;-) | 20:57 |
| dobey | and considering they are often the same, especially with packages that are runtime-interpreted code | 20:57 |
| dobey | whatever | 20:57 |
| dobey | if you have a problem with the way it exists in the tree, file a bug and a merge proposal and we can argue about it there | 20:57 |
| dobey | i don't want to waste time arguing about it in irc with multiple people about different aspects of the same thing | 20:59 |
| robru | dobey, i don't have a problem, I am following guidelines as I have done for literally hundreds of packages ;-). you're the one that has a problem. | 20:59 |
| dobey | i have no problem. i am following the policy as i have done for literally hundreds of packages :) | 20:59 |
| dobey | a) i shouldn't have to run wrap-and-sort, ever. b) my editor is just fine. "get a real editor" is not an argument, it's an insult | 21:00 |
| robru | dobey, what policy? I have never seen a policy that says "use one space indents and make sure you leave off trailing commas in a diff-ugly way." | 21:00 |
| dobey | sigh | 21:00 |
| kenvandine | dobey, the thing is, with the move to daily release this is the standard we've been given to follow | 21:01 |
| kenvandine | and it has been so for quite a while | 21:01 |
| kenvandine | so lots of packaging already following this | 21:02 |
| kenvandine | everything in daily release is | 21:02 |
| robru | kenvandine, have you ever seen a package with one-space indents? I have literally never seen that before. if you're going to argue that there should be a fixed number of spaces, at least pick a sane number like 4 or 8. | 21:02 |
| kenvandine | robru, i have... lots of them | 21:02 |
| dobey | every package i've ever put in ubuntu has one space indents | 21:02 |
| dobey | every Description: line has one space indents | 21:03 |
| robru | dobey, in the description line, it's a hack to tell the parser that it's a continuation line instead of starting a new field.... for dep lists i've never seen it that way | 21:03 |
| kenvandine | dobey, this isn't about difference of opinions... it's about standardizing all the packages we have auto landing for the distro | 21:03 |
| dobey | kenvandine: no, this is about commiting chages to a tree without a merge proposal | 21:04 |
| kenvandine | that would have helped understand, sure | 21:04 |
| kenvandine | but that isn't what we're talking about now... it's about packaiging preferences | 21:05 |
| dobey | we're only talking about that because i'm reverting the changes that were committed without an MP | 21:05 |
| dobey | and they happen to be packaging preferences | 21:06 |
| kenvandine | dobey, ok, so you'd take the changes if robru proposed them? | 21:06 |
| dobey | no | 21:06 |
| dobey | but i'd discuss them in the MP | 21:06 |
| dobey | and with didrocks | 21:06 |
| kenvandine | i guess that's the next step | 21:06 |
| kenvandine | and you can try to convince didrocks :) | 21:07 |
| dobey | if the commit i was reverting were style changes to c++, we wouldn't be having this conversation | 21:07 |
| dobey | and all these are, are style changes | 21:07 |
| robru | dobey, you're absolutely right, they are style changes... that's why they should be uncontrovercial and left alone. | 21:11 |
| dobey | robru: by that argument, you should never had made the change then, because the existing style should not be controversial and it should be left alone :) | 21:12 |
| robru | dobey, no... i'm following a standard as I was told to do by my team lead. | 21:13 |
| kenvandine | dobey, these debates with you are so much more fun with rum | 21:14 |
| dobey | i can go get some | 21:15 |
| dobey | it's about that time anyway | 21:15 |
| kenvandine | it is... and getting cold enough that it would be nice for warmth | 21:15 |
| dobey | kenvandine: https://ubuntuone.com/08D8LG5Pg8wG5Nx6i7bOKe | 21:16 |
| kenvandine | that looks much better than this banana stuff i have left from the virgin islands... | 21:17 |
| dobey | it is very good indeed | 21:19 |
| dobey | oh man, i should have bought some bananas, caramel, and ice cream, when i went to the grocery for lunch. could totally be having some tasty bananas foster right now if i had. | 21:30 |
| dobey | might have to go back after work | 21:30 |
| dobey | kenvandine: how many packages are using the split mode for daily release anyway? | 21:32 |
| kenvandine | tons | 21:34 |
| kenvandine | gotta be well over a hundred | 21:34 |
| kenvandine | everything in cu2d | 21:34 |
| dobey | so nothing a 30 line python script couldn't handle for turning them into proper native packages? | 21:35 |
| kenvandine | probably | 21:36 |
| kenvandine | it would still be a pain | 21:37 |
| dobey | pfft | 21:37 |
| dobey | only because you'd have to review all the MPs that took only 10 minutes to create with the 30 line python script :) | 21:38 |
| kenvandine | yup :) | 21:38 |
| kenvandine | and they'd all have to get in the distro too | 21:38 |
| dobey | and you'd know i'd do it just to annoy you and kill the "there are too many packages using split mode already" argument | 21:38 |
| * kenvandine debates a drive to visit dobey | 21:39 | |
| kenvandine | :-D | 21:39 |
| dobey | heh | 21:39 |
| dobey | i have rum :) | 21:39 |
| kenvandine | :-D | 21:39 |
| dobey | well the Kraken is almost gone, but the Flor de Cana is still almost a full bottle i think | 21:40 |
| dobey | of course there's a packey that's only 1 mile away, so not really an issue :) | 21:41 |
| kenvandine | :p | 21:42 |
| dobey | aka "the man" in virginia, since you can't buy it anywhere else | 21:42 |
| kenvandine | would need lots of rum to convince me switching to native is a good idea | 21:42 |
| dobey | i don't need to convince you. you already know i'm right, and native format is right, because the packages are already native, and split mode is a hack to probably workaround some issue we wouldn't have anyway if we properly used native packages :) | 21:43 |
| kenvandine | you might be right... | 21:44 |
| dobey | heh | 21:44 |
| kenvandine | i really don't know... but i'd hate to change everything just because | 21:45 |
| dobey | well yeah, i wouldn't change it "just because" | 21:45 |
| === fginther changed the topic of #ubuntu-ci-eng to: Ubuntu CI Engineering Team | Vanguard: cihelp | Landing instructions: http://paste.ubuntu.com/6292280/ | Known issues: - | ||
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!
