**** BEGIN LOGGING AT Sat Sep 11 02:59:57 2010
Sep 11 08:34:28 <gonzobrains> anyone here?
Sep 11 08:34:32 <gonzobrains> hey blathijs
Sep 11 08:34:34 <gonzobrains> sup?
Sep 11 09:29:29 <blathijs> ey gonzobrains
Sep 11 09:29:41 <blathijs> I'm not really here though, busy cleaning the bathroom ;-)
Sep 11 09:43:13 <Barbon> blathijs is fun XD
Sep 11 11:14:53 <JonTheNiceGuy> Good morning all. I have good news, and maybe good news :)
Sep 11 11:15:02 <JonTheNiceGuy> Oh, and an offer too :)
Sep 11 11:15:22 <JonTheNiceGuy> Good news: I was wrong, my previous builds had completed :)
Sep 11 11:16:01 <blathijs> heh
Sep 11 11:16:19 <blathijs> Good :-)
Sep 11 11:16:42 <JonTheNiceGuy> Maybe good news: I've started hacking on the OpenVPN scripts and code. But, I'm really nervous of screwing up my router - especially at the weekend, when the missus is likely to want to use it. Can I send the diffs to someone to test my changes? ;)
Sep 11 11:18:43 <JonTheNiceGuy> And lastly, the offer; while I'm trying to get my head around Git, I've performed a git-svn clone against the FON svn repo. To save anyone else having to do that again, would it be worth putting that git-svn version somewhere? I can host it at my webserver, we can put it on gitorious, or somewhere else, or should we just leave it as is? :)
Sep 11 11:19:11 <blathijs> heh, I just did an git-svn clone myself this week :-)
Sep 11 11:19:17 <JonTheNiceGuy> D'oh :)
Sep 11 11:19:37 <blathijs> and I was also playing with the idea of hosting it somewhere, so GMTA :-)
Sep 11 11:19:45 <JonTheNiceGuy> :D
Sep 11 11:20:24 <JonTheNiceGuy> I can strongly recommend gitorious, especially as the project already has it's own trac etc. instance.
Sep 11 11:21:01 <blathijs> I also have my own gitweb setup, but I guess gitorious might be nice as well
Sep 11 11:21:14 <JonTheNiceGuy> Or, if it's to stay within the fonosfera service, it might be worth making an action on an SVN commit to run a git-svn fetch against the repo, so the two stay in sync
Sep 11 11:21:22 <blathijs> Though we'd have to think a bit about how git would fit into the workflow, of course
Sep 11 11:21:42 <blathijs> Yeah, perhaps something like that
Sep 11 11:22:00 <JonTheNiceGuy> Actually, that could be pretty useful.
Sep 11 11:22:02 <blathijs> I guess I should poll on the fonosfera-development list if there's interest in having a git mirror
Sep 11 11:22:31 <blathijs> As for your openvpn diffs, I won't have any real time available this weekend for testing, but if you give me the diffs, I can do a quick review
Sep 11 11:22:38 <JonTheNiceGuy> No worries.
Sep 11 11:22:56 <blathijs> furthermore, you're very unlikely to break your internet connection just by messing around with luci code
Sep 11 11:23:26 <JonTheNiceGuy> On the fon 2.0n's is the lua is compiled by default?
Sep 11 11:23:29 <blathijs> (though if you mess up the network configuration with your luci code, then it can of course happen ;-p)
Sep 11 11:23:36 <blathijs> yes, it is
Sep 11 11:23:46 <JonTheNiceGuy> My main issue is flashing a new image to it, and then it not working.
Sep 11 11:26:06 <JonTheNiceGuy> I've just attached the diff to the ticket 907
Sep 11 11:26:37 <JonTheNiceGuy> That's the keepalive one. I figured, it was probably the easiest one to get started with, as it should just be a binary "Use it or not" switch.
Sep 11 11:31:40 <blathijs> As for flashing, you should be pretty safe with that. The procedure is pretty solid. And there's always the restore procedure.
Sep 11 11:31:55 <blathijs> Alternatively, you can also test by just replacing the lua files with sourcecode yourself
Sep 11 11:32:07 <JonTheNiceGuy> Oh, ok, that works :)
Sep 11 11:32:12 <JonTheNiceGuy> Didn't know you can do that!
Sep 11 11:32:13 <blathijs> the firmware doesn't care if .lua files contain precompiled code or source
Sep 11 11:32:16 <blathijs> hehe
Sep 11 11:32:35 <blathijs> I just suggested you replace the entire firmware, so you can look around more easily and start editing right away in files
Sep 11 11:32:44 <JonTheNiceGuy> Ahhhhh
Sep 11 11:32:45 <JonTheNiceGuy> Right
Sep 11 11:32:46 <JonTheNiceGuy> :)
Sep 11 11:32:50 <blathijs> (Sometimes it needs a bit of searching to find out where a .lua file came from in the sources)
Sep 11 11:33:15 <JonTheNiceGuy> That's what find's for, though, isn't it :D
Sep 11 11:33:18 <JonTheNiceGuy> :D
Sep 11 11:33:51 <blathijs> true :-)
Sep 11 11:33:56 <blathijs> You changed "sed" into "set", btw
Sep 11 11:34:38 <JonTheNiceGuy> D'oh. That wouldn't work heh
Sep 11 11:35:02 <blathijs> And you didn't need to remove the safe surf remark (for one it is unrelated to this particular patch, and we will be re-enabling that in the next firmware)
Sep 11 11:35:31 <JonTheNiceGuy> Ahhhhh, cool. I just spotted it. I didn't recall whether I'd kept it in in that patch.
Sep 11 11:35:51 <JonTheNiceGuy> OK, I'll fix those two, and re-send the diff
Sep 11 11:36:46 <blathijs> Always read a .diff file before submitting or committing it :-)
Sep 11 11:37:05 <blathijs> Finally, you can probably simplify the first part of the diff a bit. You have separate handling for "" and "0" now
Sep 11 11:37:22 <blathijs> Just say:
Sep 11 11:37:31 <blathijs> KEEPALIVE=";keepalive 10 120"
Sep 11 11:37:37 <blathijs> [ "$VALUEKEEPALIVE" = "1" ]  && KEEPALIVE="keepalive 10 120"
Sep 11 11:37:52 <blathijs> so the default is disabled, and a value of "1" enables it
Sep 11 11:38:14 <blathijs> And I would be grateful if you could actually test the patch before resubmitting it :-)
Sep 11 11:39:55 <JonTheNiceGuy> No worries, now I know I can just stick this lot into my router, I'll be glad to check them out first ;)
Sep 11 11:40:56 <blathijs> thanks :-)
Sep 11 11:41:01 * blathijs is off again
Sep 11 11:41:05 <blathijs> (happy 1337, btw)
Sep 11 11:43:21 <JonTheNiceGuy> :)
Sep 11 14:10:42 <JonTheNiceGuy> OK, so that's the diffs added to #907 and #908. I've tested both against my router, and they seem to work. These were the two critical ones for providing remote support to my family... the others, not so much so :)
Sep 11 14:12:46 <JonTheNiceGuy> I'll have a play around with the others at some point, but not today. Thanks for all your help blath ijs (adjusted so it doesn't bring you back to the channel just to get a thanks from wherever you may be! :D )
Sep 11 14:13:20 <JonTheNiceGuy> Oh, and thanks to every last one of you who's helped to make this project so interesting and great!
**** ENDING LOGGING AT Sun Sep 12 02:59:57 2010