*** mike_k has joined #crux-devel | 01:29 | |
jaeger | morning | 04:53 |
---|---|---|
jaeger | teK_: ignore my comment about the master hook and individual emails; it's a for loop in the hook that generates the emails anyway, so that wouldn't matter | 04:54 |
jaeger | http://pastebin.com/qevbGEeE <-- example of the 'pre-receive' hook barfing on a 'master' branch push | 07:12 |
*** joe9 has quit IRC | 07:12 | |
jue | jaeger: thanks for fixing our mail mess | 07:43 |
jaeger | no problem :) I'm working on better git hooks now, I'll post them here for review | 07:43 |
jaeger | the pre-receive is ready, I think, just need to upload | 07:43 |
jaeger | http://crux.nu/~jaeger/pre-receive.txt | 07:45 |
jaeger | I've also written a post-receive to do the email notifications but it's fairly ugly in bash. I'll convert it to python | 07:47 |
jue | pre-receive looks good to me | 07:57 |
jaeger | good, thanks for looking :) | 07:57 |
jue | np | 08:15 |
*** horrorSt1uck has joined #crux-devel | 08:46 | |
*** horrorStruck has quit IRC | 08:48 | |
jaeger | ok, testing the post-receive email generator as a pre-receive hook on my local git so I can verify the emails are sent properly :D | 09:39 |
jaeger | and spam the hell out of myself | 09:41 |
jue | :) | 09:43 |
jue | video-intel -> http://lists.x.org/archives/xorg/2013-February/055450.html :) | 09:56 |
jue | guess GKH forgot to mention that libdbus finally will be called systemd-dbus -> http://lwn.net/Articles/537017/ | 10:10 |
jue | sorry, will stop now with pasting random stuff here :) | 10:11 |
jaeger | jue: http://crux.nu/~jaeger/jaeger-post-receive.txt | 10:20 |
jaeger | that's a log from my latest test of the post-receive (running as a pre-receive so I could do it repeatedly without actually pushing anything) | 10:21 |
jaeger | I limited it to 25 emails just to keep the output reasonable | 10:21 |
jaeger | http://crux.nu/~jaeger/email.txt <-- one of the emails it sent | 10:22 |
jaeger | I added X-Git-Commit and X-Git-Refname headers, though maybe nobody will care about them | 10:23 |
jue | cool, looks very nice :) | 10:25 |
jaeger | I will post the script itself after I add some commenting | 10:25 |
jue | I'm off for today now but will look at it tomorrow morning | 10:27 |
jaeger | take care | 10:33 |
jaeger | ok, here are the finished hooks: http://crux.nu/~jaeger/pre-receive.txt and http://crux.nu/~jaeger/post-receive.txt | 10:55 |
jaeger | teK_, frinnst, jue: comments/questions welcome. I'm not putting them in place until everyone signs off on them, just in case. | 10:56 |
jaeger | this post-receive hook fixes the revert bounces, too, tested that | 10:57 |
jaeger | crux-commits@lists.crux.nu posting should be fixed now, too. The list was set to discard everything sent to it, regardless of who sent it | 11:37 |
jaeger | now it should be set to accept mail from crux@crux.nu, hold mail from any subscribed member in the moderator list, and discard everything else immediately | 11:37 |
frinnst | nice weekend work :D | 12:25 |
frinnst | from what i can tell reading the code it looks ok | 12:28 |
frinnst | no "rm -rf /" or anything | 12:28 |
frinnst | ;p | 12:28 |
jaeger | :) | 12:30 |
frinnst | I dont really speak python so | 12:33 |
jaeger | Well, if you want me to clarify something that isn't explained by the comments feel free to ask | 12:43 |
frinnst | well, assuming the code does what it says in the comment - it looks good :) | 12:57 |
jaeger | It does in my tests but I do appreciate you looking at it, I'm certainly not perfect | 12:59 |
jaeger | No need to learn python, of course, I just figured if anyone spotted a flow problem or something I could fix it | 12:59 |
*** mike_k has quit IRC | 13:54 | |
*** sepen has joined #crux-devel | 14:02 | |
sepen | jaeger: around here? | 14:03 |
jaeger | yes | 14:10 |
sepen | pfffff | 14:12 |
sepen | I read the irclog and seen all the work you had to do for my fault, I'm really sorry and thank you for being where I was not | 14:12 |
jaeger | Don't worry about it, mistakes happen :) Not a big problem | 14:12 |
sepen | I'm really embarrassed, I thought I did a shit solved in git but I paid no attention to the hook scripts, pffffffff | 14:12 |
jaeger | It did help expose some problems with our git hooks :) | 14:12 |
sepen | yeah I read your new scripts | 14:13 |
sepen | but I'm really ashamed | 14:13 |
jaeger | Don't worry, it's not a big deal | 14:13 |
sepen | thanks | 14:13 |
sepen | but I can not reduce the impact I've had on the list subscribers | 14:14 |
jaeger | Well, we caught it before it sent them all, so it could have been worse | 14:14 |
jaeger | just think, it tried (and failed) to send 7.3k messages to crux-commits, but only 280 to crux | 14:14 |
jaeger | :) | 14:14 |
sepen | yeah, thats for what I'm really sorry | 14:14 |
sepen | can't imagine the impact could happened without you | 14:14 |
sepen | pfff and this compromises the seriousness of your job due to me | 14:15 |
jaeger | A few emails won't really hurt anyone in the long run, it's ok | 14:15 |
sepen | is all the work done? can I help on something? | 14:15 |
jaeger | Yeah, it's all done now except to replace the git hooks. I just wanted to post them for review before replacing them | 14:16 |
jaeger | I disabled the current 'update' hook so it won't send any emails right now but everything else works | 14:16 |
sepen | http://wiki.list.org/pages/viewpage.action?pageId=4030681 | 14:16 |
sepen | http://lists.crux.nu/pipermail/crux/2013-February/thread.html | 14:17 |
sepen | I could edit the mbox file manually if that helps | 14:17 |
jaeger | If you want to, sure... I don't have a strong feeling about the archives | 14:17 |
jaeger | up to you :) | 14:17 |
sepen | nah, I'll copy the mbox file and do the job locally | 14:18 |
sepen | I feel bad, I have to do something | 14:18 |
sepen | sorry again :P | 14:18 |
jaeger | hehe, it's ok | 14:19 |
sepen | you are a great guy | 14:19 |
sepen | muchas gracias | 14:19 |
jaeger | I appreciate that but don't tell anyone! I've got my reputation to uphold | 14:19 |
sepen | ajajajaj | 14:20 |
jaeger | frinnst: that handbrake thing looks like some inception-style shit, heh | 14:22 |
jaeger | a52dec configure.in inside the contrib dir in the source | 14:22 |
frinnst | yeah | 14:24 |
frinnst | hah, damn | 14:25 |
sepen | jaeger: please, could you copy /usr/mailman/archives/private/crux to my home? | 14:26 |
frinnst | it fetches sources from the makefile | 14:26 |
jaeger | sepen: sure, sec | 14:30 |
jaeger | copying | 14:32 |
sepen | thanks | 14:32 |
jaeger | lot of files :) | 14:32 |
sepen | oops | 14:32 |
jaeger | ok, it's done | 14:32 |
sepen | well, only 2013-february its need it | 14:33 |
jaeger | true, didn't think of that before I copied it :) it's only 151M, just lots of files | 14:33 |
sepen | nah | 14:33 |
sepen | I'll scp only related files to my pc | 14:34 |
*** sepen has quit IRC | 15:13 | |
*** sepen has joined #crux-devel | 15:18 | |
sepen | jaeger: still here? | 17:01 |
sepen | sed 5512,47929d -i 2013-February.txt will delete all opt (master) messages | 17:02 |
sepen | jaeger: I need crux.mbox file to test | 17:12 |
sepen | thanks in advance and good night | 17:12 |
*** sepen has quit IRC | 17:12 | |
jaeger | sepen: the only crux.mbox file on the server is in cptn's homedir, dated 2006 | 17:30 |
jaeger | it seems like the current mailman doesn't use an mbox file | 17:30 |
jaeger | ooh, wait, never mind, locate was lying to me | 17:31 |
jaeger | ok, copied into your homedir | 17:31 |
*** joe9 has joined #crux-devel | 17:53 | |
*** mavrick61 has quit IRC | 19:45 | |
*** mavrick61 has joined #crux-devel | 19:46 | |
*** deus_ex has joined #crux-devel | 20:07 | |
*** deus_ex has quit IRC | 20:49 | |
*** deus_ex has joined #crux-devel | 21:03 | |
*** joe9 has quit IRC | 21:05 | |
*** mike_k has joined #crux-devel | 23:26 |
Generated by irclog2html.py 2.11.0 by Marius Gedminas - find it at mg.pov.lt!