tsdgeosltinkl: do you have a sec?08:47
ltinkltsdgeos, ye sure09:18
tsdgeoswill come to you in a bit09:18
tsdgeossorry :D09:18
tsdgeosltinkl: you've got mail09:24
ltinkltsdgeos, got it09:24
tsdgeosltinkl: please uncompress09:24
tsdgeosrun qmake && make in the ManglingImport folder09:25
tsdgeosand then from the root run09:25
tsdgeos qmlscene -I . main.qml09:25
tsdgeosboth with the Image { } from MySingleton commented and not commented09:26
tsdgeosi think both show bugs :D09:26
tsdgeosbut i'd like you to have a look at all the code and tell me what you think before i convince you09:26
ltinkltsdgeos, convince of what? :)09:27
tsdgeosthat it's a bug in both cases (two different bugs)09:27
ltinklI get "QML Image: Cannot open ..." in both cases09:27
tsdgeosthat's not the bug09:28
tsdgeosread the code please09:28
tsdgeosthough you're doing it wrong09:28
tsdgeosif you comment the Image  {} from MySingleton you should not be getting that09:29
tsdgeos$ qmlscene -I . main.qml09:29
tsdgeosIgnoring the url you set to  QUrl( "file:///home/tsdgeos_work/extendedTypeBug/RegularImage" )09:29
tsdgeosIgnoring the url you set to  QUrl( "file:///home/tsdgeos_work/extendedTypeBug/thisShouldBeIgnoredByImport" )09:29
tsdgeosqml: MoMoMoMo MoMoMoMo09:29
tsdgeosltinkl: do you get that?09:30
ltinkltsdgeos, yup I'm getting qml: MoMoMoMo MoMoMoMo09:31
tsdgeosand if you uncomment the image09:31
tsdgeosyou get the "cannot open" variant09:32
tsdgeosdo you agree that both behaviours are wrong?09:33
ltinklnot sure, it's either getting the image from the C++ plugin or from the Image QML class right?09:34
tsdgeosthe problem is09:37
tsdgeosthat if the singleton loads an image09:37
tsdgeosthe qmlRegisterExtendedType that gets loaded later is ignored09:37
tsdgeosthat's problem A09:37
tsdgeosproblem B is09:37
tsdgeosif the singleton is not loading the image09:37
tsdgeosthe loading of ManglingImport in MyItem.qmll viralizes to the Image in main.qml that knows nothing about such import09:38
ltinklproblem A - perhaps it's designed that way, so that QML imports get precedence over C++ imports09:39
ltinklproblem B - ye, I get that, this is eird09:39
ltinklbut understandable imho, you're extending the QQuickimageBase, so basically Image is no longer the one shipped by Qt but your provided by the plugin09:40
ltinklno matter where you import it09:40
ltinklso once you import it, there's no going back to the original Image imo09:41
ltinklhow would QML know then which one to use?09:42
tsdgeosbecause it knows09:42
tsdgeosit has different metaobjects09:42
tsdgeosfor different imports09:42
tsdgeosi.e. the image of 2.0 is different from the image of 2.409:43
tsdgeosit has a different metaobject *09:43
tsdgeosthey just need to also extend that for extended stuff09:43
tsdgeosi agree it's hard09:43
tsdgeosbut not undoable09:43
tsdgeosand A it can't be designed that way09:43
tsdgeosit's a oversight i'd say that they forgot about it09:44
tsdgeosanyway will file two bugs and see what happens09:44
tsdgeosBTW A+QMLcache is what is making us crash in unity8 some cases09:44
ltinklaha I see09:45
ltinklye definitely ask those 2 basic questions (bug reports) and see what they have to say09:45
tsdgeospstolowski: greyback landed something so you need to rebuild the silo11:26
greybackand hopefully there's no conflicts11:28
tsdgeosgreyback: can you do https://code.launchpad.net/~aacid/unity8/workaround_qt47709/+merge/267647 ?11:29
greybacktsdgeos: lgtm11:32
tsdgeosgreyback: i think the last landing makes a test crash11:43
tsdgeoslet me confirm11:43
greybacktsdgeos: which test?11:43
tsdgeosnot sure if it's new though11:44
tsdgeoshow nice would be to have cI back ...11:44
greybackhmm, that's an old one11:44
greybacksomehow has resurfaced11:44
greybackI recall there were times "app" was null11:44
greybackwhich didn't make sense11:45
tsdgeosgreyback: do i open a bug? assign it to who?11:47
greybacktsdgeos: open bug anyway, will give it to dandrader11:47
tsdgeosisn't he off for like the next two weeks?11:48
greybackI *think* that's from tomorrow11:48
pstolowskitsdgeos, rebuilding11:49
pstolowskitsdgeos, have you found anything about the error i had?11:50
tsdgeospstolowski: hadn't had time to test, promise first thing after lunch11:51
tsdgeoslunch now!11:51
tsdgeosgreyback: https://bugs.launchpad.net/ubuntu/+source/unity8/+bug/148367511:51
ubot5Ubuntu bug 1483675 in unity8 (Ubuntu) "testLauncherModel crashes" [Undecided,New]11:51
=== alan_g is now known as alan_g|lunch
=== om26er_ is now known as om26er
tsdgeosdandrader: vivid or wily?12:48
dandradertsdgeos, vivid12:49
tsdgeosdandrader: weird, dist-upgraded?12:49
dandradertsdgeos, yes12:49
* tsdgeos scratches head12:50
tsdgeospstolowski: you mentioned some issues with the silos not installing properly12:54
tsdgeospstolowski: what do i do to fix it? or should work?12:54
pstolowskitsdgeos, yeah, you need to make sure you get versions from the silos12:54
pstolowskitsdgeos, apt-cache policy <foo> for every package12:55
pstolowskitsdgeos, then apt-get install foo=<version> using versions from ppa if needed12:55
pstolowskitsdgeos, but i'm still battling with silo 4. rebuilt failed12:56
pstolowskitsdgeos, looks like something landed also in unity api, so rebuilding12:56
=== alan_g|lunch is now known as alan_g
tsdgeospstolowski: i guesswe need that rebuild13:08
tsdgeosi'm getting13:08
tsdgeos[1439298272.585212] <ERROR> mircommon: Caught exception at Mir/EGL driver boundary (in queueBuffer): /build/mir-VroDxW/mir-0.14.0+15.04.20150722/src/client/rpc/stream_socket_transport.cpp(168): Throw in function virtual void mir::client::rpc::StreamSocketTransport::send_m13:08
tsdgeosessage(const std::vector<unsigned char>&, const std::vector<mir::Fd>&)^M13:08
tsdgeosand unity8 restarts/crashes13:08
tsdgeoslet me try to see on the desktop13:10
tsdgeospstolowski: yeah can reproduce this Playlist thing on the desktop, investigating13:26
pstolowskitsdgeos, ok13:26
tsdgeospstolowski: pushed a change, sorry13:30
pstolowskitsdgeos, only unity8?13:31
tsdgeoshow do i test it, the music scope was empty13:32
tsdgeosam i supposed to get something?13:32
pstolowskitsdgeos, do you have local music?13:32
tsdgeosprobably not13:32
tsdgeosi mean i do have it13:32
tsdgeosbut not where media-scanner wants it i guess13:32
tsdgeosi'll copy some stuff to ~/Music13:32
tsdgeosis it?13:32
pstolowskitsdgeos, yes13:33
pstolowskitsdgeos, then go to Tracks department in My Music13:33
tsdgeoshmm wait13:35
tsdgeosi do actually have stuff in ~/Music13:35
tsdgeosCaught an error from create_query(): unity::scopes::MiddlewareException: unity::ResourceException: RegistryObject::ScopeProcess::exec(): exec aborted. Scope: "mediascanner-music" took longer than 4000 ms to start.13:35
tsdgeospstolowski: ↑ ?13:36
pstolowskitsdgeos, do you have all the versions from ppa? including mediascanner2.0 and unity-scope-mediascanner2?13:36
pstolowskitsdgeos, hmm. need to wait for rebuild. what phone image do you have?13:38
tsdgeospstolowski: this is on the pc13:38
tsdgeospstolowski: on the phone can't test because the mir crash/restart13:38
pstolowskitsdgeos, ah... i can't test on the PC ;)13:38
pstolowskitsdgeos, waiting for the build, will check on the phone13:39
tsdgeosi'll uninstall the stuff from the PC13:39
tsdgeosand do something else while waiting for the rebuild :)13:39
pstolowskitsdgeos, keep fingers crossed for the build if your hands are free ;)13:41
tsdgeoshe he13:43
pstolowskitsdgeos, can you check if https://code.launchpad.net/~aacid/unity8/dash_activation_no_special_casing/+merge/264024 merges ok with trunk?14:07
tsdgeosit should14:08
tsdgeoslet me check14:08
tsdgeospstolowski: yeah merges fine here14:08
pstolowskitsdgeos, ok thanks14:08
pstolowskitsdgeos, silo 4 built14:18
tsdgeospstolowski: cool14:19
=== dandrader_ is now known as dandrader|afk
tsdgeospstolowski: seems to "do things" :D14:44
tsdgeospstolowski: if i play the first track is it supposed to play the next?14:44
pstolowskitsdgeos, only if from same album14:44
* tsdgeos waits 2:2014:44
pstolowskitsdgeos, does it integrate with indicator?14:45
tsdgeospstolowski: not yet, jhodapp said he wanted to wait for something that used it to enable it14:47
tsdgeosor that's what i understood14:47
tsdgeosdoesn't seem the play next song worked14:50
tsdgeoslet me add some console.log14:50
jhodapptsdgeos, correct14:52
tsdgeosjhodapp: so silo 415:00
jhodapptsdgeos, silo 4?15:02
tsdgeosjhodapp: if you join it with silo 38 has a somewhat working audio card support on the dash15:03
jhodapptsdgeos, oh very nice15:03
jhodapptsdgeos, thanks15:03
tsdgeosfor some reason playing the first time fails15:03
tsdgeosyou have to play twice15:03
jhodapptsdgeos, so I assume you can't land silo 4 until the indicator controls work?15:03
tsdgeosand the playlists don't work either15:03
tsdgeosbut not sure whose fault it is15:04
tsdgeosdebugging a bit15:04
tsdgeosjhodapp: correct15:04
pstolowskijhodapp, also we need to wait for silo 27 to land and then rebuild15:04
jhodapptsdgeos, let me give you a Test.qml file that you can use to compare things against15:04
pstolowskitsdgeos, last time i checked with scopes-client it was exposing playlists as expected. perhaps we have a mismatch in names or something trivial like that somewhere15:05
pstolowskitsdgeos, i mean between dash and scope15:05
jhodapptsdgeos, alright, shared with you on google drive15:06
tsdgeospstolowski: sure, also debugging that15:06
jhodapptsdgeos, just use qmlscene Test.qml --desktop_file_hint=/usr/share/applications/mediaplayer-app.desktop15:07
jhodapptsdgeos, and make sure to change the sources listed in there to ones you have on your test device15:07
tsdgeosk tx15:07
jhodapp*music sources15:07
tsdgeospstolowski: yeah looks good, seems i'm losing it somehow at some point15:10
tsdgeosbooo me15:16
tsdgeosyou can't obviously store an array in a url :D15:16
tsdgeosif the sdk wasn't overflowing us with warnings15:16
tsdgeosmaybe even the qml interpreter told me15:16
tsdgeosat some stage15:16
tsdgeosalso the song i'm trying is longer that it says it is?15:20
tsdgeospstolowski: 2:47 vs 2:45 ?15:20
tsdgeoswooo, works :)15:20
pstolowskitsdgeos, afaik the duration is read from mp3 tags or some such, may not be precise?15:20
tsdgeosmay be15:21
pstolowskialecu, ^^^15:21
tsdgeosthe file doesn't seem to have that in the tags15:22
alecupstolowski: is that inline playback?15:22
pstolowskitsdgeos, hmm in any case this is what mediascanner backend tells me. probably it gets it from gstreamer15:22
pstolowskialecu, yes15:23
alecupstolowski: what's the silo number?15:23
alecupstolowski: wonderful15:23
pstolowskialecu, 4 + 38. but indicator intergation is not enabled yet15:23
tsdgeosit's pretty nifty going back and forth15:26
tsdgeosand still works :D15:26
tsdgeoseven on the album preview15:27
tsdgeoseven if you played it from the tracks list15:27
tsdgeosjhodapp: the debug is weird15:35
tsdgeosthe first time i add things to the playlist i have15:35
tsdgeoswhile the second i have15:36
tsdgeosbut i'm passing the urls the same way15:36
tsdgeoslet me console.log them15:36
tsdgeosjhodapp: can it be that addSources to a playlist are ignored if it's not part of a player?15:40
tsdgeosthat may explain why the second time works15:40
* tsdgeos tries15:40
jhodapptsdgeos, yes there's a possible bug in the QDeclarativePlaylist impl15:41
jhodapptsdgeos, let me get back to you in a little bit, on a hangout15:41
=== dandrader|afk is now known as dandrader
tsdgeosyeah i sthat15:46
tsdgeosi guess i can workaround it in my code easily15:46
tsdgeosi mean i just did15:46
tsdgeosbut would make sense fixing it anyhow15:46
tsdgeospstolowski: can you rebuild the silo so that it includes the two last fixes i made?15:49
tsdgeospstolowski: unity815:49
pstolowskitsdgeos, ok15:49
jhodapptsdgeos, yes indeed, there will be several bugs to fix15:53
jhodapptsdgeos, can you file that bug against qtmultimedia-opensource-src package?15:53
pstolowskitsdgeos, rebuilding15:54
tsdgeosjhodapp: https://bugs.launchpad.net/ubuntu/+source/qtmultimedia-opensource-src/+bug/148380615:58
ubot5Ubuntu bug 1483806 in qtmultimedia-opensource-src (Ubuntu) "addSource to a Playlist are ignored if it's not part of a player" [Undecided,New]15:58
tsdgeospstolowski: cool15:58
jhodapptsdgeos, thanks!15:58
jhodapptsdgeos, oh right, yes it clears the playlist when it attaches to a player15:59
jhodapptsdgeos, I didn't realize that's what you were trying to do...this is on purpose15:59
tsdgeosit's not a big thing i just did it the other way around15:59
jhodappwell actually no this is a valid bug, nm16:00
tsdgeosbut if that's the behaviour, make sure it's documented :D16:00
* tsdgeos eods16:00
jhodappit will clear any existing playlist attached to the player16:00
pstolowskidavmor2, does silo 27 work for you now?16:07
davmor2pstolowski: I'm on another silo currently will return to that after16:15
pstolowskidavmor2, ok16:16
pstolowskianyway, eod. cu16:16
=== alan_g is now known as alan_g|EOD
=== dandrader is now known as dandrader|afk
=== jhodapp is now known as jhodapp|errand
=== jhodapp|errand is now known as jhodapp
=== dandrader|afk is now known as dandrader
dandraderjosharenson, ping19:02
josharensondandrader: pong19:18
dandraderWhen I run "make tryWideView" all users have a "Tap do unlock" button. Some of them should display a text entry for instance, for typing the password19:20
* josharenson runs that test19:21
dandraderjosharenson, do you know what's wrong there (with the mocks most likely)19:21
josharensonsorry didn't have a good build, compiling now19:24
josharensondandrader: ah I see. Nothing happens when you "tap to unlock" but there should be a text box, yes?19:28
josharensondoes it hang for you?19:29
dandraderjosharenson, yeah, nothing happens.19:31
josharensonlooking into it, either a problem with the mock, or the wrong lib is being loaded in the test I would imagine19:31
dandraderjosharenson, The "Has Password" user should naturally display a message box for typing the password, for instance19:31
dandraderjosharenson, I wanted to debug that login message box and because of this bug in the test there's no easy way to do so :/19:33
josharensondandrader: ok, give me a few, I'll ping you when I find something19:33
dandraderjosharenson, hey20:46
josharensondandrader: hi20:46
dandraderjosharenson, does PhysicalKeysMapper have some logic to ensure you don't change the volume while your'20:46
dandraderjosharenson,  you're pressing the volume keys to get a screenshot?20:47
josharensondandrader: it should, saviq/I fixed that a while ago I thought20:47
josharensondandrader: I honestly don't remember what the exact solution ended up being, I'd have to look at it20:47
dandraderjosharenson, we just don't have a test guarding this feature20:53
josharensondandrader: I could swear I wrote some... let me look20:54
dandraderjosharenson, I bet https://code.launchpad.net/~lukas-kde/unity8/globalshortcuts/+merge/267188 will inadvertently break it20:54
josharensondandrader: I see. Either that code needs logic to handle not showing volume notifications, or it needs to let volume key events through, which is a bad idea20:58
dandraderjosharenson, just commented there that it might break this behavior20:59
josharensondandrader: I'll try and do a review if I have some time later20:59
dandraderjosharenson, if you want to. I already did my second pass over there21:00
dandraderor third, actually21:00
=== dandrader is now known as dandrader|afk
dandrader|afkjosharenson, but it would be great if you could confirm me whether we have a test for this screenshot vs. volume case. I didn't find any in tst_PhysicalKeysMapper.qml21:02
josharensondandrader|afk: ok ill look for it for sure. Also, before you go away, I can't find anything wrong w/ the greeter wide view21:03
josharensonoh wait21:03
josharensonhang on21:03
josharensondandrader|afk: ah, there was a negative test when the screenshot sequence was Power+VolDown but it looks like it never changed when we reverted to VolUp+VolDown21:12
dandrader|afkok. see you next week21:20
=== g4mby is now known as PaulW2U

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!