/srv/irclogs.ubuntu.com/2010/03/17/#ubuntu-devel.txt

keesGuest36494: yeah, timing issues for me.00:23
keesGuest36494: and sometimes i/o stalls :(00:23
KeybukI should plug the wrong AC adapter into Dells more often00:44
Keybukit's great for debugging00:44
Keybukyou get a nice *BEEP*BEEP* and have to press F100:44
Keybukotherwise it won't switch on/reboot00:45
lifelessheh00:45
lifelessstill a dell adaper?00:45
Keybukso I can then press F1 and hold down shift ;)00:45
keesslangasek, cjwatson: I thought P-a-s was in sync with debian?00:46
keeswe don't seem to have this change: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=54360100:47
ubottuDebian bug 543601 in buildd.debian.org "prelink compiles on all architectures" [Normal,Open]00:47
slangasekkees: it's manually synced00:47
keesgar00:47
cjwatsonlp:~ubuntu-core-dev/packages-arch-specific/ubuntu00:47
cjwatsonmerged from lp:packages-arch-specific00:47
* kees fixes00:47
cjwatsonwe've had our own branch for ... maybe a year now?00:47
keesokay, I've merged the Debian changes from Feb and Mar now00:52
Keybukdamnit bzr00:54
Keybukif you don't know a push location, why not try and see if you can push to the parent hmm?00:54
* slangasek grins00:54
lifelessKeybuk: because thats usually the wrong thing00:55
Keybukno it isn't00:55
RAOFlifeless: Really?  When is pushing to the parent the wrong thing?00:55
Keybukbzr branch lp:blahblahblah00:55
Keybukwork on it00:55
Keybukbzr push00:55
lifelessbbiab00:56
lifelessRAOF: everytime you don't own the branch00:56
Keybukif you don't own it, you can't push to it00:56
Keybukso it'll fail00:56
Keybukand then you can say "no push location"00:56
Keybukit could at least *try* :)00:56
keeslool: is there any way to run ARM with >256M of memory?  all it seems to do is segv if I try to change it.00:57
Keybukhell, surely at branch point it knows whether or not you can write?00:57
Keybukso bzr branch could save the push location as the parent if it happens to know it's writable00:57
barryi'm wondering if anybody can help me understand something in the intersection between dh_install and policy kit on lucid01:02
slangasekbarry: shoot01:04
barryslangasek: thanks.  so i'm nearly done with my dbus refactoring of computer janitor.  it needs to install a polkit policy.  mvo extended computerjanitor.install to install the policy file to /usr/share/PolicyKit/policy but that doesn't work.  i believe it needs to go to /usr/share/polkit-1/actions01:05
barryslangasek: now, i've hacked the .install file to do that, but i had to explicitly mention the PolicyKit path and i'm wondering, shouldn't dh_install install it to the right place automatically?01:06
Sarvattkees: thats a limitation on the platform emulated by qemu afaik, I think lool has a patch to allow 512MB though but not sure it applies to the more recent qemu's or it'd be there I imagine01:06
barryslangasek: otoh, i don't know all the magic that dh_install actually does ;)01:06
slangasekbarry: dh_install is the generic "install files where I tell you to" command01:06
slangasekbarry: there are many other specialized commands that install common files of various sorts in the obvious places, but there's no dh_policykit that I'm aware of01:07
barryslangasek: i think you're right, i couldn't find a dh_installpolicykit either01:07
barryslangasek: so, mvo had this originally in the computer-janitor.install file:01:07
barry/usr/share/PolicyKit01:08
barryi tried changing that to01:08
barryer, no leading slash01:08
barryusr/share/polkit-101:08
barryand dh_install --fail-missing complained01:08
slangasekthat was "/usr/share/PolicyKit" on a line by itself?01:09
barryslangasek: usr/share/PolicyKit on a line by itself01:09
slangasekbarry: right, that's a magic rule meaning "copy debian/tmp/usr/share/PolicyKit to a tree of the same name in the package"01:11
slangasekbarry: so you need to do usr/share/PolicyKit/* usr/share/polkit-1 instead if you want dh_install to magic this away for you, I think01:11
barryslangasek: how did computerjanitord/data/com.ubuntu.computerjanitor.policy get into usr/share/PolicyKit in the first place?01:12
slangaseksec, let me pull the package to see01:12
barryslangasek: oh wait.  i bet it's the setup.py that did it01:13
slangasekcould be01:13
barryslangasek: cool, that fits together then.  let me try that.  thanks for the help!01:14
slangaseksure thing01:14
slangasek(fwiw, the package I grabbed doesn't seem to do what you're talking about, oops :)01:14
barryslangasek: it would be lp:~barry/computer-janitor/dbus-merged01:15
barryslangasek: ^^ is a new version of c-j i'm trying to get landed01:16
slangasekok01:16
barryslangasek: yep, that was it.  thanks again01:18
Keybukbryceh: still around?02:07
brycehyeah, but a bit busy, what's up?02:10
Keybukbryceh: random question for you02:14
KeybukX multi-head02:14
Keybuklaptop has widescreen netbookish LVDS02:15
Keybukplug into 4:3 external display02:15
KeybukX picks a very random resolution to clone both together02:15
Keybuk800x600 @ 60 hz or something02:15
Keybukit's not even the highest common resolution they both support02:15
Keybukwhy?02:15
brycehbrain damage02:15
brycehusually the Xorg.0.log will list out the logic as to why it arrives at a particular resolution arrangement02:16
ionSame here. The laptop has a 1280×800 display and it’s connected to a 1920×1200 monitor. X picks something like 800×600 by default.02:16
Keybukbryceh: xrandr --auto doesn't fix this either02:16
KeybukI have to --preferred each display02:16
brycehwith cloning, it seems like there's three different ways it could be done, and no one of the three will cover all possible use cases02:17
brycehupstream seems to have picked an approach that suits the smallest of the three use cases.02:17
Keybukwhy does it clone by default at all?02:17
KeybukI guess that's what confused me02:18
ions/800×600/1024×768/ actually02:18
brycehI think it's for the projector use case, and assuming people will want to see everything on both screens02:18
brycehso it tries to find the lowest common matching resolution02:19
brycehwith 4:3 and an HD monitor, there really isn't a good common denominator02:19
Keybukit's kinda annoying02:19
Keybukyou see02:19
brycehI guess way back when, laptops also had 4:3 so that worke02:19
brycehd02:19
Keybukwe put some amount of effort to support multi-head with plymouth02:19
Keybukso even if you have a monitor02:19
Keybukthe splash screen looks sexy on both the panel and the monitor02:19
brycehnow days it generally falls all the way down to one of the vesa modes02:19
Keybuknot stretches02:19
Keybukall text on both02:20
Keybuketc.02:20
Keybukand that looks nice02:20
Keybukthen gdm starts02:20
Keybukand you end up in the resolution from hell02:20
Keybukand the smooth transition doesn't look so slick after all ;)02:20
* bryceh nods02:20
brycehbrain damage, like I said02:21
jmghi all02:25
jmgwho knows about mono02:25
RAOFA number of people.  They're more likely to answer if you ask an actual question, though.02:27
jmgok02:27
jmglooking for a ppa of mono2.602:27
jmgspecifically libapache2-mod-mono 2.602:27
RAOF...as long as that question is on-topic for #ubuntu-devel :)02:27
jmgotherwise the status of mono 2.6 in ubuntu, and roadblocks if any02:28
RAOFStatus of mono 2.6 in Ubuntu Lucid: 2.6 is not a long term support release, 2.4 is.  We're sticking with 2.4.02:28
jmgffuuuu02:29
jmghmmm02:30
RAOFLaunchpad's pretty good at searching in PPAs now; just browse to the mono source package, and it'll list PPAs which build mono.02:30
jmgok thanks02:30
=== plars` is now known as plars
dholbachgood morning07:32
pittiGood morning07:33
dholbachhey pitti07:33
=== crypt-0 is now known as test
=== test is now known as crypt-0
=== tkamppeter_ is now known as tkamppeter
pittiKeybuk: shall I commit your gdm changes to bzr, or did you do already and just need to push?07:48
tkamppeterpitti, mvo, hi08:18
loolkees: I don't think you need the rootfs anymore, but in case you do http://people.canonical.com/~lool/rootfs.img08:27
PetrolCBHi, I'm really looking forward to Beta 1... Is the exact time of release known? I know it's tomorrow, but when exactly does it become available?08:57
persiaPetrolCB: There's no precise time : sometime after the images are tested and confirmed beta-quality.09:07
PetrolCBok, thanks09:07
persiaPetrolCB: If you'd like to help out with testing, you may be able to help make it be sooner.09:07
PetrolCBI've subscribed to ubuntu-announce so I guess I'll get an email not too long after it's been released09:08
PetrolCBsure, how can I help?09:08
persiaHead over to #ubuntu-testing where the coordination happens, and someone will surely help you get started.09:09
tseliotpitti: shall I upload nvidia too (in addition to fglrx) since they won't be approved anyway?09:12
pittitseliot: go ahead09:13
tseliotpitti: ok, thanks09:14
tkamppeterpitti, hi09:15
pittihi tkamppeter09:15
tkamppeterI have tried around with the dbus problem, and tried to use a session bus service.09:17
pittitkamppeter: what is the problem again?09:17
tkamppeterThe problem is that if a printer is connected and detected by HPLIP and it needs firmware, the udev script (running as root) sends a dbus message so that a program of the user's desktop can catch the signal and run a firmware download GUI (running as the user).09:20
pittitkamppeter: ah, right; was that added to update-notifier?09:20
tkamppeterThere was a suggestion to use ubuntu-notifier as the listener and mvo has written a piece of code to do the task: http://pastebin.ubuntu.com/396286/09:21
PetrolCBthanks, persia09:21
tkamppeterWith this code ubuntu-notifier listens for a NeedPlugin signal on com.hp.hplip which root sends via09:22
tkamppeterdbus-send --system --dest=com.hp.hplip --type=signal /com/hp/hplip com.hp.hplip.NeedPlugin09:22
pittihmm09:22
pittithis adds a listener to the system bus09:23
pittimvo: ^ wouldn't that collide if you have more than one session?09:23
tkamppeterProblem is probably here also that com.hp.hplip is already used by HPLIP for PolicyKit.09:23
mvopitti: the problem is that dbus-send does not really gives the right kind of signal, it need to come from the com.hp.hplip dbus server that is already there09:23
pittiyes, it should be com.ubuntu.UpdateNotifier09:23
mvo or update-notifier needs to add low level matchers (I ahve code for this but its ugly)09:24
tkamppeterMy suggestion is to add a service to the session bus.09:24
pittimvo: oh, nevermind me; I read it the wrong way round09:24
pittimvo: u-m just connects to the existing hplip D-BUS service, right?09:24
pittiand reads signals from it09:24
mvoyes09:24
mvoso we should add the signal there IMO09:24
tkamppeterFor a quick test I edited /usr/share/hplip/hpssd.py adding a new method to com.hplip.StatuisService:09:24
pittitkamppeter: with your dbus-send, you are sending a signal *to* the hplip D-BUS service09:25
pittitkamppeter: what you really need is having the D-BUS service (the process for com.hp.hplip) send out the NeedPlugin signal09:26
tkamppeterhttp://pastebin.ubuntu.com/396603/09:26
tkamppeterName of the method is InstallPlugin09:26
tkamppeterProblem of this is that it works only if the message is sent by the user, not by root.09:27
pittitkamppeter: so, the way it's meant to be is to have the hplip daemon listen to udev events (with libudev), pick out the "needs firmware" thing, and send the signal from its d-bus service09:28
pittisending that from udev rules doesn't work, since there is no process/D-BUS service attached to an udev rule invocation09:28
pittiyou can't send signals in the name of another process09:29
tkamppeterpitti, but if an HPLIP daemonlistens, it could directly start the plugin-installer, without using update-notifier.09:29
pittitkamppeter: hplip runs as a root system daemon, though?09:29
pittiand I thought you need a GUI09:29
tkamppeterpitti, HPLIP does not have a root system daemon.09:30
tkamppeterpitti, if hplip-gui is installed, there is a session daemon running as the user, hp-systray.09:30
pittitkamppeter: now, that is the client side component which could pick up a signal, or udev event09:31
pittitkamppeter: i. e. this could use libudev to watch out for firmware events09:31
pitti(instead of the udev rule, dbus-send, and the other hackery)09:31
tkamppeterhp-systray can pop up a GUI program when it gets a dbus message.09:32
pittitkamppeter: but if hplip doesn't have a daemon, there's nothign to send the d-bus message from; also, it'd be the wrong way around09:33
tkamppeterpitti, you mean that the listener which runs as the user and pops up the plugin installer should listen for udev events and not for dbus messages?09:33
pittitkamppeter: I think hp-systray or u-m should just listen to the udev event09:33
pittiit's how the system is meant to work, and the most robust way without hacks09:33
pittitkamppeter: correct09:33
tkamppeterpitti, is a daemon needed to send dbus messages? dbus-send is not a daemon.09:34
pittitkamppeter: no, please drop the dbus-send stuff09:34
pittitkamppeter: you just need libudev (or python-gudev if it's python), and install a listener for the subsystem that the udev rule is currently listening to09:34
pitti"firmware"?09:34
pittior usb, or whatever it's listening to09:34
=== MacSlow is now known as MacSlow|errand
pittitkamppeter: is that C or Python?09:35
tkamppeterSo what I need would be a small Python program which is started for the user session (via /etc/xdg/autostart/*.desktop) and it listens for plug events of firmware-needing printers?09:36
pittinonononono, please no more additional python programs09:36
seb128yet another python process in the session?!09:37
pittitkamppeter: I thought there already was a hp-systray thing?09:37
pittitkamppeter: if there is none, we can use update-notifier09:37
pittiit already uses gudev and listens to events09:37
tkamppeterCan I make somehow make use of /lib/udev/rules.d/56-hpmud_support.rules09:37
pittiit's dead cheap to add it there, instead of the current dbus listener09:37
mvopitti++09:37
mvoI'm happy to add the code for that, its pretty trivial09:37
pittieither to u-n, or system-config-printer09:38
pittithe latter runs in the session anyway, too09:38
tkamppetermvo, can you chabge the src/hplip.c there, so that it listens to udev and not to dbus?09:38
mvotkamppeter: sure09:39
pittitkamppeter: the rule needs to be rewritten into C code, so you can use the logic of the rule09:39
tkamppetermvo, it is important that it works with both plugging the printer during the session and also if the printer got plugged before the session started.09:39
pittii. e. you listen for the correct subsystem (firmware/usb/whatever) and then check the attributes/properties of the udev_device you get in the event handler09:40
pittitkamppeter: for the latter you won't get an uevent; for that you need to iterate over all existing udev devices and pick out the existing ones09:40
pittibut one thing at a time09:40
tkamppeterImportant in the rule is the use of hp-mkuri, as this HPLIP utility determines whether the printer needs the plugin or not.09:40
tkamppeterpitti, mvo, in general, for the packaging it works well for now to have the listener in update-notifier, but for the future one needs to think about the following:09:45
tkamppeter1. system-config-printer is not a good place as upstream will never accept it. Fedora does not only have a "Do not ship proprietary software" policy, but a "Do not tell the user that proprietary software exists" policy.09:46
tkamppeter2. Is update-notifier GNOME-only? Or is it also available in Kubuntu?09:47
pittitkamppeter: 2. KDE has a counterpart, but it's different code09:47
pittiI'm off IRC for some minutes to debug a suspend bug09:48
tkamppeterpitti, mvo, so the code stub which we are adding to ubuntu-notifier now we better put into a separate C program which we ship with HPLIP09:49
tkamppeterpitti, mvo: This would also be better for HPLIP upstream/09:49
tkamppeterpitti, are you back?10:05
pittitkamppeter: yes10:06
tkamppeterpitti, mvo, so the code stub which we are adding to ubuntu-notifier now we should better put into a separate C program which we ship with HPLIP, so that also KDE users get it working and in addition we can submit the code to HPLIP upstream, so that all HPLIP users get their firmware easily installed.10:08
pittitkamppeter: can we put it into u-n for lucid? it's a much less intrusive change, avoids yet another daemon to start up, and more packaging changes10:09
tkamppeterpitti, OK, I will start separating it later.10:10
Riddelltkamppeter: what does the code do?10:11
tkamppeterRiddell, if udev discovers an HP printer which needs its firmware loaded whenever it gets turned on (only HP has such printers) then it should be popped up a GUI to download and install the proprietary firmware file.10:12
Riddelltkamppeter: ok that sounds non-trivial to get ported to Kubuntu but maybe send me an e-mail so I don't forget about it10:14
tkamppeterRiddell: HPLIP has a utility which is called by the udev rule and determines whether the firmware needs to be installed. For now only a message pops up, telling the user to run the firmware installer, but the firmware installer is not actually popped up.10:14
Riddellan e-mail pointing at where the code is in update-notifier once it's in10:14
tkamppeterpitti, mvo, Riddell: My idea of the separate code was that HPLIP gets added two files: The listener in C and an /etc/xdg/autostart/*.desktop file to start it and this way on all desktops the firmware installer pops up if needed.10:16
tkamppeterpitti, mvo, Riddell: The update-notifier solution does not add files, but it is GNOME-only, would also need an additional KDE implementation, and so is perhaps more code addition to Lucid.10:18
Riddelltkamppeter: if it's GUI code we would want it re-written for KDE preferably.  however there is a rewrite of the printer applet part of system-config-printer-kde happening currently so it might make sense to add it to that project10:18
tkamppeterRiddell, the listener has no GUI at all. It is simply a process running as the desktop user, listening for udev messages and starting a program named hp-plugin-ubuntu (this program ships with HPLIP).10:20
tkamppeterRiddell, so the listener is exactly the same for KDE and GNOME.10:21
Riddelltkamppeter: but there must be a GUI part too no?10:22
tkamppeterRiddell, we should also avoid to put any code for the proprietary plugin of HPLIP into s-c-p, as s-c-p upstream will not accept it.10:22
tkamppeterRiddell, the GUI part is in hp-plugin-ubuntu.\10:22
Riddellright10:22
tkamppeterRiddell, HPLIP itself has only Qt-based GUIs, so hp-plugin-ubuntu checks whether Qt is there and if so starts the native HPLIP GUI for the plugin download, if no Qt is there it starts the interactive text mode interface for the plugin download in a terminal.10:24
Riddelltkamppeter: so where does update-notifier come into it?10:25
tkamppeterRiddell, update-notifier was only used to make use of an already existing user-session daemon. The idea was not to introduce another one.10:47
Riddelldyfet: do we care about your plasma-widget-message-indicator upload for beta 1?11:03
persiaRiddell: You're probably best poised to answer that: it's an rdepend of kubnutu-desktop and kubuntu-netbook.  Currently it's stuck at 0.5.1 due to FTBFS, the upload would get 0.5.2.  You uploaded 0.5.2 on 10 march for some reason :)11:10
=== jussi01 is now known as o1
=== o1 is now known as jussi01
Riddellpersia: fair point :)11:17
Riddellev: on bug 534473 what's a Cruzer Micro?  I get that bug on my normal CD installs11:18
ubottuLaunchpad bug 534473 in ubiquity "Wrong CD device used when installed from a Cruzer Micro." [High,Triaged] https://launchpad.net/bugs/53447311:18
evRiddell: it's a usb disk.  I think they're the same bug, regardless.11:18
Riddellev: seems quite a nasty bug to have in for beta, means OEM install can't work.  I guess I can test in a virtual machine11:20
=== Klanticus_ is now known as Klanticus
evindeed, I hadn't realized it affected regular CD installs until you just pointed it out11:21
evhrm11:21
evmaybe this isn't the same bug after all11:21
evRiddell: did you have more than one CD inserted?11:21
Riddellev: no only 1 CD11:22
evyikes, okay11:22
evI'll undupe your bug then11:22
Riddellhappens on both of my machines11:23
cjwatsonmaybe we should go back to bind-mounting /cdrom (though perhaps onto /media/apt or something) if we have it rather than letting apt-cdrom use libudev for that bit11:24
cjwatsonRiddell: FWIW I'm pretty sure it doesn't happen everywhere11:24
cjwatsonit's not clear exactly what's going on - I suspect that apt's libudev integration might be subtly wrong and doesn't catch all cases11:24
cjwatsonI'll do a test to double-check, but I'm pretty sure that that part of the installation works OK in kvm at least11:25
cjwatsonin which case I think we could release-note it for now11:25
Riddellright, I don't get the problem in a virtual machine11:25
cjwatsonI bet it depends on the exact type of CD drive11:26
cjwatsone.g. whether it's also DVD-capable11:26
evI looked at the code last week and thought the parameters we set in apt cause it to not go down the libudev path, but I could be very wrong.11:26
cjwatsonI think maybe in base-installer but not in ubiquity11:27
cjwatsonwhich is probably an oversight!11:27
cjwatsonlist-devices (in debian-installer-utils) certainly checks several more cases than apt-cdrom does11:28
cjwatsonis there 'udevadm info -q all' output for one of these devices around somewhere?11:28
cjwatson('udevadm info -q all -n /dev/blah')11:29
evno, but I did get a udisks dump: http://launchpadlibrarian.net/40499554/udisks.txt11:32
cjwatsonsdc?11:32
evcorrect11:33
evwith sr0 being the fake cdrom for windows drivers11:33
cjwatsonnot clear from that though, or else I'm not smart enough to extract it11:33
tkamppetermvo. pitti, can you do the code change in update-notifier to pop up the plugin installer ("hp-plugin-ubuntu", not "hplip-plugin-ubuntu"), and tell me when you are ready, then I will do the HPLIP changes.11:48
=== MacSlow|errand is now known as MacSlow
mvotkamppeter: for the dbus stuff or does that need to be converted to udev ?12:40
tkamppetermvo, this must be converted to UDEV as I understand from our discussion.13:45
tkamppetermvo, if needed, I can add something to the UDEV rules file, so that these printers are somehow distinguishable from ordinary printers in the list of UDEV-reported devices.13:46
mvotkamppeter: if you could do the udev rule part, that would be nice, my experience there is limited13:49
tkamppeterpitti, can you help on the UDEV part in the C code of update-notifier?13:50
pittitkamppeter: yes, but not today/tomorrow (hands full with beta stuff)13:50
tkamppeterpitti, mvo, so let us move it out for after the beta release.13:51
mvotkamppeter: the udev C bits are no problem, the udev rule file is what I do not have a lot of experience iwth13:55
tkamppetermvo, the meaning is the following:13:57
pittitkamppeter: can you paste it somewhere? (I don't have it here); mvo, I'm happy to interpret it/help with tanslating to gudev13:58
tkamppeterWhen a USB device is added and it has vendor ID 0x03f0 and product IS 0xXX17 (XXX are two arbitrary hex digits) then the command line is executed. The com,mand line sets the environment variable hp_model to the model ID (text vertsion) of the USB device and then runs "hp-mkuri -c" in the background.14:01
Keybukpitti: I did push!14:01
Keybukthat branch is owned by the importer anyway - so if I hadn't pushed, it would have shown up14:01
pittiKeybuk: ah, -EWRONGBRANCH then14:02
pittiKeybuk: I'll commit it, don't worry14:02
Keybukpitti: ? I used lp:ubuntu/gdm14:03
pittiVcs-Bzr: https://code.launchpad.net/~ubuntu-desktop/gdm/ubuntu14:03
tkamppeter"hp-mkuri -c" checks whether the printer whose model name is suppied via the hp_model variable needs firmware and whether the firmware is already installed. IF and only if the printer needs firmware and the firmware is NOT installed, it generates a desktop notification and also sends a message to the system (currently via dbus, but I can change it to any other type of message.14:03
tkamppetermvo,  ^^^14:03
Keybukpitti: ah, I missed that14:03
Keybukpitti: still using debian-only packaging then?14:03
pittiyes, for now; we'd loose all the history14:04
pitti(and would be stuck with a much worse performance)14:04
Keybukpitti: of course, the nice thing is you can cherry-pick the change from lp:ubuntu/gdm no?14:05
tkamppeterpitti, mvo, the udev-rules file is /lib/udev/rules.d/56-hpmud_support.rules and is part of the hplip package. Here it is pasted for all the Samsung printer users who have uninstalled HPLIP: http://pastebin.ubuntu.com/396702/14:06
pittiKeybuk: I grabbed the diff from LP and committed it; all good now14:06
Keybukpitti: I've never figured out how to deal with these separated debian branches14:06
KeybukI can never get to the point I can build from them14:06
pittiKeybuk: bzr-builddeb is awesome for those14:07
seb128Keybuk, bzr-buildpackage14:07
Keybukstill doesn't give me the source14:07
seb128one command, gets the tarball and do everything for you too14:07
seb128it does14:07
pittiit auto-downloads the orig.tar.gz, and everything14:07
seb128it gets the archive one or use the watch for new versions14:07
pittiKeybuk: so usually you'd do "bzr bd-do", hack the (now full source) tree, and if you exit the shell it copies back the changes to debian/14:08
pittiKeybuk: bzr bd -S -> build source, bzr bd -- -b -> build binaries, etc.14:08
seb128bzr bd-do and edit-patch usually14:08
KeybukI did use edit-patch the other day14:08
Keybukkudos to mvo14:08
pittimvo: so basically, you use http://www.kernel.org/pub/linux/utils/kernel/hotplug/gudev/GUdevClient.html#g-udev-client-new to listen to "usb" subsystem events (what the udev rule sya)14:09
mvopitti: ok, adding the code now14:10
mvoKeybuk: :)14:10
pittimvo: and whenever you get an add or change event, you use http://www.kernel.org/pub/linux/utils/kernel/hotplug/gudev/GUdevDevice.html#g-udev-device-get-sysfs-attr to check idVendor/idProduct14:10
pittimvo: you can just copy & paste the general listener stuff from the jockey firmware code14:11
pittimvo: or, even easier, just add it to that event handler, so that we don't need two listeners :)14:11
pittiit's firmware either way, after all14:11
mvopitti: good point14:11
mvopitti: should I test for idVendor, idProduct or should it rather add a PROPERTY to look for in the udev rule file ?14:14
mvopitti: and I test on that property then?14:14
pittimvo: we should just drop the udev rule14:17
pittimvo: just check the attribute, that's fine14:17
mvoaha, ok14:17
pittino need to needlessly add more stuff into the chain14:17
=== sconklin-gone is now known as sconklin
mvotkamppeter, pitti: what is %E{ID_MODEL} ? sysfsattr, property? something else?14:35
pittimvo: property14:35
mvothanks14:35
pittimvo: http://www.kernel.org/pub/linux/utils/kernel/hotplug/gudev/GUdevDevice.html#g-udev-device-get-property14:36
pittimvo: but I wonder whether hp-mkuri is actually the program that we have to call? tkamppeter?14:36
mvothanks, I figured that :)14:36
mvotkamppeter: do I need to call hp-mkuri -c to check if the firmware is needed? or can I just unconditionally call the hp-plugin-ubuntu helper?14:37
jcastromvo: don't forget to respond to debian squid guy at some point!14:39
mvojcastro: weeh, right14:39
mvotkamppeter: I commited the changes to update-notifier now, it currently calls hp-mkuri -c, if that is unneeded, just let me know and I remove it14:52
mvotkamppeter: get it with lp:update-notifier and build with bzr-buildpackage to try it out14:52
tkamppetermvo, got it, thank you. I am building it now.15:24
mvotkamppeter: thanks, let me know about hp-mkuri -c (but feel free to just modify the code yourself)15:27
kobrienhi, can someone direct me to a resource describing conventions in coding C for ubuntu if there are any?15:43
Keybukkobrien: there aren't any15:45
Keybukif you want to read some conventions, a good idea is to download a copy of the GNU Standards, print them out, and then BURN THEM!15:46
Keybuk(actually, they're probably as good a guide as any - at least to building.  ie. your project should use autoconf + automake, etc.)15:46
=== highvolt1ge is now known as highvoltage
evshtylman, Riddell: any idea why the kde frontend of ubiquity has started to consume 100% cpu (bug 538505)?  It's not happening in the GTK+ frontend, and my attempts so far at cracking this have proven unsuccessful.15:55
ubottuLaunchpad bug 538505 in ubiquity "Extremely slow reponsiveness / high CPU usage" [High,Confirmed] https://launchpad.net/bugs/53850515:55
Riddellev: I've not seen that issue (and I've done quite a few installs today)15:57
evRiddell: odd, I can reproduce it in KVM and on real hardware15:57
Riddellev: I'm more worried about https://launchpad.net/bugs/540275 currently15:58
ubottuUbuntu bug 540275 in ubiquity "Installing in French breaks the keyboard setup page" [Undecided,New]15:58
evgah16:01
=== binarylooks is now known as binarylooks_away
Riddellev: it's a shtylman_ bug from his lovely keyboard page, something is being translated which shouldn't be, I'll take a look after I've had some lunch16:02
* ev shakes his fist at shtylman ;)16:02
evRiddell: cool, thanks16:03
Riddellfish shaking is the wrong approach I'm sure, intravenous irn bru would work much better16:03
Riddellumm, fist shaking :)16:03
shtylman_ev Riddell :)16:05
shtylman_life is boring without bugs16:06
shtylman_ev: I will test the 100% cpu issue tonight.. ive never seen 100% cpu .. but do find parts of the installer slow... so maybe there is some race or other crazyness happening16:07
tkamppetermvo, I am currently fine-tuning your code by testing it with the actual printer (and also testing with HP printers which do not need firmware).16:14
=== beuno is now known as beuno-lunch
tseliotRiddell: do you know where debug() (e.g. server.c) in kdm writes to?16:17
tselioti.e. to what log file16:17
Riddelltseliot: /var/log/kdm.log ?16:17
tseliotRiddell: ok, thanks16:18
mvotkamppeter: sweet, once its ready let me know and I merge/upload16:19
mvotkamppeter: thanks!16:19
=== yofel_ is now known as yofel
=== binarylooks_away is now known as binarylooks
tkamppetermvo, first bug: in on_uevent() it must read16:28
tkamppeterif (g_strcmp0 (action, "add") != 0 && g_strcmp0 (action, "change") != 0)16:28
tkamppeter(&& and not ||)16:28
tkamppetermvo, after having corrected that, the firmware dialog actually popped up, but for all HP LaserJet and HP Color LaserJet printers, not only for my LaserJet 1020.16:29
kobrienKeybuk: cheers, I use the gnu standards as it is. cool16:30
mvotkamppeter: aha, sorry. I think its not properly checking the return code of hp-mkuri, let me fix that16:30
tkamppetermvo, then I checked your use of hp-mkuri and saw that you do not check the exit status. See "hp-mkuri -h". Firmware installation should only happen if the exit status is 2 or 5.16:31
mvotkamppeter: ok, fixing ths now16:31
tkamppetermvo, I added if (ret != 2 && ret != 5) return TRUE; before calling the firmware installer.16:31
tkamppetermvo, In addition, I added more g_debug lines and this way I saw that after the hp-mkuri call ret was set to 512. Is ret not the exit status?16:33
mvotkamppeter: it needs to be converted via WEXITSTATUS()16:33
tkamppetermvo: 512/256 = 2, is the conversion shifting ret by 8 bits to the right?16:34
mvotkamppeter: man waitpid has the info, its available via the macro WEXITSTATUS()16:36
tkamppetermvo, thanks, trying WEXITSTATUS(ret) ...16:36
mvotkamppeter: or run bzr up16:37
mvotkamppeter: I added your fixes to my branch16:37
mvo(including this one)16:37
keeslool: cool, thanks.  if anything goes really weird in mind rootfs, I can compare it to yours.16:45
cjwatsonKeybuk: so, do we know that it's definitely on 'plymouth deactivate', rather than in ply_terminal_close?16:51
Keybuksorry, not following your question16:51
cjwatsonI'm grepping for paths that call ply_terminal_set_buffered_input16:51
cjwatsonthere seem to be two, and I was wondering if we'd confirmed which one16:52
Keybukset_buffered_input should be guarded16:52
Keybukthe one in ply_terminal_close() should return without doing anything because it's called in the deactivate path16:52
Keybukwhat has happened before is that we ended up with a ply_terminal_set_unbuffered_input() between those two16:52
Keybukso ply_terminal_close() set buffered again16:52
Keybukthat was caused, amongst other things, by the fact every damned terminal write did it <g>16:53
tkamppetermvo, it is working now, do you need a patch?16:53
tseliotcjwatson: any ideas as to what can be causing this code to exit with return code 10 ? http://pastebin.ubuntu.com/396800/16:55
cjwatsonI'm not following where the guard is16:55
cjwatsonbut if this is boring, feel free to tell me and I'll stop pursuing it16:55
cjwatsontseliot: have you used DEBCONF_DEBUG=developer?16:55
mvotkamppeter: plesae check if trunk/ works correctly, if it does we are good, otherwise I need the patch16:55
mvotkamppeter: I fixed the bits you mentioned16:55
tseliotcjwatson: well, I can't reproduce the problem here. A user reproduced it when he was updating the kernel16:56
cjwatsonKeybuk: I'm finally getting around to remastering the CD to strace plymouth; is that still a useful thing for me to do, or are you already past it?16:56
cjwatsontseliot: best ask them for that information16:56
tkamppetermv, I see you have already committed the correct version. Can you upload the package? I will update the HPLIP package.16:56
tseliotok, thanks16:56
cjwatsontseliot: 10 can mean various things, I don't see anything blindingly obvious16:56
tseliotme neither16:57
cjwatsontseliot: it means "bad parameters" in general16:57
Keybukcjwatson: I have a CD with a plymouth installed that has -g -O0 no stripped and the source code in the right place :p16:57
cjwatsonit's like EINVAL16:57
KeybukI'm hoping this will help16:57
cjwatsonbut, it typically comes with more information in the debconf debug output16:58
=== beuno-lunch is now known as beuno
tseliotcjwatson: ok, there must be something else that triggers this (as DKMS used to do). I'll ask the user to try again with that variable16:58
tseliotthanks16:58
tkamppetermvo, stop, as expected, it does not work, I will tell you what needs to be changed ...16:58
mvotkamppeter: ok16:59
tseliotcjwatson: it looks like that we have that kind of information already: http://pastebin.ubuntu.com/396803/17:01
tseliotwell, I don't know if it's the same as debconf debug17:01
cjwatsontseliot: that'll do, and there you go, it tells you the problem17:02
cjwatsonDEBCONF_DEBUG=developer is less noisy for the same effect, but17:02
cjwatson+ RET='10 nvidia-common/obsolete doesn'\''t exist'17:02
tkamppetermvo, sorry, when I updated yourt code got mixed up with my code. Now I have reloaded your code and it works. Upload the package exactly as you have it in your repo.17:02
dmartDoes anyone know how to enable debug on a network interface in Ubuntu?17:03
mvotkamppeter: thanks for the fixes and the tests, I will do the upload now17:03
tkamppetermvo, does you new code also cover cold-plugging (printer is already connected and turned on before update-notifier starts)?17:03
tseliotcjwatson: yes, I noticed that but I don't understand why it's happening17:03
dmartBSD's ifconfig has a "debug" option, but Ubuntu's ifconfig doesn't seem to support this.  Currently I needed to write my own tool to set IFF_DEBUG on an interface...17:04
mvotkamppeter: no, not yet, shall we wait on this? I guess so, I can work on this after dinner or tomorrow17:04
zygamvo: I want to start filing bugs on each ubuntu package that is using the update-alternatives comment in any script17:05
cjwatsontseliot: well, it just means that it isn't in the template database, which probably means the package that provides that template isn't installed yet (or is doing something mad)17:05
zygamvo: I want to make a blueprint on adding more metadata to debian packages to indicate all possible update-alternatives that a package install script might perform17:05
zygamvo: does this sound insane or should I go ahead with this project?17:05
keesslangasek: as samba maintainer, what do you think of upstream's recent change: http://gitweb.samba.org/?p=samba.git;a=commitdiff;h=bd269443e311d96ef495a9db47d1b95eb83bb8f417:05
keesslangasek: if we backport this, wide links would be disabled by default, which could break poeple using them.17:06
keesslangasek: but it closes the CVE-2010-0926 hole17:06
ubottuThe default configuration of smbd in Samba before 3.3.11, 3.4.x before 3.4.6, and 3.5.x before 3.5.0rc3, when a writable share exists, allows remote authenticated users to leverage a directory traversal vulnerability, and access arbitrary files, by using the symlink command in smbclient to create a symlink containing .. (dot dot) sequences, related to the combination of the unix extensions and wide links options. (http://cve.mitre.org/cgi-bin/cvena17:06
Riddellev, shtylman_: I do see CPU at 90% now I look at top while ubiquity is running :(17:06
shtylman_Riddell: it pains me to hear these things :(17:06
tseliotcjwatson: I think nvidia-common was already installed (and I call debconf in the postinst anyway) as in this case its postinst was called by a kernel hook17:07
tseliotmaybe something removed the template?17:07
tseliotBTW it's bug #53397017:08
ubottuLaunchpad bug 533970 in linux "package linux-image-2.6.31-20-generic 2.6.31-20.57 failed to install/upgrade: run-parts: /etc/kernel/postinst.d/nvidia-common exited with return code 10" [Undecided,Confirmed] https://launchpad.net/bugs/53397017:08
=== RoAk is now known as RoAkSoAx
cjwatsontseliot: I don't know, you're probably better-equipped to take it from here than I am TBH as you know those packages better17:08
cjwatsondebconf is just reporting the condition17:08
slangasekkees: omitting the loadparm.c change would at least mitigate the user impact; widelinks would still be disabled by default *because* unix extensions are on by default, but anyone who's turned off unix extensions will have widelinks ok17:09
tseliotcjwatson: ok, thanks17:09
cjwatsonsorry17:09
slangasekkees: fwiw, as maintainer my answer to the bug reporter was "fixed in unstable, not screwing around with stable for this unless the security team asks me to"17:09
mdeslaurslangasek, kees: that's not the commit I was looking at, hold on17:09
keesmdeslaur: oh? that's what was tied to the CVE17:10
mdeslauroh, yes it was17:10
mdeslaurplease ignore me, like usual :)17:10
mvozyga: sounds good, ideally we would make it a declarative think and not part of maintainer scripts. but metadata is a start17:11
mdeslaurslangasek, kees: it was further improved with http://gitweb.samba.org/?p=samba.git;a=commitdiff;h=16e73d88944ce644cccfa19a99338f5903c061f017:11
zygamvo: not part of the scripts, part of control file17:11
zygalike x-provides-alternavies: xxx, yyy17:11
keesslangasek: but that's just a small subset.  the people we're worried about are those using both wide links and unix extensions.17:11
slangasekkees: well, I don't think most people are *using* wide links, I think they just unwittingly have them enabled17:13
slangasekso I'm not sure how bad the impact on users would really be17:13
slangasekbut as I said, I'm not pushing to make this change in lenny17:13
* kees nods17:13
mdeslaurslangasek: so, you're okay with everyone inadvertently sharing the whole filesystem in lenny?17:14
keesslangasek: the problem is that it allows (DAC-limited) arbitrary file read from the server if a user has the ability to create symlinks17:14
mdeslaurthere's a cool metasploit module for it :)17:14
cjwatsonw/wg 4717:15
cjwatsondok17:15
zygamvo: great, I assume I have your backing on this, I'll patch the harvester to support this and add this as a proof of concept to selected package, if it wokrs I'll draft a blueprint for review17:16
slangasekmdeslaur: there are no public shares by default, users are discouraged from running Internet-facing samba servers; and as kees says it's DAC-limited.  Vs. suddenly breaking things for users who are intentionally using wide links, yeah, I'm ok with that17:16
slangasekexpecting your authenticated users to not be able to read world-readable files on the server just because they're not shared via SMB is right up there with expecting PHP safe mode to work, I think17:18
mdeslaurslangasek: well, by "everyone", I meant "everyone who set up a corporate file server" :)17:18
tseliotcjwatson: just to double check, the template should be /var/lib/dpkg/info/$package_name.templates,  right? (it's /var/lib/dpkg/info/nvidia-common.templates in my case)17:19
mdeslaurslangasek: huh? I would assume most people who are running a samba file server don't let their users log into the box...17:19
keesslangasek: I think samba shares are held to a higher standard that php safe mode.  :P17:19
cjwatsontseliot: yes, though something needs to arrange to load that; see debconf-devel(7)17:19
cjwatsontseliot: does nvidia-common ship a config file?17:20
tseliotcjwatson: no, it doesn't anymore17:20
slangasekmdeslaur, kees: my point is that I don't regard it as privilege escalation.  But you guys do what you think is best.:)17:20
keesslangasek: okay, cool, thanks.17:20
mdeslaurslangasek: ok, our samba update will disable php, thanks for your input :)17:21
tseliotcjwatson: I got rid of the config file and moved the logic into the postinst as you suggested: http://bazaar.launchpad.net/~ubuntu-branches/ubuntu/lucid/nvidia-common/lucid/revision/2317:23
cjwatsontseliot: hm, right, so it's possible the templates file hasn't been loaded yet if the template is being used from places other than the postinst file, as you indicate17:24
tseliotoh17:24
cjwatsontseliot: I think the best option would be to add a config script that is simply as follows:17:24
cjwatson#! /bin/sh17:24
cjwatson. /usr/share/debconf/confmodule17:24
cjwatsonexit 017:24
cjwatsonthat will cause debconf to load that templates file up-front17:25
cjwatsonmight want a comment as well to explain why it exists :-)17:25
cjwatsonthis is a bit of a gotcha, it's bitten me as well in the past.  the way debconf arranges to load templates at the right time is known-hackish17:25
tseliotcjwatson: sure, I'll do that, thanks17:25
mathiazjelmer: hi - I was trying to run bzr update on a remote git repository: http://paste.ubuntu.com/396819/17:30
mathiazjelmer: ^^ is this a known bug?17:30
jelmermathiaz: Hi17:31
jelmermathiaz: Yep, it's a known bug - seems to be caused by a newer version of python that's now in Lucid17:31
cjwatsonKeybuk: I have the strace if it's any use, though I suspect it isn't17:31
jelmermathiaz: It's been fixed in bzr-git trunk, but I haven't had the time to upload new packages to Debian/Ubuntu yet (at a sprint atm)17:31
mathiazjelmer: great than - is there a workaround for now (or a bug number)?17:31
mathiazjelmer: ok - I'll look at the bzr-git trunk and fix it locally then17:32
Keybukcjwatson: please17:32
Keybukbecause right now, I'm about to say "it's not plymouth"17:32
cjwatsonI can see a few TCSETS at the end17:33
mathiazjelmer: is the fix revision 743?17:33
jelmermathiaz: yeah, sorry about the uninformative commit message17:33
cjwatsonstrace doesn't tell me details17:33
cjwatsonKeybuk: attached to bug 54025617:33
ubottuLaunchpad bug 540256 in plymouth "enter kills X when booting with text plugin" [High,Confirmed] https://launchpad.net/bugs/54025617:33
Keybukhuh17:34
Keybukok, well17:34
KeybukI just traced every line of plymouth17:34
Keybukand it did not fail17:34
Keybukbut enter doesn't kill X either17:34
cjwatsonoh, strace does tell me, I'm just being dim17:35
cjwatson273   17:29:05.425303 ioctl(8, SNDCTL_TMR_CONTINUE or TCSETSF, {B38400 opost isig icanon echo ...}) = 017:35
cjwatsonKeybuk: there's a SIGWINCH terminal change right beforehand.  I have to say that this looks very much like it's in ply_terminal_close17:35
Keybukyes, but it can't be17:36
Keybukthere's a great big guard at the top17:36
KeybukSIGWINCH though => hmmmmm17:36
Keybukwhy would you have SIGWINCH?!17:36
Keybuklet me talk myself through the strace for a minute17:36
Keybuk273   17:28:25.803402 read(7, "D\0", 2) = 217:36
Keybuk^ received deactivate17:37
cjwatsonit's not receiving SIGWINCH, it's ditching its handler for it17:37
=== binarylooks is now known as binarylooks_away
Keybuk273   17:28:25.815653 rt_sigaction(SIGUSR1, {SIG_DFL, [USR1], SA_RESTART}, {0x6df710, [USR1], SA_RESTART}, 8) = 017:37
Keybuk273   17:28:25.815938 rt_sigaction(SIGUSR2, {SIG_DFL, [USR2], SA_RESTART}, {0x6df710, [USR2], SA_RESTART}, 8) = 017:37
Keybuk273   17:28:25.816149 ioctl(8, VIDIOC_ENUM_FMT or VT_SETMODE, 0xbfbfbe38) = 017:37
Keybukrestore VT back to VT_AUTO17:37
Keybuk(and stop watching the signals)17:37
Keybuk273   17:28:25.816299 ioctl(8, SNDCTL_TMR_TIMEBASE or TCGETS, {B38400 opost isig icanon echo ...}) = 017:37
Keybukput terminal back into Canonical Mode17:37
Keybuk273   17:28:25.816633 write(7, "\6", 1) = 117:38
Keybukdone17:38
Keybukright17:38
Keybukso that looksok17:38
cjwatsonand X starts at 17:28:26, according to its log file17:38
cjwatsonand will presumably switch back to raw mode17:38
Keybuk273   17:28:25.842325 read(7, "V\0", 2) = 217:38
Keybuk273   17:28:25.842451 write(7, "\6", 1) = 117:38
Keybukdo we have an active vt?  => yes17:39
Keybuklots of polling, times and an occasional update of the screen while we wait for X to start17:39
Keybukok17:40
Keybukno ioctl in that lot17:40
Keybuknow17:40
Keybuk273   17:29:05.412303 read(7, "Q\2", 2) = 217:40
Keybukthat's "quit"17:40
Keybukso X is up, gdm wants us to leave17:40
Keybuklooks like it's with --retain-splash given the \1\017:40
Keybuk273   17:29:05.412435 read(7, "\1\0", 2) = 217:40
Keybuk273   17:29:05.424512 ioctl(8, PIO_CMAP, 0x82a88f0) = 017:41
Keybukreset the colour map17:41
Keybuk(so vts don't stay purple forever)17:41
Keybuk273   17:29:05.425021 rt_sigaction(SIGWINCH, {SIG_DFL, [WINCH], SA_RESTART}, {0x6df710, [WINCH], SA_RESTART}, 8) = 017:41
Keybuk273   17:29:05.425141 ioctl(8, SNDCTL_TMR_TIMEBASE or TCGETS, {B38400 -opost -isig -icanon -echo ...}) = 017:41
Keybuk273   17:29:05.425227 ioctl(8, SNDCTL_TMR_TIMEBASE or TCGETS, {B38400 -opost -isig -icanon -echo ...}) = 017:41
Keybuk273   17:29:05.425303 ioctl(8, SNDCTL_TMR_CONTINUE or TCSETSF, {B38400 opost isig icanon echo ...}) = 017:41
Keybuk273   17:29:05.425394 ioctl(8, SNDCTL_TMR_TIMEBASE or TCGETS, {B38400 opost isig icanon echo ...}) = 017:41
Keybukand17:41
Keybuklike you say17:41
Keybukrestore SIGWINCH17:41
cjwatson(I'm rebuilding plymouthd with --enable-tracing)17:41
Keybukthen fuck around with the console mode17:41
Keybukand17:42
Keybuklike you say17:42
Keybukthis must be ply_terminal_close17:42
Keybukwe did each of those steps17:42
Keybukwe restored the colour palette17:42
Keybukwe removed the epoll watch on the tty fd17:43
Keybukwe removed the signal for SIGWINCH17:43
Keybukthen ply_terminal_set_buffered_input () gets called17:43
Keybukthe first ioctl is TCGETS17:43
Keybukso that's the "get current terminal attributes"17:43
cjwatsonthe guard at the top of ply_terminal_close is "has ply_terminal_close already been called?"17:43
Keybukit doesn't have ICANON17:43
Keybukso we don't return17:43
cjwatsonand AFAICS deactivate doesn't close the terminal?17:44
cjwatsonquit_splash, OTOH, does17:44
Keybukthen we restore terminal settings17:44
Keybukso right17:44
Keybukthis looks like we went through ply_terminal_set_buffered_input17:44
Keybukand reset the terminal back to Canonical Mode because X reset it to Raw Mode17:44
Keybuk--17:44
Keybukno17:44
Keybukthe guard at the top of ply_terminal_set_buffered_input17:45
Keybuk   if (!terminal->is_unbuffered)17:45
Keybuk     return true;17:45
Keybukclearly that must be false17:45
Keybukis_unbuffered starts false17:45
Keybukit's set to true by ply_terminal_set_unbuffered_input17:45
cjwatsonis it definitely the same ply_terminal_t?17:45
Keybukand set to false by ply_terminal_set_buffered_input17:45
Keybukshould be state->terminal yeah17:46
Keybukohhhhh17:47
Keybukhmmm17:47
Keybuklook at the deactivate ioctl17:48
Keybuk273   17:28:25.816299 ioctl(8, SNDCTL_TMR_TIMEBASE or TCGETS, {B38400 opost isig icanon echo ...}) = 017:48
Keybuk...17:48
Keybukthe terminal was *already* ICANON17:48
Keybuksomething reset the terminal to Canonical Mode under plymouth!17:48
cjwatsoner, are we looking at the same one?17:48
Keybukscroll up17:48
cjwatsonoh I see what you mean17:49
Keybukso when deactivate was called17:49
Keybukwe went into ply_terminal_set_buffered_input17:49
Keybukthat did a tcgetattr (that ioctl)17:49
Keybukand was informed that the terminal was already in canonical mode17:49
Keybuknow look at the code ...17:49
Keybukset_buffered_input returns at that point17:49
Keybukand *never* resets terminal->is_unbuffered17:49
Keybukso when we call the same function in ply_terminal_close ... the guard is wrong17:50
cjwatsonright, so we could work around this by updating the state flag there, but we need to know what's fiddling with the console17:50
Keybukso that tries again harder to set the terminal into buffered mode17:50
Keybukand since X has reset it back to raw mode since then17:50
Keybukplymouth helpfully resets it back17:50
Keybukwe do yes17:50
cjwatsonconsole-setup might be the obvious culprit, but it's really careful to touch only tty[1-6]17:50
Keybuksince if the terminal is in Canonical Mode under plymouth - then input and stuff doesn't work17:50
cjwatson'zactly17:50
Keybukand this explains why, when I stopped and restarted plymouth17:51
KeybukI couldn't replicate the problem17:51
Keybukit obviously happens relatively early in the boot sequence17:51
cjwatsonjust a silly question, but17:52
cjwatsonit isn't upstart is it?17:52
cjwatsonthat would explain the difference between a boot with plymouth in the initramfs (live CD) and one without (most installed systems)17:52
KeybukI was just thinking the same thing17:52
cjwatsonsystem_setup_console sets ICANON on whatever /dev/console is, which would be the foreground console, right?17:52
Keybukyes17:52
Keybukso17:53
Keybukputting plymouth in the initramfs, and rebooting17:53
Keybukshould replicate this on any machine17:53
lucasIs it normal to wait almost a week for syncs to be processed? should I upload fake syncs instead when I need the package in ubuntu ASAP?17:55
cjwatsonlucas: during beta freeze, yes it is.  what's the bug?17:55
cjwatson+number17:55
lucas#53827117:55
cjwatsonit's in main, but not AFAICT on CDs17:56
* cjwatson greps just to make sure17:56
cjwatsonno, that's on the server CD17:57
cjwatsonwhich is currently final for beta-1 and isn't due a respin (yet ...)17:57
lucasok, I see17:58
cjwatsonlucas: can this wait until after beta-1?  it should only be a day or so now17:58
cjwatsonI can make sure to do it ASAP after that17:58
lucascjwatson: it can, it's just that I would like to get as much testing in ubuntu as possible17:58
cjwatsonor is there a strong reason why it should be specifically in beta-1?17:58
Keybukcjwatson: confirmed, setting FRAMEBUFFER=y on a normal machine does the same thing17:59
cjwatsonmathiaz: ^- (can't see ttx), what do you think?17:59
cjwatsonif we need to rev upstart, we might well need to respin everything - this bug would break cryptsetup on a server system, for example17:59
mathiazcjwatson: bug 538271?17:59
lucascjwatson: well, puppet apparently breaks with the current ruby1.8 version. it's probably not major enough to rebuild the server CD.17:59
ubottuLaunchpad bug 538271 in ruby1.8 "Sync ruby1.8 1.8.7.249-2 (main) from Debian unstable (main)" [Wishlist,New] https://launchpad.net/bugs/53827117:59
cjwatsonmathiaz: yes17:59
mathiazcjwatson: right - I don't think it's worth respinning an iso for beta118:00
cjwatsondo you have test cases for puppet?18:00
mathiazcjwatson: not yet18:00
cjwatsonperhaps we can address lucas' concerns by making sure that's explicitly and formally tested for beta-2?18:00
cjwatsonat least partially address, anyway18:00
mathiazcjwatson: yes - that's the plan18:00
lucascjwatson: well by testing I mean user testing.18:00
mathiazcjwatson: we're planning more QA testing on puppet later18:01
lucascjwatson: the original bug is a random+obscure hang18:01
mathiazcjwatson: as far as beta1 is concerned I don't it's worth a respin18:01
highvoltagelamont: hi, are you around?18:01
mathiazcjwatson: I don't *think* it's worth a respin18:02
cjwatsonmathiaz: how about the upstart/plymouth bug in scrollback above?18:02
cjwatsonwell, upstart's involvement is still conjectural but seems likely at this point18:02
mathiazlucas: does that seem reasonable to you (sync ruby after beta1 is released)?18:02
Keybukcjwatson: no, it is upstart18:02
KeybukI just commented out that code, and rebooted18:02
Keybuknow enter doesn't kill X18:02
cjwatsonexcellent18:03
lucasmathiaz: yes18:03
Keybukcjwatson: I'm just trying to work out why that code is even there18:03
cjwatsonlucas: sorry we didn't catch this earlier, it was probably just at the start of beta freeze18:03
KeybukI think it's a case of "sysvinit did it"18:03
mathiazcjwatson: bug https://launchpad.net/bugs/540256?18:04
ubottuUbuntu bug 540256 in plymouth "enter kills X when booting with text plugin" [High,Confirmed]18:04
cjwatsonKeybuk: mm, doesn't the kernel set sane defaults?  (or does it, I'm not sure I trust the vt code to do anything ...)18:04
lucascjwatson: np18:04
cjwatsonmathiaz: yes18:04
Keybukcjwatson: I'm just reviewing that now18:04
cjwatsonmathiaz: don't go solely by the bug summary though; 17:50 <Keybuk> since if the terminal is in Canonical Mode under plymouth - then input and stuff doesn't work18:04
KeybukCanonical Mode being Purple ;-)18:04
mathiazcjwatson: ok - let me catch up on the bug18:04
highvoltageheh, that explains why my X session was killed on the livecd when I typed 'sudo -s' (I didn't expect it to be the actual enter key that did it :) )18:05
cjwatsonmathiaz: I think the practical impact on server is that if you are using cryptsetup (and so the initramfs is built with FRAMEBUFFER=yes) then neither fsck nor cryptsetup will/may be able to accept input18:05
cjwatsonhighvoltage: right, that's roughly where we came in, it's just taken all day to narrow it down18:05
Keybukwell, it took all afternoon to narrow it down18:05
Keybukit took until then to wait for me to get out of bed :p18:05
cjwatsonI was working on it in the morning too, just less competently :-)18:06
mathiazcjwatson: so the impact is limited to cryptsetup server installation?18:06
cjwatsonmathiaz: I think so, I can't think of other situations where you get an initramfs with FRAMEBUFFER=yes on server?18:06
cjwatsonperhaps somebody can verify that from more reliable memory18:06
highvoltagewell, thanks to both of you for getting out of bed today :)18:07
Keybukthe two things that I know that set that are cryptsetup and casper18:07
cjwatsonI get rousted out of sleep at god-awful-o-clock by a small child18:07
cjwatsondoesn't matter when I went to bed18:07
highvoltageheh18:07
Keybukright18:07
KeybukI'm going to take a few minutes to collect state18:07
Keybuknot to mention update the bug18:08
cjwatsonI'm going to leave at this point, as I'm due elsewhere.  Please SMS / google-talk me if I need to log in and poke the publisher or whatever, and I can do that18:08
mathiazcjwatson: I think it's *not* worth a -server iso respin for the plymounth/upstrart bug18:09
dahudHow far do I need to get in my education before I can begin to understand and work on Ubuntu?  Are there some lower-level tasks to be done?18:09
mathiazcjwatson: given that it only affects cryptsetup installations18:09
Keybukcjwatson: I can do that ;)18:09
cjwatsonmathiaz: OK, that will probably help matters, thanks18:09
highvoltagedahud: reading https://wiki.ubuntu.com/ContributeToUbuntu and its sub pages will give you some idea18:10
dahudhighvoltage: Thanks18:10
highvoltagedahud: anyone can with enough work and patience18:10
cjwatsondahud: there's e.g. https://bugs.launchpad.net/ubuntu/+bugs?field.tag=bitesize - but personally my advice has always been to find something that annoys you personally and seems like it ought not to be too hard, grab the code, and read through it until you've figured out how to fix it18:11
cjwatsondahud: repeat a couple of hundred times :-)18:11
=== binarylooks_away is now known as binarylooks
cjwatsonand then if you're (un)lucky you end up maintaining it ...18:11
psusisay has any thought been given to disabling partition detection in the kernel and moving it to udev with partx?  I can't find any mention of it on the wik or -devel archive18:12
Keybukpsusi: look in lkml18:12
cjwatsonIME working on things that other people have listed tends to be less fulfilling18:12
dahudOK.  Another question: what IDE do you all use?18:12
zygadahud: I don't want to repeat anything everyone else said but basically most of the stuff depends on your ability to read/write code, many people can contribute without that feature18:12
cjwatsondahud: you'll get a wide variety of answers, but many experienced long-term Ubuntu developers just use an advanced editor (vim or emacs) plus command-line tool18:13
cjwatsons18:13
cjwatsonobviously there are IDEs available as well18:13
dahudI mainly want the practice working with real code18:13
=== sconklin is now known as sconklin-gone
Keybukcjwatson: oh I wonder18:14
Keybukyou know that whole plymouth-can't-catch-enter thing?18:14
Keybukcould that be because Enter looks a lot like SIGQUIT to plymouth too ?18:14
Keybukor is this while still in casper?18:15
cjwatsonthat's on shutdown, I think18:15
zygadahud: if you can read (x10) and write (x1) c/python you have access to vast majority of stuff around ubuntu :-),  still - please read that wiki first18:15
Keybukah, ok18:15
Keybukso a new plymouth18:15
cjwatsonwell, at least one of the instances is18:15
psusiKeybuk: as an experiment I just tried using fdisk to add a new partition and the re-read ioctl fails since another partition is in use, but partx -a has no problem adding the new partition... I might have to look into getting gparted to use that18:15
cjwatsonIIRC there's a problem with CD checking too, which is on startup18:15
Keybukthat's while within the initramfs though, right?18:16
cjwatsongood point, yes it is18:17
cjwatsonanyway, really gone18:17
psusidamn... my search is turning up zero instances of partx on lkml...18:22
ograKeybuk, apparently the crash is already reported as bug 537262 and the flickering isnt reproducable on subsequent boots anymore18:39
ubottuLaunchpad bug 537262 in plymouth "plymouth pid missing from OMITPIDS and terminated by sendsigs" [High,Confirmed] https://launchpad.net/bugs/53726218:39
Keybukthat's not a crash then, is it? :p18:40
ograwell, it triggers apport18:40
evRiddell: confirmed and fixed18:54
evI'll do an upload now18:54
Riddellev: ooh18:57
Riddellev: so I was too quick to defame shtylman_?18:57
evwe both were :)18:57
evI fixed this bug in console-setup at the sprint18:57
evbut forgot to port it over to ubiquity18:57
RiddellI always knew shtylman_ was unimpeachable18:58
shtylman_Riddell ev: :)19:00
evlol19:01
=== radoe_ is now known as radoe
mvoKeybuk: i see this at boot now: "Your disk drives need to be checked for error, this may take some time " \o/ -- last week there was only a [C] :)19:42
jdongis there a way to convince apport not to do dupe checking when filing bugs?19:43
jdongin a telepathy-butterfly bug, it's pretty certain that a user's getting a TypeError crash on a different codepath as the bug that apport claims is a dupe19:43
jdongand the user claims apport refuses to allow him to submit a new bug report, saying the bug is already reported at the bug opened in the web browser?19:44
lamonthighvoltage: am now19:48
ScottKdoko: All the Main stuff that was broken due to qt4-x11 on IA64 is built now.  Thanks for taking care of it.19:56
chrisccoulsonjdong - it's likely that the bug pattern needs editting19:57
* ScottK wants apport --please-for-the-love-of-god-let-me-file-this-bug19:57
chrisccoulsonjdong - try checking here: http://bazaar.launchpad.net/~ubuntu-bugcontrol/apport/ubuntu-bugpatterns/annotate/head%3A/telepathy-butterfly.xml19:58
chrisccoulsonit's quite possible that the submitters bug inadvertently matches an existing pattern. If that's incorrect, then the pattern needs fixing19:59
slangasekjdong: if the new bug is being refused, a bug pattern is set for it server-side; you'd need to have pitti change the bug pattern (maybe someone else has access, but I don't know who)19:59
micahgslangasek: bug control should have access to the patterns20:00
micahgso devs included20:00
chrisccoulsoni just pointed a link to the telepathy-butterfly patterns :)20:00
micahgI don't know offhand how often they update20:00
slangasekmicahg: well, I can guarantee that not all of bug control has a clue where to set them :)20:00
thopiekarhi .I'm a member of the Canola project and atm almost the only person working on Canola.. I need more people here at #canola to improve the code.. the player is great and many plugins are available.. please help. the developters that where working on it in the past were paied to work on it - now they have other priorities so - we need you!20:16
* Keybuk has returned20:42
Keybukzombies may crave brains, but Keybuks crave Spring Vegetable Soup and a mug of hot Tea20:42
CaesarDoes the "compiled with the following options" chunk of this excerpt of Universe's Packages file look strange to anyone else?20:44
Caesarhttp://paste.ubuntu.com/396924/20:44
KeybukCaesar: I guess someone was taking notes in the control file and forgot to delete them ;-)20:49
lamontstgraber: around?20:49
CaesarKeybuk: zomg, debian/rules puts it there20:50
Keybukprobably trying to add it to the description20:51
Caesarand failing dismally20:52
CaesarOh fun, it works fine in Debian20:58
CaesarIt's something to do with the insertion of the Original-Maintainer maintainer field in Ubuntu20:58
Keybukyes20:59
Keybukit goes at the bottom20:59
CaesarNot in this package20:59
CaesarIt's in the middle (see the pastebin thingy)20:59
lamonthighvoltage: you around?21:15
highvoltagelamont: yep, hi!21:16
lamontcrying a little here...21:17
highvoltagelamont: do you think you'll have a chance to look at that additional squashfs image on the edubuntu disc thisweek?21:17
highvoltagelamont: crying? how so?21:17
lamontIs there any way of getting the livecd tarball build to _NOT_ require install inetd and half the networking daemons in existance?21:18
lamontwow.  that was almost english21:18
highvoltageI understood it at least :)21:18
c_kornis lucid going to have support for the SSD's trim command ? I read it will be in the 2.6.33 kernel.21:19
highvoltagelamont: it currently tries to install that during the build? because it shouldn't21:19
lamontiproute, tcpd, and openbsd-inetd are particularly offensive21:19
highvoltageperhaps I accidentally made ltsp-livecd depend on ltsp-server (one moment while I check)21:19
lamontltsp-server depends on them, and ltsp-server provides ltsp-build-client and lts-update-image21:19
lamontso I think you intentionally made it depend on ltsp-server21:20
highvoltagewe want ltsp-server to be shipped on the disc, not to beinstalled, I just checked and ltsp-livecd definitely doesn't depend on ltsp-server21:20
* highvoltage checks the seeds21:20
lamontit's installed inside the chroot where we build the image21:21
lamontwhich is what I'm bitching about21:21
highvoltagejust so that we're clear, inside the edubuntu chroot or in the ltsp chroot?21:21
lamonton terranova is a build-lucid-live/chroot-lucid chroot in which we build build/chroot-live (which becomes the casper image), and the ltsp image chroot, which becomes your file21:22
lamontmy complaint is about ltsp-server's dependencies getting installed in build-lucid-live/chroot-lucid21:23
lamontnbd-server belongs on the list too21:23
lamontOTOH, we block all the services from starting, so it's not particularly bad, it just makes me want to stab myself in the eyes21:23
highvoltagelamont: ok, I'm not sure where that comes from but I'll find out now and get it out, it really shouldn't be there21:23
highvoltagelamont: heh, understood21:24
lamontlivecd-rootfs (installed in build-lucid-live/chroot-lucid) Depends: ltsp-server (to get those commands), which Depends: all kinds of crap that we don't actually start, so we don't actually need it.21:25
lamonthighvoltage: IOW, building the client image shouldn't be something that requires me to run it on the server.21:26
lamontbut that's a design bug that won't get fixed for lucid21:26
highvoltagelamont: ok, I'm going to talk to stgraber, that package is in main so I can't change it myself, but I'll ask him to remove it and then give you a ping. thanks a lot for the feedback21:28
lamontI won't let it block us this go round, but can we pretty please fix it for lucid+1?21:29
Keybukdear pbuilder, if I give you a changes file, you should know what to do - you're not bzr, you don't have to be pedantically annoying and difficult21:32
highvoltagelamont: ok, for sure21:32
highvoltagelamont: ok, so the fix for ltsp-server would be to add conditional dependencies on either something like dnsmasq and/or a dummy package that satisfies ltsp-server's dependencies (that would make you squirm less and we'd have a better looking livecd), unfortunately a dummy package would require a FFE and a promotion to main, so it will probably be done early in the M cycle.21:52
lamontwhat's wrong with splitting the package into the part that builds the client and the part that needs the daemons?21:53
highvoltagelamont: splitting it would require FFE's and main promitions right?21:55
lamontyeah, for M21:55
highvoltagelamont: yep, that could also work.21:56
lamontbecause as soon as I upload 1.107, I'm going to file a bug against ltsp to split it, and then a bug against livecd-rootfs dependent on that bug to "NOT DEPEND: ltsp-server, but rather just the bits we need"21:56
lamontI hate dummy packages21:56
highvoltageso what would you recommend, having the scripts in a package that's something like ltsp-server-scripts and then have ltsp-server depend on that?21:57
lamontthey're nothing but clutter.  Also, I'd like to deploy ltsp in a couple of places, and can't use the packages, since the NFS server, tftp server, and dhcp server are 3 different machines, and no way I'm installing all of the daemons on all of the boxes21:57
lamontyeah - that would be a good appraoch21:58
highvoltagelamont: ok, great21:59
highvoltagestgraber: nice timing :)21:59
highvoltagestgraber: lamont suggess that (for the M release) the scripts from ltsp-server should rather go into something like ltsp-server-scripts that doesn't depend on anything, and then ltsp-server can depend on that22:00
highvoltagestgraber: that also makes sense in the cases where (like his use case) people don't want everything on one server22:00
lamontLiveCD/lucid/edubuntu/20100317.1/livecd.edubuntu-ltsp-20100317.1-i386.squashfs <-- highvoltage22:17
lamontSquashfs filesystem, little endian, version 4.0, 55992121855 bytes, 38459 inodes, blocksize: 13 bytes <-- sound about right?22:18
lamontthe 13 byte blocksize seems a little funnhy22:18
highvoltagelamont: *hug*22:21
highvoltagelamont: yes it seems a lot funny22:21
lamontit's also hardy's file command printing that22:21
lamontanyway, once 1.107 is in lucid, wait a day and the image file will be there and you can pester the CD build process to deliver the file for you22:21
highvoltageok thanks, I'll look at it as soon as it's available22:22
lamontyeah - seems there's this pesky beta1 freeze or something. :-(22:22
amitktearjerker22:43
=== oSoMoN_ is now known as oSoMoN
cjwatsonso where does this "does not terminate at computer shutdown" apport thing come from?23:02
cjwatsonah, /usr/share/apport/unkillable_shutdown23:03
sebnercjwatson: do you know if "using stuff in Trash folder" will be fixed in Gnome3? I consider this as br0ken by design (gfvs) as you should be able to open an image (e.g on windows)23:04
cjwatsonsebner: hmm, do I have my fake plastic GNOME developer hat on or something? :)23:07
cjwatson(I have no idea, I don't normally work on GNOME)23:07
slangaseksuperm1: may I suggest http://mythbuntu.org/10.04/beta1 as a URL, since we're getting 2 betas this time 'round?23:07
seb128sebner, what is broken?23:08
sebnercjwatson: I see you as someone very wise and (nearly) omniscient :P23:08
sebnerseb128: broken by design. Files in Trash are not usable. e.g open a picture like it works unter windows23:08
cjwatsonsebner: not about this, try a desktop developer. :)23:08
seb128sebner, how does it work under microsoft os-es?23:09
seb128sebner, double clicking on an image in the trash do works in Ubuntu23:10
seb128not sure what your issue is23:10
chrisccoulsonsebner - what image viewer do you use?23:10
sebnerseb128: not here, eog tells me about not found location and gvfs crashes :P23:10
sebnerchrisccoulson: eog23:10
chrisccoulsonhmm, it works fine here23:10
seb128wfm23:10
sebnervideoplaying is also not working, with any player23:10
sebnerchrisccoulson: lucid23:10
sebner?23:10
chrisccoulsonsebner, yes23:10
sebnerchrisccoulson: weird23:11
cjwatsonwow, when I say "broken by design" it usually means I've confirmed this against the design ;-)23:11
superm1slangasek, yeah that should be fine, i'll make sure the guys know23:11
slangaseksuperm1: ok, thanks23:11
chrisccoulsonsebner, normally issues like that will arise for applications not using gio23:11
seb128sebner, trash is a known uri for gvfs, any app using gio will be able to use it23:11
chrisccoulsonbut that is not the case here23:11
seb128sebner, if you have a crash send it using apport23:12
sebnerseb128: apport is too br0ken to send xD23:12
seb128reinstall your box?23:12
seb128it seems your install is screwed23:12
chrisccoulsoni was just thinking the same thing ;)23:12
sebnerseb128: not wondering. using it since lucid openend. I have to admit that it works *now* but I'm sure tomorrow I'll be able to produce a crash log23:13
sebnerchrisccoulson: lucid ftw! :P23:14
sebnerseb128: ever heard about apport complaining about not being able to send the bug report because there is no internet connection (when there *is* one) ?23:14
seb128no23:14
seb128do you use a proxy?23:14
=== robbiew is now known as robbiew_
chrisccoulsonsebner - there was a launchpad issue over the weekend. did you try recently?23:15
sebnerseb128: nope23:16
sebnerchrisccoulson: recently = 1-2 days ago, yes23:16
sebnerseb128: I guess my system is __really__ b0rken23:16
sebnerseb128: chrisccoulson : sorry for the noise and thx for your time :)23:18
chrisccoulsonyw ;)23:18
seb128yw23:19
sebnerseb128: but playing videos is *not* supported I guess?23:19
seb128?23:20
seb128you would image any modern operating system not able to play a video?23:20
seb128oh, you mean from the trash23:20
seb128totem should support gio locations just fine23:20
seb128sebner, works there23:21
seb128I just moved an ogg video in the trash and double clicked on it23:21
seb128totem play it23:21
sebnerseb128: ah, here too (now), vlc is a bad boy then :)23:22
ionvlc probably just doesn’t know what to do with a gvfs URI.23:22
ionSomething like trash:///blah probably gets passed to it.23:22
sebnerkk :)23:23
seb128yes23:23
seb128ion, well if it does that's because it's .desktop is buggy23:23
seb128hum, maybe not for trash: though, not sure about this one23:23
seb128nautilus passes the .gvfs mounts for apps who don't handle uris23:24
Lord-Readmanhello23:29
KeybukI'm going to write  abot23:29
Keybuka bot too23:30
Lord-Readmancan someone please tell me the package that holds System > Administration > Printing in gnome ubuntu 10.0423:30
Lord-Readman?23:30
Keybukthis bot, once a day, will go through all the bugs in LP that are at Confirmed or Triaged23:30
Keybukand will post23:30
KeybukTHIS BUG IS STILL THERE!!!! OMG!!!!23:30
ionThat could be an internal feature of Launchpad.23:30
micahgKeybuk: you should have the bot post it once for each subscriber...23:31
RAOFion: I'm sure that performance would be greatly improved with a local connection to the database :)23:31
cjwatsonLord-Readman: here's how you find out:23:31
cjwatsonLord-Readman: run System -> Administration -> Printing; open a terminal window and run 'xprop | grep WM_CLASS', then click on the window23:32
cjwatsonit says:23:32
cjwatsonWM_CLASS(STRING) = "system-config-printer.py", "System-config-printer.py"23:32
cjwatsonso run 'dpkg -S system-config-printer.py'23:32
cjwatsonsystem-config-printer-gnome: /usr/share/system-config-printer/system-config-printer.py23:32
elmois nano particularly highly scored in lucid or am I losing my mind?23:33
Lord-Readmanthanks23:33
rickspencer3slangasek, for the beta announcement, should I remove things that were new in Alpha 3, but are no longer new for Beta 1?23:39
slangasekrickspencer3: no, certainly not - the beta announcements are targeted to a different audience than the alphas23:39
rickspencer3thanks23:40
rickspencer3slangasek (that saves me a bit of work ;) )23:40
=== gnomefreak76 is now known as gnomefreak

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