| davidcalle | Good morning all | 07:21 |
|---|---|---|
| tsdgeos | dednick: lots of commented stuff in that MR? | 08:46 |
| dednick | tsdgeos: stuff in VideoPlayer is waiting on a design review (may need the ShaderEffect), and the other is the sharing via content hub. | 08:48 |
| dednick | tsdgeos: will be removed before merging | 08:48 |
| tsdgeos | k | 08:48 |
| tsdgeos | dednick: why the "find_package(Qt5Multimedia 5.4 REQUIRED)" ? | 08:50 |
| dednick | tsdgeos: you mean the version, or the actual requirement? | 08:52 |
| tsdgeos | the requirement | 08:52 |
| dednick | the tests use qtmultimedia. | 08:52 |
| dednick | mocks i mean | 08:53 |
| dednick | tsdgeos: ^ | 08:53 |
| tsdgeos | dednick: do they? | 08:54 |
| tsdgeos | i can't find where | 08:54 |
| tsdgeos | i.e. tests/mocks/QtMultimedia/CMakeLists.txt only links to qml and quick | 08:54 |
| dednick | tsdgeos: oh. hm. maybe it was something i was doing before. | 08:56 |
| dednick | tsdgeos: actually, i think i was getting an import failure when running. | 08:59 |
| dednick | tsdgeos: qtdeclarative5-qtmultimedia-plugin is in the control file. so shouldn't it be in cmake? or only build reqs in cmake? | 08:59 |
| tsdgeos | dednick: only build reqs i'd say | 09:00 |
| tsdgeos | dednick: the import failure was on running tests or real running? | 09:01 |
| dednick | tsdgeos: real | 09:01 |
| tsdgeos | is that because you didn't have qtdeclarative5-qtmultimedia-plugin and then the cmake made you install it? | 09:01 |
| dednick | tsdgeos: well, it didnt make me install, because there wasn't a req in the cmake. | 09:02 |
| tsdgeos | aaaaand i lost intenret | 09:04 |
| tsdgeos | dednick: right, i mean it was fixed because you added the req in cmake | 09:05 |
| tsdgeos | and then it was fine | 09:05 |
| dednick | ya. i think so :/ if i'm remembering corrrectly | 09:06 |
| tsdgeos | ok, i guess it doesn't hurt | 09:06 |
| tsdgeos | dednick: autopkg doesn't like your new requirement https://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-wily/35/console | 10:28 |
| tsdgeos | so i guess remove it if it's really not needed on build time? | 10:28 |
| tsdgeos | and i disconnected again | 10:31 |
| tsdgeos | this internet is awful today too :( | 10:31 |
| dednick | tsdgeos: hm. weird | 10:46 |
| tsdgeos | cimi: https://code.launchpad.net/~aacid/unity8/fallbackImage/+merge/258399 ? | 10:47 |
| dednick | tsdgeos: works on mine. | 10:47 |
| tsdgeos | dednick: what works? the build? of course because you have the package | 10:48 |
| dednick | tsdgeos: cmake | 10:48 |
| tsdgeos | dednick: the thing is that the debian/control file is not asking for qtmultimedia | 10:49 |
| tsdgeos | and then cmake is | 10:49 |
| tsdgeos | so it won't build | 10:49 |
| cimi | tsdgeos, ok | 10:49 |
| dednick | tsdgeos: doesnt qtdeclarative5-qtmultimedia-plugin supply that? | 10:50 |
| tsdgeos | dednick: no, that's just the qml plugin | 10:51 |
| tsdgeos | the actual cmake stuff you're asking most likely comes in qtmultimedia5-dev | 10:51 |
| dednick | ok. but thats why i added it :) because the import failed... whatever. i removed it | 10:51 |
| === tsdgeos_ is now known as tsdgeos | ||
| === alan_g is now known as alan_g|lunch | ||
| === alan_g|lunch is now known as alan_g | ||
| === mzanetti is now known as mzanetti|otp | ||
| tsdgeos | mzanetti|otp: top approve! https://mail.google.com/mail/u/1/#inbox/14d94eb7ba7d37d3 | 13:32 |
| === mzanetti|otp is now known as mzanetti | ||
| mzanetti | tsdgeos, not sure what that link should be | 14:08 |
| ltinkl | secret message | 14:08 |
| tsdgeos | lol | 14:08 |
| tsdgeos | mzanetti: this one | 14:09 |
| tsdgeos | https://code.launchpad.net/~aacid/unity8/oxide_regression_workaround/+merge/260271 | 14:09 |
| mzanetti | tsdgeos, happroved! nice one | 14:10 |
| dandrader | mzanetti, so what should I be flashing nowadays when testing a merge proposal? | 14:10 |
| tsdgeos | mzanetti: and https://code.launchpad.net/~aacid/unity8/make_try_no_cpu_usage/+merge/260288 too | 14:10 |
| mzanetti | dandrader, in theory wily... let's see how long that'll work in practice | 14:10 |
| tsdgeos | dandrader: maybe you? ↑↑ | 14:10 |
| tsdgeos | damn, stupid me | 14:18 |
| tsdgeos | mzanetti: i'll resubmit https://code.launchpad.net/~aacid/unity8/make_try_no_cpu_usage/+merge/260288 based on https://code.launchpad.net/~aacid/unity8/oxide_regression_workaround/+merge/260271 otherwise CI will fail | 14:18 |
| tsdgeos | you'll have to re-approve | 14:18 |
| mzanetti | ah right | 14:18 |
| mzanetti | yeah, sure | 14:19 |
| dandrader | mzanetti, but my desktop is still vivid+overlay_ppa, should I move to wily as well? | 14:23 |
| mzanetti | dandrader, yeah, we'll probably have to. not right now as wily and vivid+overlay are still in sync for what is affecting us | 14:31 |
| mzanetti | but probably wont take long until we have to upgrade | 14:31 |
| greyback_ | dednick: fyi, this small part of the card is ready for review: https://code.launchpad.net/~gerboland/qtubuntu/window-close-support/+merge/249579 | 14:45 |
| greyback_ | tsdgeos: hey, could you please bump the version in https://code.launchpad.net/~aacid/qtmir/gles-sync-shellRotation/+merge/260114 - I had to rebuild qtmir in the PPA | 14:47 |
| greyback_ | tsdgeos: 0.4.5+15.10.20150527.1-0ubuntu1 | 14:47 |
| tsdgeos | sure | 14:48 |
| tsdgeos | done | 14:49 |
| greyback_ | thanks | 14:49 |
| === dandrader is now known as dandrader|afk | ||
| === dandrader|afk is now known as dandrader | ||
| dednick | greyback_: yup. i saw that already. | 15:47 |
| greyback_ | ok, just in case | 15:47 |
| dandrader | mzanetti, we (eg qtmir) depend on mir 0.13 but wily still has mir 0.12.1. not sure what to do :-? | 15:55 |
| dandrader | :-/ | 15:55 |
| dandrader | (when it comes to testing branches, that is) | 15:55 |
| dandrader | mzanetti, I guess vivid+overlay_ppa is in a better shape? | 15:56 |
| mzanetti | yeah, sil is supposed to sync it over to wily | 15:56 |
| mzanetti | kgunn, is there any update on that? ^ | 15:56 |
| kgunn | mzanetti: ok....long story short, we (mir team) goofed | 15:57 |
| kgunn | broke an abi, need to rebuild qtubuntu as well | 15:57 |
| kgunn | this is happening now | 15:57 |
| kgunn | and when this silo hits wily it'll make everything happy....and flow into wily | 15:58 |
| kgunn | mzanetti: there was a check in wily, that wasn't in overlay...so didn't catch the goof | 15:58 |
| dandrader | ok, will defer testing for a couple of days then. | 15:58 |
| kgunn | dandrader: mzanetti ....sorry | 15:59 |
| === alan_g is now known as alan_g|EOD | ||
| === dandrader is now known as dandrader|lunch | ||
| kgunn | @unity hey does the grid unit web tool that ben shared in his blog work for everyone ? | 18:38 |
| kgunn | or maybe i don't know how to use it... | 18:39 |
| mzanetti | kgunn, define "work"? | 18:40 |
| mzanetti | I seem to be unable to use it too atm | 18:40 |
| kgunn | mzanetti: yeah, thot i should see icons/gridlines....all i can do is select portrait/landscape and n7/n10...but nothing else | 18:41 |
| mzanetti | I can control stuff, and that works, but not figured yet how to actually add things | 18:41 |
| kgunn | was wondering if it was cause i'm in chrome | 18:42 |
| mzanetti | yeah... same thing I guess | 18:42 |
| * mzanetti firefox | 18:42 | |
| === dandrader|lunch is now known as dandrader | ||
| mzanetti | kgunn, same in safari | 18:44 |
| greyback__ | I can add lines clicking the + in the layouts bar | 18:44 |
| kgunn | mzanetti: ok,mailed ben and cc'd you | 18:45 |
| mzanetti | kgunn, hmpf... clicking the +es does something | 18:45 |
| kgunn | hehe...oops | 18:46 |
| kgunn | yep works here | 18:46 |
| mzanetti | interesting... | 18:48 |
| mzanetti | kgunn, am I understanding this correctly that the gu size would change on rotation? | 18:57 |
| greyback__ | "It can only be 40, 50, or 90 whole units along the short edge but the long edge can be variable" | 19:03 |
| greyback__ | which I'd interpret as a: no, gu size invariant of rotation | 19:04 |
| kgunn | but if you click it, it does change no ? | 19:05 |
| kgunn | it being between landscape/portrait | 19:06 |
| kgunn | oh wait i see what you mean.... | 19:06 |
| kgunn | right fixed to short edge | 19:06 |
| greyback__ | yep | 19:07 |
| kgunn | but it does change what is a vertical column | 19:07 |
| mzanetti | greyback__, example, the dash | 19:07 |
| mzanetti | device N7 | 19:07 |
| greyback__ | I'm not saying I understand the whole concept tho! | 19:07 |
| mzanetti | portrait, 50 gu | 19:07 |
| kgunn | anyone understand the colors ? | 19:08 |
| mzanetti | then click landscape | 19:08 |
| greyback__ | this is more confusing to me than properly responsive UI | 19:08 |
| mzanetti | which gives us 80 gu in width | 19:08 |
| mzanetti | thing is, the dash is now painted along those 80 gu in width | 19:08 |
| mzanetti | kgunn, no, haven't figured the colors yet | 19:09 |
| greyback__ | mzanetti: since the dash is a single column, just stretching the contents to fit is just fine IMO (the gridview relayouts to take advantage of the extra width) | 19:10 |
| greyback__ | but for a multi-column/layout app, ben's doc makes useful suggestions | 19:10 |
| mzanetti | yes | 19:10 |
| greyback__ | tho dash could evolve into a multi-column UI, i.e. it could be 2 cols in landscape mode, second column being another lens | 19:11 |
| greyback__ | that's up to design | 19:11 |
| mzanetti | but what you mean with streching? | 19:12 |
| mzanetti | isn't to point of this to not stretch? | 19:12 |
| greyback__ | mzanetti: just adjust its width to suit, and redraw to use that width (as a gridview will do) | 19:14 |
| greyback__ | again, just my interpretation, thye may have other ideas | 19:15 |
| mzanetti | that would cause only 4 columns on landscape N7. not sure if that's the intention | 19:15 |
| === dandrader is now known as dandrader|afk | ||
| === dandrader|afk is now known as dandrader | ||
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!