/srv/irclogs.ubuntu.com/2011/06/01/#ubuntuone.txt

nessitaok, I'm gone00:19
nessitabye all!00:19
=== txwikinger2 is now known as txwikinger
nigelbdobey: There was always a KDE thing called Neon.  Its the project which gets daily builds of kde.06:49
nigelbbut yeah, lack of imagination.  We should have an app which spits out random project names :)06:49
duanedesignhello nigelb06:52
nigelbhey duanedesign :)06:52
ralsinamorning07:08
duanedesigno/07:47
mandelmorning all08:44
duanedesigno/08:44
ralsinamorning mandel08:56
mandelralsina: buenos dias :)08:56
faganmorning09:10
ralsinamorning fagan09:11
faganralsina: want me to look into that nm bug09:12
ralsinafagan: good idea09:12
faganralsina: cool I have a good idea about how and what to do09:12
ralsinafagan: great!09:13
faganralsina: ill ask if I dont know anyway09:13
ralsinafagan: ok, go ahead09:21
* duanedesign wonders if mr rye is around?12:01
* fagan break12:27
nessitahello everyone!12:35
duanedesigno/12:35
faganmorning nessita12:35
faganlooking forward to london nessita?12:36
nessitafagan: I have different feelings, sometimes yes, sometimes I would like to stay at home (too much traveling lately). How about you?12:37
fagannessita: well im just interested in whats going on more as a learning experience and all that12:37
fagannessita: its always get being in those kind of meetings12:38
fagannessita: like I learned a lot at UDS when I was there, I learned a lot at the windows sprint so any more experiences to take in is a plus12:39
nessitaright12:41
nessitaralsina: hello there, you around?12:45
duanedesignanyone aware of any Tomboy-1.6.0 dataloss bugs that would cause you to lose notes?12:51
nessitaI'm not, maybe rye knows?12:57
=== teknico is now known as teknico_away
ralsinamorning nessita13:11
ralsinawas having lunch13:12
duanedesignmmmm, lunch13:12
ralsinaduanedesign: there is no data losage reported in the last month or so, AFAIK13:12
nessitaralsina: hello! was it a good lunch?13:17
ralsinanessita: pretty good yes!13:17
nessitaralsina: QT question, when you have a moment13:17
ralsinaright now is good13:17
nessitaok, so13:17
nessitaI need to "walk" a QTreeWidget in order to do some asserts over its content. Google suggest using a QTreeWidgetItemIterator, which kinda sounds right, but when I constructs a QTreeWidgetItemIterator from my treewidget, the result is None13:18
nessitathis is my code:13:18
nessitatree_iter = gui.QtGui.QTreeWidgetItemIterator(self.ui.ui.folders)13:18
nessitaand tree_iter is None13:19
ralsinalet me check...13:19
ralsinawell, according to the C++ docs that should work13:20
ralsinanow let me see if it's broken on python...13:20
ralsinanessita: workaround: http://www.mail-archive.com/pyqt@riverbankcomputing.com/msg11348.html13:21
* nessita reds13:22
nessitareads*13:22
ralsinabut there are examples of people using the iterator, so can you point me to the code?13:22
ralsinain fact, I don't know how tree_iter could be None, since that's a class instantiation :-D13:24
nessitalet me re confirm13:25
nessitait may be PICNIC13:25
* nessita checks for the third time13:25
nessitaralsina: so, you're right (as usual in this area) :-)13:34
nessitawhat I can't do, like you said, is iterate over that result13:34
ralsinahaha, you have found my area of expertise ;-)13:34
ralsinaI knew I had one!13:34
nessita:-)13:36
* fagan thinks he found the bit of code he needs13:39
fagannessita: Bug #790717 is in the cp not the sso13:40
ubot4`Launchpad bug 790717 in ubuntu-sso-client "Changes to the network manager break network detection (affects: 1) (heat: 6)" [Undecided,Confirmed] https://launchpad.net/bugs/79071713:40
fagannessita: its just in the detection of the network state13:41
nessitafagan: nopes13:41
nessitafagan: the main code is located in ussoc13:41
nessitasee the imports13:41
fagannessita: oh13:41
nessita     34 from ubuntu_sso import networkstate13:41
faganahhhhh13:41
nessita;-)13:41
fagannessita: well that shows why I was confused all morning13:42
nessitahehe13:42
faganralsina: if I make a branch and it gets merged into trunk to fix the network state thing wouldnt that break the nightlies for everyone no on 11.1013:43
fagan*not13:43
ralsinafagan: you tell me! You should make sure it doesn't ;-)13:43
faganralsina: good point13:44
mandelralsina: pong13:45
nessitafagan: if you try to fix it, please take into account the following:13:47
=== m_conley_away is now known as m_conley
nessitafagan: the code needs to be test driven or at least be accompanied with tests covering all the changes. And it has to keep working with natty, maverick and lucid, since we provide PPA for those :-)13:48
ralsinain fact, this is a neat case where a "small" fix is going to keep you busy for... a week or so ;-)13:49
fagannessita: yeah ill make sure its working on at least natty and 11.1013:49
nessitafagan: ping me for reviews!13:49
fagannessita: yeah harsh harsh reviews :D13:49
nessitanooooooo, lies!13:50
* fagan will break everything13:50
nessitaI'm a good girl13:50
ralsinanessita harsh? naaaaaaaah13:50
nessitaexactly13:50
* nessita is a sweet dolly doll-like girl13:50
faganoh one question13:50
* nessita thinks this f****ng trees are making my life miserable 13:51
beunonessita, wasn't chucky a doll as well?13:51
duanedesignlol13:51
nessitabeuno: but that was a male doll :-D. Though hard to guess, I'm a girl doll13:51
faganactually nope false alarm no question13:51
nessitaor did chucky have a girlfriend?13:51
beunonessita, he did in Chuky 2!13:52
beunoa bride13:52
duanedesignbeuno: the one and only time I ever hitchhicked the person had a Chucky doll in their car. o.O13:52
nessitaLOL13:52
duanedesignthe bride of Chucky :)13:53
beunoduanedesign, does this story end in a tub full of ice?13:53
faganIs it wrong that I dont know what you guys (and gal) are talking about13:53
ralsinafagan: yes, it's unforgivable13:53
nessitathe unforgiven!13:53
faganhahah13:53
ralsinaspecially about "Bride of Chucky"13:53
* nessita requests metallica13:53
duanedesignbeuno: :D fortunately no.13:54
* fagan googles13:54
ralsinafagan: careful, that movie contains puppet intercourse13:54
faganralsina: oh no then its not my kind of film13:55
ralsinafagan: it was necessary for the plot!13:55
* fagan is still a little sick from a flash of two girls one cup13:55
faganralsina: names of vars13:57
ralsinafagan: it's not THAT kind of movie, you sicko13:57
faganralsina: I need to do a second state number for 11.10+ so what should I call it13:57
faganralsina: hahah13:57
ralsinafagan: I don't follow you13:57
faganralsina: well at the moment they are like NM_STATE_DISCONNECTED = 413:58
faganthats a different number for the new spec13:58
faganso id need two vars 1 for the old style one for the new style13:59
voidspaceralsina: Chipaca: ping13:59
ralsinaif there is a way to see the NM version, just check and set the right numbers13:59
faganwhat should I call the var13:59
ralsinavoidspace: pong, saw the CV13:59
voidspaceralsina: cool, after I sent it William realised he forgot to include details of his education in the cv13:59
faganoooh that gives me an interesting idea on how to take on this13:59
ralsinavoidspace: I am thoroughly impressed and want to have his children, or something like that ;-)13:59
voidspaceralsina: I have an updated version if you want it - but he already put the updated version into the system14:00
voidspaceralsina: hah :-)14:00
nessitame14:00
ralsinabesides, I *have* tried to write a PLY thing to parse excel into python and am amazed someone succeeded ;-)14:00
faganme14:00
ralsinavoidspace: I will mail him in like 15 minutes14:00
voidspaceralsina: anyway, snap him up soon or we'll hire him for our team - Gustavo has already emailed him to arrange a voice chat about working on ensemble14:00
ralsiname14:00
voidspaceralsina: so the race is on...14:00
ralsinavoidspace: :-)14:01
voidspaceralsina: yeah - the whole excel formula syntax -> python translate pipeline was great fun14:01
voidspaceralsina: I worked on it quite a bit too14:01
nessitaalecu, thisfred, dobey?14:01
thisfredme14:01
voidspaceralsina: but most of it was already implemented (by Will) when I joined Resolver - although we continually added new features14:02
ralsinavoidspace: you know the original excel parser is not actually a parser, right? It's just a stack of handmade if chains14:02
dobeyme14:02
voidspaceralsina: I didn't know that14:02
faganralsina: mandel too no?14:02
voidspaceralsina: we did it properly :-)14:02
nessitaDONE: more Cloud folders tab, started adding tests for that, got stucked and unstucked several times (but I keep moving forward).14:02
nessitaTODO: please, finish the cloud folders!14:02
nessitaBLOCKED: not ATM14:02
nessitaNEXT: fagan14:02
nessitafagan: nopes14:02
faganDONE14:02
fagan* finding where to do the work for the nm bug14:02
fagan* payroll14:02
faganTODO14:02
fagan* fix the bug14:02
fagan* investigate if there is a way to query the nm version to set the var nicely rather than have some extras14:02
faganBlocked14:02
fagan* nope14:02
voidspaceralsina: anyway, I'll leave you in peace - but really you'll need to move quickly if he's interesting...14:02
faganralsina: I think is next14:02
faganyep gogo14:03
ralsinaDONE: meetings, reviews, pinged design to confirm we have signoffs for everyhting, administrivia (yes, the same thing every day)14:03
ralsinaTODO: same as every day. Try to conquer the world pinky!14:03
ralsinaBLOCKED: no14:03
nessitathat only happens at nights!14:03
ralsinathisfred?14:03
thisfredDONE: cry hot couchdb tears of sadness. Also was unhelpful with at twisted trial issues. TODO: more couchdb debugging BLOCKED: no NEXT: dobey14:03
dobeyλ DONE: bug #789299, lots of debugging on 78930014:03
dobeyλ TODO: bug #789300, bug #77148814:03
dobeyλ BLCK: Twisted.14:03
ubot4`Launchpad bug 789299 in ubuntuone-dev-tools "DBusTestCase sometimes connects to real session bus (affects: 1) (heat: 5)" [Critical,In progress] https://launchpad.net/bugs/78929914:03
ubot4`Launchpad bug 789300 in ubuntuone-dev-tools "DBusTestCase needs to work with Qt main loop as well (affects: 1) (heat: 6)" [Critical,In progress] https://launchpad.net/bugs/78930014:03
ubot4`Launchpad bug 771488 in ubuntuone-dev-tools "u1trial should unset GTK_MODULES (affects: 1) (heat: 4)" [Medium,In progress] https://launchpad.net/bugs/77148814:03
ralsinavoidspace: I will go really quick now!14:04
nessitadobey: any idea why your branch is not being merged?  https://code.launchpad.net/~dobey/ubuntuone-dev-tools/dbus-priv/+merge/63022 I approved it this morning14:04
dobeyno, i'll look14:04
nessitathanks14:04
nessitaralsina: is alecu coming today?14:04
ralsinanessita: AFAIK he should14:04
* fagan hopes tarmac doesnt blow up before the sprint 14:05
dobeywell it's not a huge deal if it does next week. not like there will be a massive amount of branches being landed14:05
ralsinafagan: mashalla! <knock on table>14:05
faganok then I thought there would be at least some development going on next week14:06
thisfreddobey: actually from what I heard there will be more coding and less talking than last yeat14:06
thisfredyear14:06
dobeywtf cron14:06
faganI suppose I wasnt exactly fully aware of the entire content of the week14:07
dobeythisfred: i thought there were just going to be more cage matches (team building)14:07
Chipacadobey: 4 of the 5 days are "mostly" hands-on14:07
thisfreddobey: remember the first rule of sprint club14:07
dobeyChipaca: those are the best kinds of cage matches14:07
faganthisfred: ooh I know "dont talk about sprint club"14:09
alecuHola gente!14:09
thisfredholalecu!14:09
ralsinaalecu: say me!14:09
alecusorry I'm late, got delayed getting to kinder.14:09
alecume14:09
=== teknico_away is now known as teknico
ralsinaalecu, go ahead ;-)14:10
* fagan hugs dfeet14:11
dobeymake sure you wash dfeet before you hug them14:11
fagandobey: I like it dirty14:12
fagan:14:12
fagan:D14:12
alecuDONE: worked on the "devices" tab; worked on a hard merge with nessita's latest branch14:12
alecuTODO: finish merge and devices branch14:12
alecuBLOCKED: some weird dbus initialization issues are blocking the merge14:12
* fagan grew up in the christina agulara generation 14:12
alecuyuk14:12
dobeyshen me niao!14:13
dobeyok, so why would cron just stop working all of a sudden?14:14
dobey'cron' appears to be running, as does 'atd'14:14
dobeybut it hasn't processed any jobs for 2.5 hours14:14
ralsinadobey: you know noone will know. You can give me shell access and I can probably guess using strace ;-)14:16
dobeywell i restarted cron and atd, but i think i did it just after the time had passed for it to run again, so will wait until 9:30 and see14:17
* fagan is terrible with dbus 14:18
dobeyralsina: ah, apparently there was a pam security update that broke it14:24
dobeyralsina: i guess it got installed automatically on my lucid server just before 07:00 today14:24
dobeywhich is why the last run of tarmac was at 06:45 on that box14:24
ralsinadobey: fun!14:24
dobeyok14:25
dobey5 minutes and my branch should land then14:25
dobeyhuzzah, it is back14:32
thisfreddobey: hmm, I wonder if that explains why my mail suddenly stopped being fetched automatically yesterday. What did you do to fix cron?14:32
thisfredor does it fix itself?14:33
dobeythere is another update today for pam that fixed it14:34
dobeyso just upgrade again14:34
thisfredah ok14:34
dobeyok, back to this awful twisted mess14:37
* fagan sees the irony of the name twisted when dobey talks that way 14:40
dobeyi'm not entirely sure what to do here14:42
* ralsina records the moment for posterity14:43
dobeyok, well i'm sure of what to do, but you won't let me do it :)14:44
* fagan break 14:44
ralsinadobey: you are probably right ;)14:44
dobeyand it still wouldn't solve the problem in the case where we do have to use twisted14:44
fagandobey: can14:45
ralsinadobey: care to explain a little bit what the problem is? Sometimes it helps14:45
dobeytwisted's TestCase does not have a setUp or tearDown defined itself14:45
* fagan was going to ask a question but then realised he read it in a book already anyway :)14:46
dobeyand it expects all subclasses to return deferreds in setUp/tearDown14:46
dobeyand also in test_foo()14:46
thisfreddobey: well, not exactly true: If *any* of the subclasses return a deferred in setUp or tearDown, they will all have to and take care of properly chaining those deferreds14:47
dobeybut, it seems, since it doesn't have those methods, when we do super() to call them, the ultimate setUp which gets called, is from unittest.TestCase, which returns None (because it's just an empty function)14:47
dobeythisfred: i am just trying to make it work correctly in devtools at least first; and it's not complex enough to hit that issue :)14:47
thisfredright, that's the real problem14:47
dobeyand so even when we check for the None result, and set up our own deferred, the method i added as a callback, never gets called, so the setUp pieces never actually get done14:48
thisfreddobey: well if there wasn't the dbus test case, I'm pretty sure the super would just work14:48
dobeythisfred: well it does work, if we don't try to deal with deferreds14:49
ralsinahmmmm14:49
* thisfred stops confusing the issue14:49
thisfredright14:49
dobeyexcept when we try to be good citizens, and deal with deferreds, it breaks14:49
ralsinawe *could* monkeypatch  TestCase14:49
* ralsina ducks14:50
dobeyi don't think it matters14:50
dobeythe problem isn't necessarily that the parent TestCase has no setUp/tearDown14:50
dobeythe real problem is the deferreds14:50
dobeyto fix it properly, i think we'd need to rewrite much of the twisted TestCase14:50
ralsinaright14:51
dobeybecause, basically, there's no way to do synchronous code with it14:51
dobeyand we need to make synchronous calls14:51
ralsinaI'm surprised noone ever ran into this14:51
dobeywhich is why all of our tests always end up failing completely randomly with nonsensical errors14:51
thisfredwell we're doing synchronous code with asynchronous superclasses14:52
thisfredhaving  a completely synchronous TestCase hierarchy works.14:53
ralsinawe aren't the first ;-)14:53
ralsinahttp://twistedmatrix.com/pipermail/twisted-python/2008-July/018120.html14:53
dobeyyeah the inlineCallbacks rubbish is what breaks :)14:55
ralsinadobey: see? Everything has been tried before ;-)14:55
dobeywhich is ironic14:56
dobeybecause "do what twisted wants" is when everything breaks14:56
dobeybut "do what seems right to me" is when everything works14:56
dobeyso i can only take this to mean that twisted is obviously just completely wrong14:56
dobeybut alas14:57
thisfreddobey: have we tried using returnValue()15:01
thisfredI think that may be the missing part15:01
dobeywhat is that?15:01
thisfredwith inlineDeferred, the method cannot return a value, because it's a generator, so it always yields None by default15:02
thisfredbut twisted has a returnValue() that can return the deferred instead15:02
thisfredhttp://twistedmatrix.com/documents/8.1.0/api/twisted.internet.defer.html#inlineCallbacks15:02
thisfredand http://twistedmatrix.com/documents/8.1.0/api/twisted.internet.defer.html#returnValue15:03
fagan\o/ ralsina I got the version of network manager15:03
dobeywtf15:03
thisfredyeah15:03
thisfredtwisted is a mess15:04
ralsinafagan: cool, you are 5% there ;-)15:04
faganralsina: well now all I need to do is sub in the other values and it should work15:04
faganralsina: (when the version is greater than the version im on since that was when the sate values changed15:05
fagan(15:05
fagan)15:05
ralsinafagan: making it work is 10% of the work. You need to create a test case that fails without your fix, and works with the fix, too15:05
faganralsina: yeah I know ill have it tested to hell15:06
thisfredwell, ideally you'd start with that test15:06
ralsinafagan: what thisfred said ;-)15:06
thisfredwhich is easier than writing it afterwards15:06
ralsinanow you have to *remove* the fix before writing the test15:06
faganyeah I know im trying to make sure that it is just right15:07
dobeyhrmm15:07
dobeyreturnValue seems to make things worse, not better15:07
dobeybut that follows with my experience of using twisted :-/15:07
dobeyso not exactly unexpected15:07
dobeyhrmm, i think i need to postprocess some of my mp3s, particularly the classical pieces; so that the average volume is higher15:08
dobeycan barely hear this symphony 1 in d minor recording15:09
thisfredmp3gain/vorbisgain are your firends15:09
dobeyyeah15:09
dobeybut "having to do stuff" is not my friend15:10
ralsinaok, coffee break for me.15:10
dobeysays the guy who is designing a language15:10
thisfreddobey: find -L . -iname '*.mp3'  -print0 | xargs -0 mp3gain -k -r -f15:10
thisfreddone15:10
thisfredalthough that does not do album gain, which you may want for classical15:11
dobeyright15:11
dobeyactually, what i *want* is for the music player to be smart15:12
dobeybut i suppose that's asking too much in the world of foss15:12
fagan:( the version numbers in nm are dumb15:12
dobeyuh, no they aren't15:12
faganwell the version number is 0.8.9997 in 11.1015:13
faganIt means ill have to do a str comparison and id prefer to do a float15:14
dobeyyes15:14
dobeyuhm15:14
faganim up for suggestions on how to do it though I may be just seeing it wrong15:15
faganI was thinking of dropping the leading 0. and doing a float and having it going >= the version that state changed15:16
thisfredIt's not that hard if you don't have to handle alphas, betas, rcs etc15:16
thisfredfagan: then what happens when it hits 1.0?15:17
dobeyseriously, version comparisons is like the one thing that python can actually do well15:18
fagandobey: could you explain it to me why that is15:18
faganthisfred: well id say im just overcomplicating it because im overlooking something that other peole might know15:19
dobeybecause it is dynamic15:19
dobeyalso, floats are stupid15:20
thisfredfagan: so write a test that covers a single case, and make that test pass, then think of a use case that is not covered by the existing tests, and repeat15:20
thisfredthis prevents analysis paralysis15:21
faganthisfred: ok then thats cool so ill make it work first and work backwards15:21
faganweird question, why are we doing the network state thing when we could use the bool that says if you are connected instead15:25
faganid say that would be more of a question for alecu since he wrote it15:25
dobeymeta-x pyshcoanalyze-zippy15:26
faganand it would be backwards compatable too if we did it with the bool15:26
dobeyi don't know what bool you're talking about, but you are wrong15:27
fagandobey: there is a var in the dbus that we have access to that says if you are connected or not15:27
faganNetworkingEnabled15:28
dobeythat has nothing to do with connectivity15:28
dobeystop making assumptions15:28
fagandobey: hmmm well I see what you mean now sorry15:29
dobeywhy oh why is twisted so insane15:35
alecufagan, make a tuple out of the version string, and compare with a version tuples15:36
faganalecu: cool will try that, just working it out with strings first and ill do that after15:39
dobeythe only base type in python is object. any US-ASCII encoded number in a "string" can be an int15:46
dobeysigh15:48
dobeyException dbus.exceptions.DBusException: DBusException('Did not receive a reply. Possible causes include: the remote application did not send a reply, the message bus security policy blocked the reply, the reply timeout expired, or the network connection was broken.',) in <bound method BusName.__del__ of <dbus.service.BusName com.ubuntuone.controlpanel on <dbus._dbus.SessionBus (session) at 0xeaaaf8c> at 0xeb1768c>> ignored15:50
dobeyi wonder where that came from. it is weird15:50
dobeybut i get that in control-panel, and it has some very very weird usage of test cases15:51
nessitaralsina: ping?15:53
dobeyok this is messed up15:54
dobeytcole: ping15:58
dobeyhrmm, does super() do the right thing with MI?15:59
* dobey wonders16:03
thisfreddobey: It does, though not always the thing you'd expect, depending on how well you know the MRO16:06
thisfreddobey: if you're wondering MI is the problem, change the order of the baseclasses in the class that does MI16:07
dobeyinteresting16:07
thisfredthis *should* only matter if both of them implement the method you're supering to16:08
thisfredbut if they do, it does matter16:09
dobeyok16:09
dobeyNOT interesting :'(16:09
dobeyswitching to super() with @inlineCallbacks in the u1client tests, caused memory usage to go to > 1.3G RSS, and caused lots of other errors in the tests directly16:10
dobeyso uhm16:10
dobeytime to rewrite i think16:10
dobeyi think i'm going ot have to reboot now, just to fix up my memory usage16:11
dobeyswap == slllllllllllllllllllloooooooooowwwwwwwwwwwwwww16:12
dobeyso fixing ubuntuone-client seems to be not an option right now16:21
thisfredlet me know if there's anything I can pick up or look at16:23
dobeyi am not sure there is16:26
dobeyi think "do what actually works now" is better than "do what twisted seems to expect, but always fails to work correctly"16:26
dobeyso i think i'll just propose what i have now16:26
thisfredright. Fixing twistedtestcase is out of scope. We should contact jml to see if he knows a workaround that does work, I think, but barring that...16:29
dobeywell, i am not sure it is as simple as "fixing twisted's testcase"16:31
thisfredmaybe not16:32
dobeyhttps://code.launchpad.net/~dobey/ubuntuone-dev-tools/dbus-strict-priv/+merge/6313416:38
thisfredlooking16:39
dobeywrong button...16:40
dobeydon't click the (x)16:40
thisfredactually I hit ^R to refresh my browser while xchat still had focus16:41
thisfredI do that a lot16:41
thisfreddobey: so what I don't understand is, if we're not using inline callbacks, why do setUp and tearDown still need to be generators?16:42
dobeywell, it doesn't matter, but i'd rather have the change to go back to inlineCallbacks be the simplest possible16:43
thisfredok, looks good to me, +1ing16:44
thisfredthough tim should probably add his 2¢16:45
faganThink I have it16:48
dobeytcole: ^^16:50
dobeyi guess tcole's hilight in irc is acting up16:50
thisfredor he might not be up yet, west coast slacker ;)16:51
dobeyhe's usually up early16:52
dobeyhe's usually up earlier in EST, than when he actually lived over here :)16:52
thisfredI'm actually seriously considering moving to Portland next year, but getting up at 6 or so will take some getting used to16:52
thisfredprobably 5 if I have to walk the dog16:53
dobeyalso, it's portland16:53
thisfredyeah, we covered that :)16:53
dobeyi don't know how you'd survive without the weekly murders16:53
faganAnyone on 10.10 or 10.04 that can test a branch for me16:53
fagannessita: ^?16:53
dobeywhat branch?16:53
nessitafagan: I'm 11.0416:54
fagandobey: the one that fixes the nm thing16:54
fagandobey: and hopefully doesnt break everything else16:54
dobeynm hasn't changed since before lucid, until O16:54
nessitadobey: he's proposing a change to work with O, and he can't break former clients :-)16:55
fagandobey: well 11.10 the states have changed16:55
dobeynessita: yes, but 11.04 is old enough :)16:55
faganoh you guys refer to the new release as 0?16:55
dobeyfagan: it is Oneiric Ocelot16:56
nessitadobey: right, but he should (the same) test older clients, as a final test16:56
dobeyor Ocular Oroborus16:56
fagandobey: I cant spell it so I just go 11.1016:56
fagan:D16:56
dobeynessita: well we don't test every branch on 10.04 before putting code in trunk16:57
nessitadobey: right. Maybe we all should, but since fagan is new to our source tree we're asking him to16:57
fagannessita: well ill make sure it doesnt break on 11.04 first and since it hasnt changed in the previous releases I can just ask for a review on it and walk them through whats up16:58
faganrather than going back in the releases since it didnt change anyway16:59
* fagan tests (just to make sure)16:59
* alecu will have lunch16:59
dobeyah lunch17:00
dobeyi need to get lunch too. and also have an appointment and some errands to tend to. bbl17:00
fagannessita: ok just tested it myself and its working on 11.0417:08
fagancan you check it out for me17:08
faganI can do it in a merge im pretty sure17:09
faganor ralsina for that matter can look at it17:09
faganfeck it ill just do the merge proposal and ask for you guys to look at it if I did it wrong ill fix it but it should work17:10
nessitafagan: what's the link to the merge proposal?17:11
nessitayes, I can look at it, just send me the link17:11
nessitaI'll look after lunch, which is ready right now! :-)17:11
* nessita brbs17:11
=== beuno is now known as beuno-lunch
fagannessita: cool ill make the merge and ping you with it17:13
faganreviews please https://code.launchpad.net/~shanepatrickfagan/ubuntu-sso-client/nm-state-bug-fix/+merge/6313617:13
fagannessita: we can ask for 3 reviewers for me for this one just to make sure17:14
faganI have to say I was being a bit cheeky when I wrote the line that checks the version number but it works perfect so I cant complain17:16
faganit may have been unnessiary though I probably did a few things wrong17:17
* fagan checks if he could have done it differently 17:17
faganok working the easier way and tested on both versions again17:22
* fagan feels good about this now 17:22
faganOk have to go to the shop be back in a few minutes and then will EOD17:23
nessitafagan: there is no tests for this change :-)17:31
nessitafagan: you need to add tests to assert over each branch of the if17:32
fagannessita: well sudo python install and then fire up the cp17:32
nessitafagan: no no, you're not following :-)17:32
nessitafagan: we require all changes to be accompanied with unit tests17:33
nessitabesides the usual IRL testing that reviewers should do17:33
fagannessita: with mocker?17:33
* fagan probably is still lost :S17:34
nessitafagan: with whatever is needed, you should check networkstate/tests/linux.py17:34
nessitaand add the proper tests for this new functionality17:34
fagannessita: ahh ok well ill do that tomorrow then I suppose17:34
fagansince thats something new to figure out too17:34
nessitaok, I'll add this comments to the merge proposal17:34
fagannessita: other than that does the code look ok?17:35
nessitafagan: in general, yes. But for example you define self.NM_STATE_UNKNOWN in one branch of the if and not in the other17:35
nessitathat will make the code not work properly for all cases17:36
fagannessita: cool will fix that (its the same in both cases I just forgot to move it)17:37
nessitaack17:37
faganlike that var is 0 for both17:37
faganfixed17:37
faganok so thats something nice to work on tomorrow17:39
faganactually writing the test cases17:39
nessitafagan: thanks for helping with this!17:39
fagannessita: well making myself useful is always good17:39
faganlearning + helping = winning17:39
nessitafagan: also, tests do not run in your branch, I'll add that to the MP as well17:40
fagannessita: in what way?17:40
faganso they dont run at all on it?17:40
nessitaright, see the comment I added17:41
faganwill check it out17:41
faganah so the test imports the vars the way they were before17:42
faganI think I can fix that pretty easy17:42
fagananyway thats work for tomorrow EOD17:43
nessitaenjoy17:44
fagannessita: thanks for the help17:45
nessitaanytime17:45
=== didrocks1 is now known as didrocks
thisfredbah, couchdb+erlang in natty is broken, as well as in oneiric18:02
thisfredbut differently18:02
thisfredralsina: I don't know what the best way forward is wrt ^ / bug #66502418:03
ubot4`Launchpad bug 665024 in erlang (Ubuntu) (and 3 other projects) "Desktopcouch doesn't replicate with json_encode error 500 (affects: 8) (dups: 2) (heat: 46)" [High,Confirmed] https://launchpad.net/bugs/66502418:04
thisfredFor Oneiric, packaging couchdb 1.0.3 or 1.1.0 (to be released this week, with any luck)18:04
thisfredFor natty, patches to erlang-ssl and maybe couchdb.18:05
thisfredI think18:05
thisfredI best get on those I guess18:05
nessitathisfred: any idea when those broke?18:06
thisfrednessita: not really, erlang-ssl has a problem in r13, which was fixed in r14, but there was never an official bug for it anywhere18:07
thisfrednessita: the problem is the bug is apparently not always triggered18:07
thisfredit has to do with chunking and ssl, I bet18:08
nessitaright, otherwise we had have tons of reports18:08
thisfredwell, we have quite a few18:08
thisfredbut I didn't see it myself until today18:08
thisfredand the logs don't show any useful info unless you turn debug level logging on. And then ask someone in #couchdb to explain the logs... :S18:09
nessitathisfred: maybe we should email ubunet-discuss to ask for advice?18:10
nessitaare the fixes SRUables?18:10
thisfredyes, in the sense that the erlang-ssl patch is not big, but since it touches ssl, it will have to be reviewed quite carefully, by someone who speaks erlang, I suspect18:11
thisfredthis is the patch https://gist.github.com/60380818:11
thisfredI am first gonna see if it even solves the problem, and if so, put the version into our nightlies, so at least some people have the benefit of the fix18:12
thisfredthen try to sru it18:12
nessitathisfred: makes tons of sense18:12
thisfredshould help with my goal of becoming an Ubuntu dev, I guess.18:12
thisfredglass -> 1% full18:13
faganthisfred: are you an ubuntu member?18:13
thisfrednope18:13
thisfreddepending on what you mean by member18:13
faganthisfred: well do you have @ubuntu.com email reallu18:13
fagan*really18:13
thisfredyeah  I think I do, though I've never used it.18:14
=== beuno-lunch is now known as beuno
thisfredI think everyone at canonical gets that still, but we're no longer automatically ubuntu developers18:14
thisfrednot sure though18:14
faganthisfred: I dont think everyone at canonical gets one I remember the release manager had to apply and she got turned down inially for it18:15
faganthisfred: you could try to send an email to your lp id + @ubuntu.com and see anyway18:16
faganAnyway I have to reboot see you all tomorrow18:16
beunoeveryone does get it18:16
beunowe're not suppose to use it unless we're Ubuntu members  :)18:16
thisfredah right. So I'm a good boy18:18
thisfredcookie?18:18
* beuno pets thisfred 18:18
* beuno throws a treat in the air18:18
=== kepyla_ is now known as kepyla
* dobey shakes his head18:42
* teknico shakes his snakes18:43
dobeyshaking snakes can be dangerous dude18:53
nessitaalecu: you still fighting the big merge? :-(18:56
thisfredIs that teknico in the photo on http://en.wikipedia.org/wiki/Snake_handling ?19:08
ralsinathisfred: still around?19:12
thisfredralsina: yep19:15
ralsinaI suppose what you said is the right course, even if it's a huge pain19:17
thisfredralsina: yeah, building erlang with the patch now, to see if it even works19:17
ralsinaI don't know who we have handy that can back you up on erlang19:17
ralsinaspecially for something security-related19:17
nessitaI would say Elliot...19:19
thisfredmaybe, or Gustavo maybe19:19
ralsinacool, then, good luck with that19:20
nessitaralsina: you available for a review?19:20
nessitanot sure how late is there19:20
thisfredanyway, if we propose the SRU, and motivate it, the SRU review process should take care of that19:21
ralsinanessita: I just had a 4 hour coffee break, I am here for an hour or two ;-)19:21
nessitaralsina: yey! https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/list-folders/+merge/6314919:21
nessitaralsina: be strict with the QT stuff, please19:22
ralsinanessita: ok, I'm looking at it19:22
ralsinanessita: will try to curb my easygoing nature19:22
nessitaI'll build some mate in the mean time19:22
nessitaralsina: jejeje19:22
dobeymake: *** No rule to make target `mate'.  Stop.19:23
nessitalol19:23
ralsinanessita: what's the format of the icon images? jpg, png?19:23
nessitaralsina: what icons? the one I'm setting by name?19:23
ralsinaalecu, nessita, quick mumble in say 15 minutes?19:23
nessitasure19:23
dobeybmp FTL19:23
ralsinanessita: yes, the files for those icons19:23
ralsinanessita: there is a packaging problem with non-png images19:24
nessitaralsina: the files are being distributed by the theme, not by us19:24
ralsinanessita: ok, we need to have a copy for the windows version...19:24
nessitaralsina: right19:24
nessitawe can ask the icons to our designer and make those the defaults if the theme's are not available19:25
nessitasince fromTheme accepts an optional argument for a fallback19:25
ralsinahahaha, humanize goes to YiB :)19:25
nessitawel...19:25
nessitawell, Chipaca inspired me ;-)19:26
ralsinaI know, same effort ;-)19:26
=== zyga is now known as zyga-afk
nessitaok, now yes, build mate time19:29
dobeyralsina: png? i thought they had to be bmp?19:29
ralsinadobey: nope, png19:30
ralsinadobey: the problem is with deploying Qt modules, and the only "non-plugin" image format is png19:30
ralsinatouch mate.c; make mate19:30
ralsinaof course, it mails, but is more interesting that just make mate19:31
dobeyralsina: ok, that shouldn't be a big issue then19:31
ralsinadobey: nope, it was more annoying on SSO because the captcha is a JPG19:31
dobeyright19:32
ralsinas/mails/fails 2 before19:32
nessitaralsina: but in SSO the captcha is not distributed, but downloaded...19:33
nessitawas that an issue as well?19:33
ralsinanessita: yes, because we needed to create a QPixmap out of it to display it19:33
nessitaah...19:34
ralsinamanuel did a workaround using PIL but that's ugly19:34
nessitaso is difficult to show a JPG using QT? sounds... poor19:34
ralsinano, shwing it using Qt is trivial19:34
dobeywell, apparently not :)19:35
ralsinaShowing it using PyQt from a frozen exe on windows... not trivial. It breaks when you have PyQt installed19:35
ralsinanessita: line 492 f the diff19:36
ralsinawhat you are doing probably works, but did you check every possible flag? Keep in mind you are restting all the others19:36
nessitaralsina: yes, I did19:38
nessitaralsina: is there any other (cleaner) way?19:38
ralsinacleaner?  well, you can take item.flags() and OR them with the ones you want set19:39
nessitaright, but I want to unset flags19:39
nessitashall I OR with the ~ of those I need unset?19:39
ralsinayes, too19:40
ralsinathat's why I am not sure it's actually cleaner ;-)19:40
nessitaright, in this case I think I prefer the absolutely explicitness19:41
ralsinaalecu: ping?19:41
nessitaif we miss a flag we will find out soon enough19:42
ralsinasurely19:42
ralsinanessita: if it was intentional, it's A-OK19:42
ralsinanessita: +1 from me19:43
nessitaralsina: thanks!19:43
nessitaalecu: review please? https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/list-folders/+merge/6314919:43
ralsinaalecu seems to be gone19:44
nessitaright, he mentioned having lunch at 1pm ART, maybe he got delayed?19:44
ralsinaok, I can wait for him another 15 minutes19:44
nessitaralsina: want me to call him or text him?19:44
ralsinaor I can come back in one hour and have our talk19:45
nessitais cheap for me :-)19:45
ralsinaok, at 5PM your time we talk19:45
nessitaok, I'll let him know19:45
nessitaI texted him, let's see if he responds19:48
dobeytcole: are you there?19:48
nessitaralsina: alecu says he will be here in 15', shall we have the meeting then or at 5pm ART?19:48
ralsinaok, in 15 is fine19:50
dobeyhrmm, i really want to get this branch in19:54
dobeyoh well, brb; gotta drop a couple things in the post19:54
tcoledobey: pong19:57
tcoledobey: what's up?19:57
alecuralsina, nessita: I'm back. Sorry for the delay, had to go to the bank and run some "trámites"20:06
nessitaralsina: so, meeting? are we still in time?20:06
ralsinaI'm here20:06
ralsinalet's have a quick chat in #meeting2011?20:07
nessitaah, no mumble? ok20:07
ralsinaNo mumble because tato is asleep20:07
dobeytcole: can you review https://code.launchpad.net/~dobey/ubuntuone-dev-tools/dbus-strict-priv/+merge/63134 please?20:10
tcoledobey: okay, I'll have a look20:10
thisfrederlang fix seems to work and not break other stuff. Uploaded to nightlies20:11
dobeyhmm20:14
tcoledobey: +120:19
dobeyyay!20:19
dobeythanks!20:20
dobeyweird20:34
dobeyi think lp automatically scheduled rebuilds of a recipe, when i pushed the packaging branch20:34
dobeyor i requested the builds and for some reason don't remember doing it20:34
dobeyconfusing20:35
dobeythisfred: uhm20:45
thisfreddobey?20:46
dobeythisfred: are you running oneiric already?20:46
thisfredwhat'd I break?20:46
thisfrednope20:46
dobeythisfred: why'd you upload erlang to the nightlies ppa as an oneiric package then?20:46
thisfreddobey: I tried yesterday, and it completely broke my laptop20:46
dobeythisfred: it's older than the version which is already in oneiric20:47
thisfreddobey. hmm, I branched it from the natty package, so why would it say oneiric20:47
dobeythisfred: are you *sure* you're not on oneiric? :)20:47
thisfreddobey and I know it's older than the O version20:48
dobeywhere did you branch it from?20:48
thisfredthat one doesn't have the problem (though it has another one)20:48
thisfreddobey ubuntu:n/erlang20:48
dobeythisfred: hrmm, and you should have added the ~seriesN to the end20:49
thisfreddobey yes, I realized that just after the upload. I'll delete it from the ppa and try again20:49
dobeyok20:49
dobeymake sure changelog says 'natty' this time too :)20:50
thisfredyeah, weird it would change that to O20:50
thisfreddobey what would ~seriesN be in this case?20:50
dobeywell for natty i think ~natty1 is appropriate20:51
thisfredkk20:51
thisfreddobey: done. And yeah dch -i sets it to oneiric by default. Which I guess is smart, since SRUs would be the exception rather than the rule20:55
dobeyno it shouldn't do that on 11.04 i don't think20:56
dobeyit doesn't here20:56
dobeydch -i gives me a new entry with 'natty' as the series target20:56
dobeynot -proposed or -backports or anything, just natty20:57
thisfreddobey: no idea why it doesn't here then20:57
thisfredI am definitely not on Oneiric20:57
dobeyweird20:58
dobeyblah why is it taking so long for lp to publish this package21:01
=== alecu_ is now known as alecu
* alecu will run to fetch Amelia from kinder, and will be back later.21:32
=== m_conley is now known as m_conley_away
dobeyalright, need to go do other stuff; have a good evening all!22:04
nessitadobey: you too22:04
thisfredlater all22:45
jeffreyfhello.  I am getting File Sync error. (auth failed (AUTH_FAILED)).  It brought in my other devices that are registered, registers this computer also....but it doesn't sync23:45
jeffreyfthis is what I did23:57
jeffreyfhttp://pastebin.com/REkZrmDM23:57

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