[04:56] <Mirv> Saviq: I'm not using quilt, but maybe it works only when sources are in the same repo as packaging?
[08:00] <Saviq> Mirv, of course <facepalm>
[08:45] <Saviq> Mirv, both branches linked to https://bugs.launchpad.net/unity-lens-applications/+bug/1271158 - if you need me to do anything else with them, pray tell :)
[09:00] <Mirv> Saviq: thanks
[09:00] <Mirv> I'll look at those
[09:01] <Saviq> Mirv, you're in London, right?
[09:10] <Saviq> Mirv, were you *still* or *already* up at 5am? ;)
[09:15] <Saviq> dednick, hey, could I ask you to do https://code.launchpad.net/~saviq/unity8/add-doxygen/+merge/204123
[09:16] <dednick> Saviq: sure
[09:16] <Saviq> dednick, probably no need to read through the whole Doxyfile.in
[09:16] <Saviq> dednick, but you might want to grep for @ and see if the substitutions make sense
[09:17] <Saviq> dednick, hmm actually now that I think of it...
[09:17] <Saviq> dednick, we probably shouldn't build the docs all the time
[09:18] <Saviq> since it takes a few seconds
[09:18] <dednick> Saviq: yeah, i just saw that
[09:18] <Saviq> let me tweak it a bit
[09:18] <dednick> graphvis can probably take a bit of time
[09:19] <dednick> or dot rather...
[09:20] <Saviq> dednick, doxygen itself is enough
[09:20] <Saviq> dednick, especially since it needs to pipe the qml through doxyqml, which is python
[09:21] <dednick> Saviq: but it needs to be built for packaging though...
[09:21] <Saviq> dednick, yup, tweaking debian/rules to do that
[09:21] <dednick> ok
[09:21] <Saviq> let's see if I did good
[09:27] <Saviq> dednick, pushed
[09:27] <dednick> Saviq: ta
[09:28] <Saviq> dednick, I just saw an issue - "/build/unity8-KAx5z5/unity8-7.84+14.04.20140129" got included in the paths, /me fixes
[09:32] <mhr3> sil2100, automerger still ignoring that one branch... what's up with that? (https://code.launchpad.net/~marcustomlinson/unity-scopes-api/smartscopes_proxy/+merge/203943)
[09:32] <Saviq> dednick, fixed
[09:32] <Mirv> Saviq: haha. no, I'm in London only next week, not this week
[09:32] <Saviq> Mirv, oh ok :)
[09:32] <Mirv> Saviq: I expect to wake up at weird hours anyhow also next week, even if I wouldn't want to ;)
[09:33] <Saviq> Mirv, 3h difference?
[09:33] <Mirv> Saviq: I think only 2h. the thing is I also wake up locally at 6am normally because of the daily schedule
[09:33] <Saviq> Mirv, right ;)
[09:33] <Mirv> so it furthers the observed delta
[09:34] <Saviq> Mirv, just go for drinks on Sunday, it'll help you accommodate :D
[09:34] <Mirv> but in London the work begins at 9am (ie. 11am here) so it's 4h difference in practice
[09:37] <ricmm> morning
[09:37] <Saviq> hey ricmm
[09:37] <ricmm> Saviq: so yea reading back I think daniel got the point
[09:38] <Saviq> dednick, https://code.launchpad.net/~saviq/unity8/packaging-and-cleanup/+merge/203829 another small one
[09:38] <ricmm> some logic needs to happen in unity8 to setup the new ipc
[09:38] <ricmm> and then plug that in p-api
[09:38] <Saviq> seb128, could you do a quick checkup on packaging changes for:
[09:38] <Saviq> https://code.launchpad.net/~saviq/unity8/add-doxygen/+merge/204123
[09:38] <Saviq> https://code.launchpad.net/~saviq/unity8/packaging-and-cleanup/+merge/203829
[09:39] <dednick> Saviq: ok
[09:39] <Saviq> ricmm, what changed between then and now? why can't we use android's copypaste service?
[09:39] <ricmm> android has no copy paste service
[09:40] <ricmm> ClipboardService is an implementation in the android ubuntuappmanager
[09:40] <Saviq> ricmm, ah!
[09:40] <ricmm> as in, code we wrote
[09:40] <Saviq> ricmm, that was the detail I was missing
[09:40] <ricmm> that uses binder for ipc and manages all through the app manager
[09:40] <ricmm> it would be the same thing to be fair
[09:41] <ricmm> just that living in unity-mir somewhere
[09:41] <Saviq> Cimi, forgot to put prerequisite on textSummary
[09:41] <Cimi> Saviq, yeah!!
[09:42] <Cimi> Saviq, done
[09:43] <Saviq> Cimi, also, while you're at it... you could implement the See more | See less buttons there, and a "title" for it
[09:43] <Saviq> Cimi, https://docs.google.com/a/canonical.com/file/d/0B-a_7E3tDxOgY2daVlFzaVB3ZFk/edit
[09:43] <Cimi> Saviq, cool
[09:43] <Cimi> Saviq, was bored of simply adding a label :)
[09:43] <Saviq> Cimi, make the see more / see less a component
[09:43] <Saviq> Cimi, we'll be using it in the dash as well
[09:44] <Cimi> Saviq, ok
[09:44] <Saviq> Cimi, I'm adding "title" to the text widget JSON
[09:44] <Saviq> Cimi, I'd say optional
[09:44] <Saviq> Cimi, so hide if empty
[09:44] <Cimi> Saviq, so textSummary needs what?
[09:44] <Cimi> column of two labels
[09:45] <Cimi> plus see more/less
[09:45] <Cimi> ?
[09:45] <Saviq> Cimi, yes, and it needs to accept a modelData object for its data, not text directly
[09:46] <Cimi> Saviq, but no array
[09:46] <Saviq> Cimi, no, a single value, yes
[09:46] <Cimi> Saviq, var modelData then?
[09:46] <Saviq> Cimi, yeah, or so
[09:46] <Saviq> Cimi, make it an interface, btw
[09:46] <Saviq> Cimi, so PreviewWidget { property var data }
[09:47] <Saviq> Cimi, or so
[09:47] <Saviq> Cimi, and base all of the preview widgets off of that
[09:47] <Cimi> adding an interface to all widgets you mean
[09:47] <Cimi> ok
[09:47] <Saviq> Cimi, yup, the only common prop between widgets will be data, I'd say
[09:47] <Saviq> Cimi, also, use Text.Wrap
[09:47] <mhr3> Saviq, any chance to try to hookup the new previews today?
[09:48] <mhr3> although we landed columns api, should change the models to support it
[09:48] <Saviq> mhr3, I'd rather have at least one widget first ;)
[09:48] <mhr3> yea, that would help i guess :)
[09:48] <Saviq> Cimi, otherwise long words will overflow / get clipped
[09:49] <Saviq> mhr3, so Monday? I reworked a few things with the Card (+CardTool), so carousel will work now
[09:50] <Saviq> meaning when I complete the tests and merge it ;)
[09:50] <Saviq> and will make sure today grid is centered
[09:50] <mhr3> Saviq, k, re columns, i'll make scope.preview() return a master preview model that will have columns role which will have per-column widget models
[09:51] <mhr3> widget model*
[09:51] <Saviq> mhr3, yup, sounds good
[09:51] <Saviq> mhr3, so we'll wrap it in a stack of previews later?
[09:51] <mhr3> Saviq, activations and stuff will go to the master object though
[09:51] <Saviq> mhr3, fine
[09:51] <mhr3> Saviq, still not sure where to put the stack
[09:52] <Saviq> mhr3, mhm
[09:52] <mhr3> Saviq, i guess the master will emit a signal when it gets a nested child
[09:52] <mhr3> s/nested/stacked/
[09:52] <Saviq> mhr3, mhm
[09:53] <mhr3> so don't think we really need to expose the entire stack to the shell
[09:53] <mhr3> always just the active one
[09:54] <mhr3> but maybe you don't think so? :)
[09:55] <Saviq> mhr3, well, I don't want to destroy the inactive one
[09:55] <Saviq> mhr3, since there can be data the user entered
[09:55] <mhr3> true
[09:55] <Saviq> mhr3, and they expect it there when, went "back" from the stacked one
[09:57] <mhr3> i just find the model hierarchy pretty complex already, but well
[09:57] <Saviq> mhr3, well, it's modeling a complex thing, so...
[10:00] <mhr3> Saviq, ok, so scope.preview() returns PreviewStack, that contains PreviewModel, which has columns and those have PreviewWidgetModel which finally has the individual widgets
[10:00] <sil2100> mhr3: give me a moment and I'll check
[10:05] <dednick> Saviq: is unity8.shell.tests.test_system_integration.SystemIntegrationTests.test_networkmanager_integration a known failure at the mo?
[10:08] <dednick> Saviq: http://pastebin.ubuntu.com/6848599/
[10:08] <Saviq> dednick, https://code.launchpad.net/~mterry/unity8/disable-nm-test/+merge/203980
[10:09] <dednick> Saviq: that was just using dpkg-buildpackage
[10:09] <Saviq> dednick, uh, what that...
[10:10] <dednick> :/ it was second time was building. maybe not cleaned?
[10:10] <karni> Saviq: if there's been any chages to card-tool brach, push 'em up please, reading now :)
[10:11] <Saviq> karni, no, not yet
[10:11] <Saviq> dednick, semop is "systemv semaphore operations"...
[10:11] <dednick> Saviq: hm. i had to ctrl-C out. build got stuck.
[10:11] <Saviq> dednick, try a clean one, or better yet - https://wiki.ubuntu.com/SimpleSbuild
[10:12] <Saviq> dednick, it builds everywhere else, so...
[10:13] <karni> ack
[10:13] <dednick> yay. an updated [possibly] not impossible to understand sbuild wiki!
[10:14] <Saviq> dednick, oh, didn't you see it before?
[10:14] <mhr3> Saviq, btw ok to push http://paste.ubuntu.com/6848630/ ? it doesn't exactly work, but makes it easier to fix :)
[10:14] <dednick> Saviq: only this on. https://wiki.ubuntu.com/SecurityTeam/BuildEnvironment
[10:14] <Saviq> mhr3, fine
[10:15] <Saviq> dednick, yeah, I extracted the info useful for the normal person in the SimpleSbuild one
[10:16] <karni> :D
[10:17] <Saviq> dednick, https://wiki.ubuntu.com/CrossBuilding some info about cross-building here, too
[10:18] <Saviq> dednick, unity8 is cross-buildable through that
[10:18] <Saviq> didrocks, maybe you could you do a quick checkup on packaging changes for:
[10:18] <Saviq>  https://code.launchpad.net/~saviq/unity8/add-doxygen/+merge/204123
[10:18] <Saviq>  https://code.launchpad.net/~saviq/unity8/packaging-and-cleanup/+merge/203829
[10:18] <dednick> Saviq: ta
[10:19] <seb128> Saviq, just gave a +1 to the first one
[10:20] <seb128> Saviq, I'm unsure about the second one, maybe ping Mirv? (did that package got renamed in 5.2?)
[10:20] <seb128> Saviq, well unsure in the sense of "I don't know why you have the alternative name, I didn't follow the qt packaging enough to say if that's correct"
[10:21] <Saviq> seb128, see http://packages.ubuntu.com/saucy/qtdeclarative5-private-dev vs. http://packages.ubuntu.com/trusty/qtdeclarative5-private-dev
[10:21] <Saviq> seb128, qtdeclarative5-private-dev in trusty depends on libqt5v8-5-private-dev
[10:22] <Saviq> seb128, and libqt5v8-5-private-dev doesn't exist for qt >= 5.2, it's "built into" qtdeclarative5-private-dev
[10:22] <seb128> Saviq, ok, thanks for the pointer, makes sense now
[10:22] <seb128> Saviq, +1 that one as well
[10:22] <Saviq> seb128, thanks
[10:23] <dednick> Saviq: dpkg not generating unity8-doc package
[10:24] <Saviq> dednick, :/
[10:24] <dednick> need dh_install rule as well for doc?
[10:24] <Saviq> dednick, https://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-amd64/2792/artifact/work/output/
[10:24] <Saviq> dednick, jenkins disagrees
[10:24] <seb128> dednick, building from the vcs linked previously?
[10:24] <seb128> dednick, you have the new .install?
[10:24] <dednick> Saviq: it's building, but not generating the package
[10:25] <dednick> Saviq: no, just using dpkg still. Working on sbuilder env now
[10:26] <Mirv> seb128: yes Qt's V8 is no more, it's replaced by V4 insside qtdeclarative.
[10:26] <dednick> Saviq: hm, actually, it's not giving me autopilot either....
[10:26] <Mirv> Saviq: but, qtdeclarative5-private-dev 5.0.2 depends on libqt5v8-5-private-dev
[10:26] <Mirv> nowadays, to avoid that need
[10:26] <Saviq> Mirv, not in saucy
[10:26] <Saviq> Mirv, that change is for saucy
[10:26] <Saviq> Mirv, https://code.launchpad.net/~saviq/unity8/packaging-and-cleanup/+merge/203829
[10:26] <seb128> dednick, seems like you don't build arch all packages?
[10:27] <Mirv> Saviq: aha.. a stable release backport, no PPA usage? the SDK Release PPA has qtdeclarative for saucy too that has that.
[10:27] <dednick> seb128: er. dpkg-buildpackage -us -uc -B ?
[10:27] <Saviq> dednick, drop -B
[10:27] <Mirv> (and precise/quantal/raring)
[10:27] <dednick> -B not
[10:27] <dednick> hm
[10:27] <seb128> dednick, -B is the issue indeed
[10:28] <Mirv> Saviq: https://launchpadlibrarian.net/159906274/qtdeclarative-opensource-src_5.0.2-6ubuntu4_5.0.2-6ubuntu5~saucy1~test1.diff.gz
[10:28] <dednick> ah, no arch-independant
[10:28] <Saviq> Mirv, hmm we require the sdk ppa anyway, maybe that change isn't needed then
[10:28] <dednick> doh
[10:28] <dednick> 3rd times a charm!
[10:28] <Mirv> Saviq: yes we require SDK PPA if it's about developers
[10:29] <Saviq> Mirv, ok, /me undo's
[10:29] <Mirv> thanks
[10:29] <seb128> Mirv, thanks for reviewing it ;-)
[10:30] <Saviq> seb128, Mirv, done
[10:32] <didrocks> Saviq: sorry, too many fuzz here
[10:32] <didrocks> thanks seb128 for the review :)
[10:32] <seb128> didrocks, yw!
[10:32] <Saviq> didrocks, no worries, got many eyes on it already :)
[10:33] <Mirv> approved
[10:33] <Saviq> Mirv, seb128, thanks!
[10:33] <Saviq> dednick, pushed OPTIONAL for the install of doc, so that cmake doesn't complain if you didn't build it
[10:40] <Cimi> Saviq, interface https://code.launchpad.net/~unity-team/unity8/unity8.previews_audioPlayer/+merge/203387
[10:41] <Saviq> Cimi, 55	+ model: data
[10:41] <Saviq> Cimi, that's wrong
[10:41] <Saviq> Cimi, you will get the whole object as described in the docs
[10:41] <Saviq> Cimi, so you need data["tracks"]
[10:41] <Cimi> damm
[10:42] <Saviq> Cimi, also, be explicit - name the top level item
[10:42] <Saviq> Cimi, and reference data  by that name
[10:42] <Cimi> root.data
[10:42] <Cimi> ok
[10:42] <Saviq> Cimi, yup, something like that
[10:42] <Cimi> Saviq, data["tracks"] will contain the model then?
[10:43] <Cimi> so I'll have to redo the mocks?
[10:43] <Saviq> Cimi, yes
[10:43] <Saviq> Cimi, well, not "redo", just wrap them in { "tracks": [] }
[10:43] <Cimi> Saviq, got a link handy
[10:43] <Cimi> of the json?
[10:44] <Saviq> Cimi, https://docs.google.com/a/canonical.com/document/d/1NmiM4UCnJgf6IEawmfyTOHRNAA5ZGrqpyrPqPOibwc8/edit#heading=h.6fng39a8onr
[10:44] <dednick> Saviq: does doxygen not generate for the qml files?
[10:44] <Saviq> dednick, it does, when they're documented
[10:44] <Saviq> dednick, Carousel should be there
[10:44] <dednick> Saviq: ahh
[10:44] <Saviq> dednick, I've hidden non-documented files, thought it didn't make sense otherwise
[10:45] <Saviq> Cimi, 405	+ audioPlayer.stop() I don't think that's needed - and if it would be, then it needs to be part of the interface
[10:45] <Saviq> Cimi, or maybe not..
[10:45] <Cimi> Saviq, currently
[10:45] <Cimi> Saviq, on current previews
[10:45] <Cimi> Saviq, there's a stop when the preview isCurrent turns falser
[10:46] <dednick> Saviq: they're not listed under 'Files', only 'Classes'
[10:46] <Cimi> Saviq, since this is no longer top level preview but a widget, thought of adding a function
[10:47] <Saviq> dednick, yeah, that seems a limitation of doxyqml
[10:48] <Saviq> dednick, or well, it doesn't add .cpp files either
[10:48] <Saviq> dednick, I'll have a chat with agateau (maintainer of doxyqml)
[10:48] <Saviq> Cimi, I'm just thinking what would be calling it
[10:49] <Cimi> Saviq, the preview page
[10:49] <dednick> Saviq: approved.
[10:49] <Saviq> Cimi, yeah, but that would basically mean that the preview should go away, so the player would get destroyed anyway
[10:49] <Cimi> Saviq, do they go away?
[10:49] <Cimi> Saviq, or they are listviews?
[10:50] <Saviq> Cimi, there's only 3 in a ListVIew
[10:50] <Saviq> Cimi, so if you move away by more than one
[10:50] <Saviq> Cimi, it gets destroyed
[10:50] <Saviq> Cimi, only use case I can think of is when you get a preview on top of your current one - but whether it should pause I'm not sure
[10:50] <Cimi> Saviq, but you want to move one, not two
[10:50] <Cimi> Saviq, you switch album
[10:50] <Cimi> and it should stop
[10:50] <Saviq> Cimi, yes, I know
[10:51] <Saviq> Cimi, just wondering whether it should really get destroyed or not
[10:51] <Saviq> Cimi, anyway, so let's say PreviewWidget gets a new property
[10:51] <Saviq> Cimi, "focused"
[10:51] <Cimi> ok
[10:51] <Saviq> Cimi, and preview widgets react to that
[10:54] <mhr3> sil2100, so what's the news on the branch?
[10:54] <mhr3> sil2100, you gave us a silo and that mp would like to be landed
[10:57] <sil2100> mhr3: ok, so I poked fginther about it, since it seems it's something stranger
[10:57] <sil2100> mhr3: oh, and we just recently published some silo, right?
[10:58] <Cimi> Saviq, data seems to cause troubles as a name
[10:58] <Cimi> var name
[10:58] <Cimi> can I change it to something else?
[10:58] <Cimi> previewData ?
[11:02] <fginther> sil2100, mhall119, the MP is failing to merge because jenkins is complaining that it has an unapproved pre-requisite, but that's not the case (it's actually merged). Will need to dig more.
[11:02] <sil2100> mhr3: ^
[11:02] <sil2100> ;)
[11:02] <sil2100> fginther: thanks!
[11:03] <mhr3> fginther, so i'll just remove the pre-req since it's merged
[11:04]  * mhr3 really wants it in
[11:05] <Saviq> Cimi, right, of course
[11:05] <Saviq> Cimi, widgetData
[11:05] <Saviq> Cimi, "data" is an internal QML name
[11:06] <Saviq> Cimi, sorry about that
[11:07] <fginther> mhr3, can I re-approve that MP? the last push to merge to trunk does not match the approved version.
[11:08] <mhr3> fginther, i resubmitted it a minute ago
[11:08] <mhr3> and approved
[11:09] <mhr3> so should be good afaict
[11:10] <fginther> mhr3, ack. I'll watch it in case it doesn't start
[11:15] <Saviq> Cimi, you could add some docs - we'll be generating them soon enough
[11:15] <Saviq> Cimi, see http://bazaar.launchpad.net/~saviq/unity8/card-tool/revision/677 for an example
[11:27] <Saviq> /food
[11:29] <anpok> i think you client does not suppor that
[11:29] <anpok> *your
[11:34] <dandrader> greyback, hey. it unity8 already using the mir socket and protocol as a general purpose mechanism to exchange information with apps (and vice versa)?
[11:34] <dandrader> s/it/is
[11:41] <karni> Saviq: FYI I'm on app/sdk sprint next week in Orlando, my presence may vary
[11:57] <Saviq> karni, ok
[11:57] <Saviq> karni, will you get us an updated look at your guys' priorities?
[11:58] <karni> Saviq: just noticed something interesting on my phone. flashed trusty-proposed, added demo-stuff ppa, rebooted, still see old scopes
[11:58] <karni> Saviq: yes, will do
[11:58] <Saviq> karni, apt-cache policy unity8?
[11:59] <Saviq> anpok, ;)
[11:59] <karni> Saviq: looks correct http://paste.ubuntu.com/6849035/
[12:00] <Saviq> karni, no it doesn't, there's a newer unity8 from distro than there is from ppa
[12:00] <karni> oh, indeed
[12:01] <mhr3> didrocks, deps between landed packages not really handled? https://launchpad.net/~ci-train-ppa-service/+archive/landing-009/+packages
[12:01] <karni> hrm
[12:01] <Saviq> karni, probably because I didn't merge trunk into new-scopes
[12:01] <Saviq> karni, pushed, kicking build now
[12:02] <karni> Saviq: thank you, sir!
[12:02] <mhr3> didrocks, what will i have to do once the dep pkg publishes?
[12:04] <Saviq> mhr3, click "merge & clean"
[12:05] <Saviq> mhr3, and for the deps, just click "retry this build"
[12:05] <Saviq> mhr3, oh no you can't
[12:05] <Saviq> didrocks, ↑ valid issue
[12:05] <mhr3> yea, i can't
[12:06] <mhr3> the good news is that the build today was about 20x faster than last time
[12:07] <didrocks> mhr3: it's in dep wait
[12:07] <mhr3> didrocks, yes, and the dep built now
[12:07] <didrocks> mhr3: ah, you have to wait for a cronjob IIRC in launchpad
[12:07] <didrocks> it's not triggered by signals)
[12:07] <didrocks> but launchpad is handling that
[12:07] <mhr3> didrocks, does that really work in ppas?
[12:08] <didrocks> AFAIK, it does
[12:08] <Saviq> didrocks, yeah, I've never seen "dependency wait" to switch to "needs build" by itself
[12:09] <Saviq> let's see, then...
[12:09] <dednick> Saviq: just trying to understand unity-mir at the mo. how does mir know where to draw the surface? ie how does it know its position given a stage? Can't find unity telling it anywhere...
[12:09] <Saviq> dednick, currently unity-mir has it hardcoded
[12:09] <didrocks> Saviq: mhr3: you can ping wgrant about it (I'm ddosing on pings right now)
[12:09] <dednick> Saviq: ahh. i c.
[12:09] <Saviq> ;D
[12:10] <dednick> thought i was going a bit mad
[12:10] <mhr3> didrocks, k
[12:10] <Saviq> @unity: any branches you guys would like landed? There's five in queue currently
[12:11] <Saviq> elopio, can you please fill checklist on https://code.launchpad.net/~elopio/unity8/generic_preview/+merge/202793
[12:11] <dednick> mzanetti: did you take a look at https://code.launchpad.net/~nick-dedekind/unity8/plugin-qml/+merge/203810 ?
[12:12] <Saviq> dednick, mzanetti's refusing to approve branches before CI goes green again ;
[12:12] <Saviq> ;)
[12:12] <dednick> ahha. dang
[12:12] <dednick> then no, i guess
[12:12] <Saviq> dednick, which it will with this lannding
[12:12] <karni> Saviq: which row of landing spreadsheet where you referring to?
[12:12] <dandrader> Saviq, no
[12:13]  * karni doesn't see unity8-fix-mascot in the sheet
[12:13] <Saviq> karni, none, yet, just https://code.launchpad.net/~unity-team/unity8/trunk/+activereviews
[12:13] <karni> ah
[12:13] <mzanetti> dednick: I looked at it, but CI seems still red
[12:13] <karni> cool
[12:13] <dednick> mzanetti: no worries, no rush on that one
[12:16] <Saviq> sil2100, I can has silo for unity8?
[12:17] <sil2100> Saviq: let me look at that
[12:17] <Saviq> sil2100, thanks
[12:18] <Saviq> karni, I'll have patched (blurriness) qtbase packages for you in a few minutes
[12:18] <karni> Saviq: w00t!
[12:23] <Saviq> karni, new unity8-newscopes built in demo-stuff for amd64/i386, soon there for armhf
[12:23] <sil2100> Saviq: so, we have a limited number of silos right now, so it might take a while at most - we're trying to bump the number of silos anyway, but it's something not for today
[12:23] <karni> Saviq: very neat, very neat :)
[12:23] <Saviq> sil2100, so no spot for us yet? ;(
[12:24] <Saviq> sil2100, 002, 007 and 010 look empty? are they queued for someone or waiting for cleanup?
[12:24] <Saviq> sil2100, not complaining, just trying to get more data
[12:27] <Saviq> thostr_, "When you allocate a silo just go ahead and hit build at the same time.  Thanks!" delegating already, nice one!
[12:28] <thostr_> Saviq: ?
[12:29] <thostr_> Saviq: the comment. yes, I think that is a pattern we should apply
[12:29] <Saviq> thostr_, indeed, no point in assigning a silo without building, agreed
[12:29] <thostr_> Saviq: at least I started this and e.g. initiated bfillers requests, so when he starts his day he should have all his packages built
[12:30] <Saviq> thostr_, yup
[12:47] <sil2100> Saviq: so, I had a talk with Didier, and I would be grateful if you could wait with silo assignment for unity8 for now
[12:48] <sil2100> Saviq: just a little bit ;)
[12:49] <Saviq> sil2100, ok, can you let me know before you'd assign them in case we'd like another branch to get in?
[12:49] <sil2100> Saviq: sure!
[12:49] <Saviq> dednick, you have some time - merge the test fix and resubmit your branch with prerequisite if you want it in! ;D
[12:56] <Saviq> aargh stupid optipng ;/
[12:57] <Saviq> Mirv, you really need to disable OPTIPNG for qt packages... it's a half hour already here that it's built and just mangling already-optimized pngs...
[12:57] <Saviq> Mirv, or something...
[12:58]  * Saviq finds a way to disable optipng system-wide
[12:59] <karni> Saviq: card-tool -- looking good :) let me know when you want further feedback/when it goes out of WIP
[12:59] <Saviq> karni, yeah, I just need to add some tests, will happen soon (I hope)
[13:00] <karni> very cool
[13:01] <Mirv> Saviq: it could be similarly temporarily disable to how I'm temporarily enabling debug options
[13:02] <Mirv> Saviq: I know I've bookmarked your disabling trick _somewhere_ :)
[13:02] <Saviq> Mirv, I just feel like it's wasting so much builders' time
[13:02] <Saviq> Mirv, export NO_PKG_PNG_MANGLE=1
[13:02] <Saviq> export NO_PNG_PKG_MANGLE=1
[13:02] <Saviq> PKG_PNG vs. PNG_PKG :/
[13:05] <mhr3> sil2100, more train questions
[13:05] <elopio> Saviq: I already did: https://code.launchpad.net/~elopio/unity8/generic_preview/+merge/202793/comments/475007
[13:05] <Saviq> elopio, yeah, saw that
[13:05] <Saviq> elopio, please put submitter checklists in description
[13:05] <Saviq> elopio, reviewer in comments, as there may be more
[13:05] <Saviq> elopio, I copied it already
[13:05] <elopio> Saviq: ok. Thanks.
[13:06] <mhr3> sil2100, the build failed, do i need to check any of the options now when i try to rebuild?
[13:10] <Cimi> Saviq, it's fine now https://code.launchpad.net/~unity-team/unity8/unity8.previews_audioPlayer/+merge/203387
[13:11] <Saviq> Cimi, \brief not needed if only one line
[13:11] <Cimi> ok
[13:11] <sil2100> mhr3: no, you can specify what specific packages you want to rebuild but you don't have to
[13:12] <Cimi> done
[13:12] <Saviq> Cimi, no tests for subtitle, but otherwise looks good, I'll leave to Albert to review properly on Monday
[13:12] <Cimi> Saviq, how can I test subtitle?
[13:12] <Cimi> oh sorry
[13:12] <mhr3> sil2100, well, i need to rebuild the one that succeeded cause that broke the one that failed :)
[13:12] <Cimi> subtitle I think I tested it
[13:13] <Saviq> Cimi, not there in mocks?
[13:13] <Saviq> Cimi, wrong, it's there
[13:13] <Saviq> Cimi, sorry, it's good
[13:13] <Cimi> Saviq, time is not tested
[13:13] <Cimi> Saviq, not sure how I can test it
[13:13] <Cimi> Saviq, because it's processed
[13:13] <Cimi> from seconds I have a string
[13:13] <Saviq> Cimi, well, yeah, you should supply one value and check that the string is what you expect?
[13:13] <mhr3> sil2100, so i just click the rebuild without checking any of the options?
[13:14] <Saviq> Cimi, that's the easiest thing to test :) - input, output
[13:14] <Saviq> Cimi, have a separate test function for it
[13:14] <Cimi> Saviq, yes but that will mean adding an extra field to the mock
[13:14] <sil2100> mhr3: yep :)
[13:15] <Cimi> I wanted to keep the mock like the one it will receive
[13:15] <Cimi> but I can do it
[13:15] <sil2100> mhr3: it will then take all the components, fetch latest revisions of the branches and build
[13:15] <Saviq> Cimi, wait
[13:15] <Saviq> Cimi, no, no more fields in mocks, why would you need it there?
[13:15] <Cimi> Saviq, I don't know wehre
[13:15] <Cimi> if not there
[13:15] <Cimi> it's the easiest...
[13:15] <karni> Saviq: unity-scope-tool requires 14.04, right?
[13:15] <Saviq> karni, should be fine in 13.10
[13:16] <karni> ack
[13:16] <Saviq> Cimi, so well, I wouldn't have gone your route of just putting the model in _data
[13:16] <Saviq> Cimi, but instead I'd go with three separate test functions for the three different models
[13:16] <Saviq> Cimi, and then in _data() you put the expected values
[13:16] <Saviq> Cimi, one of which would be the expected processed time
[13:17] <Saviq> Cimi, but even now you can just add an array to {tag: "0 tracks", tracksModel: tracksModel0},
[13:17] <Cimi> I can add an array to _data for expected length in case
[13:17] <Saviq> Cimi, yeah, it's a bit convoluted, but yeah
[13:17] <mhr3> sil2100, can i make it build just one source pkg and the other later?
[13:17] <Saviq> Cimi, and to really test the **it out of that function
[13:18] <mhr3> sil2100, cause i know right now that the second one will fail until the first one doesn't build
[13:18] <Saviq> Cimi, ah, take out the lengthToString() function from under the Repeatre
[13:18] <Saviq> Cimi, have a separate _data() and test_ pair, put it in Audio for example - we only need one copy of that function
[13:19] <Saviq> Cimi, then, have a separate _data() and test_ pair, findChild() the Audio component
[13:19] <Saviq> Cimi, and just test what you get out when supplying undefined, 0, -1, 1, 60, 120, 3600, 7201 etc.
[13:20] <Cimi> ok
[13:20] <Saviq> so your _data() would look more or less:
[13:21] <Saviq> [ { tag: "0", value: 0, result: "0:00" }, { tag: "-1", value: -1, result: undefined } ] or so
[13:21] <Cimi> sure
[13:27] <Saviq> https://code.launchpad.net/~saviq/unity8/card-tool/+merge/204093 ready for review
[13:28] <Saviq> karni, actually let me do one more thing
[13:29] <Saviq> karni, or well, a few more things...
[13:30] <didrocks> mhr3: Saviq: see, it retried automatically
[13:31] <Saviq> didrocks, yeah, an hour - too long ;)
[13:31] <Saviq> didrocks, we might need a button "retry builds"
[13:31] <karni> Saviq: ack, ping me when ready :)
[13:31] <didrocks> Saviq: or fix launchpad to get a quicker retry?
[13:31] <Saviq> didrocks, or that ;)
[13:32] <didrocks> in the archive, it's even based
[13:32] <didrocks> not sure why the ppas can't
[13:37] <mhr3> Saviq, removing rating-input::submit-icon per katie's comment
[13:37] <Saviq> mhr3, ok
[13:43] <Saviq> karni, ready
[13:43] <karni> ack
[13:44] <karni> Saviq: do you want to still own it, or pass to unity-team?
[13:45] <karni> Doesn't matter for me, just a check.
[13:45] <Saviq> karni, we only push as ~unity-team if we foresee someone else needing to commit to it
[13:45]  * karni nod
[13:45] <karni> Saviq: neat way to OWN stuff ;)
[13:47] <Saviq> karni, crap, it's 300MB of .debs for qtbase...
[13:47] <karni> hahahhaha
[13:48]  * Saviq tests his pipe's upload capabilities
[13:48]  * karni chuckles
[13:48] <Saviq> 1.2MB/s, it's fine ;P
[13:48] <karni> :D
[13:48] <Saviq> 3 minutes
[13:51] <karni> Saviq: Just a thought, not for this MP - I think that referencing test cases by index (from the drop down component) is error prone. One has to keep in mind which case are you referencing. Adding a test case within (instead of at the end) would screw test suite beautifully. Maybe something we can get better one day?
[13:51] <karni> Less things you keep in mind when coding = better code (well.. you get the point)
[13:51] <Saviq> karni, we can probably id them by name instead of index somehow
[13:51] <Saviq> karni, can definitely be improved
[13:51] <karni> Saviq: yes, that's what I mean
[13:51]  * karni nods
[13:53] <karni> Saviq: waiting for CI run
[13:53] <Saviq> karni, yup
[13:54] <Saviq> karni, http://people.canonical.com/~msawicz/qtbase_amd64.tar.xz
[13:54] <karni> \o/
[13:54]  * Saviq likes his pipe
[13:54] <Saviq> karni, armhf is still in the works, optipng will probably take a while still :?
[13:54] <karni> :)
[13:56] <Saviq> we have more qml tests in unity8 now that employees in Canonical ;D
[13:56]  * Saviq likes :D
[13:56] <Saviq> even if we did cross the magical 600 employees boundary
[13:57] <Saviq> 611 today
[13:57] <karni> Saviq: downloaded
[13:57] <karni> Saviq: hahah
[13:58] <karni> Saviq: wow, number of packages. do you remember which one contained the fix, or should I just install em all x)?
[14:00] <Saviq> karni, for i in $( dpkg -l `for i in *.deb; do echo $i | cut -d_ -f1; done` 2>/dev/null | grep 5.0.2 | cut -d' ' -f3 | cut -d':' -f1 ); do echo -n "$i*.deb "; done; echo
[14:02] <karni> Saviq: I also like bash, but you just slaped the 'awesome' sticker on your 'already-awesome' perception from my POV haha
[14:02] <Saviq> ;)
[14:02] <Saviq> karni, thought you'd enjoy that
[14:02] <karni> Saviq: btw that last echo was intentional or got trimmed?
[14:03] <karni> oh yeah :)
[14:03] <Saviq> karni, just so you don't end up with prompt at the end of the lsit
[14:03] <Saviq> list
[14:03] <karni> :D
[14:05]  * karni joined scope sync with Scott and Joe
[14:06] <mhr3> sil2100, so, the prereq package re-built, how do i make retry building the second one?
[14:06] <mhr3> sil2100, i don't have powers that be to see a "retry this build" for the ppa
[14:10] <sil2100> mhr3: I'll try that, but it also happens automatically - just it probably takes some time ;/
[14:10] <sil2100> mhr3: we're thinking on how to deal with this
[14:10] <mhr3> sil2100, not the same thing, the build failed, it isn't in dep wait
[14:11] <sil2100> mhr3: ah, you mean, it failed and you just want to rebuild it? You can always do it from the Build job - just list the source package name you want to rebuild
[14:12] <mhr3> sil2100, into the prepare-only?
[14:13] <sil2100> mhr3: yes, it will then fetch all the merges related to that source and push them to the PPA again
[14:13] <mhr3> sil2100, since the mps didn't change do i need "force-rebuild"?
[14:14] <sil2100> mhr3: no, it's fine
[14:14] <mhr3> ok, let's see what happens
[14:14] <sil2100> At least in theory
[14:14] <sil2100> ;)
[14:27] <karni> Saviq: thostr_: Who would we need to talk to about porting news and weather scopes to the new server scope api?
[14:28] <karni> Cc ssweeny
[14:28] <thostr_> karni: lucio
[14:28] <karni> ack, thank you thostr_
[14:28] <thostr_> karni: but hurry up he's only here today, then he goes on vacation
[14:28] <karni> ouch
[14:28] <karni> omw
[14:32] <mzanetti> @unity standup
[14:32] <dandrader> mzanetti, already there
[14:32] <mzanetti> it should be possible to exclude people :)  like  @unity -dandrader :D
[14:33] <karni> mzanetti: lol
[14:33] <Saviq> like that'd work ;D
[14:33] <mzanetti> :D
[14:33] <Saviq> greyback, standup
[14:33] <karni> @greybackrocks
[14:33] <greyback> darn
[14:40] <Cimi> Saviq, added tests
[14:40] <Saviq> Cimi, o/
[14:40] <dednick> elopio: what did you need help with? sorry, i missed your request.
[14:41] <elopio> dednick: qt5.2 errors.
[14:41] <dednick> elopio: in qmltests?
[14:42] <elopio> dednick: I'm just starting to look at them. https://bugs.launchpad.net/unity8/+bug/1274484
[14:42] <elopio> Mirv reported 5 autopilot errors. I'll run the unit tests, and the autopilot tests and report some more detailed bugs.
[14:43] <karni> Saviq: Card size was applied only to first card of the carousel. While I believe I do understand your branch well, I would not understand why card size would not apply to all carousel Cards before.
[14:43] <Saviq> karni, because it was set wrong
[14:44] <Saviq> karni, it needs to be "synced" between the delegates and the carousel itself
[14:44] <Saviq> karni, so it got confused (chicken'n'egg issue)
[14:44] <karni> Saviq: Well, I can't be happier than, can I! :)
[14:45] <Saviq> karni, so yeah, I wanted to wait until this lands in trunk and merge into new-scopes and integrate CardTool into CardFilterGrid, CardCarousel and friends
[14:45] <karni> Saviq: that's a blast!
[14:45] <Saviq> karni, but might need to change the plans if we don't get a silo for a while still
[14:45]  * karni nods
[14:46]  * mzanetti goes away for an hour and a half
[14:53] <Saviq> karni, top-ack?
[14:53] <karni> sry
[14:53] <karni> Saviq: done
[14:53] <Saviq> thanks
[14:53] <karni> yw!
[14:55] <Saviq> dednick, can you please verify docs still work for you, and include qml sources now?
[14:55] <Saviq> dednick, and re-approve to pick up the new rev
[14:57] <dednick> Saviq: sure. give me a few minutes
[14:58] <Saviq> dednick, thanks
[15:02] <karni> Saviq: CardFilterGrid works fine rendering wise, maybe we could prioritize integration of CardTool into CardCarousel (DashCarousel, or whichever would be appropriate)?
[15:03] <karni> anyway, I'll try to work out the list of our needs asap, trying to multithread
[15:03] <Saviq> karni, it's at the top of my prio
[15:03] <karni> Saviq: <3
[15:03] <Saviq> karni, the next thing is to fix CardFilterGrid not being centered
[15:03]  * karni nods
[15:03] <karni> :)
[15:03] <Saviq> karni, I'll probably prepare a branch while we don't have cardtool in trunk
[15:04] <dednick> Saviq: cool, still works + qml is there. approved
[15:04] <karni> Saviq: great :)
[15:07] <Saviq> dednick, thanks
[15:11] <Saviq> karni, apparently forgot about carousel ;)
[15:12] <karni> Saviq: excuse me?
[15:12] <Saviq> karni, in CardTool
[15:12] <Saviq> karni, no carousel ;)
[15:12] <karni> d'uhhhhhh
[15:12] <karni> xDD
[15:12] <karni> I'm a terrible reviewer
[15:13] <karni> Saviq: since it's write protected with ~saviq :D can you still push up a fix?
[15:13] <Saviq> karni, sure, just cooking it up
[15:13] <karni> phew
[15:14]  * karni reboot
[15:32] <karni> Saviq: with all the work that you've put in, I'd really like to make it work here. I installed the packages your bash line returned, rebooted, mascot still blured. But don't let this take your focus away from CardTool related improvements!
[15:33] <Saviq> karni, must be it's actually a different issue
[16:08] <Saviq> sil2100, FYI unity-mir waiting for a silo now, too
[16:09] <sil2100> Saviq: ok, I guess soon we might figure out the regression and then we can resume normal landings
[16:09] <Saviq> sil2100, yup, thanks
[16:12] <Saviq> dednick, you around?
[16:12] <Saviq> dednick, ah, here you are :)
[16:23] <Saviq> karni, sorry for the delay, pushed card-tool
[16:23] <karni> Saviq: np, compiling notes from a meeting I just had
[16:25] <thostr_> sil2100: where I can see if you guys clicked the publish button?
[16:25] <thostr_> sil2100: (except trying to see if the package is available via launchpad)
[16:25] <sil2100> thostr_: it's usually saying then that it's migrating the package to the archive
[16:26] <thostr_> sil2100: can you do that for some of the silos?
[16:27] <thostr_> sil2100: seems like 5 silos are waiting for the publish button to be clicked?
[16:27] <sil2100> thostr_: yes yes, we didn't publish them yet since we're trying to resolve a regression first
[16:28] <thostr_> sil2100: ah, ok
[16:38] <karni> Saviq: sorry, handling other stuff along ETOOLITTLEHANDS will get to your review asap
[16:39] <Saviq> karni, no worries
[16:40] <Saviq> mhr3, hmm I'm not getting results in new scopes, that expected?
[16:40] <Saviq> mhr3, and btw "restart scope-registry" just killed my session a second ago ;)
[16:45] <mhr3> Saviq, how what why?
[16:46] <Saviq> mhr3, not sure, but it happened
[16:46] <Saviq> ⟫ restart scope-registry
[16:46] <Saviq> restart: Connection was disconnected before a reply was received
[16:46] <Saviq> mhr3, and my whole session went away
[16:46] <Saviq> mhr3, but anyway, unimportant ;P
[16:46] <Saviq> mhr3, no results in new scopes?
[16:46] <mhr3> are you running a unity8 session?
[16:47] <mhr3> Saviq, apt-cache policy libunity-scopes0
[16:47] <Saviq> mhr3, no, not unity8 session, 0.2.6+14.04.20140128-0ubuntu1
[16:48] <mhr3> ps aux | grep scoper
[16:48] <mhr3> Saviq, upstart crash i guess?
[16:48] <Saviq> mhr3, probably, yeah
[16:48] <Saviq> mhr3, I do get the scope, but no results
[16:48] <Saviq> mhr3, i.e. I get the header and the category
[16:49] <mhr3> then you broke the renderer :)
[16:49] <mhr3> cause no results = no category
[16:49] <Saviq> mhr3, if someone broke it, it's you ;)
[16:49] <Saviq> mhr3, I didn't push to new-scopes for a while
[16:49]  * Saviq tries again
[16:49] <Saviq> karni, something along those lines lp:~saviq/unity8/newscopes-card-tool should improve grid and carousel
[16:49] <mhr3> Saviq, i wanted to, but they don't want to publish the new versions :P
[16:49] <Saviq> karni, but I've no results yet :/
[16:50] <Saviq> karni, caution recommended
[16:50] <karni> Saviq: ack! will look asap, gotcha
[16:50] <Saviq> mhr3, ok, results are back
[16:51] <mhr3> Saviq, see, you just need to ping me, then it fixes itself :P
[16:52] <Saviq> mhr3, and I did break the renderer indeed
[16:52] <Saviq> ok /me goes for food now
[16:52] <mhr3> lol
[16:52] <mhr3> at least once i'm right :P
[16:56] <Cimi> Saviq, http://paste.ubuntu.com/6850317/
[16:56] <Cimi> Saviq, component name is ugly :D
[16:57] <Cimi> I called it SeeMore.qml
[17:02] <Saviq> karni, ok, fixed grid, it looks great now
[17:02] <karni> Saviq: WOHOOO
[17:02] <Saviq> karni, carousel needs tweaking, but it's ok, too
[17:03] <karni> And I have some friggin broke news for you guys ;D You'll get a laugh.
[17:03]  * karni preps a paste
[17:03] <Saviq> /food
[17:05] <karni> mhr3: (Cc ssweeny) http://paste.ubuntu.com/6850362/
[17:05] <karni> Saviq: for your entertainment ↑
[17:06] <karni> Saviq: mhr3: ignore the mascot: art line, it was unrelated when I tested this
[17:06] <mhr3> NO
[17:06] <mhr3> WAY
[17:06] <karni> I pushed the renderer.json to the phonee twice
[17:06] <karni> mhr3: want a screenshot? hahaha. I'm serious, tho :|
[17:07]  * karni tries this trick with scope-onlinemusic hehe
[17:07] <mhr3> i won't believe until i run that on my device before my very own eyes
[17:07] <mhr3> eh, *in front of
[17:08] <karni> mhr3: I'll ping you in a sec with a supposedly broken json file
[17:12] <Saviq> karni, whoa
[17:12] <karni> Trying to reproduce this on scope-onlinemusic
[17:13] <Saviq> karni, I can has the whole renderer.json in both versions?
[17:13] <karni> yes, in a sec
[17:21]  * karni takes time to reproduce this, I already said on another channel that's most ridiculous thing I ever said in programming
[17:22] <Saviq> karni, fixed carousel
[17:23] <karni> Saviq: you're amazing :)
[17:23] <Cimi> Saviq, quick opinion on that component? ^
[17:24] <Saviq> Cimi, use AbstractButton instead of the custom MouseArea
[17:24] <Saviq> Cimi, so AbstractButton { Label { } }
[17:24] <Cimi> Saviq, but I'll have to use style
[17:25] <Saviq> Cimi, AbstractButton is not visual
[17:25] <Cimi> Saviq, abstract button already has text property
[17:25] <Saviq> orly
[17:25] <Cimi> and ion
[17:25] <Cimi> icon
[17:25] <Cimi> but doesn't draw anything for me
[17:25] <Saviq> Cimi, yes, which is fine
[17:26] <Saviq> Cimi, it implements the behavior of a button
[17:26] <Saviq> Cimi, either way
[17:27] <Saviq> Cimi, no need for enabled: !
[17:27] <Cimi> Saviq, it's to reduce mouse areas
[17:27] <karni> Saviq: card-tool bzr pull -- branches diverged. you overwrote/uncommited something?
[17:27] <Saviq> karni, newscopes-card-tool is on top of new-scopes
[17:28] <Saviq> Cimi, yeah, but I'd rather eat that event and not react to it than to let it through
[17:28] <Saviq> Cimi, and it doesn't "reduce" them in any meaningful way
[17:28] <Cimi> Saviq, shall I really use abstract buttons?
[17:28] <Saviq> Cimi, I'd go s/seeMoreEnabled/seeMore/ or even s/seeMoreEnabled/more/
[17:28] <Saviq> Cimi, your call
[17:29] <Cimi> nope then
[17:29] <Saviq> Cimi, other than that property name it looks fine
[17:29] <Saviq> Cimi, and SeeMore.qml is fine with me
[17:29] <sil2100> mhr3: hi!
[17:30] <Cimi> Saviq, will add tests and push
[17:30] <Saviq> Cimi, how about some behaviour on opacity?
[17:30] <sil2100> mhr3: so, I published your changes and it seems I made a mess because of that
[17:30] <karni> Saviq: I wanted to +1 last two commits at https://code.launchpad.net/~saviq/unity8/card-tool/+merge/204093
[17:30] <mhr3> sil2100, how so?
[17:30] <karni> Saviq: should I be reviewing newscopes-card-tool instead.. ?
[17:31] <karni> nvm, I'll just clean branch it
[17:31] <Saviq> karni, no
[17:31] <Saviq> karni, review on card-tool is good
[17:31] <Saviq> karni, newscopes-card-tool is just prep for when card-tool gets into trunk
[17:31] <karni> ack
[17:31] <Saviq> karni, to actually use card-tool
[17:31] <sil2100> mhr3: tell me, why are you directly depending on libunity-scopes0 in debian/control on one of the packages in unity-scopes-shell ?
[17:31] <Saviq> karni, there's little on top of card-tool there
[17:32] <mhr3> sil2100, cause the pkg isn't maintaining proper abi compability yet
[17:32] <sil2100> mhr3: hm, what do you have in mind?
[17:33] <karni> Saviq: +1 card-tool
[17:33] <Saviq> karni, cool thanks
[17:33] <mhr3> sil2100, it deps on libunity-scopes0, but that keeps changing its abi, so we need to specify exact version to not get into a state where the are installed, but incompatible
[17:33] <karni> Saviq: thank you
[17:34] <Saviq> Cimi, I'd go for Behavior on just one of those labels
[17:34] <Saviq> Cimi, and then the other's opacity based on the first one
[17:34] <mhr3> sil2100, like it just upgraded just unity-plugin-scopes without upgrading libunity-scopes
[17:34] <Saviq> Cimi, so that they're always opposite, without the need for two Behaviors
[17:35] <mhr3> sil2100, if you just upgraded...*
[17:35] <sil2100> slangasek: ^
[17:35] <sil2100> slangasek: not sure if that's a proper reason, but I'm not sure how to resolve it in any other way then
[17:36] <slangasek> sil2100: "keeps changing its ABI" - ok, for the record I will not sign off on any library being uploaded to the Ubuntu archive that changes ABI without *at minimum* changing its binary package name
[17:37] <slangasek> I was aware that there were libraries currently being managed this way; this needs to stop
[17:37] <mhr3> slangasek, and it will once we declare the library stable
[17:37] <slangasek> I'm not going to block someone else signing off on it, but the currently-employed method also doesn't guarantee consistency
[17:39] <slangasek> mhr3: I don't consider "once we declare the library stable" to be sufficient.  These are baseline requirements for anything in the Ubuntu archive
[17:42] <mhr3> the library is under heavy development and we're forced to keep pushing it in the archive
[17:43] <mhr3> all its users are currently internal and they're well aware the api and abi unstable
[17:43] <slangasek> yes, but your current declarations fail to actually enforce consistency
[17:44] <slangasek> they only prevent the inconsistency in *one* direction but not the other
[17:44] <mhr3> we could have just dropped it and noone would ever know
[17:45] <mhr3> we chose to do at least one-way enforcing instead
[17:45] <Saviq> Cimi, on that note, are you sure you should be using raised/black everywhere?
[17:45] <Saviq> Cimi, I don't see that in the designs
[17:45] <Cimi> Saviq, no
[17:45] <Cimi> Saviq, will fix designs when we'll have something working
[17:45] <Cimi> s/designs/implementations
[17:46] <Saviq> Cimi, ok, drop the style at least, it's not there in the designs
[17:46] <Cimi> ok
[17:46] <Saviq> Cimi, and there's a separator you could add
[17:46] <Cimi> Saviq, how?
[17:46] <Cimi> Saviq, I thought about it
[17:46] <Cimi> rectangle?
[17:47] <slangasek> mhr3: it's absolutely not true that people wouldn't know.  Playing fast and loose with ABI declarations significantly impedes the CI infrastructure being able to automatically land the right thing
[17:47] <Saviq> Cimi, no, isn't it just a rotated ThinDivider?
[17:47] <Cimi> dunno
[17:48] <karni> Saviq: mhr3: For your entertainment, I can't get summary to show up on our custom scope at all (as opposed to sample "summary": "artist" mapping working just fine in scope-onlinemusic). I'll get back to you guys when I have less ridiculous news, I hope you enjoyed the read anyway ;)
[17:49] <karni> Saviq: Would you like me to review anything else (carousel fix)?
[17:50] <mhr3> slangasek, isn't this actually poor man's solution so that it actually does land the correct thing?
[17:50] <Cimi> Saviq, tried with hebaviour
[17:50] <Cimi> Saviq, doesn't look nice
[17:51] <Saviq> Cimi, ok
[17:51] <Saviq> karni, let me see if I can make this diff sane
[17:51] <karni> :)
[17:52] <mhr3> slangasek, eventually anyway :)
[17:54] <Saviq> karni, https://code.launchpad.net/~saviq/unity8/newscopes-card-tool/+merge/204301 looks ok
[17:54] <Saviq> karni, if you want a local diff, merge card-tool into new-scopes first
[17:56] <Saviq> karni, you need to re-top-approve card-tool, 'cause it holds the old revision as approved
[17:56] <karni> Saviq: done
[17:57] <Saviq> karni, tx
[17:59] <karni> ssweeny: This is comedy. I got the summary to appear, and now I can't get it to break. =_=
[18:02] <slangasek> mhr3: why are we resorting to poor-man's solutions, which have failure scenarios that we can actually handle?
[18:04] <Cimi> Saviq, any clever idea to animate maximumLineCount?
[18:04] <Saviq> Cimi, clip + behaviour on height
[18:05] <Cimi> Saviq, so I don't use it?
[18:05] <Saviq> Cimi, you do
[18:05] <Saviq> just when you change maximumLineCount
[18:05] <Saviq> Cimi, height is behaviourized
[18:05] <mhr3> slangasek, cause as you said, we're playing fast
[18:06] <Cimi> Saviq, adding behaviour on height doesn't change anything
[18:09]  * Saviq wants to get rid of DashRenderer so bad...
[18:10] <sil2100> Saviq: I'll assign a silo for unity8 for you after I eat something and get to the hotel, ok? :)
[18:10] <Saviq> sil2100, sure, no worries
[18:15] <Cimi> Saviq, thank you
[18:15] <Cimi> why is it so bad? :D
[18:15] <Saviq> Cimi, so does it work or not?
[18:15] <Cimi> Saviq, I was joking about dashrenderer
[18:16] <Cimi> Saviq, no, the behaviour doesn't work
[18:16] <Saviq> Cimi, because all the properties are going three ways
[18:16] <Saviq> Cimi, you might need height: implicitHeight, although that'd be weird
[18:16] <Cimi> Saviq, height changes
[18:16] <Cimi> Saviq, but it doesn't animate
[18:19] <Cimi> mmm
[18:19] <Cimi> this component requires a bit more work
[18:19] <Cimi> cannot finish it today
[18:20] <Cimi> it has to show SeeMore depending on the number or lines of text displayed
[18:20] <Cimi> I guess I'll do this monday
[18:20] <Cimi> :P
[18:22] <Cimi> Saviq, I can use height+clip instead of maximumLineHeight
[18:22] <karni> Saviq: mhr3: ssweeny: Sent you good and bad json renderer files, with more details, and debs to try.
[18:23] <karni> ssweeny: With that said, you should be able to make screenshots today.
[18:23] <Saviq> Cimi, http://qt-project.org/doc/qt-5.0/qtquick/qml-qtquick2-text.html#lineCount-prop
[18:23] <Cimi> Saviq, I know this
[18:23] <Cimi> Saviq, but height is fine
[18:24] <Cimi> Saviq, if description is taller than X height
[18:24] <Cimi> Saviq, I add seeMore that clips to X
[18:24] <Cimi> otherwise I don't set height
[18:24] <Saviq> Cimi, yeah, but that won't clip at line boundary
[18:24] <Cimi> or just set it to lineHeight*lineCOunt
[18:25] <Cimi> Saviq, I'll clip it smart
[18:25] <Saviq> Cimi, whenever you think about calculating something based on lineHeight, stop ;)
[18:25] <Cimi> Saviq, we could potentially fade with a gradient on last line
[18:25] <Saviq> Cimi, http://paste.ubuntu.com/6850737/ *almost* works
[18:26] <Saviq> Cimi, I forgot what I did to delay the clipping of text
[18:26] <Saviq> Cimi, but I did it somewhere already
[18:27] <Saviq> Cimi, if all else fails - ShaderEffect { live: false }...
[18:29] <Saviq> karni, I knew it
[18:29] <Saviq> "summary": "summmary"
[18:30] <karni> Saviq: + huge 1 on newscopes-card-tool
[18:30] <Saviq> TOOMANYMs
[18:30] <karni> you're fking with me..
[18:30] <Saviq> karni, 'fraid I'm not
[18:30] <Saviq> mhr3, ↑↑
[18:30] <karni> ssweeny: ... ↑
[18:30] <karni> ssweeny: you owe me a beer, and I owe myself a bitchslap for not noticing that.
[18:31] <karni> That's clear sign to EOW
[18:31] <Saviq> indeed
[18:31] <karni> Saviq: I won't comment on ETOOMANYMs :P
[18:31] <karni> Saviq: You've been great this week, thank you :)
[18:31] <Cimi> Saviq, I'm sure it works if we change height and not maximumLineCount
[18:32] <Saviq> Cimi, yes, but you can't know the correct height
[18:32] <Saviq> Cimi, well, there is one thing that can help
[18:32] <karni> I guess that's why I couldn't reproduce this problem, ssweeny. Because I CAN type "summary" :D hahahah And I'm blind.
[18:32] <Cimi> Saviq, lineCount * lineHeight?
[18:32] <Saviq> Cimi, http://qt-project.org/doc/qt-5.0/qtquick/qml-qtquick2-text.html#onLineLaidOut-signal
[18:32] <mhr3> karni, see, when i say NO WAY i mean no way :)
[18:33] <Cimi> sorry
[18:33] <Saviq> Cimi, you can do onLineLaidOut: and find out when the third line is laid out
[18:33] <Saviq> Cimi, and store that value in collapsedHeight
[18:33] <karni> mhr3: I couldn't believe myself. And I'm the one that can type "summary" correctly hahah
[18:33] <Cimi> Saviq, contentHeight?
[18:33] <karni> mhr3: Please, let's not get back to that ;D
[18:33] <Cimi> contentHeight is maximum
[18:34] <mhr3> karni, it's eow i'll cut you some slack :P
[18:34] <Cimi> each line is contentHeight / lineCount
[18:34] <Saviq> Cimi, property real collapsedHeight; onLineLaidOut { if (line.number <=3) collapsedHeight = line.y + line.height }
[18:34] <Cimi> Saviq, I want to try with contentHeight
[18:35] <Saviq> Cimi, that would probably work, too, yah
[18:35] <Saviq> or close enough
[18:36] <Cimi> Saviq, tested, works well
[18:36] <Cimi> Saviq, will do the automatic appear/disappear of seeMore controls on monday
[18:48] <Saviq> karni, one last treat for this evening https://code.launchpad.net/~saviq/unity8/newscopes-new-dash-look/+merge/204322
[18:48] <Saviq> karni, not really review-ready, but it's getting closer and closer
[18:51] <Saviq> karni, in scope-tool only, btw, actual shell/dash needs work
[18:58] <Saviq> aand EOW
[18:58] <Saviq> o/
[18:58]  * greyback hitting EOW too, have a good weekend all