| Keybuk | how are you reproducing? | 00:16 |
|---|---|---|
| mbiebl | unfortunately, I don't have a core file | 00:17 |
| mbiebl | reproducing is easy | 00:18 |
| mbiebl | open /etc/init.d/rc | 00:18 |
| mbiebl | and add "runlevel" after line 72 | 00:18 |
| mbiebl | it also seems, I was on the wrong track with startpar | 00:19 |
| Keybuk | Debian or Ubuntu ? | 00:19 |
| mbiebl | Ubuntu | 00:20 |
| Keybuk | after "export runlevel previous"? | 00:20 |
| mbiebl | yeah | 00:20 |
| mbiebl | who is responsible for setting the env variables PREVLEVEL and RUNLEVEL in rc.conf? | 00:21 |
| Keybuk | nobody | 00:21 |
| Keybuk | telinit sets them in the runlevel event | 00:21 |
| mbiebl | does telinit read /var/run/utmp to get that information? | 00:22 |
| Keybuk | yes | 00:22 |
| Keybuk | and its environment | 00:22 |
| mbiebl | I changed rc.conf a little | 00:23 |
| mbiebl | script | 00:24 |
| mbiebl | echo "RUNLEVEL: $RUNLEVEL" | 00:24 |
| mbiebl | echo "PREVLEVEL: $PREVLEVEL" | 00:24 |
| mbiebl | exec /etc/init.d/rc $RUNLEVEL | 00:24 |
| mbiebl | end script | 00:24 |
| Keybuk | hmm, runlevel just works for me there | 00:24 |
| mbiebl | sometimes PREVLEVEL is just empty on reboot | 00:24 |
| Keybuk | can't you do | 00:24 |
| Keybuk | ulimit -c unlimited | 00:24 |
| mbiebl | sometimes it's 2 | 00:24 |
| Keybuk | runlevel | 00:24 |
| Keybuk | ? | 00:24 |
| mbiebl | looks like a race somewhere | 00:24 |
| mbiebl | ulimit -c unlimited in /etc/init.d/rc ? | 00:25 |
| Keybuk | before calling runlevel | 00:25 |
| Keybuk | to get a coredump | 00:25 |
| Keybuk | oh, I know | 00:26 |
| Keybuk | it's that env RUNLEVEL= | 00:27 |
| Keybuk | should probably be =N ? :) | 00:27 |
| mbiebl | the core files should be in /? | 00:30 |
| Keybuk | aye | 00:30 |
| mbiebl | hm, there isn't any | 00:30 |
| mbiebl | even with ulimit | 00:31 |
| Keybuk | bah | 00:31 |
| Keybuk | if you change it to =N does it work? | 00:31 |
| mbiebl | rc-sysinit.conf? | 00:32 |
| Keybuk | yeah | 00:32 |
| Keybuk | or try 1165 | 00:34 |
| Keybuk | more in 1166 | 00:36 |
| mbiebl | Ok, if I set RUNLEVEL=N | 00:42 |
| mbiebl | I no longer get any asserts | 00:42 |
| mbiebl | runlevel then reports: N N | 00:42 |
| Keybuk | after boot? | 00:43 |
| mbiebl | immediately after the startup event | 00:43 |
| Keybuk | or for rcS? | 00:43 |
| Keybuk | ok | 00:43 |
| mbiebl | Not sure, if the legacy runlevel reported just "N" in that case | 00:43 |
| mbiebl | anyway, I guess runlevel should be more fault tolerant, when RUNLEVEL is not set | 00:44 |
| mbiebl | regarding the reboot problem | 00:44 |
| mbiebl | What I figured out so far is this: | 00:45 |
| Keybuk | I think I fixed that now | 00:45 |
| mbiebl | If I just hit strg+alt+del, I get 2 6 | 00:45 |
| mbiebl | When I login and type reboot, I get N 6 | 00:45 |
| Keybuk | that implies your /var/run/utmp is mucked up? | 00:50 |
| mbiebl | reboot seems to break it, | 00:51 |
| Keybuk | hah | 00:52 |
| Keybuk | by writing a shutdown line | 00:53 |
| Keybuk | and now I get the assert \o/ | 00:55 |
| Keybuk | try 1168 | 00:57 |
| mbiebl | so this should only happen if you *don't* use tmpfs for /var/run? | 00:57 |
| Keybuk | no, this was just a bug | 00:57 |
| Keybuk | reboot writes the shutdown time, which is a special runlevel record | 00:58 |
| Keybuk | that overwrites the runlevel record in utmp | 00:58 |
| mbiebl | I mean the assert in runlevel on boot? | 00:58 |
| Keybuk | but it did that before calling /sbin/shutdown, which meant it couldn't get the runlevel ;) | 00:58 |
| Keybuk | that was just bad sanity checking | 00:59 |
| mbiebl | Keybuk: looks good now | 01:09 |
| mbiebl | no more asserts, no more problems on reboot \o/ | 01:09 |
| mbiebl | btw., I asked on #vim if there is a standard way of installing vim addons | 01:10 |
| mbiebl | apparently there isn't a straighforward procedure | 01:11 |
| Keybuk | include it in the vim source ;) | 01:11 |
| mbiebl | So best is probably, to just ship a README how to install it manually | 01:11 |
| mbiebl | oh yeah, when the upstart syntax is frozen | 01:11 |
| mbiebl | then this is probably the best idea | 01:11 |
| Keybuk | I mean in Debian/Ubuntu | 01:11 |
| === Keybuk changed the topic of #upstart to: PLEASE TEST TRUNK! | Upstart 0.5.3 "Britain's Flag Carrier" | Upstart 0.3.11 "For Friday, June 19th 2009, I'm Jon Masters" | http://upstart.ubuntu.com/ | ||
| Keybuk | anyway bed | 01:12 |
| sadmac | Keybuk: ah | 01:12 |
| Keybuk | I shall release in the morning if there's no show-stoppers | 01:12 |
| sadmac | Keybuk: caught you | 01:12 |
| sadmac | :) | 01:12 |
| sadmac | Keybuk: just filed the state transfer patch | 01:12 |
| Keybuk | sadmac: yup, quickly! | 01:12 |
| sadmac | Keybuk: look at it in the morning if you like :) | 01:12 |
| Keybuk | cool | 01:12 |
| Keybuk | will do | 01:12 |
| mbiebl | Keybuk: make check no longer passes :-/ | 02:04 |
| mbiebl | fyi, I started packaging trunk: http://debs.michaelbiebl.de/upstart/ | 02:11 |
| mbiebl | in case you are interested | 02:11 |
| Keybuk | mbiebl: which revno did make check fail on? | 09:07 |
| ion | keybuk: Does trunk handle the Ubuntu jobs as-is – that is, can i just install to /sbin/init.temp and test with init=/sbin/init.temp? | 09:08 |
| Keybuk | ion: no, they're in different paths and slightly different format | 09:08 |
| Keybuk | I decided it was better to make it easy for 0.10 to be backwards compatible with 0.6 | 09:08 |
| ion | Good :-) | 09:09 |
| ion | % autoreconf -i | 09:17 |
| ion | autopoint: *** cvs program not found | 09:17 |
| ion | le sigh | 09:17 |
| plautrba | sadmac: fyi, i've just finished rpm from todays trunk - http://plautrba.fedorapeople.org/upstart/ | 09:21 |
| ion | An Upstart build non-benchmark on a dualcore laptop: http://pastebin.com/mbb57e6b | 09:47 |
| ion | keybuk: How about tagging the job files with a format version number? Enforce the file to begin with “upstart 0.6\n” and parse the rest as usual perhaps. It would also make file(1)’s job easier, as well as editor syntax highlighter defintions’. | 09:53 |
| Keybuk | I thought about it | 09:59 |
| Keybuk | but I don't really like those kinds of things | 09:59 |
| ion | http://heh.fi/tmp/vim-highlight-upstart-rcS.conf http://github.com/ion1/vim-highlight | 10:08 |
| Keybuk | ion: is that using mbiebl's stuff? | 10:09 |
| ion | I just copied contrib/vim/* to ~/.vim and ran vim-highlight -t upstart conf/rcS.conf foo.html. | 10:10 |
| ion | Yeah, contrib/vim seems to be written by him. :-) | 10:10 |
| ion | -t upstart wouldn’t have been needed, had it been able to automatically recognize the file type. ;-) | 10:11 |
| ion | virtualbox-ose-source doesn’t seem to work with 2.6.31-2-generic-pae. Let’s not test Upstart in a VirtualBox VM then. Let’s see how ubuntu-vm-builder works. | 10:29 |
| Keybuk | wow | 12:22 |
| Keybuk | the test suite is much faster when compiled -O2 ;) | 12:22 |
| Keybuk | make check 55.36s user 4.29s system 63% cpu 1:34.17 total | 12:25 |
| mbiebl | ion: wanna help with polishing the vim syntax file? | 12:42 |
| mbiebl | Keybuk: is the message_iter_abandon_... patch for dbus already in Ubuntu/karmic? | 12:45 |
| Keybuk | yes | 12:45 |
| mbiebl | was it accepted upstream? | 12:46 |
| Keybuk | not yet | 12:46 |
| Keybuk | I need to write a test case for it | 12:48 |
| mbiebl | Keybuk: what about a small configure check and a ifdef? | 12:58 |
| Keybuk | mbiebl: #if YES_I_WANT_TO_ASSERT_ON_ENOMEM ? :) | 12:58 |
| mbiebl | something like that, yes :-) | 12:59 |
| mbiebl | only as long as the patch is not in the official dbus upstream | 13:00 |
| mbiebl | In which can you can bump the pkg-config check | 13:01 |
| Keybuk | I think it's better to subversively force everyone to apply that patch | 13:01 |
| mbiebl | then ./configure should simply abort | 13:01 |
| mbiebl | which would be trivial to do | 13:02 |
| mbiebl | together with a helpful error message, pointing to the fd.o bug report | 13:02 |
| mbiebl | want a patch :-) | 13:04 |
| Keybuk | actually, that is a good point | 13:07 |
| Keybuk | configure should check for that function and abort | 13:07 |
| Keybuk | yes please | 13:07 |
| mbiebl | ok, will do | 13:08 |
| mbiebl | <Keybuk> mbiebl: which revno did make check fail on? | 13:51 |
| mbiebl | 1172 | 13:51 |
| mbiebl | http://paste.debian.net/41435/ | 13:51 |
| Keybuk | hhe | 13:59 |
| Keybuk | isn't that because upstart is already running? | 14:00 |
| plautrba | Keybuk: my "make check" fail - http://pastebin.dqd.cz/939 | 14:08 |
| plautrba | but i have to leave now, bb at monday | 14:08 |
| Keybuk | plautrba: do yoiu not have -lrt in ../nih/libnih.la ? | 14:09 |
| plautrba | no | 14:12 |
| Keybuk | that's weird | 14:14 |
| Keybuk | don't support you mind tarring up your build directory for me so I can examine it? | 14:14 |
| mbiebl | Keybuk: could be | 14:15 |
| mbiebl | when make distcheck passed yesterday | 14:15 |
| mbiebl | I was still running 0.3.11 | 14:15 |
| plautrba | Keybuk: http://plautrba.fedorapeople.org/upstart/upstart-0.6.0-1172-build.tar.bz2 | 14:16 |
| plautrba | Keybuk: it's build from http://plautrba.fedorapeople.org/upstart/upstart-0.6.0-0.1172.fc11.src.rpm | 14:16 |
| Keybuk | ok | 14:21 |
| Keybuk | plautrba: what revno of libnih? | 14:22 |
| Keybuk | ah, 963 | 14:22 |
| Keybuk | -r968 fixed that | 14:22 |
| Keybuk | make check 61.55s user 4.41s system 65% cpu 1:40.97 total | 14:23 |
| Keybuk | (with -Os) | 14:23 |
| Keybuk | that means -O2 was slightly quicker than -Os | 14:23 |
| ion | mbiebl: If i ever get anything done, i’ll post a patch. :-) | 15:53 |
| mbiebl | argh, manpage-de ships init.8 | 16:02 |
| mbiebl | so I alway have to run LANG=C to get the correct man page | 16:03 |
| Keybuk | mbiebl: did you have that configure.ac patch to hand? | 16:05 |
| mbiebl | yeah, could you quickly point me to the fd.o bug again? | 16:10 |
| mbiebl | bugzilla so sucks | 16:11 |
| mbiebl | found (via google) ;-) | 16:16 |
| mbiebl | http://paste.debian.net/41446/ | 16:23 |
| mbiebl | wait a sec | 16:24 |
| mbiebl | We don't want -ldbus-1 added to LIBS | 16:26 |
| mbiebl | this one's better http://paste.debian.net/41447/ | 16:27 |
| Keybuk | I came up with a better way ;) | 16:30 |
| Keybuk | I bumped the D-Bus GIT version to 1.2.15 and just dep on that | 16:30 |
| Keybuk | which gives us the timeout patches too | 16:30 |
| mbiebl | Keybuk: but that makes it harder for distros which ship dbus+the patch | 16:37 |
| Keybuk | mbiebl: D-Bus 1.2.16 will be out in NOW+$SOMETIME | 16:38 |
| mbiebl | Is Collin on it? | 16:39 |
| Keybuk | yup | 16:39 |
| mbiebl | cool | 16:39 |
| Keybuk | <walters> ok, i'll try to do one by tomorrow | 16:39 |
| mbiebl | even better ;-) | 16:40 |
| Keybuk | build test, check | 17:13 |
| Keybuk | distcheck, check | 17:13 |
| Keybuk | package built, check | 17:13 |
| * Keybuk reboots | 17:13 | |
| ion | I wouldn’t mind testing the package. | 17:13 |
| ion | ...as long as your system doesn’t fail to boot with it. :-P | 17:14 |
| Keybuk | I'm just about to upload it :D | 17:15 |
| ion | Ah, cool. | 17:15 |
| Keybuk | sweet | 17:16 |
| Keybuk | it booted | 17:16 |
| ion | That’s always a nice bonus. | 17:17 |
| mbiebl | Keybuk: so what is the make-check-fails-when-upstart-already-running about? | 17:18 |
| Keybuk | mbiebl: not sure, it doesn't do it for me | 17:19 |
| Keybuk | I figured I'll fix that in trunk quickly after the release when I figure it out :) | 17:19 |
| === Keybuk changed the topic of #upstart to: Upstart 0.6.0 "How appropriate, you fight like a cow" | Upstart 0.5.3 "Britain's Flag Carrier" | Upstart 0.3.11 "For Friday, June 19th 2009, I'm Jon Masters" | http://upstart.ubuntu.com/ | ||
| Keybuk | mbiebl: hmm | 17:24 |
| Keybuk | you had a previously failing test case right? | 17:24 |
| Keybuk | try pkill -f test_ | 17:24 |
| Keybuk | the test suite doesn't clean up in case of failure (since I want to be able to debug it usually) | 17:25 |
| mbiebl | I don't think I have any running test_ processes | 17:26 |
| Keybuk | kooky | 17:28 |
| Keybuk | I shall investigate and fix | 17:28 |
| sadmac2 | Keybuk: I applaude your release codename | 17:43 |
| ion | Indeed | 17:44 |
| ion | keybuk: How much did you say the test suite takes to run? I built the Ubuntu package for 0.6.0 locally, and it didn’t take very long. It did seem to run the test suite. Unfortunately, the build log doesn’t seem to contain any timestamps. | 17:51 |
| ion | Wait. What am i talking about? I’ve had the runtime of the previous command in zsh prompt for a while now. debuild -b -j3 of 0.6.0-1 took 491.5 seconds. | 17:52 |
| sadmac2 | ion: where's your underscore? | 17:53 |
| ion | A bit more than 8 minutes, including the time it took to type my PGP password when debuild signed the package. :-P | 17:53 |
| ion | sadmac2: Huh? | 17:53 |
| sadmac2 | ion: you've been ion_ forever | 17:54 |
| ion | Ah. #elsewhere: 2009-06-24 00:25:50 < ion> Woot! I got the nick back. I registered it back in the 1800s and someone immediately complained, claiming he had been using the nick in Freenode before me. I was teh benevolent and gave it to him (i guess i got some good karma, since someone kindly gave the nick back to me in IRCnet, since i had been using it there long before him). It seems he hasn’t been at Freenode for six months and the registration expired. | 17:55 |
| sadmac2 | pimp! | 17:55 |
| ion | ii upstart 0.6.0-1 event-based init daemon | 17:59 |
| ion | Let’s reboot and see whether i come back. | 18:00 |
| ion | It’s alive! | 18:05 |
| ion | keybuk: So, is something wrong with my system or your system (re: test suite runtime)? :-P | 18:09 |
| mbiebl | Keybuk: when will the new daemon monitoring code land in trunk? | 18:57 |
| sadmac2 | notting: how do we feel about 0.6.0 in RHEL? are we going to try to swing it? | 19:11 |
| sadmac2 | plautrba: ^^ | 19:11 |
| notting | gah. | 19:12 |
| sadmac2 | gah? | 19:13 |
| notting | "i would prefer not to deal with the headaches of a rushed migration at this particular exact point in time" | 19:22 |
| sadmac2 | notting: never heard that acronym... | 19:23 |
| notting | not acronym. approximation of an unintelligible verbal noise | 19:23 |
| sadmac2 | notting: its very meaning-dense | 19:23 |
| notting | (see also: 'meh', 'bleah', etc.) | 19:23 |
| * sadmac2 should probably port rawhide this weekend | 19:24 | |
| sadmac2 | I'll be at the beach, so nothing better to do :) | 19:29 |
| mbiebl | Keybuk: is there an equivalent to the old "initctl events" in 0.6.0? | 19:56 |
| sadmac2 | mbiebl: last we discussed it it was removed by design choice | 19:57 |
| === michael__ is now known as mbiebl | ||
| mbiebl | Keybuk: so you merged the debian packages again? | 22:32 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!