Re: [PATCH 2/2] gnu: Add openvpn service.

2017-01-20 Thread Ludovic Courtès
Alex Kost skribis: > Ludovic Courtès (2017-01-19 12:51 +0100) wrote: [...] >> The problem is that people using indent-code.el typically have raw Emacs >> with scheme-mode, but most likely without Geiser and Emacs-Guix. So we >> need a mechanism that works in that case. > >

Re: [PATCH 2/2] gnu: Add openvpn service.

2017-01-19 Thread Alex Kost
Ludovic Courtès (2017-01-19 12:51 +0100) wrote: > Alex Kost skribis: > >> Ludovic Courtès (2017-01-15 23:11 +0100) wrote: >> [...] >>> I see. So maybe we should just add a rule for ‘match’ in >>> .dir-locals.el; any downside to that? >> >> I'm against it, as I think

Re: [PATCH 2/2] gnu: Add openvpn service.

2017-01-19 Thread Ludovic Courtès
Alex Kost skribis: > Ludovic Courtès (2017-01-15 23:11 +0100) wrote: > >> Alex Kost skribis: >> >>> Ludovic Courtès (2017-01-14 22:16 +0100) wrote: >>> > +(define (uglify-field-name name) > + (match name > + ('verbosity "verb") > +

Re: [PATCH 2/2] gnu: Add openvpn service.

2017-01-16 Thread Alex Kost
Ludovic Courtès (2017-01-15 23:11 +0100) wrote: > Alex Kost skribis: > >> Ludovic Courtès (2017-01-14 22:16 +0100) wrote: >> +(define (uglify-field-name name) + (match name + ('verbosity "verb") + (_ (let ((str (symbol->string name)))

Re: [PATCH 2/2] gnu: Add openvpn service.

2017-01-15 Thread Ludovic Courtès
Alex Kost skribis: > Ludovic Courtès (2017-01-14 22:16 +0100) wrote: > >>> +(define (uglify-field-name name) >>> + (match name >>> + ('verbosity "verb") >>> + (_ (let ((str (symbol->string name))) >>> + (if (string-suffix? "?" str) >>> +

Re: [PATCH 2/2] gnu: Add openvpn service.

2017-01-15 Thread Alex Kost
Ludovic Courtès (2017-01-14 22:16 +0100) wrote: >> +(define (uglify-field-name name) >> + (match name >> + ('verbosity "verb") >> + (_ (let ((str (symbol->string name))) >> + (if (string-suffix? "?" str) >> + (substring str 0 (1- (string-length

Re: [PATCH 2/2] gnu: Add openvpn service.

2017-01-14 Thread Hartmut Goebel
Am 12.01.2017 um 18:57 schrieb Julien Lepiller: > * gnu/services/vpn.scm: New file. > * gnu/local.mk (GNU_SYSTEM_MODULES): Add it. > * doc/guix.texi (VPN Services): N Great! Thanks. What about adding a system example with a openvpn configuration defining one or two peers, an dummy "up" script?

Re: [PATCH 2/2] gnu: Add openvpn service.

2017-01-14 Thread Ludovic Courtès
Howdy, Julien Lepiller skribis: >> Besides, could you think of a system test that would allow us to test >> both services? Perhaps a single config that has both the OpenVPN >> server and client running? Thoughts? > > I don't think you can connect a client to itself, but

Re: [PATCH 2/2] gnu: Add openvpn service.

2017-01-14 Thread Julien Lepiller
On Fri, 13 Jan 2017 18:58:18 +0100 l...@gnu.org (Ludovic Courtès) wrote: > Julien Lepiller skribis: > > > * gnu/services/vpn.scm: New file. > > * gnu/local.mk (GNU_SYSTEM_MODULES): Add it. > > * doc/guix.texi (VPN Services): New section. > > Woow, neat! > > Overall LGTM!

Re: [PATCH 2/2] gnu: Add openvpn service.

2017-01-13 Thread Ludovic Courtès
Julien Lepiller skribis: > * gnu/services/vpn.scm: New file. > * gnu/local.mk (GNU_SYSTEM_MODULES): Add it. > * doc/guix.texi (VPN Services): New section. Woow, neat! Overall LGTM! The following comments are about cosmetic issues, but I think it’s best to get them right.

[PATCH 2/2] gnu: Add openvpn service.

2017-01-12 Thread Julien Lepiller
* gnu/services/vpn.scm: New file. * gnu/local.mk (GNU_SYSTEM_MODULES): Add it. * doc/guix.texi (VPN Services): New section. --- doc/guix.texi| 356 + gnu/local.mk | 1 + gnu/services/vpn.scm | 482