/srv/irclogs.ubuntu.com/2014/09/02/#ubuntu-unity.txt

=== _salem is now known as salem_
=== ssweeny` is now known as ssweeny
=== salem_ is now known as _salem
=== _salem is now known as salem_
=== salem_ is now known as _salem
=== larsu_ is now known as larsu
Saviqmounin'07:23
tsdgeosSaviq: landing stuff today?08:00
tsdgeosCimi: can you re-review this oneliner? https://code.launchpad.net/~aacid/unity8/dashOverviewFromTempScopePreview/+merge/23237808:02
Saviqtsdgeos, yeah, later today08:09
tsdgeos:)08:09
SaviqI mean landing08:09
Saviqwill review now ;)08:09
tsdgeoshow does one run the shell in desktop nowadays?08:49
tsdgeosnor run.sh nor the binary start here08:50
tsdgeosok -f "worked"08:52
mzanettitsdgeos: for me it hangs with a wait_condition on the Audio {} element in the notifications08:54
mzanettihad to comment that away08:54
Saviqmzanetti, yeah, it's some Gst fookup09:02
Saviqseems to be a problem for unity8 desktop session, too09:02
anpok_https://bugs.launchpad.net/ubuntu/+source/camera-app/+bug/136425809:02
ubot5Ubuntu bug 1364258 in ubuntu-ui-toolkit (Ubuntu) "[desktop] camera-app and unity8 freeze on startup while initializing gstreamer-0.10" [Undecided,New]09:02
CimiI had same issue in the wizard09:04
Cimisince uses notifications09:04
anpok_if either of the two libraries that pull in somehow conflicting gst versions is removed the problem goes away but fails later09:04
tsdgeosSaviq: you wanted me to kill the dash comunicator09:12
tsdgeosi actually have to extend it ^_^09:13
tsdgeosneed to know from the shell if the overview is shown or not09:13
Saviqtsdgeos, nooooooo09:13
Saviqtsdgeos, why?09:14
tsdgeosso that i know if the left swipe has to chnge the scope or not09:14
tsdgeosthe code is right now09:14
Saviqtsdgeos, why does *shell* have to know that09:14
tsdgeosshell code: if (dashSwipe && ApplicationManager.focusedApplicationId !== "unity8-dash") {09:14
tsdgeosi can either move the ApplicationManger back to the dash09:15
tsdgeosor the overview status to the shell09:15
Saviqtsdgeos, doesn't dash know whether it's focused or not itself?09:15
Saviqtsdgeos, shell should just say "hey, dash, go to apps scope"09:15
Saviqtsdgeos, then dash should make the decision what do to (animate if focused, no animate when not)09:16
tsdgeosSaviq: i don't know, how would it know if it's focused or not?09:16
Saviqtsdgeos, how does any QQuickView know if it's focused or not?09:16
Saviqit's a standard window focus thing09:16
Saviqit should *actually* be visible or not, but later09:17
tsdgeosdidn't get that last sentence09:18
Saviqtsdgeos, later we'll actually need "am I visible at all" and not "am I focused" (on desktop)09:18
Saviqtsdgeos, and only animate if visible09:18
Saviqtsdgeos, but for now, I really don't think shell should ever know state of dash, that's why I wanted to rid us of dash communicator, so that we don't get tempted :P09:19
Saviqit's just an app, tell it to go to scope foo (ideally via url) and be done with it09:19
Saviqalthough we might not be able to go through url atm 'cause url-dispatcher will request focus of dash, not sure how that will play out09:20
Saviqor at least not through url dispatcher, we could go through the uri handler just fine09:20
tsdgeosok...09:20
zbenjaminSaviq: hey, still working on scope support in QtCreator ;). After the new scope exited i want to automatically show the clickscope, but the url dispatching does not work if a scope is shown that is not in favourites09:41
zbenjaminSaviq: and the scope i currently develop on is never in the favourites09:42
zbenjaminSaviq: you can easily try by opeing a scope that is not in your favs and just invoke url-dispatcher scope://clickscope from the shell09:43
Saviqzbenjamin, sure it works09:49
zbenjaminSaviq: not for me :(09:49
Saviqzbenjamin, ah you mean it doesn't *close* the non-favourite scope09:50
zbenjaminSaviq: yeah the clickscope is not shown09:50
Saviqzbenjamin, yeah, that'd be a bug in unity8, can you please log one?09:50
zbenjaminSaviq: lets say i have Amazone scope open and want to go back to clickscope09:50
Saviqzbenjamin, yeah, I even know what's happening09:50
Saviqzbenjamin, mention non-favourite scope and overview, as expect that to not work well either...09:51
* Saviq hates how many weird cases we have there :|09:51
zbenjaminSaviq: yeah in a non fav scope you also cannot pull up the overview09:52
zbenjaminSaviq: no you can09:52
Saviqzbenjamin, you can, that's per design09:52
Saviqzbenjamin, not in a preview though09:52
Saviqzbenjamin, and then not in a non-fav opened from overview09:52
* tsdgeos is very confused09:53
zbenjaminSaviq: exactly , non fav openend from overview has no pull up09:53
Saviqzbenjamin, again, by design09:53
tsdgeosi've edited /usr/share/unity8/Shell.qml to be syntactically incorrect09:53
tsdgeosyet the shell still shows up09:54
tsdgeoswhy?¿09:54
Saviqtsdgeos, sure it's not running your local one?09:54
Saviqzbenjamin, because the overview is already open behind the scope opened09:54
tsdgeosSaviq: i'm rebooting the phone, how would it be running my local one?09:54
Saviqtsdgeos, just asking ;)09:54
Saviqtsdgeos, you didn't mention it's on the phone09:54
Saviqtsdgeos, so yeah, no idea ;P09:55
tsdgeosSaviq: so yes09:58
tsdgeosfor some reason beyond my understanding09:58
tsdgeosit was using file:///home/phablet/lvwph_non_interactive_header_animation/qml/Shell.qml09:58
tsdgeosnow that i've removed the file it doesn't start09:58
Saviqtsdgeos, whoaps, rebooted?!09:58
tsdgeosi've rebooted like 10 times09:58
Saviqwth09:58
tsdgeosmaking the syntax worse every time :D09:58
tsdgeosanyway that's fine, i've been being bad with the phone09:59
zbenjaminSaviq: https://bugs.launchpad.net/unity8/+bug/136430609:59
ubot5Ubuntu bug 1364306 in Unity 8 "scope:// url dispatching does not correctly work if a non favorite scope is shown" [Undecided,New]09:59
Saviqzbenjamin, thanks10:02
Saviqmight be indeed10:23
SaviqI think I was only getting bubbles at event time, not reminder10:24
Saviqand that was the datetime indicator10:24
Saviq</typing in wrong channel>10:24
dednickSaviq: ping10:32
Saviqdednick, hey10:32
Saviqdednick, https://code.launchpad.net/~nick-dedekind/unity8/prompts-in-prompts/+merge/232362/comments/56810310:32
dednickSaviq: howdy. I'm having some trouble with some unicode encoded strings in u8 (time formatter). string is: "%a %d %b\u2003%H:%M"10:33
Saviqdednick, that in javascript or c++?10:33
dednickSaviq: c++10:33
dednicksomething funky is happening in u8 passing it to g_date_time_format that doesn't happen in indicator-datetime.10:34
Saviqdednick, I assume that's priv->format ?10:35
dednickSaviq: yeah, but the problem isn't actually qstring related. just doing: g_date_time_format(g_date_time_new_local(2014, 9, 9, 9, 9, 9), "%a %d %b\u2003%H:%M") in u8 fails, while in indicator-datetime it works10:36
Saviqdednick, iinteresting, wonder if we need to say somewhere that's utf-8 encoded?10:36
dednickSaviq: ya. that's what i'm wondering. if we specificlly forbid it or something10:37
Saviqdednick, that's compiler error maybe?10:37
dednickSaviq: guessing so10:39
Saviqdednick, well, if you're not getting a compiler error, there's nothing to guess ;)10:39
dednickomg. someone is renevating on the roof directly under my feet!10:39
Saviqticklish?10:40
greybackskylight?10:40
Saviq:D10:40
dednickyeah. hammering away, and it's causing my NAS to beep complain about the shock!10:40
dednickSaviq: oh right, i thought you meant compiling problem :)10:41
dednickSaviq: hm. indicator-datetime is compiling that code with std=c9910:44
dednickand its a c file10:45
Saviqdednick, yeah, I'm thinking we need to convert it to pass to g10:48
dednickSaviq: convert to what?10:48
Saviqdednick, like *gchar or something? don't we get std::string for literals by default?10:49
dednickSaviq: hm. maybe.10:49
Saviqdednick, like https://developer.gnome.org/gtkmm-tutorial/3.9/sec-basics-ustring.html.en maybe10:50
dednickSaviq: or http://en.cppreference.com/w/cpp/language/string_literal10:50
Saviqdednick, right, yeah10:51
Saviqso u8""10:51
Saviqdednick, and then you can use the actual character, no need for escaping10:51
dednickSaviq: this is to do with that relative time formatter. not sure if it would work in indicator-datetime domain if not the same10:52
Saviqdednick, mhm10:52
Saviqdednick, so, any idea about prompts-in-prompts and hw keyboard input?10:52
dednickSaviq: ah. haven't looked yet10:53
Saviqdednick, that's blocking prompts from landing, fwiw, so whenever you can10:53
dednickSaviq: yeah, sorry. will do now10:53
Saviqdednick, no need to be sorry :)10:53
Saviqdednick, just letting you know, I know you wanted to get it done, and that people are waiting10:53
facundobatistaHolas11:04
=== MacSlow is now known as MacSlow|lunch
Cimidednick, is now possible to know when a connection is established?11:18
CimiWellark, ^11:18
dednickCimi: for unitymenumodel?11:18
Cimiyes11:18
dednickCimi: nameowner is valid string i think11:18
Cimidednick, camelCase?11:19
dednickCimi: most likely. i cant remember11:19
dednickCimi: yes11:20
Cimidednick, nah11:30
Cimidednick, nameOwner is 1.5 here11:33
Cimiand doesn't change11:33
dednickCimi: ? that means it's connected i think11:34
dednickCimi: if it isn't empty, it's connected to the bus11:34
Cimidednick, but if i connect to a broken ap11:34
Cimidednick, is still says 1.511:34
dednickCimi: ... i asked "for unitymenumodel?", not "for access point?"11:35
Cimidednick, ah crap :D11:35
Cimidednick, I use unitymenumodel for the access point11:35
dednickCimi: you can't. it's just active/inactive.11:35
Cimidednick, there is active property then?11:36
dednickCimi: https://bugs.launchpad.net/indicator-network/+bug/134937111:36
ubot5Ubuntu bug 1349371 in Network Menu "Current access point should not change until it is active" [Low,Triaged]11:36
dednickSaviq: the datetime is the LC_ALL=C flag btw. not sure why though11:40
Saviqdednick, aah well, maybe because it doesn't know it's UTF8 then?11:41
dednickSaviq: C.UTF-8 ?11:43
Saviqdednick, don't think such a thing exists...11:44
Saviqdednick, but if it does, sure :)11:44
Cimido you guys have wifi strength icons in system settings11:51
Cimi?11:51
dednicki need to find another place to work... my head is exploding from hammering!11:51
Cimidednick, wanna come here?11:51
dednickCimi: heh. can't need to tidy up today for parent's visit tomorrow :)11:52
Cimiseb128, hi! do you have wifi icons in system settings?11:53
seb128Cimi, no, icons are in the theme11:53
seb128Cimi, oh, and hey ;-)11:53
dednickSaviq: which tests failed without LC_ALL=C ?11:54
Cimiseb128, inside the wifi page, all access points are without signal strength to me11:55
seb128Cimi, check with Wellark or Satoris11:57
Cimiseb128, yours!11:58
seb128Cimi, ?11:58
Cimiseb128, can you run system settings and see if you have the icons?11:58
seb128I don't11:58
seb128I only have names and checkboxes11:58
Cimimpt, ping11:58
seb128Cimi, they should have icons, see https://wiki.ubuntu.com/Networking11:59
seb128Wellark should have a look I guess11:59
=== bregma_ is now known as bregma
=== MacSlow|lunch is now known as MacSlow
=== _salem is now known as salem_
mptCimi, did you report a bug about it?12:51
Cimimpt, nope, is a bug though12:51
Cimimpt, I was wondering if was a design change12:51
Cimiquite a crazy one :D12:51
mptA network should be presented the same way regardless of whether it’s in the indicator, in System Settings, or in the first-run setup <https://wiki.ubuntu.com/Networking#wi-fi-network>12:52
mptusing the same code, ideally12:54
mptCimi, reported bug 136436612:59
ubot5bug 1364366 in ubuntu-system-settings (Ubuntu) "Wi-Fi networks in System Settings lack strength or authentication icons" [Undecided,New] https://launchpad.net/bugs/136436612:59
=== zequence_ is now known as zequence
=== Malsasa_ is now known as Malsasa
Saviqdednick, testPreviewPayments13:19
dednickSaviq: hm. that fails even with LC_ALL=C13:21
Saviqdednick, note that the env in QmlTests.cmake is broken13:22
Saviqdednick, export as you call the test13:22
dednickSaviq: ah13:22
Saviqdednick, I never got around to redoing the test macros to gain some sanity in them13:23
dednickSaviq: mk13:23
dednickSaviq: C.UTF-8 don't work anyhow13:23
Saviqdednick, yeah, we never understood what the � there is ;)13:23
dednickwhy can't we dismiss notifications?!13:31
dednickor can we and i'm stupid?13:31
Saviqdednick, MacSlow is working on that13:33
dednickSaviq: where can i find the AP tests for silo qa?13:35
dednickmy keyboard is working fine on phone, but under AP the test fails. but i think my failure is due to the "Failed to add storage device" notification being constantly open13:38
Saviqdednick, right, reboot your phone13:38
dednicksince i think it's trying to do a search, which button is covered by the notification13:38
Saviqdednick, yeah, should go away after reboot13:38
Saviqadb shell gdbus introspect --system -p -d org.freedesktop.UDisks2 -o /org/freeedesktop/UDisks2/block_devices/mmcblk0 /org/freedesktop/UDisks2/block_devices/mmcblk0p2 | grep System13:39
Saviqdednick, ↑ to verify your phone's in a good state, that should say true13:39
dednickSaviq: doesn't say anything13:39
dednickafter i rebooted13:39
Saviqdednick, krillin is it?13:41
dednickSaviq: no. but test was fine (failed due to lack of input)13:41
dednickit stopped doing the notification13:42
Saviqdednick, k, so yeah, it must be we don't have focus on the dash any more or some such13:42
Saviqdednick, and vkb works because it works over dbus13:42
Saviqor well, maybe we have focus but the keyboard filter doesn't work properly13:42
Saviqdednick, WindowKeyFilter thing in Shell.qml13:43
dednickSaviq: is it keyboard input rather than touch input?13:43
Saviqdednick, yes, ap goes through keyboard input directly13:43
dednickah13:43
dednickwould explain why the touch is working then13:43
Saviqyes13:43
Saviqwhich, btw, is a good indication we'd break desktop session again...13:43
dednickmeh.13:44
dednick:)13:44
jgdxSaviq, do you know if there's a dbus call uss can make to reboot the phone?13:44
dednicki'll take a look13:44
Saviqjgdx, yeah13:44
jgdxSaviq, great. Care to share? :)13:45
Saviqjgdx, trying to get you a link ;)13:45
jgdxSaviq, woo!13:45
Saviqjgdx, http://bazaar.launchpad.net/~unity-team/unity8/trunk/view/head:/plugins/Unity/Session/dbusunitysessionservice.cpp13:45
jgdxSaviq, golden. Thank you13:46
Saviqjgdx, the login1 calls are probably the interesting ones to you13:46
jgdxSaviq, seems so:)13:46
Saviqmterry, hey, you know your rtm silo got NACK from QA on the two settings app bugs mentioned?13:55
mterrySaviq, yeah...  I'm trying to test / see what's up13:55
Saviqmterry, here's the report from ToyKeeper https://trello.com/c/9B6QLvDd/37-rtm-landing-003-mterry-unity8-ubuntu-system-settings-gsettings-ubuntu-touch-schemas13:57
Saviqmterry, not sure what to answer on the "makes it possible to launch apps" front13:57
mterrySaviq, that's a whole separate thing13:58
mterrySaviq, that at one point was how design wanted it, so the app was ready to go, but I can certainly imagine it being a problem13:58
MacSlowdednick, snap-decisions are currently getting the "swipe-to-dismiss" feature13:58
dednickMacSlow: cool.13:59
Saviqmterry, right, we should maybe start it and kill it if unlock failed / cancelled...13:59
Saviqmterry, but yeah, separate issue13:59
mterrySaviq, I don't think that helps -- I figured the problem was that an app might do something on startup?13:59
Saviqmterry, ok, let's have a chat about this at some point with security folk14:00
Saviqwe should file a bug14:00
mterrySaviq, I'm trying to schedule a meeting between design and security for an unrelated thing, I can ask about the launched-apps issue then too14:21
Saviqmterry, sure, please do14:21
=== Malsasa_ is now known as Malsasa
CimiSaviq, better Qt.rgba(0, 0, 0, 0.5) or color: "black"; opacity: 0.5 ?14:39
SaviqCimi, that's different behavior14:39
SaviqCimi, #7F should be 0.514:42
=== dandrader is now known as dandrader|afk
CimiSaviq, yup, using the calculator :)14:44
SaviqCimi, yeah, I was just unsure between 7F and 80 ;)14:45
CimiSaviq, is actually 8014:46
CimiSaviq, I just double checked14:46
SaviqCimi, well, that's 128, more than 0.5 * 255 ;)14:47
SaviqCimi, but yeah, it's .5 either way14:47
CimiSaviq, it's 7F.814:48
CimixD14:48
=== charles_ is now known as charles
SaviqCimi, ;)14:52
Cimimterry, https://code.launchpad.net/~cimi/ubuntu-system-settings/wizard.wifi-fixes/+merge/23307114:54
mterryCimi, can you link to a bug and/or expand the description?14:55
Cimimterry, done14:58
Cimimterry, the changes in Page.qml are cosmetics14:58
Cimimterry, the margin changed to 3 times when we added the pagination, but that was removed and the padding not restored14:59
Cimimterry, wordwrap was added to avoid wrapping Wi-14:59
Cimifi14:59
mterryCimi, why switch to WordWrap?14:59
Cimiin italian14:59
mterryCimi, ah... hmm.  But then doesn't Wi-Fi overlap the screen?14:59
Cimimterry, it shouldn't...15:00
Cimiis still there though15:00
Cimiweird15:00
Cimimterry, wi-fi is split here15:00
Cimimterry, you can try italian15:01
mterryCimi, we should just switch to a unicode non-breaking hyphen15:01
mterryCimi, "‑"15:01
Saviq15:03
Cimimterry, how?15:06
CimiSaviq,15:06
mterrySaviq, that's for parentheticals, right?15:06
Cimican I paste that?15:06
Saviqmterry, no idea, that's just compose + ---15:06
CimiSaviq, basically I have15:06
Cimi"Connetti alla rete Wi-15:07
Cimifi"15:07
Cimitranslations should be updated too15:07
mterrySaviq, yeah, that's an em dash, which is for parentheticals.  The en dash is for ranges and such.  The hyphen is for word joins15:07
mterryCimi, just copy paste the hyphen I gave you above15:08
mterryhttp://english.stackexchange.com/questions/2116/when-should-i-use-an-em-dash-an-en-dash-and-a-hyphen15:08
mterryCimi, I *assume* that Qt's word breaking logic takes the unicode semantics into account, but worth a test15:08
Cimimterry, works15:12
Cimimterry, but all potfiles will require update15:12
mterryCimi, what else is new, we change those strings all the time.  And design isn't done with the wizard15:13
Cimimterry, pushed15:14
Cimimterry, ok we need to fix button visuals now15:14
Cimimterry, they look shite15:14
Cimimterry, either in sdk or locally15:14
CimiI'd go locally15:14
Cimithen also push in sdk15:14
Cimiwizard is a custom themed app, I don't mind having a static theme for a button15:15
mterryCimi, I'd really rather just fix it in one place15:17
Cimimterry, so when they change the visuals there we will have a broken wizard?15:18
mterryCimi, ?15:18
Cimimterry, we want monochromatic theme here15:19
Cimimterry, the sdk is not now15:19
Cimiand I don't think we can go and say "let's change all buttons to white"15:19
Cimibecause it will affects all apps using surugradient15:20
mterryCimi, well that's my question -- I was under the impression that the visuals for the wizard were made using an updated version of suru.  If design intended those buttons to be wholly-custom to the wizard, that should be clarified -- I've got a meeting with them tomorrow to talk about a security thing too, I can ask then15:21
Cimimterry, ok15:21
=== alecu_ is now known as alecu
CimiSaviq, infographics, worth me looking or not?15:54
Saviqdandrader|afk, could you have a look at the changes to dednick's unity8 prompts branch, there was a focus issue16:06
SaviqCimi, looking at what? merging? sure, I hope to land this week16:06
=== dandrader|afk is now known as dandrader
dandraderSaviq, ok16:14
dandraderdednick_, could we have a qmltest covering that "issue with session focus"?16:16
greybackSaviq: fyi we've a few other qtmir we'd like to land after prompts-in-prompts does, can you let me know when it lands so I can get a silo for it16:22
Saviqgreyback, sure16:23
Saviqgreyback, unless it gets ACKed overnight and kgunn lands it, won't be earlier than tomorrow noon-ish16:24
greybackSaviq: that's ok, I just want to avoid parallel silos of qtmir, as it just causes pain16:25
dednick_dandrader: i can add a test, but i'm having some problems reproducing in qmltests17:17
dednick_but off for the day now17:17
dednick_well, at least until later17:17
=== elopio_ is now known as elopio
Cimimterry, commented inline17:26
Cimimterry, replaced ListView with Repeater + column + flickable17:26
kgunngreyback: so, dednick's prompt in prompt doesn't look like it'll land until tomorrow earliest (still got some needs fixin' going on)17:26
kgunnis it ok to hold you off17:26
kgunnfor maybe a 2 days ?17:27
kgunnor that too much ?17:27
CimiSaviq, in regards of this review https://code.launchpad.net/~cimi/ubuntu-system-settings/wizard.wifi-fixes/+merge/233071 better to have hiddenComponent or setting sourceComponent to null?17:28
SaviqCimi, of course better to null, why would you have a blank component?17:42
CimiSaviq, residue of the past17:42
CimiSaviq, C17:42
CimiSaviq, avoiding null pointers as much as possible :)17:42
Cimimterry, pushed with null component17:43
Cimishould be better now, works fine17:43
Cimithe default wallpaper on the phone has 1px bright line on right edge of the screen17:47
Cimican you guys confirm that?17:47
Cimihttps://bugs.launchpad.net/ubuntu/+source/unity8/+bug/136453717:54
ubot5Ubuntu bug 1364537 in unity8 (Ubuntu) "phone background has 1 px bright line on its right edge" [Undecided,New]17:54
Cimi(it was annoying me)17:54
=== dandrader is now known as dandrader|afk
Cimimterry, those fixes are fine for now, but I believe we'll be asked to add hidden network too...18:06
=== dandrader|afk is now known as dandrader
=== dandrader is now known as dandrader|afk
CimiSaviq, mzanetti do we have a bugreport for this gst audio issue stopping apps to work on the desktop?19:31
SaviqCimi, yeah there is a slew of 'em actually19:32
SaviqCimi, bug #136331419:32
ubot5bug 1363314 in Ubuntu Clock App "App fails to launch on up-to-date utopic desktop" [Critical,Confirmed] https://launchpad.net/bugs/136331419:32
CimiSaviq, thx19:33
=== dandrader|afk is now known as dandrader
bregmar7b...21:01
bregmabrzbup21:02
* bregma is having keyboard trouble21:02
dednickSaviq: you still about?21:18
Saviqdednick, kinda21:18
dednickSaviq: :) having some really straing behaviour in a test case.21:18
Saviqdednick, try me21:18
dednickSaviq: http://paste.ubuntu.com/8218316/21:20
dednickthe test down below SessionManager.createSession returns a qobject pointer with c++ ownership.21:21
dednickin testXXX the test function creates 2 copies of Session, in tryXXX it only creates ones like it should21:21
dednickSaviq: when assigning "sessionContainerLoader.item.session = fakeSession;" it creates another "fakeSession" object21:21
dednickSaviq: if i don't use a loader, it seems to work ok...21:22
Saviqdednick, what type is SessionContainer.session?21:23
dednickSaviq: QtObject21:23
Saviqok so nothing special21:23
dednickSaviq: yeah, same happens if it's a var.21:24
Saviqdednick, just FYI, if a prop is of a complex type, even if you go : null, it will create the object on creation and replace it with null just after :|21:24
Saviqbut not the issue here of course21:25
dednickSaviq: ya21:25
Saviqdednick, so is it calling .createSession twice?21:25
Saviqor does it copy fakeSession or something?21:26
Saviqdednick, so the symptom is that your .createSurface is called on a different object than the fakeSession you just got from .createSession()?21:27
dednickSaviq: it's calling createSession twice.21:27
Saviqdednick, did you try breaking in createSession() to see the trace?21:27
dednickSaviq: hm. that's odd21:27
dednickit's like it's creating a function pointer from createSession21:28
dednickSaviq: no, not yet21:29
Saviqdednick, that'd be really odd, check the type of fakeSession?21:32
Saviqdednick, but even if that was the case, then the object would be of wrong type in item.session (not even QtObject IIRC)21:33
Saviqdednick, and there's nothing that would actually call it21:33
dednickQDEBUG : qmltestrunner::SessionContainer::test_interative() qml: Session(0x2478800)21:34
dednicktrying to debug now21:34
dednickdangit. how the hell do you add a custom run to qtcreator?!21:41
dandraderdednick, are you by any chance creating a session on the test init/cleanup funtions?21:50
dednickdandrader: nope21:51
dednickdandrader, Saviq: looks like something funky is going on though... i removed all the other code from the test and it stopped doing it :/21:56
Saviqdednick, stupid question: is the test function only called once?21:57
dandraderwhen I'm haunted by mysterious issues like that, usually the best thing is approaching it again later, with a fresh mind, next morning.21:57
dednickSaviq, dandrader: sorry, it looks like i'm a dumbass. the createSurface function is being called by something else21:58
dandraderdednick, np, glad you found it :)21:59
Saviqhaving a sounding board is often helpful :)21:59
=== salem_ is now known as _salem
dednickkgunn: i've added the test for the problem we were having now23:26
dednickzzzzzz23:26

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