[03:36] <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.
[07:13] <mzanetti> TheMuso: QApplication
[07:13] <mzanetti> (I guess)
[07:28] <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:37] <tsdgeos> mzanetti: ping
[07:57] <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:58] <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:59] <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
[08:00] <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:02] <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:03] <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:05] <tsdgeos> mzanetti: it may help
[08:10] <mzanetti> tsdgeos: is it a good idea if I +1 it?
[08:11] <tsdgeos> not really sure
[08:17] <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:18] <mzanetti> tsdgeos: which kinda is the case for me... but still... the "gang push" is what concerns me
[08:19] <mzanetti> tsdgeos: he suggested to remove the preferredHighlightBegin/End (aka disable snapping), right?
[08:20] <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:21] <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:54] <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:55] <MacSlow> dednick, I can do it afterwards... what do you need to have checked?
[08:55] <MacSlow> dednick, does the installed image matter?
[08:56] <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:57] <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.
[09:00] <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:01] <dednick> larsu: i've made some more updates to qmenumodel. added impl for 'action name' and 'update state'.
[09:02] <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:03] <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:08] <dednick> mzanetti: cool. do you have the video scope assets on your device?
[09:08] <mzanetti> dednick: the demo asses, yes
[09:09] <dednick> mzanetti: you mind checking if changing the volume in the indicator updates the volume of the video?
[09:10] <mzanetti> dednick: do I need todays image?
[09:10] <dednick> mzanetti: nope
[09:11] <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:12] <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:13] <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:14] <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:15] <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:18] <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:19] <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:20] <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:21] <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:22] <dednick> Saviq: doh.
[09:22] <dednick> Saviq: what does it show?
[09:23] <Saviq> dednick, an empty shape and stars
[09:24] <Saviq> dednick, http://pastebin.ubuntu.com/5932127/ is the console output
[09:24] <Saviq> paulliu, ↑
[09:25] <Saviq> dednick, found it :/
[09:26] <Saviq> http://pastebin.ubuntu.com/5932135/
[09:26] <Saviq> mhr3, ↑ your fault again ;P
[09:26] <dednick> lol
[09:27] <dednick> wonder why mine is working
[09:27] <dednick> ah. maybe that isnt in image?
[09:28] <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:29] <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:30] <Saviq> dednick, we generally should be running --pending
[09:30] <Saviq> dednick, to be on the edge
[09:30] <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:31] <dednick> heh
[09:34] <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:37] <tsdgeos> Saviq: did you give the branches i posted yesterday a try?
[09:38] <dednick> Saviq: we using pulse on device now?
[09:39] <tsdgeos> dednick: we've always been using pulse as far as i know
[09:40] <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
[11:12] <tsdgeos> MacSlow: you guys need a copyright header in https://code.launchpad.net/~macslow/unity8/notification-autopilot-tests-dbus/+merge/177780
[11:19] <Cimi> seb128, ping :)
[11:19] <seb128> Cimi, hey
[11:20] <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:21] <Cimi> ok
[11:21] <Cimi> seb128, lp?
[11:21] <seb128> Cimi: lp:ubuntu-system-settings plugins/language/language-plugin.cpp getLocales()
[11:34] <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:35] <seb128> it's in a private dir, so not sure if you can import it easily (out of hardcoding the path to it)
[11:36] <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:37] <Saviq> tsdgeos, usual ,we're in with Jane at that time
[11:37] <tsdgeos> oka
[11:46] <Cimi> seb128, but I need to set this in the welcome wizard
[11:47] <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:48] <seb128> yes
[11:48] <seb128> larsu is in a train today, travelling to GUADEC
[11:48] <seb128> so dednick
[11:48] <dednick> yo
[11:49] <dednick> oh. that was a statement.
[11:50] <dednick> Cimi: what is the welcome wizard?
[11:50] <dednick> Cimi: intial device setup?
[11:50] <Cimi> dednick, yes
[11:53] <bregma> soooo, all the ibus-related failures in the Unity 7 daily were expected, in case anyone was wondering....
[11:54] <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:55] <Cimi> I can do mock now
[11:55] <Cimi> waiting for a model...
[11:56] <seb128> bregma, the fixes are ready waiting to land as well? ;-)
[11:57] <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:58] <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:59] <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
[12:02] <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:03] <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:04] <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:05] <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:07] <dednick> Cimi: hm. should be able to import the qml library ok...
[12:08] <dednick> Cimi: you can probably use all the page code as well.
[12:13] <dednick> Cimi: will need some modifications to be able to load different profiles. at the moment it's hardcoded to load phone.
[12:17]  * Cimi <--- lunch 
[12:49] <mzanetti> Saviq: we're not having the hangout today, do we?
[12:49] <Saviq> mzanetti, no
[13:17] <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:50] <dandrader> mzanetti, there's a nuisance with your bzr plugin: it runs in any bzr repository, not just unity8
[13:51] <bregma> seb128, sounds vaguely familiar but I can't seem to be able to find the bug
[14:03] <mzanetti> dandrader: huh? I throught that should be fixed
[14:04] <mzanetti> dandrader: it checks if there is a Shell.qml in the repositories root
[14:05] <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:06] <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:09] <mzanetti> seb128: ping
[14:10] <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:11] <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] <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:13] <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:35] <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:37] <tsdgeos> XmlListModel ?
[14:38] <tsdgeos> seems like it probably has lots of C++ code behind :D
[14:39] <dandrader> tsdgeos, yeah, it must be this component
[14:58] <mzanetti> MacSlow: the script is executable?
[14:58] <MacSlow> mzanetti, yes
[14:59] <MacSlow> mzanetti, that's wrong?!
[14:59] <mzanetti> no... that would be correct
[15:00] <MacSlow> mzanetti, hm... but there's a hashbang line in it
[15:00] <MacSlow> mzanetti, maybe that's causing the issue?
[15:01] <mzanetti> which one is it?
[15:03] <MacSlow> mzanetti, "#!/usr/bin/env python" is used in create_interactive_notification.py although that's not were the error is reported
[15:04] <mzanetti> MacSlow: hmm... still weird... try changing it to /usr/bin/python
[15:05] <mzanetti> MacSlow: but it I don't think this is it
[15:07] <mzanetti> MacSlow: ~macslow/unity8/notification-autopilot-tests/ ?
[15:07] <MacSlow> mzanetti, no lp:~macslow/unity8/notification-autopilot-tests-dbus
[15:10] <mzanetti> MacSlow: oh... its the setsid
[15:11] <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:13] <tsdgeos> hmmm
[15:14]  * 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:16] <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:50] <sil2100> pstolowski: hi!
[15:50] <pstolowski> sil2100: hey!
[15:50] <sil2100> pstolowski: you know if mhr3 will still be around today?
[15:51] <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:52] <sil2100> pstolowski: thanks!
[15:52] <sil2100> jamesh: hi!
[15:52] <pstolowski> sil2100: yw
[15:57] <jamesh> sil2100: hi.
[15:58] <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:59] <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
[16:01] <pstolowski> sil2100, jamesh: I think it should go to main?
[16:02] <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:03] <jamesh> I think we'll want this in main eventually, since it'll be going on the phone if everything goes to plan
[16:05] <pstolowski> kenvandine, sil2100: I see, sounds good, thanks
[16:14] <Saviq> dandrader, hey, since it's your last day, can you post an update on anything you've found about bug #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:15] <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:16] <Saviq> dandrader, ;P
[16:16] <dandrader> it's shame we don't have bug dependencies in launchpad...
[16:18] <Saviq> dandrader, bug #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:19] <pete-woods> either we need to make the debuild build generate the .cc files, or we need to patch the .cc files directly
[16:26] <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:36] <dandrader> Saviq, bug updated
[16:38] <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:39] <seb128> pete-woods, thanks
[16:39] <Saviq> dandrader, thanks, and talk to you in a month! ;)
[16:40] <Saviq> dandrader, hmm... no comment in bug #1206991 ?
[16:49] <Saviq> dednick, hey, can you give me a small update on what do we still need to transition to indicator-network?
[16:50] <Saviq> dednick, one thing I'm not clear on is will we still have/use the same password page?
[16:53] <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:55] <tedg> pete-woods, ^
[16:55] <tedg> :-)
[17:00] <dednick> Saviq: : isnt this enter password dialog we need the same as the extended snap descisions MacSlow is working on?
[17:01] <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:02] <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:03] <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:04] <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:05] <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:06] <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:07] <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:08] <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:09] <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:10] <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:11] <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:12] <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:13] <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:14]  * 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:16] <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:17] <tedg> If it has to be running throughout the session it probably makes sense to just put it in indicator-network-service.
[17:17] <tedg> I for some reason thought that you couldn't "just register" as then anything could do it.
[17:18] <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:19] <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:20] <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:22] <tedg> pete-woods, Hmm, yeah, it seems like that's the case.  A bit scary.
[17:22] <pete-woods> tedg: yep, anyway - I've gotta go now, but hopefully this will tick over in your brain tonight :)
[17:23] <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!
[21:51] <TheMuso> mzanetti: Nothing, I'm still new to Qt, and I thought our platform had something that I should have used.