/srv/irclogs.ubuntu.com/2012/03/06/#ubuntuone.txt

afeijohi guys02:19
afeijo2 weeks ago I changed my pictures from internal Atrix storage to my new sdcard. I moved all the files and configured the camera app to save there, so far so good. But U1 is not uploading my pictures anymore... I checked in the Settings and I have no place to tell it that it should now monitor my sdcard. How can I fix it?02:20
beunoafeijo, hrm, I don't know, our android devs are sleeping atm, could you open up a question in askubuntu?02:21
beunotag it as ubuntuone?02:21
afeijoaskubuntu.com?02:21
beunoyes02:22
afeijook02:22
karniafeijo: wait :)02:22
beunoah02:22
* afeijo halt02:22
beunoI forget karni never ever ever sleeps02:22
afeijoyeah, he does not02:22
afeijokarni, shall I reinstall?02:22
karniafeijo: This will be fixed in one, at most two updates from now. We're aware of the issue, we'll allow any location to be source for uploads.02:23
karniafeijo: No, I'm sorry, it is not supported just yet. If you want to upload to U1 automatically02:23
afeijocool :)02:23
afeijoso I just need to wait for a few more days, then an update will get my last pics and sync?02:23
karniafeijo: You'd need to use the /mnt/sdcard/ mount point (whatever that is for you). There are vendors out there with /mnt/sdcard-ext or /mnt/ext-sd or what not02:23
afeijoin atrix, the internal is /mnt/sdcard, the external is /mnt/sdcard-ext02:24
karniafeijo: Yes, although 'few days' is rather tricky - I'm leaving for few days to rest. But rest assured, when I'm back, it won't be far to the update.02:24
karniafeijo: Right.02:24
karniafeijo: Well.. ;> /me chuckles02:24
afeijonp, I'll just plug the cable now and drag the files, not many tho, under 20 I guess02:25
karniafeijo: Do you have a minute?02:25
afeijokarni, to you I have 10 minutes :)02:26
karniafeijo: Perfect. Then let me see what I can do about that ;)02:26
afeijokarni, oh, the folder in my PC is named Pictures - MB860, is it possible to rename it to Pictures - Atrix?02:26
karniafeijo: yes. are you using ubuntu?02:27
afeijonot right now02:27
karniafeijo: Anyway - what you need to do is create a folder called however you like on your PC, and sync it with Ubuntu One (make it a cloud folder)02:29
karniafeijo: Then, you can point Ubuntu One Files to upload to that location02:29
afeijoah, got it, thanks02:29
afeijokarni, what can I do to help about the sdcard-ext ?02:31
karniafeijo: you already did. give me a moment :)02:31
karnilet's make that experiment and see how trivial it'd be to make it work for you02:32
afeijok02:32
karniomg.. looks like GreenDroid has broken stuff again02:32
karniafeijo: I'll try, anyway.02:33
afeijoGreenDroid should buy you a couple drinks :)02:33
karniI'm good ;)02:33
karniyesss02:34
karnistep 1, check02:34
karnik, patching02:37
karniafeijo: do you keep your pictures in 'DCIM' folder on your /mnt/sdcard-ext ?02:38
afeijoyes02:38
karniafeijo: perfect02:38
karniafeijo: Try this, http://goo.gl/UDfxJ in your browser, or http://goo.gl/UDfxJ.qr with your QR code scanner02:41
karniafeijo: When you do, launch the app once. Make sure auto-upload is enabled (you can also change the upload folder from there). What interest me more is - will it detect the new picture if you take a new one with your camera.02:42
karniafeijo: I'll grab a sandwitch, back in few minutes.02:43
afeijok02:43
afeijoinstalled & it is uploading 1 of 6 files02:44
afeijomake it 1902:44
afeijoperfect so far02:44
karniafeijo: did you snap a picture or have you installed fresh and pressed "all and new photos" ?02:46
afeijoI snap a pic02:46
karniafeijo: sweet ;) you are welcome! ^_^02:46
karniafeijo: Thanks, now I know this will work for sdcard-ext people02:46
karniafeijo: So, like I said - if you want to upload to a different cloud folder, just create it from your computer, and direct U1F to it from auto-upload settings.02:47
afeijokarni, you're the best, unbelievable fast to solve stuff :)02:47
karniafeijo: Also - I'm still working on it, so whenever you think somethings broken - don't hesitate to contact me02:47
afeijobut if I open a new folder and move the files, it will upload it all over again?02:47
karniafeijo: Thank you, it was easy ^_^ Just little time to make it a nice "Setting"02:47
karniafeijo: Well. I tell you what!02:48
karniafeijo: Current version would upload only new pictures (if you finish uploading these). If you move them into the new folder, it should never reupload them.02:48
afeijofunny that my folder for my xoom I remember labeling it Pictures - Xoom, the app asked me for the name when I first installed. But in my atrix it didnt02:48
karniafeijo: There's more! The new version that is coming would not even upload a file, that you sent to U1 from your computer. Magic! ^_^02:49
afeijoheh02:49
karniafeijo: Hrm. FWIW, the Android app never asks. It defaults to ~/Pictures - <device name>02:49
karniBut you can always change that. We'll also allow to create cloud folders from the android app, too, it's a matter of time.02:50
afeijomaybe I named my xoom but didnt find where to do it in the atrix? android 2.3 let me name it?02:50
karniafeijo: Negative, all default to ~/Pictures - <device name>, unless you (later) change the upload directory.02:50
ralsinakarni: you never sleep do you? :-)02:50
karniralsina: hehehe02:50
karniralsina: I'm in positive mood. I like to fix stuff ad-hoc, when there's people testing around ^_^02:51
ralsinakarni: cool, as long as you are having fun ;-)02:51
karniralsina: YES ^_^ !02:51
karniralsina: I totally agree with you :)02:51
* ralsina goes to sleep. Since it's late 6 FREAKING TIMEZONES TO YOUR LEFT ;-)02:52
afeijodid you guys saw that box.com giving 50gb for those that install their android app? sound like unfair competition heh02:52
karnihahahaha02:52
karniralsina: I'll be leaving really soon, too. Night!! :)02:52
afeijogmt -0300 here, not so late yet02:52
karniafeijo: 4AM here02:52
afeijowow02:52
afeijoI wake up 7 am, cant do that02:53
afeijobtw, some times I need to open U1 app in Windows for it to download the new pictures... it should be automatic right?02:53
karniafeijo: Well. It's called competition. I think I've tried the box.com Android app. And, as all, it has it's up sides, but I didn't like it.02:53
afeijokarni, the current box version has a pretty good interface, blue and white02:54
karniafeijo: It should be. You can drop by at any work day and poke nessita or mande'l (without the aphostrpohe)02:54
afeijono ssh support, but it has webdav02:54
ralsinaafeijo: the u1 in windows should start on login, yes02:54
karniafeijo: Also - I'm not saying our's better. No it's not. I'm saying they're's not good enough for me. Also, they appear in 'Share' dialogs when they shouldn't, so it's wrongly configured intent filters.02:54
afeijoindeed02:55
karniralsina: I didn't mention you, as I didn't want to bother you with these kind of things ^_^ You're the guru here!02:55
ralsinakarni: nah, I am just a sleepy manager :-)02:55
* karni chuckles02:55
karniAll righty, I'm either going to get some sleep or work done, gotta say goodbyes!02:56
ralsinasame here. Bye!02:56
karniBye o/02:56
afeijog'night!!02:56
karnitake care afeijo02:56
=== smb` is now known as smb
mandelmorning all!09:03
jalcineMorning mandel!09:07
jalcineHow are you?09:07
mandeljalcine, great, getting ready to get some work done :)09:07
jalcinesame here :)09:09
jalcinejust finishing boiling some more water for tea.09:09
mandeljalcine, tea or coffee is always needed :P09:14
JamesTaitGood morning, all! :D09:31
Chipacadavidcalle: ARTE+7 broke :(10:21
davidcalleChipaca, :'(10:21
Chipacadavidcalle: (it was scraping, and they changed their site) (and i haven't had time to fix it yet)10:21
davidcalleChipaca, ok10:22
Chipacanot the first time it happens, and to be expected. Tempted to move to ARTE rss feeds.10:22
Chipacabut there's a lot less content hanging off of http://videos.arte.tv/fr/videos/meta/index-3188674-3223978.html than there is via search10:22
davidcalleChipaca, I thought you were already using the feeds.10:22
Chipacano, because i'm greedy10:22
Chipacabut rethinking that10:23
davidcalleChipaca, greedy screen scraping, the best kind of scraping :)10:23
ChipacaTED also scrapes; I could change it to talk to the google spreadsheet instead, but that's a pain10:23
Chipacawe'll see10:24
davidcalleChipaca, let's make a video scrapers workshop at UDS :)10:24
Chipacadavidcalle: some companies already do this, and they have huge teams only dedicated to keeping the scrapers running10:25
Chipacagranted, they are more ambitious than us; they want to snarf the actual video, as opposed to sending users to sites10:26
Chipacadavidcalle: I like the idea of croudsourcing scrapers, but then the idea of source curation becomes complicated and politicsish10:27
davidcalleChipaca, right.10:27
davidcalleChipaca, I could push some scopes devs in this direction and have scopes/sources.10:28
mandelChipaca, any experience with libsoup dealing with wrong ssl certiicates? I'm getting a connection error problem while on Qt I get a nice PyQt4.QtCore.QString(u'SSL handshake failed'), PyQt4.QtCore.QByteArray()10:43
mandelChipaca, I have been stuck with that for a while now.. :(10:43
Chipacamandel: none, i'm afraid10:46
ralsinamandel: if you want, you can cry on my shoulder/we can take a look at it in about 1.5 hours10:46
ralsinamandel: not that I have experience, but sometimes a new set of eyes help10:46
ralsinamandel: and good morning ;)10:47
mandelralsina, lol at least the qt version works.. so I don't understand why libsoup is freaking out..10:47
ralsinamandel: because it's not qt ;-)10:47
mandelChipaca, don't worry, I guess that if I find out wtf is going on Iit will be a nice blog post10:47
ralsinamandel: different libraries different painful spots and all that10:47
ralsinamandel: just curious, is there a libsoup mailing list? May be worth asking there10:48
mandelralsina, there should be, I'll see if there is an irc channel too10:48
mandelbut I'll finish with the qt implementation first10:48
ralsinamandel: yes please10:54
gatoxgood morning!11:11
jalcineMorning gatox :)11:33
gatoxjalcine, hi! o/11:34
jalcineHow are you?11:34
gatoxjalcine, fine, you?11:34
jalcineWell, I've got like two source packages (with about 10, 11 packages they provide in all) now to work and build before the 27th.11:35
gatox:P11:35
jalcineShouldn't be too much, just need to get started, lol.11:36
gatoxjalcine, jeje always the hard part11:37
* mandel hates ssl certs.. jesus! this is plainly stupidly hard!11:40
gatoxi need to leave for a while... brb!11:49
=== gatox is now known as gatox_brb
ralsinagood morning!12:13
czajkowskiralsina: aloha12:20
ralsinahi there czajkowski12:20
gatox_brbback!12:27
=== gatox_brb is now known as gatox
gatoxralsina, mandel my mac mini is here! :D12:30
ralsinagatox: cool!12:30
* ralsina still has to order his12:30
gatoxralsina, i'm going to see how to share it via web12:30
mandelwait, everyone is getting one but me.. dammed I should not have had one already!12:30
gatoxmandel, i bought my own12:30
ralsinamandel: noone is stopping you. Noone is paying you for it either ;-)12:31
ralsinagatox: something like logmein may work12:31
ralsinagatox: but it's PAINFUL12:31
ryedobey, now nautilus plugin (in precise) does not listen to PublicFilesList, PublicFileAccessChanged12:31
mandelralsina, had to try :P12:32
mandelgatox, careful, you might like it12:32
gatoxmandel, naaaaaaa.... but it's necessary12:32
ryeelopio, do we have any tests for nautilus plugin?12:38
ralsinarye: too early for dobey yet12:38
ralsinarye: and probably for elopio too12:39
mandelralsina, I have a question about design.. not UI design but the design of the api of the web client12:40
mandelralsina, all string should not have the app name (Ubuntu One) because sso is not app specific, the problem is that the webclient does not have the app as the input in its constructor.. is it a valid things to do?12:41
ralsinamandel: shoot!12:41
ralsinamandel: ENOPARSE12:41
mandelralsina, ok, so the ssl dialog as it is in trunk has a string like the following: If you are not sure about this server, do not use it to connect to %(app_nmae)s12:42
ralsinamandel: right12:42
mandelralsina, and it is the webclient the one that call the ssl dialog via the spawn process passing as an arg --appname 'my app'12:43
mandelralsina, but atm welclient does not know the appname, this means it has to be passed somehow..12:43
ralsinaright12:43
* ralsina is starting to need a diagram to follow where this is going12:44
mandelI'm thinking about the __init__ since appname is not something that changes that often but I think it is a very crappy api for a webclient lib12:44
mandelralsina,  ^12:44
ralsinamandel: ok, this is a webclient for our own usage. If we need to make it a bit less general to present a better UX, that's ok12:44
ralsinamandel: so, I feel the appname belongs in the dialog, yes12:45
ralsinamandel: and that means the webclient needs to know about it, so, where else could you pass it?12:45
mandelralsina, so, wc = WebClient('Ubuntu One') for example, right?12:45
ralsinamandel: right.That means "This is a WebClient we are using to connect to Ubuntu One"12:45
ralsinamandel: unless the wc needs to connect to more than one "app" so to speak.12:46
mandelralsina, in theory not, but it could, is more of a This is a WebClient used by Ubuntu One12:46
mandelralsina, same as you can have with " This is a WebClient used by Software center"12:46
ralsinamandel: ok, then that12:47
mandelralsina, so, its ok, right?12:47
mandeleven though I find it a little ugly :P12:47
ralsinamandel: not all that ugly IMHO12:48
=== jalcine is now known as JackyAlcine_
duanedesigno/13:29
mandelduanedesign, morning!13:30
* duanedesign woke up late. Stayed up to late playing with his arduino13:31
mandelalecu, nessita, ralsina: I'm off to have lunch, but here is some food for though, I'm thinking of storing the pinned certs by the user in in ~/.cache/ubuntu-sso/ rather than ~/.cache/ubuntuone/ what do you think?13:31
mandelgatox, briancurtin and urbanape  you too ^13:32
ralsinamandel: hmmmm both feel wrong :-/13:32
mandelralsina, then we have a problem 'cause we have to store it somewhere, I don't mind the keyring.. but there is not api for that yet13:33
ralsinamandel: right13:33
ralsinamandel: let's talk about it after your lunch13:33
mandelralsina, ok, them I'm off to have it :)13:33
dobeyrye: eh? if it doesn't listen now, then it never has. that code hasn't changed this cycle.13:36
ryedobey, it used to show when the files are published, using an emblem, now it does not, and it requires to re-publish the file to get the link on every nautilus restart13:37
dobeyrye: then something else must have broken it, if it worked before, because that code has definitely not changed.13:38
dobeyin fact, i think the only thing that's changed in the nautilus extension, is removing the locationbar widget "ribbon"13:39
ryedobey, okay, i will take a look and will run in circles, scream and shout in case i find the reason13:40
dobeyrye: testing with the same account, on natty and oneiric VMs also, would probably be helpful. it should be working, but as you know published files have been giving us grief for a long time :-/13:41
alecuhello!13:41
gatoxalecu, hi!13:42
* mandel lunch13:43
ralsinahello alecu113:43
nessitabuenos días!13:43
ryeare we near u1sdtool unhanging?13:44
gatoxnessita, buenas13:44
ryemuy buenas!13:44
ralsinadobey: would fixing bug #947711 require a UIFe?13:44
nessitahola rye, gatox :-)13:44
ralsinahello nessita13:44
nessitahola ralsina13:44
ralsinanessita: removed the version from the manpage if you want to merge it13:45
nessitaralsina: already looking13:45
ralsinanessita: also, I have a brach to fix bug #947711 (we are showing the wrong filechooser), but am not sure if it needs a UIFe13:45
nessitaralsina: looking at the bug report13:46
nessitadobey: hola! were you able to check if tarmac for u1client was stucked? I have this branch approved for 22 hours now :-) https://code.launchpad.net/~nataliabidart/ubuntuone-client/15-minutes/+merge/9590013:46
dobeyralsina: i don't think so13:47
ralsinadobey, nessita: if it doesn't, I would like to have that merged in the release (it's a very simple fix)13:47
nessitaralsina: what I don't like is having 2 different file choosers depending on the platform... is there any way to research this a little bit further to see if we can use the same in both?13:48
dobeynessita: i guess when openstack went boom yesterday, it had a lock held. should get picked up now13:48
ralsinanessita: the logical thing is to use the native chooser on each platform13:48
nessitaralsina: also, I personally don't see this bug as a release blocker... would you agree to have it merge for the next release? (is still before beta-2_13:49
nessita)13:49
ralsinanessita: we are not using the native one on windows because of a Qt bug, but that's temporary13:49
ralsinanessita: sure, no problem13:49
nessitaralsina: so, if we swicth to use natuve file choosers... we still need to workaround the bug in windows, no?13:49
* dobey has a bunch of branches that need reviews13:49
ralsinanessita: we don't switch on windows until the bug is fixed on Qt. That way we have a windows-only bug13:50
ralsinanessita: instead of a bug in both platforms :-)13:50
ralsinadobey: I did one!13:50
nessitaralsina: regarding the manpage branch... perhaps you did not push the changes? I don't see the new revno on LP nor when local branching13:51
dobeyhttps://code.launchpad.net/~dobey/libubuntuone/fix-crits/+merge/9600813:51
dobeyhttps://code.launchpad.net/~dobey/ubuntuone-installer/string-change/+merge/9603713:51
dobeyhttps://code.launchpad.net/~dobey/libubuntuone/u1ms-load-fixes/+merge/9604613:51
dobeyhttps://code.launchpad.net/~dobey/rhythmbox-ubuntuone/url-handler/+merge/9604513:51
dobeyhttps://code.launchpad.net/~dobey/rhythmbox-ubuntuone/fix-preview/+merge/9598213:51
=== JackyAlcine_ is now known as jalcine
nessitadobey: I think you missed one change in the installer branch... we also need to change 'Discover your personal cloud"13:52
nessitadobey: ie, remove "the freedom of"13:53
=== jalcine is now known as JackyAlcine_
=== JackyAlcine_ is now known as jalcine_
dobeyeh?13:54
ralsinanessita: let me check the man branch13:54
ralsinanessita: indeed not pushed sigh13:54
ralsinanessita: there it is13:54
dobeyoh13:54
dobeythat is just silly13:55
dobey:-/13:55
nessitadobey: why?13:57
nessitaralsina: ack13:57
dobeynessita: because it changes the messaging in a slightly pointless way, and seemingly for no reason other than for the sake of changing something. but whatever, i don't feel like bothering about it any more. i already changed/pushed it13:58
nessitadobey: thanks14:00
elopiorye: just manual tests14:01
ryeelopio, could you please point me towards them?14:01
elopiorye: http://testcases.qa.ubuntu.com/Applications/UbuntuOne/11.10#Sharing_and_publishing14:03
ryeelopio, thank you14:03
elopiorye: np.14:03
ralsinanessita, dobey: are we not doing the change in installer to show the cloud image? Ad if so why?14:05
dobeyralsina: because this is the result of the discussion yesterday of the strings being too specific about plan and feature availability.14:07
ralsinadobey: that is about strings, but what about the image itself?14:07
dobeyralsina: see the last comment from rtgrant on the bug.14:07
ralsinadobey: bug number handy?14:08
nessitadobey: approved14:08
dobeyhttps://bugs.launchpad.net/ubuntuone-installer/+bug/94498214:08
ubot5Ubuntu bug 944982 in ubuntuone-installer trunk "[UIFe] Changes to the ubuntuone-installer Splash Screen" [High,Triaged]14:08
ralsinadobey: thanks, looking14:08
nessitaralsina: I'm looking at https://code.launchpad.net/~ralsina/ubuntuone-control-panel/go-native/+merge/96123, is there any constant for the 0 in the linux side?14:14
ralsinanessita: 0 is the default14:14
nessitaralsina: that was no my question :-P14:14
ralsinanessita: ok, no, there isn't one :-)14:15
ralsinadobey, nessita: we are going to go with a mix of both mockups: the cloud with the simpler texts14:16
ralsinadobey, nessita: expect a third mockup soon14:16
dobeywhat? no. stop changing things :(14:18
ralsinadobey: deep breaths14:18
nessitaralsina: *another* change? why? we talked about this directly with rtgrant yesterday14:18
ralsinanessita: because rtgrant understood that we couldn't do the cloud thing14:19
nessitaralsina: we never ever said that14:19
ralsinanessita: which is why he went with the other design14:19
ralsinanessita: miscommunications happen14:19
nessitaralsina: we were very clear that the text does not scale, we never mentioned the image...14:19
dobeyit doesn't mean they always have to be corrected!14:19
dobeythe kind of miscommunication that turns a potentially very large branch, into a trivial one, isn't bad :)14:20
ralsinadobey: he14:21
ralsinadobey: but you know. People keep on talking ;-)14:21
dobeylet them talk.14:23
dobeymeanwhile, i'll get things done :)14:23
ralsinadobey: cool, let's do that ;-)14:24
dobeyi did it already. but now you've gone and changed things, so it's no longer done. :(14:26
briancurtinralsina: my ideas for the windows root problem suddenly fell apart. i'm on XP as admin, running an admin command prompt, and sd runs fine. i commented on an issue to get more user info because i can only reproduce this if i really go out of my way, which people probably are not doing14:34
ralsinabriancurtin: yes, that's what happened with my "solution" :-/14:34
ralsinabriancurtin: I could never make it fail, but a bunch of users can14:35
briancurtinthe only way i got it was to shut down the sd that gets started up for you, then run your own in an explicit admin cmd, but i cant imagine why anyone would be doing that (most dont even know how)14:35
briancurtin(on win7)14:36
ralsinabriancurtin: yes, in that case, it's ok to fail :-(14:36
ralsinabriancurtin: maybe it's something about the user profiles14:36
briancurtinralsina: could be, hopefully i can get more info back14:37
ralsinabriancurtin: maybe you can contact some of the users who had the issue14:37
gatoxnessita, ralsina when you have a moment please: https://code.launchpad.net/~diegosarmentero/ubuntu-sso-client/ugly-reset/+merge/9613014:38
briancurtinralsina: the bug i'm assigned to (#930398) was created by joshuahoover based on what users are seeing. i'll dig around and try to find more reports14:38
joshuahooverbriancurtin: we were getting these reports mostly from xp users and win7 users who appeared to be running as the administrator user14:39
* mandel back14:42
mandelralsina, alecu, nessita do you fancy to have a chat about where to store the fingeprints of pinned ssl certificates?14:42
ralsinamandel: on mgmt call for 1 hour at least14:43
mandelack14:43
nessitamandel: do we need to store those?14:43
nessitaI mean, what happens if we don't store that?14:43
alecumandel, oh, right.14:43
mandelnessita, well designed added a checkbox that says remember my decision in the ssl dialog.. so we have a problem there14:44
alecunessita, the ssl dialog would show up on every reboot.14:44
alecunessita, reboot, or SD restart, or... you get the idea.14:44
mandelalecu, exactly, which would be a PITA14:44
alecumandel, where does firefox store them?14:44
mandelalecu, FYI, qt implementation does work well with the ssl certs, libsoup on the other hand.. I need to look closer to that one14:44
mandelalecu, its own storage14:45
mandelalecu, /usr/share/ca-certificates/mozilla/14:45
alecumandel, right. But is it encrypted?14:45
nessitagatox: the branch you linked requires a UIFe for the bug. Would  you plaase add some details to the bug such as a screenshot before, and one after?14:46
alecumandel, that surely is not the pinned list. User process don't have access there!14:46
gatoxnessita,14:46
gatoxok14:46
alecumandel, unless I'm misunderstanding what you mean by "pinned"14:46
nessitagatox: thanks\14:46
mandelalecu, that is one of the places, I'm looking at the API from mozilla14:46
alecumandel, "dont' have *write* access there"14:46
mandelalecu, you are right, those are the global ca ones..14:47
nessitadobey: I will start with the releases, I will also do protocol since you have a lot in your plate right now14:47
alecumandel, probably is inside one of the sqlite dbs in .mozilla/profiles/etc....14:48
mandelalecu, this is it: https://developer.mozilla.org/en/Using_nsILoginManager14:48
dobeyok14:48
mandelalecu, yes.. so, the deal is, there is nothing we an re-use AFAIK14:48
alecumandel, probably not. But I don't want to store that list in plain text without knowing the security implications of that.14:49
alecumandel, hmmm14:49
mandelalecu, we surely don't want that.. it is a hard problem to solve, of course, I can propose the qt implementation (which might need a UIFe duw to a string change) and then move to libsoup and that14:50
alecumandel, I think the list of accepted certificates should be of similar value to the fingerprints of servers in ~/.ssh/known_hosts14:50
alecumandel, so, my guess is that having a plaintext list of accepted certificates would be fine14:51
mandelalecu, yes, I was thinking of using a similar things under a diff location, we just need the fingerprints, nothing else14:51
alecumandel, so, if we make sure that folder where we store it has permissions 600, I think it would be fine.14:51
mandelalecu, I was thinking of ~/.cache/ubuntu-sso-client14:52
nessitaalecu, mandel: what about asking to the security team?14:52
mandelnessita, very good point14:52
nessitaalecu: you can ask the same guy about the ssl certs "thing"14:52
mandelalecu, what channel is that?14:53
briancurtinjoshuahoover: i'm on XP trying as actual admin (like root), as admin-granted regular user (like someone in sudoers), and a regular user -- not seeing this problem at all on XP :/14:53
alecumandel, nessita: firefox uses a plaintext file with permissions 600, called "cert_override.txt"14:53
joshuahooverbriancurtin: very strange...we got so many complaints, it seemed rather wide spread14:54
nessitabriancurtin: the problem appears once you install U1 from the installer, as far as I know14:54
joshuahoovernessita: right, briancurtin, if you're not installing from the installer then it may not reproduce14:54
briancurtini just ran the installer in a clean XP VM and it's working great14:55
nessitajoshuahoover, briancurtin: and I also think this was caused by an older version of the installer, were the perms were raised for the installation and then they were never downgraded14:55
alecunessita, mandel: https://developer.mozilla.org/En/Cert_override.txt14:55
joshuahooverbriancurtin: which version of the installer?14:55
briancurtinwhatever i get from the website just by default, i'll find a version #14:55
nessitajoshuahoover, briancurtin: so, our installer will promote itself to admin, and then it will not demote itself before running syncdaemon14:55
joshuahooverbriancurtin: we rolled back the broken version14:55
joshuahooverbriancurtin: it needs to be 2.99.314:56
alecumandel, I think we should use a text file similar to that14:56
mandelalecu, I think that using the same format seems logical but I don't want to add that code to the current implementation, the branch will get too big14:56
mandelalecu, you type faster :P14:56
joshuahooverbriancurtin: this is the one causing problems: http://one.ubuntu.com/windows/ubuntuone-2.99.3-windows-installer.exe14:56
alecumandel, absolutely true, it can come in a subsequent branch.14:57
briancurtinjoshuahoover: thanks, i'll try that or get another env setup on this XP box14:57
joshuahooverbriancurtin: cool14:57
mandelalecu, so, I'll do the following, qt implementation no pinned certs, libsoup implementation not pinned, pinned certs for both14:57
alecumandel, you may cross out "libsoup" from that list.14:57
mandelalecu, why?14:58
nessitabriancurtin: one thing to note is that we added the 'root' check late in the code, so I would advice checking in which revno we implemented the "is_root" method in the os_helper modules14:58
mandelalecu,  we are going to ignore it?14:58
briancurtinnessita: i'll check, thanks14:58
mandelme15:01
ralsinabriancurtin, nessita: honestly, the check added more problems than it fixed15:01
gatoxme15:01
nessitame15:01
briancurtinralsina: running the tests with "def is_root(): return False" on windows seemed to work just fine, FWIW15:01
ralsinabriancurtin: that was our previous implementation ;-)15:01
briancurtin(although i cant see that being very acceptable)15:01
ralsiname15:02
briancurtinme15:02
dobeymeh15:02
nessitaalecu, urbanape, me?15:02
urbanapeme15:02
alecume15:02
nessitamandel: go15:02
mandelDONE: Got the Qt implementation working with ssl certs and the ssl dialog.15:02
mandelTODO: Talk with josh and nessita, there is a new string, should it be translated. Maybe not. Look at pinned certs.15:02
mandelBLOCKED: kinda with libsoup and certs but I'm ignoring the issue atm.15:02
mandelnext: gatox15:02
gatoxDONE:15:02
gatoxPropose a couple of branches yesterday (most of them landed, one pending for review), propose a branch for: Bug #94506515:02
gatoxTODO:15:02
gatoxKeep fixing things in SSO.15:02
gatoxBLOCKED:15:02
gatoxNo15:02
ubot5Launchpad bug 945065 in Ubuntu Single Sign On Client "ResetPassword page has some widgets really close to the right margin [UIFe]" [Medium,In progress] https://launchpad.net/bugs/94506515:02
gatoxnessita, go15:02
nessitaDONE: reviews and final landings for today's releases, teach lead call15:02
nessitaTODO: releases15:02
nessitaBLOCKED: nopes15:02
nessitaNEXT: ralsina15:02
ralsinaDONE: mgmt call, design call, reviews, native file_chooser bug, created manpage, some other branch I can't recall, reviews. TODO: more  reviews, some minor branch, more calls, briancurtin 1-1, chec outsome qml/u1 code aquarius found, stuff BLOCKED: no NEXT briancurtin15:02
briancurtinDONE: poking around the root issue, looked into a few other impls, tried a few OS configurations15:02
briancurtinTODO: take the info i just got a few minutes ago into consideration and try to reproduce/fix15:02
briancurtinBLOCKED: NONE15:02
briancurtinNEXT: dobey15:02
dobeyλ DONE: bug #928356, bug #933010, bug #934235, bug #947381, bug #944982 (apparently not), bug #947477, bug #947480, poked rb upstream about tarball release15:02
dobeyλ TODO: upload new rb, releases, twisted glib2/gireactor debugging15:03
dobeyurbanape15:03
ubot5Launchpad bug 928356 in Ubuntu One for Rhythmbox trunk "fails to play preview" [High,In progress] https://launchpad.net/bugs/92835615:03
ubot5Launchpad bug 933010 in libubuntuone trunk "Search on Music lens turns up items for purchase. Clicking on them should bring up rhythmbox, but does not" [High,In progress] https://launchpad.net/bugs/93301015:03
dobeyλ BLCK: none.15:03
ubot5Launchpad bug 934235 in rhythmbox-ubuntuone (Ubuntu Precise) "ubuntu one plugin is not enabled by default" [High,Confirmed] https://launchpad.net/bugs/93423515:03
ubot5Launchpad bug 947381 in Ubuntu One for Rhythmbox trunk "Clicking songs in My Downloads to play in library not working" [Medium,In progress] https://launchpad.net/bugs/94738115:03
ubot5Launchpad bug 944982 in ubuntuone-installer trunk "[UIFe] Changes to the ubuntuone-installer Splash Screen" [High,Triaged] https://launchpad.net/bugs/94498215:03
urbanapeDONE: Didn't make a lot of progress on the filesystem events stuff15:03
urbanapeTODO: Put together a branch for the ubuntu-sso Mac stuff15:03
urbanapeBLOCK: None15:03
urbanapealecu15:03
alecuDONE: a branch that starts and stops the tunnel. IRL of syncdaemon running thru an nonauth proxy.15:03
alecuTODO: fixes, push reviews15:03
alecuBLOCKED: no15:03
nessitaany comments anyone?15:04
ralsinaNOTE: whoa, lots of progress yesterday people! Nice!15:04
ralsinaEom?15:04
nessitaeom!15:04
alecugatox, did you land branches yesterday?15:04
alecugatox, I remember nessita saying "don't land branches on monday nor tuesdays!"15:05
nessitaalecu: yes, he did land some release-cblockers15:05
alecuoh, ack.15:05
mandelnessita, joshuahoover I've got a question regarding the string used for the ssl cert details, atm is in eng since is ssl information, do you think it should be translated?15:05
nessitaalecu: I said no branch landing unless approved by me or ralsina :-P15:05
gatoxalecu, fiuuuu.... i was really scared15:05
alecu:-)15:05
nessitaalecu: but thanks for paying attention to that!15:05
joshuahoovermandel: bug #?15:05
mandelnessita, joshuahoover doing so is another UIfe and I don't think is necessary15:05
nessita(really)15:06
alecugatox, you should "fear the nessita"15:06
gatoxjeje i know15:06
mandeljoshuahoover, hm.. let me find it15:06
joshuahoovermandel: if it's a new string then it probably needs to be translated15:06
mandeljoshuahoover, the string is as follows: http://paste.ubuntu.com/871566/15:08
gatoxjoshuahoover, i have this bug for UIFe if you can take a look at it: https://bugs.launchpad.net/ubuntu/+source/ubuntu-sso-client/+bug/94506515:08
ubot5Ubuntu bug 945065 in Ubuntu Single Sign On Client "ResetPassword page has some widgets really close to the right margin [UIFe]" [Medium,In progress]15:08
joshuahoovergatox: can we "bundle" this change with another that is making changes to strings? seems like they could go together and not cause much trouble...we're hammering the release team with all these freeze requests15:10
gatoxnessita, what do you think?  ^15:10
nessitajoshuahoover: which other is the strings change?15:11
joshuahoovermandel: and those labels (i.e. Organization, Comman Name, etc.) are displayed consistently, right? meaning they are displayed whenever a self-signed cert needs to be accepted15:11
joshuahoovernessita: well, we have 16 FEs open, so i'm guessing one of those :)15:11
joshuahooverhttp://is.gd/u1fes15:11
mandeljoshuahoover, yes they are, the string is generated with the info from the cert15:11
nessitajoshuahoover: we're closing a bunch today with the releases15:11
joshuahoovermandel: are those labels static (in the code) or provided by the cert?15:12
nessitajoshuahoover, gatox: what about if we wait to tomorrow to see which UIFe remains opened?15:12
nessitathen we try to stick it there15:12
joshuahoovernessita: i would prefer that15:12
nessitasure15:12
gatoxok15:12
mandeljoshuahoover, are in a string in the code, but they are the default issuer info tags15:12
joshuahoovermandel: if it's in the code then we need to allow translations15:12
mandeljono, I mean, I can use the sort ones, line CN and L etc.. instead15:13
dobeymandel: you're as bad at tab completion as gatox is15:14
mandelfuck, sorry!15:15
gatoxjejeje15:15
mandeljoshuahoover, I mean, I can use the sort ones, line CN and L etc.. instead15:15
joshuahoovermandel: can you provide an example?15:15
mandeljoshuahoover, sure, few secs please15:16
mandeljoshuahoover, this is using the actual tags: http://paste.ubuntu.com/871573/15:17
joshuahoovermandel: i think we need the strings you provided in the first example...can you file a bug for that and include the strings that are going to need translation?15:18
mandeljoshuahoover, of course15:18
joshuahooverthanks15:18
* dobey wonders if anyone is reviewing his branches15:20
nessitadobey: shoot links15:22
dobey08:51 < dobey> https://code.launchpad.net/~dobey/libubuntuone/fix-crits/+merge/9600815:23
dobey08:51 < dobey> https://code.launchpad.net/~dobey/ubuntuone-installer/string-change/+merge/9603715:23
dobey08:51 < dobey> https://code.launchpad.net/~dobey/libubuntuone/u1ms-load-fixes/+merge/9604615:23
dobey08:51 < dobey> https://code.launchpad.net/~dobey/rhythmbox-ubuntuone/url-handler15:23
dobey/+merge/9604515:23
dobey08:51 < dobey> https://code.launchpad.net/~dobey/rhythmbox-ubuntuone/fix-preview/+merge/9598215:23
dobeynessita: ^^ minus the installer one of course15:23
nessitaack15:23
mandeldobey, afaik, I'm not :)15:39
mandeldobey, need an other pair of eyes?15:39
nessitadobey: would you please remind me how to test the libubuntuone branch?15:39
dobey./autogen --prefix=/usr && make && LD_PRELOAD=libubuntuoneui/.libs/libubuntuoneui-3.0.so.1.0.0 rhythmbox -n15:41
nessitathanks15:43
mandeljoshuahoover, do you use any tag for the UIFe bugs?15:44
nessitaralsina: may I have a couple of stable-3-0 update reviews? https://code.launchpad.net/~nataliabidart/ubuntu-sso-client/stable-3-0-update-2.99.90/+merge/96154 and https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/stable-3-0-update-2.99.90/+merge/9615515:44
mandeljoshuahoover, bug 94811915:47
ubot5Launchpad bug 948119 in Ubuntu Single Sign On Client "Add a translatable string for the ssl certificate details" [Undecided,Triaged] https://launchpad.net/bugs/94811915:47
ralsinanessita: yes, but I am having lunch in 5' so they may take a bit15:49
nessitaralsina: don't forget me :'(15:50
nessita:-P15:50
ralsinanessita: I will think of you with every bite of salad!15:50
gatoxnessita, about this issue: https://bugs.launchpad.net/ubuntu-sso-client/+bug/93450015:53
ubot5Ubuntu bug 934500 in Ubuntu Single Sign On Client "Qt UI: Logging too verbose (and adds no useful info)" [Medium,Triaged]15:53
nessitagatox: yes, tell me15:53
gatoxnessita, i think we can remove the logs from the methods that are checking inn real time when the user type15:53
nessitagatox: for sure15:53
gatoxnessita, i don't think that it's useful to log for every key the user type15:53
nessitagatox: what I also would like is that we change the messages a bit15:53
gatoxwe have a log when the form check the email at the end15:53
nessitagatox: logging just the method name is useless15:54
nessitagatox: the goal of the bug report is to cleanup all the logging inside the ubuntu_sso/qt modules15:54
gatoxnessita, ok.... i'll look what we can do with that15:54
nessitagatox: so feel free to add logging, remove, change, edit15:54
gatoxnessita, roger that.... i'll revview the uis and check the logs15:54
nessitagatox: please consider what info is important if we later need to debug something by just looking at the logs :-)15:54
gatoxoperations15:54
nessitathanks15:54
gatoxnessita, yep15:54
nessitadobey: rythmbox is opened from your branch, but I can't find the U1 store15:56
dobeynessita: which branch are you testing?15:56
nessitau1ms-load-fixes15:56
dobeynessita: do you have the extension installed? and which version of rhythmbox do you have?15:57
nessitaSetting up rhythmbox-data (2.95+r7914-21~precise1) ...15:58
nessitaSetting up rhythmbox (2.95+r7914-21~precise1) ...15:58
nessitaSetting up rhythmbox-plugin-cdrecorder (2.95+r7914-21~precise1) ...15:58
nessitaSetting up rhythmbox-plugins (2.95+r7914-21~precise1) ...15:58
nessitadobey: installing our plugin noqw15:58
nessitaSetting up rhythmbox-ubuntuone (3.1+r109-15~precise1) ... :_)15:58
dobey:)15:58
nessitadobey: it loaded, anything else I need to confirm/check/test?15:59
nessitaI saw a "the store is loading" message and a progress bar15:59
nessitathen the store appeared15:59
nessitano guns n'roses in the store??? this is outrageous16:00
dobeyright. that branch is needed by the url-handler branch, to fix some racing that happens16:01
dobeygnr is probably in the us/uk stores. but the world store won't have16:01
* nessita is oficially mad at the store16:02
nessitadobey: ok, approved16:02
nessitaoops, I approved the incorrect one16:02
nessitadobey: by mistake I approved https://code.launchpad.net/~dobey/rhythmbox-ubuntuone/fix-preview/+merge/95982, will not revert the vote but will IRL tests when the libubuntuone nightlies is available16:03
dobeynessita: the fix-preview branch doesn't require those changes. it just requires the latest rhythmbox which has a fix for the nasty crash i was debugging last week16:06
nessitaah ok, will review16:07
mandeldobey, do we have to do anything for pylint to ignore TODO comments? or are they just printed but are not an error?16:11
dobeymandel: printed but not an error16:11
mandeldobey, sweet, thx!16:11
joshuahoovermandel: thanks16:12
dobeyok, need to get lunch. bbiab16:14
* gatox lunch16:17
briancurtinralsina: i think reverting the is_root change might be the best. on XP if you're an admin you're always an admin, and most people run with admin-based accounts. at least in vista/7 there's elevation and even the broken version works for me there16:24
briancurtini'm also not seeing anywhere in the installer that we specify elevation of privileges, unless that's handled inside Qt. however, i never get a UAC popup, oddly16:25
joshuahooverralsina, nessita, mandel: i added a uife for bug #948119 ...in theory we don't HAVE to have it but if we don't get it in then it's going to be rather confusing for many users behind proxy servers who get the ssl cert prompt16:27
ubot5Launchpad bug 948119 in Ubuntu Single Sign On Client "[UIFE] Add a translatable string for the ssl certificate details" [Medium,Triaged] https://launchpad.net/bugs/94811916:27
mandeljoshuahoover, ok, I though so, the cert tags are rather confusing..16:27
joshuahoovermandel: yes they are16:28
ralsinabriancurtin: the thing is, the installer runs as admin because it writes to system folders16:28
ralsinabriancurtin: so, we had another bug about u1cp and syncdaemon being started with elevated privs16:28
ralsinabriancurtin: I fixed that, but as a safety measure, I wanted to have syncdaemon fail instead of writing files the user can't later read16:29
ralsinabriancurtin: then, the "safety measure" turned out to break syncdaemon for a bunch of folks16:29
briancurtini guess i didn't gather that from the bug report. maybe we can talk about it during 1-1 in a bit16:29
ralsinabriancurtin: sure16:30
ralsinabriancurtin: it's a story spread over 2 or 3 bug reports :-)16:30
joshuahooverbriancurtin: heh, bad bug report...who filed that thing? oh wait, nevermind16:31
* ralsina goes back to his risotto16:31
gatoxpeople..... i'm taking half day off for sickness (starting now :P) because i'm not felling very wel... see you tomorrow!16:34
=== gatox is now known as gatox_away
mandelgatox_away, should be know as gatox_ill :P16:50
=== zyga-xchat is now known as zyga
ralsinanessita: if you have any bugs that I could hack, I have the evening semi-open after I do your reviews16:53
nessitaralsina: I'm going thru the bug right now, so I will assign some to you16:53
ralsinanessita: cool, thx16:53
ralsinanessita: there is no need to manually close the incomplete/no answer bugs, they go away at 60 days (allegedly)16:57
nessitaralsina: only if there is no asignee, I guess16:57
nessitaralsina: I'm closing bugs a year old :-/16:57
nessitaor 6 months old16:57
ralsinanessita: nice wrinkle ;-)16:57
nessitaralsina: I marking several as Incomplete with no assignee16:58
nessitato see if that works better16:58
ralsinanessita: ok, let's see16:58
ralsinaalthough if it only works without asignee, it's quite useless, since we don't want to have unassigned bugs16:59
briancurtinralsina: 1-1 when you're ready?17:00
ralsinabriancurtin: in 1'17:01
gatox_awaymandel, i thought about that..... but was too melodramatic :P .....17:05
* gatox_away return to being away17:05
nessitaralsina: hey you already have a bug assigned to you, https://bugs.launchpad.net/ubuntuone-control-panel/+bug/94046517:05
ubot5Ubuntu bug 940465 in Ubuntu One Control Panel "The --switch-to and --alert options don't work" [Undecided,New]17:05
nessitaralsina: I was about to assign it to you :-P17:06
nessitagatox_away: go away!17:06
ralsinanessita: yes, but I thought that required a FFe17:06
ralsinanessita: and maybe a UIFe for changing the help17:06
nessitaralsina: hum, I don't think so... would you please ask in #ubuntu-release? is fun :-)17:07
ralsinanessita: hahaha17:07
ralsinanessita: no it's not!17:07
mandelalecu, ping17:09
alecumandel, hola!17:09
mandelalecu, did you update the branhces you had for review?17:10
alecumandel, yes, I did!17:10
alecumandel, and before EOD I'll be submitting two more for review.17:10
mandelalecu, I have to go to rugby but i want to review them when I'm back, can you send me an email with the urls?17:11
alecumandel, ack17:15
mandelalecu, ralsina, nessita asking at #libsoup at #gimpnet I have been told that libsoup does not support https proxies.. I'm asking for some docs stating that and not only believing a guy in an irc channel17:15
dobeynessita: oops. fixed/pushed the unused import in fix-preview17:16
nessitadobey: ack17:19
nessitamandel: did he say why libsoup does not support https?17:19
mandelnessita, they said it does not support https proxies, it does support doing requests to an https page17:20
nessitayes, I meant that, sorry17:20
dobeymandel: who is the guy that answered you in irc?17:21
nessitalunchtime, brb17:21
mandeldobey, danw is the one17:21
mandeldobey, I'm talking with him since he says that is no possible.. lets see how far I get17:22
dobeymandel: well, you should listen to what he said then. he wrote libsoup, after all.17:22
dobeymandel: are you installing the gnome proxy resolver?17:24
mandelalecu, dobey, nessita, ralsina the https proxy option is gnome is so that you can configure a diff proxy for those pages using https thatn those using http, therefore I thinkwe have to rethink the way we do the webclient request, atm I think we are using the http proxy for all the requests in the qnetwork implementation17:25
mandeldobey, yes I am, I was trying to do some tests, but there is no point :)17:25
dobeyright. i'm pretty sure that's what the config is for in firefox as well17:27
mandeldobey,  is a matter of testing, but if it is like that, I'm very happy :)17:28
mandelanyways, EOD for me, which is good because after those news I should be able to finish this tom morning :)17:28
dobeymandel: i'm pretty sure the "ssl proxy" is "connections using ssl should go through this proxy", in firefox17:30
ralsinanessita: ok, confirmed --help is not covered by UIFreeze17:31
mandeldobey, then, we have a small bug in the qtnetwork implementation which is not a problem what so ever, but means that we have to changes tests.. etc17:31
mandelalecu, what do you call an alligator in a vest?17:34
mandelan investigator17:34
ralsinanessita: +1 on sso and u1cp17:40
nessitaralsina: thanks! so, wanna start with the --help fixes bug?17:45
nessitaralsina: I can assign more later, but want to move forward with releases now17:45
ralsinanessita: yes, I will start with that one17:45
nessitanice17:46
ralsinanessita: one is enough :-)17:46
nessitadobey: question, this kind of changes "player = self.shell.get_property('shell-player')" are RB API changes?17:47
dobeynessita: yes17:49
nessitadobey: and that code will also work in older releases? (just checking)17:50
dobeynessita: older as in the gtk2 rhythmbox?17:51
nessitafor example, but I meant older as in oneiric/lucid17:51
dobeylucid would be gtk2. oneiric might be ok, but probably has some other bugs fixed in newer rhythmbox we'd depend on being fixed17:52
dobeyand there is no way the current rhythmbox-ubuntuone will also work on gtk2 rhythmbox17:52
dobeyand there isn't really much we can do about it17:53
nessitaack17:54
nessitadobey: approved fix-preview17:54
dobeythanks17:56
dobeyi replied to your comment on url-handler as well17:56
ralsinaso, Unity.Launcher.get_for_desktop_file("foobar") always succeeds, regardless of whether foobar exists or not. Amusing.18:01
dobeyralsina: eh?18:02
ralsinadobey: literally, you can put anything in "foobar" and you get a "valid" UnityLauncher  which, of course, does nothing visible18:03
ralsinadobey: exploratory programming of the Unity API ;-)18:03
ralsinadobey: and yes, I know it's already encapsulated in u1-client18:04
dobeywhat are you trying to do? the quick lists stuff?18:04
ralsinadobey: usually, when I start coding for a new API, I play with it a bit following the docs. I am looking at implementing --alert18:04
ralsinadobey: and yes, there is a trivial way to do it, no problem, it just surprised me when I was playing with it18:05
dobeyoh, what is --alert supposed to do?18:05
ralsinadobey: set urgency on the launcher18:06
ralsinadobey: or rather, start *and* set urgecy18:06
dobeythat seems silly to have as a command line argument for the thing you're setting the urgency on18:06
dobeyi think it may have ben an attempted hack to do the "pop up behind everything else and set the urgency on the launcher"18:07
ralsinadobey: it's so syncdaemon can start u1cp as "urgent" when we run out of quota, that kind of thing18:07
ralsinadobey: IIRC18:07
dobeywhich i /think/ i convinced nessita and thisfred of being totally evil18:07
ralsinadobey: k, then after the releases are done, I would like to talk with you about a bug18:07
dobeyok18:08
nessitadobey: I guess you did not convince me. Why is evil?18:08
thisfredurgency does not really work, or it didn't last time I looked at it18:08
thisfredor maybe we did fix it18:09
ralsinathisfred: oh, interesting!18:09
dobeybecause it's unnecessary18:09
thisfredlong time ago18:09
ralsinaOTOH, syncadaemon seems to be setting/unsetting urgency itself18:09
ralsinaso --alert seems pretty useless18:09
dobeywell, syncdaemon sets it. control-panel would unset it18:10
nessitadobey: you sure do_impl_add_uri no longer needs to return a bool?18:10
ralsinadobey: hmmm even more useless then18:10
dobeynessita: yes. the parent method prototype returns void18:11
nessitaack, thanks18:11
ralsinanessita, dobey: ignore me then, do the releases, I will move to other areas of the bug18:11
ralsinathisfred: you too ^18:11
* thisfred is in the 7th circle of manual memory management18:12
* dobey releases the cerberae18:12
thisfredwoo I think I have it working,  maybe18:14
ralsinathisfred: sme people have a problem. They think, I will allocate memory manually!. Now they have no problem, or two problems and room  for only one, or have a problem they don't know about.18:16
dobeyOthers use a gc, and then wonder where all their RAM went, and why their hard disk light keeps blinking.18:17
ralsinadobey: details18:17
ralsinadobey: oh, a pretty light!18:17
dobeywhat color is it? ultra indigo?18:18
ralsinadobey: a sort of lemony-green with a dash of pink. You just had to be there, I guess.18:19
thisfredralsina: believe me, I'd prefer to not have to write C. Though it's probably good for me that I'm forced to. Build some character type thing ;)18:19
dobeyyou're not writing it in go? for shame.18:20
thisfredout of 4 attempts so far, only 2 have segfaulted18:20
thisfreddobey: I wish. (And I will, but not on company time, I guess ;)18:21
ralsinathisfred: fridays! On desktop you have fridays! Sometimes!18:22
dobeyfridays? what are those?18:23
thisfredralsina: yeah, sometimes, and I bequeathed my next few mythical fridays to aquarius18:24
ralsinadobey: we are calling them second-thursdays until new notice18:24
thisfredhehe18:24
* ralsina realizes he never actually told anyone "stop your fridays". How very responsible you all are.18:25
dobeyit's ok. we haven't been fixing bugs in older releases and doing SRUs on wed/thurs either. :-/18:26
ralsinadobey: win some lose some!18:26
nessitathisfred: if you have questions about C and mem management, I'm happy to answer/help18:28
thisfrednessita: you rock!18:28
=== zyga is now known as zyga-afk
thisfredANd I will take you up on that. For now I fixed the stupid myself ;)18:29
nessitaI also have some complex analogies between memory management and shoe boxes, is interesting :-) (though student seems to understand mem allocation a bit better using it)18:29
dobeyyou can't do pointer math?18:32
dobeynewb.18:32
ralsina1 pointer + 1 pointer = Lame Darth Maul laser sword?18:33
=== yofel_ is now known as yofel
dobeyhttps://code.launchpad.net/~dobey/ubuntuone-installer/update-from-trunk/+merge/9620718:41
nessitadobey: packaging question: when removing patches from a package branch, do I need to say something about it in the changelog file?18:46
thisfrednessita: dobey: my first C merge proposal! https://code.launchpad.net/~thisfred/u1db/c-nested_index/+merge/96215 John will do the actual review, but any feedback at this point is welcome. It's a very small diff.18:46
nessitaa ver...18:46
nessitathisfred: any reason to initialize result but not tmp_expression?18:47
thisfrednessita: no, should I do neither or both? :)18:48
nessitathisfred: I'd say yes... and initialize both, in C you can have any garbage as initial value for a variable18:48
thisfredok18:48
thisfredthx18:48
nessitathisfred: also, is "expression" a C string or can it be binary data?18:48
dobeynessita: yes; just list them and "removed patches which were included upstream."18:49
thisfredit's a string18:49
nessitabinary data == the content of an image, for example18:49
thisfrednessita: no it's always a real "string"18:49
nessitadobey: great. If not patches were left, shall I remove the patches/ dir? and/or the series file?18:49
nessitathisfred: you 100% sure? otherwise strdup will do nonsenses :-)18:49
thisfrednessita: I am sure :)18:50
dobeynessita: yes18:50
nessitathisfred: great18:50
nessitadobey: just to be sure, I need to remove everything under patches, right?18:50
thisfredit's a database index expression, which is a string we interpret to build the index18:50
dobeynessita: if there are no more patches left, you can just remove the patches dir, yes18:51
nessitadobey: ack thanks18:51
nessitathisfred: regarding your XXX there, did you read the strtok man page? it reads18:53
nessita "The strtok() function uses a static buffer while parsing, so it's not thread safe.   Use strtok_r() if this matters to you."18:53
thisfrednessita: I read it was not thread safe, I was just not sure if it mattered in this context, so that's a question for John18:54
nessitaah, I see18:54
thisfredbut it's good to know about strtok_r, I did not see that18:54
thisfredI may just use that anyway18:54
nessitathisfred: so, branch looks very good. I'm not sure if you should free tmp_expression, can't find if strtok dismembers the original string passed to it18:55
thisfredit does yes18:55
thisfredif by dismember you mean change18:56
thisfredand samuele told me to use free after strdup18:56
nessitathisfred: for sure, you need to free the mem given to you. Though I'm not that sure when putting strtok in the middle, that mem is not psuedo destroyed by strtok itself. Let me do an IRL test after I finish these releases :-) but branch looks good, like I said18:57
thisfredthank you for your time so far!18:58
nessita:-)18:58
thisfredconverted to strtok_r19:06
ralsinastrtok is pretty evil19:06
ralsinastrtok uses freaking global variables19:07
nessitaalecu, dobey: as far as we know there was no changes regarding ussoc package dependencies, right?19:09
dobeynessita: i don't know19:09
nessitadobey: just confirming if you perhaps added/removed something from nightlies19:10
* nessita will check nightlies log19:10
alecunessita, "no changes regarding ussoc" <- since *when* are you asking?19:12
nessitaalecu: since last release, 2.99.5, 2 weeks ago19:13
nessitaalecu: regarding ussoc package dependencies19:13
thisfredralsina: what you call global variables, I call 'open closures'19:13
alecunessita, no idea. I can try to find out.19:13
nessitaalecu: if we could cross check the info... I was hoping youc could confirm regarding proxy support only (the rest I got it covered)19:14
briancurtinralsina: https://code.launchpad.net/~brian.curtin/ubuntuone-client/930398-windows-is_root/+merge/96224 should fix is_root19:18
ralsinabriancurtin: yay!19:19
ralsinabriancurtin: looking19:19
briancurtinralsina: i haven't built an installer to *fully* test. i can try to fit that in next. this is just ubuntuone-client changes19:20
briancurtin(but based on what we talked about, i think it should work all around)19:20
ralsinabriancurtin: if you could, it would be great. I doubt a full installer is buildable thugh19:20
ralsinabriancurtin: you could add this patch to 2.99.3 or so I suppose19:21
alecunessita, I could not find any new dependency. I'm looking at the merged sso branches for the past two weeks by mandel.19:22
alecunessita, I see a few new .py files, though.19:22
nessitaalecu: thanks for the check19:22
nessitaalecu: yes, that's fine19:22
nessitaalecu: thanks again!19:22
alecunessita, btw: I see that mandel's branch added "bin/ubuntu-sso-ssl-certificate-qt" to setup.py19:24
nessitaalecu: yes, saw that as well, thanks for pointing it out19:25
alecunessita, do you know how/if we should add a new binary in u1-client?19:25
alecubecause one of my latest branches adds the tunnel binary.19:25
nessitaalecu: not for this release, I guess, but yes, we'd need a new binary, ideally not in the /usr/bin but in the /usr/lib/ubuntuone-client dir19:25
alecunessita, sure, not for this release, obviously.19:26
nessitaalecu: details about how to add it will have to be asked to dobey...19:26
nessitahow to add it in the Makefile I mean19:26
nessitaalecu: grepping for ubuntuone-login may give us an idea :-019:27
nessita:-)19:27
dobeyin the same place syncdaemon is19:28
alecudobey, libexec_SCRIPTS in Makefile.am, right?19:29
alecudobey, nessita, thanks!19:29
dobeyalecu: yes19:31
ralsinaI have to run some errands. Will be back in about one hour,  IRC stays open, feel free to drop requests...19:39
thisfredalfajores por favor19:48
nessitathisfred.request++19:56
dobeynessita: can you review https://code.launchpad.net/~dobey/ubuntuone-installer/update-from-trunk/+merge/96207 real quick?20:02
nessitadobey: yes20:02
nessitadobey: approved20:06
dobeynessita: thanks. can you rereview https://code.launchpad.net/~dobey/rhythmbox-ubuntuone/url-handler/+merge/96045 as well please?20:07
dobeythisfred: care to do some reviews as well? :)20:08
thisfredsure thing20:09
dobeythisfred: https://code.launchpad.net/~dobey/libubuntuone/fix-crits/+merge/9600820:09
dobeythisfred: https://code.launchpad.net/~dobey/libubuntuone/u1ms-load-fixes/+merge/9604620:09
dobeythisfred: https://code.launchpad.net/~dobey/rhythmbox-ubuntuone/url-handler/+merge/9604520:09
nessitadobey: can you please help me finding the new requirements for bumping to standard-version 3.9.3?20:14
nessitaI'm googling but I can't find the proper page20:14
dobeythere are new requirements for that?20:14
nessitadobey: well, lintian complained20:15
nessitaout-of-date-standards-version20:15
nessitadobey: and I made the bump carelessly because I honestly thought it was an administrative thing to complain with. But micahg pointed out I should check if my package meets the criteria, so I was looking for that20:16
nessitasorry, I mean "compliant with" :-)20:17
dobeyoh, well, if it's not compliant, i would think lintian would complain about something else afterward, no?20:17
nessitadobey: I guess, but anyways, I commit to checking the new standard, so I would like to stick to my promise :-)20:18
dobeylintian probably really shouldn't complain about that, though.20:18
ralsinaaaaaaand I amback20:18
dobeyok. i don't know any specific details about it20:18
dobeyand it seems odd for that to change this late in the cycle, anyway20:18
nessitadobey: thanks, will ask in ubuntu-release20:19
dobey"The major patch level will be changed for any change to the meaning of the standards, however small"20:21
nessitadobey: not sure exactly what that means20:25
dobeynessita: it means the policy changes were very minor and probably don't break in terms of the package in question20:26
nessitadobey: ah, perfect20:26
nessitadobey: found the details at http://www.debian.org/doc/packaging-manuals/upgrading-checklist.txt20:26
dobeyhow did you even find that page?20:27
nessitadobey: lots and lost of googling20:27
ralsinasometimes, when you guys discuss policy, I feel like I am in a parallel universe of enterprisey regulations20:28
ralsinathen I remember it comes from Debian, and it all makes sense ;-)20:28
nessitaheh20:28
thisfreddobey: +2. You win: http://spelunkyworld.com/xbla.html20:29
ralsinanessita, dobey, it seems --alert was not used by anyone, ever. So, kill it? Also, opening u1cp-qt should unset the urgent flag, agreed?20:30
thisfreddobey: oh, I reviewed the wrong links20:30
thisfrednow doing the right ones20:30
dobeyyeah i didn't write spelunky20:31
dobeyralsina: focusing the window should unset the urgency. i'm not sure how best to do that from qt20:31
ralsinadobey: a handler for the focusEvent, is trivial20:32
ralsinadobey: and unsetting is not hard. Adds an optional dependency on gi.Unity but that's ok, right?20:32
dobeyralsina: i mean the talking to unity part. i would hope it had a focus event signal or whatever they call such things :)20:32
dobeyralsina: using gobject-introspection from the qt panel makes sense?20:32
ralsinadobey: let me see and I tell you :-)20:33
thisfreddobey: +2 more20:33
ralsinadobey: doesn't seem to break anything.20:34
nessitadobey: from that link, specifically from the copyright-format item, does it mean we need to add a Version line to the copyright file?20:34
ralsinadobey: property gets set, too20:34
nessitaralsina: sorry, was reading a complex page20:34
ralsinanessita: it's ok :-)20:34
dobeynessita: i don't think so20:34
ralsinadobey: the icon even wiggles20:35
nessitaralsina: so, yes to unset urgency when cp is focused20:35
ralsinadobey: ok, the wiggling is something else. No urgent arrow visible, but that's some other lameess20:35
dobeyralsina: if it works, i guess we can do that20:35
ralsinadobey: cool20:35
nessitaralsina: and is it too hard to implement --alert? (just weighting to remove it or not)20:35
ralsinanessita: no, it's a piece of cake20:36
ralsinanessita: if I can remove the urgent bit, I can set it too ;-)20:36
nessitaralsina: I would leave it in that case, *may* be handy20:36
dobeyeh20:36
ralsinanessita: ack20:36
thisfredbriefly afk20:38
nessitadobey: sorry, I know you requested a review... but I gotta run to a start-of-teaching-semester meeting at the uni20:41
nessitaralsina, dobey: ussoc and protocol uploaded to ubuntu, will finish u1cp tomorrow20:41
dobeywell i guess i won't release that today then20:42
ralsinanessita: col20:42
ralsinacool20:42
nessitadobey: what do you need reviewed?20:43
nessitaurl-handler?20:43
dobeynessita: yes; you voted needs info on it20:43
nessitaah, got that done already20:43
nessitawill approve, seems like I forgot20:43
dobeyok, thanks20:43
nessitaapproved20:43
nessitathisfred: will write this test script I mentioned tomorrow... is that ok?20:44
nessitaok, I'm gone.... bye all!20:45
briancurtinralsina: so i tried creating a new bundle to use on the XP machine and it's giving unrelated errors (looking in C:\python27\lib\site-packages... for stuff). since i have to leave soon i dont think i'll be able to more appropriately test it today20:47
ralsinabriancurtin: it's ok, I will make a try tomorrow20:47
briancurtinand by it, i mean just starting the control panel from my built bundle20:47
ralsinabriancurtin: yes, things are not in good shape in windows. Might as well fix them :-)20:48
briancurtinadded this to my todo list20:48
dobeyralsina, thisfred, briancurtin: one of you care to give a quick look over https://code.launchpad.net/~dobey/ubuntuone-client/update-from-trunk/+merge/96237 please?20:49
ralsinadobey: on it!20:49
briancurtini'm off to catch a plane, see you guys in a few days20:55
ralsinadobey: +121:01
dobeythanks21:11
=== jalcine_ is now known as JackyAlcine_
=== JackyAlcine_ is now known as jalcine_
=== jalcine_ is now known as jalcine
dobeyhttps://code.launchpad.net/~dobey/libubuntuone/update-from-trunk/+merge/9624921:36
dobeyhttps://code.launchpad.net/~dobey/rhythmbox-ubuntuone/update-from-trunk/+merge/9625021:36
dobeyok, those are the last two merges to stable-3-0 i need to do i think.21:37
dobeyneed to get off here for a while though, and will finish up releases and do uploads in a bit.21:38
dobeyttfn21:38
ralsinadobey: will try to check early tomorrow morning, I am tired now21:45

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