/srv/irclogs.ubuntu.com/2014/03/27/#ubuntu-unity.txt

=== duflu_ is now known as duflu
=== _salem is now known as salem_
=== salem_ is now known as _salem
=== jamesh_ is now known as jamesh
pstolowskiSaviq, ping08:21
tsdgeos_salem: where do crashes like https://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-mako/6138/ get uploaded to? errors.u.c?08:21
Saviqpstolowski, hey08:21
Saviqtsdgeos, yes08:21
Saviqtsdgeos, but truth be told they don't always end up in the reports there for some reason08:22
pstolowskiSaviq, hi! do you know if the 'finished' flag (set by shell/plugin and sent to scope) for click installation is set when it actually finishes intallation, or just on download completion?08:23
Saviqpstolowski, I'd say just download08:24
Saviqpstolowski, since we're using the DownloadTracker, which doesn't know about the installation afaict08:24
pstolowskiSaviq, ok.. I think that's causing race for me on click side :(08:25
Saviqpstolowski, very possible08:25
Saviqpstolowski, I didn't like this from the beginning, but we never had time to implement this proper08:25
pstolowskiSaviq, ok, will investigate further, perhaps will need some workaround in click08:25
Saviqpstolowski, problem is there's nothing else running... just the download tracker08:26
Saviqs/tracker/manager/08:26
Saviqpstolowski, so after that's finished, we'd have to do another progress bar for the installer08:27
pstolowskiSaviq, ah, I see. is there an 'installed' signal you could subscribe to instead when progress reaches 100%08:28
pstolowski?08:28
pstolowskiSaviq, and delay 'finished' until you get it?08:28
Saviqpstolowski, the download manager does not install it08:29
tsdgeosSaviq: do we have any proper howto of how to get the info of those crash files? I remember we spoke on how to do it but have sincerely forgotten about it08:29
Saviqtsdgeos, you mean on errors.u.c or locally?08:29
tsdgeosSaviq: locally08:29
Saviqtsdgeos, you wanna do on device or on host?08:30
tsdgeosSaviq: whichever is easier08:30
Saviqtsdgeos, then device, probably08:30
Saviqtsdgeos, install apport-retrace08:30
Saviqtsdgeos, and then `apport-retrace -g file.crash` will get you in a gdb session in that crash08:30
pstolowskiSaviq, ack, what i'm after is if you could subscribe to some other service to get notified about installation status?08:30
tsdgeosSaviq: okidoki08:31
Saviqtsdgeos, installing more dbg symbols on the phone will then let you see more08:31
tsdgeossure08:31
Saviqpstolowski, that's the thing, there isn't any08:31
pstolowskiSaviq, ok..08:31
Saviqpstolowski, and that would be building knowledge into the shell/dash, we need to make it generic08:31
pstolowskiSaviq, who triggers actual installation?08:34
Saviqpstolowski, I think there's a hack in the download manager08:35
pstolowskiSaviq, ah, one more hack, ok :/08:35
Saviqpstolowski, it was discussed on the ubuntu-phone ML some time recently08:36
tsdgeosSaviq: getting "BFD: Warning: /tmp/apport_core_0gtdfi is truncated: expected core file size >= 232255488, found: 36765696.", is that usual?08:36
pstolowskiSaviq, thanks08:36
Saviqtsdgeos, yeah, we need to up the kill timeout08:36
Saviqtsdgeos, means the app was killed by upstart before apport managed to collect the crash report :|08:37
tsdgeos:/08:37
tsdgeosso useless report08:37
Saviqyeah08:37
Saviqtsdgeos, https://code.launchpad.net/~saviq/unity8/increase-kill-timeout/+merge/21300308:40
tsdgeosdone08:41
* Saviq flushes unity8 queue, then08:44
Saviqtsdgeos, top-ack?08:45
tsdgeosoh08:45
tsdgeossleept08:45
tsdgeoshmmm09:18
tsdgeosthe carousel is a bit bad in regards to delegate creation on startup09:19
Saviqtsdgeos, yeah, that's what I expected09:19
tsdgeoshttp://paste.ubuntu.com/7161682/09:21
tsdgeosnow to find out why :)09:21
tsdgeosdednick: https://code.launchpad.net/~nick-dedekind/unity8/indicators.hint-interval/+merge/20210409:24
tsdgeossizing seems to kick too late09:26
tsdgeoscards are 0x0 and thus carousel seems to think it needs to create them all09:26
tsdgeosthen they get resized09:26
CimiSaviq, saw this http://dorey.github.io/JavaScript-Equality-Table/ \o/09:44
Cimitsdgeos, what's the issue?09:48
Cimidednick, https://code.launchpad.net/~nick-dedekind/ubuntu-settings-components/access-point-init/+merge/212913/comments/50335909:49
tsdgeosCimi: still don't know looking at it :)09:49
Cimido we have some sort of semaphores to avoid using boolean vars like this? https://code.launchpad.net/~nick-dedekind/ubuntu-settings-components/access-point-init/+merge/212913/comments/50335909:50
Cimiin qml09:50
dednickCimi: responded09:51
Cimidednick, component.oncompleted is the first call?09:53
Cimidednick, well yes09:54
Cimidednick, but are we sure we won't have races?09:54
SaviqCimi, shame it shows everything twice ;)09:55
Saviqi.e. it's symmetrical by the diagonal09:56
Cimiyep09:56
CimiI'm gonna share with us :D09:56
tsdgeosat least the table is symmetrical10:00
tsdgeoscould be worse :D10:00
dednickCimi: the oldValue is only used because those functions can be called from each other. nothing can call Component.onCompleted other than qt event loop (or whatever system it uses)10:04
Cimidednick, sure10:05
Cimidednick, but what happens if it is executing component.oncompleted10:06
Cimidednick, then menu.checked changes?10:06
Cimiit will skip10:06
tsdgeosCimi: Saviq: https://code.launchpad.net/~aacid/unity8/carouselNoNoSizeDelegates/+merge/21301310:15
Saviqtsdgeos, cool10:17
Cimitsdgeos, Saviq I might use > 0 instead != 0 ?10:17
tsdgeossure, let me change that10:18
=== dandrader is now known as dandrader|afk
dpmtsdgeos, hi! A quick question about Qt locales: I'm looking at a MP for the Calendar app that adds the SDK's DatePicker component, and I've noticed that for 'ca' it shows the wrong "$DAYNR $DAYNAME" (it should be the other way around) http://i.imgur.com/JXt1w7N.png - zsombi tells me that comes straight from the locale definitions, in which case it seems that ordering would be wrong for Catalan and would need to be changed upstream?10:21
dpmit seems to get it from Qt.locale.dateFormat(Locale.ShortFormat)10:21
dpmlet me try if that prints the right order10:21
tsdgeosdpm: you mean you want "Dissabte 26" and not "26 Dissabte"?10:22
dpmyes10:22
tsdgeosmay very well be what zsombi says10:22
dpmtsdgeos, so where are those locale definitions upstream?10:26
tsdgeosdpm: are you doing a small testcase or want me to?10:26
tsdgeosoh there's a big magic file with magic numbers10:26
tsdgeosonce you see it you won't be able to sleep anymore :D10:27
dpmhahaha10:27
tsdgeoslet me look for it10:27
dpmcool, thanks!10:27
tsdgeosSaviq: Cimi: pushed the != -> >10:27
tsdgeosor may well be the posix locales too, give me a few mins10:28
dpmno rush, thanks10:29
=== dandrader|afk is now known as dandrader
Cimidednick, ?10:34
dednickCimi: it can't happen while Components.onCompleted is executing...10:43
Cimidednick, ah ok10:43
Cimidednick, so all connections are blocked until component.oncompleted is executed?10:43
CimiI mean, everything?10:43
Cimidon't know how multithreading works here10:43
dednickCimi: it's a single thread as far as i'm aware10:44
Cimimight worth asking10:47
CimiSaviq, tsdgeos hey10:47
tsdgeosyes?10:48
Cimisee this http://bazaar.launchpad.net/~nick-dedekind/ubuntu-settings-components/access-point-init/view/head:/Ubuntu/Settings/Menus/AccessPointMenu.qml10:48
Saviqdednick, this never worked afaict10:48
Saviqdednick, i.e. there was never a Changed signal unless there was a binding (and so the evaluation of the binding delayed setting of its value)10:49
Cimidednick, can we use an alias to start?10:55
Cimidednick, aliasing menu.checked with checkbox.checked?10:56
dednickCimi: there is a comment there saying why we can't use a binding11:04
dednickit worked because the object used to be completed before we started setting any values11:04
dednickCimi: because we used a Loader.11:05
Cimidednick, sorry I don't understand11:06
dednickLoader { Component.onCompleted: { menuModel = model } } kind of thing11:06
Cimidednick, I trust you, I'm just trying to understand11:07
Cimidednick, so if we do for id:menu (parent), property alias checked: checkbox.checked11:07
Cimidednick, what will happen, example?11:07
Cimidednick, otherwise we can have a second property like serverChecked?11:08
dednickCimi: at the moment it doesnt work in your code, because we use a binding to set the value of checked in the listview delegate. Because you are directly using the AccessPoint as a delegate, the object is NOT COMPLETE when you set "checked: model.isToggled" (or whatever), so the Connection to menu.checked in AccessPointMenu is never triggered11:08
dednickin the settings page, or in indicator code: We use a Loader, which we wait unitl it is complete, before setting the values.11:09
Cimidednick, I understand. So why an alias won't work?11:11
Cimisorry for asking and asking, but all this binding stuff and connections and loaders is confusing me now11:11
Cimimaybe I need a second coffee11:11
Cimimaybe it's aging11:11
Cimisince yesterday was my bday11:11
dednickCimi: we can't use an alias, because we dont want the menu.triggered(menu.checked) code to run when the server updates the value.11:11
Cimidednick, can't we use onCheckedChanged and so for that?11:12
dednickwe only want to send info to server when the user changes the value manually.11:12
Cimiand trigger only when there is an actual click on the checkbox?11:12
Cimidednick, like with that mousearea?11:13
Cimidednick, menu.triggered is emitted only when             target: menu.__mouseArea11:14
Cimi            onClicked: {11:14
Cimi                checkbox.clicked();11:14
Cimi            }11:14
dednickCimi: no11:14
Cimiok11:14
dednickCimi: signals are not emitted order. checked might or might not have changed when we get that clicked signal.11:15
dednickCimi: // FIXME : should use Checkbox.toggled signal11:15
dednick:)11:15
Cimiyes11:15
Cimibut nobody approves11:15
dednickCimi: it's the same for the slider. there is no "user changed value" signal11:16
Cimiwhat's saviq's opinion on those widgets?11:16
dednickalthough florian says the behaviour is well defined11:16
dednickCimi: i have no idea :)11:18
dednickCimi: believe me, i didnt want it this way. which is why i tried to add the toggle signal11:19
dednickit's a friggin pain in the arse11:19
dednickCimi: i'll take a look at the checkbox. maybe they've changed something, but when i tried it didnt work11:21
* Saviq just got the polishItems crash :|11:27
Saviqdednick, Cimi, let me know if you need me11:31
CimiSaviq, we have this ugly code here http://bazaar.launchpad.net/~nick-dedekind/ubuntu-settings-components/access-point-init/view/head:/Ubuntu/Settings/Menus/AccessPointMenu.qml11:31
CimiSaviq, nick wrote this because apparently it's the only way, we are both unhappy about it11:31
CimiSaviq, connections and re-setting values11:32
Cimiinstead bindings or such11:32
SaviqCimi, so two-way sync? I don't think there's a better way than that11:32
dednickCimi: been testing the Checkbox. looks like clicked slots do com after internal ones. But I have no idea if it's a "well known" order11:33
dednickSaviq: do you know if there's an order for signal handlers to be triggered?11:33
dednick"slots" or connections11:33
Saviqdednick, nothing to rely on11:33
SaviqCimi, bindings only work one way, so if you need to sync two places, connections to onChanged is the only solution11:33
dednickSaviq: right. that's what i thought11:33
tsdgeosSaviq: what were you doing? can repro?11:34
Saviqtsdgeos, of course I can't11:34
Saviqtsdgeos, well, will try in a bit11:34
Saviqtsdgeos, but what I was doing is I managed to crash the click scope11:34
tsdgeos:/11:35
Saviqtsdgeos, you can try: my steps → uninstall and install webbrowser, search again11:35
CimiSaviq, but when we have those booleans11:36
Saviqtsdgeos, it will get stuck on "Install", 'cause you shouldn't be able to uninstall it anyway11:36
Saviq(that's when scope crashes)11:36
Saviqtsdgeos, it crashed when I tried searching again11:36
CimiSaviq, property bool enableCheckConnection: true11:36
Cimiit's a rudimental semaphore11:37
Cimiis there anything smarter in qml?11:37
SaviqCimi, not really, but I'm not sure it should really be necessary, since it will just feed back once11:37
SaviqCimi, well, it might be worse for slider, for exampe11:37
Saviqexample11:37
Saviqbut for bools it would just come back as the same value that's there on the target property already11:38
dednickCimi: there was some talk about creating an object to manage the 2 way sync so we didnt have to do it in different classes, but i never got around to it.11:38
dednickfor now we just have to stick it out11:39
Saviqtsdgeos, got it again, different steps, but searching seems to trigger it for me11:55
tsdgeos'nteresting11:55
Saviqtsdgeos, searching in the apps scope was what I was doing in both cases11:56
tsdgeosSaviq: phone?11:56
Saviqtsdgeos, yes11:56
tsdgeosso you're runnign with run_on_Device?11:56
pete-woodsSaviq: was it the same as this crash? https://bugs.launchpad.net/ubuntu/+source/unity-scope-click/+bug/129729711:56
ubot5Ubuntu bug 1297297 in unity-scope-click (Ubuntu) "Click scope crash during search" [Undecided,New]11:56
tsdgeosor how do you detech the crash?11:56
tsdgeosdetech! new word11:57
Saviqtsdgeos, it just stops responding :)11:57
Saviqtsdgeos, and apport hogs the cpu11:57
Saviqtsdgeos, and there's a .crash file in /var/crash11:57
tsdgeosSaviq: but how do you know it's the polishItems thing?11:57
tsdgeosah because it has time to dump11:57
Saviqpete-woods, no, yours is different11:57
pete-woodsokay11:57
pete-woodsthanks for looking11:57
Cimidednick, is there like a property on the unitymenumodel I can use to see if I'm connected to a network?11:58
Saviqpete-woods, but I was actually talking about a unity8 crash right now, got another one in the scope11:58
Saviqpete-woods, let me find it11:58
pete-woodsSaviq: it's more to help the guys figure out what happned11:58
pete-woodsmy crash file didn't seem to help them much11:58
Saviqpete-woods, https://bugs.launchpad.net/ubuntu/+source/unity-scope-click/+bug/129831411:58
ubot5Ubuntu bug 1298314 in unity-scope-click (Ubuntu) "scoperunner crashed with SIGSEGV in UbuntuOne::Token::signUrl()" [Medium,New]11:58
pete-woodsah, okay, quite different then11:59
Saviqpete-woods, mine has easy steps to repro, too, it's trying to install a non-click, already installed app (e.g. webbrowser)12:00
pete-woodsyeah, mine was just I searched, it crashed, ??12:00
pete-woodsand it didn't happen the next time12:00
Saviqpete-woods, our "bigger" issue is not that the scope crashes, but unity812:00
pete-woodsyeah, that's totally fair, unity8 trumps almost any other crash12:01
pete-woodsexcept maybe kernel :p12:01
Saviqtsdgeos, and again12:07
Saviqtsdgeos, this seems like a reliable way to repro, just search, clear, search, clear12:07
Saviqtry with slow / fast typing, selecting an item from the history and such12:07
tsdgeosok12:07
* tsdgeos tries12:07
tsdgeosgot a crash, let's see if i can see if it's polishitems12:08
tsdgeosnothing in /var/crash12:09
tsdgeosconfusing12:09
Cimidednick, ?12:09
Cimidednick, I'm looking for a way to know if I'm connected to a wifi or not, from wizard12:10
Saviqtsdgeos, assign yourself to bug #1297240 please12:11
dednickCimi: sorry. no, i dont believe there is12:11
ubot5bug 1297240 in unity8 (Ubuntu) "unity8 crashed with SIGSEGV in QQuickWindowPrivate::polishItems()" [High,Confirmed] https://launchpad.net/bugs/129724012:11
Cimidednick, but the indicators12:11
Saviqtsdgeos, hmm? and that was unity8 launched just as usual, nothing special?12:11
Cimidednick, the wifi indicator says connected12:11
tsdgeosSaviq: think so, yes12:11
Cimidednick, on the panel12:11
Saviqtsdgeos, you might want to up the kill timeout in /usr/share/upstart/sessions/unity8.conf12:11
Cimidednick, is this something I can fetch?12:11
dednickCimi: oh. connected to wifi. er12:12
dednickthere's an action12:12
Saviqtsdgeos, but there should be a .crash file regardless12:12
dednickCimi: er, but i think it's just the root item icon, not a "i am conencted"12:12
tsdgeosscope runner crashed12:13
Cimidednick, deom unitymenumodel?12:13
Cimi*from12:13
tsdgeosis there anyway to recover from that other than reboot?12:13
=== _salem is now known as salem_
=== MacSlow is now known as MacSlow|lunch
Saviqtsdgeos, restart scope-registry12:26
dednickCimi: what do you actually need to show?12:35
Cimidednick, I want to change label of the continue button from skip to connect when it's connected12:36
Cimiopposite12:36
Cimiwhen it's connected continue12:36
Cimiwhen it's not skip12:36
Cimidednick, I can use a global variable set by the delegates when isToggled is true for one of them12:37
mzanettiSaviq: did you already trigger the right-edge review somehow?12:37
Cimidednick, but was looking for better solutions12:38
Cimidednick, thinking that I can have the same logic of the indicator row12:38
Saviqmzanetti, yes12:38
Saviqmzanetti, let me find out what's the progress12:38
mzanettiawesome :)12:38
dednickCimi: dont think there is a way. probably best to check with ted or Wellark12:40
Cimidednick, I'll go for the global variable12:40
Cimidednick, but one day we will need it12:40
Cimidednick, we need an "you have network, you don't"12:41
dednickCimi: then check with the indicator gues12:41
dednickguys12:41
dednickthere used to be an action for it, which is what the wifisection was using12:42
=== alex_abreu is now known as alex-abreu
CimiWellark, ? :)12:43
=== jhodapp|afk is now known as jhodapp
Cimidednick, do you remember this action?12:44
dednickCimi: it was in the chewie network indicator12:44
Cimiah ok12:44
Saviqmzanetti, they have a device and your ppa on it, so that's good progress!12:45
Cimidednick, he might be on holiday12:48
CimiWellark12:49
mzanettiSaviq: cool, thanks12:54
dednickCimi: well for now, i dont think the action exists. but maybe you should add a bug12:56
Cimidednick, I did a quick workaround12:56
Cimidednick, http://paste.ubuntu.com/7162565/12:57
Cimimterry, can we start with fullscreen or not?13:00
dednickCimi: > 0 ?13:00
=== dandrader is now known as dandrader|afk
Cimidednick, well, if it has 2 is not connected...13:00
mterryCimi, in wizard?13:00
dednickCimi: one of them may be :)13:00
Cimidednick, that's why I put 113:00
Cimiahah13:01
Cimimterry, yup13:01
Cimimterry, makes debugging on desktop harder though13:01
mterryCimi, I mean we can.. I don't remember the logic in its main.cpp off the top of my head13:01
Cimimterry, maybe I should just set it to be taller?13:01
Cimimterry, the wizard has black empty space at bottom13:01
Cimi@unity is there a way to have apps filling screen but not being fullscreen?13:02
mzanettiCimi: ?13:02
Saviq???13:02
Cimilike, if panel is shown, anchors down the panel, otherwise fullscreen?13:02
mterryCimi, yeah, you could get the size of the screen and resize the view13:02
Saviqmterry, I'm not sure that's actually possible13:03
=== jhodapp is now known as jhodapp|brb
Saviqgreyback, ↑ what are we doing with surfaces these days?13:03
mzanettiit should be as long as you don't call showFullscreen() but just show() or similar13:03
CimiSaviq, wizard starts outside unity, so should be fullscreen13:03
mzanettihmm, yeah... unity-mir might override it13:03
SaviqCimi, yes, so why shouldn't it be fullscreen?13:03
Cimibut if you run on desktop or inside unity, should not be fullscreen13:04
greybackSaviq: you'll have to be more specific13:04
mzanettiah... showMaximized() perhaps helps with that13:04
Saviqmzanetti, I don't think that'll work13:04
SaviqCimi, when will it ever be run on desktop or inside unity?13:04
CimiSaviq, maybe we want to13:05
Cimirun it after?13:05
SaviqCimi, but how does "fullscreen, but not fullscreen" solve that?13:05
CimiSaviq, should fill the screen13:05
SaviqCimi, so fullscreen :D13:06
Cimibut stay under panel if panel is in13:06
SaviqCimi, that's not the screen13:06
SaviqCimi, that's available geometry13:06
SaviqCimi, and that's what maximized is for13:06
CimiSaviq, X11 had workarea13:06
Cimiok13:06
Saviqbut I don't think we're handling maximized at all13:06
Cimiah ok13:07
SaviqCimi, but I really think you're overdoing it13:07
CimiSaviq, so answer is not for now13:07
mzanettiSaviq: on the phone. I think its just ignored and shown (non-fullscreen)13:07
Saviqmzanetti, well, it's just always maximized anyway ;)13:07
=== alan_g is now known as alan_g|lunch
mzanettiyep13:07
mzanettiso yeah, showMaximized might do the trick. but I haven't tried13:07
Saviqmzanetti, it won't13:07
mzanettiok. I give up :)13:08
Saviqmzanetti, we don't have struts and such13:08
Saviqmzanetti, so it will just always be under the panel, even if it's not there13:08
Saviqi.e. there's no way to tell what's maximized atm13:08
mzanettiah... I see13:08
mterryCimi, the wizard isn't going to be run as an app though, just fyi.  not like some random click app13:10
Cimimterry, yeah I know13:10
Cimimterry, I'll do fullscreen13:10
mterryk13:10
=== MacSlow|lunch is now known as MacSlow
Cimimterry, seb128 https://code.launchpad.net/~unity-team/ubuntu-system-settings/wizard.wifi/+merge/21267513:12
Cimione thing I want to make sure is that the upgrade of the deb will remove the 20-wifi.disabled file in /usr/share/ubuntu/settings/wizard/Pages13:13
Cimiqml/paged13:13
Cimipages13:13
=== jhodapp|brb is now known as jhodapp
=== alan_g|lunch is now known as alan_g
greybackhey folks, anyone here a qmake guru? I'm having a problem generating a pkgconfig file using qmake13:50
=== dandrader|afk is now known as dandrader
tsdgeosgreyback: what's your problem?14:10
tsdgeosand why are you still using qmake? :D14:11
greybacktsdgeos: existing project, no choice14:11
greybacktsdgeos: http://bazaar.launchpad.net/~gerboland/qtubuntu/server-custom-input-destination/view/head:/src/platforms/ubuntu/ubuntumir/ubuntumirserver/ubuntumirserver.pro14:11
greybacktsdgeos: it just doesn't create the file at all14:12
tsdgeosgreyback: the interwebs say14:15
tsdgeos CONFIG += create_pc14:15
greybacktsdgeos: I had that initially, which didn't help. Then had http://stackoverflow.com/questions/6823862/qmake-creating-a-pc-file14:16
tsdgeosgreyback: don't know moar14:16
tsdgeosi'd go with reading qmake's code14:17
tsdgeosdid it once, it's not that hard14:17
greybackI was hoping to avoid that :)14:17
greybackbut yeah14:17
tsdgeoshmmmm14:22
tsdgeosi'm running the shell manually on the phone and i have no keyboard14:22
tsdgeosany idea why?14:22
tsdgeosis there an upstart job for it or somethig?14:23
tsdgeosyep14:24
tsdgeosmaliit-server14:24
tsdgeosworst name ever14:24
Cimiseb128, mterry when can you review that branch?14:33
seb128Cimi, why did you set it as merged?14:34
Cimiseb128, I set needs review, bloody dropdown menu14:34
Cimiseb128, I redid immediately to needs review14:34
seb128Cimi, weird, I got an email saying merged14:34
seb128Cimi, but adding to my review queue, it might not be for today though14:34
Cimiseb128, I'll rebase my location branch on top of that then14:35
Cimikgunn, do we needs tests for the wizard?14:35
Cimiusually answer is we need tests for everything14:36
Cimibut this interaction things with funcionality might be more tricky and time consuming14:36
kgunnCimi: how might it be more tricky ?14:37
kgunnCimi: but yes, i would say let's create a nice AP test for that14:38
Cimikgunn, because what you test if not the settings are correctly done?14:38
Cimiit's not about testing UI here14:38
Cimikgunn, nice AP is utopia :P14:39
CimiI can do an AP14:39
Cimi:D14:39
mterrykgunn, if you have some seconds, rebuilding unity-system-compositor and unity8 in silo 004 would be neat.  But not urgent14:41
kgunnCimi: :D14:41
kgunnCimi: i will take something less than utopia14:41
greybackSaviq: QSG_RENDERER_DEBUG=render,build,change,upload,roots,dump,noalpha,noopaque,noclip <- pick & choose what ones you want. "render" handiest14:42
Cimikgunn, can I do this later?14:42
Cimiat a later time14:42
kgunnCimi: can you do something super minimal at least ?14:43
Cimikgunn, I'll have  alok14:43
Cimikgunn, code is ready, just missing tests14:44
kgunnthanks Cimi14:45
kgunnmterry: kicked14:46
mterrykgunn, thanks!14:46
Cimitsdgeos, Saviq an item's height does not take into account margins?15:13
tsdgeoshmmm15:16
tsdgeoswhat od you you mean with that?15:16
Cimitsdgeos, if I set margin 100px, height is not calculated15:16
Ciminevermind, did in a different way15:16
tsdgeosok15:17
Saviqdavidcalle, glad you're liking the tool :D15:28
Saviqdavidcalle, we actually plan(ned) to build some more logic into it, but it's definitely a good first step15:29
davidcalleSaviq, since I don't have a device, I've been using it for a couple of months now, it's my favorite 14.04 feature :p15:30
Saviq:D15:30
=== alan_g is now known as alan_g|tea
paulliuSaviq: https://code.launchpad.net/~paulliu/unity8/fix-lazyimage1/+merge/21172815:39
paulliuSaviq: BTW, I'm too unfamiliar with jenkins. Is there any newbie docs for that?15:40
Saviqpaulliu, it's rather unstable recently15:41
Saviqpaulliu, so don't worry15:41
paulliuSaviq: I access it through VPN but feel nothing to do there.15:41
Saviqmterry, hey, on the auto background15:41
Saviqmterry, should we not dpkg-depend on whatever provides that schema?15:41
Saviqmterry, I think it's a relatively new key?15:41
Saviqmterry, or maybe check that the schema/key exists, so we don't crash if it doesn't?15:42
mterrySaviq, yes, we should.  I was under the impression we already depended on it, but I see we don't15:43
Saviqmterry, make sure to put it in ./build, too15:43
mterrySaviq, I thought that was automatic now15:44
Saviqmterry, only for build deps15:44
Saviqmterry, not for runtime ones (somewhat impossible to be automatic due to some of them being generated)15:44
tsdgeosSaviq: are you adding the small optimization stuff i did in this landing?15:45
Saviqtsdgeos, I will15:45
tsdgeosok15:45
mterrySaviq, ah...  it is already a dep, just with the old package name15:45
tsdgeosthere's 3 small ones yet to be reviewed15:45
mterryor maybe I'm wrong15:46
Saviqtsdgeos, not really testable the carousel one is it...15:46
tsdgeosSaviq: i guess i could test it yes15:46
tsdgeosstoring how many delegates are created15:47
tsdgeos?15:47
Saviqtsdgeos, count them15:47
tsdgeosyou can't check how many there will be at the end15:47
tsdgeosbecause it'll be correct anyway, but the total should be doable15:47
mterryyup, guess desktop-schemas and ubuntu-schemas are different15:47
tsdgeoslet's see if i see how to do it though :D15:47
Saviqtsdgeos, well, yeah, only on creation15:47
Saviqmterry, gsettings-ubuntu-schemas ships that schema I think15:47
Saviqtsdgeos, I'll land it separately then, want to unblock the silos asap, we're running low15:48
mterrySaviq, yeah I got my aliases mixed up.  We were depending on gsettings-desktop-schemas.  Which of course is a different thing  :)15:48
tsdgeosSaviq: land separately the test or the whole thing?15:48
mterrySaviq, branch updated15:48
Saviqtsdgeos, the test I think, that's an important, and somewhat obviously correct, thing15:48
Saviqmterry, thanks15:48
tsdgeosSaviq: ok15:48
Cimidednick_, the access point menu is basically an extension to the checkmenu, can we merge them?15:54
dednick_Cimi: an extension? not really. It has a checkbox in it, but it's not the same15:55
dednick_as far as i remember15:56
Cimidednick_, almost like15:56
Cimidednick_, it has some extra stuff15:56
Saviqtsdgeos, actually, I'll land that with the right edge, don't want to wait for rebuilding15:57
Saviqand right edge I'll need to rebuild anyway15:57
Cimidednick_, I find it confusing we have checkbox on left here15:58
Cimidednick_, while in optionmenu the tick is on the right15:58
tsdgeosSaviq: ok15:59
dednick_Cimi: the AccessPoint alignment is different, it also has an icon between text and checkbox, as well as an icon on the right.16:00
Cimiok16:01
Cimidednick_, reusing that checkbox with the logic for the connection would be nice though16:02
dednick_Cimi: yeah. that's the point of having a bi-directional syncing object16:03
dednick_just havent dont it yet16:04
Cimidednick_, where is checkable menu used?16:09
dednick_Cimi: um. not sure it is anymore. used to be on bluetooth or something16:11
dednick_but i think it's using switch now16:11
Cimidednick_, it is16:11
Cimidednick_, let's drop it?16:11
=== alan_g|tea is now known as alan_g
=== gatox is now known as gatox_lunch
tsdgeosSaviq: Cimi: https://code.launchpad.net/~aacid/unity8/test_carouselNoNoSizeDelegates/+merge/21310916:50
tsdgeoslooks huge16:50
tsdgeosbut it's only a few lines + indentation of the test16:50
tsdgeostest->rest16:50
Cimitsdgeos, does this work?16:51
tsdgeoswhat work?16:51
Cimitsdgeos, thought the prevopus was unit test16:51
Cimiprevious16:51
tsdgeos?¿16:52
Cimitsdgeos, this one requires the window16:52
tsdgeosyes16:52
Cimitsdgeos, from early times, I remember having unit tests16:52
Cimiand qmltests16:52
tsdgeosah16:52
tsdgeossee what you men16:52
tsdgeosa16:52
Cimithis looks like it's a qmltests so requires cmake change16:52
tsdgeosright need to move it16:53
tsdgeosyes16:53
tsdgeoswell spotted16:53
Cimicool16:53
Saviqtsdgeos, also, could we move TestCase outside of Carousel?16:53
tsdgeosSaviq: why?16:53
tsdgeosSaviq: we have all the tests like this one16:53
tsdgeosSaviq: wait, it's outside the carousel, no?16:53
tsdgeosoh, it' not :D16:54
tsdgeosCimi: Saviq: pushed16:54
Saviqtsdgeos, I generally put TestCase next to the tested item, not inside it16:55
tsdgeosyes yes16:55
tsdgeosdone16:55
Saviqok16:55
Saviqbiab16:56
kgunntsdgeos: ...who would be best to look at this ?17:06
kgunnhttps://bugs.launchpad.net/ubuntu/+source/qtubuntu/+bug/129790017:06
ubot5Ubuntu bug 1297900 in qtubuntu (Ubuntu) "crash QUbuntu: Could not create application instance" [Undecided,New]17:06
tsdgeoskgunn: gerry :D17:07
tsdgeosor ricmm17:07
tsdgeosmaybe17:07
tsdgeosi can have a look i guess too17:07
kgunntsdgeos: yep...i was thinking of begging for ricmm17:07
kgunnmmm17:07
tsdgeosbut i'm not the optimal guy17:07
kgunni think ricmm is in the process of moving17:07
kgunnmzanetti: ^ maybe ?17:08
kgunngerry's kinda busy17:08
greybackhe is. But lemme take a glance17:08
kgunntsdgeos: i know you're working on the other crasher...17:08
kgunnif you're done, then yes pls17:08
tsdgeosi am working in the other crasher yes17:08
greybackthat crash /always/ happens when unity8 rejects the client's connection17:08
greybackso the output of unity8.log would help a lot17:09
greybackusually due to using the --desktop_file_hint badly17:09
greybackkgunn: commented on bug, not with answers, but with where someone should look17:12
=== gatox_lunch is now known as gatox
mzanettistrange... the link in that bug shows a 100% pass rate17:19
mzanettiah, there's the crash17:19
mzanettikgunn: greyback: yep, this is in the log: ApplicationManager REJECTED connection from app with pid 4983 as no desktop_file_hint specified17:21
kgunnmzanetti: so dialer app is not special...gotta have a hint just like everyone?17:22
mzanettiI think so, yes17:23
greybackyou should check how the apps are being launched - via command line with --desktop_file_hint, or via upstart.17:23
greybackwell upstart-app-launch17:23
josharensonkgunn: I'm in contact with fginther re: ci tests, but he is currently in a meeting. Do you know where the tests live so I can start poking around?17:42
kgunnmmm17:44
kgunnmzanetti: if josharenson wanted to modify jenkins ci job to add a whole new test (e.g. glmark2 on mir)...where would he look ?17:45
mzanettikgunn: hmm. I could definitely help him get started. I think we should talk to the ci guys first, to ask for permission etc17:47
mzanettifginther: hello :)17:47
mzanettifginther: are you ok with us creating some jenkins jobs to try in getting set up something new? And me giving permissions to josharenson to do that?17:48
mzanettijosharenson: your vpn connection and everything set up already?17:49
fginthermzanetti, josharenson, this all depends on what your trying to do17:50
josharensonI don't think I have vpn...17:50
mzanettiindeed... josharenson: what would the requirements be for such a job? any special packages needed or hardware requirements?17:51
kgunnfginther: mzanetti josharenson... so this is at my request :) basically adding glmark217:51
josharenson^^17:51
kgunnwhich is a seperate project to be part of the ci run17:51
kgunnfor mir17:51
fgintherI can setup you up with a sandbox job to get something started if you need to experiment, but if you already have a test, there is no need for that17:51
fgintherkgunn, glmark on touch? on desktop?17:52
josharensonfginther: I need to experiment. I am new here and have never seen the tests.17:52
kgunnfginther: glmark2 on touch/armhf17:52
kgunnultimately we'd like to get a report out as well....in order to track trend17:52
kgunnyou may remember the qa report dashboard that robotfuel put together17:53
kgunnthen we hope to expand that into more ui tests that will measure perf as well17:53
kgunnso baby step 1 out of about 6 here17:53
fgintherkgunn, ok so this is part of a big picture :-)17:53
fgintherbut it's good to know where this needs to go17:54
kgunnyeah step1...add glmark2 as a test for arm mir ci on mir/devel17:54
kgunnstep2 reporting17:54
fgintherjosharenson, do you have a phone device to test on?17:54
josharensonyes17:54
kgunnstep3 (semi-optional) - create a warning on jenkins....(in addition to failure)17:54
josharensonfginther: I have nex7 flashed w/ ubuntu currently17:55
fgintherjosharenson, that'll work.17:55
fgintherjosharenson, we have the ability to add tests to mir ci as is being requested. the first step is to get the test into something that can be provisioned and executed in a somewhat generic environment17:56
fgintherjosharenson, i.e. if all that is needed is to add some packages and execute a single command, we'll be in good shape to add this17:57
josharensonfginther: how about parsing the results from the execution?17:58
mzanettijosharenson: you need to produce a certain xml file18:00
mzanettican't reacall the name. the same as autopilot or qmltestrunner produce18:00
mzanettiother than that, you can collect whatever files you want18:01
fgintherjosharenson, that's less straight forward. From what I know of glmark, these results don't really fit th junitxml format18:01
fginthercorrect, we can collect any artifacts when the job runs. The post processing could be done elsewhere (which is what the dashboard does)18:01
fginthergetting the performance regression feedback kgunn mentions in step 3 isn't something we've implemented yet, but are working on18:02
kgunnfginther: ack...i'll live with whatever....i just want to get this in place :)18:03
mzanettijosharenson: you can browse other jenkins jobs to see what they produce, and to learn how jenkins works (unless you already know that)18:03
mzanettiI'll send you some links18:03
josharensonI'll run the benchmark and see if it leaves behind any parsable artifacts18:04
josharensonI'm sure there is _someting_18:04
fgintherjosharenson, kgunn, there's already a bug this https://bugs.launchpad.net/ubuntu-ci-services-itself/+bug/125293318:05
ubot5Ubuntu bug 1252933 in Ubuntu CI Services "Mir performance testing as a merge requirement" [Undecided,New]18:05
Cimikgunn, the two relevant branches for the welcome wizard are here, but they might need more refinements with the review: lp:~unity-team/ubuntu-system-settings/wizard.wifi and lp:~cimi/ubuntu-system-settings/wizard.privacy18:06
Cimias well as AP test18:06
fgintherit was originally from an email from thomi. it's a broader focus than what you're asking for here, so we can probably chunk this into smaller pieces18:06
Cimi(and I'm waiting for the build for the phone)18:07
kgunnCimi: thanks for the ap test!18:08
Cimikgunn, HAVEN'T DONE YET :p18:08
Cimiops caps lock18:08
Cimiseb128, in the diagnostics's PageComponent, there's this /* Confirm the setting stuck and reflect it in the UI. */18:57
Cimiseb128, what's that?18:58
kgunnmterry: gonna add this mp for the flicker bug19:41
kgunnhttps://wiki.canonical.com/InformationInfrastructure/IS/VPN19:41
kgunnwhoops19:41
kgunnor that19:41
kgunnhttps://code.launchpad.net/~andreas-pokorny/mir/no-initial-display-configuration-sent-to-hosting-server/+merge/21312619:41
mterrykgunn, ok nice19:44
kgunnmterry: also...they think osk bug is fixed on tip of mir-devel...so we should get that on rebuild19:44
mterrykgunn, I'm investigating why notifications in the greeter appear too tall  :-/  may have a unity8 rebuild request when I figure that out19:44
seb128Cimi, dunno, ev wrote that setting page, ask him for details19:54
mzanettikgunn: https://bugs.launchpad.net/ubuntu/+source/qtubuntu/+bug/1297900/comments/419:55
ubot5Ubuntu bug 1297900 in qtubuntu (Ubuntu) "crash QUbuntu: Could not create application instance" [Undecided,Confirmed]19:55
* mzanetti EOD19:56
vini_hi19:57
vini_i get an error of not enough disk space on /boot while trying to run system-update19:57
seb128Saviq, ^ not sure if the issue mzanetti just listed is https://errors.ubuntu.com/problem/85c403c4a05cd32a48a73b226340850faa45e785 but it could be19:58
vini_sudo apt-get clean doesn't help, any other soultion?19:59
mzanettivini_: check how much free disk space there is and what is occupying it20:00
mzanettivini_: this might help: http://askubuntu.com/questions/2793/how-do-i-remove-or-hide-old-kernel-versions-to-clean-up-the-boot-menu-or-free-sp20:01
Cimiseb128, that code is quite weird20:01
Cimiseb128, who approved?20:01
seb128Cimi, Laney iirc, but he's eod for today20:02
Cimiseb128, ok20:02
seb128Cimi, if you see issues feel free to do changes/send patches, ev has only been sending that merge request and moved to other things then20:03
Saviqseb128, I think it'd say SIGABRT at least20:03
Cimiseb128, might do20:03
Saviqseb128, as that above would be a raise() + abort()20:03
Cimiseb128, this is the code, you might agree with me http://paste.ubuntu.com/7164686/20:03
Saviqseb128, and there's completely nothing in the trace20:03
Saviqah waity20:03
Cimiseb128, first sets a variable, then check for it to be different20:04
seb128Saviq, it does?20:04
Saviqseb128, yeah, it's a raise() + abort()20:04
Saviqseb128, so yeah, it could be that, we'd have to see stderr to know for sure20:04
=== jhodapp is now known as jhodapp|bbl
seb128Cimi, I don't understand the logic of that code, so I'm not sure why it's this way ... there could be a reason or it could be buggy, better to check with ev tomorrow20:05
CimiI will20:05
seb128Saviq, right, in any case we need to add some hook so qmlscene issues log extra info20:06
Saviqseb128, yeah20:06
Saviqmterry, hmm indicator-sound segfault :/21:28
mterrySaviq, :(  during build?21:29
Saviqmterry, yeah, tests21:29
Saviqhttps://launchpad.net/~ci-train-ppa-service/+archive/landing-004/+sourcepub/4046039/+listing-archive-extra21:29
mterrySaviq, I think ted_ has a branch for that.  let me see21:29
mterrySaviq, lp:~ted/indicator-sound/test-failures21:29
mterrySaviq, while there, maybe add lp:~ted/indicator-sound/phone-greeter-mute21:29
ted_They're dependent on each other :-)21:30
mterryted_, ah yes.  lp:~ted/indicator-sound/phone-greeter-mute should be sufficient21:30
Saviqmterry, kicked21:31
mterrythanks!21:32
=== jhodapp|bbl is now known as jhodapp
=== salem_ is now known as _salem
=== jhodapp is now known as jhodapp|afk
=== jhodapp|afk is now known as jhodapp
=== jhodapp is now known as jhodapp|afk
=== mhall119 is now known as mhall119|offline

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