| === duflu_ is now known as duflu | ||
| xnox | mdeslaur: hm, is that on the desktop only atm? | 03:36 |
|---|---|---|
| * xnox haven't seen that on Touch yet. | 03:36 | |
| mdeslaur | xnox: oh, I didn't realize he was referring to touch...perhaps it's not on touch yet | 04:09 |
| xnox | mdeslaur: maybe he wasn't =) /me only has touch on my mind these days | 04:10 |
| mdeslaur | xnox: yes :) | 04:10 |
| xnox | but would be cool to have them on touch as well =) | 04:10 |
| mdeslaur | definitely :) | 04:19 |
| === iahmad is now known as iahmad|afk | ||
| tsdgeos | we got stuff merged in \o/ | 08:15 |
| lesshaste | mdeslaur, how can I use that to get the calendar to show the right date? | 08:43 |
| tsdgeos | anyone knows what's the new s-jenkins ip? | 08:45 |
| tsdgeos | mzanetti: Saviq: MacSlow: ↑↑↑ ? | 08:45 |
| MacSlow | tsdgeos, isn't it 10.97.0.26 | 08:46 |
| tsdgeos | may be | 08:46 |
| tsdgeos | my firefox autocompletion said 10.97.2.10 | 08:46 |
| MacSlow | tsdgeos, that's what I have here in my hosts file... and it seems to work still | 08:46 |
| mzanetti | 10.97.0.26 | 08:46 |
| tsdgeos | okidoki | 08:46 |
| tsdgeos | tx | 08:46 |
| mzanetti | oh... that might have been wrong | 08:47 |
| tsdgeos | ok, big question | 08:48 |
| tsdgeos | why is there no huge queue of autolanding? | 08:48 |
| tsdgeos | we need to set to needs review->approve again? | 08:48 |
| mzanetti | tsdgeos: Saviq wanted to trigger them manually | 08:49 |
| tsdgeos | mzanetti: " [paulliu] Swipe between previews: INPROGRESS" is you, right? | 08:49 |
| mzanetti | tsdgeos: yeah | 08:49 |
| tsdgeos | oka | 08:49 |
| mzanetti | tsdgeos: blueprint? | 08:49 |
| tsdgeos | yep | 08:49 |
| tsdgeos | moved it | 08:49 |
| mzanetti | thanks | 08:49 |
| tsdgeos | there some other stuff in https://blueprints.launchpad.net/ubuntu/+spec/client-1303-unity-ui-dash that needs moving to month 11 i guess | 08:51 |
| * tsdgeos moves | 08:52 | |
| mzanetti | implement the music preview: TODO | 08:52 |
| mzanetti | didn't know that was assigned to me. but ok. shouldn't be a biggie | 08:52 |
| tsdgeos | :D | 08:52 |
| tsdgeos | so we're Saviq? shouldn't he be jetlagged? :D | 08:53 |
| mzanetti | which might be related to the fact you cant find him :) | 08:54 |
| mhr3 | dednick, it's here again! no time indicator on my desktop now! | 09:47 |
| mhr3 | dednick, also how was sf? | 09:48 |
| dednick | mhr3: better weather than here :) | 09:48 |
| dednick | mhr3: otherwise, pretty busy | 09:48 |
| mhr3 | hmm, really? who would have thought that :) | 09:48 |
| dednick | mhr3: what version of indicator-datetime you running? | 09:48 |
| * mhr3 checks | 09:49 | |
| mhr3 | Installed: 13.10.0+13.10.20131016.2-0ubuntu1 | 09:49 |
| larsu | mhr3: https://bugs.launchpad.net/indicator-session/+bug/1239710 | 09:50 |
| ubot5 | Ubuntu bug 1239710 in indicator-session (Ubuntu Saucy) "indicator-datetime or -session missing ~10% of the time" [High,Fix committed] | 09:50 |
| larsu | my fixes got merged after 16.10.2013. They should trickle in with an SRU | 09:50 |
| mhr3 | larsu, yey, so it's finally tackled, cool | 09:51 |
| mhr3 | larsu, what was the issue? | 09:51 |
| larsu | mhr3: a deadlock in glib's type system | 09:51 |
| larsu | the fixes are work arounds | 09:51 |
| mhr3 | oh that? | 09:51 |
| larsu | desrt wants to fix it this cycle | 09:51 |
| mhr3 | i remember reporting it a year and a half ago :) | 09:51 |
| larsu | yeah its quite old... | 09:52 |
| seb128 | hey mhr3, larsu | 10:00 |
| seb128 | mhr3, yeah, the same good old bug biting back ;-) | 10:01 |
| === _salem is now known as salem_ | ||
| sil2100 | bregma: ping! | 10:50 |
| sil2100 | bregma: https://bugs.launchpad.net/unity/+bug/1247787 | 10:54 |
| ubot5 | Ubuntu bug 1247787 in Unity "Unity FTBFS from lp:unity on trusty" [Undecided,New] | 10:54 |
| dednick | mhr3: ping. | 11:31 |
| mhr3 | dednick, pong | 11:31 |
| dednick | mhr3: hey. do you know if you can send and receive a dbus message on same interface and same thread when using g_dbus_proxy_call_sync ? | 11:33 |
| dednick | I'm guessing not.... | 11:34 |
| dednick | given that it says sync :) | 11:34 |
| tsdgeos | mzanetti: ping | 11:34 |
| mhr3 | dednick, it's _sync cause it'll block until you get a reply | 11:34 |
| mhr3 | dednick, otherwise all _async methods will return reply in the same thread where you initiated the request | 11:35 |
| dednick | mhr3: right. but will it spin without pumping glib queue? | 11:35 |
| mhr3 | given there's a mainloop running there | 11:35 |
| mhr3 | dednick, you mean "will it spin the mainloop" or "will it flush the previous dbus msgs"? | 11:36 |
| mhr3 | it will the latter | 11:36 |
| mhr3 | spinning mainloop... not sure | 11:37 |
| dednick | mhr3: ok. so if the message is being handled by the same thread, it will just time out i guess. Which is what i'm seeing... | 11:37 |
| dednick | sigh | 11:38 |
| mhr3 | yea, don't do that :) | 11:38 |
| dednick | mhr3: means we can't create notifications in Unity8. | 11:38 |
| dednick | bit of a bugger | 11:38 |
| mhr3 | dednick, why can't you use the async variant? | 11:39 |
| larsu | mhr3: why did you set bug #1239710 to confirmed? Is it still happening after you updated? | 11:39 |
| ubot5 | bug 1239710 in indicator-datetime (Ubuntu) "indicator-datetime or -session missing ~10% of the time" [High,Confirmed] https://launchpad.net/bugs/1239710 | 11:39 |
| dednick | because it's libnotify | 11:39 |
| dednick | mhr3: ^ | 11:39 |
| mhr3 | larsu, cause it was marked as released, and it's not | 11:39 |
| mhr3 | dednick, eek :/ | 11:39 |
| larsu | mhr3: ah, it's only in proposed. Sorry about that :) | 11:39 |
| dednick | mhr3: and to made it even worse, notification actions dont work when you put it on separate thread! happy days! | 11:41 |
| dednick | stupid | 11:41 |
| larsu | dednick: unity8 itself implements org.freedesktop.Notifications | 11:41 |
| larsu | just use internal API to show a notification | 11:42 |
| dednick | larsu: yeah, i know. but only really the server. | 11:42 |
| dednick | the client is too rudimentary. | 11:42 |
| larsu | dednick: right. If you block unity8 when sending the message, it can't receive the message | 11:42 |
| dednick | but i guess we can change that | 11:42 |
| nic-doffay | tsdgeos, ping | 11:43 |
| tsdgeos | nic-doffay: hi | 11:43 |
| larsu | dednick: but really, don't hit the bus from the same process | 11:43 |
| mzanetti | tsdgeos: pong | 11:43 |
| tsdgeos | mzanetti: still missing a test, but can you have a look at https://code.launchpad.net/~aacid/unity8/lvwph_bad_header_position_1245824/+merge/193754 to see if it works for you ? | 11:44 |
| nic-doffay | tsdgeos, quick Q. Is the index on a repeater delegate a QModelIndex or just an int? | 11:44 |
| tsdgeos | nic-doffay: just an int | 11:44 |
| nic-doffay | tsdgeos, cool ta | 11:44 |
| tsdgeos | nic-doffay: QML doesn't really expose the QModelIndex stuff since it's motly list-based | 11:45 |
| mzanetti | tsdgeos: on it | 11:45 |
| nic-doffay | tsdgeos, right I'll remember that. Thanks! | 11:49 |
| Saviq | tsdgeos, I *am* | 11:52 |
| tsdgeos | oh my | 11:52 |
| tsdgeos | the google mail thing | 11:52 |
| tsdgeos | has put the travel agency stuff into the spam folder | 11:52 |
| tsdgeos | awesome | 11:52 |
| tsdgeos | ... | 11:52 |
| Saviq | tsdgeos, like you'd *ever* need that | 11:53 |
| * Saviq is useless, hates jetlag, takes swap-day | 11:53 | |
| Saviq | ping if needed | 11:53 |
| tsdgeos | Saviq: but shouldn't you be jetlagged into the other direction? :D | 11:53 |
| tsdgeos | Saviq: what do we do with landing stuff? | 11:53 |
| dandrader | mzanetti, is autolanding working again for unity8? | 11:53 |
| mzanetti | tsdgeos: https://code.launchpad.net/~aacid/unity8/lvwph_bad_header_position_1245824/+merge/193754/comments/446487 | 11:53 |
| tsdgeos | mzanetti: tx | 11:54 |
| mzanetti | dandrader: yes. but afaik still manually. Saviq didn't want to let the queue fight on its own | 11:54 |
| Saviq | tsdgeos, dandrader, mzanetti yep - manual | 11:54 |
| dandrader | Saviq, hey, I want my branch in! :) who do I bug to get it merged? you? | 11:55 |
| tsdgeos | Saviq: meaning we trigger the CI and if it approves we manually merge it? | 11:55 |
| Saviq | tsdgeos, no, meaning we trigger unity8-autolanding manually | 11:55 |
| Saviq | tsdgeos, or - I am | 11:55 |
| tsdgeos | oh | 11:55 |
| tsdgeos | ok | 11:55 |
| tsdgeos | Saviq: want us to take over for today? | 11:55 |
| dandrader | before it bitrots | 11:55 |
| Saviq | tsdgeos, whoever's got the rights to access http://s-jenkins:8080/job/unity8-autolanding/build | 11:56 |
| tsdgeos | i had afair | 11:57 |
| * tsdgeos checks | 11:57 | |
| Saviq | it's just about filling landing_candidate merge_proposal and candidate_revision there | 11:57 |
| tsdgeos | yep, can do that | 11:58 |
| Saviq | tsdgeos, I think I'm jetlagged the right way, i.e. I couldn't sleep until 5am ('cause it was still 8pm in SF) | 11:58 |
| === MacSlow is now known as MacSlow|lunch | ||
| tsdgeos | Saviq: ah right | 11:58 |
| tsdgeos | Saviq: ok, let me try to merge dandrader's MR | 11:59 |
| tsdgeos | Saviq: candidate_revision is the rev on landing_candidate we want to merge, right? | 11:59 |
| dandrader | tsdgeos, https://code.launchpad.net/~dandrader/unity8/drag-crash-1228336/+merge/192183 thanks!!! | 12:00 |
| Saviq | tsdgeos, the one that's approved on the merge, to be exact | 12:00 |
| nic-doffay | Saviq, still a maximum of three searches for the SearchHistory yeah? | 12:00 |
| Saviq | tsdgeos, I just triggered for https://code.launchpad.net/~cimi/unity8/fix-1204300/+merge/192006 | 12:00 |
| Saviq | nic-doffay, yes | 12:00 |
| nic-doffay | Saviq, cool | 12:00 |
| Saviq | tsdgeos, you can put the next one in queue | 12:00 |
| Saviq | and I'll try and be around to look at it anyway, but in case I pass out you know what to do ;) | 12:01 |
| tsdgeos | :D | 12:01 |
| tsdgeos | added to the queue | 12:01 |
| Saviq | tsdgeos, https://code.launchpad.net/~aacid/unity8/lvwph_bad_header_position_1245824/+merge/193754 no test? | 12:02 |
| tsdgeos | Saviq: i need to think how to do the test, wanted to get mzanetti to try it first | 12:03 |
| Saviq | tsdgeos, ah kay, thanks | 12:03 |
| tsdgeos | the other test i made for the other bug is not liking the CI though :_/ | 12:03 |
| mzanetti | yep. hence the comment only and not a direct approve | 12:03 |
| Saviq | tsdgeos, btw, Qt 5.2 is there in qt5-beta-proper ppa, weirdly fails unity8 package tests | 12:07 |
| Saviq | i.e. https://launchpadlibrarian.net/155434488/buildlog_ubuntu-trusty-amd64.unity8_1%3A7.83%2B14.04.20131028.1-0%2B201310302134~481~ubuntu14.04.1_FAILEDTOBUILD.txt.gz | 12:07 |
| Saviq | which is like... "230 hours ago" != "yesterday"... ORLY? | 12:08 |
| Saviq | or "230 months ago" != "yesterday", for that matter | 12:08 |
| tsdgeos | boo :D | 12:08 |
| tsdgeos | Saviq: want me to have a look at that after i finish the test for this fix? | 12:08 |
| Saviq | tsdgeos, not a priority | 12:09 |
| tsdgeos | ok | 12:09 |
| Saviq | tsdgeos, but the sooner we fix it, the sooner we switch to it, or at least start testing | 12:09 |
| Saviq | I worry that's v4 failing to do the date maths properly | 12:09 |
| tsdgeos | Saviq: though tbh if it's a bug in 5.2 it'd be cool to get it fixed in 5.2.0 | 12:09 |
| Saviq | tsdgeos, true | 12:09 |
| tsdgeos | so if we can report it and get someone else to fix it | 12:09 |
| tsdgeos | instead of it having to be me | 12:09 |
| tsdgeos | it's a win :D | 12:09 |
| Saviq | tsdgeos, sure, do your thing :) | 12:10 |
| Saviq | Cimi, hint: you don't need to commit tag changes, they're separate | 12:14 |
| Cimi | so what shall I do? | 12:15 |
| seb128 | mhr3, why did you change the indicator bug back to "confirmed"? | 12:21 |
| Cimi | Saviq, how am I supposed to deal with them then? | 12:23 |
| Saviq | Cimi, just push | 12:23 |
| Cimi | Saviq, I cannot push without a commit | 12:23 |
| Saviq | Cimi, ah, that's where you direct the "bzr tag" command directly at the remote branch | 12:24 |
| Cimi | Saviq, unless you push and says "nothing to push" it's a bug | 12:24 |
| Saviq | Cimi, my mail did that | 12:24 |
| Cimi | ah I see | 12:24 |
| Cimi | Saviq, I obviously just copied past, remember my lazy nature :) | 12:24 |
| Cimi | *paste | 12:24 |
| mhr3 | seb128, @12:39 | 12:30 |
| seb128 | mhr3, it should be "fix commited" | 12:31 |
| seb128 | mhr3, or "in progress" at least | 12:31 |
| mhr3 | seb128, we should have "in distro" / "not in distro" status for pkg bugs :) | 12:33 |
| seb128 | ;-) | 12:33 |
| seb128 | mhr3, what distro do you run? | 12:33 |
| mhr3 | seb128, what a silly question :P | 12:33 |
| seb128 | mhr3, because the fix is released in trusty, so the status was correct... | 12:33 |
| seb128 | mhr3, saucy I guess? ;-) | 12:34 |
| greyback | Saviq: https://launchpad.net/~canonical-qt5-edgers/+archive/qt5-beta-proper <- I don't see Qt5.2 packages in there. Or are the version numbers not bumped? | 12:34 |
| mhr3 | seb128, of course, noone likes to be on dev while foundations team are playing there :) | 12:34 |
| seb128 | mhr3, still, that's the correct status, can you set it back? | 12:35 |
| Cimi | who wants to help me with some cake? I have to port the wizard from quake to cmake | 12:35 |
| mhr3 | seb128, done | 12:36 |
| seb128 | mhr3, thanks | 12:36 |
| Saviq | greyback, hmm maybe somewhere else? /me checks | 12:37 |
| Saviq | greyback, https://launchpad.net/~canonical-qt5-edgers/+archive/qt5-beta2 | 12:38 |
| Saviq | tsdgeos, fyi ↑ | 12:38 |
| greyback | Saviq: I've 5.2 in a chroot. Have had 1 bug fixed already anyway | 12:38 |
| Saviq | tsdgeos, btw, https://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-trusty/90/? looks like a valid fail | 12:40 |
| tsdgeos | checking | 12:40 |
| tsdgeos | but at least it fails too here | 12:41 |
| tsdgeos | so that's good :d | 12:41 |
| mhr3 | Cimi, mhhh, cake :) | 12:44 |
| Cimi | hah | 12:45 |
| Cimi | stupid autocorrection | 12:45 |
| tsdgeos | stupid -7.10543e-15 not being 0 | 12:47 |
| === alan_g is now known as alan_g|lunch | ||
| pabloalmeidaff9 | hello | 12:54 |
| pabloalmeidaff9 | anyone can help a willing developer who's trying to fix a bug? | 12:54 |
| greyback | pabloalmeidaff9: what's the bug? When we know that, we can point you to the best person | 12:56 |
| pabloalmeidaff9 | it's a problem with drag and drop in Unity | 12:56 |
| pabloalmeidaff9 | you can drag and drop to many places, except websites | 12:56 |
| pabloalmeidaff9 | like the Gmail composer | 12:57 |
| pabloalmeidaff9 | I guess I found the cause | 12:57 |
| pabloalmeidaff9 | it looks related to how the URI of the icon is encoded | 12:57 |
| pabloalmeidaff9 | since Firefox understands the URI as a lot of Chinese characters | 12:57 |
| pabloalmeidaff9 | while the terminal understands it just fine | 12:57 |
| pabloalmeidaff9 | I followed the path and ended up on the Files lens and scope | 12:58 |
| pabloalmeidaff9 | but it's in Vala | 12:58 |
| pabloalmeidaff9 | is there a default encoding in Unity results uris? | 12:58 |
| greyback | pabloalmeidaff9: oh wow, that's great, you've figured out the issue! Could I ask you first to log a bug against the unity project, it helps us keep track of the work | 13:00 |
| pabloalmeidaff9 | allright | 13:01 |
| pabloalmeidaff9 | I'll be doing that right now | 13:01 |
| greyback | pabloalmeidaff9: thanks. Then I'll have to point you to "bregma" - he should be able to help you - but he may not be awake yet | 13:02 |
| pabloalmeidaff9 | it's my first time on IRC. I'll have to leave in one hour. If he's not awake until then, is there a more offline way of working on this? Will he be following the bug report? | 13:03 |
| pabloalmeidaff9 | I've filled a lot of reports in the past, some even with the patches, but the overall impression is that nobody listens | 13:03 |
| === MacSlow|lunch is now known as MacSlow | ||
| greyback | pabloalmeidaff9: we get a lot of bug reports, it cane be hard to keep track. https://launchpad.net/~unity-dev is a Mailing List where the right people would definitely see your contribution | 13:11 |
| pabloalmeidaff9 | greyback: reported. Link: https://bugs.launchpad.net/unity/+bug/1247834 | 13:12 |
| ubot5 | Ubuntu bug 1247834 in Unity "Drag and drop files to an area inside a website from the Dash doesn't work" [Undecided,New] | 13:12 |
| greyback | pabloalmeidaff9: I suggest you write a mail, adding the bug and your proposed fix/asking for help. That's probably the best way to do this in a more offline way | 13:13 |
| greyback | pabloalmeidaff9: if you've other bugs with patches, it would do no harm to list them in a mail too. We like patches :) | 13:14 |
| pabloalmeidaff9 | greyback: Allright. I'll be doing more research until it's time to leave the office and if bregma is not here until then, I'll email him | 13:14 |
| greyback | pabloalmeidaff9: nice, thank you. | 13:15 |
| pabloalmeidaff9 | greyback: Well, the last one I tried was a bug that is more like an opinion. It was about sorting the wifi networks list by signal strength instead of by name. The problem is that I could not build the thing at all. The solution is simple, so I guess it works, but I have no way of knowing. | 13:16 |
| pabloalmeidaff9 | greyback: I don't remember ever being able to build any package GNOME package ever. | 13:16 |
| === dandrader is now known as dandrader|afk | ||
| Saviq | dandrader|afk, there's a bunch of (non-flaky) failures for your branch http://10.97.0.26:8080/job/unity-phablet-qmluitests-trusty/92/ | 13:25 |
| Saviq | tsdgeos, you need to fill "merge_proposal", too | 13:26 |
| Saviq | tsdgeos, otherwise it won't update its status | 13:26 |
| === dandrader|afk is now known as dandrader | ||
| dandrader | Saviq, interesting. gonna check those | 13:37 |
| sil2100 | Trevinho: ChrisTownsend: ping ;) | 13:42 |
| ChrisTownsend | sil2100: Hi! What's up? | 13:43 |
| sil2100 | ChrisTownsend: hello, could you maybe take a look at the recent unity FTBFS issue? | 13:43 |
| sil2100 | ChrisTownsend: https://bugs.launchpad.net/unity/+bug/1247787 | 13:44 |
| ubot5 | Ubuntu bug 1247787 in Unity "Unity FTBFS from lp:unity on trusty" [High,New] | 13:44 |
| * ChrisTownsend Looks | 13:44 | |
| sil2100 | ChrisTownsend: thanks :) | 13:45 |
| ChrisTownsend | sil2100: Looks very strange. I'll look into it. Thanks for pointing that out. I'm wayyyyy behind on email, so I probably wouldn't see it for a while:) | 13:45 |
| greyback | pabloalmeidaff9: sorry, had meeting. "apt-get build-dep <package-name>" should download all the dependencies needed to build a package. It should install all you need | 13:47 |
| tsdgeos | Saviq: ah, ok | 13:55 |
| tsdgeos | Saviq: did you do that or? | 13:55 |
| === rachelliu_ is now known as rachelliu | ||
| === alan_g|lunch is now known as alan_g | ||
| tsdgeos | Saviq: ok, i did | 14:00 |
| mhr3 | sil2100, is this the week we can get the new pkgs in trusty? | 14:10 |
| mhr3 | sil2100, mostly capnp | 14:10 |
| sil2100 | mhr3: yes, I guess it will! I already have a snapshot packaging ready, there are only a few issues that need to be resolved and we can push it out | 14:11 |
| mhr3 | sil2100, would be nice if we could get python bindings for it too :) | 14:12 |
| mhr3 | but that's a bonus ;) | 14:13 |
| tsdgeos | dednick: nic-doffay: standtup | 14:31 |
| tsdgeos | dednick: oh you're there :D | 14:31 |
| dednick | tsdgeos: the second you said :) | 14:31 |
| greyback | tsdgeos: just an FYI: https://bugreports.qt-project.org/browse/QTBUG-34493 | 14:34 |
| tsdgeos | greyback: ouch | 14:35 |
| greyback | tsdgeos: that breaks just swiping the greeter away showing dash. Here's workaround: http://pastebin.ubuntu.com/6359069/ | 14:37 |
| kgunn | mterry: curious...as you try the nested mir (on desktop) were you also attempting on touch as well ? | 14:38 |
| greyback | tsdgeos: I'm also seeing a fast swipe-greeter-away doesn't set dash scale = 1, instead seems stuck around something like 0.8. Probably another JS issue. | 14:38 |
| mterry | kgunn, I've mostly been working with nested Mir on phone | 14:38 |
| kgunn | mterry: cool...that's even better | 14:38 |
| mterry | kgunn, my benchmarking will be on phone for example | 14:38 |
| kgunn | mterry: you're the greatest! | 14:38 |
| mterry | kgunn, I wasn't even thinking of benchmarking on desktop... maybe I should, but I assume it will be smaller delta than on phone | 14:39 |
| tsdgeos | greyback: i guess we can get that patch in, it "should work", but the less name overlapping the better i'd say | 14:40 |
| kgunn | mterry: right the impact will really be on the phone... alf and i were just chatting about this very thing | 14:40 |
| greyback | tsdgeos: +1 | 14:40 |
| tsdgeos | greyback: i always get confused by the fact that we have billions of stuff named the same :D | 14:40 |
| tsdgeos | root | 14:40 |
| kgunn | mterry: i think the mir guys should do nested-bypass work for phone first vs desktop | 14:40 |
| tsdgeos | everywheeeere | 14:40 |
| tsdgeos | greyback: so are you proposing the MR? | 14:41 |
| greyback | tsdgeos: I like "root" everywhere. It refers to the root of the qml doc you're in. Also IMO is a kind of scoping, makes it much harder to probe QML objects | 14:42 |
| mterry | kgunn, sure, makes sense. phone is priority after all | 14:42 |
| greyback | tsdgeos: yep, gimme a sec | 14:42 |
| tsdgeos | greyback: well, i like it some days, get confused some others :D | 14:42 |
| greyback | tsdgeos: fair enough :) | 14:42 |
| tsdgeos | i agree it's good it gives some kind of "predictability" | 14:42 |
| kgunn | mterry: yeah...i kinda view it as: we should strive for functional across both phone & desktop...but include consideration for performance on phone, delayed focus on perf for desktop | 14:43 |
| tsdgeos | mzanetti: ok, switching-previews next up the queue | 14:44 |
| mzanetti | tsdgeos: \o/ thanks | 14:45 |
| nic-doffay | mzanetti, got time for a review? The search scope is ready. | 14:45 |
| mzanetti | nic-doffay: in a moment | 14:45 |
| nic-doffay | mzanetti, cool here's the link for whenever: https://code.launchpad.net/~nicolas-doffay/unity8/scope-search-refactor/+merge/193787 | 14:45 |
| greyback | tsdgeos: https://code.launchpad.net/~gerboland/unity8/stop-name-clash/+merge/193789 | 14:46 |
| mzanetti | greyback: ah, what I wanted to ask: | 14:47 |
| mzanetti | greyback: on the weekend I've been working a bit more on some apps and got hit by this again: | 14:47 |
| mzanetti | greyback: run an app from cmdline, lock the screen, ctrl+c on the app, unlock the screen | 14:47 |
| mzanetti | greyback: it will get in a state where you have to reboot | 14:48 |
| greyback | mzanetti: eek | 14:48 |
| greyback | mzanetti: bug please :) | 14:48 |
| mzanetti | greyback: https://bugs.launchpad.net/unity8/+bug/1247864 | 14:53 |
| ubot5 | Ubuntu bug 1247864 in Unity 8 "device needs to be rebooted when an app closes while stopped" [Undecided,New] | 14:53 |
| greyback | mzanetti: mind if I rename? | 14:54 |
| mzanetti | feel free | 14:54 |
| mzanetti | greyback: also, feel free to increase priority :P | 14:54 |
| greyback | mzanetti: does unity8 actually crash, or just hang? | 14:54 |
| mzanetti | greyback: I don't know... can't restart it any more | 14:55 |
| mzanetti | maybe it hangs | 14:55 |
| greyback | mzanetti: ok | 14:55 |
| Cimi | mterry, hey michael! | 14:57 |
| mterry | Cimi, hello! | 14:57 |
| Cimi | mterry, I'm working on porting the wizard over cmake | 14:58 |
| mterry | Cimi, oh neat | 14:58 |
| mterry | Cimi, did system-settings switch over? | 14:59 |
| Cimi | mterry, yes | 14:59 |
| Cimi | mterry, but I'm failing and I need an help :) | 15:00 |
| Cimi | mterry, I pushed there lp:~unity-team/ubuntu-settings-components/wizard-cmake | 15:00 |
| dandrader | Saviq, tsdgeos fixed (at least locally) the qmltest failures in lp:~dandrader/unity8/drag-crash-1228336. | 15:01 |
| mterry | Cimi, the account plugin is missing a CMakeLists.txt file? | 15:03 |
| Cimi | mterry, might forgot to addd | 15:04 |
| Cimi | mterry, added | 15:04 |
| Cimi | mterry, but I dunno with the wizard app | 15:04 |
| Cimi | mterry, and the potfiles | 15:05 |
| tsdgeos | dandrader: you can ask for the type | 15:05 |
| dandrader | tsdgeos, it can be an EdgeDragArea or DirectionalDragArea. I think duck-typing is more convenient | 15:05 |
| tsdgeos | it can be more convinient | 15:06 |
| tsdgeos | but it'll break | 15:06 |
| dandrader | tsdgeos, does type checking also consider inheritance? | 15:06 |
| tsdgeos | don't know tbh, probably not | 15:06 |
| dandrader | tsdgeos, well, I think javascript is all about duck typing | 15:06 |
| tsdgeos | since at least what i know is "typeof" | 15:06 |
| mterry | Cimi, your accounts CMakeFile.txt is missing a closing parens on first lin | 15:07 |
| mterry | e | 15:07 |
| dandrader | tsdgeos, as opposed to hard type checking | 15:07 |
| tsdgeos | so it'll give you the "child-most" item | 15:07 |
| mterry | Cimi, why does wizard app have its own po file anyway? | 15:07 |
| tsdgeos | dandrader: that doesn't mak it right ;-D | 15:07 |
| tsdgeos | dandrader: anyway it's not me that was reviewing that MR | 15:07 |
| dandrader | tsdgeos, it's a different way of doing things. not right or worng | 15:07 |
| tsdgeos | mzanetti: can you do the re-review of last thing at https://code.launchpad.net/~dandrader/unity8/drag-crash-1228336/+merge/192183 ? | 15:08 |
| tsdgeos | dandrader: tell me that when you end up with an item with those properties and it breaks :D | 15:08 |
| mzanetti | tsdgeos: dandrader: hmm... isn't there a less intrusive solution for this? | 15:10 |
| dandrader | mzanetti, I'm open for suggestions | 15:10 |
| mzanetti | hmm | 15:10 |
| dandrader | mzanetti, one idea is for the UnityTest base class to already call this method | 15:12 |
| dandrader | in his initTestCase | 15:12 |
| mzanetti | dandrader: yeah, sounds better, but let me think for one more minute | 15:12 |
| dandrader | although I don't know what happens if an intance of it overrides this method | 15:13 |
| Cimi | mterry, because I thought of keeping it separate | 15:13 |
| Cimi | mterry, you have commit access, it's on unity-team | 15:14 |
| mzanetti | dandrader: fixing the flick() method to do stuff within the allowed range is not an option? | 15:14 |
| mterry | Cimi, oh, fair | 15:14 |
| dandrader | mzanetti, I didn't get it | 15:15 |
| mzanetti | dandrader: mhm... ok. well, please try to hide it more then | 15:15 |
| mterry | Cimi, ok, I fixed a couple syntax errors, so now you can at least get to the make portion of building | 15:17 |
| mterry | Cimi, looks like you aren't passing libhybris includes/ldflags to the accounts plugin | 15:17 |
| === dandrader is now known as dandrader|lunch | ||
| Cimi | mterry, thanks | 15:19 |
| Cimi | mterry, I'm *bad* at those things | 15:19 |
| Cimi | makefiels | 15:19 |
| Cimi | makefiles | 15:19 |
| Cimi | well, especially bad :) | 15:19 |
| tsdgeos | greyback: eh, the time test error in 5.2 is actually bad coding on our side | 15:25 |
| tsdgeos | Saviq: ↑ | 15:26 |
| tsdgeos | "A JSONObject is an unordered collection of name/value pairs" | 15:26 |
| tsdgeos | it just happen that 5.0 was returning it to us ordered | 15:26 |
| tsdgeos | and now it isn't | 15:26 |
| greyback | tsdgeos: interesting, maybe a little optimization v8 did? | 15:27 |
| tsdgeos | don't know tbh | 15:27 |
| tsdgeos | but we're not the only ones to hit this | 15:28 |
| tsdgeos | http://stackoverflow.com/questions/5773950/how-to-keep-an-javascript-object-array-ordered-while-also-maintaining-key-lookup | 15:28 |
| tsdgeos | this is browser-related | 15:28 |
| tsdgeos | but it's the same issue :D | 15:28 |
| tsdgeos | any taker? https://code.launchpad.net/~aacid/unity8/object_to_array/+merge/193801 | 15:40 |
| tsdgeos | mzanetti: greyback: ↑↑↑ | 15:40 |
| tsdgeos | it's easyyyyy | 15:40 |
| mzanetti | ok. can do | 15:41 |
| mzanetti | tsdgeos: and you're sure this doesn't break anything? | 15:42 |
| tsdgeos | i am | 15:42 |
| tsdgeos | what would it break? | 15:42 |
| mzanetti | ok... given that we only import this in a test | 15:43 |
| tsdgeos | lol | 15:43 |
| tsdgeos | that's also true | 15:43 |
| tsdgeos | why just not kill it? | 15:43 |
| mzanetti | yeah | 15:43 |
| mzanetti | ah right... this was used in the clock on the greeter iirc | 15:44 |
| mzanetti | and that one was dropped in favor of the indicator one | 15:44 |
| tsdgeos | well, approve this one and then let's see if we want to kill it or not | 15:44 |
| tsdgeos | tx | 15:47 |
| tsdgeos | hmmm | 15:55 |
| * tsdgeos confused | 15:55 | |
| tsdgeos | why is the autolander green but not merged?¿?¿ | 15:56 |
| tsdgeos | beucase it failed | 15:57 |
| tsdgeos | grrr | 15:57 |
| tsdgeos | my mitsake | 15:58 |
| tsdgeos | grrrrr2 | 15:58 |
| tsdgeos | third attempt :D | 15:59 |
| tsdgeos | dandrader|lunch: do you think we can make it so removeTimeConstraintsFromDirectionalDragAreas is invoked on the shell for all the tests? | 16:00 |
| === dandrader|lunch is now known as dandrader | ||
| Saviq | dandrader, thanks | 17:05 |
| === tvoss is now known as tvoss|dinner | ||
| === alan_g is now known as alan_g|EOD | ||
| davidcalle | mzanetti, hi, I'm testing some scopes with your music previews branch and I'm wondering if the cover art having a fixed height is by design. | 18:13 |
| * greyback eod | 18:28 | |
| === dandrader is now known as dandrader|bbl | ||
| === dandrader|bbl is now known as dandrader | ||
| === salem_ is now known as _salem | ||
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!