=== alecu_ is now known as alecu === marcusto_ is now known as marcustomlinson [09:13] Saviq: hey! Any progress on LP: #1535397 ? [09:13] 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] sil2100, yes, we're gearing up a silo [09:32] sil2100, https://requests.ci-train.ubuntu.com/#/ticket/963 [09:32] sil2100, it's gonna take a few hours still (conflicts, reviews), but I'm hopeful [09:36] Saviq: excellent! [09:36] What about the greeter experience thing? [09:37] sil2100, yeah, will add that, too [09:38] That would be awesome, we'd have almost everything we need then [09:38] Saviq: thanks! [09:38] sil2100, you'll take care of the livecd-rootfs bits? [09:38] Saviq: sure, I suppose in this case I could release those a bit earlier === JMulholland_ is now known as JMulholland === bpierre_ is now known as bpierre === alan_g is now known as alan_g|lunch [13:38] 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] mterry, hey, I just installed your branch, still says "phablet" [13:38] mterry, ok, still sorting it out for the silo [13:39] mzanetti, you need an image created with https://code.launchpad.net/~mterry/livecd-rootfs/ubuntu-real-name/+merge/284637 [13:39] ah ok [13:39] mzanetti, or change your user's gecos field in /var/lib/extrausers* [13:39] mterry, what does GECOS stand for? [13:40] mzanetti, haha I don't know [13:40] :) [13:40] mzanetti, "General Electric Comprehensive Operating Supervisor" [13:40] https://en.wikipedia.org/wiki/General_Comprehensive_Operating_System [13:41] "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] mterry, so... how did you pick it as the variable for the user's name then? :D [13:42] mzanetti, *I* didn't, that' just the standard [13:42] mzanetti, it also holds the users phone number [13:42] mzanetti, and their fax :) [13:42] mzanetti, all separated by commas. But if you just put one string in there, it will be the real name [13:42] mzanetti, I think you can use chfn to change it? [13:42] mzanetti, don't know if that will support extrausers or not [13:43] mterry, ok, understood [13:43] mterry, works now [13:43] \o/ [13:45] mterry, you know we already have this user real name in DBusSesssionService plugin? :) [13:46] 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] (or duplicate a DBusSesssionService object in the second plugin) [13:47] mterry, yeah, calling dbus from another dbus service makes me feel uncomfortable :) === daek_ is now known as Daekdroom [13:49] mterry, 2 more inline comments [14:02] mterry, who deletes the gchar* ? [14:02] 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] mzanetti, so I guess it is a leak :) [14:02] mzanetti, but not one via use of the API [14:03] mterry, ok, got it [14:03] mterry, I'm ok with not dropping the demo stuff yet... mind adding a trello card for it? [14:04] 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] 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] mzanetti, fair [14:05] making trello card [14:05] mterry, btw, in that case you might want to do "QString realName = QStringLiteral(g_get_real_name());" [14:06] not sure if that does with the g tho [14:06] mzanetti, but it's not a string literal, right? [14:06] oh well... no prob.. this works and doesn't get called often enough to worry, really :D [14:06] approving [14:06] nope [14:07] mzanetti, mterry: QString realName = QStringLiteral(g_get_real_name()); wrong! [14:07] yeah... [14:07] mzanetti, mterry: that screws my name already when I look at it :) === alexabreu is now known as alex-abreu [14:07] ltinkl, well, this returns a pointer to a static string [14:08] mzanetti, mterry: QString realName = QString::fromUtf8(g_get_real_name()); is what you want [14:08] ltinkl, it's not guaranteed to be utf8 [14:08] fair enough [14:08] ltinkl, glib gives no guarantee as to encoding... [14:08] ltinkl, but I guess utf8 is as good a guess as any [14:08] mterry, not guaranteed but that's what all (sane) unix tools do [14:09] * mterry makes that change real quick. [14:09] ltinkl, what is default encoding for QString constructor? [14:12] mterry, hmm... have some unexpected tags on your branch [14:13] mzanetti, ugh! [14:13] mterry, hmm Qt docu says it's utf8 indeed but not sure about QStringLiteral; lemme check for sure [14:13] mterry, not really sure why tho... they don't look that wrong [14:13] 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] mzanetti, just cleaned them [14:13] ack [14:14] mterry, approved [14:15] mzanetti, awesome thanks [14:15] mterry, #define QStringLiteral(str) QString::fromUtf8("" str "", sizeof(str) - 1) [14:16] mterry, all good, sry :) I believe this wasn't the case before [14:18] dednick, what's the issue this branch solves? https://code.launchpad.net/~unity-team/unity8/stagedFullscreen.form-factor/+merge/285053? [14:19] mterry, so I added the test you requested: function test_switchKeymap() in tst_DesktopStage.qml, good enough? [14:19] ltinkl, I like the new qmltest in your kbdLayout branch yeah [14:20] ltinkl, you go to pains to avoid actually just comparing against "fr" though :) [14:20] mzanetti, pad.lv/1535397 [14:20] ltinkl, doing stuff like AccountsService.keymaps[0].split("+")[0] instead. But that's fine [14:20] ah [14:20] mterry, I'm not touching that branch again until the silo build has finished :) [14:20] ltinkl, you can still just delete the other test: test_setKeymap [14:21] and the spy [14:21] mzanetti, for now all the logic will live in qtubuntu, but we need to pass formFactor down to it [14:21] 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] mterry, yeah, after the silo, otherwise Saviq kills me if something conflicts there again :) [14:21] ltinkl, :) [14:21] ltinkl, nope, go for it [14:21] ltinkl, assuming you don't introduce more conflicts (but if you're working in tests, doubt you would) [14:21] Saviq, famous last words [14:21] ltinkl, and it's building already, so no point in waiting [14:22] mzanetti: the u8 bit is just to get the form factor over to client. [14:22] Saviq, mterry: ok [14:22] ltinkl, Saviq never kills people, he just convinces them it would be the best thing to do [14:22] <20:48> ltinkl, it's your fault you're still here, so I'm gonna kill you ;P [14:22] :D [14:23] that was later than that, wasn't it? ;) [14:23] dednick, ok, got it. [14:23] that's UTC or something [14:23] yeah... [14:24] /really_food [14:26] mterry, done, removed [14:30] Saviq, dednick: https://code.launchpad.net/~unity-team/unity8/stagedFullscreen.form-factor/+merge/285053/comments/724853 [14:30] Saviq, dednick, that said, I understand this is temporary. [14:30] so... maybe a FIXME or TODO comment explaining what's happening? [14:31] Saviq, will this go to trunk or just the silo? [14:32] ltinkl, you still have the useless killAllRunningApps call :) [14:37] mterry, removed [14:38] ltinkl, awesome, love it [14:40] ltinkl, oh wait, don't we need debian/control and CMakeLists.txt changes to require latest qtmir / unity-api? [14:41] mterry, they are in https://code.launchpad.net/~unity-team/unity8/packaging/+merge/284814 [14:41] mterry, in the same silo [14:41] ltinkl, oh weird ok [14:41] ltinkl, what's with CI on your branch? [14:41] ltinkl, I see build errors, but I don't see why [14:42] mterry, the CI fails because of that [14:42] ltinkl, ah yeah, I see the error now [14:42] ltinkl, qtmir indeed [14:44] ltinkl, ok approved. that's all the kbdLayout branches, right? [14:44] mterry, yup, I think so (let me check again) [14:45] mterry, yeah, the silo says so [14:45] ltinkl, cool === sil2100_ is now known as sil2100 === alan_g|lunch is now known as alan_g [15:05] mzanetti, temporary in the sense that it sets it all to the same value, or? [15:06] Saviq, yeah [15:06] 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] mzanetti, right, if it's easy, we should probably set it for the screen the shell is on [15:07] +only [15:07] mzanetti, because yes, we're planning to land this normally [15:08] mhm... not sure if the Screens api allows to know on which screen a certain QML item is on [15:20] mzanetti, each QML item has an attached Screen object with its properties [15:20] mzanetti, http://doc.qt.io/qt-5/qml-qtquick-window-screen.html [15:21] dednick, ^ [15:23] mzanetti: screen, not "screens" :) [15:23] i'll look at trying to get it on only the shell screen. === dandrader is now known as dandrader|lunch === dandrader|lunch is now known as dandrader === alan_g is now known as alan_g|EOD === edzweistein2 is now known as anina [19:36] 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] So I would know where to find the sourcecode of the sound settings, because I think there might be a bug?! [19:37] first the sourcecode of the GUI then I can maybe find out why it is not applying the profile properly === dandrader is now known as dandrader|afk