josharenson | Does anyone know where indicator-sound writes logs, and why changing volume on mako is so laggy? | 00:16 |
---|---|---|
tsdgeos | i'm getting a black screen on unity8-desktop-mir-session | 09:00 |
tsdgeos | any idea why? | 09:00 |
tsdgeos | unity8 doesn't seem to be running, that'd be a reason i guess :D | 09:01 |
tsdgeos | seb128: ↑ ? | 09:02 |
tsdgeos | greyback: want me to top approve https://code.launchpad.net/~mir-team/qtubuntu/port-to-mirclient/+merge/245164 or you want to give it a go too? | 09:02 |
greyback | tsdgeos: feel free to top approve | 09:03 |
seb128 | tsdgeos, didn't see that bug yet, do you have anything useful in .cache/upstart? | 09:03 |
tsdgeos | that's weird | 09:03 |
tsdgeos | seems like tried to use X11 instead of mir | 09:04 |
tsdgeos | log says | 09:04 |
tsdgeos | QXcbConnection: Could not connect to display | 09:04 |
seb128 | maybe some desktop backend missing? | 09:04 |
tsdgeos | seb128: don't know, i just installed unity8-desktop-session-mir and hoped it'd work :D | 09:09 |
seb128 | tsdgeos, it should, not sure what's going on | 09:10 |
tsdgeos | seb128: it can be started from regulra X11 lightdm, right? | 09:10 |
seb128 | yes | 09:10 |
tsdgeos | let me try rebooting then | 09:13 |
seb128 | greyback, hey | 09:25 |
seb128 | greyback, willcooke hit the same battery icon/unity8 stucked issue yesterday | 09:25 |
willcooke | running away, eh? | 09:26 |
seb128 | lol | 09:27 |
seb128 | willcooke, hey | 09:27 |
willcooke | o/ | 09:27 |
seb128 | willcooke, did you have the same issue on the tablet? or was that something else? | 09:27 |
willcooke | seb128, indeed - using silo0 the greeter, lockscreen, whatever it's called, | 09:28 |
willcooke | just kinda got stuck | 09:28 |
willcooke | I could see half the pin code thing | 09:28 |
willcooke | but couldnt type on it | 09:28 |
willcooke | your fix sorted it | 09:28 |
seb128 | so that was on the n7 as well | 09:28 |
willcooke | yah | 09:28 |
seb128 | thanks for confirming | 09:28 |
seb128 | Saviq, ^ | 09:28 |
willcooke | Just to confirm, I am running a non-standard set up - but I think the problems here are the same root cause | 09:29 |
seb128 | right | 09:30 |
seb128 | it just means the issue is not desktop specific | 09:30 |
seb128 | but it had no reason to be | 09:30 |
seb128 | greyback, hey | 09:51 |
greyback | seb128: hi! | 09:51 |
seb128 | greyback, did you see what I was saying before? | 09:52 |
greyback | you got the filename | 09:52 |
seb128 | yeah | 09:52 |
greyback | the battery icon | 09:52 |
seb128 | it's not specific to it though | 09:52 |
greyback | no, hmm | 09:52 |
seb128 | this morning the battery is less charged | 09:52 |
seb128 | and it has the same issue on battery-060 | 09:52 |
greyback | what device do you see this with? | 09:52 |
seb128 | greyback, willcooke is getting that bug on his n7 tablet and on his test desktop as well | 09:52 |
tsdgeos | so | 09:53 |
greyback | ok so | 09:53 |
tsdgeos | i got it to work | 09:53 |
seb128 | insipiron test laptop | 09:53 |
tsdgeos | and i know why it's broken | 09:53 |
seb128 | tsdgeos, oh, tell us! | 09:53 |
tsdgeos | just not sure why only there | 09:53 |
* greyback perks ears | 09:53 | |
tsdgeos | seb128: basically the sdk is a bad icon provide | 09:53 |
tsdgeos | it gets asked for a icon of size 24 | 09:53 |
tsdgeos | and returns a pixmap of 30, 24 | 09:53 |
tsdgeos | then qml goes, oh wait you gave me something of 30 | 09:54 |
tsdgeos | and asks for a size of 30 | 09:54 |
tsdgeos | and gets 34, 30 | 09:54 |
tsdgeos | ad infinitum | 09:54 |
greyback | eek | 09:54 |
tsdgeos | just that it seems in some cases | 09:54 |
tsdgeos | it detects a binding loop | 09:54 |
tsdgeos | so this is if i run it from X11 | 09:54 |
tsdgeos | http://paste.ubuntu.com/10288127/ | 09:54 |
seb128 | tsdgeos, are you talking about my bug now, not about your x11 bug? | 09:55 |
tsdgeos | it detects a loop and stops | 09:55 |
tsdgeos | seb128: yes | 09:55 |
seb128 | k | 09:55 |
seb128 | great! | 09:55 |
seb128 | https://bugs.launchpad.net/ubuntu/+source/unity8/+bug/1422835 if you want a bug to comment on | 09:56 |
ubot5 | Launchpad bug 1422835 in unity8 (Ubuntu) "Unity8 stucks in icon loading code" [Undecided,New] | 09:56 |
tsdgeos | this is on the unity8 session http://paste.ubuntu.com/10288142/ | 09:56 |
tsdgeos | and ad infinitum | 09:56 |
tsdgeos | at some point i was getting | 09:56 |
seb128 | right | 09:56 |
seb128 | see my bt on ^ | 09:56 |
tsdgeos | request "battery-080-charging" QSize(12769, 8268) | 09:56 |
greyback | aha | 09:56 |
greyback | hence the massive memfill | 09:57 |
tsdgeos | the question is why | 09:57 |
tsdgeos | request "battery-080-charging" QSize(37, 24) | 09:57 |
tsdgeos | LoadIcon 37 QSize(57, 37) | 09:57 |
seb128 | is that a regression with new qt? | 09:57 |
seb128 | that started after the qt5.4 landing for me | 09:57 |
greyback | tsdgeos: what is the "37" | 09:57 |
greyback | seb128: I was seeing it before in 5.3 | 09:57 |
tsdgeos | width, height | 09:57 |
greyback | but maybe less often | 09:58 |
tsdgeos | now since that is on the correct aspect ration, i'd expect it to return 37, 24 again, not 57, 37 | 09:58 |
tsdgeos | but i guess the sdk icon loader has a bug somewhere | 09:58 |
greyback | tsdgeos: the requested width/height? Am guess the qsize is the returned | 09:58 |
willcooke | seb128, I can't get Bluetooth to turn back on with the N7 - can I do it from a command line? | 10:04 |
tsdgeos | give me a sec i think i have a patch | 10:06 |
tsdgeos | will be back! | 10:06 |
seb128 | willcooke, what happens when you try to turn it on? | 10:07 |
willcooke | seb128, in the indicator the slider just turns off again, in the settings app it tuns on and stays on, but nothing happens, no scanning, no connecting, etc - and when I quit the app and go back in it's back to off again | 10:08 |
seb128 | willcooke, not sure, try to sudo restart bluetooth maybe? | 10:14 |
willcooke | seb128, heh - that just seems to get stuck doing nothing. | 10:17 |
willcooke | I'll turn it off and back on again | 10:17 |
seb128 | willcooke, is bluetoothd running? | 10:17 |
willcooke | appears not | 10:17 |
willcooke | oh | 10:18 |
willcooke | well | 10:18 |
willcooke | service --status-all says yes, ps aux says no | 10:18 |
willcooke | hrm | 10:21 |
willcooke | Running bluetoothd manually works | 10:21 |
mzanetti | greyback: did you already install silo0 today? | 10:21 |
mzanetti | greyback: it's working just fine for me. can't repro kevin's issues | 10:22 |
greyback | mzanetti: just testing it now | 10:22 |
greyback | mzanetti: I share kevin's problem | 10:22 |
seb128 | willcooke, it probably segfaulted or something and the init job doesn't restart it in those cases, I sent a patch upstream recently for that | 10:29 |
willcooke | thanks seb128 | 10:29 |
tsdgeos | seb128: if you have time can you try https://code.launchpad.net/~aacid/ubuntu-ui-toolkit/nonsquareicons/+merge/250110 ? | 10:49 |
=== vrruiz_ is now known as rvr | ||
=== dpm_ is now known as dpm | ||
=== marcusto_ is now known as marcustomlinson | ||
tsdgeos | larsu: you may be interested in https://code.launchpad.net/~aacid/ubuntu-ui-toolkit/nonsquareicons/+merge/250110 ? | 11:45 |
larsu | tsdgeos: it's really hard to see what you actually changed :/ | 11:50 |
* larsu looks at commit diffs | 11:50 | |
tsdgeos | larsu: yeah look at the first one | 11:50 |
larsu | why did you move the class into the header? Nothing else needs it... | 11:50 |
tsdgeos | the second one is actually adding the test (and a small fixlet for the png branch) | 11:50 |
tsdgeos | larsu: the test does | 11:50 |
tsdgeos | though i'm not very happy on how the test looks tbh | 11:51 |
larsu | oh, you're testing the class itself now? | 11:51 |
larsu | weird | 11:51 |
tsdgeos | larsu: ? | 11:51 |
tsdgeos | since results depend on the suru theme changing or not | 11:51 |
larsu | tsdgeos: I would have just tested the provider | 11:52 |
larsu | but fair enough | 11:52 |
tsdgeos | larsu: yes i'm testing only the provider | 11:52 |
tsdgeos | except for the small bit that i want to make sure i'm testing a svg and a png | 11:52 |
tsdgeos | and thus i need access to the functions that return the filename for a given icon | 11:52 |
tsdgeos | which are part of the theme | 11:52 |
tsdgeos | i'm open to a suggestion on how to better test it | 11:53 |
tsdgeos | couldn't find a different way | 11:53 |
larsu | hm, we could make a test theme with differently named svg and png | 11:54 |
tsdgeos | yeah | 11:54 |
larsu | would also get rid of the problem that suru might change | 11:54 |
tsdgeos | but you still do | 11:54 |
larsu | but might not be worth it... | 11:54 |
tsdgeos | theme = IconTheme::get("suru"); | 11:54 |
tsdgeos | in the constructor | 11:54 |
tsdgeos | though maybe i can use a default param | 11:54 |
tsdgeos | to fix that | 11:54 |
tsdgeos | let me see | 11:54 |
larsu | tsdgeos: you could also call the test icon theme "suru" and run the test in an env where it only finds that | 11:55 |
tsdgeos | right | 11:55 |
larsu | I wonder why we hardcoded "suru" back then | 11:55 |
larsu | I guess there's no way to fetch the icon theme name? | 11:56 |
=== MacSlow is now known as MacSlow|lunch | ||
dandrader | Saviq, ping | 12:12 |
dandrader | was the last unity8 release a "special", manual one? The commits are all under a merge commit (rev 1616) instead of being merged individually | 12:17 |
tsdgeos | larsu: looks much easier to read now https://code.launchpad.net/~aacid/ubuntu-ui-toolkit/nonsquareicons/+merge/250110 :) | 12:25 |
larsu | indeed, thanks | 12:26 |
tsdgeos | dandrader: which branch? | 12:26 |
larsu | I'll have a look in a bit | 12:26 |
dandrader | tsdgeos, lp:unity8 | 12:26 |
tsdgeos | dandrader: right | 12:27 |
tsdgeos | maybe the train has changed? | 12:27 |
tsdgeos | robru: ↑ ? | 12:27 |
tsdgeos | dandrader: ah you're back | 12:28 |
dandrader | yes :) | 12:28 |
tsdgeos | dandrader: https://code.launchpad.net/~aacid/unity8/qmluitests54/+merge/250021 is mostly for you | 12:28 |
tsdgeos | the tests need some coercing to pass on touch related stuff due to changes in aggregation of touch events | 12:30 |
tsdgeos | i couldn't find any impact in real world | 12:30 |
tsdgeos | but yuo should have a look and in case anything touch related shows up in vivid-only to know it can be related to that Qt 5.4 change | 12:30 |
=== alan_g is now known as alan_g|lunch | ||
dandrader | tsdgeos, checking it | 12:46 |
Trevinho | dpm: happy BD! :) | 13:01 |
dpm | thanks Trevinho! :-) | 13:10 |
=== dandrader_ is now known as dandrader|afk | ||
=== alan_g|lunch is now known as alan_g | ||
=== dandrader|afk is now known as dandrader | ||
larsu | tsdgeos: can't test your changes, uitk fails to build (test failure in localizedApp) | 13:42 |
tsdgeos | larsu: fails to build? | 13:43 |
tsdgeos | built fine here | 13:43 |
tsdgeos | which error do you have? | 13:43 |
larsu | well, test failure | 13:43 |
tsdgeos | ah | 13:43 |
larsu | msgfmt: error while opening "localizedApp/share/locale/en/LC_MESSAGES/localizedApp.mo" for writing: No such file or directory | 13:43 |
larsu | but apparently tests are run by default on `make` ?! | 13:43 |
tsdgeos | no idea | 13:43 |
larsu | :/ | 13:50 |
tsdgeos | i did have no problems running make | 13:52 |
tsdgeos | are you building the deb? | 13:52 |
tsdgeos | dandrader: updated https://code.launchpad.net/~aacid/unity8/qmluitests54/+merge/250021 | 13:52 |
larsu | tsdgeos: no, I'm running make | 13:52 |
tsdgeos | weird | 13:53 |
=== MacSlow|lunch is now known as MacSlow | ||
larsu | tsdgeos: I can't reproduce the infinite growing problem | 14:12 |
tsdgeos | larsu: where? in the test? | 14:12 |
larsu | tsdgeos: no, with a sample script | 14:12 |
tsdgeos | larsu: well there's a bug, you can try unity8 | 14:13 |
tsdgeos | on the desktop or on the tablet | 14:13 |
larsu | is there a way to make the bug show up in a small example? | 14:13 |
larsu | I guess we'll want to include that as a test as well | 14:14 |
tsdgeos | larsu: well ins't the current test enough? why would asking for a icon of 24x16 give me a pixmap of 37x24? | 14:14 |
tsdgeos | which is what the current code does | 14:15 |
seb128 | tsdgeos, sorry, took me a while, your fix restore my desktop test machine | 14:15 |
seb128 | tsdgeos, commented on the mp saying so | 14:16 |
larsu | tsdgeos: you didn't say that in the MR or the bug... | 14:16 |
tsdgeos | larsu: i didn't say what? | 14:17 |
larsu | tsdgeos: why would asking for a icon of 24x16 give me a pixmap of 37x24? | 14:17 |
tsdgeos | larsu: https://code.launchpad.net/~aacid/ubuntu-ui-toolkit/nonsquareicons/+merge/250110/comments/619927 ? | 14:18 |
seb128 | larsu, he said that on the MR | 14:18 |
tsdgeos | larsu: also the commit message | 14:18 |
tsdgeos | Without this patch we could end up in loops like | 14:18 |
tsdgeos | requested icon "battery-080-charging" QSize(24, 24) | 14:18 |
tsdgeos | returned QSize(37, 24) | 14:18 |
larsu | tsdgeos: meh, write it in the description or the bug next time | 14:20 |
tsdgeos | larsu: as said, it's also written in the commit message right on the top | 14:20 |
larsu | tsdgeos: and like I said, I cannot reproduce that | 14:21 |
seb128 | I wonder why only some people get the issue :-/ | 14:21 |
larsu | a commit message saying "without this patch, X doesn't work" is not very helpful.. | 14:21 |
larsu | seb128: because we haven't found the real issue yet | 14:22 |
tsdgeos | larsu: we haven't foudn the reasl issue? | 14:22 |
tsdgeos | you kidding? | 14:22 |
larsu | no, of course not | 14:22 |
tsdgeos | larsu: so you're really arguing the bug is somewhere else and not there? | 14:22 |
larsu | tsdgeos: no. I'm saying I don't know what the issue is yet | 14:23 |
tsdgeos | the issue is | 14:23 |
tsdgeos | that function is returning the wrong pixmap size | 14:23 |
tsdgeos | and there's a test to prove it | 14:23 |
tsdgeos | seb128: it only happens for some people because the other people are getting png icons, that don't have this bug | 14:24 |
seb128 | tsdgeos, why isn't everyone getting the same icons? we use the same environment and icon theme... | 14:25 |
tsdgeos | seb128: e.g. i have to unplug my battery-powered-mouse, otherwise the icon that gets shown is the one for battery+mouse and that's a png one that doesn't show the bug | 14:25 |
seb128 | oh ok | 14:25 |
tsdgeos | larsu: honestly not sure what more you want besides this test | 14:25 |
larsu | tsdgeos: I'd love to be able to run it for a start | 14:26 |
tsdgeos | larsu: are you on vivid? | 14:26 |
larsu | yes | 14:26 |
larsu | running tst_iconProvider manually doesn't work either | 14:27 |
tsdgeos | can you paste the erorr you get? | 14:27 |
tsdgeos | larsu: make check | 14:27 |
larsu | tsdgeos: another test is failing | 14:28 |
tsdgeos | larsu: make check in the tst_iconProvider directoyu | 14:28 |
larsu | actually, I don't even know. `make` is failing | 14:28 |
larsu | make check in the directory complains about a .pro file missing | 14:28 |
tsdgeos | ok, so maybe i forgot to commit the pro file :) | 14:29 |
tsdgeos | no it's there | 14:29 |
tsdgeos | can you please paste the error? | 14:29 |
larsu | ah, that one's on me, wrong branch | 14:29 |
dandrader | mzanetti, there might be some regression in trunk. the launcher no longer hides itself automatically after a couple of seconds being idle | 14:43 |
dandrader | mzanetti, have you noticed it? | 14:43 |
greyback | dandrader: would your mouse cursor be over it by any chance? | 14:59 |
mzanetti | dandrader: ? | 14:59 |
mzanetti | dandrader: how do you reveal it? | 14:59 |
mzanetti | by a left edge drag or via mouse? | 14:59 |
dandrader | mzanetti, left edge drag on the device | 15:02 |
dandrader | mzanetti, dragging it with the mouse in "make tryLauncher" | 15:03 |
dandrader | (but note that mouse sends touch events in tryFoo runs) | 15:03 |
dandrader | greyback, no | 15:03 |
greyback | dandrader: was worth a shot, it happened to me | 15:04 |
dandrader | mzanetti, so you're not getting this behavior? | 15:04 |
mzanetti | dandrader: it's not supposed to hide any more | 15:04 |
mzanetti | and doesn't do so since december or so | 15:04 |
dandrader | mzanetti, hmm, why do we still have this dismissTimer in Launcher.qml for? | 15:04 |
mzanetti | because it hides when reviealing by mouse hover | 15:05 |
tsdgeos | in an unmerged branch | 15:05 |
mzanetti | isn't that merged too now? | 15:05 |
tsdgeos | https://code.launchpad.net/~mzanetti/unity8/reveal-launcher-with-mouse-hover/+merge/248913 | 15:05 |
mzanetti | ah ok. | 15:05 |
mzanetti | then no, not merged yet | 15:05 |
tsdgeos | actaully now that i realized i can run unity8 on the desktop just fine | 15:05 |
tsdgeos | i'll review it in a bit | 15:06 |
mzanetti | dandrader: anyhow, I didn't want to drop the timer back then because I knew I need it again for the mouse hover thing | 15:06 |
mzanetti | which is happening in that branch albert just posted | 15:06 |
dandrader | ok | 15:06 |
=== dandrader is now known as dandrader|lunch | ||
josharenson | Saviq: lp:indicator-sound is the correct project for unity8, right? When I make changes to it, I am not seeing them propagate. | 15:51 |
larsu | tsdgeos: [sorry, was on the phone] reviewed your patch. thanks | 15:51 |
larsu | tsdgeos: I'm not sure why it fixes your problem, but glad it dos | 15:51 |
larsu | *does | 15:51 |
tsdgeos | mzanetti: so i'm trying your hover patch and i can't get it to show the launcher | 15:53 |
mzanetti | tsdgeos: how are you trying? | 15:54 |
mzanetti | hmm... could be we only get it on silo0 yet | 15:55 |
tsdgeos | mzanetti: i got your branch, installed it in my pc | 15:55 |
mzanetti | tsdgeos: so you're running on Mir | 15:56 |
tsdgeos | mzanetti: yes | 15:56 |
tsdgeos | well i guess | 15:56 |
mzanetti | for that you'd need to have qtmir updated to use mirevent2 I think, https://code.launchpad.net/~mir-team/qtmir/port-to-msh-shell-and-mirevent2/+merge/248433 | 15:56 |
mzanetti | otherwise it won't give us mouse events yet | 15:56 |
mzanetti | you can try on X11 though | 15:56 |
mzanetti | by just running ./builddir/src/unity8 | 15:57 |
mzanetti | don't use make tryLauncher, because the qmlscene we use in "try" converts all the mouse events to touch events too | 15:57 |
mzanetti | so you can test on: a device with silo0 installed | 15:57 |
mzanetti | or X11 running the unity8 binary without wrappers (run.sh etc) | 15:58 |
mzanetti | actually, you could also install silo0 on the pc, if it's a VM and not a problem to break/recover | 15:58 |
tsdgeos | mzanetti: so one we land this ./run and tryLauncher will be broken "forever"? | 16:04 |
mzanetti | tsdgeos: no, why? | 16:05 |
mzanetti | tsdgeos: it just won't work with revealing by mouse hover | 16:05 |
mzanetti | tsdgeos: but the mouse is threated like a finger in those cases | 16:05 |
mzanetti | so you can still drag it from the left edge | 16:05 |
tsdgeos | ok | 16:08 |
tsdgeos | mzanetti: the dismisstimer seems to not have effect | 16:15 |
tsdgeos | i moved it to 5000 and the thing still hides immediately | 16:15 |
mzanetti | hmm | 16:15 |
mzanetti | that doesn't sound right | 16:15 |
tsdgeos | MacSlow: how hard is to get a test for https://code.launchpad.net/~macslow/unity8/unbreak-notification-positioning-fix-1422711 ? | 16:25 |
MacSlow | tsdgeos, depends how hard it is to fake a unityMenuModel attached as a hint... I've not tried it yet as the shell-rotation-AP test still keeps me very busy | 16:29 |
tsdgeos | MacSlow: added some comments to the MR | 16:39 |
MacSlow | tsdgeos, done | 17:06 |
=== dandrader|lunch is now known as dandrader | ||
=== alan_g is now known as alan_g|EOD | ||
=== boiko_ is now known as boiko | ||
=== dandrader is now known as dandrader|afk | ||
josharenson | so, still can't figure out why my changes don't show up in indicator-sound... is there a special install process/variable to set? | 19:48 |
josharenson | tedg: ^^^ ? | 20:18 |
tedg | josharenson, I'm confused, which changes? | 20:21 |
josharenson | tedg: lets say i want to add a print statement... I add it, do make/make install/ and never see it in the logs | 20:22 |
tedg | josharenson, make install may not put it in the correct place. You probably want "bzr bd" which will build a deb for you. | 20:22 |
tedg | Also make sure to restart | 20:23 |
josharenson | tedg: ok, do you know if its possible to X-compile? | 20:23 |
* josharenson tries native in the meantime | 20:23 | |
tedg | josharenson, Should be, but I haven't tried. | 20:23 |
josharenson | tedg: thanks, ill let you know if it still doesn't work | 20:24 |
=== dandrader|afk is now known as dandrader |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!