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

=== greyback__ is now known as greyback
CimiSaviq, tsdgeos what's the difference between openScope and gotoScope?09:45
SaviqCimi, fav vs. non-fav09:46
Cimiok09:46
Saviqor the other way 'round09:46
Cimiyeah other way09:46
tsdgeosyeah09:46
tsdgeosopenscope is non fav09:46
tsdgeosgoto scope is09:46
Cimiwhy don't we have a single call for them?09:46
SaviqCimi, legacy, really09:46
SaviqCimi, and the fact that the dash doesn't directly know whether it's fav or not, so the plugin gives us a hand09:47
=== timon_ is now known as t-mon
CimiSaviq, to fix run/run_on_device, shall we restart also unity8-dash or we can pass a variable to unity8 upstart with the bin for unity8-dash?10:07
SaviqCimi, no, unity8-dash would need to be restarted, but it didn't work straight away when I tried it10:07
Saviqtsdgeos, network trouble?10:08
tsdgeosno, was playing with appmenu-qt5 in my irc client that just got upgraded to qt510:08
tsdgeosi have two menus now :D10:08
Saviq:)10:09
CimiSaviq, it does not, because we specify in run.sh just the binary for unity8, not for unity8-dash10:11
CimiSaviq, so unity8 job then restarts the dash job which is the system-wide one10:11
CimiSaviq, we have two options: restart in run.sh the local dash job10:11
SaviqCimi, thanks for explaining, but guess what, I thought of that, and I did restart the dash with the right binary, but it didn't work locally IIRC10:12
CimiSaviq, or make the unity8 job look for the local dash build, maybe with a variable or so10:12
CimiSaviq, it worked here...10:12
SaviqCimi, the unity8 job knows nothing about the dash10:12
SaviqCimi, sure, if you solved this, MP it, we'll be sure to review10:12
CimiSaviq, yesterday when I tried with restart unity8-dash BINARY=/path/to/your/unity8-dash10:12
SaviqCimi, yes, I know that works10:12
SaviqCimi, but when I put that in run.sh, it did not work well (either on device or locally, don't remember)10:13
CimiSaviq, so my question is how can we tweak unity8 job to run that?10:13
SaviqCimi, we don't10:13
SaviqCimi, the unity8 job knows nothing about the dash10:13
SaviqCimi, we need to tweak the run scripts to restart unity8-dash with the correct binary10:13
CimiSaviq, well thje dash job restarts on unity810:14
CimiSaviq, if we could set a variable pointing to the right binary...10:14
SaviqCimi, it doesn't restart, it starts and stops10:14
SaviqCimi, that's exactly what "restart unity8-dash BINARY=..." does10:15
SaviqCimi, but you can't do that in the unity8 job10:15
SaviqCimi, because it has no idea about the dash10:15
SaviqCimi, you need that in the run scripts, that will start the correct unity8 and then restart dash with the correct binary10:15
CimiSaviq, I meant adding something to the unity8 dash job, check if a variable is set ($UNITY8_DASH_BINARY), otherwise run system wide10:16
SaviqCimi, you'd have to set a global var like that, not sure we want that10:17
CimiSaviq, we set a variable in run.sh, and we unset the variable at stop of unity810:17
CimiSaviq, so the variable is always unset, apart when we run run.sh10:17
SaviqCimi, sure, that could work, not much different than going "restart unity8-dash BINARY=..." in the same run.sh script10:18
Saviqonly thing is that the system-wide dash would be started first, but that's not really a problem IMO10:18
SaviqCimi, if we did that, we should make the unity8 job itself respect that env var10:19
SaviqCimi, but I'm worried it would easily get stale on Ctrl+C and such10:19
Saviqand then you'd get stuck with the local build until you reset that var manually10:21
CimiSaviq, problem is that when we start unity8 in run.sh, this will start the dash10:21
SaviqCimi, yes, and then we can restart dash with the right binary10:21
CimiSaviq, with a sleep ?10:22
SaviqCimi, no, straight away10:22
SaviqCimi,10:22
Saviqunity8-dash starts as soon as unity8 started10:22
Cimiyeah10:22
Saviqwhich initctl start unity8 exits even later than that10:22
Saviq-which10:22
Cimiah ok, cool10:22
Cimithought was running in parallel10:23
CimiSaviq, doing ctrl-c will not run the stop script of unity8 job?10:24
Cimior post-stop10:24
SaviqCimi, it will, assuming you actually deliver the ctrl+c10:24
SaviqCimi, if you just unplug USB, nothing will happen10:25
SaviqCimi, but recovering from that is a simple "restart unity8" away10:25
Saviqwhereas with the global env you'd need to unset that env first10:25
Cimiwe could unset the variable in unity8-dash start10:26
Cimiafter the binary ran10:27
SaviqCimi, that's putting more and more test considerations in a real-life job, wouldn't like that10:28
CimiSaviq, ok, so let's just restart in run.sh10:28
SaviqCimi, I'd rather us put an .override in ~/.config/upstart and remove "itself" in post-stop10:28
Saviqthat could actually work and could mean we don't need to recommend installing unity8 on desktop, and make sure you always use the new job10:29
mzanettiSaviq: wouldn't it make more sense to make the dash not start at all with ./run.sh ?10:33
mzanettiyou can just run ./builddir/src/Dash/unity8-dash to run the dash alone10:34
Saviqmzanetti, that depends, it's used on device as well10:34
mzanettioh ok... I see the value for the device case10:34
Saviqmzanetti, run_on_device calls run.sh10:34
mzanettibut for local runs I think it's not useful10:34
Saviqdunno, as long as it runs the right one, I'd be happy10:35
Saviqespecially since you have to remember -mousetouch to use the bottom edge etc.10:35
mzanettiSaviq: I would rather rather have a run-dash.sh10:35
mzanettiif you only need the shell, it's quite annoying that the dash is always popping up too10:35
Saviqsure, we could refactor all that10:36
Saviqand do away with .sh in the process ;P10:39
tsdgeosCimi: Saviq: can you reproduce https://launchpadlibrarian.net/194886158/bug.png ?10:54
Saviqtsdgeos, no, I wonder if it's the Spanish version maybe10:55
tsdgeosi'm on spanish krillin10:55
tsdgeoscan't either10:55
Saviqtsdgeos, do you have the "Lugares emblemátic..." category?10:56
tsdgeosyeah10:56
tsdgeosit fits just fine10:56
tsdgeoshttp://i.imgur.com/QalsvMJ.png10:57
Saviqoh that's interesting10:58
Saviqso it got elided10:59
Saviqso basically the chevron got huge, and the label is right-anchored to it10:59
Saviqso it elided10:59
Saviqnow why would the chevron get huge?10:59
Saviqif I had to guess, RowLayout is playing with us11:01
Cimisdk sdk sdk! :D11:01
tsdgeosit's weird11:01
SaviqCimi, nope, it's all our components11:01
tsdgeosbecause victor mentions it's fine and just wrong after scrolling11:02
tsdgeosand the height of the icon is set to gu(2)11:03
MacSlowtsdgeos, mzanetti, Saviq: is there any way to alter the qmltest-makefiles to run uqmlscene under gdb?11:16
mzanettiMacSlow: make gdbtestFoo11:16
MacSlowmzanetti, thx11:16
SaviqMacSlow, you can always just copy-paste the qmltestrunner line, too11:17
MacSlowok11:18
tsdgeosSaviq: how have you tried the new dash branches on rtm? i get errors if i try the branch directly11:45
tsdgeoshttp://paste.ubuntu.com/9755298/11:45
tsdgeoslike i guess the sdk is too old or whatver11:45
Saviqtsdgeos, I cherry-picked11:45
tsdgeosi see11:46
Saviqtsdgeos, and depends what you mean "new dash branches"?11:46
tsdgeosi'll try that11:46
tsdgeosproperVRangesCurrentScope and properRangesHorizontalCategories11:46
Saviqyeah, in general we can't run trunk on rtm any more, you need to cherry pick onto lp:unity8/rtm-14.0911:46
Saviqyeah those I didn't try yet11:46
tsdgeosah ok11:46
tsdgeosSaviq: i tried vivid, but rtm krillin has more interesting scopes11:47
Saviqbiab11:47
Saviqtsdgeos, yeah, definitely worth to try there11:47
dandradermzanetti, did you enable the touch logging (TouchRegistry, DirectionalDragArea and TouchGate) in your dogfooding phone?11:59
mzanettidandrader: not yet. was quite busy yesterday. wanted to do that *now*11:59
mzanettidandrader: do you perhaps have some rtm packages with that enabled?12:02
dandradermzanetti, I don't12:02
mzanettiI don't even know how to enable it...12:03
=== MacSlow is now known as MacSlow|lunch
=== dandrader is now known as dandrader|afk
=== alan_g is now known as alan_g|lunch
=== dandrader|afk is now known as dandrader
=== MacSlow|lunch is now known as MacSlow
=== alan_g|lunch is now known as alan_g
=== seb128_ is now known as seb128
=== alan_g is now known as alan_g|afk
tsdgeoswhat is this thing? https://code.launchpad.net/~gerboland/unity8/initialSurfaceGeometry/+merge/23172614:44
tsdgeos Proposed by Gerry Boland on 2014-08-21 ?14:44
tsdgeosgreyback: ↑ ¿14:44
greybacktsdgeos: yeah, it's taken a while...14:44
greybacktsdgeos: it's ready to review, that and it's children. Saviq did a review ages ago, but found an issue that AP tests would hang.14:45
greybackReason that it hung was because of the dash dbus blocking the mainloop thing14:45
greybackso now that fix landed, can now try land this!14:46
Saviqyeah, let's14:46
tsdgeosSaviq: you taking care or want me to have a look?14:52
Saviqtsdgeos, feel free, /me nursing silos still14:53
tsdgeosok14:53
kgunnmterry: so rick said he never saw the poweroff dialog, but did experience the busy/slow resume15:12
Saviqkgunn, looks like the same that mterry experienced then https://bugs.launchpad.net/ubuntu/+source/unity8/+bug/1410830/comments/215:20
ubot5Launchpad bug 1410830 in unity8 (Ubuntu) "Shutdown dialog appears on resume, after a long delay in screen turning on" [Undecided,In progress]15:20
Saviqmterry, do we even want the band-aid then?15:20
mterrySaviq, I missed kgunn's comment that you are responding to15:20
Saviq<kgunn> mterry: so rick said he never saw the poweroff dialog, but did experience the busy/slow resume15:21
mterrySaviq, I believe Rick was testing with the band-aid15:21
Saviqmterry, right, my comment was more about your experiences then15:21
mterrySaviq, Rick had previously reported being able to reproduce the shutdown dialog even with the MP that landed in rtm.15:22
mterrySaviq, so the difference now between him not reproducing seems to be the band-aid...  So I'd vote +1 for band-aid since it isn't making anything *worse* and might be helping15:22
Saviqthat is true15:22
mterrySaviq, I guess the result of my findings in comment2 you linked is that the band-aid should be attached to Rick's bug, not mine15:25
* mterry adjusts the LP connections15:25
kgunnmterry: yep, +1, i saw we get that code reviewed (just a couple of lines) and landed15:30
Cimitsdgeos, any improvements on mem or CPU usage with your dash branches? did you measure that?15:41
=== dandrader is now known as dandrader|lunch
tsdgeosCimi: there's less mem usage, but wouldn't say it's noticeable15:44
Cimisorry I dropped15:51
Cimitsdgeos, any improvements on mem or CPU usage with your dash branches? did you measure that?15:51
Saviq<tsdgeos> Cimi: there's less mem usage, but wouldn't say it's noticeable15:51
Cimiwell. less is good no?15:52
Saviqdednick, there's FTBFS with your rtm branch, could you have a look please?15:52
Cimitsdgeos, one thing I saw when playing with signals, is that we send signals to the whole list of scopes15:52
dednickSaviq: sure15:52
Cimitsdgeos, like a openScope sends stuff to all of them15:52
Cimiwe need to take that into account when we connect to those15:53
Cimiif we have like 20+ scopes all doing stuff when an event occurs15:53
sil2100j ubuntu-meeting15:54
sil2100Crap...15:55
dednickSaviq: fixed.15:55
tsdgeosCimi: hmm openScope sends signals to what?15:55
Saviqdednick, fast ;)15:57
dednickjust forgot to remove a include :)15:57
Cimitsdgeos, nevermind might be wrong on that15:58
Cimitsdgeos, seems like was some debugging leftover15:58
tsdgeosgreyback: do we still need the  QTimer::singleShot(0, this, SLOT(create())); in https://code.launchpad.net/~gerboland/unity8/initialSurfaceGeometry/+merge/231726 ?16:10
greybacktsdgeos: no, that shouldn't be needed. Will remove16:11
greybacktsdgeos: pushed16:15
tsdgeostx16:15
Saviqdednick, "modelactionrootstate.cpp: multiple new lines at end of file"...16:27
dednickSaviq: fixed16:30
Saviqdednick, thanks16:30
Cimitsdgeos, Saviq I'd like to call openScope(scope) from tst_Dash.qml to open a non-fav scope, do you know easily how I could get the argument scope?16:39
tsdgeosCimi: the fake scopes have a get scope i think16:39
Cimimmm16:40
tsdgeosyeaj16:40
Cimiyeah get scope16:40
* Cimi tries16:40
tsdgeosthat may only return fav scopes though16:41
tsdgeosCimi: getScopeFromAll16:41
=== dandrader|lunch is now known as dandrader
=== dandrader is now known as dandrader|afk
=== dandrader|afk is now known as dandrader
Saviqgreyback, testing wakelock here, krillin/vivid, anything special I should test? IIUC the camera and media issues were not visible outside of krillin/rtm?18:21
greybackSaviq: I could repro the camera issue on stock vivid18:22
greybackSaviq: the impact is pretty minor otherwise18:22
greybackkeep an eye on "sudo power-cli list" - it should list nothing 3 seconds after display off18:22
Saviqgreyback, 1.5s, got the lower timeout in that silo as well ;)18:25
greybackSaviq: ah ok18:25
Saviqok, done18:27
Saviqgreyback, btw, noticed an interesting hack in unity-scope-click18:29
Saviqgreyback, they added a moot autopkgtest to just go through building the package, which runs the usual unit tests18:29
Saviqgreyback, sounds like qtmir would be a good candidate for that18:29
greybackSaviq: I'll add it to my list ;)18:30
greybackSaviq: em, where do I see it?18:31
Saviqgreyback, https://code.launchpad.net/~saviq/qtmir/add-autopkgtest/+merge/24662518:33
greybackSaviq: oh thanks18:34
Saviqgreyback, to test: `adt-run .// --- schroot vivid-amd64-shm`18:41
Saviqvivid-amd64-shm being an available schroot18:42
Saviqnote bug #1396955 if you use byobu18:43
ubot5bug 1396955 in autopkgtest (Ubuntu) "adt-run fails if sourcing ~/.profile fails" [Medium,Triaged] https://launchpad.net/bugs/139695518:43
=== alan_g|afk is now known as alan_g
dandraderSaviq, when a process crashes a coredump is automatically generated, right? do you know where it is stored? will this whoopsie thing clean it up?19:08
=== tedg` is now known as tedg
=== irsol_ is now known as irsol
=== TheMuso` is now known as TheMuso
=== dkessel_ is now known as dkessel
=== robru_ is now known as robru

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