Re: [PATCH] wg-quick: add restart command

2020-06-21 Thread Tore Anderson
* Garrit Franke > Do we have an alternative for non-systemd users? My patch ought to work just fine for non-systemd users. Example usage: $ wg-quick reload wg0 Try it out and let me know if it works fine for you? Tore

Re: [PATCH] wg-quick: add restart command

2020-06-20 Thread Tore Anderson
* Jason A. Donenfeld > On Wed, Jun 17, 2020 at 2:17 AM Eric Light wrote: > > As a purely Debian user, the 'service x restart' pattern is far more > > memorable than the syncconf method. I know personal preference isn't a > > great reason to add a knob, but Garrit's method is probably going to

Re: [PATCH] wg-quick: add restart command

2020-06-18 Thread Garrit Franke
Thanks for your comments! I really like the systemctl reload approach. My main intention with this patchset was to add this feature to wg-quicks arsenal because (at least for me) it's the most obvious approach. I mainly use `wg-quick down wg0 && wg0 up wg0`, I think you guys see where I'm coming fr

Re: [PATCH] wg-quick: add restart command

2020-06-17 Thread Eric Light
Oh hey that sounds like a great way to do it. Seems like it'd be simpler than this patch set as well, which is always good. E Q: Why is this email five sentences or less? A: http://five.sentenc.es On Wed, 17 Jun 2020, at 20:19, Jason A. Donenfeld wr

Re: [PATCH] wg-quick: add restart command

2020-06-17 Thread Jason A. Donenfeld
On Wed, Jun 17, 2020 at 2:17 AM Eric Light wrote: > > As a purely Debian user, the 'service x restart' pattern is far more > memorable than the syncconf method. I know personal preference isn't a great > reason to add a knob, but Garrit's method is probably going to be much more > familiar to

Re: [PATCH] wg-quick: add restart command

2020-06-17 Thread Eric Light
Sorry, failure to complete my thought before I sent: "wg-quick wg0 restart" fits much more tightly with the familiar "service x restart" pattern. E Q: Why is this email five sentences or less? A: http://five.sentenc.es On Wed, 17 Jun 2020, at 20:16,

Re: [PATCH] wg-quick: add restart command

2020-06-17 Thread Eric Light
As a purely Debian user, the 'service x restart' pattern is far more memorable than the syncconf method. I know personal preference isn't a great reason to add a knob, but Garrit's method is probably going to be much more familiar to many users. As to _when_ you'd need this... during a config

Re: [PATCH] wg-quick: add restart command

2020-06-17 Thread Jason A. Donenfeld
Thanks for the patchset. I'm wondering what the intended use case of this is. When do you need to set the interface down and then immediately up again? Most changes to the config file can be reflected with a more simple: wg syncconf wg0 <(wg-quick strip wg0)

[PATCH] wg-quick: add restart command

2020-06-17 Thread Garrit Franke
This feature seems quite trivial, so I was wondering if there is any reason not to have this command implemented. I didn’t bother implementing it for android, since there is probably next to no chance of anyone using it on that platform. I’d be happy to take your feedback! Thanks, Garrit Franke