[00:16] <josharenson> Does anyone know where indicator-sound writes logs, and why changing volume on mako is so laggy?
[09:00] <tsdgeos> i'm getting a black screen on unity8-desktop-mir-session
[09:00] <tsdgeos> any idea why?
[09:01] <tsdgeos> unity8 doesn't seem to be running, that'd be a reason i guess :D
[09:02] <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:03] <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:04] <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:09] <tsdgeos> seb128: don't know, i just installed unity8-desktop-session-mir and hoped it'd work :D
[09:10] <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:13] <tsdgeos> let me try rebooting then
[09:25] <seb128> greyback, hey
[09:25] <seb128> greyback, willcooke hit the same battery icon/unity8 stucked issue yesterday
[09:26] <willcooke> running away, eh?
[09:27] <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:28] <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:29] <willcooke> Just to confirm, I am running a non-standard set up - but I think the problems here are the same root cause
[09:30] <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:51] <seb128> greyback, hey
[09:51] <greyback> seb128: hi!
[09:52] <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:53] <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:54] <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:55] <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:56] <seb128> https://bugs.launchpad.net/ubuntu/+source/unity8/+bug/1422835 if you want a bug to comment on
[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:57] <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:58] <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
[10:04] <willcooke> seb128, I can't get Bluetooth to turn back on with the N7 - can I do it from a command line?
[10:06] <tsdgeos> give me a sec i think i have a patch
[10:06] <tsdgeos> will be back!
[10:07] <seb128> willcooke, what happens when you try to turn it on?
[10:08] <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:14] <seb128> willcooke, not sure, try to sudo restart bluetooth maybe?
[10:17] <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:18] <willcooke> oh
[10:18] <willcooke> well
[10:18] <willcooke> service --status-all says yes, ps aux says no
[10:21] <willcooke> hrm
[10:21] <willcooke> Running bluetoothd manually works
[10:21] <mzanetti> greyback: did you already install silo0 today?
[10:22] <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:29] <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:49] <tsdgeos> seb128: if you have time can you try https://code.launchpad.net/~aacid/ubuntu-ui-toolkit/nonsquareicons/+merge/250110 ?
[11:45] <tsdgeos> larsu: you may be interested in https://code.launchpad.net/~aacid/ubuntu-ui-toolkit/nonsquareicons/+merge/250110 ?
[11:50] <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:51] <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:52] <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:53] <tsdgeos> i'm open to a suggestion on how to better test it
[11:53] <tsdgeos> couldn't find a different way
[11:54] <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:55] <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:56] <larsu> I guess there's no way to fetch the icon theme name?
[12:12] <dandrader> Saviq, ping
[12:17] <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:25] <tsdgeos> larsu: looks much easier to read now https://code.launchpad.net/~aacid/ubuntu-ui-toolkit/nonsquareicons/+merge/250110 :)
[12:26] <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:27] <tsdgeos> dandrader: right
[12:27] <tsdgeos> maybe the train has changed?
[12:27] <tsdgeos> robru: ↑ ?
[12:28] <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:30] <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:46] <dandrader> tsdgeos, checking it
[13:01] <Trevinho> dpm: happy BD! :)
[13:10] <dpm> thanks Trevinho! :-)
[13:42] <larsu> tsdgeos: can't test your changes, uitk fails to build (test failure in localizedApp)
[13:43] <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:50] <larsu> :/
[13:52] <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:53] <tsdgeos> weird
[14:12] <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:13] <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:14] <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:15] <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:16] <seb128> tsdgeos, commented on the mp saying so
[14:16] <larsu> tsdgeos: you didn't say that in the MR or the bug...
[14:17] <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:18] <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:20] <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:21] <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:22] <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:23] <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:24] <tsdgeos> seb128: it only happens for some people because the other people are getting png icons, that don't have this bug
[14:25] <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:26] <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:27] <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:28] <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:29] <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:43] <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:59] <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?
[15:02] <dandrader> mzanetti, left edge drag on the device
[15:03] <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:04] <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:05] <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:06] <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:51] <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:53] <tsdgeos> mzanetti: so i'm trying your hover patch and i can't get it to show the launcher
[15:54] <mzanetti> tsdgeos: how are you trying?
[15:55] <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:56] <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:57] <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:58] <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
[16:04] <tsdgeos> mzanetti: so one we land this ./run and tryLauncher will be broken "forever"?
[16:05] <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:08] <tsdgeos> ok
[16:15] <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:25] <tsdgeos> MacSlow: how hard is to get a test for https://code.launchpad.net/~macslow/unity8/unbreak-notification-positioning-fix-1422711 ?
[16:29] <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:39] <tsdgeos> MacSlow: added some comments to the MR
[17:06] <MacSlow> tsdgeos, done
[19:48] <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?
[20:18] <josharenson> tedg: ^^^ ?
[20:21] <tedg> josharenson, I'm confused, which changes?
[20:22] <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:23] <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:24] <josharenson> tedg: thanks, ill let you know if it still doesn't work