[00:02] tkamppeter_: you can't. You can boot from sdcard or you can boot via OTG mini-usb port. [00:02] tkamppeter_: google for ogra_ blog posts about booting via mini-usb. [00:04] you can have uboot on the sd card but the rootfs on usb stick === slank is now known as slank_away === _salem is now known as salem_ === hggdh_AFK is now known as hggdh [00:17] You may as well use an SD card if you are thinking of using a USB stick. IMO a rotery USB disk is better. [00:17] xnox, thanks. [00:17] For the root F sanyway. [01:05] Laney: ;-) here be dragons === cpg|away is now known as cpg === imbrando1 is now known as nodnaebmi === salem_ is now known as _salem === V is now known as SATAN === SATAN is now known as Guest24219 === Guest24219 is now known as V [06:07] Good morning [07:17] good morning [08:52] dholbach, congratulations :-) [08:52] thanks diwic :) [09:01] xnox: hoho [09:04] Hi, is this a place one can draw attention to a bug report for raring? [09:05] bug #1085342 FWIW [09:05] bug 1085342 in totem (Ubuntu) "Pulseaudio applications hang (Totem, GNOME Shell etc.)" [Undecided,Confirmed] https://launchpad.net/bugs/1085342 [09:06] rye, could you check if it is resolved in PA 3.0? We have packages incoming for PA 3.0, let me give you the link [09:07] rye, *reading* hmm, maybe it's more of a threading error than PA error? [09:08] diwic: that's glibc error as far as I understand [09:08] in NPTL [09:09] rye, ok, then I'm not the right person to answer. PA 3.0 is available for testing at https://launchpad.net/~ubuntu-audio-dev/+archive/ppa btw. [09:10] diwic: will test that too [09:11] diwic: basically the bug can be triggered very easily - totem videofile -> seek back and forth a bit. Sometimes totem does not even start playing so in case PA 3.0 has a workaround for this glibc bug then it should stop manifesting. Otherwise all aps will still fail and pulseaudio will look like the one to blame [09:11] * rye has not yet rebuilt libc locally [09:23] diwic: ok, just a heads up, PA 3.0 also hangs under these conditions [09:23] rye, ack, thanks for testing [09:40] diwic: If that's the bug I'm thinking of, it's fixed in glibc 2.17, which I'm bouncing to the archive soonish. [09:40] rye, ^ [09:40] infinity, sounds good. Does it affect any stable releases? [09:42] diwic: I'm a bit less clear on that. [09:43] ok [09:43] infinity, probably not as I haven't heard any riots about it [09:43] diwic: Always happy to have more datapoints on the topic, if you can easily reproduce with a backported package or something. [09:44] infinity, ok [09:44] I'm seeing (something which sounds like) this which correlates with my recent dist-upgrade to raring. === henrix_ is now known as henrix [09:45] diwic: For me it happens with Spotify too; don't know if that gives you any more data (this uses libasound2) [09:46] Laney: If I give you a glibc 2.17 to test, can you tell me if it goes away? [09:46] ok [09:46] Hrm, I'll have to find a build that isn't scary broken. I trust my source, but none of my binaries... [09:48] infinity: please subscribe me too to glibc 2.17 testing [09:49] infinity: also this can be easily reproduced in VM so if it is broken that's not the end of the world :) === Quintasan_ is now known as Quintasan [09:52] infinity: for tracking purposes - bug #1085342 [09:52] bug 1085342 in totem (Ubuntu) "Pulseaudio applications hang (Totem, GNOME Shell etc.)" [Undecided,Confirmed] https://launchpad.net/bugs/1085342 [09:53] rye: Same as Debian #694962? [09:53] Debian bug 694962 in libc6 "libc6:amd64: pulseaudio hangs in pthread_cond_wait" [Important,Open] http://bugs.debian.org/694962 [09:54] infinity: yes [09:55] infinity: linked it there [10:01] indeed, my traces have pthread_cond_wait at the top too [10:02] Right, let me just lob this at a PPA, then. People can have a hammer at it. [10:07] rye, diwic, Laney: Uploaded to https://launchpad.net/~adconrad/+archive/ppa for testing fun. [10:07] cheers [10:07] pitti: I suspect this is what you reported in #-desktop too ^^^ [10:08] ev: I've been looking at the LP implementation of phased updates. Do you have any thoughts/requirements/etc. on how frequently the phase value for a given package version should be changed? [10:08] rye, diwic, Laney: There's one test failure in there that still scares the crap out of me, plus a few more things I need to land, so I don't recommend this for "production". :P [10:08] Laney: sounds right indeed [10:08] infinity, okay [10:08] ev: I assume it isn't worth smoothing it out *too* much [10:18] cjwatson: I think we can cope with changes up to "fairly frequently", given the proposed design: https://wiki.ubuntu.com/ErrorTracker/PhasedUpdates [10:19] Yeah, I was just hoping for actual numbers on that :-) Is once a day "fairly frequently"? Twice a day? [10:19] hah, sorry. Frequently would be several times a day. I think this is something we're going to have to tune once we start getting real world data in. So perhaps start at twice a day? [10:20] and then we can adjust from there [10:20] The reason I'm asking is that it looks like the easiest way to do this in LP is to treat phase as an override, and use changeOverride for it; this should be very simple but involves creating new publications each time [10:20] ah [10:20] Which I think is essentially required anyway since we're changing something in the Packages stanza [10:20] so not something you want to do that often [10:21] It's not horrible since there's only so much in -updates that hasn't been phased up to 100% yet, but I wouldn't want to be doing it every half-hour, no [10:22] ev: Also, what granularity do you need? Would a 0-100 int be fine, or does it need to be a floating-point type for some reason? [10:22] (Will need to get the database column created more or less first) [10:22] I think an int would be fine [10:22] OK, good [10:22] cjwatson: we should involve bdmurray in these chats, by the way, as he's taking care of a lot of the implementation [10:22] Yeah, I was talking to him a bit last night [10:22] ah, excellent [10:23] Mostly I want to get the LP side done and then get out of the way ;-) === Sweetsha1k is now known as Sweetshark [10:33] pitti: do you have any strong opinions on whether silent reports should be implemented as a new filename suffix (.silentcrash) or as a key off the report itself? I'm inclined the say the former, even though it requires changes to whoopsie as well, as it keeps us from having to read through every report file in /var/crash to find the silent ones. [10:33] context: https://wiki.ubuntu.com/ErrorTracker#error [10:33] that bulleted list will grow [10:33] to include most types of system-internal reports [10:34] which we can only make the distinction on by reading .desktop files. Bum. [10:34] ev: where would we actually save effort by introducing another extension? I'm rather wary of doing that, it requires more changes [10:34] e. g. in GNOME's mime handler to open them, etc. [10:35] ev: and no, we can by no means send an error automatically if it appears during logout [10:35] pitti: in what would become "def get_all_silent_reports" [10:35] if that is for "[ ] Report internal errors, too", we need to reprocess and load those reports anyway [10:35] they're not being sent automatically unless the user has explicitly asked for them to be sent automaticlly [10:36] automatically* [10:36] also, I don't think we can decide about silence in the core pipe handler in all cases, can we/ [10:36] ? [10:36] in the vast majority of cases, they appear with the next regular application crash [10:36] but if the user has checked the box in system settings for "just send the things", then it will be sent automatically [10:36] for detecting .desktop files etc. it needs to happen in the apport-gtk post-processing stage, not in the core pipe handler [10:36] yeah, that's what I was wondering [10:37] right, it sounds like this just needs to be a key in the report [10:37] ev: right, but wouldn't that be implemented by creating teh .upload files at the time of the crash, not by some GUI having to go through and create them for you? [10:37] and getting all of the silent reports to show with the current application crash involves reading all the report files in /var/crash [10:37] or I guess we could drop another 0-byte hint file [10:37] I thought whoopsie would only consider those with an .upload (and without .uploaded) stamp [10:38] pitti: this is for showing them in the UI at all [10:38] so as an example === henrix is now known as henrix_ [10:38] lets say colord crashes [10:38] that should not pop up a dialog [10:38] later, the gimp crashes [10:38] now a dialog appears saying that gimp crashed and an internal application crashed [10:38] asking you to report both at once [10:39] okay [10:39] but we need to call apport (post-processing) to find out whether or not a report file is system-internal in the first place === henrix_ is now known as henrix [10:40] I thought we decide this based on whether it has a .desktop file, don't we? [10:40] indeed, my brain just hadn't gotten that far :) [10:40] we do [10:40] we probalby have more, such as having a $DISPLAY in env [10:40] (which we can detect in the core pipe handler already) [10:40] we still need to note that the crash file we've just processed is a silent error, to be picked up later though [10:41] so do you prefer a 0-byte .silent file alongside the .crash file, or a key in the report? [10:41] I'm just still thinking of the implementation of "get_all_silent_reports" [10:41] ev: update-notifier calls apport-checkreport; in principle this could go through and do the "is system internal" decision, create .stamp files for the "silent uplaod" ones, and add the desktop information to the .crash; and then report in its exit status whether there are "interactive" ones left [10:42] okay [10:42] ev: I don't like yet another stamp file TBH, it starts being confusing [10:42] I think apport-checkreports could just "do" it, without having lots of other async components in between [10:42] that's my gut feeling, anyway [10:42] it does, but we need some way of quickly getting the entire set of silent reports that still need to be presented to the user [10:43] they're only silent in the sense that they don't get their own dialog [10:43] perhaps silent is a poor term for this [10:43] grouped might be better [10:43] it does get confusing, that is - I agree [10:43] ev: so all that update-notifier does is to call "apport-gtk" without arguments [10:44] if we define "grouped" in terms of "system vs. app", then we already have API to decide that for a particualr report, don't we? [10:44] we might want to show dialogs for some system internal reports, as mpt pointed out yesterday [10:44] for instance, if pulseaudio crashes [10:45] the user probably wants to know why their sound stopped working [10:45] so ui.py's run_crashes() already iterates through all of them; it coudl then do the decision which ones to show interactively and which ones to batch? [10:45] i. e. that wouldn't need rewriting the .crash files yet another time, or adding more stamps; or am I missing someting? [10:45] pitti: there might be some time between colord crashing and the gimp crashing [10:45] I don't think this can happen in one run [10:46] oh [10:46] why, apport-gtk would see the old colord crash when it is being called for the gimp crash? [10:46] right [10:46] isn't that how it was supposed to be? [10:46] I just caught that [10:46] but [10:46] what happens when we have all system internal crashes and no regular application crashes? [10:46] oh, the same thing, I guess [10:46] I see your point [10:46] yeah, that's the problem with that model in general; they would never be sent [10:47] if the user configured "send them automatically", then apport-checkreports could create the .upload files [10:47] would we not just leave them unprocessed for the next run of run_crashes? [10:47] but if not, then we need the UI to ack the reports [10:47] (apport-checkreports is being called at desktop startup and each crash) [10:50] pitti: I think I've got enough to take a stab at the algorithm here. Thank you so much for the chat. [10:50] I'll put an MP up for us to discuss more in depth [10:50] ev: no worries; we can also have a mumble if you want to discuss in more detail [10:50] or that :) [10:50] :) [10:51] thanks for working on this! looking forward to it [10:51] ps. my kingdom for code-inline merge proposal comments [10:51] cheers [10:51] I think for now we can identify the "batched" ones with "system crash", right? [10:51] correct [10:51] mpt and I are going to refine that on Thursday [10:52] you're welcome to join us via mumble or gtalk [10:52] for the pulseaudio case, we might actually change it so that it appears as an application carsh [10:52] hm, that's an interesting angle [10:52] ev: please [10:52] noted [10:52] mpt: what time on Thursday? [10:52] so that it appears immediately, with some appropriate verbiage [10:52] right [10:52] ev, 1.30? [10:52] pitti: does that work for you? [10:53] ev: yes, both UTC and CET [10:53] looks like we need to do a bit more sponsoring again [10:53] great [10:54] would anyone be available to demo a small bug fix or two? we have one 30m slot left: https://wiki.ubuntu.com/UbuntuDeveloperWeek/Timetable [11:12] @pilot in === udevbot changed the topic of #ubuntu-devel to: Ubuntu 12.10 released | Archive: Open | Dev' of Ubuntu (not support or app devel) | build failures -> http://qa.ubuntuwire.com/ftbfs/ | #ubuntu for support and dicussion of hardy -> quantal | #ubuntu-app-devel for app development on Ubuntu http://wiki.ubuntu.com/UbuntuDevelopment | See #ubuntu-bugs for http://bit.ly/lv8soi | Patch Pilots: tjaalton === _salem is now known as salem_ [11:40] ev,bdmurray: Also, do you think it's a problem not to support different phases on different architectures? Supporting that with the apt-ftparchive-based override handling turns out to be a right pain [11:40] * dholbach hugs tjaalton [11:41] Although I still think putting the phase on BPPH is the right place - it'd just be a limitation of the override handling that if the phases were different across architectures it would effectively just pick one [11:43] cjwatson: I think that's entirely okay, mostly because we hadn't considered it ourselves ;) [11:44] second pass if we really need it [11:47] ev: Yeah, not that I'm sure how to do it even in a second pass :-) [11:47] ev: Looking something along the general lines of http://paste.ubuntu.com/1537437/ so far [11:55] Hmm, that's not quite right, need to be able to set the phase to None ... [11:55] Half-tempting to invert the phase value in the database so that I can have the default be 0 :-P [11:56] hi [11:56] is this te right room for questions about ubuntu-defaults-image? [11:57] the* [11:58] dholbach: thanks for adding me to the sponsors & review teams :) [11:58] tjaalton, anytime :) [12:00] I'am trying to build 32bit iso on 64bit host, and according to logs it builds 64bit every time. I'am using --arch i386 switch [12:00] here is build script https://github.com/dz0ny/ubuntu-si/blob/12.04/Makefile [12:02] --arch didn't work right until quantal [12:02] dholbach: already forgot that it was TheMuso who added me to -sponsors, whoops :) [12:02] anyway.. [12:03] bug 1016121 [12:03] bug 1016121 in ubuntu-defaults-builder (Ubuntu Precise) "wrong architecture (i386,amd64) packages downloading" [Undecided,Fix released] https://launchpad.net/bugs/1016121 [12:03] tjaalton, ain't anything stopping you now ;-) [12:03] though that indicates that you should be able to upgrade to ubuntu-defaults-builder 0.31.1 to fix it [12:09] cjwatson: thx === zyga is now known as zyga-afk [12:33] infinity: It certainly seems to be stable again with 2.17 [12:33] Aphex Twin are helping me to confirm this === MacSlow is now known as MacSlow|lunch [12:54] infinity: are there any other bugs that should have been resolved by 2.17 upgrade i could verify - pulseaudio no longer wants to hang [12:56] * Laney is relieved that it's not gstreamer's fault :P [13:06] Q:when files that are not part of the upstream tarball needs to be added to a package, how should it be done ? quilt patch even for new files ? [13:07] for instance, I wrote a script that uses binaries from the tarball and I want to create a new pkg; should that new script be a quilt patch ? [13:08] Laney, hi, are there are eglibc 2.17 packages available for raring somewhere? [13:09] ricotz: infinity provided a PPA of his unfinished packages (still some test failures). ppa:adconrad/ppa [13:09] ah i see :) [13:09] (and he didnt told me) [13:10] we were testing a specific bug fix [13:10] caribou: patch or under debian/ [13:10] plus you weren't online to be pung [13:11] Laney, alright ;) [13:11] OdyX: thanks [13:11] i will test it === yofel_ is now known as yofel === cpg is now known as cpg|away [13:25] what to do with a merge request that is marked "needs fixing" for a month now and polluting the sponsor queue? [13:25] change the status to 'work in progress' at the top [13:25] ah, thanks [13:25] ~curse usb-creator-{gtk,kde} badly [13:28] hrw: why?! =) [13:29] xnox: took me a while to find out that it expects lot of things to be done by user before using tool. [13:30] xnox: such as formatting usbstick as vfat, mounting it etc === zyga-afk is now known as zyga [13:30] hrw: it can format by itself, but one needs to download the img/cd. [13:30] "erase disk button" [13:31] xnox: did that once. UI frozen, no progress [13:32] and it remounts stick as 'sync' [13:36] so it takes 467 minutes according to progressbar instead of copying in 1-2 minutes, doing all needed and then syncing [13:36] hrw: in raring? yeah i reverted sync, need to reupload. [13:36] xnox: yes, in raring [13:59] nice set of languages in xubuntu install when booted to live: English, Spanish, Polish, Portugese, xhosa, Chinese. [14:00] RAOF, it looked like you were looking at my cloud-init -proposed, i'd *really* appreciate it if you could push those through. (per comment on bug 1073077, i'm not sure how you're "supposed" to show 'verfication-done' for two different releases) [14:01] bug 1073077 in zsh (Ubuntu) "zsh complains about locale_warn on launch" [Undecided,New] https://launchpad.net/bugs/1073077 [14:17] smoser: I'll do it shortly [14:21] cjwatson, thanks. [14:21] smoser: done === MacSlow|lunch is now known as MacSlow [14:26] cyphermox: have you had a chance to take a look at bug 1099155 ? [14:26] bug 1099155 in network-manager (Ubuntu) "[raring] No ip assigned to bridge and no routes added for routed network" [Critical,Confirmed] https://launchpad.net/bugs/1099155 [14:28] mpt: is it correct that only third-party non-packaged applications still have a "ignore future errors of this type" checkbox? https://wiki.ubuntu.com/ErrorTracker#non-app-crash [14:29] ev, no, those three kinds should [14:29] app thread, 3rd-party non-packaged, and OS package [14:29] okay, thanks [14:30] potentially three checkboxes [14:30] No [14:30] oh [14:30] right [14:30] because that would be regular applications [14:30] No, not because of that [14:31] Because of "*and* there is no “Report previous internal errors too” checkbox (so that we don’t show more than two checkboxes at once)." [14:31] Is that too fiddly? :-) [14:31] ahhh [14:31] sorry, I sped through that initially [14:31] no, I think that makes sense [14:32] I was worried about the interaction between those two checkboxes [14:32] yeah [14:32] I guess "of this type" would be ambiguous if the "previous internal errors" checkbox preceded it, so we avoid that problem too [14:33] right === dannf` is now known as dannf === slank_away is now known as slank [15:03] hallyn: I have, and upstream is working on it [15:05] cyphermox: cool, thanks === henrix is now known as henrix_ === henrix_ is now known as henrix [15:19] ev, I just realized why that part of the spec was unclear: In the default wiki theme, table cell borders are invisible. I use the "modernized_cms" theme, where they're visible. [15:20] ah [15:20] well, thanks to moinmoin, you can add them in, cell by cell ;) [15:21] yeah [15:33] reported bug 1100325 [15:33] bug 1100325 in ubuntu-website-content "Table cell borders are invisible in default wiki theme" [Undecided,New] https://launchpad.net/bugs/1100325 [16:02] =) [16:08] @pilot out === udevbot changed the topic of #ubuntu-devel to: Ubuntu 12.10 released | Archive: Open | Dev' of Ubuntu (not support or app devel) | build failures -> http://qa.ubuntuwire.com/ftbfs/ | #ubuntu for support and dicussion of hardy -> quantal | #ubuntu-app-devel for app development on Ubuntu http://wiki.ubuntu.com/UbuntuDevelopment | See #ubuntu-bugs for http://bit.ly/lv8soi | Patch Pilots: === henrix is now known as henrix_ === henrix_ is now known as henrix [17:11] cjwatson, but it would be something like gcc-4.6-$HOST_TRIPLET [cross] === deryck is now known as deryck[lunch] [17:16] doko: do you mean "cross profile"? [17:16] which I think is currently spelled ? [17:17] doko: I don't think it's necessary since this would be something sbuild/dpkg-checkbuilddeps would synthesise when they already know they're cross-building; though you can think of it that way if you like [17:18] cjwatson, hmm how? this is about the additional b-d for gcc-4.7 when cross built [17:19] doko: That's a different issue [17:20] doko: *That* will have to be solved using profiles [17:20] doko: I'm talking about packages that build-depend *on* gcc-4.6 or whatever, and that it would be useful to automatically translate that build-dep into gcc-4.6:native, gcc-4.6-HOST on the fly [17:20] doko: This does not prevent using profiles for build-dependencies of the toolchain itself [17:21] Entirely orthogonal [17:21] doko: But the reason I was asking you was that it would be helpful to have some way to detect that there's a -HOST version available that doesn't involve going and looking in the apt cache, because it'd be a layering violation for dpkg-checkbuilddeps to do that [17:23] doko: I do see that the problem you describe isn't quite simple enough that profiles will magically solve it; but it's not the same as the one I was describing and I was hoping to be able to solve problems one at a time :-) [17:24] ok [17:25] when you said "target-triplet" in that sbuild bug - we're not building a cross-compiler in this case, so is target == build or == host here? [17:26] cross building a native compiler, so host == target [17:37] doko: OK, so my proposal isn't actually wrong for you, it just doesn't solve the entire problem === zequence_ is now known as zequence === deryck[lunch] is now known as deryck === slank is now known as slank_away [19:04] cjwatson: anything I can do to help a precise fix for bug #1060404 ? Most of our juju handouts/website's documentation runs into this. [19:04] bug 1060404 in grub2 (Ubuntu Precise) "update-grub runs and fails in containers" [High,In progress] https://launchpad.net/bugs/1060404 [19:10] jcastro: fairly sure I already uploaded it, at least the grub-install side (which is as much as is in raring) [19:10] jcastro: it's probably waiting for queue review or something [19:10] (can't check right now) [19:10] SpamapS: ^^ [19:11] there's no grub in the precise-proposed queue [19:12] SpamapS: I'll recheck when I'm not ircing from a phone [19:13] cjwatson: ACK [19:17] SpamapS: also I need clarity on whether this is affecting grub-install or also update-grub [19:17] because when I looked at this last I'd only changed the postinst (grub-install) and apparently that was making people happy [19:18] if it's also causing people problems on kernel upgrades as well as grub-pc upgrades, then I think there's been some relevant fix somewhere else [19:19] cjwatson: I'm not familiar enough with the mechanics to know for sure. The reports are coming from people installing new grub's, not new kernels. [19:19] SpamapS: ok, good, so that suggests that the fix I have is sufficient and I only need to look into where it went [19:25] hmm, I think perhaps infinity asked for an unrelated change and I forgot to make that - it's in rejected [19:25] SpamapS: I can sort that out later this evening [19:27] cjwatson: thanks for the timely response! \o/ [19:27] cjwatson: cool, ping me on upload and I'll review it === henrix is now known as henrix_ [20:50] quantal kernels in precise are not packaged correctly for multiarch [20:50] linux-header-`uname -r` needs to be Multi-arch: foreign [20:51] #ubuntu-kernel - but might be better to file a bug on linux-lts-quantal === salem_ is now known as _salem [22:24] xnox: hey, why did you break su? :) [22:25] why not! [22:25] xnox: http://paste.ubuntu.com/1539328/ [22:26] xnox: I'm suspecting the new upstream version to be the problem as cjwatson's no change rebuild is fairly unlikely to have caused this (and I can't find old binaries for -1ubuntu1 to check) [22:27] hmm, well, not no change rebuild, but no code change rebuild :) [22:48] stgraber: That's disconcerting. === V is now known as heHATEme [22:53] stgraber: $(tty) works, I wonder what's breaking with /dev/tty [22:54] infinity: yeah, not too sure, but I have a few scripts that spawn a shell at some point through su and were rather unhappy about the /dev/tty situation ;) === heHATEme is now known as Obama [23:01] stgraber: Possibly due to the fix for http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=628843 [23:01] Debian bug 628843 in login "login: tty hijacking possible in "su" via TIOCSTI ioctl" [Critical,Fixed] [23:02] stgraber: In fact, almost certainly due to that fix. [23:04] infinity: yep, that looks like it, not sure if there's a way around it for cases where you specifically want access to the tty (such as when you're starting a shell and don't want it to be close to impossible to use) [23:04] stgraber: If you're starting an interactive shell, don't use su -c? [23:05] infinity: sure, but I have that small problem that I use su in arkose and that the actual command is passed by the user, so I have no clue as to what they'll be running and quite a lot of them (well, it's the default) happen to call /bin/bash :) [23:06] stgraber: If this is a shell script that's expecting to output to a tty, try $(tty)? [23:07] stgraber: I guess I'm not sure what the specific use case is that's leading to this failure, since "echo foo > /dev/tty" seems like an artificial reproducer. [23:07] stgraber: And if that's actual representating code, the fix is s,/dev/tty,$(tty), [23:08] infinity: "arkose -h" was the original source of the bug, it'd create an LXC container, then spawn a shell inside it, which would be as root (as lxc requires root privilege), then use su to drop back to user privileges inside that shell and then run a user provided command from there. That user provided command defaults to /bin/bash === Obama is now known as vorian [23:08] http://paste.ubuntu.com/1539530/ [23:10] stgraber: Yeah, I guess I'm still failing to see how/where that fails. [23:10] stgraber: perhaps have it spawn su -l for the special case of /bin/bash? [23:10] Sure, but even this works fine here: [23:10] su root -c "/bin/sh -c '/bin/bash'" [23:10] So, I'm still a bit confused what the actual failure is in arkose. [23:12] infinity: try doing ctrl+c in that shell, you'll see the problem :) [23:12] Session terminated, terminating shell... ...killed. [23:12] Cute. [23:13] infinity: with a simple su, it's not a big problem as it'll get you out of your shell and just mess a bit with your console, with arkose as it's wrapped in python, that means that the container won't get killed and you end up with a stack trace and a dozen extra mount points (as arkose can't catch the fact that the user exits and call the cleanup function) [23:15] stgraber: Yeah, that's vaguely unpleasant. [23:15] sarnold: that's because you're assuming that this patch doesn't apply to -l ;) it does [23:16] stgraber: oh :) [23:16] stgraber: Well, but when you detect that the command-line is simple /bin/sh or /bin/bash, you can skip the -c entirely. [23:17] stgraber: And "su root" or "su - root" (or similar) doesn't have this problem. [23:17] infinity: yep, which gets me back, to shipping a list of all known shell in the package... fun... [23:17] stgraber: It goes haywire on the nested shells. [23:17] stgraber: /etc/shells [23:18] stgraber: Read from the container, of course, not the host. [23:18] stgraber: (Since that file is dynamically updated when shells are installed/removed) [23:18] infinity: oh, right, forgot about that one ;) and I'll also need a list of all X terminal emulator as unfortunately arkose supports X application, so xterm will also be affected [23:22] so yeah, seems like I'll have to spend way more time on arkose this cycle than I had first planned... Porting to python3-lxc should avoid some of the su-related issues and the rest I can re-implement the rest of the privilege dropping directly in python so that it's less likely to break than su (and will work the same way across distro) [23:25] stgraber: Some more discussion of the problem in http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=659878 [23:25] Debian bug 659878 in login "cannot set terminal process group (-1): Inappropriate ioctl for device" [Normal,Open] [23:26] infinity, hi, eglibc 2.17 works nicely here so far, deadlocks are gone [23:26] stgraber: The suggestion to not drop the terminal when suing to root could be a bit of a workaround for your case, maybe. Still leaves it weird for others. [23:26] ricotz: Good to hear. [23:27] infinity: nope, I drop from root to user, so it's specifically the case the patch is meant to prevent [23:27] stgraber: Oh, but the initial su (or, at least, the one you gave as an example) is root. [23:30] infinity: yeah, my example was so that nobody could tell me it was some kind of weird permission issue ;) === adam_g_ is now known as adam_g