| === fginther is now known as fginther|away | ||
| === fginther|away is now known as fginther | ||
| Saviq | dednick, hey, could you review https://code.launchpad.net/~unity-team/unity8/caching-image-provider/+merge/224415 please | 07:21 |
|---|---|---|
| dednick | Saviq: sure | 07:23 |
| Saviq | dednick, thanks | 07:37 |
| mzanetti | moin | 07:42 |
| Saviq | o/ | 07:44 |
| Saviq | dednick, btw, https://code.launchpad.net/~nick-dedekind/unity8/move-indicator-qml/+merge/223947/comments/539798 | 07:44 |
| Saviq | dednick, the Dash one is actually a crash, probably same as testShell | 07:44 |
| Saviq | mzanetti, could you review https://code.launchpad.net/~mterry/unity8/is-active/+merge/223653 on this fine mornin' please? | 07:44 |
| mzanetti | ok | 07:45 |
| Saviq | ohnoes | 07:50 |
| Saviq | mhr3, you landed unity-api without corresponding changes to unity8 and -scopes-shell did ya? | 07:50 |
| mhr3 | Saviq, right, but it's compatible | 07:52 |
| Saviq | mhr3, sure, but FTBFS :P | 07:52 |
| mhr3 | Saviq, you can bump u8 when you implement support for customizations | 07:52 |
| mhr3 | Saviq, what? | 07:52 |
| Saviq | mhr3, pure virtual | 07:52 |
| mhr3 | oh.. mocks | 07:52 |
| Saviq | yup | 07:52 |
| mhr3 | right... sorry totally forgot about that | 07:52 |
| mhr3 | Saviq, do you want me to prep an mp? | 07:53 |
| Saviq | mhr3, doing, you'll review | 07:53 |
| mhr3 | ok | 07:53 |
| Saviq | truth be told mocks should live in unity-api | 07:54 |
| Saviq | mhr3, https://code.launchpad.net/~saviq/unity8/mock-customizations/+merge/224757 | 08:01 |
| Saviq | brb | 08:02 |
| Cimi | is unity-mir Unity.Application 0.1, right? | 08:06 |
| mzanetti | Cimi: yes | 08:06 |
| Cimi | this Unity.Application comes from unity-mir | 08:06 |
| Cimi | ok | 08:06 |
| Cimi | so weird | 08:06 |
| mzanetti | why? | 08:07 |
| Cimi | I need gerry :) | 08:07 |
| Cimi | https://bugs.launchpad.net/ubuntu-welcome-wizard/+bug/1334429 | 08:08 |
| ubot5 | Ubuntu bug 1334429 in Ubuntu Welcome Wizard "touch event allowed through OSK to what is underneath" [High,New] | 08:08 |
| Cimi | but we're using OSK from unity mir | 08:08 |
| Cimi | so I don't know why... | 08:08 |
| mzanetti | Cimi: I think we have an InputFilterArea in unity | 08:08 |
| Cimi | ah nio | 08:09 |
| Cimi | nevermind | 08:09 |
| Cimi | missed point 3 | 08:09 |
| mzanetti | Cimi: should this probably be reassigned to the welcome wizard? https://bugs.launchpad.net/ubuntu/+source/ubuntu-system-settings/+bug/1334203 | 08:18 |
| ubot5 | Ubuntu bug 1334203 in ubuntu-system-settings (Ubuntu) "ubuntu-system-settings-wizard crashes/freezes when using the back button" [Undecided,New] | 08:18 |
| Cimi | mzanetti, do you have a stacktrace? | 08:19 |
| Cimi | doesn't crash for me | 08:19 |
| mzanetti | Cimi: its 100% reproducable | 08:19 |
| mzanetti | Cimi: did you follow the instructions in the bug precisely? | 08:19 |
| mzanetti | step 1 mostly | 08:19 |
| Cimi | I just saw it | 08:19 |
| Cimi | will bootstrap | 08:20 |
| Cimi | mzanetti, if you could share that crash log | 08:23 |
| Cimi | mzanetti, probably not a bug in the wizard, but a broken plugin | 08:24 |
| Saviq | Cimi, could you please review https://code.launchpad.net/~saviq/unity8/fix-card-vjournal/+merge/224727 quickly | 08:26 |
| Saviq | Cimi, I'd like to land this fix with the FTBFS fix | 08:26 |
| mhr3 | Saviq, jumped the gun too soon on the ftbfs fix, should bump deps in control | 08:26 |
| Saviq | mhr3, mhm | 08:27 |
| Cimi | need to reboot | 08:27 |
| Cimi | bloody usb not working | 08:27 |
| Cimi | http://paste.ubuntu.com/7710021/ | 08:27 |
| Cimi | Saviq, ^ | 08:27 |
| Cimi | mzanetti, ^ | 08:27 |
| Cimi | this is my dmesg :( | 08:28 |
| Saviq | Cimi, yup | 08:28 |
| Saviq | Cimi, reboot | 08:28 |
| Saviq | mhr3, pushed | 08:28 |
| mhr3 | Saviq, yep, +1 | 08:29 |
| Cimi | Saviq, ok works, what was that? | 08:29 |
| mzanetti | Cimi: did you reboot the machine or the tablet? | 08:30 |
| Cimi | mzanetti, pc | 08:30 |
| mzanetti | really weird... | 08:30 |
| Saviq | Cimi, https://code.launchpad.net/~saviq/unity8/fix-card-vjournal/+merge/224727 | 08:31 |
| Cimi | Saviq, not that | 08:31 |
| Cimi | Saviq, the reason reboot is required | 08:31 |
| Saviq | Cimi, dunno, something in Ubuntu usb gets confused | 08:31 |
| Saviq | Cimi, adb/mtp/blah | 08:31 |
| Cimi | ah ok | 08:31 |
| Saviq | Cimi, to test the MP easily: http://paste.ubuntu.com/7710041/ | 08:32 |
| Saviq | Cimi, and then in tryDash you'll see in trunk 3 columns on top of another | 08:32 |
| Cimi | ok | 08:32 |
| Saviq | with the fix - 3 separate columns | 08:32 |
| Saviq | complete review fail on my part :| | 08:32 |
| Saviq | Cimi, if you have doubts about width/height, Loader assumes the loaded item's dimensions, or forces its own if are explicit | 08:33 |
| Saviq | Cimi, so Loader { width: foo; sourceComponent: blah } | 08:33 |
| Cimi | \i see | 08:34 |
| Saviq | Cimi, basically means blah will be foo-wide, and loader will be blah-wide | 08:34 |
| Saviq | Cimi, FFR http://qt-project.org/doc/qt-5/qml-qtquick-loader.html#loader-sizing-behavior | 08:34 |
| Cimi | Saviq, subtitle is cut at the bottom | 08:39 |
| Saviq | Cimi, you ACKed this fix, remember https://code.launchpad.net/~saviq/unity8/fix-header-height/+merge/224269 ? | 08:39 |
| Saviq | ;) | 08:39 |
| Cimi | cool | 08:39 |
| Cimi | ok | 08:39 |
| Cimi | well fix works | 08:39 |
| Saviq | Cimi, yeah, needs tests, I know | 08:40 |
| Saviq | Cimi, but let's just get it in, fix is obvious | 08:40 |
| Cimi | mzanetti, can I bootstrap without reflashing? | 08:44 |
| mzanetti | Cimi: I don't know... probably you can wipe ~ and /userdata, but I never really tried it | 08:45 |
| mhr3 | dednick, replied on the mp | 08:45 |
| Saviq | @unity please grap an MP to review (and claim it in the MP page when doing so), we're drowning here | 08:51 |
| Cimi | Saviq, I have few that can be grabbed :P | 08:52 |
| Saviq | when you have some time / strength to do so, of course | 08:52 |
| Saviq | Cimi, yup, works both ways | 08:52 |
| Saviq | dednick, btw, how do you find the reworked QmlPlugin.cmake? any comments, other than making it support folder hierarchies? | 08:54 |
| Cimi | mzanetti, yeah i can reproduce | 08:58 |
| mzanetti | ok | 08:58 |
| Cimi | mzanetti, problem I need to install dbg symbols | 08:58 |
| Cimi | but I have no wifi :) | 08:58 |
| mzanetti | :) | 08:58 |
| Cimi | and no rw filesystem | 08:59 |
| mzanetti | you'll figure a solution, I'm sure | 08:59 |
| Cimi | mzanetti, I'll remove the back button :) | 08:59 |
| mzanetti | Cimi: so the ro file system is not a problem | 08:59 |
| mzanetti | Cimi: I also had this when the wizard landed | 08:59 |
| mzanetti | and I didn't bootstrap and had a rw image | 08:59 |
| mhr3 | Saviq, was there ever solution to :native not working on jenkins? | 08:59 |
| mzanetti | Cimi: I guess cleaning up every data that the welcome wizard produces would be enough | 09:00 |
| Saviq | mhr3, I believe they fixed it, yes | 09:01 |
| Cimi | mzanetti, it produces no data | 09:01 |
| Saviq | mhr3, bug #1322294 remains, though | 09:01 |
| ubot5 | bug 1322294 in launchpad-buildd "Recipes don't support :native multi-arch dependency" [High,Triaged] https://launchpad.net/bugs/1322294 | 09:01 |
| mhr3 | Saviq, not sure about that https://jenkins.qa.ubuntu.com/job/unity-team-unity-scope-click-devel-utopic-amd64-ci/129/console | 09:01 |
| Saviq | mhr3, they did just strip :native before, maybe they reverted this | 09:02 |
| mhr3 | right... "fixed" | 09:02 |
| Saviq | mhr3, or maybe they didn't "populate" all the jenkins | 09:02 |
| Saviq | mhr3, well, until they start x-compiling, everything's native ;) | 09:03 |
| mhr3 | Saviq, anyway, i guess fginther will know | 09:03 |
| Saviq | mhr3, I *think* it was psivaa who knew about it before, too, so try with him | 09:03 |
| mhr3 | Saviq, k, thx | 09:04 |
| Saviq | dednick, reviewed u-s-c menu.plugin | 09:09 |
| dednick | Saviq: you mean the QmlPlugin.cmake in the suru-switch branch? | 09:17 |
| Saviq | dednick, hm? no, the one you copied to menu.plugin | 09:17 |
| Saviq | dednick, "new" as in the one we have in unity8 for some time | 09:18 |
| Saviq | dednick, was just wondering whether we should put it in cmake-extras already or is there still work on it needed | 09:18 |
| dednick | Saviq: sorry, getting my branches mixed up. i meant https://code.launchpad.net/~unity-team/unity8/manual-qmltypes/+merge/224578 | 09:20 |
| dednick | which is the one i copied from | 09:20 |
| Saviq | dednick, well, yeah | 09:20 |
| Saviq | dednick, that one | 09:20 |
| dednick | but yeah, it's good for usc. should put it in cmake-extras | 09:21 |
| dednick | didnt have to mod | 09:21 |
| mzanetti | Saviq: well, we have that commit hook... its just that noone seems to be using it | 09:25 |
| Saviq | mzanetti, yeah exactly... | 09:25 |
| mzanetti | well, I use it all the time... | 09:25 |
| mzanetti | and have also updated it here to check for stale tags | 09:25 |
| Saviq | mzanetti, well, also, commit is probably too often | 09:26 |
| mzanetti | works fine for me... its less than 1 sec | 09:26 |
| Saviq | mzanetti, yeah, but you might want to split things into several commits | 09:26 |
| Saviq | mzanetti, and then the hook will tell you NO | 09:26 |
| mzanetti | the hook doesn't disallwo committing... it just tells you: this will fail CI if you continue | 09:26 |
| mzanetti | and keeps a copy of the commit message so you don't have to type it again in case you decide to fix it and recommit | 09:27 |
| Saviq | mhm | 09:28 |
| Saviq | mzanetti, hmm where is it? | 09:28 |
| mzanetti | .bazaar/plugins | 09:28 |
| Saviq | mzanetti, I think we need a script in . | 09:31 |
| Saviq | mzanetti, that will do the checks | 09:31 |
| Saviq | mzanetti, and the plugin (if installed) will call it | 09:31 |
| Saviq | mzanetti, this way you can run it without the hook | 09:32 |
| mzanetti | yep, works for me | 09:32 |
| Saviq | like a pre_push.py | 09:32 |
| mzanetti | also, I'd like to have the hook more visible/easy to install | 09:33 |
| mzanetti | but haven't gotten a good idea for that yet | 09:34 |
| mzanetti | I'm sure more people would use it if they'd be aware of it | 09:34 |
| Saviq | mzanetti, this script in . could check for the hook | 09:34 |
| Saviq | mzanetti, and say "if you want this to run on commit, do foo" | 09:34 |
| mzanetti | yeah, sounds reasonable | 09:34 |
| Saviq | mzanetti, no point for it being in .bazaar, either | 09:34 |
| Saviq | mzanetti, we can move it to tools/ | 09:35 |
| mzanetti | right... | 09:35 |
| Saviq | since you can't have a branch-local plugin | 09:35 |
| mzanetti | yeah, too bad... | 09:35 |
| mzanetti | that'd be useful | 09:35 |
| Saviq | ooh guess what, git does! ;P | 09:36 |
| mzanetti | you can? | 09:36 |
| mzanetti | ah | 09:36 |
| mzanetti | I read "it does", missed the g | 09:36 |
| mzanetti | of course git does | 09:36 |
| Saviq | ;) | 09:36 |
| mzanetti | Saviq: btw. when you touch this, if you can think of a better to figure the branch, one that doesn't break when we move shell.qml to another subdir :D | 09:39 |
| Saviq | mzanetti, that's the thing | 09:40 |
| Saviq | mzanetti, the hook will just check for a script in . | 09:40 |
| Saviq | mzanetti, if there - run it, otherwise ignore | 09:40 |
| Saviq | mzanetti, script with a known name | 09:40 |
| mzanetti | ok... it'll break when you move the script to another dir... but I guess already better than some "random" qml file of the codebase | 09:40 |
| Saviq | mzanetti, yeah, and we won't move the script to another dir, since we want it to be visible | 09:41 |
| Saviq | mzanetti, and available at your fingertips | 09:41 |
| mzanetti | ack | 09:41 |
| mzanetti | Saviq: maaan... that new-header branch develops to a beast... | 09:48 |
| Saviq | mzanetti, story of my life ;P | 09:49 |
| mzanetti | figured that we actually could get rid of the PageHeader::childItem and put the uitk's header in there... | 09:49 |
| mzanetti | then there's already a back button | 09:49 |
| mzanetti | which doesn't match design | 09:49 |
| Saviq | mzanetti, indeed | 09:49 |
| mzanetti | then it has like a thousand states I mustn't break :D | 09:49 |
| Saviq | mzanetti, you probably could | 09:50 |
| Saviq | mzanetti, since the states are meant to deal with the search entry | 09:50 |
| Saviq | mzanetti, which with the new header should probably behave differently | 09:51 |
| mzanetti | hmm... should it? | 09:51 |
| Saviq | mzanetti, well, ultimately we should take the behaviour from UITK | 09:51 |
| Saviq | mzanetti, but since it's not there yet... | 09:51 |
| Saviq | mzanetti, I think we should get the UITK header as is and replace its contents with {text,image}, text entry, just sliding up/down on search activation | 09:52 |
| Saviq | mzanetti, since the search icon doesn't make sense when the search entry is on screen | 09:52 |
| Saviq | mzanetti, and vice-versa | 09:52 |
| mzanetti | Saviq: right... that's what it does now... but it hides the back button, which is a probme | 09:53 |
| mzanetti | problem | 09:53 |
| Saviq | mzanetti, right, because it replaces the whole header from UITK | 09:53 |
| mzanetti | so I could obviously just leftMargin the textfield | 09:53 |
| mzanetti | but I fugred that wouldn't pass your review | 09:53 |
| Saviq | mzanetti, I mean it should be UITKHeader { contentItem: OurComponent { } } | 09:53 |
| Saviq | mzanetti, where OurComponent would be {text|image} + text entry | 09:54 |
| mzanetti | I don't think the uitk's header has such a thing | 09:54 |
| Saviq | it does | 09:54 |
| * mzanetti checks again | 09:54 | |
| Saviq | it might be __ | 09:54 |
| Saviq | but will be public later | 09:54 |
| mzanetti | I must be blind | 09:55 |
| * Saviq looks | 09:55 | |
| mzanetti | ah... there's something | 09:56 |
| mzanetti | contentsContainer might do that | 09:56 |
| Saviq | yeah | 09:56 |
| Saviq | mzanetti, oh well, PageHeader still needs to wrap it | 09:57 |
| Saviq | mzanetti, because we need to add departments to it | 09:57 |
| mzanetti | true | 09:57 |
| mzanetti | also the landscape behavior would break | 09:58 |
| dednick | Saviq: thanks for review. pushed update. | 09:59 |
| Saviq | mzanetti, well, that's fine, we don't want to bring the text entry in in landscape any more | 09:59 |
| Saviq | mzanetti, only on search press | 09:59 |
| Saviq | dednick, kk | 10:02 |
| Saviq | dednick, hmm, I'd have left find_package(Qt5Core REQUIRED), otherwise you have no qt5_use_modules | 10:05 |
| Saviq | dednick, in your case it basically gets included somewhere else, but I think it's good to have the one for Core | 10:06 |
| dednick | Saviq: it's in the root cmake | 10:07 |
| dednick | Saviq: but mk | 10:08 |
| Saviq | dednick, yeah, I know, probably depends on how redistributable we want it to be | 10:08 |
| Saviq | dednick, hmm, why LGPL? | 10:08 |
| Saviq | dednick, you don't need LGPL for dlopening | 10:08 |
| dednick | Saviq: because i don't really know which one to use.... | 10:08 |
| dednick | and the other files around seemed to use it | 10:09 |
| Saviq | dednick, you only need LGPL for libs that are linked against | 10:09 |
| dednick | Saviq: does it matter? | 10:09 |
| Saviq | dednick, IANAL ;) | 10:09 |
| Saviq | it is lesser for some reason :) | 10:09 |
| dednick | :) | 10:09 |
| Saviq | dednick, we default to GPLv3 | 10:09 |
| Saviq | dednick, fall back to LGPLv3 when required | 10:10 |
| Saviq | dednick, so please make it GPL, if others are already LGPL, let's fix in another branch | 10:11 |
| dednick | Saviq: doing | 10:11 |
| Saviq | dednick, also, check what's in debian/copyright | 10:11 |
| Saviq | dednick, hmm it's LGPL there | 10:12 |
| Saviq | dednick, sorry, let's leave it be then | 10:13 |
| dednick | hehe | 10:13 |
| Saviq | dednick, and fix it across the board after verifying that's needed | 10:13 |
| dednick | Saviq: ok. pushed the fin_package change | 10:14 |
| Saviq | dednick, can't top-ack | 10:14 |
| Saviq | Cimi, add me to https://launchpad.net/~ubuntu-settings-components-team please | 10:15 |
| dednick | Saviq: need packaging approval? | 10:15 |
| Cimi | Saviq, done | 10:15 |
| Saviq | dednick, no, just not on the team ;) | 10:15 |
| dednick | since i removed the -assets package | 10:15 |
| Saviq | better | 10:15 |
| Saviq | dednick, we'll get pkg review in silo | 10:16 |
| dednick | Saviq: ok. thanks | 10:16 |
| Saviq | dednick, unless you find someone who will take a look ;) | 10:16 |
| Saviq | /fiid | 10:16 |
| Saviq | /food that is | 10:16 |
| Saviq | mzanetti, know what? just do the simplest transition possible for now, we'll get a proper one from UITK when it's there anyway | 10:18 |
| mzanetti | ack. on it :) | 10:19 |
| Saviq | mzanetti, thinking here... | 10:26 |
| mzanetti | don't :D | 10:27 |
| Saviq | I *know*, sorry | 10:27 |
| mzanetti | I'm nearly done | 10:27 |
| Saviq | mzanetti, we have basically two header configurations → [back], {text|image}, favourite, search | 10:27 |
| Saviq | back, text_entry | 10:28 |
| mzanetti | yep | 10:28 |
| Saviq | mzanetti, and if we just replace the actions in the current UITK header | 10:28 |
| Saviq | mzanetti, they will just appear/disappear as they do now | 10:28 |
| Saviq | erm | 10:28 |
| Saviq | I mean because there's no transitions in the UITK | 10:28 |
| Saviq | mzanetti, let's use two UITK headers and swap them (slide them in/out)? | 10:29 |
| mzanetti | phew... :) | 10:29 |
| mzanetti | that's what I'm doing | 10:29 |
| Saviq | good | 10:29 |
| mzanetti | or well, not really | 10:29 |
| mzanetti | I just attached a back button myself to the textfield | 10:29 |
| mzanetti | so for now it keeps behavior as it was... hope that's fine | 10:30 |
| mzanetti | well, you'll see in a minute | 10:30 |
| mzanetti | will just have to update the preview header | 10:30 |
| Saviq | mzanetti only problem we'll end up with the search entry *and* the search button | 10:30 |
| mzanetti | no | 10:30 |
| Saviq | in landscape? | 10:30 |
| mzanetti | oh... right | 10:30 |
| Saviq | mzanetti, I think what I'm asking will simplify this quite a bit | 10:31 |
| Saviq | mzanetti, like we get rid of the Narrow states in PageHeader | 10:31 |
| Saviq | mzanetti, anyway... I know you'll get there | 10:32 |
| * Saviq focuses on the cereal, or it will be keyboard food | 10:33 | |
| Cimi | how do I write http://paste.ubuntu.com/7710506/ so that returns 0 or null when unitymenumodel is not valid? | 10:40 |
| Saviq | Cimi, you probably want dednick for this | 10:45 |
| Cimi | mzanetti, I think that is the crasher | 10:45 |
| mzanetti | fix it :P | 10:46 |
| Cimi | mzanetti, model being destroyed but property being manipulated | 10:46 |
| * mzanetti rewrites PageHeader.qml :/ | 10:46 | |
| mzanetti | Saviq: so in widescreen, we behave the same as in portrait, ok? | 10:47 |
| mzanetti | having a super long textfield | 10:48 |
| Saviq | mzanetti, no, make it 60GU max, falling back to 40GU in > 60GU screens | 10:48 |
| Saviq | mzanetti, same behavior as we have with other width-dependant things | 10:48 |
| mzanetti | Saviq: and the title would then stay? | 10:49 |
| Saviq | mzanetti, no | 10:49 |
| Saviq | mzanetti, replace the title with 40GU wide text entry | 10:49 |
| mzanetti | meaning empty space at the left of the text entry | 10:49 |
| Saviq | right | 10:49 |
| mzanetti | mhm... ok | 10:49 |
| Saviq | at the right | 10:49 |
| Saviq | I mean | 10:49 |
| mzanetti | oh... moving the label to left | 10:50 |
| mzanetti | err. the textfield | 10:50 |
| Saviq | | < [text entry] | | 10:50 |
| mzanetti | right | 10:50 |
| mzanetti | ok | 10:50 |
| Saviq | | < Title f s | | 10:50 |
| Saviq | two configs like that | 10:51 |
| mzanetti | ack | 10:51 |
| mzanetti | that sounds like a looong friday :D | 10:51 |
| Saviq | mzanetti, I really don't think it should be that complex... | 10:52 |
| Saviq | more red than green, at least | 10:52 |
| mzanetti | yep... its not too bad actually | 10:53 |
| Cimi | dednick, ping :) | 10:57 |
| === MacSlow is now known as MacSlow|lunch | ||
| dednick | Cimi: sup? | 12:04 |
| === alan_g is now known as alan_g|lunch | ||
| Cimi | dednick, I have a crash in the wizard | 12:16 |
| Cimi | dednick, happens with http://paste.ubuntu.com/7710506/ | 12:16 |
| Cimi | dednick, if I am in the wifi page then I tap "back" | 12:17 |
| Cimi | dednick, it might be that unitymenumodel is deleted or something like that | 12:17 |
| dednick | Cimi: could be related to: https://code.launchpad.net/~charlesk/qmenumodel/lp-1282282-dangling-UnityMenuAction-pointer/+merge/211211 | 12:25 |
| Cimi | can we trigger https://code.launchpad.net/~charlesk/qmenumodel/lp-1282282-dangling-UnityMenuAction-pointer/+merge/211211 ? | 12:33 |
| Saviq | Cimi, wdym trigger? | 12:37 |
| Cimi | Saviq, ci | 12:37 |
| Cimi | Saviq, cannot sbuild | 12:37 |
| Cimi | https://code.launchpad.net/~charlesk/qmenumodel/lp-1282282-dangling-UnityMenuAction-pointer/+merge/211211/comments/497978 | 12:38 |
| Saviq | Cimi, done | 12:39 |
| Saviq | Cimi, it was rather old indeed | 12:39 |
| Saviq | xnox, hey, could you please look at qmenumodel's x-compiling? it's definitely missing :any for python, but not sure -gi and -dbus should also get :any? | 12:45 |
| === alan_g|lunch is now known as alan_g | ||
| === _salem is now known as salem_ | ||
| Saviq | Cimi, reviewed infog | 13:06 |
| Saviq | oh wow, suggestions in gdocs | 13:11 |
| Saviq | handy | 13:12 |
| dandrader | greyback, mzanetti should we keep using this doc to track qt comp bugs? https://docs.google.com/a/canonical.com/spreadsheets/d/1Balt7jlHQlNuf7RBuXYjnYJWaICd_wTRpSVfTYYb3cQ/edit#gid=0 | 13:24 |
| dandrader | I think we should, so I will update it with the latest problems caused by qt 5.3 and latest mir/devel | 13:24 |
| greyback | dandrader: ok with me | 13:24 |
| cwayne | greyback: mzanetti: dandrader: do you guys have any idea when actual N7 landscape support might land? | 13:24 |
| greyback | cwayne: we plan within 2 weeks from now | 13:25 |
| cwayne | greyback: wonderful, thanks | 13:26 |
| greyback | dandrader: PPA in working order, patch for input crash added to Mir | 13:26 |
| dandrader | cwayne, we had it but qt 5.3 caused a regression on that | 13:26 |
| dandrader | greyback, great! | 13:26 |
| elopio | Saviq: in order to have a flake8 make build task, we would have to add it to build-deps, right? | 13:27 |
| greyback | dandrader: I want to clear the fixed bugs from that spreadsheet | 13:27 |
| cwayne | dah, thanks a lot qt5.3 | 13:27 |
| dandrader | greyback, yeah. they are just polluting the view | 13:27 |
| Saviq | elopio, yup | 13:30 |
| Saviq | elopio, but since I wrote that | 13:30 |
| Saviq | elopio, we discussed a pre-commit check / hook that we'd add this to instead | 13:30 |
| Saviq | elopio, so let's leave it be for now | 13:30 |
| elopio | Saviq: ok. We had a problem on the toolkit adding flake8 as a build-dep because it's not on main. | 13:31 |
| elopio | a pre-commit check sounds good. | 13:31 |
| Saviq | elopio, k | 13:31 |
| elopio | Saviq: the branch is ready for review again, btw. Thanks for your comments. | 13:32 |
| Saviq | elopio, tx! | 13:32 |
| Cimi | Saviq, yes I copied, because I expect UITK delay things | 13:42 |
| Saviq | Cimi, that's not a solution I'm afraid | 13:43 |
| Cimi | Saviq, I can put the changes when we agree on API | 13:43 |
| Saviq | Cimi, if we copy it now, we'll be stuck with it forever, you know that? ;) | 13:43 |
| Cimi | Saviq, why? | 13:43 |
| Saviq | Cimi, because there will be no pressure to fix | 13:43 |
| Cimi | Saviq, well fix comes from our side anyway | 13:43 |
| elopio | Saviq: I can try to make the hook. I'll take a look at the existing one. | 13:45 |
| Saviq | elopio, we want to change stuff a little | 13:45 |
| Saviq | elopio, make a script in . | 13:45 |
| Saviq | elopio, that we can run manually | 13:45 |
| Saviq | elopio, so that it's visible | 13:45 |
| Saviq | elopio, and the hook would just check for that file and execute if there, ignore otherwise | 13:46 |
| Saviq | elopio, this way it can be used without installing the hook | 13:46 |
| Saviq | elopio, and hook could be reused | 13:46 |
| Saviq | if other projects include such a script in their root | 13:46 |
| Saviq | Cimi, UITK has been very receiving recently - they've opened v1.1 of the API | 13:47 |
| Saviq | Cimi, so please prep an MP to include the different fade modes | 13:47 |
| Saviq | Cimi, see https://code.launchpad.net/~saviq/ubuntu-ui-toolkit/add-icon-source-property/+merge/224122 for how they want the new APIs put in | 13:48 |
| === MacSlow|lunch is now known as MacSlow | ||
| Cimi | Saviq, how shall we call that API? | 14:10 |
| Saviq | Cimi, mode / fadeMode | 14:10 |
| Cimi | Saviq, and the fade modes? | 14:10 |
| Saviq | Cimi, overlay, cross | 14:10 |
| mterry | Saviq, did you mean to abstain on the platform-api MP? | 14:18 |
| Cimi | Saviq, https://code.launchpad.net/~cimi/unity8/card_emblems_single_container/+merge/223899 merged | 14:25 |
| === alan_g is now known as alan_g|tea | ||
| paulliu | Are you speaking on the mumble? | 14:33 |
| Cimi | paulliu, yes | 14:34 |
| Cimi | paulliu, still talking | 14:34 |
| paulliu | Cimi: yes. Thanks. Now it is ok. | 14:35 |
| === alan_g|tea is now known as alan_g | ||
| Cimi | dednick, unfortunately doesn't fix it | 14:40 |
| dednick | Cimi: you'll need to get me a crashdump in that case | 14:41 |
| === dandrader is now known as dandrader|afk | ||
| === dandrader|afk is now known as dandrader | ||
| === gatox is now known as gatox_lunch | ||
| fginther | dandrader, does lp:unity-system-compositor/devel-mir-next replace lp:~mir-team/unity-system-compositor/development-branch | 17:01 |
| dandrader | kgunn, ^^^ | 17:01 |
| dandrader | fginther, I would say so | 17:02 |
| fginther | dandrader, ok, I'll go with that | 17:02 |
| === alan_g is now known as alan_g|EOW | ||
| mhall119 | mhr3: ping | 17:12 |
| mhr3 | mhall119, pong | 17:12 |
| mhall119 | mhr3: cwayne pinged me about https://bugs.launchpad.net/ubuntudeveloperportal/+bug/1328559 | 17:13 |
| ubot5 | Ubuntu bug 1328559 in Ubuntu App Developer site "14.04 scopes api docs use missing framework "ubuntu-sdk-14.04-clibs"" [Undecided,Confirmed] | 17:13 |
| mhall119 | the example click manifest on there uses "framework": "ubuntu-sdk-14.04-clibs", | 17:13 |
| mhall119 | which isn't actually a framework on the phone | 17:13 |
| mhall119 | I'm also not sure if the apparmor templates "ubuntu-scope-local-content" and "ubuntu-scope-network" exist | 17:14 |
| mhr3 | mhall119, hm, we were updating that recently, should be fixed in latest docs | 17:15 |
| mhall119 | latest docs package in utopic? | 17:15 |
| mhr3 | mhall119, yep | 17:20 |
| mhall119 | mhr3: ah, yes, looks good there, I'll update production | 17:21 |
| mhr3 | although the whole situation is odd | 17:24 |
| mhr3 | 14.04 docs are really for the 14.10 version, so they don't strictly apply :/ | 17:24 |
| mhall119 | mhr3: ah, right, I was only updating the 14.10 docs previously, should I replce the 14.04 docs with the new 14.10 ones, or leave them as they were? | 17:25 |
| mhall119 | or just point to only the 14.10 docs in http://developer.ubuntu.com/scopes/api/ | 17:26 |
| mhr3 | mhall119, yea, that will probably be the best | 17:26 |
| mhall119 | ok, I'll do that then | 17:27 |
| mhall119 | cwayne: kicking it back to you to decide what you want to do with the rest of the bug, I made it "Won't Fix" for the devportal | 17:28 |
| === dandrader is now known as dandrader|lunch | ||
| === gatox_lunch is now known as gatox | ||
| cwayne | mhall119: so the site is going to point at the 14.10 docs now? | 18:02 |
| kgunn | fginther: yes, sorry bout that | 18:12 |
| mhall119 | cwayne: already does | 18:21 |
| mhall119 | the old URL is still up, but nothing should link to it anymore | 18:21 |
| mhall119 | cwayne: see http://developer.ubuntu.com/scopes/api/ | 18:22 |
| cwayne | okay, so then essentially it should be considered fixed i'd say | 18:22 |
| mhall119 | cwayne: can I mark it fixed for savilerow too? | 18:28 |
| cwayne | mhall119: please do | 18:29 |
| mhall119 | cwayne: done | 18:29 |
| cwayne | mhall119: <3 thanks | 18:30 |
| === dandrader|lunch is now known as dandrader | ||
| === rsalveti_ is now known as rsalveti | ||
| === irsol_ is now known as irsol | ||
| dandrader | kgunn_, do you know the situation of the "dash as an app" work? | 21:01 |
| dandrader | mzanetti, are you still around? | 21:02 |
| === irsol_ is now known as irsol | ||
| === salem_ is now known as _salem | ||
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!