/srv/irclogs.ubuntu.com/2015/01/19/#ubuntu-unity.txt

liuxghow are we going to call today?06:02
=== seb128_ is now known as seb128
=== Malsasa_ is now known as Malsasa
=== timon is now known as Guest78191
=== Malsasa is now known as Guest43085
=== Malsasa_ is now known as Malsasa
BexHi!09:06
BexI'm trying to set up a development environment for unity 8, but I get some trouble with the dependencies.09:06
BexIs 14.04 supported for development?09:07
tsdgeosi'm not sure, most probably not09:09
tsdgeosmaybe not even 14.10 is09:09
Cimitsdgeos, morning!09:32
tsdgeoshi ho09:32
Cimitsdgeos, you know a bit on how we open scopes/close them in the dash?09:33
tsdgeosa bit09:33
Cimiif we open the cinema scope from the store, then we should close the store in background, right?09:33
Cimitsdgeos, I am not sure we are doing that at the moment, not that I can see many closeScope in Dash.qml09:34
Cimitsdgeos, then fake_scope.ccp at line 211 seems to expect sth else09:34
tsdgeoswe're opening a temp scope from a temp scope?09:35
Cimim_openScope that is never updated09:35
Cimitsdgeos, from the app store yes...09:35
tsdgeosi don't think that was ever tested09:35
tsdgeosnor specified what it should do09:35
tsdgeosso it's probably not working yes09:37
tsdgeosreading the code i see something wrong already09:37
tsdgeosand don't trust fake_scope either09:37
tsdgeosit's just a fake scope :D09:37
tsdgeosCimi: do we know what has to happen when you open a temp scope from a temp scope interaction wise?09:38
CimiCimi, I believe replacing the temp scope with the opened scope?09:40
tsdgeossure, but no animation or anything at all09:40
tsdgeosjust plain straight replace?09:40
Cimitsdgeos, I don't know...09:43
tsdgeosand back will skip the store and bring you to dash?09:43
tsdgeosCimi: well we need to know before coding anything09:43
Cimino idea for that09:43
Cimitsdgeos, https://bugs.launchpad.net/ubuntu/+source/unity8/+bug/141033709:45
ubot5Launchpad bug 1410337 in unity8 (Ubuntu) "Launching a scope once installed is broken" [High,Triaged]09:45
tsdgeosCimi: let's ask jounih then09:46
tsdgeoswhat, why does my phone keep rebooting itself!09:46
tsdgeosagain09:49
tsdgeosthis is nuts09:49
tsdgeosaaaaaaaaaaaaaaaaaaah09:52
tsdgeosstopit!09:52
Wellarkhey, I'm trying to build unity8 trunk on vivid desktop and the build fails with http://pastebin.ubuntu.com/9783989/10:42
Wellarktsdgeos: --^10:42
* tsdgeos clicks10:42
tsdgeosWellark: how are you building? is it a clean build?10:42
Wellarktsdgeos: yep. with the build.sh10:42
Wellarkfirst build.sh -s10:43
Wellarkand then build.sh10:43
Wellarktsdgeos: debian rules has an interesting note10:43
Wellark# libMockLightDM-qml.so links against liblightdm-qt5-2.so which doesn't exist10:43
Wellarkoverride_dh_shlibdeps: dh_shlibdeps -XlibMockLightDM-qml.so10:43
tsdgeosi never had this issue10:43
WellarkSaviq: --^10:44
Wellarksomeone...10:44
Wellark:(10:44
tsdgeosi just cleaned and built from scratch just fine10:48
Wellarktsdgeos: could you run dpkg-query -S lightdm-qt5-210:49
tsdgeoshttp://paste.ubuntu.com/9784026/10:49
tsdgeoshmmm10:51
tsdgeosgive me a sec10:51
CimiWellark, hey :) we would like to update https://bugs.launchpad.net/ubuntu/+source/ubuntu-system-settings/+bug/1363400 with what we needed10:51
ubot5Launchpad bug 1363400 in ubuntu-system-settings (Ubuntu RTM) "[wizard] allows to "Continue" without connecting to network" [High,Triaged]10:51
Cimiideally something that tells us when a connection is established with an AP10:52
WellarkCimi: you need to write a plugin that talks directly with NM to get that information10:54
tsdgeosWellark: the dpkg -S returns empty10:54
tsdgeosWellark: ccache playing tricks on you?10:54
tsdgeosWellark: do you have the result of http://paste.ubuntu.com/9784026/ ?10:55
Wellarktsdgeos: nope10:57
Wellarkseems they are not being built10:57
tsdgeostry to find out why10:57
tsdgeosthey should10:57
Wellarktsdgeos: hmm.. the targets don't seem to build bydefault on my system11:05
Wellarkmanual invokation works11:06
Wellark$ ninja MockLightDM-single11:06
Wellark[4/4] Linking CXX shared library tests/mocks/LightDM/single/liblightdm-qt5-2.so11:06
tsdgeosah you're nijaing?11:06
tsdgeosdon't do that :D11:06
tsdgeosdoes proper make work?11:06
Wellarktsdgeos: build.sh picked ninja on it's own11:07
tsdgeosthat may be11:08
tsdgeosi don't know11:08
tsdgeosdon't use ninja11:08
tsdgeosit works for me11:08
tsdgeosi can see how ninja might build stuff diferently11:08
Wellarktsdgeos: well, it _should not_11:08
Wellarkbut sometimes there are differencies11:09
tsdgeosit should not11:09
tsdgeosbut i can see how it can happen11:09
Wellarkwhich usually are bugs in cmake :)11:09
Wellarkok.. I will modify build.sh11:09
tsdgeossee if that helps11:09
tsdgeosif it does open a bug and say ninja build fails11:10
Wellarktsdgeos: you should try ninja11:12
Wellarkit makes compilation a lot faster11:12
tsdgeosbut doesn't work ^_^11:16
Wellarktsdgeos: ok. forcing a build without ninja worked11:16
tsdgeosWellark: can you report a bug?11:16
tsdgeosor wait a bit11:16
Wellarktsdgeos: yep. will file it11:16
tsdgeosi'll see if i can fix it11:16
tsdgeosin between compiles11:16
Wellarktsdgeos: one thing that might cause ninja to "misbehave"11:17
Wellarkis the fact11:17
Wellarkthat for example in tests/mocks/LightDM/single/CMakeLists.txt11:17
Wellarka library target is created11:17
Wellarkadd_library(MockLightDM-single SHARED ${LibLightDM_SOURCES})11:17
Wellarkbut on the very same file11:17
Wellarkthere is qt5_use_modules(MockLightDM-single Gui)11:17
Wellarkoh, wait no11:18
Wellarksorry. disregard that11:18
Wellarktsdgeos: actually it might be one level up11:20
Wellarktests/mocks/LightDM/CMakeLists.txt11:20
mzanettidednick: ping11:20
Wellarktsdgeos: add_library(MockLightDM-qml MODULE ${QMLPLUGIN_SRC} )11:20
dednickmzanetti: plop11:20
mzanettidednick: hey. I'm testing silo 6 which contains this branch: https://code.launchpad.net/~nick-dedekind/unity8/rtm-1385331/+merge/24640311:21
mzanettidednick: seems the indicators are invisible11:21
Wellarktsdgeos: target_link_libraries(MockLightDM-qml -L${CMAKE_CURRENT_BINARY_DIR}/full -llightdm-qt5-211:21
Wellarkthere is no dependency specified to the single, single-pin, single-passphase targets11:21
Wellarkso ninja might just go ahead and have a different build order as it does not know that -llightdm-qt5-2 actually requires some additional targets to run11:23
mzanettidednick: http://i.imgur.com/8ASuvJE.png?111:23
dednickmzanetti: hm. opening doesnt show anything?11:24
dednickmzanetti: is there anything in the log?11:24
mzanettidednick: I get this repeatedly in the logs: file:///usr/share/unity8/Panel/Indicators/IndicatorDelegate.qml:23:46: Unable to assign [undefined] to bool11:25
dandradermzanetti, just read www.rpadovani.com/ubuntu-calculator-app-reboot-store11:26
dandradermzanetti, so, will the calculator app now gracefully handle rotations (as in the shellRotation branch)?11:26
mzanettidednick: this is when I pull them down11:26
mzanettihttp://i.imgur.com/5828NBx.png11:26
mzanettiso seems ok, but the animation is somehow broken as they're invisible in the starting position11:27
mzanettidandrader: heh11:27
mzanettidandrader: yes11:27
mzanettidandrader: I've reworked the calculator keyboard to deal with that11:27
dandradermzanetti, awesome11:27
tsdgeosgreyback: ping11:28
greybacktsdgeos: pong11:28
greybackjust pushing fix for qtmir conflict11:28
tsdgeosgreyback: i'm trying the https://code.launchpad.net/~gerboland/unity-api/surfaceSizerCallback/+merge/231698 and friends11:28
tsdgeosi don't knwo what i should look at11:29
mzanettidandrader: http://i.imgur.com/tQVNgLF.png11:29
mzanetti(on a non-rotating shell still)11:30
mzanettihence the panel at top11:30
greybacktsdgeos: launch webbrowser - and watch the URL bar closely. Without that patch, you'll see the url bar reposition a few frames after it appears. With this patch, that reposition stops11:30
greybacktsdgeos: the main issue I'm fixing is this: when an app creates a surface, it asks mir/qtmir is the size for hte surface it wants is ok. Currently qtmir always overrides the size to be a default of "the size of the display"11:31
tsdgeossure i understand the code11:31
tsdgeosi just didn't know what to look at11:31
greybacktsdgeos: once the app draws its first frame, the surface is added the QML scene, which causes the surface to resize to fit - i.e. under the panel11:31
greybackI'm removing that initial surface resize11:32
dednickmzanetti: hm. ok, that's weird. dont suppose that branch contains shell rotation stuff as well?11:32
dandradermzanetti, nice11:32
greybacktsdgeos: does that make sense?11:33
mzanettidednick: no, it doesn't11:33
dednickweird. i dont think i touched anything visual11:33
tsdgeosgreyback: yes11:33
mzanettidednick: this is the silo: http://people.canonical.com/~platform/citrain_dashboard/#?distro=ubuntu-rtm&q=landing-00611:33
tsdgeosgreyback: i've been trying the gallery though11:33
tsdgeosand it's ugly11:33
dednickmzanetti: thanks. i need to flash.11:33
tsdgeosi can see it grow11:33
tsdgeosnot sure if it was already there without the patch11:33
greybacktsdgeos: I think that was there before, there's something really slowing down that app->fullscreen animation11:34
greybackbut don't trust me on that11:34
greybacktsdgeos: one way I really notice that is, have gallery open, lock screen, unlock and enter PIN. The animation to restore the view to the gallery is terribly slow11:36
tsdgeosmay be11:36
tsdgeoslet me revert the change and see if i can still see it11:36
tsdgeosWellark: can you onfirm that http://paste.ubuntu.com/9784367/ fixes the ninja build for you?11:42
tsdgeosmzanetti: Wellark: https://code.launchpad.net/~aacid/unity8/buildWithNinja/+merge/24688312:07
=== MacSlow is now known as MacSlow|lunch
Wellarktsdgeos: it does!12:16
Wellarktsdgeos: so yes. there was a target dependency missing12:17
Wellarkplain cmake just happens to build the targets in an order that the .so files just happened to be available12:17
Wellarkbut ninja reordered the build because it didn't know about the dependency12:18
dandradermzanetti, is this new calculator a different application from the pre-loaded one? Or is the pre-loaded one just outdated?12:21
mzanettidandrader: it's one of those core app "reboots"12:21
mzanettidandrader: meaning they start from scratch, trying to avoid the mistakes from the first round12:22
mzanettisome code might be still the same, other isn't12:22
mzanettiin this case they only kept the calculator engine the same and redid the ui completely12:22
mzanettiat some point it will replace the original calc app12:22
dandradermzanetti, so will the pre-loaded calculator be replaced with this "rebooted" one?12:22
dandradermzanetti, ok12:23
mzanettiyes, when the rebooted one is considered better than the old12:23
tsdgeossomething from https://code.launchpad.net/~unity-team/unity8/trunk/+activereviews i should take?12:27
tsdgeosi'm a bit work-less at the moment12:27
dandradermzanetti, did you happen get the "can't do gestures" bug since last Friday?12:30
dandrader(ie, since you enabled the touch and gestures logging)12:30
mzanettidandrader: nope, haven't12:30
dandradermzanetti, it's both good and bad news :)12:31
mzanettidandrader: yeah, haven't really had the phone in my pocket since12:31
dandraderah12:31
mzanettigoing for a run soon, that should increase chances12:31
=== alan_g is now known as alan_g|lunch
dandradergreyback, so rotation lock is broken on latest devel-proposed. do you have any idea since when it has been like that (or even better, the cause of it)?12:39
greybackdandrader: no idea, sorry12:40
dandraderhmmm... the indicator switch to lock the orientation has no effect but locking using the system-settings app works12:41
Cimitsdgeos, for the scope bug, design ideally wants animation when opening the scope, plus getting back will go to store12:49
tsdgeosright12:51
dandraderdednick, any ideas on what would be causing this? https://bugs.launchpad.net/ubuntu/+source/unity8/+bug/141243012:51
ubot5Launchpad bug 1412430 in unity8 (Ubuntu) "Locking rotation via the indicators panel switch does not work" [Undecided,New]12:51
tsdgeosso you basically need to create an infinite number of temp GenericScopeView and stack them on top of eachother12:51
tsdgeoslunchtime12:51
Cimitsdgeos, yeah...12:51
=== Malsasa_ is now known as Malsasa
=== alecu_ is now known as alecu
=== dandrader is now known as dandrader|afk
=== alan_g|lunch is now known as alan_g
WellarkCimi: wanna hangout?13:28
greybacktsdgeos: slightly related fix: https://code.launchpad.net/~gerboland/unity8/lockscreen-height-constant/+merge/24689213:38
=== Malsasa_ is now known as Malsasa
* tsdgeos clicks13:49
=== dandrader|afk is now known as dandrader
tsdgeosgreyback: hmmmm13:54
tsdgeosis that only needed with your patches?13:54
=== MacSlow|lunch is now known as MacSlow
tsdgeosCimi: what's your plan for implementing that?13:58
tsdgeosgreyback: ah no needed in trunk too14:04
greybacktsdgeos: that fixes the bug in trunk - just while I was profiling I saw the problem and shipped up the patch14:05
greybackno fix for the issue you spotted yet14:05
tsdgeosinterestingly the bug doesn't seem to be in rtm14:05
greybackno? huh14:06
tsdgeosor at least i don't get the gallery to be ultra slow on unlock screen14:06
tsdgeosis this what it's fixing?14:07
tsdgeosmaybe i need more phtos on the rtm phone14:07
greybackyes, that is what it fixes. I get it on my kryllin with devel-proposed. Having Gallery open, I lock & unlock the phone. After PIN+Enter, the animation jutters badly14:08
tsdgeosyep14:09
tsdgeosdo you have many photos?14:09
tsdgeosor just empty?14:09
greybackcouple of photos, but that should not matter14:09
greybackit's unity8's animation that is slow14:09
greybackthe fade from Lockscreen to app surface14:10
greybackplus the panel sliding up14:10
tsdgeosi know14:11
tsdgeosbut my krillin rtm is silky smooth14:11
greybackI'm running devel-proposed14:12
greybackso maybe that's a factor14:12
tsdgeosyeah14:12
tsdgeosdevel proposed is just uglier14:12
tsdgeosit's broken :D14:12
tsdgeoswith your patch14:12
tsdgeosit doesn't look good14:13
Cimitsdgeos, I need to think about how14:13
tsdgeoswhen you unlock14:13
tsdgeosthe locker fades in place and the indicator bar scrolls up14:13
tsdgeosleaving a part of the screen that's already totally transparent while the other part is fading14:13
tsdgeoson rtm it's not like that14:13
tsdgeosit's better14:14
tsdgeosgreyback: do you have an rtm phone at hand?14:14
greybacktsdgeos: would need to flash14:14
tsdgeosif you're not losing much stuff i'd suggest you to try14:15
tsdgeosso you see the diference14:15
tsdgeosi can't be sure how to describe it14:16
greybacktsdgeos: yeah I see what you mean14:16
greybackwell spotted14:16
greybackmeans I have to refactor things a bit more :(14:16
tsdgeosinteresting thing is that this didn't happen14:17
tsdgeosso how did it happen?14:17
greybacktsdgeos: rev 143814:18
greybackthe sourceSize lines were added14:18
tsdgeosi see14:19
dednickdandrader: it's probably not hooked up correctly in the indicator14:27
dednickdandrader: indicator-rotation or whatever14:27
dandraderdednick, did it ever work?14:28
dednickdandrader: don't think so14:28
dednickdandrader: i've never seen it working anyway14:29
dandraderdednick, is it a unity8 bug? I have no idea what mechanism is used to populate those indicators14:29
dednickdandrader: i think it's a backend bug. indicator-rotation or something14:29
dednickno idea what it's called.14:30
dednickdandrader: indicator-display i think14:31
dednickdandrader: https://bugs.launchpad.net/ubuntu/+source/indicator-display/+bug/141091514:32
ubot5Launchpad bug 1410915 in indicator-display (Ubuntu) "com.ubuntu.touch.system rotation-lock can't be changed by the checkbox in indicators" [Undecided,Confirmed]14:32
dandraderdednick, ah, so I made a duplicate. thanks for finding it14:33
seb128dednick, dandrader, rotation lock seems to work fine on current krillin rtm for me14:51
dandraderseb128, might be, I'm using devel-proposed14:53
=== om26er_ is now known as om26er
=== anpok_ is now known as anpok
tsdgeosMacSlow: do you need help with your branch?15:15
MacSlowtsdgeos, nope... I've solved the issue and an on othe final stretch... thanks15:15
MacSlows/an/am15:15
tsdgeosawesome15:15
Cimitsdgeos, so shall we use a pagestack for that stack of genericscopeview?15:17
tsdgeoshmm15:19
tsdgeosi'd say no15:20
tsdgeosprobably you won't get the right to left animation like we want and stuff15:20
tsdgeosjust instantiate new elements as we go?15:20
tsdgeosactually even the first one should be instantiated when needed probably15:21
tsdgeossave some memory when we don't have a temp scope around15:21
Cimitsdgeos, stackview?15:22
tsdgeoswhat's the benefit of stackview?15:23
Cimitsdgeos, animations and stuff?15:23
tsdgeosare they the ones we want?15:23
Cimitsdgeos, how do I add infinite elements?15:23
tsdgeosyou create them15:23
Cimimmm how?15:23
Cimifrom qml?15:23
Cimilike we do for crad?15:23
Cimicard15:23
Cimi?15:23
tsdgeosCimi: http://doc.qt.io/qt-5/qtqml-javascript-dynamicobjectcreation.html15:25
tsdgeoscomponent = Qt.createComponent("Sprite.qml");15:25
tsdgeos    sprite = component.createObject(appWindow, {"x": 100, "y": 100});15:25
Cimitsdgeos, ok as I imagined...15:26
Cimitsdgeos, thanks15:26
Cimitsdgeos, I might also want to keep a list of the scopes opened, right?15:32
tsdgeosthere goes the magic15:32
tsdgeoscan you get an openScope for a temporary scope?15:33
Cimitsdgeos, I'm wondering if for now we should just land a fix for this, then add this dynamic creation15:33
tsdgeoshow are you going to land a fix for this?15:33
=== dandrader is now known as dandrader|lunch
=== mardy_ is now known as mardy
om26erSaviq, Hi!16:22
om26erSaviq, while the logs for this crash look useless, can you see if anything else hints it to be an old bug ? https://errors.ubuntu.com/oops/e9a80d32-9fee-11e4-a3ec-fa163e37368316:23
=== dandrader|lunch is now known as dandrader
greybackom26er: Saviq is on holidays this week16:52
om26ergreyback, oh, can you have a peek at that then ?16:53
greybackom26er: it appears to be a null pointer dereference somewhere from libdbus-cpp, but there's no other information in that report I can use16:55
dandradermzanetti, did you figure out that issue about building packages for RTM on a PPA?17:39
mzanettidandrader: I know why, but no solution17:39
dandrader:(17:39
mzanettidandrader: so this is a ubuntu ppa, it can't build rtm packages17:39
dednickmzanetti: just pushed some changes; seems to fix the issue17:39
mzanettidandrader: in the end I just compiled the plugin on my phone17:39
mzanettidednick: cool, thanks17:39
=== 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!