all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: ludo@gnu.org (Ludovic Courtès)
To: Julien Lepiller <julien@lepiller.eu>
Cc: guix-devel@gnu.org, Alex Kost <alezost@gmail.com>
Subject: Re: [PATCH 2/2] gnu: Add openvpn service.
Date: Sat, 14 Jan 2017 22:16:53 +0100	[thread overview]
Message-ID: <87pojpjq9m.fsf@gnu.org> (raw)
In-Reply-To: <20170114143938.45d8f59e@lepiller.eu> (Julien Lepiller's message of "Sat, 14 Jan 2017 14:39:38 +0100")

Howdy,

Julien Lepiller <julien@lepiller.eu> 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 you could run two
> machines, one client and one server. Is it possible?

It’s possible to run two VMs and have one talk to the other, but there’s
currently no test that does that, so it’d be potentially a little bit
more work.

> From 8de1245e8887ae90d34e0262e1eadab091eda603 Mon Sep 17 00:00:00 2001
> From: Julien Lepiller <julien@lepiller.eu>
> Date: Sat, 7 Jan 2017 20:16:00 +0100
> Subject: [PATCH] gnu: Add openvpn service.
>
> * gnu/services/vpn.scm: New file.
> * gnu/local.mk (GNU_SYSTEM_MODULES): Add it.
> * doc/guix.texi (VPN Services): New section.

[...]

> +@node VPN Services
> +@subsubsection VPN Services
> +@cindex VPN

Please make it:

   @cindex VPN (virtual private network)
   @cindex virtual private network (VPN)

Readers might be looking for either of these.

> +(define (uglify-field-name name)
> +  (match name
> +         ('verbosity "verb")
> +         (_ (let ((str (symbol->string name)))
> +              (if (string-suffix? "?" str)
> +                  (substring str 0 (1- (string-length str)))
> +                  str)))))

Not sure why but indent-code.el got ‘match’ indentation wrong, although
it works for me.  Alex, any ideas?

Anyway, please push, thank you!

Ludo’.

  reply	other threads:[~2017-01-14 21:17 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-01-12 17:57 [PATCH 1/2] gnu: openvpn: Update to 2.4.0 Julien Lepiller
2017-01-12 17:57 ` [PATCH 2/2] gnu: Add openvpn service Julien Lepiller
2017-01-13 17:58   ` Ludovic Courtès
2017-01-14 13:39     ` Julien Lepiller
2017-01-14 21:16       ` Ludovic Courtès [this message]
2017-01-15 18:25         ` Alex Kost
2017-01-15 22:11           ` Ludovic Courtès
2017-01-16 19:32             ` Alex Kost
2017-01-19 11:51               ` Ludovic Courtès
2017-01-19 19:01                 ` Alex Kost
2017-01-20 13:45                   ` Ludovic Courtès
2017-01-14 22:07   ` Hartmut Goebel
2017-01-13 17:29 ` [PATCH 1/2] gnu: openvpn: Update to 2.4.0 Ludovic Courtès

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87pojpjq9m.fsf@gnu.org \
    --to=ludo@gnu.org \
    --cc=alezost@gmail.com \
    --cc=guix-devel@gnu.org \
    --cc=julien@lepiller.eu \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/guix.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.