[09:13] <sil2100> Saviq: hey! Any progress on LP: #1535397 ?
[09:13] <ubot5`> Launchpad bug 1535397 in unity8 (Ubuntu) "[enhancement] Implement support for QWindow::visibility set to Automatic" [High,In progress] https://launchpad.net/bugs/1535397
[09:32] <Saviq> sil2100, yes, we're gearing up a silo
[09:32] <Saviq> sil2100, https://requests.ci-train.ubuntu.com/#/ticket/963
[09:32] <Saviq> sil2100, it's gonna take a few hours still (conflicts, reviews), but I'm hopeful
[09:36] <sil2100> Saviq: excellent!
[09:36] <sil2100> What about the greeter experience thing?
[09:37] <Saviq> sil2100, yeah, will add that, too
[09:38] <sil2100> That would be awesome, we'd have almost everything we need then
[09:38] <sil2100> Saviq: thanks!
[09:38] <Saviq> sil2100, you'll take care of the livecd-rootfs bits?
[09:38] <sil2100> Saviq: sure, I suppose in this case I could release those a bit earlier
[13:38] <mterry> ltinkl, in kbdLayout, you can drop the test_setKeymap test completely.  It really does just test the mock.  Which means you can drop the spy too.  And you don't need to call killAllRunningApps at the beginning of a test function -- cleanup() does it for you
[13:38] <mzanetti> mterry, hey, I just installed your branch, still says "phablet"
[13:38] <ltinkl> mterry, ok, still sorting it out for the silo
[13:39] <mterry> mzanetti, you need an image created with https://code.launchpad.net/~mterry/livecd-rootfs/ubuntu-real-name/+merge/284637
[13:39] <mzanetti> ah ok
[13:39] <mterry> mzanetti, or change your user's gecos field in /var/lib/extrausers*
[13:39] <mzanetti> mterry, what does GECOS stand for?
[13:40] <mterry> mzanetti, haha I don't know
[13:40] <mterry> :)
[13:40] <mterry> mzanetti, "General Electric Comprehensive Operating Supervisor"
[13:40] <mterry> https://en.wikipedia.org/wiki/General_Comprehensive_Operating_System
[13:41] <mterry> "Some early Unix systems at Bell Labs used GECOS machines for print spooling and various other services,[3] so this field was added to carry information on a user's GECOS identity."
[13:41] <mzanetti> mterry, so... how did you pick it as the variable for the user's name then? :D
[13:42] <mterry> mzanetti, *I* didn't, that' just the standard
[13:42] <mterry> mzanetti, it also holds the users phone number
[13:42] <mterry> mzanetti, and their fax :)
[13:42] <mterry> mzanetti, all separated by commas.  But if you just put one string in there, it will be the real name
[13:42] <mterry> mzanetti, I think you can use chfn to change it?
[13:42] <mterry> mzanetti, don't know if that will support extrausers or not
[13:43] <mzanetti> mterry, ok, understood
[13:43] <mzanetti> mterry, works now
[13:43] <mterry> \o/
[13:45] <ltinkl> mterry, you know we already have this user real name in DBusSesssionService plugin? :)
[13:46] <mterry> ltinkl, oh nice.  But calling code in one plugin from another is tough.  So I would have had to add a call to DBus directly in our fake liblightdm plugin.  Since gecos is always in sync with AccountsService anyway, seemed fine to grab it from tehre
[13:46] <mterry> (or duplicate a DBusSesssionService object in the second plugin)
[13:47] <ltinkl> mterry, yeah, calling dbus from another dbus service makes me feel uncomfortable :)
[13:49] <mzanetti> mterry, 2 more inline comments
[14:02] <mzanetti> mterry, who deletes the gchar* ?
[14:02] <mterry> mzanetti, no one.  It's a string owned by glib.  Looks like glib keeps info on the current user hanging around inside it forever
[14:02] <mterry> mzanetti, so I guess it is a leak  :)
[14:02] <mterry> mzanetti, but not one via use of the API
[14:03] <mzanetti> mterry, ok, got it
[14:03] <mzanetti> mterry, I'm ok with not dropping the demo stuff yet... mind adding a trello card for it?
[14:04] <mterry> mzanetti, sure.  I mean, I'm fine with dropping it.  Didn't mean to appear like I was pushing back.  I genuinely am not sure if we care about demo'ing multi user...
[14:04] <mzanetti> mterry, I don't think we do. But well... you never know... let's just add a reminder to clean it up when we replace it with the real multiuser thing
[14:05] <mterry> mzanetti, fair
[14:05] <mterry> making trello card
[14:05] <mzanetti> mterry, btw, in that case you might want to do "QString realName = QStringLiteral(g_get_real_name());"
[14:06] <mzanetti> not sure if that does with the g tho
[14:06] <mterry> mzanetti, but it's not a string literal, right?
[14:06] <mzanetti> oh well... no prob.. this works and doesn't get called often enough to worry, really :D
[14:06] <mzanetti> approving
[14:06] <ltinkl> nope
[14:07] <ltinkl> mzanetti, mterry: QString realName = QStringLiteral(g_get_real_name()); wrong!
[14:07] <mzanetti> yeah...
[14:07] <ltinkl> mzanetti, mterry: that screws my name already when I look at it :)
[14:07] <mzanetti> ltinkl, well, this returns a pointer to a static string
[14:08] <ltinkl> mzanetti, mterry: QString realName = QString::fromUtf8(g_get_real_name()); is what you want
[14:08] <mterry> ltinkl, it's not guaranteed to be utf8
[14:08] <mzanetti> fair enough
[14:08] <mterry> ltinkl, glib gives no guarantee as to encoding...
[14:08] <mterry> ltinkl, but I guess utf8 is as good a guess as any
[14:08] <ltinkl> mterry, not guaranteed but that's what all (sane) unix tools do
[14:09]  * mterry makes that change real quick.
[14:09] <mterry> ltinkl, what is default encoding for QString constructor?
[14:12] <mzanetti> mterry, hmm... have some unexpected tags on your branch
[14:13] <mterry> mzanetti, ugh!
[14:13] <ltinkl> mterry, hmm Qt docu says it's utf8 indeed but not sure about QStringLiteral; lemme check for sure
[14:13] <mzanetti> mterry, not really sure why tho... they don't look that wrong
[14:13] <mterry> mzanetti, like a few weeks ago, I started seeing tags on my branches.  Don't know why I have them now.  I haven't visited any direputable bzr servers!
[14:13] <mterry> mzanetti, just cleaned them
[14:13] <mzanetti> ack
[14:14] <mzanetti> mterry, approved
[14:15] <mterry> mzanetti, awesome thanks
[14:15] <ltinkl> mterry,  #define QStringLiteral(str) QString::fromUtf8("" str "", sizeof(str) - 1)
[14:16] <ltinkl> mterry, all good, sry :) I believe this wasn't the case before
[14:18] <mzanetti> dednick, what's the issue this branch solves? https://code.launchpad.net/~unity-team/unity8/stagedFullscreen.form-factor/+merge/285053?
[14:19] <ltinkl> mterry, so I added the test you requested: function test_switchKeymap() in tst_DesktopStage.qml, good enough?
[14:19] <mterry> ltinkl, I like the new qmltest in your kbdLayout branch yeah
[14:20] <mterry> ltinkl, you go to pains to avoid actually just comparing against "fr" though  :)
[14:20] <Saviq> mzanetti, pad.lv/1535397
[14:20] <mterry> ltinkl, doing stuff like AccountsService.keymaps[0].split("+")[0] instead.  But that's fine
[14:20] <mzanetti> ah
[14:20] <ltinkl> mterry, I'm not touching that branch again until the silo build has finished :)
[14:20] <mterry> ltinkl, you can still just delete the other test: test_setKeymap
[14:21] <mterry> and the spy
[14:21] <Saviq> mzanetti, for now all the logic will live in qtubuntu, but we need to pass formFactor down to it
[14:21] <dednick> mzanetti: swtiching between desktop & phone/tablet preserves window state; which we don't want to happen. ie staged apps like camera shouldn't be fullscreen when we switch to desktop.
[14:21] <ltinkl> mterry, yeah, after the silo, otherwise Saviq kills me if something conflicts there again :)
[14:21] <mterry> ltinkl, :)
[14:21] <Saviq> ltinkl, nope, go for it
[14:21] <Saviq> ltinkl, assuming you don't introduce more conflicts (but if you're working in tests, doubt you would)
[14:21] <mterry> Saviq, famous last words
[14:21] <Saviq> ltinkl, and it's building already, so no point in waiting
[14:22] <dednick> mzanetti: the u8 bit is just to get the form factor over to client.
[14:22] <ltinkl> Saviq, mterry: ok
[14:22] <mzanetti> ltinkl, Saviq never kills people, he just convinces them it would be the best thing to do
 <Saviq> ltinkl, it's your fault you're still here, so I'm gonna kill you ;P
[14:22] <mzanetti> :D
[14:23] <Saviq> that was later than that, wasn't it? ;)
[14:23] <mzanetti> dednick, ok, got it.
[14:23] <Saviq> that's UTC or something
[14:23] <ltinkl> yeah...
[14:24] <Saviq> /really_food
[14:26] <ltinkl> mterry, done, removed
[14:30] <mzanetti> Saviq, dednick: https://code.launchpad.net/~unity-team/unity8/stagedFullscreen.form-factor/+merge/285053/comments/724853
[14:30] <mzanetti> Saviq, dednick, that said, I understand this is temporary.
[14:30] <mzanetti> so... maybe a FIXME or TODO comment explaining what's happening?
[14:31] <mzanetti> Saviq, will this go to trunk or just the silo?
[14:32] <mterry> ltinkl, you still have the useless killAllRunningApps call  :)
[14:37] <ltinkl> mterry, removed
[14:38] <mterry> ltinkl, awesome, love it
[14:40] <mterry> ltinkl, oh wait, don't we need debian/control and CMakeLists.txt changes to require latest qtmir / unity-api?
[14:41] <ltinkl> mterry, they are in https://code.launchpad.net/~unity-team/unity8/packaging/+merge/284814
[14:41] <ltinkl> mterry, in the same silo
[14:41] <mterry> ltinkl, oh weird ok
[14:41] <mterry> ltinkl, what's with CI on your branch?
[14:41] <mterry> ltinkl, I see build errors, but I don't see why
[14:42] <ltinkl> mterry, the CI fails because of that
[14:42] <mterry> ltinkl, ah yeah, I see the error now
[14:42] <mterry> ltinkl, qtmir indeed
[14:44] <mterry> ltinkl, ok approved.  that's all the kbdLayout branches, right?
[14:44] <ltinkl> mterry, yup, I think so (let me check again)
[14:45] <ltinkl> mterry, yeah, the silo says so
[14:45] <mterry> ltinkl, cool
[15:05] <Saviq> mzanetti, temporary in the sense that it sets it all to the same value, or?
[15:06] <mzanetti> Saviq, yeah
[15:06] <mzanetti> Saviq, I mean, right now, the Shell.qml only runs on 1 screen at a time. still this code sets its value to all the screens
[15:07] <Saviq> mzanetti, right, if it's easy, we should probably set it for the screen the shell is on
[15:07] <Saviq> +only
[15:07] <Saviq> mzanetti, because yes, we're planning to land this normally
[15:08] <mzanetti> mhm... not sure if the Screens api allows to know on which screen a certain QML item is on
[15:20] <ltinkl> mzanetti, each QML item has an attached Screen object with its properties
[15:20] <ltinkl> mzanetti, http://doc.qt.io/qt-5/qml-qtquick-window-screen.html
[15:21] <mzanetti> dednick, ^
[15:23] <dednick> mzanetti: screen, not "screens" :)
[15:23] <dednick> i'll look at trying to get it on only the shell screen.
[19:36] <anina> hey guys :) I am using ubuntu 15.10 and unity 7.3.2. I am facing problems with my bluetooth speaker, the A2DP profile can be selected but it does not really apply it. Then I downloaded the blueman package and with the blueman-manager I can set the profile properly.
[19:36] <anina> So I would know where to find the sourcecode of the sound settings, because I think there might be a bug?!
[19:37] <anina> first the sourcecode of the GUI then I can maybe find out why it is not applying the profile properly