/srv/irclogs.ubuntu.com/2010/08/13/#ayatana.txt

klattimerseb128: in lp:ubuntu/maverick/ibus the old app indicators patch has been applied09:05
klattimereven though the patch still exists in the debian/patches folder09:05
klattimerseb128: in fact, my patch has been applied and then modified09:08
seb128klattimer, hi09:09
seb128to be honest I didn't check what the guy who did the update do09:09
klattimer:/09:10
seb128he probably just used the patch we had in lucid09:10
klattimerseb128: no, it's weirder than that09:10
klattimerthe old patch is in the debian/patches folder09:10
seb128well feel free to drop the patch and see if your version applies or need update09:10
klattimerbut, the new patch has been applied to the code and checked in09:10
seb128no09:10
klattimer?09:11
seb128would be very surprising09:11
seb128I think it's just the new source format09:11
klattimerit's been modified from what I wrote too09:11
seb128the patch is in the debian dir but already applied when unpacking the source09:11
klattimerand a comment added saying Icon from filename not supported yet for app indicator09:11
klattimerseb128: it's all different09:12
seb128hum09:12
seb128so maybe he tried to take the work in progress version you had at the time he did the update09:12
klattimerlike the patch in lp:ubuntu/maverick/ibus is different to what's already been applied to the code and different to my ibus patch09:12
seb128the guy who did the update is not a hacker09:12
klattimer:(09:13
seb128he tried to make some of the patch apply without conflict09:13
klattimerall that needed to be done is change the 05_appindicator.dpatch09:13
seb128well, just drop everything in that version and make your change apply to the new version09:13
klattimergrr, this is going to be hard to unpick now09:13
seb128don't bother about packaging09:13
seb128unpick?09:13
klattimerwell, unless I can get a clean unpatched/broken version of ibus?09:14
seb128well09:14
klattimerwhat would the lp: address be for that?09:14
seb128tar xzf orig.tar.gz09:14
seb128apt-get source ibus09:14
klattimerright but, I think the orig might have these same hacks in it09:14
klattimerwill the source be the new version you mentioned?09:14
seb128it would mean that upstream commit some appindicator support?09:15
klattimerno09:15
seb128do apt-get source ibus09:16
seb128rm the dir09:16
seb128and untar the orig tarball09:16
klattimerseb128: I'll create a bzr branch for the packager to use, perhaps?09:16
seb128re09:20
seb128klattimer, you can probably use the current source, I'm not sure to get the issue09:20
seb128klattimer, does running "quilt pop" in the source does what you want?09:20
klattimerI think I've got it ok now thanks for your help09:21
seb128just run it a bunch of time to unapply the patches09:21
klattimerI'll create a bzr branch and push it up somewhere so the packager can use it09:21
seb128it's that the source you get has patches applied09:21
seb128quilt pop unapply those09:21
seb128you should get back to a source without the debian patches this way09:21
klattimerquilt pop 06_show_menuitem.dpatch does not remove cleanly :/09:23
seb128weird09:23
klattimerI think he's checked it in after it's failed to patch or something09:24
seb128hum09:24
klattimerbecause 05, 02,01 all patch -R -p1 < patch09:24
klattimerso I can reverse everything except 0609:24
seb128can you try to "apt-get source ibus"09:24
seb128cd ibus-1.3.709:24
seb128quilt pop09:25
seb128if you get an error copy it there09:25
seb128do it twice09:25
seb128quilt pop09:25
klattimergetting source gets me 1.2.0 not 1.3.7?!09:26
seb128sudo apt-get update09:26
klattimerok now I have a clean source tree09:27
klattimerI'll generate my ibus patch for this updated ver09:27
seb128klattimer, thanks09:29
klattimerseb128: updated patch has been uploaded to bug #56403409:44
ubot5Error: Could not parse data returned by Launchpad: list.index(x): x not in list (https://launchpad.net/bugs/564034)09:44
klattimerwhich isn't that bug09:44
klattimerhttps://bugs.launchpad.net/ubuntu/+source/ibus/+bug/56403409:45
ubot5Error: Could not parse data returned by Launchpad: list.index(x): x not in list (https://launchpad.net/bugs/564034)09:45
seb128klattimer, thank you and sorry for the trouble09:45
klattimernp09:45
klattimerthese things happen :)09:45
=== mpt_ is now known as mpt
davidbarthbratsche: https://code.launchpad.net/~kamstrup/libindicator/robust-icon-loading/+merge/3256611:56
davidbarthbratsche: do you know why ted had all of the explicit icon loading this way; kamstrup found that gicon currently uses the same scaling algorithm11:57
bratscheI'm not sure what his reason for this was.12:01
davidbarthbratsche: nevermind, the fix works; we'll check with him later today12:06
thorwilnice example of assumptions vs testing: http://ignorethecode.net/blog/2010/08/13/opinions_vs__data/12:12
adaythorwil: interesting. google are well known for having a testing-led design process. there's advantages and disadvantages to that, of course...12:26
adaythorwil: http://stopdesign.com/archive/2009/03/20/goodbye-google.html12:31
aday"that data eventually becomes a crutch for every decision, paralyzing the company and preventing it from making any daring design decisions."12:31
thorwiladay: yeah, i had to think of that article when you mentioned disadvantages :)12:31
adaythorwil: not that i think f/oss is in danger of an over-reliance on user testing ;)12:32
thorwiladay: large improvements could be had with proper design methods or even just expert reviews that are then translated into changes. all before even testing, which could then bring to a further level up12:34
adaythorwil: yeah, we're getting better at that, but there's a long way to go12:35
adaythorwil: had some amazing results yesterday with a gsoc student, actually12:35
thorwiladay: what worries me a bit is that i think some deep changes would be required for some real ux improvements. for example getting to a point where hierarchical filesystems either don't exist anymore, or are something a user absolutely never has to care about12:38
adaythorwil: yeah, document management is *the* thing that needs fixing. i've talked about this with designers in the past, but we've never managed to get very far. we're all too busy fixing what we already have12:40
=== DrPepperKid is now known as MacSlow
=== DrPepperKid is now known as MacSlow
=== DrPepperKid is now known as MacSlow
Cimidavidbarth: received your mail13:57
davidbarthCimi: hi14:06
Cimidavidbarth: I've just finished replying to the bugs14:06
Cimidavidbarth: I didn't know about "the thursday window"14:07
CimiI was not at the pc because otto was busy with other tasks and told me to take some relax waiting him to come back on radiance ;)14:08
Cimibut since yesterday we are finishing radiance14:08
=== MacSlow is now known as MacSlow|lunch
davidbarthCimi: ok, cool14:13
Cimidavidbarth: I know you love radiance, and you're gonna love this more14:13
davidbarthCimi: no worries, the upload window is an internal DX thing, but it does help to streamline the flow of changes between us and the desktop guys14:13
CimiI really like the new colors14:13
davidbarthCimi: hehe, have a screenshot?14:14
Cimiyes but I want to share it when it will be perfect ;)14:14
davidbarthCimi: ok, i'll wait then14:14
seb128klattimer, the ibus indicator doesn't seem to work there14:24
seb128ibus-daemon --verbose gives14:24
seb128"/usr/share/ibus/ui/gtk/main.py", line 61, in __init__14:24
seb128    self.__notify.attach_to_status_icon (self.__panel.get_status_icon())14:24
seb128TypeError: Notify.Notification.attach_to_status_icon() argument 1 must be gtk.StatusIcon, not None14:24
klattimerseb128: ugh14:24
klattimermaybe some of the patch corrupted14:24
klattimerI didn't rebuild the package as the patch seemed to apply OK with offsets14:24
klattimercan you try and select quit from the menu14:25
seb128can you try to install it and see if it works for you?14:25
seb128I've no menu or indicator displayed at all14:25
klattimerI got a problem 1 out of 5 times that it breaks all the menu signals14:25
klattimeroh!?14:25
klattimerthat's very strange14:25
klattimerI'll take a look at it14:25
seb128thanks14:25
klattimercan you make sure you have m17n-db installed14:26
seb128I don't14:26
klattimerthat's the package the icons come with14:26
klattimerso you should probably install that14:26
seb128installing14:26
klattimer... that /should/ already be a dep14:26
klattimer:/14:26
seb128it's not14:27
klattimer:/14:27
seb128$ apt-cache rdepends m17n-db14:27
klattimerwell, it's always depended on it14:27
seb128Reverse Depends:14:27
seb128  scim-m17n14:27
seb128  m17n-contrib14:27
seb128  libm17n-014:27
klattimeras in, always required the icons from there14:27
seb128doesn't make a difference14:28
seb128do you get the error with ibus-daemon --verbose?14:28
klattimerhave you tried restarting ibus?14:28
klattimerhold on, I have to rebuild the package14:28
seb128yes14:28
seb128I killall ibus-daemon between tests14:28
seb128well I'm trying by running ibus-daemon --verbose now14:29
seb128"/usr/share/ibus/ui/gtk/main.py", line 61, in __init__14:29
seb128    self.__notify.attach_to_status_icon (self.__panel.get_status_icon())14:29
seb128TypeError: Notify.Notification.attach_to_status_icon() argument 1 must be gtk.StatusIcon, not None14:29
seb128still getting that14:29
klattimerI don't think that should be the case14:30
klattimerso I'm gonna try and see what could be doing it14:30
seb128I might be lacking a depends14:31
seb128I tried to install some ibus-* to get input methods listed14:31
klattimerseb128: i'm having trouble building14:36
klattimerit's erroring on gir stuff14:36
klattimerIBus-1.0.gir exists in debian/tmp but is not installed anywhere14:36
klattimer...14:36
seb128klattimer, edit the rules14:37
seb128dh_install14:37
seb128drop the fail if missing thing14:37
seb128that's because you have gir build requirements installed14:37
klattimerk14:38
seb128you can "debuild binary"14:40
seb128to avoid rebuilding from start14:40
klattimerseb128: yeah I see14:49
klattimeranother file changed between 1.2.0 and 1.3.7 and now relies on the status icon14:49
klattimerI'll rework it14:49
klattimerand update the patch14:49
seb128klattimer, thanks14:51
klattimerseb128: patch fixed testing now14:54
klattimerseb128: I've fixed the patch15:19
klattimerhowever another problem exists in that I can no longer add new input methods15:19
klattimerand I think that's related to the ibus-m17 package no-longer existing in 1.3.715:19
seb128just install random ibus-..15:19
seb128apt-cache search ibus15:19
seb128I installed the latex one for testing15:20
klattimerseb128: i still can't add input methods15:21
klattimerbut if you can check out my patch on the bug15:21
seb128ok will do15:21
klattimerhttps://bugs.launchpad.net/ubuntu/+source/ibus/+bug/56403415:21
ubot5Error: Could not parse data returned by Launchpad: list.index(x): x not in list (https://launchpad.net/bugs/564034)15:21
klattimerand see if it works for you15:21
seb128you can't select those in the combo box in the second tab?15:22
klattimerI'm sure something went wrong in the packaging now that the ibus-m17 package is gone :/15:22
seb128it's a bit non obvious15:22
=== MacSlow|lunch is now known as MacSlow
klattimerI can't press the add button15:22
seb128you need to open the combo, select one and then click add15:22
klattimeroh15:22
klattimernm15:22
klattimermy fault15:22
klattimerseb128: just tested and it works fine15:23
klattimer:)15:23
seb128excellent15:23
seb128will review and upload that15:24
defrenggood evening guys :)16:11
defrengI'm the developer of the gm-notify application and just wanted to ask if there are major API changes for since lucid, so I can adapt these into the application?16:12
defrengI just scanned the wiki but couldn't find something really new16:12
defreng*API changes since16:14
kirohtolihi, y wanted to ask djsiegel a few simple questions about ubuntu unity, but he's not on line. can please anyone else help me with this?16:44
kirohtolisorry about the bad english :(16:45
kirohtoliok, thanks anyway, bye!16:47
=== bratsche_ is now known as bratsche
fR_hey are any of the KDE folks around? I was wondering if someone could update the qt package in the dx-team PPA, currently the kubuntu PPA is newer so appmenu breaks.20:20
ScottKfR_: The one in the Maverick archive should be current.20:21
fR_ScottK: ah, and that includes the appmenu patches? are there plans to backport this to lucid?20:24
ScottKProbably not.20:24
fR_okay, thanks for the info.20:26
ScottKWe may have it in a PPA somewhere.  Let me check.20:28
fR_oh, thanks. i have the kubuntu beta ppa which does include 4.7b2, but just not the appmenu patch.20:29
=== rickspencer3 is now known as saacfl
=== saacfl is now known as rickspencer3

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!