* distro packaging, makefiles, deps, systemd, new tools
@ 2017-01-04 18:50 Jason A. Donenfeld
2017-01-04 20:14 ` [PATCH 1/1] tools: remove DESTDIR for autodetection Christian Hesse
` (2 more replies)
0 siblings, 3 replies; 10+ messages in thread
From: Jason A. Donenfeld @ 2017-01-04 18:50 UTC (permalink / raw)
To: Christian Hesse, Daniel Kahn Gillmor, Dan Lüdtke
Cc: WireGuard mailing list
Hey Christian, Daniel, Dan, and other maintainers,
The next snapshot is going to have some improvements to the tooling,
and along with this comes a more complicated Makefile. I wanted to run
the changes by y'all (downstreams).
- The makefile now has a few WITH_{FEATURE}=yes|no environment flags,
as well as auto-detection. Documentation is here:
https://git.zx2c4.com/WireGuard/plain/src/tools/INSTALL
And the Makefile itself is here:
https://git.zx2c4.com/WireGuard/tree/src/tools/Makefile
Question: does this propose a desirable situation for you as a downstream?
- The wg-config script is now called wg-quick and is quite simple and
nice. Documentation:
https://git.zx2c4.com/WireGuard/about/src/tools/wg-quick.8
And source:
https://git.zx2c4.com/WireGuard/tree/src/tools/wg-quick.bash
OpenWRT should _not_ ship this. NixOS probably should not ship this.
When systemd-networkd gets support for WireGuard, systemd-based
distros should not ship this. But everybody else who just wants a
simple rough script for quickly bringing up and down VPNs can ship
this, as it's a big help to users. It's installed according to the
documentation for the Makefile above.
- There's now an "upstream" systemd unit for wg-quick, called
wg-quick@.service. It's pretty simple:
https://git.zx2c4.com/WireGuard/tree/src/tools/wg-quick@.service
Question: does this unit seem sane to you? Can you ship it downstream
as is, or will you need modifications?
Thanks,
Jason
^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH 1/1] tools: remove DESTDIR for autodetection
2017-01-04 18:50 distro packaging, makefiles, deps, systemd, new tools Jason A. Donenfeld
@ 2017-01-04 20:14 ` Christian Hesse
2017-01-04 20:26 ` Jason A. Donenfeld
2017-01-04 21:02 ` Daniel Kahn Gillmor
2017-01-04 21:16 ` distro packaging, makefiles, deps, systemd, new tools Daniel Kahn Gillmor
2017-01-05 14:02 ` Dan Lüdtke
2 siblings, 2 replies; 10+ messages in thread
From: Christian Hesse @ 2017-01-04 20:14 UTC (permalink / raw)
To: wireguard; +Cc: Christian Hesse
From: Christian Hesse <mail@eworm.de>
DESTDIR is always empty, no need to check anything there. Check the main
system instead.
Signed-off-by: Christian Hesse <mail@eworm.de>
---
src/tools/Makefile | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/src/tools/Makefile b/src/tools/Makefile
index fe483b9..8160cc9 100644
--- a/src/tools/Makefile
+++ b/src/tools/Makefile
@@ -12,12 +12,12 @@ WITH_WGQUICK ?=
WITH_SYSTEMDUNITS ?=
ifeq ($(WITH_BASHCOMPLETION),)
-ifneq ($(strip $(wildcard $(DESTDIR)$(BASHCOMPDIR))),)
+ifneq ($(strip $(wildcard $(BASHCOMPDIR))),)
WITH_BASHCOMPLETION := yes
endif
endif
ifeq ($(WITH_WGQUICK),)
-ifneq ($(strip $(wildcard $(DESTDIR)$(BINDIR)/bash)),)
+ifneq ($(strip $(wildcard $(BINDIR)/bash)),)
WITH_WGQUICK := yes
endif
ifneq ($(strip $(wildcard $(DESTDIR)/bin/bash)),)
@@ -25,7 +25,7 @@ WITH_WGQUICK := yes
endif
endif
ifeq ($(WITH_SYSTEMDUNITS),)
-ifneq ($(strip $(wildcard $(DESTDIR)$(SYSTEMDUNITDIR))),)
+ifneq ($(strip $(wildcard $(SYSTEMDUNITDIR))),)
WITH_SYSTEMDUNITS := yes
endif
endif
--
2.11.0
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 1/1] tools: remove DESTDIR for autodetection
2017-01-04 20:14 ` [PATCH 1/1] tools: remove DESTDIR for autodetection Christian Hesse
@ 2017-01-04 20:26 ` Jason A. Donenfeld
2017-01-04 21:02 ` Daniel Kahn Gillmor
1 sibling, 0 replies; 10+ messages in thread
From: Jason A. Donenfeld @ 2017-01-04 20:26 UTC (permalink / raw)
To: Christian Hesse; +Cc: Christian Hesse, WireGuard mailing list
Hey Christian,
Seems reasonable to me.
Merged.
Jason
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 1/1] tools: remove DESTDIR for autodetection
2017-01-04 20:14 ` [PATCH 1/1] tools: remove DESTDIR for autodetection Christian Hesse
2017-01-04 20:26 ` Jason A. Donenfeld
@ 2017-01-04 21:02 ` Daniel Kahn Gillmor
2017-01-04 21:06 ` Jason A. Donenfeld
1 sibling, 1 reply; 10+ messages in thread
From: Daniel Kahn Gillmor @ 2017-01-04 21:02 UTC (permalink / raw)
To: Christian Hesse, wireguard; +Cc: Christian Hesse
On Wed 2017-01-04 15:14:16 -0500, Christian Hesse wrote:
> From: Christian Hesse <mail@eworm.de>
>
> DESTDIR is always empty, no need to check anything there. Check the main
> system instead.
why are we checking either place? It's entirely possible that my
dedicated build farm doesn't have any bash completion mechanisms
installed, even if my development system does.
--dkg
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 1/1] tools: remove DESTDIR for autodetection
2017-01-04 21:02 ` Daniel Kahn Gillmor
@ 2017-01-04 21:06 ` Jason A. Donenfeld
2017-01-04 21:18 ` Jason A. Donenfeld
0 siblings, 1 reply; 10+ messages in thread
From: Jason A. Donenfeld @ 2017-01-04 21:06 UTC (permalink / raw)
To: Daniel Kahn Gillmor; +Cc: Christian Hesse, WireGuard mailing list
On Wed, Jan 4, 2017 at 10:02 PM, Daniel Kahn Gillmor
<dkg@fifthhorseman.net> wrote:
> why are we checking either place? It's entirely possible that my
> dedicated build farm doesn't have any bash completion mechanisms
> installed, even if my development system does.
Yes, the concept of auto-detection is "broken". But it's useful as a
default, for people doing `make && make install` on their dev
machines. Packagers should always set these types of flags explicitly,
as with autotools.
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: distro packaging, makefiles, deps, systemd, new tools
2017-01-04 18:50 distro packaging, makefiles, deps, systemd, new tools Jason A. Donenfeld
2017-01-04 20:14 ` [PATCH 1/1] tools: remove DESTDIR for autodetection Christian Hesse
@ 2017-01-04 21:16 ` Daniel Kahn Gillmor
2017-01-05 14:02 ` Dan Lüdtke
2 siblings, 0 replies; 10+ messages in thread
From: Daniel Kahn Gillmor @ 2017-01-04 21:16 UTC (permalink / raw)
To: Jason A. Donenfeld, Christian Hesse, Dan Lüdtke
Cc: WireGuard mailing list
[-- Attachment #1: Type: text/plain, Size: 1827 bytes --]
On Wed 2017-01-04 13:50:14 -0500, Jason A. Donenfeld wrote:
> Hey Christian, Daniel, Dan, and other maintainers,
>
> The next snapshot is going to have some improvements to the tooling,
> and along with this comes a more complicated Makefile. I wanted to run
> the changes by y'all (downstreams).
>
> - The makefile now has a few WITH_{FEATURE}=yes|no environment flags,
> as well as auto-detection. Documentation is here:
> https://git.zx2c4.com/WireGuard/plain/src/tools/INSTALL
> And the Makefile itself is here:
> https://git.zx2c4.com/WireGuard/tree/src/tools/Makefile
>
> Question: does this propose a desirable situation for you as a downstream?
Reading through INSTALL, these parameters look reasonable to me as a
downstream.
> - The wg-config script is now called wg-quick and is quite simple and
> nice. Documentation:
> https://git.zx2c4.com/WireGuard/about/src/tools/wg-quick.8
> And source:
> https://git.zx2c4.com/WireGuard/tree/src/tools/wg-quick.bash
>
> OpenWRT should _not_ ship this. NixOS probably should not ship this.
> When systemd-networkd gets support for WireGuard, systemd-based
> distros should not ship this. But everybody else who just wants a
> simple rough script for quickly bringing up and down VPNs can ship
> this, as it's a big help to users. It's installed according to the
> documentation for the Makefile above.
this also looks good.
> - There's now an "upstream" systemd unit for wg-quick, called
> wg-quick@.service. It's pretty simple:
> https://git.zx2c4.com/WireGuard/tree/src/tools/wg-quick@.service
>
> Question: does this unit seem sane to you? Can you ship it downstream
> as is, or will you need modifications?
although i haven't tested it, a quick read through the .service file
looks reasonable to me. Thanks for thinking about system integration
stuff.
--dkg
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 832 bytes --]
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: distro packaging, makefiles, deps, systemd, new tools
2017-01-04 18:50 distro packaging, makefiles, deps, systemd, new tools Jason A. Donenfeld
2017-01-04 20:14 ` [PATCH 1/1] tools: remove DESTDIR for autodetection Christian Hesse
2017-01-04 21:16 ` distro packaging, makefiles, deps, systemd, new tools Daniel Kahn Gillmor
@ 2017-01-05 14:02 ` Dan Lüdtke
2017-01-06 23:01 ` Jason A. Donenfeld
2 siblings, 1 reply; 10+ messages in thread
From: Dan Lüdtke @ 2017-01-05 14:02 UTC (permalink / raw)
To: Jason A. Donenfeld; +Cc: Christian Hesse, WireGuard mailing list
> On 4 Jan 2017, at 19:50, Jason A. Donenfeld <Jason@zx2c4.com> wrote:
>=20
> OpenWRT should _not_ ship this. =08=0D
Agreed.
Everything else looks good to me. Just had a brief look at it.
Let's see what buildbot says. Should all run smoothly!
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: distro packaging, makefiles, deps, systemd, new tools
2017-01-05 14:02 ` Dan Lüdtke
@ 2017-01-06 23:01 ` Jason A. Donenfeld
2017-01-11 16:46 ` Dan Lüdtke
0 siblings, 1 reply; 10+ messages in thread
From: Jason A. Donenfeld @ 2017-01-06 23:01 UTC (permalink / raw)
To: Dan Lüdtke; +Cc: Christian Hesse, WireGuard mailing list
[-- Attachment #1: Type: text/plain, Size: 419 bytes --]
On Jan 6, 2017 23:44, "Dan Lüdtke" <mail@danrl.com> wrote:
> On 4 Jan 2017, at 19:50, Jason A. Donenfeld <Jason@zx2c4.com> wrote:
>
> OpenWRT should _not_ ship this.
Agreed.
Everything else looks good to me. Just had a brief look at it.
Let's see what buildbot says. Should all run smoothly!
I assume youre not shipping the bash completion stuff either, right? Since
openwrt doesn't use bash.
[-- Attachment #2: Type: text/html, Size: 933 bytes --]
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2017-01-11 16:36 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-01-04 18:50 distro packaging, makefiles, deps, systemd, new tools Jason A. Donenfeld
2017-01-04 20:14 ` [PATCH 1/1] tools: remove DESTDIR for autodetection Christian Hesse
2017-01-04 20:26 ` Jason A. Donenfeld
2017-01-04 21:02 ` Daniel Kahn Gillmor
2017-01-04 21:06 ` Jason A. Donenfeld
2017-01-04 21:18 ` Jason A. Donenfeld
2017-01-04 21:16 ` distro packaging, makefiles, deps, systemd, new tools Daniel Kahn Gillmor
2017-01-05 14:02 ` Dan Lüdtke
2017-01-06 23:01 ` Jason A. Donenfeld
2017-01-11 16:46 ` Dan Lüdtke
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).