/srv/irclogs.ubuntu.com/2013/01/16/#ubuntu-devel.txt

xnoxtkamppeter_: you can't. You can boot from sdcard or you can boot via OTG mini-usb port.00:02
xnoxtkamppeter_: google for ogra_ blog posts about booting via mini-usb.00:02
mwhudsonyou can have uboot on the sd card but the rootfs on usb stick00:04
=== slank is now known as slank_away
=== _salem is now known as salem_
=== hggdh_AFK is now known as hggdh
TheMusoYou 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
tkamppeterxnox, thanks.00:17
TheMusoFor the root F sanyway.00:17
xnoxLaney: ;-) here be dragons01:05
=== 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
pittiGood morning06:07
dholbachgood morning07:17
diwicdholbach, congratulations :-)08:52
dholbachthanks diwic :)08:52
Laneyxnox: hoho09:01
ryeHi, is this a place one can draw attention to a bug report for raring?09:04
ryebug #1085342 FWIW09:05
ubottubug 1085342 in totem (Ubuntu) "Pulseaudio applications hang (Totem, GNOME Shell etc.)" [Undecided,Confirmed] https://launchpad.net/bugs/108534209:05
diwicrye, could you check if it is resolved in PA 3.0? We have packages incoming for PA 3.0, let me give you the link09:06
diwicrye, *reading* hmm, maybe it's more of a threading error than PA error?09:07
ryediwic: that's glibc error as far as I understand09:08
ryein NPTL09:08
diwicrye, 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:09
ryediwic: will test that too09:10
ryediwic: 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 blame09:11
* rye has not yet rebuilt libc locally09:11
ryediwic: ok, just a heads up, PA 3.0 also hangs under these conditions09:23
diwicrye, ack, thanks for testing09:23
infinitydiwic: 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
diwicrye, ^09:40
diwicinfinity, sounds good. Does it affect any stable releases?09:40
infinitydiwic: I'm a bit less clear on that.09:42
diwicok09:43
diwicinfinity, probably not as I haven't heard any riots about it09:43
infinitydiwic: Always happy to have more datapoints on the topic, if you can easily reproduce with a backported package or something.09:43
diwicinfinity, ok09:44
LaneyI'm seeing (something which sounds like) this which correlates with my recent dist-upgrade to raring.09:44
=== henrix_ is now known as henrix
Laneydiwic: For me it happens with Spotify too; don't know if that gives you any more data (this uses libasound2)09:45
infinityLaney: If I give you a glibc 2.17 to test, can you tell me if it goes away?09:46
Laneyok09:46
infinityHrm, I'll have to find a build that isn't scary broken.  I trust my source, but none of my binaries...09:46
ryeinfinity: please subscribe me too to glibc 2.17 testing09:48
ryeinfinity: also this can be easily reproduced in VM so if it is broken that's not the end of the world :)09:49
=== Quintasan_ is now known as Quintasan
ryeinfinity: for tracking purposes - bug #108534209:52
ubottubug 1085342 in totem (Ubuntu) "Pulseaudio applications hang (Totem, GNOME Shell etc.)" [Undecided,Confirmed] https://launchpad.net/bugs/108534209:52
infinityrye: Same as Debian #694962?09:53
ubottuDebian bug 694962 in libc6 "libc6:amd64: pulseaudio hangs in pthread_cond_wait" [Important,Open] http://bugs.debian.org/69496209:53
ryeinfinity: yes09:54
ryeinfinity: linked it there09:55
Laneyindeed, my traces have pthread_cond_wait at the top too10:01
infinityRight, let me just lob this at a PPA, then.  People can have a hammer at it.10:02
infinityrye, diwic, Laney: Uploaded to https://launchpad.net/~adconrad/+archive/ppa for testing fun.10:07
Laneycheers10:07
Laneypitti: I suspect this is what you reported in #-desktop too ^^^10:07
cjwatsonev: 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
infinityrye, 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". :P10:08
pittiLaney: sounds right indeed10:08
diwicinfinity, okay10:08
cjwatsonev: I assume it isn't worth smoothing it out *too* much10:08
evcjwatson: I think we can cope with changes up to "fairly frequently", given the proposed design: https://wiki.ubuntu.com/ErrorTracker/PhasedUpdates10:18
cjwatsonYeah, I was just hoping for actual numbers on that :-)  Is once a day "fairly frequently"?  Twice a day?10:19
evhah, 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:19
evand then we can adjust from there10:20
cjwatsonThe 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 time10:20
evah10:20
cjwatsonWhich I think is essentially required anyway since we're changing something in the Packages stanza10:20
evso not something you want to do that often10:20
cjwatsonIt'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, no10:21
cjwatsonev: 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
cjwatson(Will need to get the database column created more or less first)10:22
evI think an int would be fine10:22
cjwatsonOK, good10:22
evcjwatson: we should involve bdmurray in these chats, by the way, as he's taking care of a lot of the implementation10:22
cjwatsonYeah, I was talking to him a bit last night10:22
evah, excellent10:22
cjwatsonMostly I want to get the LP side done and then get out of the way ;-)10:23
=== Sweetsha1k is now known as Sweetshark
evpitti: 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
evcontext: https://wiki.ubuntu.com/ErrorTracker#error10:33
evthat bulleted list will grow10:33
evto include most types of system-internal reports10:33
evwhich we can only make the distinction on by reading .desktop files. Bum.10:34
pittiev: where would we actually save effort by introducing another extension? I'm rather wary of doing that, it requires more changes10:34
pittie. g. in GNOME's mime handler to open them, etc.10:34
pittiev: and no, we can by no means send an error automatically if it appears during logout10:35
evpitti: in what would become "def get_all_silent_reports"10:35
pittiif that is for "[ ] Report internal errors, too", we need to reprocess and load those reports anyway10:35
evthey're not being sent automatically unless the user has explicitly asked for them to be sent automaticlly10:35
evautomatically*10:36
pittialso, I don't think we can decide about silence in the core pipe handler in all cases, can we/10:36
pitti?10:36
evin the vast majority of cases, they appear with the next regular application crash10:36
evbut if the user has checked the box in system settings for "just send the things", then it will be sent automatically10:36
pittifor detecting .desktop files etc. it needs to happen in the apport-gtk post-processing stage, not in the core pipe handler10:36
evyeah, that's what I was wondering10:36
evright, it sounds like this just needs to be a key in the report10:37
pittiev: 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
evand getting all of the silent reports to show with the current application crash involves reading all the report files in /var/crash10:37
evor I guess we could drop another 0-byte hint file10:37
pittiI thought whoopsie would only consider those with an .upload (and without .uploaded) stamp10:37
evpitti: this is for showing them in the UI at all10:38
evso as an example10:38
=== henrix is now known as henrix_
evlets say colord crashes10:38
evthat should not pop up a dialog10:38
evlater, the gimp crashes10:38
evnow a dialog appears saying that gimp crashed and an internal application crashed10:38
evasking you to report both at once10:38
pittiokay10:39
pittibut we need to call apport (post-processing) to find out whether or not a report file is system-internal in the first place10:39
=== henrix_ is now known as henrix
pittiI thought we decide this based on whether it has a .desktop file, don't we?10:40
evindeed, my brain just hadn't gotten that far :)10:40
evwe do10:40
pittiwe probalby have more, such as having a $DISPLAY in env10:40
pitti(which we can detect in the core pipe handler already)10:40
evwe still need to note that the crash file we've just processed is a silent error, to be picked up later though10:40
evso do you prefer a 0-byte .silent file alongside the .crash file, or a key in the report?10:41
evI'm just still thinking of the implementation of "get_all_silent_reports"10:41
pittiev: 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 left10:41
evokay10:42
pittiev: I don't like yet another stamp file TBH, it starts being confusing10:42
pittiI think apport-checkreports could just "do" it, without having lots of other async components in between10:42
pittithat's my gut feeling, anyway10:42
evit does, but we need some way of quickly getting the entire set of silent reports that still need to be presented to the user10:42
evthey're only silent in the sense that they don't get their own dialog10:43
evperhaps silent is a poor term for this10:43
evgrouped might be better10:43
evit does get confusing, that is - I agree10:43
pittiev: so all that update-notifier does is to call "apport-gtk" without arguments10:43
pittiif 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
evwe might want to show dialogs for some system internal reports, as mpt pointed out yesterday10:44
evfor instance, if pulseaudio crashes10:44
evthe user probably wants to know why their sound stopped working10:45
pittiso 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
pittii. e. that wouldn't need rewriting the .crash files yet another time, or adding more stamps; or am I missing someting?10:45
evpitti: there might be some time between colord crashing and the gimp crashing10:45
evI don't think this can happen in one run10:45
evoh10:46
pittiwhy, apport-gtk would see the old colord crash when it is being called for the gimp crash?10:46
evright10:46
pittiisn't that how it was supposed to be?10:46
evI just caught that10:46
evbut10:46
evwhat happens when we have all system internal crashes and no regular application crashes?10:46
evoh, the same thing, I guess10:46
evI see your point10:46
pittiyeah, that's the problem with that model in general; they would never be sent10:46
pittiif the user configured "send them automatically", then apport-checkreports could create the .upload files10:47
evwould we not just leave them unprocessed for the next run of run_crashes?10:47
pittibut if not, then we need the UI to ack the reports10:47
pitti(apport-checkreports is being called at desktop startup and each crash)10:47
evpitti: I think I've got enough to take a stab at the algorithm here. Thank you so much for the chat.10:50
evI'll put an MP up for us to discuss more in depth10:50
pittiev: no worries; we can also have a mumble if you want to discuss in more detail10:50
pittior that :)10:50
ev:)10:50
pittithanks for working on this! looking forward to it10:51
evps. my kingdom for code-inline merge proposal comments10:51
evcheers10:51
pittiI think for now we can identify the "batched" ones with "system crash", right?10:51
evcorrect10:51
evmpt and I are going to refine that on Thursday10:51
evyou're welcome to join us via mumble or gtalk10:52
pittifor the pulseaudio case, we might actually change it so that it appears as an application carsh10:52
evhm, that's an interesting angle10:52
pittiev: please10:52
evnoted10:52
evmpt: what time on Thursday?10:52
pittiso that it appears immediately, with some appropriate verbiage10:52
evright10:52
mptev, 1.30?10:52
evpitti: does that work for you?10:52
pittiev: yes, both UTC and CET10:53
dholbachlooks like we need to do a bit more sponsoring again10:53
evgreat10:53
dholbachwould anyone be available to demo a small bug fix or two? we have one 30m slot left: https://wiki.ubuntu.com/UbuntuDeveloperWeek/Timetable10:54
tjaalton@pilot in11:12
=== 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_
cjwatsonev,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 pain11:40
* dholbach hugs tjaalton11:40
cjwatsonAlthough 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 one11:41
evcjwatson: I think that's entirely okay, mostly because we hadn't considered it ourselves ;)11:43
evsecond pass if we really need it11:44
cjwatsonev: Yeah, not that I'm sure how to do it even in a second pass :-)11:47
cjwatsonev: Looking something along the general lines of http://paste.ubuntu.com/1537437/ so far11:47
cjwatsonHmm, that's not quite right, need to be able to set the phase to None ...11:55
cjwatsonHalf-tempting to invert the phase value in the database so that I can have the default be 0 :-P11:55
dz0nyhi11:56
dz0nyis this te right room for questions about ubuntu-defaults-image?11:56
dz0nythe*11:57
tjaaltondholbach: thanks for adding me to the sponsors & review teams :)11:58
dholbachtjaalton, anytime :)11:58
dz0nyI'am trying to build 32bit iso on 64bit host, and according to logs it builds 64bit every time. I'am using --arch i386 switch12:00
dz0nyhere is build script https://github.com/dz0ny/ubuntu-si/blob/12.04/Makefile12:00
cjwatson--arch didn't work right until quantal12:02
tjaaltondholbach: already forgot that it was TheMuso who added me to -sponsors, whoops :)12:02
tjaaltonanyway..12:02
cjwatsonbug 101612112:03
ubottubug 1016121 in ubuntu-defaults-builder (Ubuntu Precise) "wrong architecture (i386,amd64) packages downloading" [Undecided,Fix released] https://launchpad.net/bugs/101612112:03
dholbachtjaalton, ain't anything stopping you now ;-)12:03
cjwatsonthough that indicates that you should be able to upgrade to ubuntu-defaults-builder 0.31.1 to fix it12:03
dz0nycjwatson: thx12:09
=== zyga is now known as zyga-afk
Laneyinfinity: It certainly seems to be stable again with 2.1712:33
LaneyAphex Twin are helping me to confirm this12:33
=== MacSlow is now known as MacSlow|lunch
ryeinfinity: are there any other bugs that should have been resolved by 2.17 upgrade i could verify - pulseaudio no longer wants to hang12:54
* Laney is relieved that it's not gstreamer's fault :P12:56
caribouQ: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:06
cariboufor 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:07
ricotzLaney, hi, are there are eglibc 2.17 packages available for raring somewhere?13:08
Laneyricotz: infinity provided a PPA of his unfinished packages (still some test failures). ppa:adconrad/ppa13:09
ricotzah i see :)13:09
ricotz(and he didnt told me)13:09
Laneywe were testing a specific bug fix13:10
OdyXcaribou: patch or under debian/13:10
Laneyplus you weren't online to be pung13:10
ricotzLaney, alright ;)13:11
caribouOdyX: thanks13:11
ricotzi will test it13:11
=== yofel_ is now known as yofel
=== cpg is now known as cpg|away
tjaaltonwhat to do with a merge request that is marked "needs fixing" for a month now and polluting the sponsor queue?13:25
Laneychange the status to 'work in progress' at the top13:25
tjaaltonah, thanks13:25
hrw~curse usb-creator-{gtk,kde} badly13:25
xnoxhrw: why?! =)13:28
hrwxnox: took me a while to find out that it expects lot of things to be done by user before using tool.13:29
hrwxnox: such as formatting usbstick as vfat, mounting it etc13:30
=== zyga-afk is now known as zyga
xnoxhrw: it can format by itself, but one needs to download the img/cd.13:30
xnox"erase disk button"13:30
hrwxnox: did that once. UI frozen, no progress13:31
hrwand it remounts stick as 'sync'13:32
hrwso it takes 467 minutes according to progressbar instead of copying in 1-2 minutes, doing all needed and then syncing13:36
xnoxhrw: in raring? yeah i reverted sync, need to reupload.13:36
hrwxnox: yes, in raring13:36
hrwnice set of languages in xubuntu install when booted to live: English, Spanish, Polish, Portugese, xhosa, Chinese.13:59
smoserRAOF, 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:00
ubottubug 1073077 in zsh (Ubuntu) "zsh complains about locale_warn on launch" [Undecided,New] https://launchpad.net/bugs/107307714:01
cjwatsonsmoser: I'll do it shortly14:17
smosercjwatson, thanks.14:21
cjwatsonsmoser: done14:21
=== MacSlow|lunch is now known as MacSlow
hallyncyphermox: have you had a chance to take a look at bug 1099155 ?14:26
ubottubug 1099155 in network-manager (Ubuntu) "[raring] No ip assigned to bridge and no routes added for routed network" [Critical,Confirmed] https://launchpad.net/bugs/109915514:26
evmpt: 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-crash14:28
mptev, no, those three kinds should14:29
mptapp thread, 3rd-party non-packaged, and OS package14:29
evokay, thanks14:29
evpotentially three checkboxes14:30
mptNo14:30
evoh14:30
evright14:30
evbecause that would be regular applications14:30
mptNo, not because of that14:30
mptBecause 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
mptIs that too fiddly? :-)14:31
evahhh14:31
evsorry, I sped through that initially14:31
evno, I think that makes sense14:31
evI was worried about the interaction between those two checkboxes14:32
mptyeah14:32
mptI guess "of this type" would be ambiguous if the "previous internal errors" checkbox preceded it, so we avoid that problem too14:32
evright14:33
=== dannf` is now known as dannf
=== slank_away is now known as slank
cyphermoxhallyn: I have, and upstream is working on it15:03
hallyncyphermox: cool, thanks15:05
=== henrix is now known as henrix_
=== henrix_ is now known as henrix
mptev, 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:19
evah15:20
evwell, thanks to moinmoin, you can add them in, cell by cell ;)15:20
mptyeah15:21
mptreported bug 110032515:33
ubottubug 1100325 in ubuntu-website-content "Table cell borders are invisible in default wiki theme" [Undecided,New] https://launchpad.net/bugs/110032515:33
xnox=)16:02
tjaalton@pilot out16:08
=== 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
dokocjwatson, but it would be something like gcc-4.6-$HOST_TRIPLET [cross]17:11
=== deryck is now known as deryck[lunch]
cjwatsondoko: do you mean "cross profile"?17:16
cjwatsonwhich I think is currently spelled <cross>?17:16
cjwatsondoko: 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 like17:17
dokocjwatson, hmm how? this is about the additional b-d for gcc-4.7 when cross built17:18
cjwatsondoko: That's a different issue17:19
cjwatsondoko: *That* will have to be solved using profiles17:20
cjwatsondoko: 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 fly17:20
cjwatsondoko: This does not prevent using profiles for build-dependencies of the toolchain itself17:20
cjwatsonEntirely orthogonal17:21
cjwatsondoko: 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 that17:21
cjwatsondoko: 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:23
dokook17:24
cjwatsonwhen 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:25
dokocross building a native compiler, so host == target17:26
cjwatsondoko: OK, so my proposal isn't actually wrong for you, it just doesn't solve the entire problem17:37
=== zequence_ is now known as zequence
=== deryck[lunch] is now known as deryck
=== slank is now known as slank_away
jcastrocjwatson: 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
ubottubug 1060404 in grub2 (Ubuntu Precise) "update-grub runs and fails in containers" [High,In progress] https://launchpad.net/bugs/106040419:04
cjwatsonjcastro: fairly sure I already uploaded it, at least the grub-install side (which is as much as is in raring)19:10
cjwatsonjcastro: it's probably waiting for queue review or something19:10
cjwatson(can't check right now)19:10
jcastroSpamapS: ^^19:10
SpamapSthere's no grub in the precise-proposed queue19:11
cjwatsonSpamapS: I'll recheck when I'm not ircing from a phone19:12
SpamapScjwatson: ACK19:13
cjwatsonSpamapS: also I need clarity on whether this is affecting grub-install or also update-grub19:17
cjwatsonbecause when I looked at this last I'd only changed the postinst (grub-install) and apparently that was making people happy19:17
cjwatsonif 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 else19:18
SpamapScjwatson: 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
cjwatsonSpamapS: ok, good, so that suggests that the fix I have is sufficient and I only need to look into where it went19:19
cjwatsonhmm, I think perhaps infinity asked for an unrelated change and I forgot to make that - it's in rejected19:25
cjwatsonSpamapS: I can sort that out later this evening19:25
jcastrocjwatson: thanks for the timely response! \o/19:27
SpamapScjwatson: cool, ping me on upload and I'll review it19:27
=== henrix is now known as henrix_
noisepub2quantal kernels in precise are not packaged correctly for multiarch20:50
noisepub2linux-header-`uname -r` needs to be Multi-arch: foreign20:50
cjwatson#ubuntu-kernel - but might be better to file a bug on linux-lts-quantal20:51
=== salem_ is now known as _salem
stgraberxnox: hey, why did you break su? :)22:24
mlankhorstwhy not!22:25
stgraberxnox: http://paste.ubuntu.com/1539328/22:25
stgraberxnox: 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:26
stgraberhmm, well, not no change rebuild, but no code change rebuild :)22:27
infinitystgraber: That's disconcerting.22:48
=== V is now known as heHATEme
infinitystgraber: $(tty) works, I wonder what's breaking with /dev/tty22:53
stgraberinfinity: 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 ;)22:54
=== heHATEme is now known as Obama
infinitystgraber: Possibly due to the fix for http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=62884323:01
ubottuDebian bug 628843 in login "login: tty hijacking possible in "su" via TIOCSTI ioctl" [Critical,Fixed]23:01
infinitystgraber: In fact, almost certainly due to that fix.23:02
stgraberinfinity: 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
infinitystgraber: If you're starting an interactive shell, don't use su -c?23:04
stgraberinfinity: 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:05
infinitystgraber: If this is a shell script that's expecting to output to a tty, try $(tty)?23:06
infinitystgraber: 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
infinitystgraber: And if that's actual representating code, the fix is s,/dev/tty,$(tty),23:07
stgraberinfinity: "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/bash23:08
=== Obama is now known as vorian
infinityhttp://paste.ubuntu.com/1539530/23:08
infinitystgraber: Yeah, I guess I'm still failing to see how/where that fails.23:10
sarnoldstgraber: perhaps have it spawn su -l for the special case of /bin/bash?23:10
infinitySure, but even this works fine here:23:10
infinitysu root -c "/bin/sh -c '/bin/bash'"23:10
infinitySo, I'm still a bit confused what the actual failure is in arkose.23:10
stgraberinfinity: try doing ctrl+c in that shell, you'll see the problem :)23:12
infinitySession terminated, terminating shell... ...killed.23:12
infinityCute.23:12
stgraberinfinity: 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:13
infinitystgraber: Yeah, that's vaguely unpleasant.23:15
stgrabersarnold: that's because you're assuming that this patch doesn't apply to -l ;) it does23:15
sarnoldstgraber: oh :)23:16
infinitystgraber: Well, but when you detect that the command-line is simple /bin/sh or /bin/bash, you can skip the -c entirely.23:16
infinitystgraber: And "su root" or "su - root" (or similar) doesn't have this problem.23:17
stgraberinfinity: yep, which gets me back, to shipping a list of all known shell in the package... fun...23:17
infinitystgraber: It goes haywire on the nested shells.23:17
infinitystgraber: /etc/shells23:17
infinitystgraber: Read from the container, of course, not the host.23:18
infinitystgraber: (Since that file is dynamically updated when shells are installed/removed)23:18
stgraberinfinity: 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 affected23:18
stgraberso 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:22
infinitystgraber: Some more discussion of the problem in http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=65987823:25
ubottuDebian bug 659878 in login "cannot set terminal process group (-1): Inappropriate ioctl for device" [Normal,Open]23:25
ricotzinfinity, hi, eglibc 2.17 works nicely here so far, deadlocks are gone23:26
infinitystgraber: 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
infinityricotz: Good to hear.23:26
stgraberinfinity: nope, I drop from root to user, so it's specifically the case the patch is meant to prevent23:27
infinitystgraber: Oh, but the initial su (or, at least, the one you gave as an example) is root.23:27
stgraberinfinity: yeah, my example was so that nobody could tell me it was some kind of weird permission issue ;)23:30
=== adam_g_ is now known as adam_g

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