unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
* [bug#64468] [PATCH] services: dhcp-client-service-type: Support DDNS.
@ 2023-07-05  0:10 Lilah Tascheter via Guix-patches via
  2023-10-11 17:14 ` Ludovic Courtès
  0 siblings, 1 reply; 4+ messages in thread
From: Lilah Tascheter via Guix-patches via @ 2023-07-05  0:10 UTC (permalink / raw)
  To: 64468; +Cc: Lilah Tascheter

* gnu/services/networking.scm (dhcp-client-shepherd-service): Enable -I
  flag on dhclient.
---
 gnu/services/networking.scm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gnu/services/networking.scm b/gnu/services/networking.scm
index 5657b141d9..0fbba1b8c7 100644
--- a/gnu/services/networking.scm
+++ b/gnu/services/networking.scm
@@ -350,7 +350,7 @@ (define ifaces
 
                          (false-if-exception (delete-file #$pid-file))
                          (let ((pid (fork+exec-command
-                                     (cons* dhclient "-nw"
+                                     (cons* dhclient "-nw" "-I"
                                             "-pf" #$pid-file ifaces))))
                            (and (zero? (cdr (waitpid pid)))
                                 (read-pid-file #$pid-file)))))

base-commit: 9122736455a9085d51b71b269b7c26a695a474ef
-- 
2.40.1





^ permalink raw reply related	[flat|nested] 4+ messages in thread

* [bug#64468] [PATCH] services: dhcp-client-service-type: Support DDNS.
  2023-07-05  0:10 [bug#64468] [PATCH] services: dhcp-client-service-type: Support DDNS Lilah Tascheter via Guix-patches via
@ 2023-10-11 17:14 ` Ludovic Courtès
  2023-10-12 20:23   ` Lilah Tascheter via Guix-patches
  0 siblings, 1 reply; 4+ messages in thread
From: Ludovic Courtès @ 2023-10-11 17:14 UTC (permalink / raw)
  To: Lilah Tascheter; +Cc: 64468

Hi,

Lilah Tascheter <lilah@lunabee.space> skribis:

> * gnu/services/networking.scm (dhcp-client-shepherd-service): Enable -I
>   flag on dhclient.

[...]

> +                                     (cons* dhclient "-nw" "-I"

Could there be unintended side effects from passing “-I”?  The
dhclient(8) man page is silent about the implications.

If there are none, the patch LGTM.

Thanks,
Ludo’.




^ permalink raw reply	[flat|nested] 4+ messages in thread

* [bug#64468] [PATCH] services: dhcp-client-service-type: Support DDNS.
  2023-10-11 17:14 ` Ludovic Courtès
@ 2023-10-12 20:23   ` Lilah Tascheter via Guix-patches
  2023-10-14 17:45     ` bug#64468: " Ludovic Courtès
  0 siblings, 1 reply; 4+ messages in thread
From: Lilah Tascheter via Guix-patches @ 2023-10-12 20:23 UTC (permalink / raw)
  To: Ludovic Courtès

Ludovic Courtès <ludo@gnu.org> wrote:
> Could there be unintended side effects from passing "-I"?

the isc-dhcp knowledgebase says that by default dhclient uses a
pre-standardization implementation of DDNS, which is incompatable with non-isc
DHCP servers. no idea why that's the default, but -I just switches to using the
standardized protocol.

kb link with more details: https://kb.isc.org/docs/aa-01091

~lunabee




^ permalink raw reply	[flat|nested] 4+ messages in thread

* bug#64468: [PATCH] services: dhcp-client-service-type: Support DDNS.
  2023-10-12 20:23   ` Lilah Tascheter via Guix-patches
@ 2023-10-14 17:45     ` Ludovic Courtès
  0 siblings, 0 replies; 4+ messages in thread
From: Ludovic Courtès @ 2023-10-14 17:45 UTC (permalink / raw)
  To: Lilah Tascheter; +Cc: 64468-done

Lilah Tascheter <lilah@lunabee.space> skribis:

> Ludovic Courtès <ludo@gnu.org> wrote:
>> Could there be unintended side effects from passing "-I"?
>
> the isc-dhcp knowledgebase says that by default dhclient uses a
> pre-standardization implementation of DDNS, which is incompatable with non-isc
> DHCP servers. no idea why that's the default, but -I just switches to using the
> standardized protocol.
>
> kb link with more details: https://kb.isc.org/docs/aa-01091

Thanks for explaining.  I added this explanation as a comment in the
code and committed.

Ludo’.




^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2023-10-14 17:45 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-07-05  0:10 [bug#64468] [PATCH] services: dhcp-client-service-type: Support DDNS Lilah Tascheter via Guix-patches via
2023-10-11 17:14 ` Ludovic Courtès
2023-10-12 20:23   ` Lilah Tascheter via Guix-patches
2023-10-14 17:45     ` bug#64468: " Ludovic Courtès

Code repositories for project(s) associated with this public inbox

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

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 read-only IMAP folder(s) and NNTP newsgroup(s).