=== _salem is now known as salem_ | ||
TheMuso | What Qt API should I be looking at if I want to get the name for the application using the qml plugin I am writing? i.e the qml bindings I am writing to connect to a service, and an app name needs to be supplied. | 03:36 |
---|---|---|
=== salem_ is now known as _salem | ||
mzanetti | TheMuso: QApplication | 07:13 |
mzanetti | (I guess) | 07:13 |
TheMuso | mzanetti: I was thinking that was the case, but wanted to be sure, given I am writing C++ code to provide qml bindings. | 07:28 |
tsdgeos | mzanetti: ping | 07:37 |
mzanetti | tsdgeos: hey | 07:57 |
mzanetti | TheMuso: so? what's the problem with QApplication? | 07:57 |
tsdgeos | mzanetti: can you read alan's comments at https://codereview.qt-project.org/#change,60535 and maybe answer? | 07:57 |
mzanetti | tsdgeos: on it | 07:57 |
tsdgeos | mzanetti: if it helps you qquickitemview.cpp has | 07:58 |
tsdgeos | /###Possibly rename these properties, since they are very useful even without a highlight? | 07:58 |
tsdgeos | qreal QQuickItemView::preferredHighlightBegin() const | 07:58 |
tsdgeos | also i am not sure if parts of his comment are against the logic of the qml or the logic of the patch :_/ | 07:59 |
mzanetti | tsdgeos: I don't understand what he means | 07:59 |
tsdgeos | " That test case looks suspicious, because it's not setting a highlightRangeMode yet is setting preferredHighlightRange." ? | 08:00 |
mzanetti | tsdgeos: ? where is that? | 08:00 |
tsdgeos | previous comment | 08:00 |
mzanetti | tsdgeos: I mean this one "I just tested it, and if you remove the preferredHighlightBegin/End then it works correctly. Which means that it's a bit of an invalid usecase, but also that the above logic is broken for the NoHighlightRange case." | 08:00 |
tsdgeos | expand the other comment | 08:00 |
mzanetti | tsdgeos: you can add highlightRangeMode: ListView.ApplyRange. That's what I do in the launcher. However, as it does not affect the issue in any way I left it away in the as-simple-as-possible example to reproduce it | 08:02 |
tsdgeos | ok | 08:03 |
tsdgeos | tbh i don't understand the rest of his comments either | 08:03 |
mzanetti | tsdgeos: should I describe my use case as a comment here? | 08:03 |
tsdgeos | mzanetti: it may help | 08:05 |
mzanetti | tsdgeos: is it a good idea if I +1 it? | 08:10 |
tsdgeos | not really sure | 08:11 |
mzanetti | tsdgeos: commented. | 08:17 |
tsdgeos | tx | 08:17 |
tsdgeos | mzanetti: i'd leave the +1 for "i do understand the code and like it" | 08:17 |
tsdgeos | otherwise it may seem we are a doing as a "gang push" | 08:17 |
mzanetti | tsdgeos: which kinda is the case for me... but still... the "gang push" is what concerns me | 08:18 |
mzanetti | tsdgeos: he suggested to remove the preferredHighlightBegin/End (aka disable snapping), right? | 08:19 |
tsdgeos | well, he said "the bug" doesn't happen without the preferredHighlightBegin/End | 08:20 |
tsdgeos | yes | 08:20 |
tsdgeos | problem is preferredHighlightBegin/End is a bad name i find | 08:20 |
tsdgeos | as the comment in the file says | 08:20 |
mzanetti | agreed. should be snapPositionBegin/End | 08:20 |
mzanetti | or something along that | 08:21 |
mzanetti | actually, this use case happens always if you want to have something like a CoverFlow with snapping in the middle | 08:21 |
dednick | tsdgeos, mzanetti, MacSlow, Saviq: anyone available to do a quick device test for me? | 08:54 |
MacSlow | dednick, sorry... in the middle of ap-testing for notifications | 08:54 |
MacSlow | dednick, I can do it afterwards... what do you need to have checked? | 08:55 |
MacSlow | dednick, does the installed image matter? | 08:55 |
dednick | MacSlow: testing if sound indicator updating volume for apps. dont know if image matters. | 08:56 |
tsdgeos | dednick: i have the unity-mir stuff, would take me quite a while to go back | 08:56 |
tsdgeos | better if someone can do | 08:57 |
dednick | MacSlow: after a fresh flash the volume will be affected, but as soon as i restart phone, it stops changing. | 08:57 |
tsdgeos | someone *else* that is :D | 08:57 |
dednick | tsdgeos: no worries. | 08:57 |
larsu | dednick: good morning. I have a first version of indicator-messages that uses an indicator file at lp:~larsu/indicator-messages/phablet-port-indicator-ng | 09:00 |
mzanetti | dednick: can help you I guess | 09:00 |
larsu | dednick: it's not thoroughly tested yet, but you can give it a spin if you're up for it | 09:00 |
dednick | larsu: \o/ . will give it a go today | 09:00 |
larsu | dednick: (I'll be on a train for the rest of the day) | 09:00 |
dednick | larsu: i've made some more updates to qmenumodel. added impl for 'action name' and 'update state'. | 09:01 |
dednick | larsu: although i didnt know how you were handing changes to the gtkmenu area. i realise it's from another project... | 09:02 |
dednick | larsu: https://code.launchpad.net/~nick-dedekind/qmenumodel/add-menumenumodel.action-name.update-state | 09:02 |
larsu | dednick: neat! Can you MR them onto my branch? I'll be at guadec, so I probably won't be online a lot. | 09:03 |
dednick | larsu: ok. no prob | 09:03 |
larsu | dednick: cool, talk to you later. Gotta catch a train :) | 09:03 |
dednick | mzanetti: cool. do you have the video scope assets on your device? | 09:08 |
mzanetti | dednick: the demo asses, yes | 09:08 |
dednick | mzanetti: you mind checking if changing the volume in the indicator updates the volume of the video? | 09:09 |
mzanetti | dednick: do I need todays image? | 09:10 |
dednick | mzanetti: nope | 09:10 |
mzanetti | dednick: dammit. just realized I only have the scopes fake data (aka the video covers) but not the video itself. gimme a minute and I'll install | 09:11 |
dednick | mzanetti: ok. thanks. | 09:11 |
mzanetti | dednick: I did another test which tells me it won't work for the videos | 09:12 |
mzanetti | dednick: I logged in and ran "alsamixer". Changing the indicators volume does change the alsamixer's sliders | 09:12 |
mhr3 | sil2100, any success on lucene++? pls brief jamesh ^ | 09:13 |
dednick | mzanetti: it shouldn't? | 09:13 |
mzanetti | dednick: d'oh! "... does NOT change the alsamixer's sliders" | 09:13 |
mzanetti | dednick: yes, it should I'd say | 09:13 |
dednick | mzanetti: :) haha. ok | 09:13 |
sil2100 | mhr3, jamesh: yes, I built the package locally and in my PPA, just need to tweak it a bit | 09:13 |
tsdgeos | mzanetti: does my reply to the snapping patch make sense to you? | 09:14 |
* mzanetti reads | 09:14 | |
dednick | mzanetti: presumably you've restarted your device since you last flahed? | 09:14 |
mzanetti | dednick: a 100 times when testing the SIM pin :D | 09:14 |
jamesh | sil2100: there was some packaging in the hollywood-team/staging PPA, if that helps | 09:14 |
dednick | mzanetti: ok thanks. confirms my suspicions. | 09:14 |
mzanetti | tsdgeos: yep. makes sense to me | 09:15 |
tsdgeos | okki | 09:15 |
Saviq | dednick, mzanetti does the video preview work for you at all? | 09:15 |
dednick | Saviq: sometimes | 09:15 |
mhr3 | sil2100, cool, jamesh do you think we could get lucene and the scanner built in an arm-enabled ppa so we could check things on the phone? | 09:15 |
mzanetti | Saviq: dunno. still don't have a video. just tested with alsamixer | 09:15 |
dednick | Saviq: it's a bit hit and miss. sometimes get a white page, sometimes black, sometimes works | 09:18 |
mzanetti | is unity8 always running on ~20% cpu on the desktop for you too? | 09:18 |
Saviq | dednick, that's on device? | 09:18 |
dednick | Saviq: ya | 09:18 |
Saviq | dednick, to me it looks like carousel is broken, grid is fine | 09:18 |
dednick | Saviq: oh, you mean the video selection widget? | 09:19 |
Saviq | dednick, in Video apps | 09:19 |
dednick | Saviq: that seems to be ok for me. | 09:19 |
Saviq | s/apps/dash/ | 09:19 |
dednick | Saviq: although i might be using mock video | 09:19 |
Saviq | dednick, yeah, that's what I mean | 09:19 |
dednick | Saviq: it's fine. I thought you meant the mediaplayer-app. | 09:20 |
Saviq | dednick, mock video scope previews for carousel are broken | 09:20 |
dednick | Saviq: that's how i've been testing my sound issues. | 09:21 |
Saviq | dednick, which version? | 09:21 |
Saviq | dednick, yeah, that's what I wanted, too ;) | 09:21 |
dednick | Saviq: latest | 09:21 |
Saviq | dednick, but the previews for my carousel are broken ;) | 09:21 |
dednick | Saviq: doh. | 09:22 |
dednick | Saviq: what does it show? | 09:22 |
Saviq | dednick, an empty shape and stars | 09:23 |
Saviq | dednick, http://pastebin.ubuntu.com/5932127/ is the console output | 09:24 |
Saviq | paulliu, ↑ | 09:24 |
Saviq | dednick, found it :/ | 09:25 |
Saviq | http://pastebin.ubuntu.com/5932135/ | 09:26 |
Saviq | mhr3, ↑ your fault again ;P | 09:26 |
dednick | lol | 09:26 |
dednick | wonder why mine is working | 09:27 |
dednick | ah. maybe that isnt in image? | 09:27 |
mhr3 | Saviq, stop using those mocks! :P | 09:28 |
Saviq | mhr3, supply me with matching backends! | 09:28 |
mhr3 | ehm, touche | 09:28 |
dednick | holy crap. my nexus7 spazed out | 09:28 |
dednick | doesnt want to flash :( | 09:29 |
Saviq | dednick, if you didn't flash with --pending, then you're on a 0727 image | 09:29 |
Saviq | 0726 | 09:29 |
dednick | Saviq: is that a bad thing? | 09:29 |
Saviq | dednick, we generally should be running --pending | 09:30 |
Saviq | dednick, to be on the edge | 09:30 |
=== alan_g is now known as alan_g|tea | ||
dednick | lol. the 'danger zone' | 09:30 |
Saviq | dednick, mzanetti https://code.launchpad.net/~saviq/unity8/fix-mock-videopreview/+merge/177777 please | 09:30 |
Saviq | nice merge number ;) | 09:30 |
dednick | heh | 09:31 |
mhr3 | Saviq, are we ok with being unable to close apps once we have a "proper" running apps scope? | 09:34 |
mhr3 | Saviq, or will you keep a hack in the renderer? | 09:34 |
=== alan_g|tea is now known as alan_g | ||
tsdgeos | Saviq: did you give the branches i posted yesterday a try? | 09:37 |
dednick | Saviq: we using pulse on device now? | 09:38 |
tsdgeos | dednick: we've always been using pulse as far as i know | 09:39 |
dednick | Saviq, tsdgeos: any idea why there is a phablet-team ppa for pusleaudio? | 09:40 |
tsdgeos | last time i checked there was some small extra patch there | 09:40 |
tsdgeos | don't really remember what it was about | 09:40 |
dednick | tsdgeos: ok | 09:40 |
=== alan_g is now known as alan_g|afk | ||
=== alan_g|afk is now known as alan_g | ||
tsdgeos | MacSlow: you guys need a copyright header in https://code.launchpad.net/~macslow/unity8/notification-autopilot-tests-dbus/+merge/177780 | 11:12 |
Cimi | seb128, ping :) | 11:19 |
seb128 | Cimi, hey | 11:19 |
Cimi | seb128, I'm writing the welcome wizard for the phone, I'm wondering how can I fetch the list of the locales | 11:20 |
seb128 | Cimi, look at the language panel in system settings | 11:20 |
Cimi | ok | 11:21 |
Cimi | seb128, lp? | 11:21 |
seb128 | Cimi: lp:ubuntu-system-settings plugins/language/language-plugin.cpp getLocales() | 11:21 |
Cimi | seb128, thanks | 11:34 |
Cimi | seb128, saw it | 11:34 |
seb128 | Cimi, yw | 11:34 |
Cimi | seb128, you think I could import it? | 11:34 |
Cimi | seb128, it sucks to duplicate code | 11:34 |
seb128 | Cimi, I'm not sure what are the best practices in qml there... | 11:34 |
seb128 | but it would make sense to share the code yes | 11:34 |
Cimi | seb128, if the plugin is locally installed | 11:34 |
seb128 | not sure how though | 11:34 |
Cimi | seb128, might import it | 11:34 |
seb128 | it's in a private dir, so not sure if you can import it easily (out of hardcoding the path to it) | 11:35 |
seb128 | ideally we would have e.g indicator-keyboard-service to publish that list over gmenu or something | 11:36 |
tsdgeos | Saviq: are we having the "UnityNext team product review" hangout today with kgunn or shall we just do the usual stand-up | 11:36 |
Saviq | tsdgeos, usual ,we're in with Jane at that time | 11:37 |
tsdgeos | oka | 11:37 |
=== ashams_ is now known as ashams | ||
Cimi | seb128, but I need to set this in the welcome wizard | 11:46 |
Cimi | seb128, we have wifi, location setting, username and language | 11:47 |
Cimi | seb128, would be great to have set them without duplicating code | 11:47 |
seb128 | Cimi, that seems quite some common work with system setting/indicators | 11:47 |
Cimi | exactly | 11:47 |
seb128 | the idea so far was to have the indicator backends to provide those features | 11:47 |
seb128 | through gmenumodel | 11:47 |
Cimi | who knows here? | 11:47 |
Cimi | larsu / dednick ? | 11:47 |
seb128 | and use unitymenumodel | 11:47 |
seb128 | to do the qt UIs | 11:47 |
seb128 | yes | 11:48 |
seb128 | larsu is in a train today, travelling to GUADEC | 11:48 |
seb128 | so dednick | 11:48 |
dednick | yo | 11:48 |
dednick | oh. that was a statement. | 11:49 |
dednick | Cimi: what is the welcome wizard? | 11:50 |
dednick | Cimi: intial device setup? | 11:50 |
Cimi | dednick, yes | 11:50 |
bregma | soooo, all the ibus-related failures in the Unity 7 daily were expected, in case anyone was wondering.... | 11:53 |
dednick | Cimi: right. not really sure how that would work, but i guess it would need to be fed by settings. Not sure we can rely on indicators as i'm not sure it's a 1-1 relationship between indicators and settings. | 11:54 |
Cimi | I can do mock now | 11:55 |
Cimi | waiting for a model... | 11:55 |
seb128 | bregma, the fixes are ready waiting to land as well? ;-) | 11:56 |
=== alan_g is now known as alan_g|lunch | ||
seb128 | dednick, well, the idea was that the indicators would have a settings profil | 11:57 |
seb128 | Cimi, dednick: they can also have a wizard profile | 11:57 |
seb128 | we already have profiles for desktop, phone, greeter, installer | 11:57 |
bregma | seb128, I'll be checking with brandon when he comes in, but he warned me there would be a slew of ibus failures in todays build -- I am assuming it has to do with dependency change sequencing or something | 11:58 |
dednick | seb128: i c | 11:58 |
dednick | just trying to think of the best way to mock it. | 11:59 |
bregma | seb128, yes, the problem is the new ibus 1.5 went in to the archive but the required Nux patches are still in review | 11:59 |
dednick | Cimi: you could probably get away with forgetting the mock for now and just using the current profiles for testing. | 12:02 |
Cimi | dednick, some code I should have a look at? | 12:02 |
dednick | Cimi: otherwise you're going to essentially need to create a mock indicator and have indicator files for them. which may not be a bad idea for testing purposes to be honest | 12:03 |
dednick | Cimi: if you want to mock, easiest approach may be to use vala to generate the code. | 12:03 |
Cimi | dednick, yep but which code shall I mock? | 12:04 |
Cimi | dednick, unity? | 12:04 |
dednick | Cimi: but if you want to use the current indicator profiles, the ui code to pick them up and process is all in plugins/Unity/Indicators | 12:04 |
Cimi | ok | 12:04 |
Cimi | dednick, might need to import this plugin from the external welcomewizard | 12:05 |
Cimi | unless we want the welcome wizard to be part of unity | 12:05 |
dednick | Cimi: the indicator backends are separate projects | 12:05 |
dednick | Cimi: hm. should be able to import the qml library ok... | 12:07 |
dednick | Cimi: you can probably use all the page code as well. | 12:08 |
=== hikiko is now known as hikiko|lunch | ||
dednick | Cimi: will need some modifications to be able to load different profiles. at the moment it's hardcoded to load phone. | 12:13 |
* Cimi <--- lunch | 12:17 | |
mzanetti | Saviq: we're not having the hangout today, do we? | 12:49 |
Saviq | mzanetti, no | 12:49 |
=== hikiko|lunch is now known as hikiko | ||
=== alan_g|lunch is now known as alan_g | ||
=== jhodapp is now known as jhodapp|afk | ||
=== _salem is now known as salem_ | ||
seb128 | bregma, do you know if there is a bug/known issue about launcher emblems showing only on one monitor? | 13:17 |
seb128 | bregma, e.g the tb badge for new emails | 13:17 |
dandrader | mzanetti, there's a nuisance with your bzr plugin: it runs in any bzr repository, not just unity8 | 13:50 |
=== jhodapp|afk is now known as jhodapp | ||
bregma | seb128, sounds vaguely familiar but I can't seem to be able to find the bug | 13:51 |
mzanetti | dandrader: huh? I throught that should be fixed | 14:03 |
mzanetti | dandrader: it checks if there is a Shell.qml in the repositories root | 14:04 |
dandrader | mzanetti, ahhh... that's because my minimal test case does have a Shell.qml in its root dir :D | 14:05 |
mzanetti | heh | 14:05 |
mzanetti | dandrader: the thing is, bzr repository branches can have any name and any origin. so its not possible to check on those either | 14:06 |
mzanetti | I know that checking for Shell.qml is not really nice | 14:06 |
mzanetti | seb128: ping | 14:09 |
seb128 | mzanetti, hey | 14:10 |
mzanetti | seb128: how far are you with the SIM pin stuff? | 14:10 |
mzanetti | seb128: did you already check out the ofono api? | 14:10 |
sil2100 | mhr3_: hi! I pushed the latest version of lucene++ for building and updated the needs-packaging bug with all the required links to branches | 14:10 |
sil2100 | mhr3_: so now just for some MOTU guys to review | 14:11 |
seb128 | mzanetti, kenvandine has been looking at the SIM stuff so far (he did the SIM services yesterday), I don't think anyone started on the pin | 14:11 |
sil2100 | mhr3_: https://bugs.launchpad.net/ubuntu/+bug/1036812 | 14:11 |
seb128 | mzanetti, no, I didn't yet | 14:11 |
ubot5 | Launchpad bug 1036812 in Ubuntu "[needs-packaging] Lucene++" [Wishlist,In progress] | 14:11 |
seb128 | mzanetti, why? | 14:11 |
mzanetti | seb128: for some reason it the amount of available pin retries does not work | 14:11 |
sil2100 | mhr3_: I'll have some of my guys do a quick packaging review as well before that | 14:11 |
mzanetti | seb128: while I'm sure my SIM card suports it, calling this: qdbus --system --literal org.ofono /ril_0 org.ofono.SimManager.GetProperties | 14:11 |
mzanetti | seb128: just returns an empty list | 14:11 |
kenvandine | i haven't looked at SIM pin stuff at all, mine doesn't use a pin | 14:13 |
kenvandine | but libofono-qt does have an API for unlock/lock | 14:13 |
dandrader | found a leak! | 14:35 |
dandrader | in Dash/Video/VideoInfo.qml | 14:35 |
dandrader | remove it and memory stops growing | 14:35 |
dandrader | now gotta find why | 14:35 |
=== alan_g is now known as alan_g|tea | ||
tsdgeos | XmlListModel ? | 14:37 |
tsdgeos | seems like it probably has lots of C++ code behind :D | 14:38 |
dandrader | tsdgeos, yeah, it must be this component | 14:39 |
=== alan_g|tea is now known as alan_g | ||
mzanetti | MacSlow: the script is executable? | 14:58 |
MacSlow | mzanetti, yes | 14:58 |
MacSlow | mzanetti, that's wrong?! | 14:59 |
mzanetti | no... that would be correct | 14:59 |
MacSlow | mzanetti, hm... but there's a hashbang line in it | 15:00 |
MacSlow | mzanetti, maybe that's causing the issue? | 15:00 |
mzanetti | which one is it? | 15:01 |
MacSlow | mzanetti, "#!/usr/bin/env python" is used in create_interactive_notification.py although that's not were the error is reported | 15:03 |
mzanetti | MacSlow: hmm... still weird... try changing it to /usr/bin/python | 15:04 |
mzanetti | MacSlow: but it I don't think this is it | 15:05 |
mzanetti | MacSlow: ~macslow/unity8/notification-autopilot-tests/ ? | 15:07 |
MacSlow | mzanetti, no lp:~macslow/unity8/notification-autopilot-tests-dbus | 15:07 |
mzanetti | MacSlow: oh... its the setsid | 15:10 |
mzanetti | MacSlow: is there a reason its required? | 15:11 |
MacSlow | mzanetti, yeah... like mentioned in the email... is it a permission-issue on jenkins? Do we need to enable something to make this work? | 15:11 |
tsdgeos | hmmm | 15:13 |
* tsdgeos realizes the "AddressBook" app is trying to do something like the LVPWH | 15:14 | |
MacSlow | mzanetti, not really... tests work without it... but they seem to take longer to start | 15:14 |
tsdgeos | and it's obviously failing | 15:14 |
tsdgeos | since we failed at it too in QML :D | 15:14 |
MacSlow | mzanetti, but I could just imagine this taking more time | 15:14 |
mzanetti | MacSlow: try it without it please. if that doesn't work we can enable stuff in jenkins | 15:14 |
mzanetti | tsdgeos: tell renato | 15:14 |
mzanetti | that was close... I entered "sudo reboot" and only the password not being "phablet" protected me from rebooting my notebook instead of the phone | 15:16 |
tsdgeos | :D | 15:16 |
MacSlow | mzanetti, seems to work (locally) without that culprit line... I'll delete it can re-commit/push | 15:16 |
mzanetti | MacSlow: ack | 15:16 |
sil2100 | pstolowski: hi! | 15:50 |
pstolowski | sil2100: hey! | 15:50 |
sil2100 | pstolowski: you know if mhr3 will still be around today? | 15:50 |
pstolowski | sil2100: he won't, he is attending guadec | 15:51 |
pstolowski | sil2100: he has few days off, but he may show up from time to time on irc | 15:51 |
sil2100 | Ah, ok :) | 15:51 |
sil2100 | pstolowski: thanks! | 15:52 |
sil2100 | jamesh: hi! | 15:52 |
pstolowski | sil2100: yw | 15:52 |
jamesh | sil2100: hi. | 15:57 |
jamesh | I was just about to head off to bed, but if it's something quick, I'm here :) | 15:58 |
sil2100 | jamesh: I remember mhr3 saying that you're interested in lucene++ in Ubuntu? ;) | 15:58 |
jamesh | sil2100: I'm interested in getting the Media scanner in Ubuntu, and lucene++ is a dependency | 15:58 |
sil2100 | jamesh: are there any specific tests you would like to run to see if the lucene++ that we're proposing to Ubuntu works? | 15:59 |
sil2100 | jamesh: since I prepared lucene++ packaging, basing on work of Openismus and kenvandine's, and kenvandine will probably sponsor it today to universe | 15:59 |
pstolowski | sil2100, jamesh: I think it should go to main? | 16:01 |
kenvandine | pstolowski, if something in main will depend on it, then we'll need to get an MIR done | 16:02 |
kenvandine | but first step is universe | 16:02 |
sil2100 | pstolowski: we can get it into main later | 16:02 |
jamesh | sil2100: The only testing I've been doing of lucene++ was building and running mediascanner, but that there are a few patches not in trunk before it'll build on Saucy | 16:02 |
sil2100 | pstolowski: first universe, then MIR to main if needed | 16:02 |
sil2100 | jamesh: ok | 16:02 |
sil2100 | jamesh: then we'll get what we have into Ubuntu | 16:02 |
jamesh | I think we'll want this in main eventually, since it'll be going on the phone if everything goes to plan | 16:03 |
pstolowski | kenvandine, sil2100: I see, sounds good, thanks | 16:05 |
Saviq | dandrader, hey, since it's your last day, can you post an update on anything you've found about bug #1201116? | 16:14 |
ubot5 | bug 1201116 in Unity 8 "[dash] constant increate in memory consumption when navigating through lenses" [Critical,In progress] https://launchpad.net/bugs/1201116 | 16:14 |
Saviq | dandrader, the CPU issue seems to be more pressing than mem, too | 16:14 |
dandrader | Saviq, sure. I'm just trimming my test case to a bare minimum | 16:14 |
dandrader | Saviq, did you report a separate bug on the CPU issue? | 16:15 |
Saviq | dandrader, no, it's the same bug | 16:15 |
Saviq | dandrader, ah, you wanted to separate it out | 16:15 |
dandrader | Saviq, well, not anymore, I've renamed it :) | 16:15 |
dandrader | Saviq, and you told there you would report one. stand by your word! :) | 16:15 |
Saviq | dandrader, ;P | 16:16 |
dandrader | it's shame we don't have bug dependencies in launchpad... | 16:16 |
Saviq | dandrader, bug #1206991 ;) | 16:18 |
ubot5 | bug 1206991 in Unity 8 "Frequent CPU hogging" [Critical,Triaged] https://launchpad.net/bugs/1206991 | 16:18 |
pete-woods | seb128: unfortunately that patch has no effect when applied to the hg files in the ubuntu package - the package build doesn't regenerate the generate files | 16:18 |
pete-woods | *generated | 16:18 |
pete-woods | either we need to make the debuild build generate the .cc files, or we need to patch the .cc files directly | 16:19 |
pete-woods | seb128: I have an updated patch that patches the .cc files if you want it? | 16:26 |
seb128 | pete-woods, otp, but yes please | 16:26 |
dandrader | Saviq, bug updated | 16:36 |
pete-woods | seb128: whenever you are ready, I put the patch against this bug https://bugs.launchpad.net/ubuntu/+source/glibmm2.4/+bug/1206997 :) | 16:38 |
ubot5 | Launchpad bug 1206997 in glibmm2.4 (Ubuntu) "Updated patch for generated .cc files" [Undecided,New] | 16:38 |
seb128 | pete-woods, thanks | 16:39 |
Saviq | dandrader, thanks, and talk to you in a month! ;) | 16:39 |
Saviq | dandrader, hmm... no comment in bug #1206991 ? | 16:40 |
ubot5 | bug 1206991 in Unity 8 "Frequent CPU hogging" [Critical,Triaged] https://launchpad.net/bugs/1206991 | 16:40 |
=== dandrader_ is now known as dandrader|lunch | ||
Saviq | dednick, hey, can you give me a small update on what do we still need to transition to indicator-network? | 16:49 |
Saviq | dednick, one thing I'm not clear on is will we still have/use the same password page? | 16:50 |
dednick | Saviq: i dont think it's finished yet. | 16:53 |
dednick | Saviq: and i dont think we will need the password page after it is. | 16:53 |
dednick | tedg: ^ | 16:53 |
tedg | pete-woods, ^ | 16:55 |
tedg | :-) | 16:55 |
dednick | Saviq: : isnt this enter password dialog we need the same as the extended snap descisions MacSlow is working on? | 17:00 |
pete-woods | tedg: I don't know the answer to this | 17:01 |
Saviq | dednick, I think not, we need both an in-indicator way and an out-of-indicator way | 17:01 |
Saviq | dednick, i.e. when you select a network in the indicator, you log in in the indicator | 17:01 |
dednick | Saviq: i thought it was a dialog popup | 17:02 |
Saviq | dednick, you only get the snap decision if you get disconnected or password changes etc. | 17:02 |
Saviq | dednick, do we have design for it? | 17:02 |
pete-woods | the question I care about is should the network-indicator be pinging unity for a password prompt? | 17:02 |
pete-woods | or should unity just be a "secret agent" (as it it currently) that naturally gets asked by the network manager for credentials? | 17:03 |
tedg | Saviq, That's not part of the design now, no in indicator way. | 17:03 |
Saviq | tedg, so no password entry in indicator? | 17:04 |
tedg | Saviq, Yes | 17:04 |
Saviq | dednick, ok ↑ | 17:04 |
tedg | So I think Unity shouldn't be a secret agent. | 17:04 |
Saviq | +1 | 17:04 |
pete-woods | so should there be a single secret agent? whose only job is to ask unity to pop up snap decisions? | 17:05 |
Saviq | tedg, pete-woods, something (network-indicator I'd say?) will request a password-entry snap decision | 17:05 |
Saviq | I believe nm-applet did it until now? | 17:05 |
dednick | tedg: so how do you connect with password? dialog? | 17:05 |
Saviq | dednick, yeah, snap decision | 17:05 |
tedg | Saviq, I thought we wanted to use the system dialog stuff instead of snap decision? | 17:05 |
dednick | ok. thought so | 17:05 |
Saviq | dednick, let's not use "dialog" - we got bitten by it anyway | 17:06 |
Saviq | tedg, it *is* a snap decision | 17:06 |
Saviq | tedg, there is no system dialogs | 17:06 |
Saviq | tedg, there are only snap decisions | 17:06 |
tedg | The problem is that the buttons need to be interactive. i.e., the button isn't sensitive until the right number of characters is entered. | 17:06 |
dednick | so is pete-woods and MacSlow doing the same thing then? | 17:06 |
Saviq | tedg, that's why we're going for a snap decision with a side-channel through *MenuModel | 17:06 |
tedg | Saviq, AH, I didn't realize. Where is that? | 17:07 |
pete-woods | dednick: I'm doing nothing to do with the GUI, I'm putting together a daemon that asks unity for a snap decision with a list of the wifi networks | 17:07 |
dednick | in the ether | 17:07 |
Saviq | tedg, in MacSlow's plans ;) | 17:07 |
dednick | pete-woods: ah. ok | 17:07 |
Saviq | tedg, he's investigationg it now | 17:07 |
Saviq | -o | 17:07 |
Saviq | or really starting to implement itnow | 17:07 |
Saviq | pete-woods, same thing should probably ask for the password | 17:08 |
dednick | Saviq: ok, so there's your status. we can display the networks but cannot connect from indicators. So we can't replace for awhile i guess. | 17:08 |
pete-woods | tedg: atm there's indicator-network and indicator-network-prompt, both of which will need to ask unity for a snap decision regarding passwords | 17:09 |
Saviq | dednick, yeah, hopefully this won't take long | 17:09 |
Saviq | gtg o/ | 17:09 |
pete-woods | tedg: does it make sense to have a common thing that is the secret agent? | 17:09 |
tedg | pete-woods, ? What is indicator-network-prompt? | 17:09 |
pete-woods | it's what I'm temporarily calling the new thing you asked me to make | 17:10 |
pete-woods | tedg: it can be called whatever you want really? | 17:10 |
tedg | Let's call it Jennifer! | 17:10 |
tedg | ;-) | 17:10 |
pete-woods | :) | 17:10 |
=== salem_ is now known as _salem | ||
tedg | Ah, okay. I didn't realize | 17:11 |
tedg | I guess I figured the prompt thing would be something that indicator-network would call, and also that nm would call. | 17:11 |
pete-woods | tedg: I just don't want to have lots of places that implement the secret agent, really | 17:11 |
tedg | Kinda the hub | 17:11 |
tedg | Why does indicator-network-service need to do a snapdecision? | 17:11 |
pete-woods | well, if unity isn't going to be an agent any more, something needs to do the snap decision for the credentials// | 17:12 |
tedg | Wouldn't that be indicator-network-prompt? | 17:12 |
pete-woods | but what about when then user picks a network? | 17:13 |
pete-woods | I thought the prompt daemon was supposed to be short-lived | 17:13 |
tedg | So indicator-network-service would say "NM Choose this one" and then NM goes all "but I don't have a password, secret agent!" and then that, like, starts the prompt utility. | 17:13 |
* tedg isn't sure if you can do a valley accent in IRC | 17:14 | |
pete-woods | :p | 17:14 |
pete-woods | does nm actually start agents? I thought they had to be running and already registered? | 17:14 |
tedg | I'm not sure on that. | 17:16 |
pete-woods | I was kinda expecting there to be a really tiny agent managed by upstart | 17:16 |
tedg | I thought they had to be registered in the system, not at runtime. | 17:16 |
pete-woods | who lived throughout the session | 17:16 |
tedg | If it has to be running throughout the session it probably makes sense to just put it in indicator-network-service. | 17:17 |
=== dandrader|lunch is now known as dandrader | ||
tedg | I for some reason thought that you couldn't "just register" as then anything could do it. | 17:17 |
pete-woods | tedg: there's actually a stub of one on there - maybe alberto was working on it before he left? | 17:18 |
pete-woods | tedg: best docs I can find on agents so far: https://projects.gnome.org/NetworkManager/developers/api/09/ref-migrating.html | 17:18 |
pete-woods | tedg: it looks to me like it's 'just' some dbus calls? but then those docs could be out of date / misleading | 17:19 |
tedg | Yeah, it just seems to easy for someone to create a phantom service if that's the case. But perhaps it is. | 17:20 |
pete-woods | tedg: yep - I'd agree with you there - evil app X could fill in all my password prompts for me with garbage | 17:20 |
tedg | pete-woods, Hmm, yeah, it seems like that's the case. A bit scary. | 17:22 |
=== _salem is now known as salem_ | ||
pete-woods | tedg: yep, anyway - I've gotta go now, but hopefully this will tick over in your brain tonight :) | 17:22 |
tedg | pete-woods, Yup, have a good night. Could you also ping MacSlow in the morning? | 17:23 |
pete-woods | tedg: sure! | 17:23 |
tedg | pete-woods, See where he is there. Perhaps there's something we can get started with. | 17:23 |
pete-woods | tedg: hopefully! | 17:23 |
=== iveand is now known as rik-shaw | ||
TheMuso | mzanetti: Nothing, I'm still new to Qt, and I thought our platform had something that I should have used. | 21:51 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!