/srv/irclogs.ubuntu.com/2015/03/27/#ubuntu-desktop.txt

pittiGood morning05:57
didrocksgood morning06:48
pittibonjour didrocks !06:51
didrockshey pitti06:52
happyaronmorning didrocks pitti :)07:25
didrockshey happyaron07:27
larsuhappy Friday every107:33
pittihey happyaron, hello larsu!07:34
didrockspitti: yeah, confirmed on an installed system, I should just have tried on the live…07:59
seb128good morningg desktopers08:00
didrockspitti: so, as this will stay a distro-patch anyway, I propose that I do the minimal workaround in systemd-machine-id-commit itself, meaning checking if there is an overlayfs mountpoint in /etc/machine-id or parent08:01
didrocksre seb12808:01
didrocksas*08:01
didrocksthat way, the systemd binary isn't at risk, and the patch is smaller08:01
pittibonjour seb128, comment vas-tu ?08:02
pittididrocks: sounds good to me08:02
didrockspitti: I don't know why the bigger patch hangs TBH, sounded sane enough to me, but maybe there is an evaluation of mount points too early or such…08:03
didrocksbut let's go minimal as upstream wants a kernel fix08:03
seb128hey pitti, ca va bien ! et toi ?08:07
pittiseb128: ça va bien aussi, merci !08:11
didrockswaow, one run, two bugs discovered on ubuntu make, I guess this guy should apply for the QA team :p08:12
* didrocks understands the first one, not yet the second though08:12
pittihaha08:30
willcookeyawn - morning08:56
didrocksmorning willcooke08:57
willcookeTheMuso, still around?08:58
=== dpm is now known as dpm-afk
=== sil2100_ is now known as sil2100
willcookehey larsu - shall we meet?09:28
larsuwillcooke: yep09:29
=== Zdra is now known as xclaesse
Sweet5harkseb128: So, I did an 4.4.1 update for the ppa including the breeze icons. Since this is FFes for kubuntu, it should be uploaded soonish. However, Friday uploads are bad, so lets target that for Monday?09:43
Sweet5harkseb128: FWIW, 4.4.2~rc2 has been tagged upstream today and likely will be 4.4.2 final. I will update the ppa as soon as this update is in -proposed. Likely still rather a 0-day SRU as it will get tight with timing otherwise.09:47
=== greyback__ is now known as greyback
didrockspitti: phew, I found an easy fix!10:41
didrocksfor the machine-commit-id10:41
didrocks(the issue is when major(parent = 0) and major(child) != 010:41
didrockssorry, I meant, we only wait to fallback in that case10:42
didrocks(as it's overlayfs-like)10:42
didrocksbut it seems we can get major(parent) = 0 and major(child) = 010:42
didrockswe do not want to fallback in that case10:42
didrocks(case of tmpfs)10:42
pittididrocks: oh, good! I sorted out the iscsi stuff, so I'd like to do a merge with Debian exp10:43
pittiso please poke me if/once you have a patch10:43
didrockspitti: do you want me to repropose a patch with the fix? (I'm going to do a full suite of test with the package installed)10:43
didrockspitti: I can give it to you if you want to run some tests in parallel10:43
pittididrocks: sure!10:44
didrockspitti: http://people.canonical.com/~didrocks/tmp/0001-Fix-mount-point-detection-on-overlayfs-and-similar-f.patch10:52
didrockspitti: the only diff is replacing if (major(b.st_dev) == 0) by if (major(b.st_dev) == 0 && major(a.st_dev) != 0)10:52
didrockspitti: I'm rebuilding the ubuntu package with this, now that I removed all my manual breaks and printf statements to ensure it's fine10:53
willcookeseb128, do we need to worry about this new glibc do you think?10:53
didrockswhile it's building, going for a run (and then I guess it will be 20 minutes for a full test with vm, vm-iso, /etc/machine-id on ro…)10:53
pittididrocks: ah, ok; thanks! I can test it on Debian exp on an installed system, if you want to test it on a live?10:54
didrockspitti: sure, the installed system isn't the most difficult, but we can do like that10:55
didrockspitti: ok, ubuntu package building, going for a run meanwhile :)10:57
pittididrocks: enjoy! I'll test on Debian installed10:57
didrocksthanks!10:57
pittididrocks: works fine on installed; let me know how it goes on live, then I'll push11:17
pittididrocks: note that there was some noise with gbp-pq import/export11:17
didrockspitti: oh right (yeah, failed to take a break, now waiting for it to finish to build)11:18
tkamppeterwillcooke, hi11:22
willcookehey tkamppeter!  Got your mail, just waiting to hear back from Rick11:23
didrockspitti: no, forget about it, there is another use case I can't get working with this patch11:38
didrockspitti: if only, I could get my printf to shows up while it's hanging…11:40
pittididrocks: ah, meh -- you didn't go running?11:42
didrockspitti: yeah, wanted to get a good news on this before going… :p11:43
didrocksefail11:43
=== dpm-afk is now known as dpm
=== MacSlow is now known as MacSlow|lunch
didrockspitti: ok, the issue is the call due to readlink_and_make_absolute when called on some /sys path12:12
* happyaron sents out bunches of emails to gsoc prospective students12:16
didrockspitti: ok, understood it, what happens is:12:17
didrocks-> /sys/<bla> is read12:17
didrocksthen, we try to readlink_and_make_absolute12:17
didrocksthis errors out12:17
didrockserrno != ENOENT12:17
didrockswe return -errno12:18
didrocksand so, systemd doesn't like it (I guess it prevents some execution with /sys setup)12:18
didrocksso, I guess instead of returning -errno, we can return a.st_dev != b.st_dev12:18
didrocksas it's the previous behavior12:18
darkxsthey seb128 did you see my comment on the schemas bug?12:25
didrockspitti: recreated the patch, building now, going finally for a run, and then, I'll test installed, live and such…12:30
seb128Sweet5hark, monday is fine12:40
seb128willcooke, glibc shouldn't be an issue for us no12:40
seb128darkxst, yeah, I saw, I didn't think much about it yet though12:41
Sweet5harkseb128: great, thx12:41
seb128yw12:41
Sweet5harkalso: c- c- c- c-combobreaker for sebs reply volley!12:42
darkxstseb128, ok, it doesn't really need thinking about now, just wasnt sure if re-assigned bugs would get back to you12:43
didrockspitti: \o/ success (both permanent, live and temporary machine-id)12:59
pittididrocks: yay! what was it?13:00
didrockspitti: yeah, I couldn't resist to get this testing before taking a break :)13:00
didrockspitti: basically on /sys path, readlink will return some errno13:00
didrocksand I guess that systemd stops going on with detecting if path is a mountpath if it returns < 013:00
didrocks(so doesn't read some /sys content and such…)13:01
didrocksthe fix is, instead of returning errno (not being able to read a symlink in a directory is a valid one anyway), use the major() comparison13:01
didrocksso basically, being a no-op for those cases13:01
didrocksI extended the same in case we are in error reading /proc/self/mounts btw13:02
didrocks(fallbacking to the previous comparison method)13:02
didrockspitti: so, that should be it: http://people.canonical.com/~didrocks/tmp/0001-Fix-mount-point-detection-on-overlayfs-and-similar-f.patch13:03
didrocksNote the "return a.st_dev != b.st_dev;" if we fail in the fallback13:03
=== MacSlow|lunch is now known as MacSlow
=== alan_g is now known as alan_g|lunch
didrockspitti: if that makes sense to you, taking my break now :)13:10
pittididrocks: I won't say anything to you before you leave :)13:10
* didrocks is gone thus :p13:11
didrocks(for real)13:11
cyphermoxpitti: didrocks: this is an interesting patch. What bug is it fixing?13:28
pitticyphermox: detecting mount points on overlayfs; the other two standard methods don't work due to kernel bugs13:29
cyphermoxI'm dealing with the CD ejection and mountpoint when casper ends, it could be useful13:30
cyphermoxthough right now the problem appears to be more with plymouth and/or plymouth's systemd units13:30
pitticyphermox: I'm not sure what util-linux' "mountpoint" program does, though13:32
pittii. e. if it also just compares stat().st_dev, or falls back to parsing /proc/mounts13:32
cyphermoxok13:32
cyphermoxI think most of casper is just looking at /proc/mounts13:32
=== alan_g|lunch is now known as alan_g
didrockspitti: so? :)14:48
* didrocks back from running14:49
didrocksah, seems you took it \o/14:50
pittididrocks: just back from running as well :)14:52
pittididrocks: yep, in exp and uploaded to vivid14:52
didrockssweet!14:52
pittihm, or did I? /me misses upload and rejection msg14:54
didrockspitti: forgot to dput? :p14:54
didrocks(nothing in -changes)14:54
* pitti dputs again *shrug*14:55
pittino, I did have an .upload14:55
* didrocks opens the black hole and looks into it… but nothing :)14:56
seb128hum, I wonder why gedit is the default handler for text/x-apport types15:07
larsuseb128: because it's the best15:17
seb128larsu, ?15:17
larsuseb128: look at the date15:17
seb128larsu, it's friday? ;-)15:18
larsuindeed15:18
seb128hehe15:18
seb128I was unsure if that was a real reply or a troll :p15:18
larsuit's remarkably silent here for a Friday15:18
seb128$ LC_ALL=C gvfs-mime --query text/x-apport15:19
seb128Default application for 'text/x-apport': gedit.desktop15:19
seb128....15:19
seb128Recommended applications:15:19
seb128apport-gtk.desktop15:19
seb128 15:19
seb128I don't get why gedit is default if the recommended one is apport-gtk15:19
seb128it's the only .desktop matching the exact mime15:19
=== JanC_ is now known as JanC
larsudid you override it acceidentally?15:20
larsu*accidentally15:20
seb128no, it's the same in a daily iso15:20
seb128in->on15:20
larsuweird15:20
seb128indeed15:22
seb128I wonder if that's a glib bug15:22
* larsu summons desrt into the second channel in the last 10 minutes15:22
seb128lol15:22
seb128oh, look at that15:24
seb128works if I LD_LIBRARY_PATH glib 2.4015:24
seb128deeessssrtt15:24
larsuuh oh15:27
seb128https://bugzilla.gnome.org/show_bug.cgi?id=74428215:28
ubot5Gnome bug 744282 in gio "gvfs-open for application/x-virt-viewer changed behaviour between 2.40 and 2.42" [Normal,New]15:28
seb128so it's a desrt bug15:29
pittiseb128: oh, I noticed that gedit is quite greedy these days; it apparently wants to open pretty much anything :/15:33
pittiI think I even had it open a PDF or iso the other day15:33
seb128pitti, right, that's due to ^15:33
seb128the pdf maybe not15:33
pittiwell, maybe not PDF, but something really strange15:33
pittiit's been a while15:33
seb128but apparently it prefers the specified handler for subtypes, e.g x-text, which is gedit, to an handle for the exact type15:34
=== alex-abreu is now known as alex-abreu|doc
* willcooke -> EOD. 17:29
willcookeHave a good weekend all, I'm on holiday next week - don't touch my stuff17:29
=== alex-abreu|doc is now known as alex-abreu
=== alan_g is now known as alan_g|EOW
=== mfisch is now known as Guest90957

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