/srv/irclogs.ubuntu.com/2008/11/09/#kubuntu-devel.txt

jtechidnargreening: 02_no_dependency_file.diff in kdepimlibs02:05
jtechidnaneeds to be gone for it to work I think02:05
coreymon77and now for my classic line02:14
coreymon77holy netsplits batman!02:14
jtechidnaHa, success03:09
jtechidnaScottK, Riddell: http://paste.ubuntu.com/69465/ <- Revert bad patch that makes kdepimlibs5-dev not dep on kdelibs5-dev03:30
jtechidnahttp://paste.ubuntu.com/69466/ <- Re-add tooltips patch to base-workspace03:30
rgreening\o/ jtechidna04:05
jtechidnaRiddell: http://paste.ubuntu.com/69477/ <- fixes kdenetwork FTBFS04:05
* jtechidna waves to rgreening04:05
=== jtechidna is now known as JontheEchidna
rgreeningJust had kgpg eating 100% CPU for last couple of hours. Dunno why04:10
rgreeningLappy smeels like burnt tennis shoes04:10
JontheEchidnaheh04:12
JontheEchidnahmm, new Qt in jaunty04:13
* JontheEchidna wonders what's new04:13
JontheEchidnaoh, kdenetwork didn't need to depend on libsm-dev04:14
JontheEchidnaRiddell: nevermind that kdenetwork diff then04:14
* JontheEchidna out04:22
rgreeningbye04:23
ScottKrgreening: I've never had much luck with kgpg.04:38
rgreeninglol04:58
nixternalfinally fixed my damn development desktop...took me long enough, but I finally decided to fork over a little cash to get some new hardware05:04
NCommanderScottK, are you interested in sponsoring an upload to main (I think I have a fix to resolve kdenetwork's FTBFS)05:47
ScottKNCommander: Possibly.  I need to get to bed.05:48
NCommanderNo issue, I'll ask someone else to upload05:48
nixternalNCommander: jaunty?06:17
nixternalif so, link me to the sources and I can upload06:18
NCommanderif we're going to be uploading 4.2 soonish06:18
NCommanderProbably not worth it06:18
nixternalok..if you change your mind I can upload for you06:19
coreymon77HI EVERYBODY!06:19
coreymon77hi dr. nick!06:19
* NCommander didn't know you were a coredev Nightrose 06:19
coreymon77lol06:19
NCommanderer nixternal06:19
nixternalhehe06:19
nixternalyup06:19
coreymon77heya nixy06:20
NCommandernixternal, next time I'm in Chicago, I'll have to say hi06:20
coreymon77whats up06:20
nixternalonly put it off for a couple of years because I was working with KDE4 so much that I didn't need core dev for anything else...always had JR, Tonio, Hobbsee, and others to upload for my lazy arse06:20
nixternalNCommander: sure enough, let me know when you come out...we can get an opensource night out on the town06:21
nixternalwe do it, like every night I think :)06:21
nixternalwasabi coreymon7706:21
coreymon77i dont like spicy stuff06:21
coreymon77:P06:21
NCommanderBah, FEEL THE BURN06:21
=== dfiloni is now known as devfil
Tm_Thmm, has there been plans to upgrade/backport cake 2.6.2 or newer to intrepid ?10:26
Tm_Ts/cake/cmake/10:26
NCommanderTm_T, you can request a backport10:47
Tm_TNCommander: oh, I'm sure it will be backported when someone starts packaging KDE 4.210:50
Tm_Tor so10:50
NCommanderKDE 4.2 isn't packaged yet?10:50
Tm_Tit isn't "ready" yet even10:51
NCommanderOh10:51
NCommanderfair enough10:51
Tm_Tcurrent development head (trunk) KDE: 4.1.72 (KDE 4.1.72 (KDE 4.2 >= 20081104))10:52
* NCommander nods10:53
NCommanderRiddell, ping11:14
jussi01mornining NCommander, didnt get to congratulate you yet, so congrats on MOTU! :D11:16
NCommanderheh11:17
NCommanderThank you11:17
Tm_Tapachelogger: hi son12:15
JontheEchidnaNCommander: I believe Riddell fixed the kdenetwork ftbfs last night when he added libsm-dev to qt4-x11's build deps12:40
NCommanderright12:40
NCommanderthat's the fix12:40
JontheEchidnaWhat can I say? The man is fast :P12:40
JontheEchidnaAnyway, if we have enough people I wouldn't mind packaging the next weekly snapshot of KDE 4.2 for Jaunty12:41
NCommanderI'll help do it12:41
NCommanderBut do we want it in a PPA, or Jaunty proper?12:41
JontheEchidnaI'd assume both, the PPA being for us mere mortals who don't want to break their systems with Jaunty12:42
NCommanderNo jaunty PPAs though12:43
NCommanderJontheEchidna, you core dev yet?12:43
JontheEchidnanope, gonna go for MOTU in a bit here12:43
JontheEchidnaI think we could use an Intrepid PPA since we wouldn't have to change much more than the series12:44
Tm_TJontheEchidna: NCommander: you will need new cmake then12:53
JontheEchidnagood point, hadn't realized that12:54
NCommanderWe can simply put the new cmake in the PPA12:54
Tm_TJontheEchidna: so please do that for intrepid backports please (;)12:54
NCommanderCan't12:54
NCommanderPPA do not build w/ backports12:54
Tm_Tyou can put it to backports _too_12:55
Tm_Tor, KDE developers fiddling with Intrepid will need other source (ppa?)12:55
Tm_Tjust for cmake that is12:55
JontheEchidnaapparently if you have 1337 powars you can even copy stuff from ppas to backports12:55
* JontheEchidna afk for breakfast12:56
Tm_TKDE 3 repository for Intrepid: http://ubuntuforums.org/showthread.php?p=606552413:03
JontheEchidnathose packages have an epoch of 7...13:08
JontheEchidnacrazy13:08
JontheEchidnaTm_T: So trunk needs cmake 2.6.2 currently?13:14
* JontheEchidna investigates and finds out the answer is "yes" :P13:15
JontheEchidnaoh, jaunty already has 2.6.213:17
JontheEchidnakewl13:17
Tm_TJontheEchidna: will need13:19
JontheEchidnakk13:19
JontheEchidnabah, dput is broken13:28
andylockranguys, I've just switched over to kubuntu (by installing kubuntu-desktop) to help with more bugs reports .etc after listening to the ubuntu-uk podcast.13:28
andylockranHow come any bugs that occur goto bugs.kde.org rather than a launchpad bug-tracker?13:29
JontheEchidnaThe KDE devs can't fix bugs they don't know about13:30
JontheEchidnaIf you're unsure whether the bug is KDE or *buntu's fault, then it's a better idea to report it at launchpad first13:30
JontheEchidnawe can generally determine whether or not it's a KDE or Kubuntu bugs13:31
JontheEchidnaOh, and if anybody can halp with the dput failure: http://paste.ubuntu.com/69628/13:31
JontheEchidnaoh, haha13:34
* JontheEchidna needs to learn to rtfm13:34
Hobbseedput -f?13:37
Hobbseeoh :)13:38
andylockranJontheEchidna: isn't there a method for passing kubuntu bugs upstread?13:59
andylockranupstream ?13:59
=== santiago-ve is now known as Guest66702
=== Guest66702 is now known as santiago-ve
jjessegood morning :016:57
apacheloggerTm_T: yo mom17:00
apacheloggerJontheEchidna: I thought I did that before I even created that patch, will look into it17:08
ryanakcaapachelogger: were you still interested in helping out with porting the wiki theme? We basically need to get a color palette and then sed ``Dark Ubuntu Brown'' for ``Dark Kubuntu Blue'' throughout the css files. (I seem to recall asking you this, not sure...)17:10
apacheloggerRiddell: about kubuntu_11_fix_openwithdialog.diff coming from upstream: no, it is a workaround for 4.1, 4.2 will carry the whole kservice object around instead of only connecting selected information to the tree, however I talked to dfaure and he aggreed that the patch is a sensible solution as well, I actually think the proper fix of carrrying around the object should be committed by now, IIRC I created that patch a couple of17:11
apachelogger weeks ago17:11
apacheloggerryanakca: get a meeting date17:12
apacheloggercolor palette is uberimportant indeed17:12
ryanakcaapachelogger: okies. Well, what I'll do is work on one, and then present it at the meeting, if everybody likes it, I can sed, commit and start poking the sysadmins.17:13
apacheloggerseele, smarter: --force-overwrite for the neon issues, KDE dragged stuff around in plasma creating some conflicts17:14
smarterapachelogger: yup, I figured it out(and I had another issue, but it was because I forgot to s/hardy/intrepid/ the neon line in my sources.list :P)17:14
ryanakcaapachelogger: did the bzr + kde packages get discussed or is it for the next meeting17:18
apacheloggernot that I would claim the neon stuff for intrepid works properly yet :P17:18
apacheloggerryanakca: next meeting17:18
apacheloggerin fact, I should get that meeting setup17:18
ryanakcaapachelogger: okies... I set the DateTime to TBD... and I'll finish adding my topic to the agenda17:19
apacheloggerNightrose: about that quassel package: only if sput can ensure we get a stable release from current HEAD by the time jaunty is going into future freeze, otherwise it probably would make sense to drag neon to the 3rd layer of abstraction which would mean that I have to introduce documentation so that I don't get lost, and we all know, documenting stuff is boring, so I'd rather not do that17:20
Nightroseapachelogger: hmmmmm i'll ask17:21
Nightrosewhen is feature freeze?17:21
apacheloggerno clue17:21
apachelogger~google ubuntu wiki schedule jaunty17:21
kubotuResults for ubuntu wiki schedule jaunty: 1. JauntyReleaseSchedule - Ubuntu Wiki: https://wiki.ubuntu.com/JauntyReleaseSchedule | 2. UbuntuOpenWeek - Ubuntu Wiki: https://wiki.ubuntu.com/UbuntuOpenWeek | 3. Xubuntu/Artwork/Incoming/Jaunty - Ubuntu Wiki: https://wiki.kubuntu.org/Xubuntu/Artwork/Incoming/Jaunty17:21
apacheloggerFebruary 19th17:22
Nightrosek17:23
Tm_Tapachelogger: cmake upgrade on the way there?17:38
apacheloggerwhere?17:38
apacheloggerScottK: I think we should create a package with some useful and well maintained scripts to be shipped with Kubuntu17:39
apacheloggermaybe based on upstream suggestions17:39
Tm_Tapachelogger: kde trunk will soon require cmake 2.6.2 or newer, so, how will you deal needed upgrade on neon?17:40
Tm_Tand will that be also pushed to ubuntu backports or some?17:40
apacheloggerTm_T: neon shipped 2.6.2 as it's first or second intrepid package17:40
Tm_Tapachelogger: aah, good18:03
Tm_Tapachelogger: I'm hoping someone push it to backports or some other more general source18:04
* apachelogger needs to get some sort of mail access working again18:04
smarterapachelogger: do you plan to package kdepim for neon?18:04
apacheloggersmarter: it is18:04
smarterapachelogger: oh, I didn't even see it :p18:04
smarterbut "Version: 20081027.6+svn876565-0neon1"18:04
apacheloggerso?18:06
smarterkdebase/kdelibs are from 110718:06
apacheloggerwell, kdebase was b0rked for the last week or so18:07
smarterso, any plan for upgrading all the other packages? :)18:07
lex79apachelogger: http://revu.ubuntuwire.com/details.py?package=kradioripper18:08
lex79http://revu.ubuntuwire.com/report.py/legal?upid=389218:08
apacheloggersmarter: technically I invoked a rebuild 3 hours ago18:08
apacheloggerbut my kmail ain't starting18:08
apacheloggerkonqueror fails to render gmail and I really don't feel like coping with the incredibly slow lp today18:09
smarterooohh, shiny, upgr4des :p18:09
* smarter wonders why aptitude decided to install exim418:10
apacheloggerlex79: upstream licensed in a very inproper way18:10
apachelogger/ (c) Tim Fechtner. Licenced under GPL 2 (yes, version 2 only!).18:10
apacheloggerthat is not my defintion of a sensible license header18:11
apacheloggerin fact, I don't even think that is valid18:11
lex79so, missing license header in all files?18:11
ScottKsmarter: Because some package wanted an MTA and Debian usually does that exim4|mail-transport-agent18:11
apacheloggerlex79: well, incomplete18:11
apacheloggerlex79: also the COPYING is missing, which is definitely invalid18:12
ScottKapachelogger: Does the package include a full copy of GPL v2 in the tarball?18:12
ScottKYeah.  That's needed.18:12
apacheloggerScottK: people who insist on version 2 only mostly didn't read the license :P18:12
smarterScottK: probably, but I don't expect a post-release upgrade to install a mail daemon/client/whatever :p18:12
ScottKapachelogger: I disagree.18:12
apacheloggerScottK: all I have seen didn't18:13
ScottKsmarter: Sure.  What else does it want to install/upgrade?18:13
apacheloggerlex79, ScottK: ah!18:13
ScottKapachelogger: I think there are very valid reasons not to like V3.18:13
apacheloggeractually there is18:13
* apachelogger was blind18:13
lex79I send a mail to author :)18:14
ScottKOK.  Well if there's a full copy of the license in the tarball, then the incomplete license/copyright headers in the files won't block it.18:14
lex79ScottK: ya18:14
apacheloggerScottK: my point is that they rely on reasons of other people without reading the licenses for themselfs18:14
ScottKIt's better if it's there.18:14
apacheloggerlex79: suggest it to upstram though18:14
smarterScottK: http://pastebin.com/m2a2196b818:14
lex79ok18:14
ScottKDefinitely (about suggesting upstream)18:14
* ScottK looks18:14
smarterit may be aptitude who wants to install recommends and these recommends need a mailer18:14
apacheloggerbug #27151618:15
ubottuLaunchpad bug 271516 in ubuntu "[needs-packaging] KRadioRipper" [Undecided,In progress] https://launchpad.net/bugs/27151618:15
ScottKsmarter: Yes.  That's about right.18:15
smarteroh, bsd-mailx rdepends on devscripts18:16
ScottKYes.18:16
ScottKsmarter: Do you have backports enabled?18:17
smarteryup18:17
ScottKThat's the new devscripts just backported from Jaunty.18:17
ScottKIt's wouldn't stun me if it pulled to much stuff in.18:17
smarterokay18:17
apacheloggerlex79: I am pretty sure kdelibs5-dev already depends on libqt4-dev and libphonon-dev, and I am also pretty sure kdesdk-dev depends on kdelibs5-dev .... that said, a) why does it depend on kdesdk-dev at all? b) why does it depend on kdelibs >= 4.1.3 and kdesdk >= 4.12? c) the depends on kdelibs5 is going to be determined by the shilbs debhelper18:18
apacheloggerlex79: no periods at the end of the initial description line please18:18
lex79a moment... :)18:18
apacheloggerlex79: no md5sums in the copyright file18:19
lex79no?18:19
lex79in control?18:19
apacheloggerno18:19
apacheloggernot at all18:20
apacheloggerone would have to update it everytime which is pointless18:20
lex79ok18:20
apacheloggerlex79: your short license is wrong18:20
apachelogger    the Free Software Foundation; either version 2 of the License, or18:20
apachelogger    (at your option) any later version.18:20
ScottKsmarter: I'll discuss it with pitti this week.18:20
ScottKThanks for pointing it out.18:21
smarteryou're welcome ;)18:21
lex79This package disgusting :)18:21
apacheloggerlex79: how so?18:21
smartermaybe it's just aptitude which is not doing the right thing18:21
* apachelogger needs a jaunty pbuilder18:21
ScottKThe or later version is optional (upstream's choice)18:21
lex79apachelogger: for depends: http://revu.ubuntuwire.com/revu1-incoming/kradioripper-0811091906/kradioripper-0.4.1/INSTALL18:22
ScottKapachelogger: Everything you need for that is in intrepid-backports18:22
ScottKsmarter: No, aptitude is installing recommends as it is supposed to.18:22
apacheloggerScottK: would I need something special for that? Oo18:22
ScottKapachelogger: Either get debootstrap from backports or add a Jaunty symlink yourself.18:23
* apachelogger loves his pbuilder :P18:23
ScottKThe devscripts backport will give you a dch that defaults to Jaunty, which can be handy.18:23
lex79apachelogger: without libphonon-dev, pbuilder does not build18:23
smarterlex79: what's the error?18:24
lex79phonon not found18:24
smarterkdelibs5-dev does depend on libphonon-dev18:24
apacheloggerlex79: I am not saying they are wrong, they are just not common ;-) you don't manual set a depends which is going to be detected by debhelper anyway (kdelibs5) ... and for the build-deps they are recursive18:24
apacheloggerkdesdk-dev depends on kdelibs5-dev, kdelibs5-dev depends on libphonon-dev and libqt4-dev18:25
apacheloggerso at least you would need to remove phonon and qt418:25
apacheloggerbut I also think kdesdk is not necessary, so I would appreciate if you could try without it18:25
lex79I try18:25
ScottKapachelogger: plasmoid-quicklauncher is looking for a 2nd advocate on REVU.18:26
* ScottK looks to sponsor JontheEchidna's kdepimlibs upload.18:27
lex79kdelibs5-dev with (>= 4.1.3) or without?18:28
apacheloggerlex79: I would assume (>= 4.1) should be enough18:29
lex79ok18:29
lex79apachelogger: without phon no build18:33
lex79http://paste.ubuntu.com/69703/18:33
apacheloggerlex79: complete log please18:33
apacheloggerbug #26009418:35
ubottuLaunchpad bug 260094 in ubuntu "[needs-packaging] plasmoid-quicklauncher" [Wishlist,Confirmed] https://launchpad.net/bugs/26009418:35
apacheloggerScottK: I feel like holding back the advocation in order to get the patch mentioned in the changelog18:36
ScottKapachelogger: What patch?18:38
apacheloggerdesktop-file-improve.patch18:38
apacheloggerScottK: plasmoid-quicklauncher18:38
ScottKThat's in the package.18:38
ScottKOh, I see18:39
apacheloggerbut not documented in the changelog18:39
ScottKapachelogger: I'd say edit the changelog and upload it then.18:39
ScottKGot it.18:39
apacheloggerwell, might be a good lesson :P18:39
ScottKapachelogger: Be careful not to teach the lesson that's it's too hard to get stuff done for Kubuntu.18:39
apacheloggerbeing to sloppy on the documentation of patches is a pretty grave issue IMHO18:39
lex79apachelogger: http://paste.ubuntu.com/69705/18:39
apacheloggerScottK: you are probably right18:40
ScottKI think if you show the right way that will be enough.18:40
apachelogger*nod*18:41
apacheloggerlex79: apparently the libphonon dep was dropped in our most recent debian merge18:42
apacheloggerI am not sure that is a very good idea though18:42
apacheloggerlex79: for now readd the phonon build-dep, but the libqt4-dev one shouldn't be needed18:42
lex79ok, add libphonon-dev18:43
ScottKapachelogger: Is there some package that used to depend on libphonon-dev that doesn't now.  My test build of kdepimlibs just failed for lack of phonon.18:53
smarterScottK: see above :P18:53
smarterScottK: apparently Debian doesn't make kdelibs5-dev depends on libphonon-dev18:54
smarterand we merged recently18:54
ScottKI see.18:54
apacheloggerwell18:54
ScottKapachelogger: Should we put that back then?18:54
apacheloggerI am not sure18:54
apacheloggerthis could easily be caused by CMake linking recursive18:54
apacheloggerwhich is going to get fixed once we upload something 4.2ish18:54
apacheloggerScottK: I guess readding it for now is the way to go18:55
* apachelogger is feeling ineffective without kmail19:00
lex79I try to make a package for this plasmoid http://www.kde-look.org/content/show.php/memusage?content=8663019:08
lex79same issue with libphonon-dev... uhm19:08
smarterthat is one ugly plasmoid19:09
smarterand hackish too, you shouldn't need a config dialog for changing the size19:09
apacheloggersmarter: I would assume it's the author's first one19:10
smarterprobably19:10
* apachelogger kicks nx19:10
apacheloggerreply ain't workin19:10
smarterlex79: I would suggest not packaging any plasmoid until we have 4.2 in the repos19:10
lex79:(19:11
smarter'cause there is a chance that some of them will not be updated19:11
apacheloggersmarter: I don't agree, it is practise19:11
smarterand then we'll have to remove them19:11
smarterwel,l you can package them, but we could wait until 4.2 is out to upload them ;)19:11
apacheloggersmarter: that is discouraging :P19:12
ScottKFinding old broken KDE3 that won't be updated and doesn't work and getting it removed would be a good idea.19:12
ScottK5 year old just had a conniption because she felt one of her older sisters wasn't taking her seriously.19:13
ScottKI told her to go to her room until she calms down and can be civil.19:13
ScottKI expect that'll get me an hour or two of quiet.19:14
ScottK;-)19:14
apacheloggerLOL19:14
smarterhttp://www.kde-look.org/content/show.php/Adjustable+Clock?content=92825 yeah, moar clocks! :p19:14
apacheloggerclocks++19:14
apachelogger~karma c19:14
kubotukarma for c: 3519:14
apachelogger~karma python19:14
kubotupython has neutral karma19:14
apachelogger~karma ruby19:14
kuboturuby has neutral karma19:14
apacheloggerye're all a bunch of c lovas :P19:15
ScottKNot /me.19:15
ScottKThe best thing about C packages is I can't fix them so all I can do is complain.19:15
apacheloggerScottK: isn't that what you did all intrepid anyway? :P19:16
smarterapachelogger: it's probably because you say c++ more often than python++ :p19:16
smarterc++++ ? :]19:16
ScottKapachelogger: Yep.  Worked too.19:16
apachelogger~karma c++19:16
kubotukarma for c++: 119:16
apacheloggerScottK: ;-)19:16
* ScottK did fix guidance-power-manager stuff too though.19:16
apacheloggerI think I should comment on sebner's motu app19:17
apacheloggerbut I can't, nx doesn't let me :S19:17
* ScottK thinks sebner syncs too many things.19:18
apacheloggerI have 5 choices: 1) install KDE 4.1 2) fix akonadi and use mailody 3) fix nx 4) fix gmail 5) just don't care19:18
smarterwhy did you remove kde 4.1 anyway?19:19
smarteror use firefox for gmail if konqueror is broken19:19
apacheloggerScottK: I think he should only be allowed to do syncs, the packaging he did from scratch and made me review was not too motuworthy IIRC19:19
ScottKapachelogger: Please comment on that then.19:19
ScottKI think he just syncs the world without a lot of consideration.19:20
apachelogger*nod*19:20
apacheloggerwill have to do some studies on his work19:20
apacheloggersmarter: no 4.1 because we don't ship 4.1 with jaunty, no firefox because I have values19:21
ScottKJontheEchidna: kdepimlibs uploaded.  Thank you for your contribution to Kubuntu.19:21
ScottKNCommander: Still got that kdenetwork fix to get uploaded?19:22
* ScottK looks at kdebase-workspace19:22
apacheloggersmarter: btw, we are going to need kubuntu-icons-mono19:22
apacheloggersmarter: for launchpad, ubiquity, jockey-kde, possibly more19:22
smarterwhy?19:22
smarteroh19:22
apacheloggersmarter: you should give trunk mono a try19:23
apacheloggerI moved a lot of stuff around19:23
smarterand we need an artist which is excited by black&white icons :P19:23
apacheloggersmarter: mono icons are pretty easy to do19:23
* apachelogger did one or two already19:23
apacheloggerdragon player is going to be difficult19:24
* apachelogger thinks juk just crashed something19:24
apacheloggeroh, kded it is :(19:25
smarteryou're using juk?19:25
apacheloggeramarok has become too sluggish for my liking19:26
apacheloggerand something is draining hard disk space19:27
apacheloggerthe network manager plasmoid is already working pretty well19:32
* smarter switched to good ol' /etc/network/interfaces and wpa-supplicant19:36
apacheloggersmarter: much more reliable indeed19:36
smarterand works without X19:36
* smarter never really understood why networkmanager couldn't be run from a console19:37
apacheloggerbecause none implemented a CLI?19:37
smarterprobably :)19:37
* apachelogger needs to get new cal thingy into neon19:40
apachelogger-.-19:40
andylockranhowdy guys21:24
Riddellhi andylockran22:02
ArbyRiddell: do you have time to look at some s-c-p-kde patches?22:14
RiddellArby: jings sorry totally failed to do that22:16
RiddellArby: I've got the one from 4th nov about the qstring -> unicode22:17
Riddellthat seems right enough22:17
Arbygood22:17
RiddellArby: got more?22:17
ArbyRiddell: next one is bug 28190522:18
ubottuLaunchpad bug 281905 in system-config-printer-kde "system-config-printer-kde.py crashed with AttributeError in on_tvNPDevices_cursor_changed()" [Undecided,Incomplete] https://launchpad.net/bugs/28190522:18
ArbyI'm pretty sure it's just a bad method name22:18
ArbyRiddell: http://paste.ubuntu.com/69775/ should fix it22:18
Arbybut I don't have the hardware to test the fix22:18
Riddellyeah, that'll be copied from the GTK version and not changed22:22
ArbyRiddell: great. the next one is more interesting22:22
ArbyRiddell: bug 28357522:22
ubottuLaunchpad bug 283575 in system-config-printer-kde "system-config-printer-kde.py crashed with AttributeError in on_btnNPApply_clicked()" [Low,In progress] https://launchpad.net/bugs/28357522:22
ArbyRiddell: should be fixed by http://paste.ubuntu.com/69776/22:23
ArbyRiddell: but the real question is what does cups have against whitespace22:23
Arbythe fix works but I wonder if it's the right solution22:24
Riddelltill kampper would be the guy to ask22:25
RiddellTill Kamppeter22:25
ArbyI'll try to catch up with him22:25
Arbydo you want to use this as a temporary solution or should we try to find the root22:26
Arby?22:26
Riddellthis is fine with me22:27
ArbyI've got one more as well. give me a minute22:27
Arbygreat, please commit whenever you have time22:28
RiddellArby: all three committed22:28
Arbythanks22:28
RiddellArby: last time I had a complaint about use of QFileDialog, it should be using KDE classes where they exist e.g. KFileDialog or KMessageBox22:28
ArbyRiddell: OK noted. I'll do some cleaning up soon then22:29
Arbylast one for now is bug 28268722:29
ubottuLaunchpad bug 282687 in system-config-printer-kde "system-config-printer-kde.py crashed with AttributeError in fillMakeList()" [Undecided,Confirmed] https://launchpad.net/bugs/28268722:29
RiddellArby: and there's another :)22:29
Arbyseems to be caused by cups not running22:30
ArbyI was wondering whether s-c-p-k should display a warning at startup if that's the case.22:30
Riddellit doesn't warn when cups isn't running at startup?22:31
Arbynope22:31
RiddellArby: it ought to22:32
ArbyI wrote some code that does this afternoon22:32
Arbyhaven't made a diff yet22:32
RiddellArby: but check the gtk s-c-p to see what it does, there might be code that can be copied, pasted and adapted22:32
ArbyRiddell: there is a try/except block for the relevant error22:33
Arbyso it gets caught but pretty much ignored22:33
Arbylet me make a diff, 2 ticks22:33
ArbyRiddell: possible solution http://paste.ubuntu.com/69784/22:37
RiddellArby: that should work, just needs i18n()22:40
Riddelland a decent dialog title e.g.  i18n("Print Server Not Running")22:41
ArbyRiddell: can I just wrap the whole string in i18n()22:41
Riddellyep22:43
Riddellnot with the tripple quotes22:43
Riddelljust use " quotes22:43
ArbyRiddell: ok, new patch coming up22:44
ArbyRiddell: new improved version http://paste.ubuntu.com/69787/22:46
ArbyRiddell:22:47
Arbyis it worth me making a pass through the whole code and trying to port to PyKDE as much as possible?22:47
Riddellyes that would be good22:47
ArbyI'll add that to the todo list then22:47
RiddellQMessageBox and QFileDialog are probably the most visible ones though22:47
Arbyok22:47
ArbyI've also made a start at implementing the ui spec that seele did22:48
smarterporting QPushButton to KPushButton should not be needed in most case22:48
Arbybut there's still a lot of work there22:48
RiddellI've never worked out what the difference is22:48
smarterKPushButton let you habe an icon next to the text22:48
smarter*have22:48
* smarter is doing to much German :p22:49
RiddellArby: excellent22:49
smarterand can take a "KGuiItem" but I don't really know what it is :p22:49
ArbyI've got most of the elements in place but the layout isn't quite right yet22:49
* smarter is interested in using his PyQt sk1llz for our distro tools22:50
smarterwhat needs love right now?22:50
Riddellqpushbuttons have had icons since qt 4.022:50
Arbyand the buttons aren't hooked up yet22:50
Riddellthe KGuiItem will be the important bit I guess22:50
smarterReally? I don't see anything about that on http://doc.trolltech.com/4.4/qpushbutton.html22:51
Riddellhttp://doc.trolltech.com/4.4/qabstractbutton.html#icon-prop22:52
smarterooohhh22:53
* smarter learned something today :p22:53
smarterthanks^^22:53
Riddellabstraction hides many things22:53
RiddellArby: committed22:53
Arbyquick bzr question. in 'bzr merge [LOCATION]' is location where I'm merging from or to?22:53
ArbyRiddell:  thanks22:53
RiddellArby: from22:53
Arbythanks again22:53
smarter(kpushbutton has drag support too)22:54
Riddellsmarter: all help lovingly received. apt-cache rdepends python-kde4, pick your favourite app, look up bugs, write patches, achieve fame and glory22:54
smarterhehe, okay ;)22:55
smarteris there a quick tutorial on pykde?22:55
* smarter knows pyqt, c++/qt, c++/kde, but not pykde (:22:55
Riddellsmarter: pykde is a pretty small step from pyqt22:55
Riddellsmarter: here's some good bugs if printer-applet interests you https://bugs.kde.org/buglist.cgi?cmdtype=dorem&remaction=run&namedcmd=printer-applet22:56
* smarter <3 printer-applet22:56
Arbybranches all tidied up, that's me done for now22:56
Riddellhmm, that URL won't work22:56
smarterimproving it will certainly interest me :)22:56
Riddellthis works https://bugs.kde.org/buglist.cgi?quicksearch=printer-applet22:57
smarterI'll try to look into that tuesday(which is an holiday here ;))22:58
smarterhttps://bugs.kde.org/show_bug.cgi?id=173091 sounds like it could be fixed with a simple foo.raise_()22:59
ubottuKDE bug 173091 in general "Click on System Tray icon should raise window" [Normal,New]22:59
Riddellfoo.show()23:00
smarter'night everybody23:16
lex79'night smarter23:20
* JontheEchidna is back23:27
RiddellJontheEchidna: we missed you23:34
JontheEchidna:)23:35
* JontheEchidna waves to a|wen-23:35
a|wen-hey JontheEchidna23:38
* a|wen- finally got his new laptop =) just installed intrepid and is migrating data now23:38

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