| * karni waves goodbye | 01:52 | |
| === smb` is now known as smb | ||
| Chipaca | morning, peeps | 08:34 |
|---|---|---|
| JamesTait | Happy Friday, everyone! :D | 09:31 |
| czajkowski | JamesTait: you're way too happy | 09:41 |
| JamesTait | czajkowski: Only because I see your face on my Facebook timeline in the mornings. ;) | 09:41 |
| czajkowski | JamesTait: ROFLOL | 09:42 |
| davidcalle | Chipaca, hello. I'm going to ask for an upload today, with what we have in trunk. Any other change we could have missed? | 09:48 |
| Chipaca | davidcalle: nope. Only remaining change in my mind at this moment would be when we move the server to something.ubuntu.com | 09:51 |
| davidcalle | Chipaca, ok | 09:53 |
| gatox | good morning! | 11:09 |
| mandel | gatox, morning! | 11:31 |
| gatox | mandel, hi | 11:32 |
| gatox | brb (in 5min) | 12:36 |
| gatox | back | 12:42 |
| nessita | hola gatox | 12:49 |
| gatox | nessita, hi | 12:49 |
| nessita | gatox: show-errors is approved, I'm looking at the last one now | 12:50 |
| gatox | nessita, great! | 12:50 |
| gatox | nessita, everything looks prettier now? :P | 12:50 |
| nessita | yes! | 12:50 |
| gatox | at least a little bit more..... | 12:50 |
| gatox | jejeje | 12:50 |
| nessita | gatox: while doing the last review, I found: https://bugs.launchpad.net/ubuntu-sso-client/+bug/944767 | 12:51 |
| gatox | mmmm maybe i miss a show_overlay there | 12:51 |
| nessita | gatox: you can queue that up for after the INTERNAL SERVER ERROR | 12:51 |
| nessita | gatox: yeap | 12:51 |
| gatox | nessita, yes | 12:51 |
| gatox | can i assign that to me? | 12:51 |
| gatox | ah..... i assume that i should queu that :P | 12:52 |
| nessita | gatox: I assigned it already | 12:52 |
| nessita | refresh pliz! :-) | 12:52 |
| nessita | gatox: also reported https://bugs.launchpad.net/ubuntu-sso-client/+bug/944769 (as a Medium) | 12:55 |
| gatox | nessita, roger that | 12:56 |
| nessita | gatox: approved | 12:57 |
| gatox | nessita, thanks | 12:57 |
| alecu | hello, all! | 12:57 |
| gatox | alecu, hi! o/ | 12:58 |
| nessita | holaaaa alecu | 12:58 |
| ralsina | good morning! | 12:59 |
| ralsina | Despite appearances, I am not working until 2PM. So I may or may not do things I should be doing. You are warned ;-) | 12:59 |
| gatox | ralsina, jeeje ok | 13:00 |
| * nessita does not see ralsina | 13:01 | |
| nessita | ralsina: is great you're coming after 2pm, I will have tons of reviews for you :-D | 13:01 |
| ralsina | nessita: awesome! | 13:03 |
| gatox | nessita, don't scare ralsina | 13:03 |
| gatox | jeje | 13:03 |
| gatox | let him enjoy his morning | 13:03 |
| ralsina | nessita: if you have any, I may do one as a communitymember in the morning, as well ;-) | 13:03 |
| mandel | ralsina, you have to sign the agreement then :P | 13:08 |
| nessita | lol | 13:09 |
| ralsina | mandel: I have! | 13:09 |
| === yofel_ is now known as yofel | ||
| nessita | mandel, gatox: if you have some minutes, would you please review https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/fix-933576-1/+merge/95557 ? | 13:45 |
| mandel | nessita, sure! | 13:46 |
| gatox | nessita, ok | 13:46 |
| mandel | nessita, I'm off to have lunch and will finish the review when I'm back | 13:55 |
| nessita | mandel: thanks! | 13:55 |
| === m_conley_away is now known as m_conley | ||
| dobey | the really annoying thing about debugging the music player, is that i can't play music while i'm doing it :( | 14:14 |
| dobey | ugh. 464M for libwebkitgtk-3.0-dbg | 14:17 |
| nessita | gatox: there is a conflict in https://code.launchpad.net/~diegosarmentero/ubuntu-sso-client/934502/+merge/94841 | 14:20 |
| gatox | nessita, ok.... fixing | 14:20 |
| alecu | gatox, ping | 14:27 |
| gatox | alecu, pong | 14:27 |
| alecu | gatox, did you ever encounter something like this in pyqt? | 14:27 |
| alecu | """QObject.connect: Cannot queue arguments of type 'MyType' | 14:28 |
| alecu | (Make sure 'MyType' is registered using qRegisterMetaType().)""" | 14:28 |
| gatox | :S..... nop | 14:28 |
| gatox | alecu, what are you doing? | 14:28 |
| gatox | nessita, approve | 14:28 |
| alecu | gatox, I'm trying to connect to this signal: http://doc.qt.nokia.com/4.7-snapshot/qabstractsocket.html#stateChanged | 14:29 |
| nessita | gatox: thanks! | 14:29 |
| * gatox looking | 14:29 | |
| alecu | gatox, in that doc it says very clearly: "QAbstractSocket::SocketState is not a registered metatype, so for queued connections, you will have to register it with Q_REGISTER_METATYPE() and qRegisterMetaType()." | 14:29 |
| alecu | gatox, but I'm not finding the right docs for those two methods in the pyqt docs | 14:29 |
| alecu | Q_REGISTER_METATYPE() and qRegisterMetaType() | 14:29 |
| gatox | alecu, give me a sec | 14:29 |
| * gatox looking | 14:29 | |
| gatox | alecu, do you have qt assistant installed? | 14:30 |
| dobey | i wonder if anyone even ever uses this feature in the music store plug-in, where we have clicking on the song in the "my downloads" page, start playing that song in your library | 14:30 |
| dobey | aquarius_: ping | 14:30 |
| aquarius_ | dobey, pong | 14:31 |
| === aquarius_ is now known as aquarius | ||
| alecu | gatox, seems I don't. what is the package for that? | 14:31 |
| dobey | aquarius_: you were doing some stuff with rhythmdb queries outside of rhythmbox, right? | 14:31 |
| aquarius | dobey, I was | 14:31 |
| dobey | aquarius: do you know if you can query it for a list of tracks from a certain library, in the list of libraries? | 14:32 |
| aquarius | I don't know off the top of my head | 14:32 |
| alecu | gatox, I've got the qt designer, but not the assistant. And I can't find a package for that | 14:32 |
| dobey | ie "all tracks in ~/.ubuntuone/Purcahsed from Ubuntu One/" ? | 14:32 |
| aquarius | could maybe work it out | 14:32 |
| gatox | alecu, i think you can install libqtassistantclient4..... if that doesn't work.... you can always install qt4-dev-tools | 14:32 |
| alecu | ok | 14:32 |
| gatox | alecu, the web documentation is not loading here....... that's why i mention qt assistant | 14:33 |
| gatox | alecu, take a look at the answer here, to see if it is useful for you: http://stackoverflow.com/questions/2585442/sending-custom-pyqt-signals | 14:36 |
| dobey | aquarius: is the code for the queries stuff you were doing somewhere i could look at it? | 14:37 |
| gatox | alecu, i think that probably using: Q_REGISTER_METATYPE() is more c++ way..... and the one in that page is more python-ish | 14:37 |
| aquarius | dobey, it's only a one-liner, but it's not helpful to you, because it was about querying the ExtDB (where the album art is stored), not the RhythmDB. | 14:37 |
| dobey | ah right | 14:37 |
| aquarius | dobey, I'm currently looking into it | 14:38 |
| dobey | i'm looking at the .gir and can't even figure out how to do a query and tell it what to query for | 14:38 |
| dobey | thanks | 14:38 |
| aquarius | I am a little worried that rhythmdb_query_parse and cousins are not bound into gir :( | 14:38 |
| aquarius | http://developer.gnome.org/rhythmbox/unstable/RhythmDB.html#rhythmdb-query-parse | 14:38 |
| dobey | hrmm | 14:39 |
| dobey | it's introspectable=0 | 14:39 |
| dobey | ah, because a RhythmDBQuery isn't an object | 14:42 |
| aquarius | dobey, but... if you've got multiple libraries, I think they're all separate sources | 14:42 |
| aquarius | so what you want to do is not construct a query; I think you want to get the list of sources, find the one in it that's your library, and then that source's query_model contains all the tracks | 14:43 |
| dobey | aquarius: nah, they're all under the same source it seems | 14:43 |
| aquarius | really? how do you get the source? | 14:43 |
| aquarius | if you can get the source I ma be able to build a query for it | 14:43 |
| dobey | libsrc = self.shell.props.library_source | 14:44 |
| aquarius | that only works inside a plugin :( | 14:48 |
| aquarius | external pytohn scripts don't get a shell object :( | 14:48 |
| === IAmNotThatGuy is now known as M0hi | ||
| aquarius | dobey, ah, you've gotta use it in a plugin; all the other stuff doesn't work safely externally. bah. So, I don't have time to fiddle with that right now :( | 14:52 |
| dobey | aquarius: right. new rb has some behavioral differences, and i think making it so that if you select the "Ubuntu One" store, you can hit play and it will play your purchased music, would be nice, and get rid of some of the weirdness i'm seeing in fixing other stuff :) | 14:54 |
| ralsina | gatox: ping | 14:57 |
| gatox | ralsina, pong | 14:57 |
| ralsina | gatox: please talk to Christine on canonical IRC *now* and give her your picture :-) | 14:57 |
| gatox | ralsina, ok | 14:57 |
| ralsina | gatox: not the darth maul one, please | 14:58 |
| gatox | ralsina, jeje okok | 14:58 |
| urbanape | briancurtin: http://pypi.python.org/pypi/MacFSEvents | 15:00 |
| briancurtin | urbanape: inttttteresting | 15:01 |
| nessita | me | 15:01 |
| briancurtin | me | 15:02 |
| nessita | gatox, mandel, alecu, dobey? | 15:02 |
| nessita | urbanape: ? | 15:03 |
| mandel | me | 15:03 |
| urbanape | me | 15:03 |
| gatox | me | 15:03 |
| nessita | alecu: standup | 15:04 |
| alecu | me | 15:04 |
| nessita | DONE: lots of reviews, helped gatox with a branch to make sso qt more beautiful, team call, more on u1cp sigin screen branch | 15:04 |
| nessita | TODO: proposed the branch for the aforementioned UIFe, finish reviews | 15:04 |
| nessita | BLOCKED: nopes | 15:04 |
| nessita | NEXT: briancurtin | 15:04 |
| * alecu writting notes | 15:04 | |
| briancurtin | DONE: got to where SSO tests are passing, moving on with client | 15:04 |
| briancurtin | TODO: push forward with client tests | 15:04 |
| briancurtin | BLOCKED: nope | 15:04 |
| briancurtin | NEXT: mandel | 15:04 |
| mandel | DONE: Debugging why libsoup throws an IO erro when a cert is wrong and ssl-strict is set to false. Some reviews for alecu, | 15:04 |
| mandel | TODO: finish nessitas review, more debugging. | 15:04 |
| mandel | BLOCKED: no, I mean, devugging.. | 15:04 |
| mandel | urbanape, please | 15:04 |
| urbanape | DONE: Made significant progress getting the ubuntu-sso-client tests passing in OS X (had to hard-code the network state). 1 random window size test failure. | 15:04 |
| urbanape | TODO: starting on the syncdaemon stuff. Need to write our own platform package. | 15:04 |
| urbanape | BLOCKED: Nada | 15:04 |
| urbanape | gatox: go | 15:04 |
| dobey | meh | 15:04 |
| gatox | DONE: | 15:04 |
| gatox | Fixed UI stuff and branches finally landed. Couple of reviews. | 15:04 |
| gatox | TODO: | 15:04 |
| gatox | Working on INTERNAL SERVER ERROR when executing an installed sso. | 15:04 |
| gatox | BLOCKED: | 15:04 |
| gatox | No | 15:04 |
| gatox | alecu, gord | 15:04 |
| gatox | go | 15:04 |
| gatox | :P | 15:04 |
| alecu | DONE: tunnel server use from Syncdaemon | 15:05 |
| alecu | TODO: fix some bugs, more of the above | 15:05 |
| alecu | BLOCKED: stuck with weird Qt signal and metatypes | 15:05 |
| nessita | dobey: go | 15:06 |
| alecu | dobey, go | 15:06 |
| dobey | λ DONE: team meeting, rb debugging, fixed one CRITICAL in rb upstream, fixed click-to-play on 'my downloads' page | 15:07 |
| dobey | λ TODO: more rb debugging, twisted glib2/gireactor debugging, tarmac locking? | 15:07 |
| dobey | λ BLCK: none. | 15:07 |
| nessita | dobey: can you please delay the twisted glib2/gireactor debugging and start working on the installer UI refactopring? the UI fix has way more priority than the other thing | 15:08 |
| nessita | ralsina: I know you're not here, but do we have the bug for the installer UI refactoring? | 15:08 |
| mandel | alecu, in https://code.launchpad.net/~alecu/ubuntuone-client/proxy-tunnel-server/+merge/95075 in FakeServerTunnelProtocol why is remote_disconnected empty? | 15:09 |
| alecu | mandel, probably it's not needed by the fake. lemme check | 15:10 |
| nessita | dobey: can you please give me an ack? :-) | 15:10 |
| nessita | I know you're not happy about that, but we have no other choice for that | 15:10 |
| dobey | there is no bug filed for installer ui | 15:11 |
| dobey | and all i have for it is a screenshot | 15:11 |
| ralsina | nessita: I don't think there is a bug | 15:11 |
| ralsina | alecu: can I help you with the weird signal/metatype thing? | 15:12 |
| nessita | ralsina: can you please file the bug with the details (or ask rtgrant to do it)? | 15:12 |
| alecu | ralsina, if you have some minutes, please do :-) | 15:12 |
| ralsina | alecu: ok! | 15:12 |
| ralsina | nessita: I will ping rtgrant about it | 15:12 |
| dobey | nessita: and no. the rhythmbox stuff has much higher priority. and it's what i'm concentrating on at the moment. | 15:13 |
| nessita | dobey: I said to delay the glib/gi debugging, not the RB stuff | 15:13 |
| alecu | ralsina, did you ever encounter something like this in pyqt? | 15:13 |
| alecu | """QObject.connect: Cannot queue arguments of type 'MyType' | 15:13 |
| alecu | (Make sure 'MyType' is registered using qRegisterMetaType().)""" | 15:13 |
| alecu | ralsina, I'm trying to connect to this signal: http://doc.qt.nokia.com/4.7-snapshot/qabstractsocket.html#stateChanged | 15:13 |
| alecu | ralsina, in that doc it says very clearly: "QAbstractSocket::SocketState is not a registered metatype, so for queued connections, you will have to register it with Q_REGISTER_METATYPE() and qRegisterMetaType()." | 15:14 |
| ralsina | alecu: are you connecting accross threads for some reason? | 15:14 |
| dobey | and i think the twisted stuff is also a higher priority | 15:14 |
| ralsina | alecu: because queued connections are accross threads and they have to be marshalled | 15:14 |
| alecu | ralsina, if I don't use "2" as the second parameter to connect I'm getting segfaults | 15:15 |
| ralsina | alecu: then yes, it's across threads | 15:15 |
| ralsina | alecu: and you are officially in some trouble :-/ | 15:15 |
| alecu | ralsina, the weird thing is that if I print the "thread.get_ident()" then it seems to be the same. | 15:15 |
| * gatox lunch | 15:15 | |
| === gatox is now known as gatox_lunch | ||
| ralsina | alecu: that is weird, yes | 15:16 |
| briancurtin | urbanape: mumble? | 15:16 |
| alecu | ralsina, this is probably qt4reactor playing with our heads. | 15:16 |
| ralsina | alecu: but keep in mind that Qt uses threads for non-blocking I/O so maybe you are later emitting the signal from another thread | 15:16 |
| urbanape | briancurtin: omw | 15:16 |
| alecu | ralsina, oh, that makes more sense. | 15:16 |
| alecu | ralsina, the thing is that I looked for .connect() in qt4reactor, and they *are not* using queued connections. | 15:17 |
| ralsina | alecu: hmmm | 15:17 |
| ralsina | alecu: let me think 1' | 15:17 |
| alecu | which sounds awfully dangerous | 15:17 |
| alecu | (not you thinking) | 15:17 |
| ralsina | alecu: usually people don't use queued connections, andthey get used automatically if needed | 15:18 |
| alecu | ralsina, what's "if needed" ? | 15:18 |
| ralsina | alecu: if you are doing it from a non-UI thread | 15:19 |
| alecu | ralsina, if I'm connecting to the signal, or using the signal? | 15:19 |
| ralsina | alecu: not sure, never tried it ;-) | 15:19 |
| alecu | ralsina, :-) | 15:20 |
| ralsina | alecu: a 0 (the default) means "auto" and it will guess if it's director queued | 15:20 |
| ralsina | alecu: 0(default) If the signal is emitted from a different thread than the receiving object, the signal is queued, behaving as Qt::QueuedConnection. Otherwise, the slot is invoked directly, behaving as Qt::DirectConnection. The type of connection is determined when the signal is emitted. | 15:20 |
| alecu | ralsina, so, state changing signals like "disconnected" or "stateChanged" look to be coming from a different thread at some point... because sometimes the process ends with a segfault. | 15:21 |
| alecu | (those are QTcpSocket signals) | 15:21 |
| alecu | as usual with thread stuff, it's impossible to reproduce. | 15:22 |
| ralsina | alecu: right | 15:23 |
| alecu | so, for instance, if I add a "print" it stops crashing. | 15:23 |
| ralsina | alecu: is it at all possible to do this without connecting that signal? | 15:23 |
| alecu | ralsina, in order for tests to pass correctly I need the tcp connections to be completely closed. | 15:25 |
| alecu | The "right" way is to use the disconnect signal, but it's not fired if the socket is looking up the hostname. | 15:25 |
| alecu | so I'm trying to use the statechanged signal. | 15:25 |
| alecu | I *could* use a "callLater" with a check of the state()... | 15:25 |
| ralsina | alecu: yes | 15:26 |
| ralsina | alecu: do some polling | 15:26 |
| alecu | but it makes the test ugly. | 15:26 |
| ralsina | alecu: and locking while looking up is not unusual behaviour | 15:26 |
| nessita | mandel: hola! | 15:26 |
| mandel | nessita, dime :) | 15:26 |
| nessita | mandel: were you able you finish my review? | 15:26 |
| alecu | ralsina, well, not exactly polling, because I'll still be releasing the cpu with the reactor.callLater... | 15:26 |
| alecu | ralsina, but it's very ugly. | 15:26 |
| ralsina | alecu: yes, a slow polling ;-) | 15:26 |
| mandel | nessita, running tests atm and will +1 | 15:26 |
| nessita | mandel: awesome, thanks! | 15:27 |
| alecu | ralsina, I'll give it a try, thanks for the input :-) | 15:27 |
| mandel | nessita, es un placer! | 15:27 |
| ralsina | alecu: sorry I couldn't solve it :-( | 15:27 |
| ralsina | alecu: but you are in the dark, seamy underground of Qt since a while ago ;-) | 15:27 |
| alecu | ralsina, I guess it would have been easier if I did this in C++; pyqt seems to be focused on the gui parts of qt :-) | 15:28 |
| ralsina | alecu: actually, maybe that's not as crazy as it sounds. Qt'sidea of C++ is very python-like | 15:28 |
| nessita | ok, need to run a quick erran, brb! | 15:29 |
| alecu | ralsina, let's see how the tunnel turns out; in any case we can side-port it to C++ later :-) | 15:30 |
| mandel | nessita, +1 done | 15:30 |
| nessita | mandel: thanks! | 15:30 |
| ralsina | alecu: indeed | 15:31 |
| ralsina | alecu: may be a fun weekend project, actually (a generic SSL over HTTP proxy tunnel?) | 15:32 |
| mandel | ralsina, alecu I'll avoid C++ if possible :P | 15:36 |
| alecu | chicken! | 15:36 |
| ralsina | mandel: C++ is the natural language for this. Or go, if there is a decent network library :-) | 15:37 |
| mandel | alecu, velociraptor! | 15:37 |
| mandel | ralsina, alecu go sounds more reasonable for this | 15:37 |
| alecu | sorry, I'm not here. I'm compiling! | 15:37 |
| mandel | lol | 15:37 |
| alecu | mandel, ralsina: Qt bindings for go? | 15:37 |
| ralsina | Some people have a problem. Then they use C++ and now the C++ points to element problem+1 | 15:38 |
| ralsina | alecu: not exists, AFAIK | 15:38 |
| mandel | alecu, you can call c from go, so we just look at the vtables :P | 15:38 |
| ralsina | and my C++ joke has a syntax error | 15:38 |
| ralsina | mandel: and do the mangling manually | 15:38 |
| mandel | did that once with c... painful! | 15:38 |
| briancurtin3 | ralsina: at least you aren't using templates in that joke | 15:39 |
| alecu | mandel, and it only worked for one compiler only... since they mangle differently. | 15:39 |
| ralsina | briancurtin3: I could try to work that in, but it would specialize on compile time. | 15:39 |
| mandel | alecu, yep.. it was a very hacky thing | 15:39 |
| briancurtin3 | and produce a 50,000 word error message | 15:39 |
| ralsina | briancurtin3: which amounts to "you left a space before the >" or "you missed a space before the >" | 15:40 |
| briancurtin3 | :) | 15:40 |
| mandel | briancurtin3, ralsina well, after dealing with erlang I think thisfred is ready to read template errors | 15:40 |
| thisfred | I explicitly declined to deal with erlang, mostly ;) | 15:40 |
| ralsina | mandel: you know C++ templates are turing complete? | 15:41 |
| ralsina | mandel: you can write your program in *templates* with no actual code. | 15:41 |
| thisfred | Just like XSLT then ;) | 15:41 |
| ralsina | http://stackoverflow.com/questions/189172/c-templates-turing-complete for example | 15:41 |
| mandel | jikes!!! ugly ugly.. | 15:42 |
| * ralsina considers generating templates from XML descriptions via XSLT | 15:44 | |
| mandel | ralsina, if you that I wont talk to you again, I'll use alecu as a proxy | 15:45 |
| ralsina | http://gitorious.org/metatrace a C++ compile-time raytracer. You write the scene, you compile it, the image is in the binary. | 15:46 |
| ralsina | Which is the greatest thing I have seen in the last 5 weeks at least | 15:46 |
| alecu | mandel, I'll be your tunnel | 15:46 |
| ralsina | alecu: yikes | 15:46 |
| mandel | alecu, that is what she said.. | 15:47 |
| mandel | lol | 15:47 |
| gatox_lunch | nessita, if i'm right..... and you can try it there...... you should go to: /usr/lib/pymodules/python2.7/ubuntu-sso-client/ubuntu_sso$ ........ and then do: sudo find . -iname '*.pyc' -delete....... and try the problem between sso and control panel again, and should be fixed..... something went wrong between the update and the pycs | 15:47 |
| gatox_lunch | it seems | 15:48 |
| === gatox_lunch is now known as gatox | ||
| gatox | ohhhh i was gatox_lunch | 15:48 |
| mandel | gatox, dude there is a guy with a nick very similar with lunch at the end, was it you? | 15:50 |
| gatox | mandel, no!! who who?? | 15:51 |
| gatox | jje | 15:51 |
| urbanape | briancurtin3: http://pypi.python.org/pypi/watchdog | 16:00 |
| urbanape | mandel, et al: what if we used watchdog for all platforms? | 16:00 |
| urbanape | it abstracts away inotify, FSEvents, and Windows stuff. | 16:00 |
| mandel | urbanape, hm..it will add an abstraction layer for all which is good, but I'm sure they have not fixed the fact that moves are diff etc.. although, you can use it for mac, and then we can consider it for windows | 16:04 |
| mandel | urbanape, I think that adding a lib in there is a smart move.. the problem is how well tested it is, does it solve all the issues and how do we hook that to twisted | 16:04 |
| mandel | urbanape, so in summary, if you have the time is ok for me but you need facundobatista to approve such a major change | 16:05 |
| urbanape | just found it looking to see if anyone had already mapped inotify events to FSEvents | 16:05 |
| facundobatista | urbanape, if you a library that abstracts them all into one set of semantics, *maybe* SD can be tuned to use those semantics, and we'd be happy | 16:07 |
| mandel | facundobatista, would mean loosing the open event for sure | 16:07 |
| mandel | facundobatista, urbanape it will also mean pushing all the dirty code to a lib maintained by others \o/ | 16:08 |
| facundobatista | mandel, which is double edged sword | 16:09 |
| mandel | facundobatista, true. | 16:09 |
| urbanape | it's got a bunch of dependencies on it, too, which we might not want to take on | 16:11 |
| urbanape | PyYAML, argh, argparse, pathtools, and Brownie | 16:11 |
| urbanape | might not actually need those (just for the extra scripting support) | 16:12 |
| dobey | ok, need to get lunch. bbiab | 16:18 |
| === briancurtin2 is now known as briancurtin | ||
| briancurtin | urbanape: sry, comcast seems extra flaky today | 16:23 |
| urbanape | no problem | 16:24 |
| mandel | ralsina, nessita I have an appointment in the ophthalmologist, so I'm leaving in 5 mins to be there on time | 16:28 |
| mandel | ok, time to go and see if i'm more blind :P | 16:32 |
| nessita | mandel: ack! | 16:36 |
| elopio | nessita, ralsina, can you take a look at bug #940884 ? | 16:59 |
| elopio | https://bugs.launchpad.net/ubuntuone-client/+bug/940884 | 16:59 |
| elopio | we will not sync files with names longer than what the os allows. Is that our problem, or the os problem? I think you'll know better. | 17:00 |
| ralsina | elopio: it's the OS fault but it's our problem | 17:01 |
| elopio | ralsina: ok, you have it on your plate. | 17:04 |
| ralsina | good evening u1 team! I am now working, so hit me with reviews ;-) | 17:05 |
| nessita | ralsina: about to! | 17:05 |
| nessita | guh, when I fix this lint issues | 17:05 |
| nessita | ralsina: would you know if rtgrant filed the bug? | 17:09 |
| gatox | ralsina, nessita really trivial review: https://code.launchpad.net/~diegosarmentero/ubuntu-sso-client/back-button/+merge/95622 | 17:10 |
| nessita | gatox: looooking | 17:10 |
| ralsina | nessita: seems he didn't. I'll do it myself | 17:11 |
| nessita | ralsina: :-/ | 17:11 |
| ralsina | nessita: I pinged (with the request) but he never ponged, maybe he's not on IRC today, andit's late in the UK already, so... | 17:13 |
| nessita | ralsina: if you consider this can wait till Monday, we can email him asking him to please file it | 17:13 |
| ralsina | I guess it can, since we are not going to file the UIFe today.Let's mail him and he can do it early monday | 17:14 |
| ralsina | nessita: I'll send the mail right away | 17:15 |
| ralsina | nessita: mailed, with you as CC | 17:18 |
| ralsina | gatox: The "enter verification" page is the one for the token you get by email for new accounts? | 17:19 |
| gatox | ralsina, yes | 17:19 |
| ralsina | gatox: why shouldn't there be a back button there? | 17:20 |
| gatox | ralsina, let me give you the bug description | 17:20 |
| ralsina | And the right way to not have a back button is to make the previous page a "commitPage" IIRC | 17:20 |
| gatox | ralsina, https://bugs.launchpad.net/ubuntu-sso-client/+bug/944769 | 17:20 |
| ralsina | gatox: looking | 17:20 |
| ralsina | what happens if you registered with a wrong email? Then you want to go back... | 17:21 |
| gatox | nessita, ^ | 17:21 |
| nessita | ralsina: well, the user can close the window and re-open, no? Or, we can leave the button there but cleanup the whole fomr | 17:25 |
| nessita | ralsina: otherwise we'll let the user think he can "retry" the registration | 17:26 |
| nessita | which he can not | 17:26 |
| nessita | ralsina: I prefer not having a back button, the user already had to enter the email *twice* | 17:26 |
| nessita | ralsina: the review you were asking for is https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/fix-933576/+merge/95627 | 17:30 |
| nessita | gatox: if you could too... ^ | 17:30 |
| gatox | nessita, on it | 17:30 |
| gatox | so.... what should i do with the back button isssue..... i'm working in another issue now...... to know if i need to modify anything there | 17:31 |
| nessita | ralsina: any thoughts re: Back button? my food is ready so you can share those with me when I came back | 17:31 |
| nessita | gatox: mark the MP as WIP until ralsina and I decide what is best | 17:31 |
| gatox | nessita, ack | 17:32 |
| ralsina | nessita: https://bugs.launchpad.net/ubuntuone-installer/+bug/944982 from Robert | 17:38 |
| ralsina | nessita: buen provecho! | 17:38 |
| dobey | nessita: "removing the dbus service" means bin/ubuntuone-control-panel-backend also? are we certain no apps in ubuntu are using it? | 17:39 |
| dobey | nessita: deja-dup uses the dbus service in control panel | 17:44 |
| ralsina | nessita: on your branch, if I login using an existing account, the SSO window is empty and u1cp is not usable until I close it manually | 17:48 |
| nessita | dobey: why would they? is not an official API :-/ | 17:52 |
| nessita | ralsina: do you have nightlies updated? | 17:52 |
| ralsina | nessita: updated to last night. let me update and retry | 17:52 |
| nessita | ralsina: (remember that SSO code is independent from the control panel code) | 17:52 |
| ralsina | nessita: yes, I was thinking if I had to file a bug on SSO and/or if you knew about it | 17:53 |
| nessita | ralsina: ah, ok. LEt's try to reproduce with latest nightlies | 17:53 |
| nessita | (please) | 17:53 |
| nessita | ralsina: with sso window empty, what do you mean? | 17:54 |
| dobey | nessita: because it's probably the only way to do it. and any published api is an official api. | 17:54 |
| dobey | nessita: if you know of a better way to do that, feel free to help mterry switch deja-dup over to it :) | 17:55 |
| nessita | dobey: I disagree with your last sentence, but will not discuss about that. Ok, if you are sure deja-dup uses it, I guess we need to leave that in place :-/ | 17:55 |
| ralsina | nessita: all widgets disappear and the window stays open | 17:56 |
| dobey | well i just asked mterry in #ubuntu-desktop, because it is depending on ubuntuone-control-panel | 17:56 |
| nessita | ralsina: weird :-/. Would you let me know if this happens with latest sso? | 17:56 |
| ralsina | nessita: yep | 17:56 |
| nessita | dobey: well, sadly, there is no better way to do the quota querying without doing all the REST thingy by hand. SO I guess we cna leave that (the controlpanel dbus service) in place. | 17:58 |
| dobey | right | 17:58 |
| gatox | nessita, ralsina another really small branch: https://code.launchpad.net/~diegosarmentero/ubuntu-sso-client/ensure-overlay/+merge/95636 | 18:08 |
| nessita | ralsina: what shall we do regarding the back button? saw my comments from before? | 18:08 |
| nessita | gatox: did you check all the screens? | 18:08 |
| gatox | nessita, yes | 18:09 |
| nessita | awesome :-) | 18:09 |
| ralsina | nessita: let's remove it and prepare a request for UX | 18:09 |
| ralsina | nessita: so that we can make it better in the not-so-close-future | 18:09 |
| nessita | ralsina: ack, gatox, did you get that ^? (also not sure if you should apply what ralsina said about the CommitPage) | 18:09 |
| gatox | nessita, yes..... i'll check about CommitPage | 18:10 |
| nessita | gatox: thanks! | 18:10 |
| nessita | gatox: I haven't tested IRL yet, but... in https://code.launchpad.net/~diegosarmentero/ubuntu-sso-client/ensure-overlay/+merge/95636, shouldn't the overlay be hide when the operation is completed? ie, in the method on_email_verified y en on_email_verification_error | 18:26 |
| gatox | ....... yep....... | 18:27 |
| nessita | gatox: I meant on_email_validated and on_email_validation_error | 18:27 |
| nessita | gatox: would you please add that? | 18:28 |
| gatox | nessita, of course.... | 18:28 |
| nessita | thanks! | 18:28 |
| nessita | ralsina: did it work? | 18:34 |
| ralsina | nessita: still upgrading | 18:34 |
| nessita | ack | 18:35 |
| ralsina | nessita: day of lots of updates, friday | 18:35 |
| dobey | meh | 18:42 |
| dobey | looks like rain | 18:42 |
| ralsina | dobey: in a wedding day? | 18:42 |
| dobey | weddings are lame anyway | 18:43 |
| ralsina | Oh. My attempt at alanis-rolling failed :-( | 18:44 |
| nessita | ralsina: I got it :-) | 18:44 |
| nessita | is like a free ride, when you already paid ;-) | 18:44 |
| ralsina | it's a good advice! | 18:44 |
| dobey | and i really do not like debugging untraceable race conditions that corrupt memory :-/ | 18:44 |
| nessita | dobey: is the RB thingy? :-/ | 18:45 |
| ralsina | dobey: ugh | 18:45 |
| dobey | yes | 18:45 |
| dobey | i am pretty certain now that it's not the entry that i'm creating in the plug-in, which is being dereferenced. | 18:46 |
| ralsina | dobey: if you had no luck with valgrind, have you tried any of the debuggig mallocs? | 18:46 |
| dobey | well i'm just doing gdb | 18:47 |
| dobey | and using G_SLICE=always-malloc makes the crash go away | 18:47 |
| ralsina | dobey: like dmalloc or whatever (I am just digging in old C stuff Ihalf-remember, so feel free to ignoreme) | 18:47 |
| ralsina | dobey: oh, a heisenbug | 18:47 |
| dobey | valgrind seems to be problematic, because it crashes in webkit when i run under vg | 18:47 |
| ralsina | dobey: valgrind isa whole freaking virtualized thing, it finds more bugs than there really are, sometimes | 18:48 |
| gatox | ralsina, the setCommitPage(True) is not working for this case | 18:49 |
| ralsina | gatox: really? remember it's on the PREVIOUS page ;-) | 18:49 |
| gatox | ralsina, yes, i'm setting that in the setup_account_page | 18:49 |
| ralsina | gatox: ok, then do it the other way around | 18:49 |
| ralsina | gatox: maybe it's because it relies on the order of the pages | 18:50 |
| gatox | ralsina, i'm checking why is not working..... | 18:50 |
| dobey | ralsina: i don't think it's finding a bug that isn't there in this case. but the trace from it isn't helpful | 18:50 |
| ralsina | dobey: right | 18:50 |
| ralsina | dobey: my sympathies, man | 18:50 |
| ralsina | dobey: that and 99 cents get you the dollar menu at Wendy's, though | 18:50 |
| dobey | nah. you need about $1.11 to buy something off the dollar menu at wendys, here anyway | 18:51 |
| dobey | yay taxes | 18:51 |
| nessita | ralsina: got another review that will not depend on your system being up to date: https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/fix-944120 | 18:51 |
| ralsina | nessita: on it! | 18:51 |
| nessita | dobey: would you please review it as well? https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/fix-944120/+merge/95642 | 18:51 |
| nessita | ralsina: merge link is ^ | 18:51 |
| gatox | ralsina, the attribute is being set correctly..... but i think that something involving how we navigate between the pages disable the commitPage thing | 18:52 |
| dobey | nessita: can you re-review my remove-gtk branch? :) | 18:52 |
| gatox | so..... the other way is already in repo | 18:52 |
| nessita | dobey: yes, already branching it | 18:52 |
| gatox | in the proposal i mean | 18:52 |
| dobey | ralsina: ^^ can you look at the remove-gtk branch as well pelase? | 18:52 |
| nessita | cd review_remove-gtk #and start reviewing ;) | 18:52 |
| ralsina | dobey: sure | 18:52 |
| ralsina | dobey: link? | 18:52 |
| dobey | https://code.launchpad.net/~dobey/ubuntuone-control-panel/remove-gtk/+merge/95252 | 18:52 |
| ralsina | dobey: thanks, queued | 18:52 |
| ralsina | dobey: (+7/-7509) 25 files modified nice! | 18:53 |
| nessita | dobey: so, your branch still has the docs/ubuntuone-control-panel-gtk.1, but I don't want you to remove but to rename it to be the Qt cp man page. Can you do that in this branch pliz? | 18:54 |
| nessita | dobey: with that we can tackle another bug ;-) | 18:55 |
| nessita | dobey: bug #933021 | 18:55 |
| dobey | nessita: i'll remove it. i won't rename it to -qt. the arguments aren't the same, i'm pretty sure | 18:55 |
| nessita | dobey: please do not remove it | 18:55 |
| nessita | dobey: you can edit the man page to be correct, no? | 18:56 |
| ralsina | let's do a branch that edits and renames | 18:56 |
| nessita | dobey: or I can do it, but do not remove it so I can cheat and re use it | 18:56 |
| dobey | nessita: why not just make a new one with the --help output from -qt instead? | 18:56 |
| dobey | it's simple enough to do. there's a tool that does it | 18:56 |
| gatox | nessita, ok..... both my branches are ready.... now i'm going to start with the tasks we talk | 18:56 |
| ralsina | help2man IIRC | 18:57 |
| dobey | yeah, that | 18:57 |
| dobey | it's easier to cheat that way, than copy the gtk one over | 18:57 |
| ralsina | Also, remember the help from the Qt version is a lie | 18:57 |
| nessita | dobey: as you wish, my gastritis is bad enough to be discussing at this time of the week | 18:57 |
| nessita | dobey: let me know when you made whatever change you decide to make to that branch, and I'll re-review | 18:58 |
| ralsina | nessita: assign me the bug, I will do it | 18:58 |
| nessita | ralsina: ack, thanks | 18:58 |
| nessita | ralsina: assigned, and updated milestone | 18:58 |
| ralsina | nessita: +1 on fix-944120 | 18:59 |
| nessita | ralsina: thanks! | 18:59 |
| dobey | nessita: are we supposed to maintain the other order on windows, and only have this order on ubuntu? | 19:00 |
| dobey | lisettte: ^^ | 19:00 |
| nessita | dobey: I don't think so, we need to have the same UI accross all platforms | 19:00 |
| nessita | dobey: the mock for the controlpanel always had the restore to the left and not default, I've probably done it wrong from the start | 19:01 |
| nessita | dobey: you can check the google doc with the spec | 19:01 |
| nessita | (which was originally for windows) | 19:01 |
| dobey | ok, well +1 anyway. i'm fairly apathetic, and it's too late to get an answer out of millbank today anyway | 19:02 |
| nessita | dobey: the mock: https://docs.google.com/a/canonical.com/drawings/d/1O_Q9EIJNbvnimroBTvoxvpFXuawEp8pA01iyfW-lHn4 | 19:02 |
| dobey | oh google docs; i hate you too. | 19:02 |
| ralsina | On a future UI refresh, there is something called a QButtonBox that's supposed to give platform-correct primary/secondary button ordering, but I can't even find the docs today | 19:03 |
| nessita | ralsina: nice to know | 19:04 |
| gatox | ralsina, are you sure about qbuttonbox?? i can't find it in qt assistant | 19:04 |
| dobey | note that there is no "platform correct" answer to that really | 19:04 |
| ralsina | gatox: it's there in designer :-) | 19:04 |
| ralsina | gatox: but I think it's not a real widget | 19:05 |
| dobey | though the ButtonBox widget makes it easy to switch directions of the buttons | 19:05 |
| gatox | ralsina, maybe is something like the "line" in the qt designer, that is actually a qframe | 19:05 |
| ralsina | gatox: yes, I think so | 19:05 |
| dobey | macos generally hasn't got dialog buttons. windows is usually "[ok] [cancel]" though basically arbitrary for non-standard apps. kde uses "[ok] [cancel]" and GNOME uses "[cancel] [ok]" | 19:06 |
| dobey | and then there are all the other environments too! | 19:07 |
| dobey | i think xfce matches gnome. and i'm not sure unity/ubuntu specifies that, but generally has followed gnome as it's been gnome. | 19:07 |
| dobey | i swear, if i have fixed this crash with this insane hack, i don't know what i'll do. maybe go get drunk right now | 19:08 |
| dobey | ok good. it didn't fix it | 19:09 |
| ralsina | dobey: right, I was thinking [ok][cancel] on windows, [cancel][ok] on windows | 19:09 |
| ralsina | oops, make one of those linux | 19:09 |
| dobey | also, i am getting tired of listening to the tron soundtrack previews | 19:10 |
| dobey | player.play_entry(entry.ref().ref().ref().ref().ref().ref(), self.source) | 19:10 |
| dobey | and it still crashed | 19:10 |
| dobey | so i'm pretty sure that entry isn't the point of unref failure :) | 19:10 |
| ralsina | bah, ButtonBox does that, but doesn't let you change the button texts. | 19:11 |
| ralsina | This is going to sound really strange, but my chromium window has turned grayscale, and all the others are ok. | 19:12 |
| dobey | compiz turns hanged/blocked/crashed applications grey | 19:14 |
| ralsina | dobey: but chromiumworks | 19:14 |
| dobey | isn't buttonbox just a specialized container that you can stick any widget in? | 19:14 |
| dobey | ralsina: i never specified the IQ of compiz. | 19:15 |
| ralsina | dobey: not in this case, it's just a gadget in designer, doesn't seem to exist in real life | 19:15 |
| ralsina | So I am now in the 1920s web | 19:15 |
| dobey | ah | 19:17 |
| dobey | hrmm, i hate these kinds of bugs :( | 19:33 |
| ralsina | nessita: with nightlies updated, I can't login with any of my test accounts :-( | 19:38 |
| ralsina | nessita: I get "authentication failed" for all | 19:38 |
| nessita | ralsina: are you sure those are valid U1 accounts? | 19:39 |
| nessita | ralsina: can you share a pair username and password? (I ask since those are test accounts) | 19:39 |
| ralsina | nessita: sure | 19:39 |
| nessita | ralsina: just checking, you fully updated and rebooted, right? | 19:39 |
| ralsina | nessita: updated, not rebooted | 19:40 |
| nessita | ralsina: shouldn't make any difference... but who knows. gatox and I had an issue with pyc files | 19:40 |
| ralsina | nessita: so, I'll reboot. This notebook fscks on every reboot so it will take a little while | 19:40 |
| gatox | ralsina, try to remove the pycs before just in case | 19:41 |
| ralsina | ok, will try that first | 19:41 |
| ralsina | nessita, gatox: yay, that did it! weird! | 19:42 |
| nessita | ralsina: there something very bad | 19:42 |
| ralsina | nessita: confirmed the branch works great IRL, the empty window thing is gone | 19:42 |
| nessita | ralsina: ffiiiuuuuuuuuuuuuuuuu | 19:42 |
| ralsina | nessita: so, I'll finish the review quick | 19:43 |
| gatox | ralsina, jejeej yep..... it happens to me too.... i figure it out modifying the files with sudo | 19:43 |
| nessita | ralsina: thanks! | 19:43 |
| dobey | RB, Y U NO REF?! | 19:43 |
| gatox | which force the generation of the pyc (wonder why it didn't do it before) | 19:43 |
| ralsina | gatox: maybe permissions on the pyc? | 19:44 |
| ralsina | gatox: but I am pretty sure I did not sudo edit ay py files :-) | 19:44 |
| gatox | ralsina, yes... it's weird | 19:44 |
| gatox | ralsina, i did it because i was trying to debug that | 19:45 |
| dobey | maybe i should look at twisted for a bit instead | 19:45 |
| ralsina | dobey: change the air | 19:45 |
| dobey | or just go try to unlock kaori on ssx | 19:46 |
| ralsina | nessita: +1 on fix-933576 | 20:04 |
| nessita | yey! | 20:05 |
| ralsina | gatox: +1 on ensure-overlay | 20:05 |
| gatox | ralsina, thanks | 20:05 |
| gatox | people! EOD here! have a nice weekend! | 20:18 |
| nessita | ralsina: doi you have https://code.launchpad.net/~diegosarmentero/ubuntu-sso-client/back-button/+merge/95622 in your queue? | 20:18 |
| nessita | who's up for a review? | 20:20 |
| * dobey hopes this bug is 'gone' now | 20:20 | |
| ralsina | nessita: looks like I didn't | 20:20 |
| ralsina | nessita: doing that one then | 20:20 |
| nessita | ralsina: thanks! | 20:20 |
| === gatox is now known as gatox_brb | ||
| dobey | hrmm is not; well the criticals aren't anyway | 20:21 |
| ralsina | nessita, gatox_brb: global approve on back-button | 20:27 |
| nessita | ralsina: nice, thanks | 20:27 |
| === gatox_brb is now known as gatox | ||
| gatox | ralsina, great thanks | 20:33 |
| cjohnston | u1filestatus.py doesn't work anymore on precise | 20:50 |
| cjohnston | has anyone updated it to work? | 20:50 |
| dobey | rye_: ^^ | 20:58 |
| rye_ | cjohnston, i know, but I haven't yet found a way to get it working | 21:07 |
| cjohnston | :-( | 21:11 |
| cjohnston | my u1 stuff just keeps getting worse | 21:12 |
| cjohnston | :-/ | 21:12 |
| dobey | how so? | 21:17 |
| cjohnston | dobey: u1 started deleting my files, and now isnt syncing right | 21:21 |
| cjohnston | I've been working with duanedesign for a week on it | 21:21 |
| dobey | hmm. that sounds quite odd | 21:24 |
| dobey | every time i think i've found something related to this bug, nothing changes :( | 21:30 |
| ralsina | EOD for me, I may do some reviews late tonight if needed for the release, please mail | 21:48 |
| briancurtin | have a good weekend ralsina | 21:48 |
| dobey | laters. i need to get off here for a while before my hands go numb from typing "bt" in gdb | 22:03 |
| === m_conley is now known as m_conley_away | ||
| === dduffey_afk is now known as dduffey | ||
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!