[09:17] and471: hey , could you upload your patches to lp and subscribe the sponsors.. [09:17] ? [09:18] vish, hey, which patches? [09:18] and471: the patches you forwarded to debian.. [09:18] or for those bugs.. [09:19] and471: debdiffs rather, sponsors prefer debdiffs.. [09:20] vish, sure, I shall try to complete them today, I have just got back form holiday and I am working through a backlog :) [09:20] *from [09:20] and471: doh! no wonder i couldnt find you on irc :) [09:21] and471: was wondering how you kept changing the bug status but i always seemed to never catch you here. :) [09:22] and471: sure not immediately , but we are nearing UIF, and translations need to be in by then.. :) [09:22] vish, hehe, yeah I did some bug stuff while I had some free time :) [09:22] vish, yup I am aware :) the shotwell guys are also bugging me as they watn to do a new release :) [09:22] *want [09:23] and471: heh, that was gonna be my follow up, "what happened to shotwell icons" ;) [09:25] vish, I won't be able to finish the whole bug I think, but I have say 1/2 of it :) [09:32] and471: redoing the icons? or not hardcoding them is 1/2? [09:33] vish, 1/2 redoing, 1/4 hardcoding [09:33] and471: oh right , re-doing them is not easy.. do you have a list that need to be redone? make i can do a few as well? [09:34] s/make/maybe [09:34] vish, sure, I shall publish them on the launchpad bug report when I get round to it :) [09:34] and471: cool! thanks.. [10:22] vish, which icon is the spinner in Humanity? [10:23] and471: which spinner? [10:23] vish, for example the one you get in nautilus when you open /usr/bin [10:25] and471: are you using n-e? [10:25] vish, yup [10:25] and471: yeah, that doesnt appear in nautilus, it was removed iirc :) [10:26] vish, ah, let me think of another example.. [10:26] and471: there is a spinner in gnome icons, which might be from where the spinner is being pulled [10:26] vish, I'll show you a screenshot [10:27] vish, the image in the right tab in http://imgur.com/On2Uv.png [10:28] and471: oh that! [10:32] and471: i cant seem to find it.. not sure where that one if from :s [10:42] vish, hehe me neither :( [10:43] vish, don't worry it isn't vital [12:20] hello [16:37] devildante: hi! [16:37] hi kiwinote :) [16:38] devildante: I adapted the appdetailsview to use package info lines rather than package info tables [16:38] devildante: you can use this for the total size info [16:38] kiwinote: thanks :) [16:38] devildante: code is in the getting-the-small-things-right branch [16:39] kiwinote: okay :) can I merge it? [16:39] devildante: sure [16:40] kiwinote: thanks :) [17:01] kiwinote: it worked, thanks again :) [17:01] devildante: great to hear! [17:01] devildante: I'll look at the navigation stuff tomorrow [17:01] kiwinote: thanks :) [17:05] kiwinote: I'm here all day, so if you want to talk, I'm available ;) [17:05] devildante: are you free-ish this week? [17:06] kiwinote: I'm a 16-year-old student who doesn't have school in summer, of course I'm free :p [17:06] devildante: ah, ok, fun :p [17:07] devildante: there are still a few things that need doing about that addons branch [17:07] kiwinote: order and I shall code :P [17:07] devildante: mostly just tidying up things and so [17:07] kiwinote: is there something interface-wise? [17:07] devildante: I think if you work on that say monday and tuesday, then by wednesday thursday it should be truly rocking ;) [17:08] kiwinote: yeah :) [17:08] devildante: yeah, a few things like the padding around the install bar [17:08] kiwinote: you mean the "apply changes" bar? [17:08] devildante: but also things that installing a normal package fails.. [17:08] devildante: yep [17:09] kiwinote: that one I done it right now :) [17:09] devildante: it's great work, it just needs a bit more polish and testing before it is ready for trunk [17:09] devildante: more so because feature freeze has passed [17:09] kiwinote: and about the pkg not installing, it's been a while I didn't test that, so I'll investigate :) [17:10] kiwinote: okay :) [17:10] devildante: don't worry too much about it all though, at the beginning of the summer I was just getting stuck into s-c code myself and facing these issues as well [17:10] kiwinote: okay :) [17:11] devildante: you already have the advantage that you are a few years younger when you start ;) [17:11] kiwinote: yeah :) [17:18] kiwinote: do you think there will be a problem with the ffe? [17:21] devildante: I think if we get it into really good shape this week, then there is a fair chance that we may get it [17:22] kiwinote: thank you for helping me :) [17:22] devildante: having said that, I aren't really too experienced with ffe's. I have seen more significant changes come later in the cycle though [17:23] devildante: no probs, it's great to see people actively working on s-c [17:26] kiwinote: argh, spotted a bug, and I don't know how to fix it [17:26] devildante: you can paste the error in the pastebin and post a link here [17:26] devildante: or if it isn't a crash, you can describe the bug here ;) [17:27] kiwinote: when removing an add-on, it is removed, but the "in progress" view doesn't show [17:27] devildante: yeah, i think that has to do with how you call the transaction [17:28] output: http://paste.ubuntu.com/478434/ [17:29] devildante: when i looked at the code I wondered why you wouldn't call install_multiple or remove_multiple rather than feeding the addons as a side thing of a real pkg [17:29] nessita, hey, how is the login dialog going? [17:29] kiwinote: it's just because remove_multiple wasn't there when I began to implement [17:30] devildante: but i haven't decided what the best way is, so don't change that behaviour until I've had time to think ;) [17:30] kiwinote: okay :) [17:30] and471: hey, you can try it on on latest maverick [17:30] devildante: ah, yes, that output looks familar. It's an aptdaemon bug most likely though [17:30] and471: you'd have to use the dbus service "by hand", but is already in main [17:31] devildante: don't think I'm comfortable enough with that code to fix it though [17:31] kiwinote: this is the exact line causing this: trans = aptdaemon.client.get_transaction(tid, [17:31] error_handler=lambda x: True) [17:31] kiwinote: pendingview.py in _on_transaction_changed [17:32] kiwinote: install_multiple and remove_multiple could fix this, you think? [17:33] devildante: i think the pending view stuff is a consequence of aptdaemon stuff though [17:33] devildante: it should bypass it, but the bug will need to be fixed at some stage anyway [17:34] so I just replace it with install_multiple/remove_multiple (in aptd.py) [17:34] ? [17:34] devildante: using install/remove multiple will mean that it sends each addon as a separate transaction, hence showing each in a separate tile in the pending view [17:34] kiwinote: I got some idea, lemme test [17:35] devildante: from memory it would need slightly more changes than that [17:35] devildante: the biggest question is when we select a pkg to install and select some addons to install, do we want say 5 entries in the pending view, or do we want one entry? [17:35] kiwinote: well my idea won't work :p [17:36] hm ;) [17:36] nessita, cool, might try it later :) [17:36] from the spec, it seems we want each add-on as a separate task! [17:37] but only when the app is installed [17:37] devildante: let me have a quick look [17:37] devildante: hm, and when the main app isn't installed we want it one transaction?? [17:37] kiwinote: yes [17:38] kiwinote: now it should be easy :p [17:39] hm, that makes it a bit messy in the code though, as we need to call install multiple if the pkg is not installed and we need to do what you're doing now for non installed apps [17:40] devildante: the nicest and cleanest solution would be to change your current code to use install/remove multiple in all cases [17:40] the "Apply changes" button won't show if the app isn't installed, though [17:40] kiwinote: it seems so [17:41] devildante: i think i need to think about it a bit, and we need to check with mpt before you should start changing the behaviour [17:41] devildante: so that's something for tomorrow ;) [17:42] kiwinote: okay, I'll do some tests though :p [17:42] devildante: nice [17:42] and471: let me know! :-) [17:42] devildante: mvo has recently added quite a few automated tests that check for regressions [17:43] kiwinote: oh yeah! [17:43] devildante: it may be interesting to look at what he has done and add some tests related to addons [17:43] kiwinote: though I don't know anything about tests :p [17:43] devildante: there are both backend tests and gui tests [17:44] devildante: heh, nor did I, but looking at what mvo has done you should be able to copy and modify some stuff ;) [17:44] kiwinote: okay :) [17:45] devildante: having tests may give bonus points for the ffe, so if you have heaps of spare time, then it may be a good thing to spend some time looking at ;) [17:45] kiwinote: all right :) [17:46] devildante: the most important part is making sure you cover all cases [17:46] devildante: anywayz, I'm going to have some dinner now [17:46] kiwinote: bye for now :) [17:47] devildante: not sure if I'll be back online this evening, we'll see ;) [17:47] kiwinote: np, take your time :) [17:48] devildante: enjoy your coding ;) cu [17:48] dammit kiwinote, I can't enjoy Coding :P [18:08] hi vish :) [18:09] devildante: o/ === JanC_ is now known as JanC === almaisan-away is now known as al-maisan === al-maisan is now known as almaisan-away === almaisan-away is now known as almaisan === almaisan is now known as al-maisan === al-maisan is now known as almaisan-away