/srv/irclogs.ubuntu.com/2013/07/31/#ubuntu-unity.txt

=== _salem is now known as salem_
TheMusoWhat 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
mzanettiTheMuso: QApplication07:13
mzanetti(I guess)07:13
TheMusomzanetti: I was thinking that was the case, but wanted to be sure, given I am writing C++ code to provide qml bindings.07:28
tsdgeosmzanetti: ping07:37
mzanettitsdgeos: hey07:57
mzanettiTheMuso: so? what's the problem with QApplication?07:57
tsdgeosmzanetti: can you read alan's comments at https://codereview.qt-project.org/#change,60535 and maybe answer?07:57
mzanettitsdgeos: on it07:57
tsdgeosmzanetti: if it helps you qquickitemview.cpp has07:58
tsdgeos/###Possibly rename these properties, since they are very useful even without a highlight?07:58
tsdgeosqreal QQuickItemView::preferredHighlightBegin() const07:58
tsdgeosalso i am not sure if parts of his comment are against the logic of the qml or the logic of the patch :_/07:59
mzanettitsdgeos: I don't understand what he means07:59
tsdgeos" That test case looks suspicious, because it's not setting a highlightRangeMode yet is setting preferredHighlightRange." ?08:00
mzanettitsdgeos: ? where is that?08:00
tsdgeosprevious comment08:00
mzanettitsdgeos: 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
tsdgeosexpand the other comment08:00
mzanettitsdgeos: 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 it08:02
tsdgeosok08:03
tsdgeostbh i don't understand the rest of his comments either08:03
mzanettitsdgeos: should I describe my use case as a comment here?08:03
tsdgeosmzanetti: it may help08:05
mzanettitsdgeos: is it a good idea if I +1 it?08:10
tsdgeosnot really sure08:11
mzanettitsdgeos: commented.08:17
tsdgeostx08:17
tsdgeosmzanetti: i'd leave the +1 for "i do understand the code and like it"08:17
tsdgeosotherwise it may seem we are a doing as a "gang push"08:17
mzanettitsdgeos: which kinda is the case for me... but still... the "gang push" is what concerns me08:18
mzanettitsdgeos: he suggested to remove the preferredHighlightBegin/End (aka disable snapping), right?08:19
tsdgeoswell, he said "the bug" doesn't happen without the preferredHighlightBegin/End08:20
tsdgeosyes08:20
tsdgeosproblem is preferredHighlightBegin/End is a bad name i find08:20
tsdgeosas the comment in the file says08:20
mzanettiagreed. should be snapPositionBegin/End08:20
mzanettior something along that08:21
mzanettiactually, this use case happens always if you want to have something like a CoverFlow with snapping in the middle08:21
dednicktsdgeos, mzanetti, MacSlow, Saviq: anyone available to do a quick device test for me?08:54
MacSlowdednick, sorry... in the middle of ap-testing for notifications08:54
MacSlowdednick, I can do it afterwards... what do you need to have checked?08:55
MacSlowdednick, does the installed image matter?08:55
dednickMacSlow: testing if sound indicator updating volume for apps. dont know if image matters.08:56
tsdgeosdednick: i have the unity-mir stuff, would take me quite a while to go back08:56
tsdgeosbetter if someone can do08:57
dednickMacSlow: after a fresh flash the volume will be affected, but as soon as i restart phone, it stops changing.08:57
tsdgeossomeone *else* that is :D08:57
dednicktsdgeos: no worries.08:57
larsudednick: good morning. I have a first version of indicator-messages that uses an indicator file at lp:~larsu/indicator-messages/phablet-port-indicator-ng09:00
mzanettidednick: can help you I guess09:00
larsudednick: it's not thoroughly tested yet, but you can give it a spin if you're up for it09:00
dednicklarsu: \o/ . will give it a go today09:00
larsudednick: (I'll be on a train for the rest of the day)09:00
dednicklarsu: i've made some more updates to qmenumodel. added impl for 'action name' and 'update state'.09:01
dednicklarsu: although i didnt know how you were handing changes to the gtkmenu area. i realise it's from another project...09:02
dednicklarsu: https://code.launchpad.net/~nick-dedekind/qmenumodel/add-menumenumodel.action-name.update-state09:02
larsudednick: neat! Can you MR them onto my branch? I'll be at guadec, so I probably won't be online a lot.09:03
dednicklarsu: ok. no prob09:03
larsudednick: cool, talk to you later. Gotta catch a train :)09:03
dednickmzanetti: cool. do you have the video scope assets on your device?09:08
mzanettidednick: the demo asses, yes09:08
dednickmzanetti: you mind checking if changing the volume in the indicator updates the volume of the video?09:09
mzanettidednick: do I need todays image?09:10
dednickmzanetti: nope09:10
mzanettidednick: 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 install09:11
dednickmzanetti: ok. thanks.09:11
mzanettidednick: I did another test which tells me it won't work for the videos09:12
mzanettidednick: I logged in and ran "alsamixer". Changing the indicators volume does change the alsamixer's sliders09:12
mhr3sil2100, any success on lucene++? pls brief jamesh ^09:13
dednickmzanetti: it shouldn't?09:13
mzanettidednick: d'oh! "... does NOT change the alsamixer's sliders"09:13
mzanettidednick: yes, it should I'd say09:13
dednickmzanetti: :) haha. ok09:13
sil2100mhr3, jamesh: yes, I built the package locally and in my PPA, just need to tweak it a bit09:13
tsdgeosmzanetti: does my reply to the snapping patch make sense to you?09:14
* mzanetti reads09:14
dednickmzanetti: presumably you've restarted your device since you last flahed?09:14
mzanettidednick: a 100 times when testing the SIM pin :D09:14
jameshsil2100: there was some packaging in the hollywood-team/staging PPA, if that helps09:14
dednickmzanetti: ok thanks. confirms my suspicions.09:14
mzanettitsdgeos: yep. makes sense to me09:15
tsdgeosokki09:15
Saviqdednick, mzanetti does the video preview work for you at all?09:15
dednickSaviq: sometimes09:15
mhr3sil2100, 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
mzanettiSaviq: dunno. still don't have a video. just tested with alsamixer09:15
dednickSaviq: it's a bit hit and miss. sometimes get a white page, sometimes black, sometimes works09:18
mzanettiis unity8 always running on ~20% cpu on the desktop for you too?09:18
Saviqdednick, that's on device?09:18
dednickSaviq: ya09:18
Saviqdednick, to me it looks like carousel is broken, grid is fine09:18
dednickSaviq: oh, you mean the video selection widget?09:19
Saviqdednick, in Video apps09:19
dednickSaviq: that seems to be ok for me.09:19
Saviqs/apps/dash/09:19
dednickSaviq: although i might be using mock video09:19
Saviqdednick, yeah, that's what I mean09:19
dednickSaviq: it's fine. I thought you meant the mediaplayer-app.09:20
Saviqdednick, mock video scope previews for carousel are broken09:20
dednickSaviq: that's how i've been testing my sound issues.09:21
Saviqdednick, which version?09:21
Saviqdednick, yeah, that's what I wanted, too ;)09:21
dednickSaviq: latest09:21
Saviqdednick, but the previews for my carousel are broken ;)09:21
dednickSaviq: doh.09:22
dednickSaviq: what does it show?09:22
Saviqdednick, an empty shape and stars09:23
Saviqdednick, http://pastebin.ubuntu.com/5932127/ is the console output09:24
Saviqpaulliu, ↑09:24
Saviqdednick, found it :/09:25
Saviqhttp://pastebin.ubuntu.com/5932135/09:26
Saviqmhr3, ↑ your fault again ;P09:26
dednicklol09:26
dednickwonder why mine is working09:27
dednickah. maybe that isnt in image?09:27
mhr3Saviq, stop using those mocks! :P09:28
Saviqmhr3, supply me with matching backends!09:28
mhr3ehm, touche09:28
dednickholy crap. my nexus7 spazed out09:28
dednickdoesnt want to flash :(09:29
Saviqdednick, if you didn't flash with --pending, then you're on a 0727 image09:29
Saviq072609:29
dednickSaviq: is that a bad thing?09:29
Saviqdednick, we generally should be running --pending09:30
Saviqdednick, to be on the edge09:30
=== alan_g is now known as alan_g|tea
dednicklol. the 'danger zone'09:30
Saviqdednick, mzanetti https://code.launchpad.net/~saviq/unity8/fix-mock-videopreview/+merge/177777 please09:30
Saviqnice merge number ;)09:30
dednickheh09:31
mhr3Saviq, are we ok with being unable to close apps once we have a "proper" running apps scope?09:34
mhr3Saviq, or will you keep a hack in the renderer?09:34
=== alan_g|tea is now known as alan_g
tsdgeosSaviq: did you give the branches i posted yesterday a try?09:37
dednickSaviq: we using pulse on device now?09:38
tsdgeosdednick: we've always been using pulse as far as i know09:39
dednickSaviq, tsdgeos: any idea why there is a phablet-team ppa for pusleaudio?09:40
tsdgeoslast time i checked there was some small extra patch there09:40
tsdgeosdon't really remember what it was about09:40
dednicktsdgeos: ok09:40
=== alan_g is now known as alan_g|afk
=== alan_g|afk is now known as alan_g
tsdgeosMacSlow: you guys need a copyright header in https://code.launchpad.net/~macslow/unity8/notification-autopilot-tests-dbus/+merge/17778011:12
Cimiseb128, ping :)11:19
seb128Cimi, hey11:19
Cimiseb128, I'm writing the welcome wizard for the phone, I'm wondering how can I fetch the list of the locales11:20
seb128Cimi, look at the language panel in system settings11:20
Cimiok11:21
Cimiseb128, lp?11:21
seb128Cimi: lp:ubuntu-system-settings plugins/language/language-plugin.cpp getLocales()11:21
Cimiseb128, thanks11:34
Cimiseb128, saw it11:34
seb128Cimi, yw11:34
Cimiseb128, you think I could import it?11:34
Cimiseb128, it sucks to duplicate code11:34
seb128Cimi, I'm not sure what are the best practices in qml there...11:34
seb128but it would make sense to share the code yes11:34
Cimiseb128, if the plugin is locally installed11:34
seb128not sure how though11:34
Cimiseb128, might import it11:34
seb128it's in a private dir, so not sure if you can import it easily (out of hardcoding the path to it)11:35
seb128ideally we would have e.g indicator-keyboard-service to publish that list over gmenu or something11:36
tsdgeosSaviq: are we having the "UnityNext team product review" hangout today with kgunn or shall we just do the usual stand-up11:36
Saviqtsdgeos, usual ,we're in with Jane at that time11:37
tsdgeosoka11:37
=== ashams_ is now known as ashams
Cimiseb128, but I need to set this in the welcome wizard11:46
Cimiseb128, we have wifi, location setting, username and language11:47
Cimiseb128, would be great to have set them without duplicating code11:47
seb128Cimi, that seems quite some common work with system setting/indicators11:47
Cimiexactly11:47
seb128the idea so far was to have the indicator backends to provide those features11:47
seb128through gmenumodel11:47
Cimiwho knows here?11:47
Cimilarsu / dednick ?11:47
seb128and use unitymenumodel11:47
seb128to do the qt UIs11:47
seb128yes11:48
seb128larsu is in a train today, travelling to GUADEC11:48
seb128so dednick11:48
dednickyo11:48
dednickoh. that was a statement.11:49
dednickCimi: what is the welcome wizard?11:50
dednickCimi: intial device setup?11:50
Cimidednick, yes11:50
bregmasoooo, all the ibus-related failures in the Unity 7 daily were expected, in case anyone was wondering....11:53
dednickCimi: 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
CimiI can do mock now11:55
Cimiwaiting for a model...11:55
seb128bregma, the fixes are ready waiting to land as well? ;-)11:56
=== alan_g is now known as alan_g|lunch
seb128dednick, well, the idea was that the indicators would have a settings profil11:57
seb128Cimi, dednick: they can also have a wizard profile11:57
seb128we already have profiles for desktop, phone, greeter, installer11:57
bregmaseb128, 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 something11:58
dednickseb128: i c11:58
dednickjust trying to think of the best way to mock it.11:59
bregmaseb128, yes, the problem is the new ibus 1.5 went in to the archive but the required Nux patches are still in review11:59
dednickCimi: you could probably get away with forgetting the mock for now and just using the current profiles for testing.12:02
Cimidednick, some code I should have a look at?12:02
dednickCimi: 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 honest12:03
dednickCimi: if you want to mock, easiest approach may be to use vala to generate the code.12:03
Cimidednick, yep but which code shall I mock?12:04
Cimidednick, unity?12:04
dednickCimi: but if you want to use the current indicator profiles, the ui code to pick them up and process is all in plugins/Unity/Indicators12:04
Cimiok12:04
Cimidednick, might need to import this plugin from the external welcomewizard12:05
Cimiunless we want the welcome wizard to be part of unity12:05
dednickCimi: the indicator backends are separate projects12:05
dednickCimi: hm. should be able to import the qml library ok...12:07
dednickCimi: you can probably use all the page code as well.12:08
=== hikiko is now known as hikiko|lunch
dednickCimi: 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
mzanettiSaviq: we're not having the hangout today, do we?12:49
Saviqmzanetti, no12: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_
seb128bregma, do you know if there is a bug/known issue about launcher emblems showing only on one monitor?13:17
seb128bregma, e.g the tb badge for new emails13:17
dandradermzanetti, there's a nuisance with your bzr plugin: it runs in any bzr repository, not just unity813:50
=== jhodapp|afk is now known as jhodapp
bregmaseb128, sounds vaguely familiar but I can't seem to be able to find the bug13:51
mzanettidandrader: huh? I throught that should be fixed14:03
mzanettidandrader: it checks if there is a Shell.qml in the repositories root14:04
dandradermzanetti, ahhh... that's because my minimal test case does have a Shell.qml in its root dir :D14:05
mzanettiheh14:05
mzanettidandrader: the thing is, bzr repository branches can have any name and any origin. so its not possible to check on those either14:06
mzanettiI know that checking for Shell.qml is not really nice14:06
mzanettiseb128: ping14:09
seb128mzanetti, hey14:10
mzanettiseb128: how far are you with the SIM pin stuff?14:10
mzanettiseb128: did you already check out the ofono api?14:10
sil2100mhr3_: hi! I pushed the latest version of lucene++ for building and updated the needs-packaging bug with all the required links to branches14:10
sil2100mhr3_: so now just for some MOTU guys to review14:11
seb128mzanetti, kenvandine has been looking at the SIM stuff so far (he did the SIM services yesterday), I don't think anyone started on the pin14:11
sil2100mhr3_: https://bugs.launchpad.net/ubuntu/+bug/103681214:11
seb128mzanetti, no, I didn't yet14:11
ubot5Launchpad bug 1036812 in Ubuntu "[needs-packaging] Lucene++" [Wishlist,In progress]14:11
seb128mzanetti, why?14:11
mzanettiseb128: for some reason it the amount of available pin retries does not work14:11
sil2100mhr3_: I'll have some of my guys do a quick packaging review as well before that14:11
mzanettiseb128: while I'm sure my SIM card suports it, calling this: qdbus --system --literal org.ofono /ril_0 org.ofono.SimManager.GetProperties14:11
mzanettiseb128: just returns an empty list14:11
kenvandinei haven't looked at SIM pin stuff at all, mine doesn't use a pin14:13
kenvandinebut libofono-qt does have an API for unlock/lock14:13
dandraderfound a leak!14:35
dandraderin Dash/Video/VideoInfo.qml14:35
dandraderremove it and memory stops growing14:35
dandradernow gotta find why14:35
=== alan_g is now known as alan_g|tea
tsdgeosXmlListModel ?14:37
tsdgeosseems like it probably has lots of C++ code behind :D14:38
dandradertsdgeos, yeah, it must be this component14:39
=== alan_g|tea is now known as alan_g
mzanettiMacSlow: the script is executable?14:58
MacSlowmzanetti, yes14:58
MacSlowmzanetti, that's wrong?!14:59
mzanettino... that would be correct14:59
MacSlowmzanetti, hm... but there's a hashbang line in it15:00
MacSlowmzanetti, maybe that's causing the issue?15:00
mzanettiwhich one is it?15:01
MacSlowmzanetti, "#!/usr/bin/env python" is used in create_interactive_notification.py although that's not were the error is reported15:03
mzanettiMacSlow: hmm... still weird... try changing it to /usr/bin/python15:04
mzanettiMacSlow: but it I don't think this is it15:05
mzanettiMacSlow: ~macslow/unity8/notification-autopilot-tests/ ?15:07
MacSlowmzanetti, no lp:~macslow/unity8/notification-autopilot-tests-dbus15:07
mzanettiMacSlow: oh... its the setsid15:10
mzanettiMacSlow: is there a reason its required?15:11
MacSlowmzanetti, 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
tsdgeoshmmm15:13
* tsdgeos realizes the "AddressBook" app is trying to do something like the LVPWH15:14
MacSlowmzanetti, not really... tests work without it... but they seem to take longer to start15:14
tsdgeosand it's obviously failing15:14
tsdgeossince we failed at it too in QML :D15:14
MacSlowmzanetti, but I could just imagine this taking more time15:14
mzanettiMacSlow: try it without it please. if that doesn't work we can enable stuff in jenkins15:14
mzanettitsdgeos: tell renato15:14
mzanettithat was close... I entered "sudo reboot" and only the password not being "phablet" protected me from rebooting my notebook instead of the phone15:16
tsdgeos:D15:16
MacSlowmzanetti, seems to work (locally) without that culprit line... I'll delete it can re-commit/push15:16
mzanettiMacSlow: ack15:16
sil2100pstolowski: hi!15:50
pstolowskisil2100: hey!15:50
sil2100pstolowski: you know if mhr3 will still be around today?15:50
pstolowskisil2100: he won't, he is attending guadec15:51
pstolowskisil2100: he has few days off, but he may show up from time to time on irc15:51
sil2100Ah, ok :)15:51
sil2100pstolowski: thanks!15:52
sil2100jamesh: hi!15:52
pstolowskisil2100: yw15:52
jameshsil2100: hi.15:57
jameshI was just about to head off to bed, but if it's something quick, I'm here :)15:58
sil2100jamesh: I remember mhr3 saying that you're interested in lucene++ in Ubuntu? ;)15:58
jameshsil2100: I'm interested in getting the Media scanner in Ubuntu, and lucene++ is a dependency15:58
sil2100jamesh: are there any specific tests you would like to run to see if the lucene++ that we're proposing to Ubuntu works?15:59
sil2100jamesh: since I prepared lucene++ packaging, basing on work of Openismus and kenvandine's, and kenvandine will probably sponsor it today to universe15:59
pstolowskisil2100, jamesh: I think it should go to main?16:01
kenvandinepstolowski, if something in main will depend on it, then we'll need to get an MIR done16:02
kenvandinebut first step is universe16:02
sil2100pstolowski: we can get it into main later16:02
jameshsil2100: 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 Saucy16:02
sil2100pstolowski: first universe, then MIR to main if needed16:02
sil2100jamesh: ok16:02
sil2100jamesh: then we'll get what we have into Ubuntu16:02
jameshI think we'll want this in main eventually, since it'll be going on the phone if everything goes to plan16:03
pstolowskikenvandine, sil2100: I see, sounds good, thanks16:05
Saviqdandrader, hey, since it's your last day, can you post an update on anything you've found about bug #1201116?16:14
ubot5bug 1201116 in Unity 8 "[dash] constant increate in memory consumption when navigating through lenses" [Critical,In progress] https://launchpad.net/bugs/120111616:14
Saviqdandrader, the CPU issue seems to be more pressing than mem, too16:14
dandraderSaviq, sure. I'm just trimming my test case to a bare minimum16:14
dandraderSaviq, did you report a separate bug on the CPU issue?16:15
Saviqdandrader, no, it's the same bug16:15
Saviqdandrader, ah, you wanted to separate it out16:15
dandraderSaviq, well, not anymore, I've renamed it :)16:15
dandraderSaviq, and you told there you would report one. stand by your word! :)16:15
Saviqdandrader, ;P16:16
dandraderit's shame we don't have bug dependencies in launchpad...16:16
Saviqdandrader, bug #1206991 ;)16:18
ubot5bug 1206991 in Unity 8 "Frequent CPU hogging" [Critical,Triaged] https://launchpad.net/bugs/120699116:18
pete-woodsseb128: unfortunately that patch has no effect when applied to the hg files in the ubuntu package - the package build doesn't regenerate the generate files16:18
pete-woods*generated16:18
pete-woodseither we need to make the debuild build generate the .cc files, or we need to patch the .cc files directly16:19
pete-woodsseb128: I have an updated patch that patches the .cc files if you want it?16:26
seb128pete-woods, otp, but yes please16:26
dandraderSaviq, bug updated16:36
pete-woodsseb128: whenever you are ready, I put the patch against this bug https://bugs.launchpad.net/ubuntu/+source/glibmm2.4/+bug/1206997 :)16:38
ubot5Launchpad bug 1206997 in glibmm2.4 (Ubuntu) "Updated patch for generated .cc files" [Undecided,New]16:38
seb128pete-woods, thanks16:39
Saviqdandrader, thanks, and talk to you in a month! ;)16:39
Saviqdandrader, hmm... no comment in bug #1206991 ?16:40
ubot5bug 1206991 in Unity 8 "Frequent CPU hogging" [Critical,Triaged] https://launchpad.net/bugs/120699116:40
=== dandrader_ is now known as dandrader|lunch
Saviqdednick, hey, can you give me a small update on what do we still need to transition to indicator-network?16:49
Saviqdednick, one thing I'm not clear on is will we still have/use the same password page?16:50
dednickSaviq: i dont think it's finished yet.16:53
dednickSaviq: and i dont think we will need the password page after it is.16:53
dednicktedg: ^16:53
tedgpete-woods, ^16:55
tedg:-)16:55
dednickSaviq: : isnt this enter password dialog we need the same as the extended snap descisions MacSlow is working on?17:00
pete-woodstedg: I don't know the answer to this17:01
Saviqdednick, I think not, we need both an in-indicator way and an out-of-indicator way17:01
Saviqdednick, i.e. when you select a network in the indicator, you log in in the indicator17:01
dednickSaviq: i thought it was a dialog popup17:02
Saviqdednick, you only get the snap decision if you get disconnected or password changes etc.17:02
Saviqdednick, do we have design for it?17:02
pete-woodsthe question I care about is should the network-indicator be pinging unity for a password prompt?17:02
pete-woodsor should unity just be a "secret agent" (as it it currently) that naturally gets asked by the network manager for credentials?17:03
tedgSaviq, That's not part of the design now, no in indicator way.17:03
Saviqtedg, so no password entry in indicator?17:04
tedgSaviq, Yes17:04
Saviqdednick, ok ↑17:04
tedgSo I think Unity shouldn't be a secret agent.17:04
Saviq+117:04
pete-woodsso should there be a single secret agent? whose only job is to ask unity to pop up snap decisions?17:05
Saviqtedg, pete-woods, something (network-indicator I'd say?) will request a password-entry snap decision17:05
SaviqI believe nm-applet did it until now?17:05
dednicktedg: so how do you connect with password? dialog?17:05
Saviqdednick, yeah, snap decision17:05
tedgSaviq, I thought we wanted to use the system dialog stuff instead of snap decision?17:05
dednickok. thought so17:05
Saviqdednick, let's not use "dialog" - we got bitten by it anyway17:06
Saviqtedg, it *is* a snap decision17:06
Saviqtedg, there is no system dialogs17:06
Saviqtedg, there are only snap decisions17:06
tedgThe 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
dednickso is pete-woods and MacSlow doing the same thing then?17:06
Saviqtedg, that's why we're going for a snap decision with a side-channel through *MenuModel17:06
tedgSaviq, AH, I didn't realize.  Where is that?17:07
pete-woodsdednick: 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 networks17:07
dednickin the ether17:07
Saviqtedg, in MacSlow's plans ;)17:07
dednickpete-woods: ah. ok17:07
Saviqtedg, he's investigationg it now17:07
Saviq-o17:07
Saviqor really starting to implement itnow17:07
Saviqpete-woods, same thing should probably ask for the password17:08
dednickSaviq: 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-woodstedg: atm there's indicator-network and indicator-network-prompt, both of which will need to ask unity for a snap decision regarding passwords17:09
Saviqdednick, yeah, hopefully this won't take long17:09
Saviqgtg o/17:09
pete-woodstedg: does it make sense to have a common thing that is the secret agent?17:09
tedgpete-woods, ?  What is indicator-network-prompt?17:09
pete-woodsit's what I'm temporarily calling the new thing you asked me to make17:10
pete-woodstedg: it can be called whatever you want really?17:10
tedgLet's call it Jennifer!17:10
tedg;-)17:10
pete-woods:)17:10
=== salem_ is now known as _salem
tedgAh, okay.  I didn't realize17:11
tedgI guess I figured the prompt thing would be something that indicator-network would call, and also that nm would call.17:11
pete-woodstedg: I just don't want to have lots of places that implement the secret agent, really17:11
tedgKinda the hub17:11
tedgWhy does indicator-network-service need to do a snapdecision?17:11
pete-woodswell, if unity isn't going to be an agent any more, something needs to do the snap decision for the credentials//17:12
tedgWouldn't that be indicator-network-prompt?17:12
pete-woodsbut what about when then user picks a network?17:13
pete-woodsI thought the prompt daemon was supposed to be short-lived17:13
tedgSo 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 IRC17:14
pete-woods:p17:14
pete-woodsdoes nm actually start agents? I thought they had to be running and already registered?17:14
tedgI'm not sure on that.17:16
pete-woodsI was kinda expecting there to be a really tiny agent managed by upstart17:16
tedgI thought they had to be registered in the system, not at runtime.17:16
pete-woodswho lived throughout the session17:16
tedgIf 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
tedgI for some reason thought that you couldn't "just register" as then anything could do it.17:17
pete-woodstedg: there's actually a stub of one on there - maybe alberto was working on it before he left?17:18
pete-woodstedg: best docs I can find on agents so far: https://projects.gnome.org/NetworkManager/developers/api/09/ref-migrating.html17:18
pete-woodstedg: it looks to me like it's 'just' some dbus calls? but then those docs could be out of date / misleading17:19
tedgYeah, it just seems to easy for someone to create a phantom service if that's the case.  But perhaps it is.17:20
pete-woodstedg: yep - I'd agree with you there - evil app X could fill in all my password prompts for me with garbage17:20
tedgpete-woods, Hmm, yeah, it seems like that's the case.  A bit scary.17:22
=== _salem is now known as salem_
pete-woodstedg: yep, anyway - I've gotta go now, but hopefully this will tick over in your brain tonight :)17:22
tedgpete-woods, Yup, have a good night.  Could you also ping MacSlow in the morning?17:23
pete-woodstedg: sure!17:23
tedgpete-woods, See where he is there.  Perhaps there's something we can get started with.17:23
pete-woodstedg: hopefully!17:23
=== iveand is now known as rik-shaw
TheMusomzanetti: 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!