=== chihchun_afk is now known as chihchun
[05:30] <Mirv> cjwatson: qtdeclarative5-ubuntu-ui-toolkit-plugin-gles Provides: qtdeclarative5-ubuntu-ui-toolkit-plugin and that used to work, otherwise eg vivid emulator wouldn't work, but now last week sil2100 was hitting similar problems on wily and was wondering what had changed.
[05:31] <Mirv> but if that's the only fix needed to have apt happy then it's good. sometimes package dependencies just become more complex, and we've had trouble guiding apt with the -gles packages before too.
=== _morphis is now known as morphis
[08:15] <dbarth> hey
[08:15] <dbarth> pinging cihelp, cause i have a build / dependency problem in silo 32, ie the oxide-qt rebuild
[08:16] <dbarth> i didn't manage to get linked to libmedia-hub-client4 despite it being available in the overlay ppa
[08:16] <dbarth> and so when we use the oxide build on the phone the media-hub audio integration is broken
[08:16] <dbarth> because of an interface binary compatibility issue (libmedia-hub-client3 and 4 are not compatible)
[08:17] <sil2100> dbarth: that's more like a trainguard thing then :)
[08:17] <sil2100> hmmm
[08:17] <dbarth> ah
[08:17] <dbarth> hi
[08:17] <dbarth> if we were to add a dependency on libm-h-c4 strictly, then it would break the desktop builds were 3 only is availble
[08:17] <sil2100> That's strange - are all the dependecies correct for it to pull in libmedia-hub-client4 during build?
[08:18] <dbarth> so what's the trick?
[08:18] <dbarth> see http://bazaar.launchpad.net/~oxide-developers/oxide/packaging.vivid/view/head:/debian/control
[08:18] <dbarth> it depends on     libmedia-hub-dev [armhf],
[08:19] <sil2100> This should dep on the latest media-hub, so 4 in case of the overlay I suppose
[08:19] <dbarth> ah, so as a special armhf thing it could be specific to libm-h-c4 then
[08:19] <dbarth> but we took 3 apparently
[08:19] <dbarth> https://launchpadlibrarian.net/214500930/buildlog_ubuntu-vivid-armhf.oxide-qt_1.8.4-0ubuntu0.15.04.1~overlay1_BUILDING.txt.gz
[08:20] <sil2100> Well, yeah, but this should work nevertheless, let me check what's up
[08:21] <sil2100> dbarth: the overlay has media-hub 3
[08:22] <sil2100> https://launchpad.net/~ci-train-ppa-service/+archive/ubuntu/stable-phone-overlay/+sourcepub/5101143/+listing-archive-extra
[08:22] <dbarth> hmm, so that's the issue
[08:22] <sil2100> This is the latest one
[08:22] <dbarth> hmm
[08:22] <sil2100> Is it required for us to have a newer media-hub, or are we ok with this here?
[08:24] <dbarth> sorry, i was wrong it's not 3 vs 4 it's 3.1 vs 3.0
[08:24] <dbarth> (re-reading justinmcp's email)
[08:24] <dbarth> but that's still something to do with dependencies it seems
[08:25] <dbarth> so the overlay has libmedia-hub-client 3.1.0+15.04.20150527.1-0ubuntu1
[08:25] <sil2100> 3.1.0+15.04.20150527.1-0ubuntu1 <- it's using the latest as expected during the build
[08:25] <dbarth> yes
[08:26] <dbarth> and so that should work
[08:26] <dbarth> ok sorry, i'll get back to testing the issue for now
[08:28] <Mirv> cihelp: what's the status of eg qtubuntu-gles boottest failing because at least in the past it tried to install that x86-only package in arm environment? http://people.canonical.com/~ubuntu-archive/proposed-migration/update_excuses.html#qtubuntu-gles
[08:28] <sil2100> dbarth: make sure you're testing the right oxide version, if you have correct pinning configured then you should fetch the ~overlay1 one
[08:28] <Mirv> cihelp that was reported some weeks ago and was said to be looked into, so I'm wondering if that's still the same issue or something else
[08:29] <dbarth> sil2100: yup, i forced that with exact version numbers to do the install
[08:30] <sil2100> Mirv, jibel, davmor2: be right there on the meeting
[08:34] <psivaa> Mirv: let me take a look at this
[08:36] <ogra_> oops
[08:46] <cjwatson> Mirv: "Provides: qtdeclarative5-ubuntu-ui-toolkit-plugin" only satisfies *unversioned* dependencies on qtdeclarative5-ubuntu-ui-toolkit-plugin; it used to work in this case because the dependency in webbrowser-app's binary packages was unversioned until recently
[08:47] <cjwatson> Mirv: though as a recent feature in dpkg/apt it's possible to say "Provides: qtdeclarative5-ubuntu-ui-toolkit-plugin (= some-version)" - I haven't tried that personally for anything, so don't know if the full stack works with it
[08:48] <cjwatson> oSoMoN: thanks for the webbrowser-app approval - would you mind sorting out a landing for it to unbreak installability?
[08:48] <oSoMoN> cjwatson, sure, I’ll file a landing request
[08:48] <cjwatson> great
[08:56] <Mirv> cjwatson: thanks for the clarification, makes sense
[08:56] <Mirv> psivaa: thanks!
[08:57] <sil2100> cjwatson: thanks! We were wondering about this failure, since it suddenly stopped working
[08:57] <sil2100> cjwatson: do you know if fixing only the webbrowser-app is enough to satisfy apt?
[08:57] <sil2100> cjwatson: since there's a LOT of qtdeclarative5-ubuntu-ui-toolkit-plugin dependencies around there, not sure if versioned ones though...
[08:58] <Mirv> sil2100: maybe only webbrowser started to use a feature only available in a newer UITK?
[08:58] <Mirv> although that wouldn't stay like that for too long even if it would be the case
[09:00] <sil2100> But from what I recall (checking that now) there was no recent change of the UITK deps to become versioned
[09:00] <psivaa> Mirv: The issue hasn't been permanently fixed. I just created a fake result for this, i'll for it to propagate
[09:02] <sil2100> hm, no, actually it got bumped recently
[09:03] <sil2100> cjwatson: anyway, thanks for explaining, at least I wasn't aware that Provides was working only for unversioned deps and that's why I didn't know what was wrong there
[09:03] <sil2100> Apparently this merge should in this case fix the world
[09:04] <cjwatson> sil2100: There are several other versioned dependencies, but they already have explicit | ...-gles
[09:04] <psivaa> Mirv: http://people.canonical.com/~ubuntu-archive/proposed-migration/update_excuses.html#qtubuntu-gles is now green
[09:04] <Mirv> psivaa: ok, thank you a lot!
[09:04] <cjwatson> (inappropriately written in some cases without the versioning on the -gles part, but that won't actually break things other than partial upgrades)
[09:05] <cjwatson> at some point it might be worth trying to make use of versioned Provides here
[09:05] <cjwatson> I'm not sure what our constraints on that are, whether we need to avoid them in the archive until trusty's apt no longer needs to parse them or something
[09:28] <sil2100> cjwatson: could you setup automatic weekly translation exports in the 15.04 series on Tuesdays evenings?
[09:30] <cjwatson> sil2100: That's already been done on the LP side, except Tuesday mornings not evenings (10:30 UTC like all the other exports).  Is that OK?
[09:30] <cjwatson> sil2100: There's something that consumes the output of that that also needs to run, but I think you need to ask pitti for that.
[09:38] <sil2100> cjwatson: oh, the auto-exports are done? Didn't know, I requested an explicit export last Friday - good... I think we would rather prefer to have it a bit later in the day to give people more time
[09:38] <sil2100> I already talked to pitti regarding langpack-o-matic
[09:38] <sil2100> cjwatson: is it a big problem to change the hour of the exports? Don't want to force anything
[09:44] <cjwatson> sil2100: I'm wary of that, because it's a big job that could end up accidentally conflicting with something else like backups
[09:44] <cjwatson> The current time is known to work
[09:44] <sil2100> cjwatson: ok, I suppose we can adjust our schedules accordingly in that case
[09:53] <oSoMoN> cjwatson, is there a bug report to track the regression my branch introduced wrt installability of ubuntu-touch on amd64 and i386 ?
[09:54] <cjwatson> oSoMoN: no
[09:54] <oSoMoN> ok
[09:54] <cjwatson> it was ~midnight, I just proposed a merge :)
[09:55] <oSoMoN> that’s alright, just wanted to check, in case there was one I would have included the link in the landing request
[09:55]  * cjwatson nods
[09:58] <oSoMoN> cjwatson, building in silo 8
[10:01] <cjwatson> oSoMoN: ta
[10:02] <sil2100> oSoMoN: dual landing?
[10:02] <sil2100> oSoMoN: since it's the case for both wily and overlay
[10:02] <oSoMoN> yes
[10:02] <sil2100> Thanks :)
[10:03] <oSoMoN> (the silo also includes two unrelated minor bug fixes)
[10:26] <Mirv> robru: almost forgot to reply, but I created bug #1484795 already on Fri
[10:26] <ubot5> bug 1484795 in CI Train [cu2d] "Can't handle a package with "-" in upstream version number" [Undecided,New] https://launchpad.net/bugs/1484795
[10:46] <Mirv> mardy: the unity8 has rebuilt in https://launchpad.net/~ci-train-ppa-service/+archive/ubuntu/landing-029 - https://wiki.ubuntu.com/Process/Merges/TestPlans/Unity8 should be executed now additionally to the earlier testing, and of course the MP should be approved before tomorrow's OTA-6 deadline (if not ok with postponing to OTA-7)
[11:16]  * sil2100 off to lunch o/
[11:19] <mardy> Mirv: thanks, I see that tsdgeos approved the branch
[11:19] <tsdgeos> mardy: Mirv: are you going to land that? because mzanetti is planning to land another unity8 silo that conflcits with that one
[11:23] <mardy> tsdgeos: ah, I see, we have the same branch in silo 14 and 29
[11:23] <Mirv> tsdgeos: well yes if the unity8 change seems ok to you, since otherwise no problems were found. but that'd sit then in the QA queue so I'm not sure.. it's not OTA-6 targetted so of lesser priority
[11:23] <tsdgeos> mardy: no no, the branch is going away from 14
[11:23] <tsdgeos> mardy: what i mean is that your branch conflicts with https://code.launchpad.net/~aacid/unity8/previews_in_order
[11:23] <mardy> tsdgeos: ok, will Mirv take care of removing it? ^
[11:24] <tsdgeos> Mirv: ok, then let us get https://code.launchpad.net/~aacid/unity8/previews_in_order in before which is a nice improvement for OTA6
[11:24] <mardy> tsdgeos: maybe since my branch is tiny, you could merge it into yours?
[11:24] <tsdgeos> mardy: care to rebase it on mine?
[11:24] <mardy> tsdgeos: sure will
[11:24] <mardy> expect a ping soon
[11:25] <Mirv> mardy: tsdgeos: if the branch can be landed also without the Qt change, then by all means merge and land it! then I can put the Qt alone into queue again.
[11:25] <mardy> Mirv: yes, it can land without the Qt change
[11:26] <Mirv> thanks then, I'll remove the branch from 029
[11:31] <mardy> tsdgeos: I pushed an updated branch; should I resubmit the MP setting your branch as diff origin, or it doesn't matter?
[11:31] <tsdgeos> mardy: makes it easier, so please do
=== _salem is now known as salem_
[11:37] <Mirv> ogra_: can you ack that https://ci-train.ubuntu.com/job/ubuntu-landing-016-2-publish/lastSuccessfulBuild/artifact/network-manager_packaging_changes.diff adds two patches under debian/...
[11:37] <Mirv> they have headers and all
[11:41] <alan_g> fginther:
=== om26er_ is now known as om26er
[11:55] <mzanetti> mardy, thanks for rebasing that branch, added it to silo 14
[11:56] <mardy> mzanetti: yw :-)
=== alan_g is now known as alan_g|lunch
=== alan_g|lunch is now known as alan_g
=== om26er_ is now known as om26er
=== charles_ is now known as chalres
=== chalres is now known as charles
=== kyrofa_ is now known as kyrofa
[14:30] <awe_> trainguards, can someone vet the packaging changes in silo-016?
[14:31] <infinity> sil2100: Please don't sponsor kernel uploads. :)
[14:32] <infinity> sil2100: Or, help usher those patches into git instead.
[14:32] <sil2100> infinity: it was an universe package, thought it would be alright :)
[14:32] <sil2100> Ok, didn't know rules for those
[14:32] <infinity> sil2100: All the Android kernels are (sort of) maintained by our kernel team.
[14:33] <sil2100> Understood, next time I'll simply direct those to git then, sorry about that
[14:33] <sil2100> Hope I didn't cause too much chaos ;)
[14:33] <ogra_> someone from the kernel team just needs to re-merge that now
=== chihchun is now known as chihchun_afk
[14:34] <sil2100> It was just a minor low-priority description fix anyway
[14:35] <sil2100> So I suppose no one would have anything against merging that in
[14:35] <ogra_> nope
[14:35] <ogra_> but someone needs to do it :)
=== chihchun_afk is now known as chihchun
[14:50] <Mirv> awe_: I tried asking ogra for that
[14:50] <awe_> I just asked cyphermox
[14:50] <Mirv> awe_: ok
[14:51] <Mirv> awe_: let me know when he acks it
[14:51] <awe_> ack
[14:51] <Mirv> was that it? ;D
[14:53] <awe_> no... sorry
[14:53] <awe_> sorry for the confusion there
[14:59] <ogra_> Mirv, oh, sorry, i'm a bit swamped, havent gotten to it yet
[15:00] <ogra_> (and these packaging changes arent really packaging changes beyond the changelog :) )
[15:00] <ogra_> so form a "packaging changes" POV this is indeed an ACK
[15:13] <sil2100> robru: ouch
[15:15] <sil2100> robru: ok, we have a possible breach in the train
[15:16] <sil2100> robru: it's a bit serious, I'll fill in a bug for it in a moment not to double type
[15:20] <Mirv> ogra_: right, it was just adding patches and no actual packaging changes. thanks.
[15:21] <Mirv> cyphermox now publish it himself
[15:24] <davmor2> sil2100: is it me or are there silos missing from the dashboard?  for example silo 011
[15:24] <cyphermox> Mirv: sorry, did I break something? awe asked me to review and publish.
[15:24] <davmor2> Mirv: ^
[15:25] <Mirv> cyphermox: no, you did the correct thing, I was just doing the same but needed core dev ack
[15:25] <cyphermox> Mirv: oh ok
[15:25] <cyphermox> Mirv: when you need reviews feel free to ask, I'm there ;)
=== chihchun is now known as chihchun_afk
[15:30] <Mirv> cyphermox: sure thanks! earlier it wasn't your TZ yet.
[15:31] <cyphermox> I'm in germany for the week
[15:39] <rvr> bfiller: Silo 47 approved
[15:39] <sil2100> davmor2: hmmm
[15:39] <sil2100> davmor2: let me check
[15:43] <sil2100> bfiller: hey! Silo 47 has unbuilt revisions in https://code.launchpad.net/~tiagosh/messaging-app/fix-1394971/+merge/264936
[15:43] <sil2100> ;/
[15:45] <sil2100> davmor2, rvr: if you guys could take a look at silo 8, it has a fix for our emulator builds
[15:45] <rvr> sil2100: I'm on it
[15:45] <sil2100> Thanks :)
[15:46] <bfiller> sil2100: let me look
[15:46] <sil2100> bfiller: we'll either have to rebuild or revert the missing revisions, but I think they might be needed
[15:47] <bfiller> salem_: ^^^ can you take alook at silo 47
[15:48] <bfiller> or boiko ^^^
[15:48] <bfiller> sil2100: should I just rebuild?
[15:48] <ogra_> sil2100, i assume you still dont have any magic tool that can re-write the distro series in a debian/changelog of a published package ? seems the android package in the overlay PPA is a wily one ...
[15:50] <sil2100> ogra_: that would require a re-publish anyway - I can re-publish it if needed after rewriting
[15:51] <ogra_> sil2100, talk to morphis ... if there is a shortcut he can take vs a full rebuild thats probably helpful
[15:52] <morphis> sil2100: I can rebrand the changelog quickly for wily
[15:52] <morphis> package is identical otherwise
[15:52] <sil2100> morphis: yeah, I think we can then publish it then ASAP
[15:53] <morphis> sil2100: without QA?
[15:53] <sil2100> Since rewriting and hacking the PPA is a bad idea in overall, as this android package was built against wily
[15:53] <ogra_> well
[15:53] <ogra_> the android package ships its own toolchain inside
[15:53] <ogra_> i doubt it matters on what distro it was build ... but yeah, do a rebuild
[15:54] <sil2100> Ok... anyway, did the package get QA last time? How did they test it?
[15:56] <john-mcaleely> sil2100, jibel I understand silo 30 is marked 'ready for qa'
[15:56] <john-mcaleely> I happen to know it's busted on krillin/vegetahd
[15:56] <davmor2> john-mcaleely: no jibel
[15:56] <john-mcaleely> and there will be a new device tarball shortly
[15:56] <john-mcaleely> aha
[15:56] <john-mcaleely> davmor2, ^ :-)
[15:56] <morphis> sil2100: see https://wiki.ubuntu.com/Process/Merges/TestPlans/Android
[15:57] <morphis> sil2100: and yes, it was QA approved last time by davmor2
[15:57] <sil2100> john-mcaleely: silo 30 landed already ;)
[15:57] <davmor2> john-mcaleely: so new device and then silo030 can be tested right which is the requirement for chinese arale as a minimum right?
[15:57] <sil2100> john-mcaleely: it was the pulseaudio trust-store one, right?
[15:58] <davmor2> sil2100: see no silo 030 listed either :(
[15:58] <john-mcaleely> sil2100, really? how did it pass QA on krillin & vegetahd
[15:58] <john-mcaleely> and (actually) arale
[15:59] <john-mcaleely> davmor2, yes, I thought it needed to land with an arale device tarball, to get chinese image
[15:59] <sil2100> john-mcaleely: what's in silo 30? Since I don't see it on request or the dashboard
[15:59] <sil2100> https://requests.ci-train.ubuntu.com/#/?search=landing-030 <- all silo 30's got published
[15:59] <sil2100> No silo 30 assigned right now
[16:00] <davmor2> sil2100: so empty silos don't show on the dashboard any more is that correct?
[16:00] <john-mcaleely> hrm. well, it was pulseaudio trust store
[16:00] <sil2100> davmor2: yes
[16:00] <john-mcaleely> and camera trust store
[16:00] <sil2100> john-mcaleely: then it landed, not sure how they tested it
[16:00] <sil2100> But it was ready for QA so they picked it u
[16:00] <sil2100> up
[16:00] <john-mcaleely> sil2100, this is the one I thought it was: https://requests.ci-train.ubuntu.com/#/?req=126
[16:00] <sil2100> I have the seed change ready
[16:00] <john-mcaleely> well, you have a busted camera then
[16:00] <john-mcaleely> on krillin, vegeta & arale
[16:01] <sil2100> davmor2, rvr: ^
[16:01] <boiko> sil2100: bfiller: yep, just rebuild it, the missing revisions are only adding a QML test, no extra code changes
[16:01] <john-mcaleely> (or I think you should have)
[16:01] <davmor2> sil2100: meeting
[16:01] <sil2100> Google issues
[16:02] <boiko> sil2100: that's regarding silo 47, I can rebuild it if you want
[16:03] <rvr> I tested silo 30
[16:03] <john-mcaleely> I think the camera is busted on it
[16:04] <sil2100> boiko: is it required? SInce we can basically try skipping that by reverting those from your branch
[16:07] <rvr> john-mcaleely: How can pulseaudio mess the camera?
[16:07] <john-mcaleely> rvr, the trust store update within also included a camera service update
[16:08] <john-mcaleely> not a really big deal. the fix is incoming :-)
[16:08] <boiko> sil2100: well, I asked salem_ to write those, messaging-app is already building again on the silo
[16:08] <boiko> sil2100: sorry for the trouble anyway
[16:09] <davmor2> john-mcaleely: silo 030 testplan on lists pulseaudio no mention of the camera app in there
[16:09] <rvr> john-mcaleely: Well, there is some deal, that should have been tested, but no word about that on the request :(
[16:09] <john-mcaleely> interesting
[16:09] <rvr> https://trello.com/c/3XaS4chY/2182-126-ubuntu-landing-030-pulseaudio-ubuntu-touch-session-abeato-diwic-mandel
[16:09] <davmor2> john-mcaleely: https://trello.com/c/3XaS4chY/2182-126-ubuntu-landing-030-pulseaudio-ubuntu-touch-session-abeato-diwic-mandel check the testplan
[16:10] <john-mcaleely> two people who are on holiday, and one fresh back in
[16:10] <john-mcaleely> it seems the error is ours then
[16:10] <john-mcaleely> given the new device tarball is incoming, probably best to just press onward
[16:11] <john-mcaleely> not sure if you have other options in the interim?
[16:11] <greyback_> cihelp: hey, qtmir stuck in wily-proposed, http://people.canonical.com/~ubuntu-archive/proposed-migration/update_excuses.html#qtmir - does boottest even work there?
[16:12] <john-mcaleely> brb
[16:12] <davmor2> john-mcaleely: yeap
[16:15] <fginther> greyback_, nope, I'll need to manually pass that one, one moment
[16:15] <greyback_> fginther: cheers
[17:04] <rvr> sil2100: Silo 8 approved
[17:05] <rvr> EOD here, see you tomorrow!
=== alan_g is now known as alan_g|EOD
[17:13] <boiko> sil2100: messaging-app rebuilt on silo 47, ready to land now
[17:19] <sil2100> o/
[17:30] <robru> morphis: uh, you sure silo 11 doesn't need QA? it's targetted at vivid overlay
[17:30] <robru> sil2100: ^
[17:30] <sil2100> Yeah
[17:30] <sil2100> I changed that
[17:31] <sil2100> It was basically tested already and it's a binary copy, no sign-off needed as it was already tested in the same state it was before
[17:31] <robru> huh ok
[17:40] <john-mcaleely> davmor2, sil2100 device tarballs to fix the busted camera on krillin/vegeta:
[17:40] <john-mcaleely> https://requests.ci-train.ubuntu.com/#/?req=180
[17:44] <davmor2> john-mcaleely: will we need one for arale too do you know?  I assume we would
[17:54] <boiko> kenvandine: could you please check and (possibly) ack the packaging changes on silo 47?
[17:55] <boiko> kenvandine: it is basically requiring some QML plugins so that the QML tests pass
[17:55] <kenvandine> boiko, i'll look
[17:55] <kenvandine> added as build deps then?
[17:55] <boiko> kenvandine: yep
[17:58] <kenvandine> boiko, +1 from me
[18:00] <sil2100> davmor2: I think there'll be another one for arale, yes
[18:00] <boiko> robru: sil2100: ^ (silo 47)
[18:00] <davmor2> sil2100: I assumed there would be
[18:01] <sil2100> boiko, robru: reviewing that
[18:04] <sil2100> boiko: ok, I think it will require a rebuild - but with an exception of not requiring QA sign-off - the ubuntu archive admins requested that all dependency changes are documented
[18:04] <sil2100> boiko: could we get the merge that modifies the deps include information about the deps being changed in the commit message?
[18:04] <kenvandine> sil2100, it's a build dep change, probably not an issue
[18:06] <sil2100> Quoting the request: "Please enforce the requirement that changelogs for train packages include an explanation of the changes to the packaging."
[18:27] <sil2100> ogra_: btw. could you merge in https://code.launchpad.net/~sil2100/ubuntu-seeds/ubuntu-touch.wily-add-iw/+merge/267370 once you have a moment?
[18:44] <sil2100> I'll upload that to the vivid seeds in the meantime
[18:52] <robru> sil2100: kenvandine: so did we come to a consensus on silo 47 then? we'll wait for boiko to rebuild with a proper changelog?
[18:52] <kenvandine> yeah
[18:52] <kenvandine> we should
[18:52] <kenvandine> boiko, ^^
[18:52] <sil2100> I wouldn't publish that, but maybe someone with power can override this ;)
=== karni is now known as karni-away
[18:55] <greyback_> fginther: hey, dobey tested qtmir from wily-proposed and it allows unity8 to come up, adds more evidence it's good!
[18:56] <dobey> "tested"
[18:56] <dobey> it fixes the unity8 crashing anyway :)
[18:56] <davmor2> video or it never happened
[18:57] <fginther> greyback_, I verified the same and pushed a passing result. It just takes a bit for it to be seen on the excuses page
[18:57] <greyback_> fginther: ah ok, thanks for that!
[18:57] <fginther> greyback_, now it's passed :)
[18:58] <greyback_> \o/
[18:59] <dobey> yay
[19:01] <pedronis> trainguards: hi, you can publish  ubuntu/landing-017 (it's the wily landing matching a recent vivid one that was QAed and is already published there)
[19:02] <robru> pedronis: sure one sec
[19:02] <pedronis> thx
[19:03] <robru> pedronis: right so the reason this wasn't published is that it's not in 'Publish without QA' state (that is, nobody knew it was ready for publishing). for future reference
[19:04] <robru> pedronis: also this merge needs to be top-approved: https://code.launchpad.net/~pedronis/ubuntu-push/fixes-to-wily/+merge/268067
[19:05] <pedronis> robru: sorry, done now
[19:06] <nikwen> pedronis: Thanks.
[19:07] <robru> pedronis: no worries, all good now
[19:07] <pedronis> robru: thx
[19:55] <davmor2> john-mcaleely: ^
[20:10] <boiko> robru: kenvandine: sorry, I had a doctor appointment, I will change the commit message and rebuild it
[20:12] <boiko> robru: kenvandine: rebuilding...
[20:16] <robru> boiko: which MP did you change?
[20:42] <boiko> robru: https://code.launchpad.net/~tiagosh/messaging-app/fix-1394971/+merge/264936
[20:59] <boiko> robru: build finished, btw
[21:03] <robru> boiko: heh, ok thanks
[21:05] <robru> boiko: https://ci-train.ubuntu.com/job/ubuntu-landing-047-1-build/18/artifact/messaging-app_packaging_changes.diff/*view*/ uhhhh the new words aren't actually in the diff.
[21:06] <robru> boiko: I think the changelog code uses git-like commit message parsing, eg the first line is the "title" and subsequent lines are snipped. Can you make the commit message be just one line and rebuild again?
[21:07] <boiko> robru: ok
[21:07] <robru> boiko: sorry for the trouble, this changelog code has a lot of issues and is definitely on the chopping block
[21:08] <boiko> robru: yeah, maybe those policy changes (like requiring the commit message to explicitly mention the packaging changes) should be better advertised?
[21:09] <robru> boiko: yeah I'm sure that's written in a policy somewhere but there could definitely be better communication about that.
[21:10] <boiko> robru: anyway, one more thing learnt, next time I will make sure the MRs are correct in that regard
[21:15] <robru> boiko: thanks, I just sent an email to ubuntu-phone as a reminder for others.
[21:15] <boiko> robru: thanks!
[21:16] <robru> boiko: you're welcome
[21:59] <boiko> robru: now the changes contains the new commit message on silo 47
=== karni-away is now known as karni
[22:12] <robru> boiko: thanks
[22:12] <boiko> robru: np
=== salem_ is now known as _salem