| === jono is now known as Guest96931 | ||
| === _salem is now known as salem_ | ||
| Saviq | Wellark, hey, could we get a review of https://code.launchpad.net/~mterry/indicator-network/watch-nm-dbus/+merge/219705 and https://code.launchpad.net/~mterry/indicator-network/greeter-profiles/+merge/219897 please? | 07:37 |
|---|---|---|
| === zequence_ is now known as zequence | ||
| tsdgeos | Saviq: 7.85+14.10.20140428.2-0ubuntu1 ? | 08:07 |
| tsdgeos | it is back | 08:07 |
| Saviq | tsdgeos, yeah, that one we never properly removed | 08:08 |
| Saviq | tsdgeos, and we probably don't care enough | 08:08 |
| tsdgeos | ok | 08:08 |
| Saviq | tsdgeos, it got added by the train | 08:08 |
| Saviq | tsdgeos, some hiccup there | 08:08 |
| === salem_ is now known as _salem | ||
| tsdgeos | mhr3: ping | 09:02 |
| mhr3 | tsdgeos, pong | 09:03 |
| === vrruiz_ is now known as rvr | ||
| tsdgeos | mhr3: mzanetti is complaining in the unity-api thing why Scopes::get returns a QVariant and not a ScopeInterface* while MockPreviewStack::get returns a PreviewModelInterface* and not a QVariant | 09:04 |
| tsdgeos | mhr3: what do you think of changing the first get to return a ScopeInterface* ? | 09:04 |
| tsdgeos | i just mimic''ed what we had in the old code | 09:04 |
| tsdgeos | but maybe this is better? | 09:04 |
| mhr3 | tsdgeos, i think there's a more fundamental issue - what should the get() return | 09:07 |
| mhr3 | in some cases it's return a dict of all row properties | 09:07 |
| mhr3 | in others it's returning one specific thing | 09:07 |
| mzanetti | fwiw, all the other unity-api's return a non-variant-wrapped pointer to a single object | 09:08 |
| mzanetti | on get() that is | 09:08 |
| tsdgeos | mhr3: well i think it should return what it is, so if it's going to be a QVariantMap, write QVariantMap instead of QVariant, if it's a ptr, the same, return the ptr instead of a QVariant | 09:10 |
| tsdgeos | Saviq: your opinion ↑↑↑↑ ? | 09:10 |
| Saviq | tsdgeos, mhr3 has a point - 'get' is not an established thing - sometimes we return an object "behind" the model, sometimes the QVariantMap | 09:12 |
| mhr3 | tsdgeos, yea, sure, the question remains whether those get() methods should always return a QVariantMap | 09:12 |
| mhr3 | tsdgeos, imo best would be get() -> QVariantMap and anything not returning a QVariantMap should be getXyz() | 09:12 |
| Saviq | mhr3, I think yes, and if we want the object behind it, we should use getObject, getRaw or something (that's what's in notifications) | 09:12 |
| Saviq | mhr3, violent agreement then | 09:12 |
| mhr3 | indeed | 09:13 |
| tsdgeos | Saviq: mhr3: ok, i'll make get() return a QVariantMap and the ptr be getScope, getPreviewModel, etc | 09:13 |
| mhr3 | tsdgeos, cool, thx | 09:14 |
| Saviq | tsdgeos, mhr3, do we need the get() at all? | 09:14 |
| mhr3 | just came to my mind :) | 09:15 |
| Saviq | we should avoid it with fire | 09:15 |
| tsdgeos | Saviq: i do not know, i'm just copying code around | 09:15 |
| mhr3 | i think shell indeed it in one case | 09:15 |
| * Saviq greps | 09:15 | |
| mhr3 | but who knows maybe it doesn't anymore | 09:15 |
| tsdgeos | this is starting to look like those Qt code reviews where people ask me to fix some other bugs unrelated to the one i'm fixing :D | 09:15 |
| Saviq | tsdgeos, no no, I thought you were renaming | 09:16 |
| mhr3 | tsdgeos, i guess that's a good thing? :) | 09:16 |
| mhr3 | tsdgeos, quality++ ;) | 09:16 |
| tsdgeos | mhr3: in general is a bit frustrating tbh | 09:16 |
| Saviq | tsdgeos, *but*, unity-api is meant to sanitize after all... | 09:16 |
| tsdgeos | sure | 09:17 |
| mhr3 | > <mhr3> i think shell indeed it in one case | 09:18 |
| mhr3 | wow | 09:18 |
| mhr3 | i use indeed to much :) | 09:18 |
| mhr3 | was meant to be needed :) | 09:18 |
| Saviq | mhr3, we get it on activate | 09:18 |
| Saviq | mhr3, to pass .result | 09:19 |
| mhr3 | right, so that's the one use-case | 09:19 |
| mhr3 | we could at least get rid of all the other get()s | 09:19 |
| tsdgeos | Saviq: but we can get it somehow else | 09:19 |
| tsdgeos | mhr3: you mean the getScope? | 09:20 |
| mhr3 | tsdgeos, i meant we could get rid of the other QVariantMap get(), if there are any | 09:20 |
| tsdgeos | no, there's none else | 09:21 |
| mhr3 | otherwise ScopeInterface* get() -> ScopeInterface* getScope() | 09:21 |
| tsdgeos | ok | 09:22 |
| * tsdgeos changes | 09:22 | |
| mhr3 | but yea, now we'll need to land unity-api, -scopes-shell and u8 in the same go | 09:24 |
| mhr3 | since we're renaming | 09:24 |
| Saviq | Cimi, you broke video carousel: http://imgur.com/p0oiNEo | 09:26 |
| tsdgeos | mhr3: you can still land unity-api first | 09:28 |
| tsdgeos | since it's "unused" | 09:29 |
| tsdgeos | and then land scopes-shell and u8 in one go | 09:29 |
| mhr3 | right | 09:29 |
| Mirv | tsdgeos: Saviq: so far the story sounds like that if we want to switch to 5.3 it has to be ~now and not any lingering later plan. hopefully once unity8 starts working the functionality would look good, but when I for example was not able to launch system settings from indicator last week when I tried and I'm not sure why (in addition to scopes problem) | 09:31 |
| Cimi | Saviq, let's revert it | 09:32 |
| Saviq | Cimi, why can't it adapt to the card size? | 09:32 |
| mhr3 | Mirv, what scopes problem? | 09:34 |
| Cimi | Saviq, I did it externally in cardCarousel | 09:34 |
| Saviq | Cimi, yeah, and? it knows the card size of course? | 09:35 |
| tsdgeos | Mirv: define "now" | 09:35 |
| Cimi | Saviq, you remember my fear of the anchors fill? | 09:35 |
| Mirv | mhr3: broken with Qt 5.3, since it relies on the delegate range patch that was not accepted by upstream. also the new unity8 branch somehow just crashes on 5.3 even though it works with the 5.3-way-of-doing-things-backported-to-5.2-qtdeclarative | 09:35 |
| Cimi | Saviq, could be that issue | 09:35 |
| Saviq | mhr3, ours | 09:35 |
| Cimi | Saviq, I am not anchoring to the artShape | 09:35 |
| Cimi | I am just filling the card | 09:35 |
| Mirv | tsdgeos: 2-3 weeks probably | 09:35 |
| Cimi | we thought filling card was enough | 09:35 |
| tsdgeos | Mirv: even without the .0 out? | 09:36 |
| mhr3 | Mirv, right, thx | 09:36 |
| mhr3 | Saviq, see that now :) | 09:36 |
| Cimi | not in case of video apparently | 09:36 |
| Mirv | tsdgeos: well we can work on 5.3 rc for now and .0 should be out this week (or at latest next if it gets delayed) | 09:36 |
| Saviq | Cimi, and? the carousel knows about the size of the card, there's no reason why it shouldn't work (with a borderimage on the shadow) | 09:36 |
| Mirv | tsdgeos: it's just that the schedule is such that we shouldn't do major things like this anymore later in June for example | 09:37 |
| Mirv | anyhow, more discussion on the subject will happen on Wed | 09:37 |
| mhr3 | Saviq, Cimi, i'd also say the height of the video carousel is weird... maybe fixing that would magically fix this ;) | 09:37 |
| Saviq | mhr3, https://docs.google.com/a/canonical.com/file/d/0B8T-xKCA6bikUm56SG1pVEktZUk/edit | 09:38 |
| Saviq | mhr3, and anyway, kinda a mediascanner scope thing ;) | 09:38 |
| mhr3 | Saviq, no, i mean there's loads of empty space beneath the carousel | 09:39 |
| mhr3 | the visual design looks ok | 09:39 |
| mhr3 | the real thing is empty | 09:39 |
| Saviq | mhr3, in any case it's just the padding | 09:40 |
| Saviq | mhr3, but agreed it seems to think it's bigger than it really is | 09:44 |
| mhr3 | it's scary when the tech lead says that the qml components think :) | 09:46 |
| mhr3 | although with all that code generation, who knows how sentient u8 is now :) | 09:48 |
| veebers | Saviq: should I be able to run unity8 on my desktop using 'start unity8' or has something changed? | 09:50 |
| Saviq | veebers, should work, make sure you don't have some stale job .conf in ~/.config/upstart | 09:51 |
| Saviq | veebers, anything in ~/.cache/upstart/unity8.log? | 09:52 |
| Saviq | veebers, and what does `initctl status unity8` say? | 09:52 |
| veebers | Saviq: ah ok, will check and try (this is from ~last week) | 09:52 |
| veebers | Saviq: `initctl status unity8` gives (before starting): unity8 stop/waiting, `initctl start unity8` hangs for ages and I accidently just Ctrl-C it :-\ but trying to start it again states ..'job is already running..' | 09:55 |
| veebers | and now `stop unity8` is hanging for ages :-\ will ctrl-c soon | 09:56 |
| Saviq | veebers, and that's current unity8? trusty or utopic? | 09:56 |
| veebers | Saviq: Hmm, trusty and should be up-to-date | 09:57 |
| Saviq | veebers, right, we should SRU this | 09:57 |
| Saviq | veebers, http://bazaar.launchpad.net/~unity-team/unity8/trunk/revision/866/data/unity8.conf | 09:58 |
| veebers | fyi: while stop was hanging status output is "unity8 stop/pre-start, process 13935" | 09:58 |
| * veebers looks | 09:58 | |
| Saviq | veebers, you need to kill any initctl still hanging, make that ↑ change to the unity8.job (you can just put current trunk's data/unity8.conf in ~/.config/upstart) and all should be good again | 09:59 |
| Saviq | veebers, and you should move to utopic already ;) | 09:59 |
| veebers | Saviq: heh right, I intend to upgrade after the sprint (knowing my luck a fresh install just before a sprint would have borked my laptop :-) ) | 10:00 |
| xnox | veebers: initctl start --no-wait unity8 | 10:00 |
| * veebers tries | 10:00 | |
| xnox | veebers: you are probably blocked on jobs that start on starting/started unity8, yet other conditions for them have not been satisfied and thus you are blocked. | 10:00 |
| Saviq | xnox, that won't help for actually starting unity8, which didn't do --no-wait when starting indicators | 10:00 |
| Saviq | xnox, nope, he's blocked on starting indicators (which unity8's pre-start does) | 10:00 |
| xnox | Saviq: horum. | 10:01 |
| Saviq | xnox, yeah, we added --no-wait since then | 10:01 |
| Saviq | need to SRU | 10:01 |
| veebers | Saviq: put trunk http://bazaar.launchpad.net/~unity-team/unity8/trunk/view/866/data/unity8.conf as ~/.config/upstart/unity8.conf `initctl start unity8` and it still hangs there (nothing popsup) | 10:04 |
| Saviq | veebers, that's because the previous job is still hanging | 10:05 |
| Saviq | veebers, make sure no initctl is still going | 10:05 |
| veebers | Saviq: ah ok, will do | 10:05 |
| Saviq | veebers, last resort restart your session... | 10:05 |
| xnox | veebers: new config takes effect, only after the job is fully stopped. | 10:05 |
| veebers | xnox, Saviq: ah ha awesome that works a treat thanks guys /me finds it amusing when IRC notifications pop up in the windowed unity8 :-) | 10:06 |
| Saviq | veebers, ;) | 10:07 |
| tsdgeos | mhr3: any idea about this? | 10:49 |
| mhr3 | tsdgeos, "this"? | 10:49 |
| tsdgeos | mhr3: http://paste.ubuntu.com/7487951/ | 10:49 |
| tsdgeos | unity::scopes::ConfigException: invalid config file name: "/tmp/Runtime.ini.J10927": missing .ini extension | 10:49 |
| mhr3 | fix for that is being landed right now | 10:50 |
| mhr3 | in u8 approved mps | 10:50 |
| mhr3 | tsdgeos, but just run it without params | 10:50 |
| mhr3 | that will work | 10:50 |
| tsdgeos | mhr3: somehow my scopes are not running :/ | 10:53 |
| tsdgeos | this was an attempt at getting it to run | 10:53 |
| tsdgeos | i get the | 10:53 |
| tsdgeos | ERROR! Caught unity::scopes::TimeoutException: Request timed out after 300 milliseconds | 10:53 |
| tsdgeos | and nothing happens | 10:53 |
| mhr3 | tsdgeos, stop scope-registry && stop smart-scopes-proxy && start smart-scopes-proxy | 10:53 |
| tsdgeos | mhr3: done that like 10 times already :D | 10:53 |
| * tsdgeos does again | 10:53 | |
| tsdgeos | same | 10:54 |
| mhr3 | tsdgeos, hm, what's in ~/.cache/upstart/scope-registry.log? | 10:54 |
| mhr3 | oh wait, you had the perm issue earlier didn't you? | 10:54 |
| tsdgeos | i did | 10:54 |
| mhr3 | so i guess you still do? | 10:54 |
| tsdgeos | no | 10:54 |
| tsdgeos | http://paste.ubuntu.com/7487972/ | 10:55 |
| tsdgeos | that was fixed somehow | 10:55 |
| mhr3 | ok, it's saying the ss-proxy doesn't work | 10:55 |
| mhr3 | so pastebin that pls | 10:55 |
| tsdgeos | mhr3: http://paste.ubuntu.com/7487977/ | 10:56 |
| mhr3 | see, still the perm issue :P | 10:56 |
| tsdgeos | ok | 10:56 |
| mhr3 | at least that's what the catalan message look like :) | 10:56 |
| tsdgeos | a chmod fixed it | 10:57 |
| tsdgeos | where do i open a bug so this doesn't happen again? | 10:57 |
| mhr3 | lp:unity-scopes-api | 10:57 |
| tsdgeos | ok, tx | 10:58 |
| === zequence is now known as GrumpyStudio | ||
| === GrumpyStudio is now known as zequence | ||
| === MacSlow is now known as MacSlow|lunch | ||
| Saviq | greyback, hey, sorry to be pushing, what do we do with the close fix branch? | 11:25 |
| tsdgeos | mhr3: ping again | 11:28 |
| tsdgeos | mhr3: Saviq: after all the get() changes only remaining comment's from mzanetti in the unity-api branch is http://paste.ubuntu.com/7488090/ | 11:29 |
| tsdgeos | what do you guys think? | 11:29 |
| Saviq | tsdgeos, I think we should kill it with fire... but we probably can't, yet | 11:29 |
| tsdgeos | so just leave it be? | 11:30 |
| Saviq | tsdgeos, let's see what mhr3 says, but I'm afraid so... | 11:30 |
| mzanetti | kill me with fire? :D | 11:30 |
| mzanetti | good thing there's lots of water around Malta | 11:30 |
| Saviq | well, we could drop it from the shell API and deprecate it in the scope api in due course... | 11:30 |
| Saviq | mhr3, ? | 11:31 |
| Saviq | Cimi, so what's the verdict for the carousel shadow? | 11:32 |
| mhr3 | tsdgeos, Saviq, well we still need to send it to the scopes, and someone needs to provide it | 11:32 |
| Saviq | mhr3, no, we don't | 11:32 |
| Cimi | Saviq, I am looking at the padding first | 11:33 |
| Saviq | mhr3, we really don't, not in the current meaning of form factor | 11:33 |
| Cimi | Saviq, had to download and push few videos | 11:33 |
| mhr3 | Saviq, you're free to rename it ;) | 11:33 |
| Saviq | mhr3, only I don't know what to rename it to, yet | 11:33 |
| mhr3 | Saviq, but scopes should know what layout they're providing results for | 11:33 |
| Saviq | mhr3, what "layout"? | 11:33 |
| mhr3 | Saviq, tv, touch, mouse | 11:34 |
| mhr3 | i don't know | 11:34 |
| Saviq | mhr3, yeah, sure, but we can't define those yet | 11:34 |
| mhr3 | but they specify the renderers, some might make more sense than others | 11:34 |
| mhr3 | for specific layouts | 11:34 |
| Saviq | mhr3, doesn't feel convergent, tbh :P | 11:35 |
| mhr3 | Saviq, convergence is about proper ui, right? :) | 11:35 |
| Saviq | mhr3, yeah, one that's not changing for no reason | 11:35 |
| tsdgeos | ok, so ignore that for the moment i guess? | 11:35 |
| mhr3 | and since scopes define renderers, they define ui | 11:36 |
| mhr3 | so should be smart about it | 11:36 |
| tsdgeos | mhr3: do you feel like adding docu for roles or shall we convince mzanetti it will just say "result returns results" | 11:36 |
| Saviq | mhr3, well, then form factor doesn't give enough info anyway | 11:36 |
| mhr3 | tsdgeos, i'd say add a FIXME, and it will be just a doc fixme | 11:37 |
| Saviq | mhr3, because 7" tablet would be the same as a 20" tablet | 11:37 |
| tsdgeos | mhr3: fixme for what? | 11:37 |
| mhr3 | tsdgeos, "document what values are supported" | 11:37 |
| Saviq | mhr3, tsdgeos asked about model roles | 11:38 |
| tsdgeos | mhr3: can you do it since you know what you want to do? :D | 11:38 |
| mhr3 | Saviq, tsdgeos, eh sorry, i was still talking about the formFactor | 11:38 |
| mhr3 | if the roles need documenting, we're doing it wrong | 11:39 |
| Saviq | I agree | 11:40 |
| mhr3 | ie the enum value is not good :) | 11:40 |
| tsdgeos | mhr3: so document the fixme for the formfactor and done? | 11:41 |
| mhr3 | yep | 11:42 |
| greyback | Saviq: it needs a little updating against the short app Id change, and trying to fix as best as possible the issue you reported. Hopefully have something in a couple of hours | 11:42 |
| Saviq | greyback, ok, just let me know if this is taking too much of your time (MAE higher prio) | 11:42 |
| greyback | Saviq: sure | 11:42 |
| tsdgeos | mhr3: ok, i'll do it | 11:42 |
| mzanetti | tsdgeos: just saw this one in the MR: 9+ -- Albert Astals Cid <tsdgeos_work@xps> Thu, 15 May 2014 17:20:44 +0200 | 11:43 |
| mzanetti | I guess should be your canonical mail | 11:43 |
| tsdgeos | probably | 11:43 |
| mhr3 | alright, i have half day off today, so cya tomorrow | 11:45 |
| === mpt_ is now known as mpt | ||
| tsdgeos | mzanetti: fixed | 11:46 |
| Cimi | Saviq, ok so the delegate of the carousel video has empty space at the bottom | 11:59 |
| Saviq | Cimi, fix it! | 11:59 |
| Cimi | Saviq, I am :D | 11:59 |
| Saviq | Cimi, tx | 11:59 |
| Cimi | Saviq, looking to see if it comes from card | 11:59 |
| === alan_g is now known as alan_g|lunch | ||
| Cimi | Saviq, cardCarousel.components && cardCarousel.components["art"]["aspect-ratio"] || 1.0 | 12:01 |
| Cimi | Saviq, this seems 1.0 in video carousel | 12:01 |
| Saviq | Cimi, unlikely | 12:02 |
| Saviq | Cimi, at least the scope sends 1.5 | 12:02 |
| Saviq | Cimi, you can see in the scope tool | 12:02 |
| Cimi | Saviq, TypeError: Cannot read property 'art' of undefine | 12:02 |
| Cimi | cardCarousel.components is undefined | 12:02 |
| Saviq | Cimi, can we please not do IRC debugging again? | 12:02 |
| Cimi | I'm live digging | 12:02 |
| Cimi | ah ok | 12:02 |
| Cimi | I like it | 12:03 |
| Saviq | Cimi, get your own channel then ;P | 12:03 |
| Cimi | Saviq, yeah but nobody will join >D | 12:03 |
| Saviq | Cimi, yeah, that's 'cause it's really unproductive for the other party | 12:04 |
| Cimi | Saviq, yes | 12:05 |
| Cimi | Saviq, btw bug is fixed | 12:05 |
| Cimi | Saviq, s/cardCarousel/cardTool/g | 12:05 |
| Saviq | good | 12:05 |
| Cimi | pushing soon | 12:05 |
| Saviq | Cimi, testable? | 12:05 |
| Cimi | Saviq, looks more like someone mistake | 12:05 |
| Cimi | maybe yeah,,, | 12:06 |
| Saviq | Cimi, that's what tests are for ;) | 12:06 |
| Cimi | we should test if the carousel tile width and height have same aspect ratio of the backend data | 12:06 |
| Cimi | can be done with some objectName magic maybe, othewise aliasing properties | 12:07 |
| Saviq | tsdgeos, will you have a look at what Mirv reported about Qt 5.3 and your patch? | 12:10 |
| Saviq | /latefood | 12:11 |
| Cimi | Saviq, ping me when you back | 12:13 |
| Cimi | mhr3, you in the office one of these days? | 12:20 |
| mhr3 | Cimi, wed probably | 12:20 |
| Cimi | ok | 12:21 |
| Cimi | mhr3, want to try my wifi card? | 12:21 |
| Cimi | or flashing your bios together? | 12:22 |
| mhr3 | Cimi, oh yea, do you want mine? | 12:23 |
| mhr3 | Cimi, i can give you mine and try to do nasty things to yours :) | 12:23 |
| Cimi | mhr3, not that I want yours | 12:23 |
| Cimi | mhr3, mine works fine | 12:23 |
| mhr3 | Cimi, but is it ac? | 12:23 |
| Cimi | mhr3, nope | 12:23 |
| mhr3 | Cimi, see, so you can have an upgrade | 12:24 |
| Cimi | mhr3, but I don't have an AC ap anyway | 12:24 |
| Cimi | mhr3, neither I know anyone with one | 12:24 |
| mhr3 | give it few months :) | 12:24 |
| mhr3 | but yea, i was trying to flash my bios back and forth and all the way around, but couldn't make it work | 12:25 |
| mhr3 | the latest bioses have some stupid security features | 12:26 |
| mhr3 | and i have a feeling that if i play a bit more with it, my laptop will just stop booting | 12:26 |
| Cimi | ok >D | 12:26 |
| Cimi | let's try mine, otherwise send it back | 12:26 |
| mhr3 | it won't just work out of box | 12:27 |
| === MacSlow|lunch is now known as MacSlow | ||
| mhr3 | there's only one wifi/wimax intel card that this laptop supports | 12:27 |
| Cimi | it could work | 12:27 |
| mhr3 | but i could flash the epprom on the card ;) | 12:27 |
| Cimi | how do you know if a card is supportd or not? | 12:27 |
| mhr3 | there's a list | 12:28 |
| Cimi | link | 12:28 |
| Cimi | mhr3, my 5100 is bundled with some thinkpads, so it might be that it works | 12:30 |
| mhr3 | Cimi, http://download.lenovo.com/parts/ThinkPad/ThinkPadEdgeE430_2012-04-10.pdf | 12:33 |
| Cimi | mhr3, nothing basically :D | 12:34 |
| mhr3 | pretty much :) | 12:34 |
| Cimi | mhr3, https://www.diggloving.com/product_info.php?language=en¤cy=GBP&products_id=2233 ? | 12:35 |
| mhr3 | yea, that's the one intel it supports | 12:35 |
| mhr3 | Cimi, but you still need to buy it from lenovo for about 25 pounds... | 12:37 |
| mhr3 | so... yea | 12:37 |
| greyback | Saviq: close fix branch updated, please lemme know if I succeeded, or if I broke something else | 12:37 |
| cwayne1 | Saviq: heya, is the whole removing "Preview" from the preview titlebar going to be discussed in Malta? | 12:37 |
| Saviq | Cimi, back | 12:53 |
| Saviq | greyback, ok, looking | 12:53 |
| pstolowski | sil2100, ping | 12:53 |
| sil2100 | pstolowski: pong :) | 12:53 |
| Saviq | cwayne1, not sure, it might come up in the dash design session | 12:55 |
| cwayne1 | Saviq: would you mind making sure it's discussed? it looks not great when we send screenshots to partners and the preview title is cut off... | 12:58 |
| Saviq | cwayne1, hmm with the planned design it might get even worse https://sites.google.com/a/canonical.com/unity8dash/toolkit/14-previews | 12:59 |
| Saviq | cwayne1, since it's meant to hold content title after all | 13:00 |
| Saviq | cwayne1, that's not signed-off design, yet, though | 13:00 |
| === alan_g|lunch is now known as alan_g | ||
| Saviq | cwayne1, in any case, I'll make sure to mention it, yes | 13:02 |
| cwayne1 | Saviq: thank you | 13:04 |
| === pete-woods is now known as pete-woods-lunch | ||
| cwayne1 | Saviq: so next thing i have to bother you about :) any update on shell rotation? | 13:05 |
| Saviq | cwayne1, you can check the status out in https://launchpad.net/~unity-team/+archive/phone-right-edge | 13:06 |
| Saviq | cwayne1, there's rough edges, but just upgrading from that PPA should give you a rotating shell | 13:06 |
| tsdgeos | Saviq: sure i will | 13:08 |
| cwayne1 | Saviq: do we plan on having it landed before MAE? or are we going to need to hack it again like MWC | 13:09 |
| Saviq | cwayne1, we'll need a custom image for sure | 13:10 |
| Saviq | cwayne1, but only one | 13:10 |
| cwayne1 | daw | 13:11 |
| Cimi | Saviq, sorry I am busy with the test | 13:18 |
| Saviq | Cimi, nw, it's you who's asked me to ping when back ;) | 13:18 |
| Cimi | Saviq, anyway I noticed that there is more padding above the carousel then at the bottom (in music) | 13:18 |
| Cimi | Saviq, this is due to the sum of internal padding of carousel centering the items | 13:19 |
| Saviq | Cimi, mhm | 13:19 |
| Cimi | Saviq, plus the empty space of the header of the category | 13:19 |
| Cimi | in Music scope | 13:19 |
| Cimi | "Songs " | 13:20 |
| Cimi | no idea how to best fix this | 13:20 |
| Cimi | design wise... | 13:20 |
| Cimi | it doesn't look nice | 13:20 |
| Cimi | Saviq, carousel has even padding top and bottom of tiles | 13:21 |
| Cimi | Saviq, but at top you add the empty space next to the label | 13:21 |
| Cimi | we should probably remove the carousel top padding | 13:22 |
| Cimi | because we know we have the padding due to the label | 13:22 |
| Saviq | greyback, only thing I didn't expect: upstart-app-stop does not remove the app from the list, but that's probably correct - until you remove it from the dash, it's meant to be restartable, even though the job goes away? | 13:23 |
| greyback | Saviq: nope, it should go away. Darn | 13:24 |
| Saviq | greyback, orly? | 13:24 |
| Saviq | so you can distinguish "expected" stop vs. "unexpected" stop? | 13:24 |
| greyback | Saviq: how does a user stop an app from the CLI? | 13:24 |
| greyback | Saviq: upstart can distinguish those events | 13:25 |
| Saviq | greyback, ok, so yeah, that'd be a problem still | 13:25 |
| greyback | so I broke something... | 13:25 |
| greyback | weird, I've a test for that and all | 13:26 |
| Saviq | greyback, to make sure we're on the same page: https://code.launchpad.net/~gerboland/unity-mir/fix-upstart-closed-apps2/+merge/218721/comments/525521 | 13:27 |
| Saviq | greyback, that'd be the only thing, all the other issues are fixed! | 13:29 |
| greyback | Saviq: ack, on it | 13:29 |
| === _salem is now known as salem_ | ||
| * greyback sees his test in wrong.... | 13:37 | |
| greyback | Saviq: what do you think it should do? Second opinion wanted :) | 13:37 |
| kgunn | mzanetti: phone right edge ppa looks likes its got all new goodies...i was gonna try, just checking if there was a reason i shouldn't ? | 13:42 |
| Saviq | greyback, I *think* you're right, it probably should close | 13:42 |
| mzanetti | kgunn: hit it! | 13:42 |
| greyback | Saviq: yep, I agree | 13:43 |
| Saviq | greyback, it should only restart when upstart didn't expect it | 13:43 |
| mzanetti | kgunn: it'll crash after screen blanking right now, and can't start apps currently. | 13:43 |
| Saviq | greyback, basically, retcode !=0 | 13:43 |
| mzanetti | kgunn: but it respawns if it crashes and it rotates | 13:43 |
| greyback | mzanetti: any idea why apps don't launch? | 13:43 |
| Saviq | mterry, first of all thanks for taking time out of your weekend to fix the conflicts | 13:43 |
| mzanetti | greyback: no. dandrader has the suspicion because of newer mir and we need to update something | 13:44 |
| Saviq | mterry, unfortunately it was a nail to your own coffin - I was able to test stuff ;) | 13:44 |
| Saviq | mterry, and found a bunch of issues that still need attention | 13:44 |
| mterry | Saviq, oh did you find a problem? (Good ish?) | 13:44 |
| Saviq | mterry, https://code.launchpad.net/~mterry/unity8/split/+merge/213149 | 13:44 |
| greyback | mzanetti: hmm ok | 13:44 |
| dandrader | greyback, has the launching scheme changed in the meantime (the way appids are handled, I don't know) | 13:44 |
| dandrader | ? | 13:44 |
| dandrader | or the appid format | 13:44 |
| mzanetti | dandrader: actually... it works here | 13:44 |
| greyback | dandrader: nothing that should break anything | 13:45 |
| greyback | any changes were backward compatible | 13:45 |
| Cimi | Saviq, in the mock of fake_categories.cpp | 13:46 |
| Cimi | Saviq, can you explain me if (index.row() < m_specialCategories.count()) at line 97 | 13:46 |
| mterry | Saviq, OK, will comment in MP, thanks for the testing! | 13:46 |
| Cimi | basically why we have this if/else | 13:46 |
| dandrader | yeah. it's working. was able to launch gallery, system settings and phone app | 13:47 |
| Saviq | mterry, yeah, some things I'm sure are expected | 13:47 |
| Saviq | mterry, just wanted to write all down | 13:47 |
| Saviq | mterry, but there's a few blockers still | 13:47 |
| Saviq | Cimi, IIUC it checks if it's not the last category | 13:47 |
| dandrader | but not calculator | 13:48 |
| Cimi | Saviq, dunno, I will copy to both if/else | 13:49 |
| Cimi | Saviq, I want to play with different aspect ratio here | 13:49 |
| Cimi | and check in test DashContent | 13:49 |
| === salem_ is now known as _salem | ||
| Cimi | the aspect ratios | 13:49 |
| Saviq | Cimi, m_specialCategories is a QList that you can override in tests | 13:51 |
| Saviq | Cimi, altho looks like it's not used any more | 13:51 |
| Saviq | Cimi, ah no, it is | 13:51 |
| Saviq | addSpecialCategory | 13:51 |
| Saviq | Cimi, Apps is a special category | 13:51 |
| Saviq | Cimi, I mean running apps in the Apps scope | 13:52 |
| Saviq | Cimi, we need to inject that category to display the running apps (not for long, though! so we'll be able to get rid of all that) | 13:52 |
| * mzanetti likes how launchpad estimates a build completed a minute ago while its still running | 13:54 | |
| === _salem is now known as salem_ | ||
| mzanetti | kgunn: hmm... I'd need to delete a package in the ppa (cause I messed up with versioning). What's your eta? | 13:58 |
| kgunn | mzanetti: i just flashed....so whenever, no rush | 13:58 |
| mzanetti | kgunn: ok... then do the upgrade and let me know onces you fetched it | 13:59 |
| === dandrader is now known as dandrader|afk | ||
| Cimi | Saviq, in make tryDashContent, I have few empty categories | 14:00 |
| Cimi | Saviq, do you have the same? | 14:00 |
| Saviq | Cimi, doesn't look like it | 14:01 |
| Saviq | Cimi, sounds like one of your renderers fails to parse | 14:01 |
| Saviq | kgunn, fyi, I cleaned up the old recipes we had in https://code.launchpad.net/~unity-team/+recipes | 14:03 |
| kgunn | cool, thanks, it was a mess | 14:04 |
| mterry | Saviq, thanks so much for the testing! I should have tested manta more recently. Got complacent with mako | 14:04 |
| mterry | Saviq, but SIM unlocking is a pain, since I don't have a SIM that unlocks to test | 14:05 |
| Saviq | mterry, nw, I'm only just flashing now properly, will test there again | 14:06 |
| Saviq | mterry, you should be able to set a PIN in Android, or do you mean there are SIMs which don't support it at all? | 14:06 |
| mterry | Saviq, I'm sure there will still be some problems, since I haven't given malta the attention it deserves | 14:06 |
| * mterry busts out his manta | 14:06 | |
| ehu | Hi! a little help please. If you have user accounts authenticated with LDAP (say), and use nss tables other than passwd to store user information, then you find that the system automatically creates an entry in the user account database on login. This is handy: it means the user gets listed by lightdm in the greeter, for example. However, these users cannot be deleted. If you use the accounts-daemon from the control center to remove them, you get that th | 14:13 |
| ehu | e user isn't in /etc/passwd | 14:13 |
| Cimi | tsdgeos, builddir/plugins/Dash/createCardComponent:40:51: Unable to assign [undefined] to QString where is that? | 14:13 |
| tsdgeos | Cimi: the card creator | 14:15 |
| Cimi | tsdgeos, yeah, but how do I get the line? | 14:15 |
| tsdgeos | it's there | 14:16 |
| tsdgeos | line 40 | 14:16 |
| tsdgeos | obviously line 40 of the generated code | 14:16 |
| tsdgeos | you'll have to print the generated code | 14:16 |
| Cimi | tsdgeos, and where is the generated code? | 14:16 |
| tsdgeos | Cimi: ? | 14:16 |
| Cimi | like CardCreator.createCardComponent(root, template, components); | 14:17 |
| Cimi | printing this? | 14:17 |
| Cimi | ouch... might be a lot | 14:17 |
| tsdgeos | ./plugins/Dash/CardCreator.js:519: return Qt.createQmlObject(code, parent, "createCardComponent"); | 14:18 |
| tsdgeos | that's where createCardComponent comes from | 14:18 |
| Cimi | I found it btw | 14:18 |
| tsdgeos | Cimi: anyway if this warning is the warning i think it's basically harmless | 14:19 |
| Cimi | tsdgeos, it's | 14:19 |
| Cimi | tsdgeos, borderSource: artShapeBorderSource; \n\ | 14:19 |
| tsdgeos | ?¿ | 14:20 |
| Cimi | tsdgeos, with property var artShapeBorderSource; - question is, where do we want to put the override check? | 14:20 |
| Cimi | tsdgeos, I added this for the carousel, to remove the borderSource of the art ubuntu shape | 14:20 |
| tsdgeos | i know | 14:20 |
| tsdgeos | but there can't be a \n\ in the generated code | 14:20 |
| Cimi | tsdgeos, it is not generated, this is the js code | 14:21 |
| Cimi | anyway | 14:21 |
| tsdgeos | you don't understand | 14:21 |
| Cimi | we need a default value | 14:21 |
| tsdgeos | the line of the error is not the .js | 14:21 |
| tsdgeos | but the generated code | 14:21 |
| tsdgeos | so don't look at line 40 of the .js | 14:21 |
| === dandrader|afk is now known as dandrader | ||
| Cimi | tsdgeos, and this js, generated the qml that causes the warning | 14:21 |
| tsdgeos | correct | 14:22 |
| Cimi | tsdgeos, this is the js line that corresponds to line 40 | 14:22 |
| tsdgeos | ok | 14:22 |
| tsdgeos | you made it pretty hard to understand ^_^ | 14:22 |
| === alan_g is now known as alan_g|tea | ||
| Cimi | tsdgeos, now, I either put the default back at property var artShapeBorderSource: ""; | 14:22 |
| Cimi | tsdgeos, or I do borderSource: artShapeBorderSource || ""; | 14:23 |
| tsdgeos | that's both bad | 14:23 |
| tsdgeos | should be the default value of artShapeBorderSource | 14:23 |
| tsdgeos | not "" | 14:23 |
| tsdgeos | but we don't have a way to get the default value of artShapeBorderSource | 14:23 |
| Cimi | tsdgeos, default value in current ubuntu shape is "radius_idle.sci" | 14:24 |
| tsdgeos | so why would we set "" there as default? | 14:24 |
| Saviq | mzanetti, what happened in the PPA? I don't think you'll be able to upload a source that's older than the one in the PPA already | 14:25 |
| mzanetti | Saviq: I think after deleting the package it should work again... | 14:25 |
| mzanetti | Saviq: before I had the recipies I bumped the version a bit too much | 14:25 |
| Cimi | tsdgeos, undefined complains | 14:25 |
| Saviq | mzanetti, mhm, I'm worried it will take a few hours to drop the previous source | 14:26 |
| tsdgeos | Cimi: my suggestion is turn it into an alias or a dummy variable depending if you have art or not | 14:26 |
| Saviq | mzanetti, you might want to just bump the epoch... | 14:26 |
| tsdgeos | Cimi: if you have art artShapeBorderSource aliases to artShape.borderSource | 14:26 |
| tsdgeos | otherwise artShapeBurderSource is just a int | 14:26 |
| tsdgeos | or a string or some other crap | 14:26 |
| mzanetti | Saviq: I deleted packages on friday already. was instant, but not sure if that really cleaned history in the background as well | 14:26 |
| Cimi | tsdgeos, but what if artShape does not exist? | 14:26 |
| mzanetti | Saviq: good idea with the epoch | 14:27 |
| tsdgeos | Cimi: if it doesn't exist, just let people set it and do nothing, it doesn't exist anyway ;) | 14:27 |
| Cimi | tsdgeos, qml will complain, no? | 14:27 |
| tsdgeos | ? | 14:27 |
| tsdgeos | why would it complain? | 14:27 |
| Cimi | tsdgeos, setting alias to inexisting id | 14:27 |
| Saviq | ehu, sounds like you should file a bug, or at least post to ubuntu-devel mailing list, it's going to be rather difficult getting a response here | 14:27 |
| tsdgeos | Cimi: you are generating code, don't alias it to an existing id | 14:28 |
| tsdgeos | do what i said and just create the variable as not an alias | 14:28 |
| tsdgeos | when there's nothing to alias to | 14:28 |
| Saviq | dandrader, re: "rotate application thumbnails in dash to follow shell rotation: INPROGRESS", keep in mind we should probably not rotate those that will only support one orientation | 14:29 |
| === pete-woods-lunch is now known as pete-woods | ||
| tsdgeos | Cimi: otoh i'm not sure you can alias to a loader item | 14:30 |
| dandrader | Saviq, once we have that information. There's also the case that shell should not rotate at all if the focused/foreground app does not support it | 14:31 |
| Saviq | dandrader, of course | 14:31 |
| Saviq | dandrader, just wanted to make sure we're on the same page | 14:31 |
| ehu | Saviq: thx | 14:40 |
| dandrader | mzanetti, is the qt compositor ppa using ubuntumirclient or ubuntuclient QPA for apps? | 14:46 |
| Cimi | tsdgeos, indeed doesn't find the id inside the loader | 14:46 |
| mzanetti | dandrader: not the new one | 14:46 |
| mzanetti | dandrader: greyback voted for keeping it with the old one for now. | 14:47 |
| Cimi | hold on, artShapeLoader.item might work | 14:47 |
| mzanetti | dandrader: however, the new one is installed along with the old one. you should be able to just export the env var to use the new one | 14:47 |
| dandrader | right | 14:47 |
| mzanetti | dandrader: the fact that its installed right now is probably a bug, but you can use it for good | 14:47 |
| greyback | dandrader: mind testing lp:~gerboland/qtubuntu/qtCompositorCompatibility/ to see if it implements the core functional changes you made in ubuntuclient QPA? | 14:48 |
| Cimi | tsdgeos, nope, alias doesn't work | 14:48 |
| Cimi | tsdgeos, running out of ideas | 14:50 |
| dandrader | greyback, after I'm done with the tasks at hand | 14:54 |
| greyback | dandrader: sure | 14:55 |
| tsdgeos | Cimi: this http://paste.ubuntu.com/7488876/ is a bit silly and recursive, but does it work? | 15:01 |
| Cimi | borderSource: artShapeBorderSource || borderSource ? | 15:01 |
| Cimi | can we do that? :D | 15:01 |
| tsdgeos | Cimi: it looks kind of evil right? | 15:03 |
| tsdgeos | :/ | 15:03 |
| Cimi | tsdgeos, hah | 15:04 |
| tsdgeos | Cimi: if it works we could have it | 15:04 |
| tsdgeos | Cimi: the other solution is only ever doing the binding in the art level if we know we're going to set it | 15:05 |
| dandrader | I wanna check in javascript if a property exists (not check if it has an assigned value or anything like that). Will that do it? "if (item.propertyName != undefined)" | 15:05 |
| === gatox is now known as gatox_lunch | ||
| tsdgeos | dandrader: for strict stuff better use the triplet comparers (i.e. === !==) | 15:06 |
| Saviq | dandrader, `if typeof foo !== "undefined"` | 15:06 |
| Saviq | dandrader, and in case of props | 15:06 |
| Saviq | dandrader, item.hasOwnProperty('foo') | 15:06 |
| dandrader | ah, that's nice. thanks | 15:06 |
| Cimi | tsdgeos, file:///usr/lib/x86_64-linux-gnu/qt5/qml/Ubuntu/Components/UbuntuShape.qml:103:5: QML Shape: Binding loop detected for property "stretched" | 15:06 |
| tsdgeos | Cimi: the other solution is defaulting to whatever ubuntu shape defaults | 15:06 |
| tsdgeos | Cimi: but that's kind of bad if they change | 15:07 |
| Cimi | tsdgeos, the latter is the new warning | 15:07 |
| tsdgeos | Cimi: so i'd prefer you investigate the possibility of only generating the code when we know we're going to use it | 15:07 |
| Cimi | tsdgeos, yes | 15:07 |
| Cimi | tsdgeos, sucks though | 15:08 |
| Cimi | tsdgeos, so in qml we don't have a way to default a property? | 15:09 |
| dandrader | Saviq, what about: if ("propertyName" in item) (from http://www.nczonline.net/blog/2010/07/27/determining-if-an-object-property-exists/) | 15:10 |
| tsdgeos | not that i know | 15:10 |
| Cimi | ok | 15:10 |
| Saviq | dandrader, that's not only for properties, would be true for methods, too, for example | 15:10 |
| Saviq | dandrader, not sure hasOwnProperty would have the same issue TBH | 15:10 |
| Saviq | dandrader, but good enough | 15:11 |
| Cimi | tsdgeos, Component.onCompleted { if (artShapeBorderSource !== undefined) borderSource = artShapeBorderSource; } ? | 15:12 |
| Cimi | works here | 15:15 |
| tsdgeos | hmmm | 15:17 |
| tsdgeos | Cimi: don't we set the bordersoure after comlpetion? | 15:17 |
| tsdgeos | i.e. it will never get any set? | 15:18 |
| Cimi | tsdgeos, it is set in the loader of CardCarousel | 15:18 |
| tsdgeos | Cimi: yes, after on loaded | 15:19 |
| tsdgeos | is that before on completed? | 15:19 |
| Cimi | apparently :D | 15:19 |
| Cimi | seems to work here | 15:19 |
| tsdgeos | Cimi: can i see the patch? | 15:20 |
| Cimi | http://paste.ubuntu.com/7488943/ | 15:20 |
| === alan_g|tea is now known as alan_g | ||
| tsdgeos | Cimi: yeah seems loaded is before completed | 15:26 |
| tsdgeos | weird | 15:26 |
| tsdgeos | Cimi: still if we're going to go that route | 15:26 |
| tsdgeos | Cimi: i think it's better if you add a onArdShapeForcedSourceChanged | 15:26 |
| tsdgeos | and set it to the somethingLoader.item.borderSource | 15:26 |
| tsdgeos | if it's not null | 15:26 |
| tsdgeos | what do you think? | 15:26 |
| Cimi | tsdgeos, onArtShapeBorderSourceChanged you mean? | 15:27 |
| tsdgeos | yes | 15:28 |
| tsdgeos | typos-- | 15:28 |
| tsdgeos | i think it's less "racy" and a bit easier to understand | 15:29 |
| tsdgeos | you can even add a comment there in why this is not a binding because we don't want to change the default bla bla | 15:29 |
| tsdgeos | what do you think? | 15:29 |
| Cimi | tsdgeos, I am wondering if this is called before the item is loaded? | 15:32 |
| tsdgeos | what? | 15:32 |
| tsdgeos | the onChanged? | 15:32 |
| Cimi | tsdgeos, yes | 15:32 |
| tsdgeos | it has to be called every time it changes | 15:32 |
| Cimi | tsdgeos, so artShapeLoader.item is not set | 15:32 |
| tsdgeos | well so you check for it in the code | 15:32 |
| tsdgeos | or not | 15:32 |
| tsdgeos | i mean it's the same you do now | 15:33 |
| Cimi | tsdgeos, now I do from the loader | 15:33 |
| Cimi | tsdgeos, when it is loaded (completed) | 15:33 |
| tsdgeos | ah right | 15:33 |
| tsdgeos | it's an async loader | 15:33 |
| tsdgeos | sorry | 15:33 |
| Cimi | tsdgeos, we can have both | 15:33 |
| tsdgeos | Cimi: right, so have both | 15:34 |
| tsdgeos | in case someone changes it "later" | 15:34 |
| tsdgeos | or | 15:34 |
| Cimi | ok | 15:34 |
| tsdgeos | propose a patch for ubuntushape so they expose their default bordershape value | 15:35 |
| tsdgeos | so we can do something like | 15:35 |
| tsdgeos | property var artShapeBorderSource: UbuntuShape.defaultBorderSource; \n\ | 15:35 |
| Cimi | tsdgeos, better not to ask more stuff for them | 15:35 |
| Cimi | tsdgeos, first thing I want the new ubuntushape | 15:36 |
| Cimi | then this is an easy patch | 15:36 |
| tsdgeos | ok :/ | 15:38 |
| greyback | Saviq: 3rd time lucky? https://code.launchpad.net/~gerboland/unity-mir/fix-upstart-closed-apps2/+merge/218721 | 15:39 |
| Saviq | greyback, will do first thing tomorrow | 15:43 |
| kgunn | dandrader: mzanetti ...i assume you guys already found the dash bug on rotation (like it changes scopes) | 15:43 |
| greyback | Saviq: thanks | 15:44 |
| mzanetti | I didn't, no... but not currently focusing on rotation | 15:44 |
| dandrader | kgunn, yes | 15:45 |
| tsdgeos | Mirv: there? | 15:47 |
| Mirv | tsdgeos: here | 15:50 |
| tsdgeos | Mirv: see https://code.launchpad.net/~aacid/unity8/useDisplayMargin/+merge/219709/comments/525668 | 15:50 |
| tsdgeos | Mirv: can you confirm that https://launchpad.net/~canonical-qt5-edgers/+archive/qt5-daily/+packages doesn't carry my lp:~aacid/unity8/useDisplayMargin changes? | 15:51 |
| Mirv | tsdgeos: ah, now the autobuild has overwritten it | 15:51 |
| Mirv | tsdgeos: yes | 15:51 |
| tsdgeos | ok | 15:51 |
| tsdgeos | still yes, the phone is crashing | 15:51 |
| Mirv | so I need to rebuild it | 15:51 |
| tsdgeos | i'll try to find out why tomorrow | 15:51 |
| tsdgeos | it's weird | 15:51 |
| Mirv | tsdgeos: but, I just heard from jhodapp that beta2 was crashing for him too, so it might be that it was not the unity8 patch that broke it, but that while waiting for it something else changed that broke stuff, for example the problem we have with unity*-api tests that requires a manual rebuild (just happening in qt5-beta2) or otherwise the build fails and falls out of sync with archive version | 15:52 |
| tsdgeos | Mirv: i see | 15:52 |
| tsdgeos | Mirv: do you want me to try to do some investigation? | 15:53 |
| Mirv | I was meaning to look at it more today but I got stuck with qtpim, so I just now only launched the beta2 manual rebuilds | 15:53 |
| Mirv | tsdgeos: well any trying out of 5.3 woud be appreciated to have more idea on whether we have possibilities of migrating to it or not | 15:53 |
| Mirv | hopefully after this round of rebuilds in qt5-beta2 it works again, and then I can copy the rebuilds to qt5-daily too to have a separate PPA for testing the qtdeclarative + your unity8 branch | 15:53 |
| Mirv | so if the unity8 branch also works with 5.3, then stuff starts to be in better shape for getting an idea of how 5.3 looks like | 15:54 |
| tsdgeos | Mirv: yeah on the desktop it works fine | 15:55 |
| tsdgeos | if i run my own compiled unity8 | 15:55 |
| tsdgeos | not the ppa since it doesn't have the patch | 15:55 |
| Mirv | yeah I just pushed a new unity8 again to the qt5-daily | 15:55 |
| tsdgeos | cool tx | 15:56 |
| Mirv | and well I'll copy the rebuilds I put to beta2 to daily already now so that I won't need to wait | 15:56 |
| tsdgeos | ok, so tomorrow i'll do another check on how it works on both desktop and phone and try to find out what's wrong if something fails/crashes | 15:56 |
| Mirv | thanks! | 15:56 |
| Mirv | I also wasn't able to launch settings from indicator which I tried to use as a workaround when scopes were absent, but let's see how it is after the scopes start working | 15:57 |
| === gatox_lunch is now known as gatox | ||
| Cimi | Saviq, I have this to check aspect ratio http://paste.ubuntu.com/7489159/ | 16:10 |
| Cimi | Saviq, currently only testing a static 1.5 for all carousels | 16:10 |
| Cimi | Saviq, how do I get the aspect ratio property of scopes? for this category? | 16:11 |
| Cimi | nevermind, found a way | 16:17 |
| Cimi | Saviq, I need lil help with C++ | 16:22 |
| Cimi | Saviq, I'd like to have artMap["aspect-ratio"] = "1." + index.row(); | 16:22 |
| Cimi | so 1.1, 1.3, 1.5... | 16:23 |
| Cimi | artMap["aspect-ratio"] is a char* | 16:23 |
| Cimi | I am trying to look for concatenation operator in C++ and I only see to use strings | 16:23 |
| Cimi | I was wondering if you know a trick | 16:23 |
| === dandrader is now known as dandrader|lunch | ||
| === salem_ is now known as _salem | ||
| Cimi | Saviq, got it to work with QString, let me know if it is fine for fake_categories.cpp http://paste.ubuntu.com/7489256/ | 16:40 |
| === alan_g is now known as alan_g|EOD | ||
| === dandrader|lunch is now known as dandrader | ||
| dandrader | Saviq, what's the idea of this "nonmirplugins"? | 17:30 |
| dandrader | in tests/mocks/Unity/Application | 17:30 |
| mzanetti | Wellark: hey | 17:41 |
| mzanetti | Wellark: https://code.launchpad.net/~mzanetti/unity8/lockscreen-enhancements/+merge/220118 | 17:41 |
| mzanetti | dandrader: can I help regarding the mockApplicationManager? | 17:41 |
| dandrader | mzanetti, yeah, I was thinking about moving the Unity.Application mock to Mir.Application. but when looking at tests/mocks/Unity/Application/CMakeLists.txt I was puzzled at this NonMirUnityApplicationQml lib and the nonmirplugins dir | 17:44 |
| * mzanetti takes a look | 17:44 | |
| mzanetti | dandrader: that's strange indeed. But I guess the idea is to have this around in case you want to run the shell on anything non-mir | 17:46 |
| mzanetti | dandrader: say you really need to test it on surfaceflinger for some reason, you could use this to start it | 17:46 |
| greyback | dandrader: hey, I've found a crash bug in qtmir's QtEventFeeder - the volume up/down & power key codes are not in the kkeyCode table | 17:46 |
| greyback | dandrader: was there any reason you didn't use xbk stuff there? | 17:47 |
| dandrader | greyback, no. that key code table is essentially a copy&paste from qtubuntu. Also, I know next to nothing about xkb | 17:48 |
| greyback | dandrader: ok. | 17:48 |
| greyback | I'll see what I can do | 17:48 |
| mzanetti | I'm still not sure why we want to rename the plugin from Unity.Application to Mir.Application | 17:50 |
| mzanetti | I mean, isn't the idea of unity-api somehow that we could replace the plugin with another implementation and it still would work? | 17:50 |
| mzanetti | and it's in fact Unity's application plugin, which just happens to implement a Mir backend | 17:51 |
| greyback | mzanetti: my thinking behind the rename is to indicate the plugin is not Unity specific, but can be used by other shells. But yeah, in basing it on unity-api, we contradict ourselves | 17:53 |
| greyback | mzanetti: but for now, I don't think the rename needed | 17:53 |
| mzanetti | greyback: it kinda is unity specific, no? | 17:53 |
| greyback | mzanetti: dandrader sounds like a chat to have next week ;) | 17:53 |
| mzanetti | greyback: probably not the libmirserver behind it | 17:54 |
| greyback | mzanetti: if we want to upstream this, we don't want it to be unity specific | 17:54 |
| mzanetti | greyback: sure... but that would the libmirserver/libmirclient, not the plugin, no? | 17:54 |
| greyback | mzanetti: without the plugin, it's not a shell. It can't do anything, start/stop apps, get & control surfaces... | 17:55 |
| mzanetti | hmm... fair enough | 17:55 |
| mzanetti | in which case, Unity.Application would be wrong | 17:56 |
| mzanetti | ok well... its not urgent anyways... lets discuss that in Malta | 17:56 |
| greyback | mzanetti: dandrader: I pushed a fix for qtmir crashing with screen blank, let me know if it causes issues | 18:34 |
| dandrader | greyback, ok, good | 18:34 |
| mzanetti | greyback: dandrader: the ppa is automatically built daily if there are changes. You both should have permissions to kick builds more often if you need so. | 18:35 |
| greyback | mzanetti: magic! Was just about to write a recipe :) | 18:35 |
| mzanetti | greyback: all there | 18:35 |
| mzanetti | greyback: https://code.launchpad.net/~unity-team/+recipe/qtmir-qtcompositor | 18:35 |
| greyback | dandrader: just a note, best to use the ./run.sh script in unity8, as that sets all the vars unity8 needs to run properly | 18:36 |
| greyback | if you run unity8 as usual user, it'll crash on screen blank with hwc error - this harder to fix, will take longer | 18:36 |
| dandrader | mzanetti, does Ubuntu SDK have the equivalent of a combo box (a component where you select one out of N available options) | 18:47 |
| mzanetti | dandrader: OptionSelector | 18:48 |
| dandrader | mzanetti, ah, nice | 18:49 |
| dandrader | interestingly it's not listed here: http://developer.ubuntu.com/api/qml/sdk-1.0/Ubuntu.Components/ | 18:49 |
| dandrader | but here http://developer.ubuntu.com/api/qml/sdk-1.0/Components.Components.OptionSelector/ | 18:49 |
| dandrader | probably a documentation bug | 18:49 |
| mzanetti | dandrader: yep. can you report it to the SDK? | 18:50 |
| mzanetti | dandrader: also the example says "import Components.Components 0.1" :) | 18:52 |
| dandrader | mzanetti, https://bugs.launchpad.net/ubuntu-ui-toolkit/+bug/1320985 | 18:54 |
| ubot5 | Ubuntu bug 1320985 in Ubuntu UI Toolkit "OptionSelector documentation does not show up in the proper page" [Undecided,New] | 18:54 |
| mzanetti | dandrader: confirmed. Yes. this is the right place to report the bug | 18:56 |
| mterry | Saviq, how did you set your locale which shows different in greeter and shell? Setting to "spain" or some non-US locale didn't do it for me | 18:59 |
| === dandrader is now known as dandrader|aflk | ||
| === dandrader|aflk is now known as dandrader|afk | ||
| === dandrader|afk is now known as dandrader | ||
| josharenson1 | kgunn, for performance testing w/ shell rotation... Do you just want glmark2 numbers w/ unity8 runnint? | 19:51 |
| josharenson1 | running* | 19:51 |
| === josharenson1 is now known as josharenson | ||
| kgunn | josharenson: yeah... | 19:56 |
| kgunn | josharenson: i suppose it won't matter a bit...other than testing out plugin compositor, which expect will mean nothing to glmark2 | 19:57 |
| josharenson | kgunn, yes I wouldn't expect any differences really | 19:58 |
| kgunn | josharenson: but than we can say "we did it" :) | 19:59 |
| kgunn | you know the deal | 19:59 |
| josharenson | kgunn, :-) im on it | 19:59 |
| josharenson | kgunn, during rotation the shell peeks through the fullscreen benchmark for a frame | 20:18 |
| josharenson | kgunn, i think ive seen this before... not sure if its a real problem or not | 20:18 |
| kgunn | josharenson: noteworthy...if the numbers are in the neighborhood its fine | 20:18 |
| josharenson | kgunn, sure... | 20:19 |
| === zequence_ is now known as zequence | ||
| === naee is now known as eean | ||
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!