[00:15] chrisccoulson: Why does Thunderbird make it so difficult to send plain-text email? [00:22] RAOF: How so? I've never had a problem sending plain text email... [00:22] Mind you I always set things up such that I view plain text emails, and make sure my replies are sent as plain text only. [00:24] RAOF: you just have to find the right places to turn it off [00:24] RAOF: i agree -- it's pretty damn obnoxious [00:24] RAOF: consider, though, that mozilla is very very much about html, as a company [00:24] and it makes a bit of sense [00:24] Yeah. [00:29] RAOF: it's one preference [00:30] micahg: Where? [00:30] Edit -> preferences -> composition -> Send OPtions [00:31] And add * to Plain Text Domains? [00:32] hrm, that's not what I thought it was [00:32] Yeah, that's where I looked first, but it doesn't have a “Just send plain text, damnit” option. [00:33] i think it's a per-account setting or something [00:33] i remember it _not_ being easy to find [00:34] ya... under account settings for each account [00:34] there is a checkbox in "Composition and addressing" [00:34] Ah, yes. There we go. [00:34] Good work on that dialog being > 768 pixels high, by the way. [00:35] WIN [00:38] HTML in email really shoudl have been banned years ago. [00:39] Its a bitch when I get emails that don't have a plain text equivalent. [00:39] Whoops, meant pain, a little milder... [00:39] heh [00:40] i'm infuriated enough when i get html-only email that i can't even imagine... [00:45] TheMuso: bitch is fine :> [00:49] heh ok. [00:50] TheMuso: if it was aimed at someone, it definitely wouldn't be [00:50] TheMuso: but its very mild on the swearometer, and you are normally -so- polite ... :) [00:51] lifeless: i find people's swearometers are calibrated differently according to country [00:53] desrt: yeah [01:39] Man, those fglrx-updates/nvidia [01:40] Man, those fglrx-updates/nvidia-updates SRUs are a gnarly mass of unknown status. [01:40] RAOF: Unknown status? [01:41] duflu: It's hard to determine what the status of the various pieces are. Have the updates been verified? Which packages are verified? What about Godot? [01:41] RAOF: So the usual... just trust us... this new version is better :) [01:42] Or are they being uploaded without any -proposed testing? [01:43] Oh, they're in -proposed now. [01:43] I'm just trying to work out whether they should be move from -proposed to -updates. [01:44] RAOF: Mind-reading of graphics vendors is in your job description, no? [01:45] Oh, thsi is a problem partially of my own making. The *bugs* are a gnarly mass of confusion. [01:47] Ok. So the only thing preventing fglrx-updates from hitting precise-proposed is https://bugs.launchpad.net/ubuntu/+source/fglrx-installer-updates/+bug/1039397 not being verified. [01:47] Ubuntu bug 1039397 in fglrx-installer-updates (Ubuntu) "[Dell Precision M6700] HDMI audio fails, HDMI/Displayport output has blank frame" [Medium,Incomplete] === bschaefer_ is now known as bschaefer === rickspencer3_ is now known as rickspencer3 [02:20] RAOF, yeah it's quite a confusing mess. Think the verification requirements are being overly strict? [02:22] bryce: Not really. Two of the bugs linked in the SRU are not actually bugs, so can't be verified. That's my fault for letting the package into proposed. [02:22] part of the problem is the installer for the proprietary drivers involves a bit of playing russian roulette [05:29] Good morning [06:36] pitteah [07:45] good morning [08:00] good morning [08:02] salut jibel! [08:05] hey didrocks, how are you? [08:06] pitti: still tired, but going in the right direction :) [08:06] yourself? [08:06] I'm quite fine, thanks; making some nice progress on mocking hardware [08:06] sweet! [08:06] gphoto2 --auto-detect already works, now working on actually mocking the data as well [08:07] excellent :) [08:07] hey didrocks === tkamppeter_ is now known as tkamppeter [10:06] hey desktopers [10:06] pitti, hey, https://launchpad.net/ubuntu/+source/gconf/3.2.5-0ubuntu6 ... did you just forget to push or did you forgot to use the vcs? [10:07] pitti, if that's the second one don't bother, I can just merge it back there from the archive version [10:07] seb128: sorry, forgot VCS; I'll commit it now, for teaching myself :) [10:07] pitti, danke ;-) [10:08] seb128: done, sorry [10:09] pitti, no worry, thanks! === vrruiz_ is now known as rvr_ === Sweetsha1k is now known as Sweetshark [11:26] seb128, hmm, i have a slight problem with gsettings on the nexus ... we ship http://paste.ubuntu.com/1562339/ ... and in dconf-editor i can see "suspend" being the setting thats used, nontheless it gives me the interactive dialog *until* i once switch the setting in dconf-editor manually once [11:26] any idea what that could be ? [11:29] ogra_, do you run glib-compile-schemas on rootfs/usr/share/glib-2.0/schemas/ubuntu-defaults-nexus7-powerbtn.override to get gschemas.compiled updated? [11:29] ogra_, the glib triggers take care of that on the system when installing package, not sure that happens for your rootfs [11:29] i thought dh does that automatically [11:30] oh, rootfs is only a package path [11:30] the binary package puts it in / [11:30] (some weird setup of the ubuntu-defaults stuff) [11:30] ogra_, gsettings get org.gnome.settings-daemon.plugins.power button-power [11:31] well, i changed it now, let me reset it first [11:31] what does that give when it's behaving buggy? [11:31] k [11:31] well, dconf-editor showed the right value [11:31] seb128: ah, wrt. https://blueprints.launchpad.net/ubuntu/+spec/desktop-r-gnome-plans-review I wasn't sure whether the "update .." ones were DONE already, as we don't yet have their final versions [11:31] ("suspend") [11:32] pitti, well, we are on the unstable serie and will track it through version, I think it's good enough for the workitems, but feel free to change it back if they are useful for tracking for you [11:32] seb128: so you think they are enough for "get to current GNOME 3.7.x version", and the rest will just be part of the normal update cycle? [11:32] pitti, yes [11:32] seb128: I'm on d-devel-list, so through that I should get a reminder for new devel releases [11:33] seb128, "set to default" in dconf-editor gets me "suspend" back in the value field and makes the entry non-bold ... though pressing the power button now gets me the dialog again [11:33] seb128: if you are also watching, please feel free to prod me if I forget [11:33] pitti, well, I'm watching http://people.canonical.com/~platform/desktop/desktop.html in any case ;-) [11:33] pitti, will do [11:33] * pitti te donne une accolade [11:33] * seb128 hugs pitti back [11:34] ogra_, if the gsettings command return the right value then there is probably a bug in gnome-session... [11:34] ogra@nexus7:~$ gsettings get org.gnome.settings-daemon.plugins.power button-power [11:34] 'interactive' [11:34] ogra@nexus7:~$ [11:34] aha [11:34] funny, why does dconf show the other value [11:35] could it be that i miss any quoting in the .override file ? [11:36] seb128: is https://blueprints.launchpad.net/ubuntu/+spec/desktop-r-gnome-fallback even relevant for this cycle still? [11:37] seb128: i. e. do we want to move the automounting code in this cycle to be prepared, or move it to free resources for nexus 7 and friends? [11:37] ogra_, dconf-editor is hackish, gsettings lacks the api it needs so it does guess work by parsing schemas itself [11:38] pitti, yes that spec is still revelant, and it would be good to clean/move the automount code, that's not the highest item on the list though [11:39] ogra_, try ='suspend' yes (with the '') [11:42] no change [11:42] even tried double quotes [11:42] it is a list value, i wonder if it needs special treatment due to that [11:44] hmm, no it says string [11:49] seb128, there is an upstream fix for bug 1101213, can you apply it? Thanks. [11:49] Launchpad bug 1101213 in gnome-control-center (Ubuntu) "G-c-c printing doesn't allow you to select a hp printer" [High,Triaged] https://launchpad.net/bugs/1101213 [11:49] tkamppeter, sure, thanks for looking to it! [11:52] Laney: btw, the 2nd blog post should answer to some of your question about how to interact with them [11:52] them beeing the PS packages [11:52] Laney: if you have any question, I think we should file a FAQ in the end [11:52] (that I'll paste to a wiki page) [11:53] didrocks: awesome, will read! [11:55] I think I want a simple document that just says [11:55] - bzr branch [11:55] - hack hack hack [11:55] - bzr commit --fixes=lp:foo; bzr push lp:~/...; bzr lp-propose ... [11:55] Laney: can you add that to the desktop pad? [11:55] - set commit message on MP [11:55] Laney: I'll grab the questions for doing the FAQ [11:56] ok I'll add my questions there [11:56] thanks :) [11:56] yubikeyyyyyyyyyy [11:56] Laney: part 3 will be really on the daily release process, what's the bots are doing and so on [11:59] seb128, argh ... [11:59] ogra_, ? [11:59] so the file needs to end in .gschema.override, not in .override [11:59] :P [11:59] * ogra_ slaps forehead [11:59] lol [11:59] sure [11:59] sorry I didn't pay attention to the filename [12:02] didrocks: what happened here: https://launchpadlibrarian.net/128756347/indicator-power_12.10.6daily13.01.17-0ubuntu1_12.10.6daily13.01.18-0ubuntu1.diff.gz - no explanation of the changes in the changelog? [12:03] Laney: because nobody attached a bug to their branch, I'm keep telling them to do that, please join the effort :) [12:03] interesting [12:03] Laney: i guess lp:indicator-power has commit messages. [12:03] I'm sure it does [12:04] right, but commit messages can be noisy and not really explanatory [12:04] that's why I decided to limit at bug reports, which is what we generally list for any releases [12:04] does it use the commit message you can set on MPs if they are set? [12:04] you also don't want an hundred commit messages listed in debian/changelog if there is an hundred commits [12:04] "g_type_init() is deprecated as of glib 2.35.4, so bump our glib minimum requirement and remove the g_type_init() call." [12:04] is the commit message and whould be useful. [12:04] xnox: in that case, yeah, but not the general rule [12:05] I did a MP for something the other day and got told to set a commit message there [12:05] there is also the changelog spam effect [12:05] didrocks: what about using first line only? (git-dch style) [12:05] should those be generally useful? [12:05] * [-r161] g_type_init() is deprecated as of glib 2.35.4, so bump our glib minimum requirement and ... [12:05] xnox: will cut in the middle a commit message, even worse [12:05] xnox: they don't use that the "short/long summary" [12:05] s/that// [12:05] clearly we should switch to git then [12:05] Laney, those are a requirement for the merger bot, not sure why anymore, something to do with the readability of the vcs logs I think [12:05] * xnox hides [12:06] Laney: and it's only on the unity stack, different teams are using different flows, other are just taking the description then [12:06] seb128: Perhaps. I've never known what they are for really but it could be useful in this case [12:06] (see part 2 ;)) [12:06] and I think this doesn't solve the changelog spam [12:07] I think important changes needs to have a bug attached [12:07] or being just specified manually in debian/changelog as part of the merge [12:07] this is more an educational thing than anything else [12:07] larsu: btw, can you keep that in mind when reviewing/doing changes please? ^ [12:09] didrocks, is a --fixes lpXXXX on the commit enough? [12:09] or associating the MR with the bug? [12:09] larsu: either the one, or the other, or even just mentionning "this fixes bug #XXXX" any commit message of the changes [12:09] s/the one/this/ [12:10] I'm quite flexible to spot that :) [12:10] you don't want to see the regexp :p [12:10] haha [12:10] okay :) [12:11] thanks! [12:11] hmmmmm [12:11] didrocks, np, I actually prefer having a bug reference myself [12:11] I just forget sometimes [12:11] and you get the praise then! [12:11] with your name :) [12:12] in the changelog :p [12:12] does it already do one changelog entry per MP? [12:12] one changelog entry per bug [12:12] as you would find in any NEWS file [12:13] ack [12:13] so it feels a bit heavy to make people file bugs when they can just JFDI a fix [12:14] Laney: well, if they don't want to file bugs, they can themselve add it to debian/changelog as part of the MP [12:14] if they feel it's important enough to get communicated [12:14] ah, that is allowed eh [12:14] yep :) [12:14] we get to my questions on the pad now [12:14] so let's leave it here :P [12:14] yep :) [12:14] and if they already referenced the bug in the changelog [12:15] there is no duplication === jhernandez is now known as jhernandez|lunch === MacSlow is now known as MacSlow|lunch [12:53] What rolling release?! Nooooo.... :> [12:53] This means less release parties :( [12:54] just make them bug fixing parties then [12:55] ogra_, I have those on a weekly basis! ;) [13:00] lol [13:19] hey seb128, how are you? [13:20] BigWhale: What are you on about it means that you have to party everyday for the newly released stuff [13:24] chrisccoulson, hey, I'm good, how are you? [13:24] seb128, yeah, not too bad thanks [13:24] chrisccoulson, just woke up? ;-) [13:24] i wish :) [13:25] seb128, i've got this for you http://paste.ubuntu.com/1562859/ ;) [13:25] chrisccoulson, will you be unhappy if I discard it? [13:26] chrisccoulson, I just noticed this morning, looking at version, that cairo 1.12.10 is out for a week and it includes that fix (with a stack of other ones) [13:26] chrisccoulson, I was going to just do the update [13:26] seb128, oh, that's ok then :) [13:26] chrisccoulson, you can do the update if you are looking for launchpad karmas though ;-) [13:27] seb128, yeah, i don't mind doing that [13:27] chrisccoulson, great, thanks [13:28] chrisccoulson, http://www.cairographics.org/releases/cairo-1.12.10.tar.xz [13:29] thanks === Ursinha_ is now known as Ursinha [13:51] g_variant_parse doesn't return a floating reference, does it? [13:59] chrisccoulson, according to gvariant-parser.c, it does [13:59] docs could be a little more clear there === jhernandez|lunch is now known as jhernandez [14:00] but I think close to everything that gives you a variant gives you a floating ref [14:00] desrt, right? === MacSlow|lunch is now known as MacSlow [14:15] Sweetshark: you know most OSS are on G+ right ;) [14:17] it's like a glossy magazine for geeks [14:17] so now I get content on there, twitter, and my inbox, 3-4 times o the same story :) fun [14:17] * czajkowski bounces her nexus 7 off a wall [14:18] testing the gravity sensor? :P [14:18] my on screen keyboard has vanished [14:19] and I've tried a lot of things now to make it come back [14:19] nothing works and I want it for fosdem [14:19] can you attach usb? [14:21] mlankhorst: yup [14:21] czajkowski, is it activated in system settings, universal access, input tab? [14:22] http://ubuntuone.com/0S97eFdCGOGC1Dk6mEeFlZ [14:23] that's probably not going to work [14:23] indeed [14:23] it's not [14:24] that's not Universal Access panel, that's Keyboard panel [14:24] czajkowski, that's not what I asked [14:24] http://ubuntuone.com/0qF9v89DMgQG7jEUGyxzOD [14:24] there? [14:24] yeah [14:24] seb128: sorry that was a paste from earlier on [14:24] czajkowski, do you have a keyboard or a ssh access to type commands? [14:24] cant get up the keyboard at all [14:25] can you ssh to it or plug an usb keyboard for debugging? [14:25] it's plugged in now. [14:26] czajkowski, can you see if onboard is running (ps ax | grep onboard) [14:32] czajkowski: echo chambers are fun ;) [14:32] not really :) [14:32] seb128: thank you [14:33] czajkowski: but one can test on G+, what works and then carry that to the outside world ... [14:33] czajkowski, yw, good luck [14:34] I have a weird issue with online accounts. It says I have an AIM account that requires authorization once I start empathy, but online accounts has no record of this AIM account. I'm searching for a bad config file somewhere, but not exactly sure where to look. Any ideas? [14:34] bcurtiswx, run empathy-accounts [14:35] bcurtiswx, I guess it's an account you configured in empathy itself by then, out of the integrated online accounts [14:35] seb128, looks to be the case. thx [14:35] yw ;-) [14:40] if i wanted to issue a command 30 seconds after I lock the screen and have it quit after i return. what would I use ? === m_conley_away is now known as m_conley [14:58] larsu, doesn't http://git.gnome.org/browse/glib/tree/glib/gvariant-parser.c#n2371 make sure that the GVariant returned is not floating? [14:59] chrisccoulson, oh, nice catch. I must have read over that, sorry :) [15:00] larsu, i think ted must have assumed it returned a floating reference too [15:00] http://bazaar.launchpad.net/~dbusmenu-team/libdbusmenu/trunk.13.04/view/head:/libdbusmenu-glib/server.c#L1061 [15:00] and http://bazaar.launchpad.net/~dbusmenu-team/libdbusmenu/trunk.13.04/view/head:/libdbusmenu-glib/menuitem.c#L1659 [15:00] in fact, everywhere it uses g_variant_parse ;) [15:01] larsu: no [15:01] larsu: only constructors [15:01] chrisccoulson, yes :( [15:01] desrt, thanks [15:01] larsu: and you have to pretend that g_variant_builder_end() is called g_variant_new_from_builder() :) [15:02] haha [15:02] fantastic :) [15:02] does anyone have some rope i could borrow for a few moments? [15:03] desrt, another question: g_main_context_pending seems to return FALSE if there are no immediate sources to be dispatched. Is there a function to find out whether there are *any* sources? [15:03] dental floss OK ? [15:03] larsu: you have to poll... [15:03] :-/ [15:03] so like prepare/query/poll/check [15:03] this makes writing a loop that stops when no events are attached very hard [15:04] heh [15:04] * larsu could use that for his tests [15:04] yup :) [15:04] desrt, care to add a function that does this? [15:04] we talked about adding this to gmaincontext [15:04] but you don't want it [15:04] why? [15:04] because 'pending' is a very difficult term [15:04] is a timeout that will fire in 1 second from now pending? [15:04] well, I just spend 20 minutes finding out "pending" is not what I thought it meant [15:05] YES. [15:05] okay [15:05] how about a pending async result that will fire 1 second from now? [15:05] yes [15:05] that's the problem [15:05] because you don't know about these? [15:05] async results don't become GSources until they are (immediately) ready to dispatch [15:05] right [15:06] so that made us think maybe we should have like g_main_context_hold() as a sort of special "don't exit now!" ref [15:06] then the whole thing just started feeling too much like GApplication [15:07] hm, and that's a bad thing? [15:07] meh [15:07] it would require changes everywhere [15:07] everyone who may at some point add an idle would have to call this hold() [15:08] i guess GTask and GSimpleAsyncResult could be updated to do that... [15:08] but this is starting to get out of hand, wouldn't you agree? [15:08] https://bugzilla.gnome.org/show_bug.cgi?id=690126 is the bug btw [15:08] Gnome bug 690126 in gdbus "Do not quit mainloop while there are pending GSources" [Normal,Assigned] [15:15] desrt, agreed. It seemed like an easy thing at first :) [15:24] larsu: did you do a lesson yet? :) [15:24] desrt, no :( [15:24] dude [15:25] you're going to be in toronto in two weeks and unable to speak the local language [15:25] there's just so much other stuff I need to take care of [15:25] lessons are quick :p [15:25] I do have the tab open, does that count? [15:25] for something? [15:26] hahahah [15:26] you TODO list by tab, hm? [15:26] not usually, only in this case [15:26] because I actually meant to do it :) [15:27] whenever i see ken he always says "haven't heard from lars yet..." [15:28] he already know me? [15:28] *knows [15:28] he knows you will be sending lessons soon :) [15:28] heh [16:11] grrr, on 3G === alex3f is now known as alexef === alexef is now known as alex3f [17:07] seb128: can i get you or someone to look at my patch on https://bugs.launchpad.net/ubuntu/+source/gnome-keyring/+bug/859600 please? [17:07] Ubuntu bug 859600 in gnome-keyring (Ubuntu Precise) "Please convert gnome-keyring to multiarch" [High,In progress] [17:16] dobey, will have a look [17:21] seb128: thanks [17:27] dobey: i think last time around this was attempted, the use case was to have the library co-installed and the problem was that the /etc/ conffile encodes multiarch path. [17:27] which means that conf file is no longer the same across all architectures and thus failing to co-install. [17:27] has this been addressed? [17:28] * xnox can't figure that bit out just by looking at the patch. [17:28] * xnox didn't build both amd64/i386 & try to co-install them yet. [17:30] xnox: which conf file? [17:31] let me dig into it. 1sec. [17:31] xnox: i have them co-installed on my machine from my ppa right now. [17:31] awesome =) [17:31] xnox: the p11kit conf file only has the module name in it [17:31] hmm.. [17:32] /etc/pkcs11/modules/gnome-keyring.module that file, that is [17:40] dobey: why is the patch against oneiric and not raring?! [17:41] xnox: eh? [17:41] +gnome-keyring (3.6.2-0ubuntu2) raring; urgency=low [17:41] dobey: i downloaded not your patch... [17:41] xnox: you're not looking at *my* patch :) [17:42] i was looking at your patch, but my clipboard had the other url in it... [17:42] oh [17:42] =) all is fine now. [17:43] thanks [18:17] chrisccoulson, still working on that cairo update? [18:18] chrisccoulson, if you do, please include http://cgit.freedesktop.org/cairo/commit/?id=fa4f48cccb6c7f4e1afb2ff4b98b906b7d8d4afc , it fixes a regression from the new version [18:18] c.f bug #1101316 [18:18] Launchpad bug 1101316 in cairo (Ubuntu) "Evince switches to wrong page while scrolling in continuous mode" [High,Invalid] https://launchpad.net/bugs/1101316 [18:36] Hello, I have Ubuntu 12.10 and a POS system EBN X-950 with touchscreen (EgalaxyTouch according to the manual), so, i tried $lsusb and it doesn't list the touchscreen controller. If I do a screen /dev/ttyS[0-4] can't get any input from the touchscreen. Already did $ modprobe -r usbtouchscreen and still doesn't detect, any ideas what's the problem? [18:43] LIDH: #ubuntu is the general help channel [18:43] ok [18:43] thank you but i've already post there, and no one helped me :s [18:45] well be patient [18:46] if you're impatient, nobody will want to help you :) [18:47] LIDH: check 'dmesg' output for information, you might get lucky [19:21] xnox, thanks for fixing bug 1099589 ;-) [19:21] Launchpad bug 1099589 in devscripts (Ubuntu) "wrap-and-sort crashed with TypeError in save(): must be str, not bytes" [Critical,Fix released] https://launchpad.net/bugs/1099589 [19:21] robru: np =) [20:13] hi seb128. sorry, was away for dinner and exercise [20:13] yeah, i'll include that change too [20:14] chrisccoulson, no worry, thanks ;-) [20:15] chrisccoulson, I hope for you it's "exercise and dinner", not good to go to cycle after eating ;-) [21:00] mterry, if you get a sec I'd like to ask you some questions about that mp you sent me [21:00] robru, the deja-dup one? sure [21:00] mterry, yeah, so, what steps should I take to ensure that your change is working [21:00] ? [21:04] robru, well make check is always good, but I only fixed the existing tests to work with the code change, I didn't add any new tests for it, because I didn't know how to fake partitions without root access [21:04] robru, so I personally verified it manually. The trick is to do a backup from a partition that is different than the partition /tmp is on [21:05] robru, then run "G_MESSAGES_DEBUG=all deja-dup --backup" and notice the arguments we pass to duplicity. There should be a --tempdir with an appropriate (same partition as source files) directory [21:05] mterry, yeah, that's what i meant. so my home directory is different than tmp [21:05] robru, so deja-dup should pass ~/.cache/deja-dup/tmp as tempdir to duplicity [21:06] mterry, oh, btw, I started using deja-dup for my actual, real $HOME backups. Will that command interfere with the backups I have going? [21:06] should i make a dummy user for this? ;-) [21:08] robru, it will use your normal settings to make a backup yeah. But it won't (shouldn't) interfere with them, just start a new backup [21:08] mterry, so it'll try to backup all 500GBs then? maybe I will make a dummy user ;-) [21:08] robru, you also might want to double confirm that ~/.cache/deja-dup/tmp is empty after the run [21:09] robru, well, it will do an incremental or whatever, but if you haven't made a first backup yet, then yah. dummy or change your backup settings momentarily [21:09] mterry, ok, great. I'll try it out shortly. thanks [21:49] mterry, I'm getting a weird xmllint error on 'make check' [21:49] xmllint? [21:50] paste me [21:50] mterry, http://paste.ubuntu.com/1564351/ does this mean anything to you? [21:50] probably a missing dep or something simple [21:50] robru, oh, I bet the gnome-docs automake stuff is missing a dep on the 'all' target for the 'check' target [21:50] but I'm not sure [21:50] robru, run make && make check [21:51] ahhh [21:52] in unrelated news, today I found a little netbook of mine that's been in storage since November. It had raring installed on it... now I'm just installing all 905 updates ;-) [21:53] mterry, ok, xmllint is passing, tests are going ok so far, just have to do the manual checks and then I'll approve your branch [21:54] robru, cool. Thanks! [21:54] robru, heh, only 905?! [21:56] mterry, well, it was put in storage *late* in november ;-) [21:56] so just 905 updates for december/jan [21:57] :) [22:07] mterry, ok, so if I see '--tempdir=/home/robru/.cache/deja-dup/tmp' in the debugging output, then that for sure means it worked? ~/.cache/deja-dup/tmp directory was never used prior to this branch? [22:08] robru, right [22:08] robru, nor was --tempdir either [22:09] mterry, k, great. just gonna peruse the diff briefly then approve ;-) [22:17] robru, I just noticed something myself looking at the diff [22:17] oh? [22:17] robru, I changed DuplicityInstance.start from normal to async. And thus dropped it's 'throw Error' line [22:17] ... ok? [22:18] robru, but, when I called it from DuplicityJob before, I would bubble up the error message via show_error(). In this new diff, the error message gets swallowed by DuplicityInstance and the user will end up just seeing a generic "Unknown error" message [22:18] robru, let me fix that real quick [22:18] ok [22:22] mterry, oh, I like the way you allow this tempdir logic to be overridden by an env variable. That's thinking ahead ;-) [22:31] robru, I fixed the issue I found locally, but I decided to write a test for this case for the future. But I have to run. So I'll update tomorrow [22:31] mterry, ok [22:34] robru, thanks for the review! [22:34] mterry, yw, have a good night! === m_conley is now known as m_conley_away