* [Buildroot] [PATCH/next v1] package/ifupdown-scripts: send udhcpc to background in case lease is not obtained
@ 2021-02-16 20:20 Peter Seiderer
2021-02-17 16:33 ` Peter Korsgaard
2021-08-01 10:07 ` Thomas Petazzoni
0 siblings, 2 replies; 3+ messages in thread
From: Peter Seiderer @ 2021-02-16 20:20 UTC (permalink / raw)
To: buildroot
- Add the 'udhcpc_opts -b' option to /etc/network/interfaces for the
system dhcp configured interface. With this option ('-b: background if
lease is not obtained') the udhcpc keeps running in case the initial
dhcp requests are not answered immediately.
Signed-off-by: Peter Seiderer <ps.report@gmx.net>
---
package/ifupdown-scripts/ifupdown-scripts.mk | 1 +
1 file changed, 1 insertion(+)
diff --git a/package/ifupdown-scripts/ifupdown-scripts.mk b/package/ifupdown-scripts/ifupdown-scripts.mk
index 5ef032142c..d3135b18b6 100644
--- a/package/ifupdown-scripts/ifupdown-scripts.mk
+++ b/package/ifupdown-scripts/ifupdown-scripts.mk
@@ -28,6 +28,7 @@ define IFUPDOWN_SCRIPTS_DHCP
echo " pre-up /etc/network/nfs_check"; \
echo " wait-delay 15"; \
echo " hostname \$$(hostname)"; \
+ echo " udhcpc_opts -b"; \
) >> $(TARGET_DIR)/etc/network/interfaces
endef
define IFUPDOWN_SCRIPTS_DHCP_OPENRC
--
2.30.0
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [Buildroot] [PATCH/next v1] package/ifupdown-scripts: send udhcpc to background in case lease is not obtained
2021-02-16 20:20 [Buildroot] [PATCH/next v1] package/ifupdown-scripts: send udhcpc to background in case lease is not obtained Peter Seiderer
@ 2021-02-17 16:33 ` Peter Korsgaard
2021-08-01 10:07 ` Thomas Petazzoni
1 sibling, 0 replies; 3+ messages in thread
From: Peter Korsgaard @ 2021-02-17 16:33 UTC (permalink / raw)
To: buildroot
>>>>> "Peter" == Peter Seiderer <ps.report@gmx.net> writes:
> - Add the 'udhcpc_opts -b' option to /etc/network/interfaces for the
> system dhcp configured interface. With this option ('-b: background if
> lease is not obtained') the udhcpc keeps running in case the initial
> dhcp requests are not answered immediately.
> Signed-off-by: Peter Seiderer <ps.report@gmx.net>
> ---
> package/ifupdown-scripts/ifupdown-scripts.mk | 1 +
> 1 file changed, 1 insertion(+)
> diff --git a/package/ifupdown-scripts/ifupdown-scripts.mk b/package/ifupdown-scripts/ifupdown-scripts.mk
> index 5ef032142c..d3135b18b6 100644
> --- a/package/ifupdown-scripts/ifupdown-scripts.mk
> +++ b/package/ifupdown-scripts/ifupdown-scripts.mk
> @@ -28,6 +28,7 @@ define IFUPDOWN_SCRIPTS_DHCP
> echo " pre-up /etc/network/nfs_check"; \
> echo " wait-delay 15"; \
> echo " hostname \$$(hostname)"; \
> + echo " udhcpc_opts -b"; \
Wouldn't it make more sense to change the value of
CONFIG_IFUPDOWN_UDHCPC_CMD_OPTIONS in our default busybox.config instead
of hardcoding it here?
Does this still do the right thing for the fallback to IPv4LL addressing
if avahi-autoipd is enabled?
--
Bye, Peter Korsgaard
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [Buildroot] [PATCH/next v1] package/ifupdown-scripts: send udhcpc to background in case lease is not obtained
2021-02-16 20:20 [Buildroot] [PATCH/next v1] package/ifupdown-scripts: send udhcpc to background in case lease is not obtained Peter Seiderer
2021-02-17 16:33 ` Peter Korsgaard
@ 2021-08-01 10:07 ` Thomas Petazzoni
1 sibling, 0 replies; 3+ messages in thread
From: Thomas Petazzoni @ 2021-08-01 10:07 UTC (permalink / raw)
To: Peter Seiderer; +Cc: buildroot
Hello Peter,
On Tue, 16 Feb 2021 21:20:22 +0100
Peter Seiderer <ps.report@gmx.net> wrote:
> - Add the 'udhcpc_opts -b' option to /etc/network/interfaces for the
> system dhcp configured interface. With this option ('-b: background if
> lease is not obtained') the udhcpc keeps running in case the initial
> dhcp requests are not answered immediately.
>
> Signed-off-by: Peter Seiderer <ps.report@gmx.net>
> ---
> package/ifupdown-scripts/ifupdown-scripts.mk | 1 +
> 1 file changed, 1 insertion(+)
There has been some feedback from Peter Korsgaard on your patch, but
you didn't reply to it. Could you follow-up so that we can move forward?
Thanks a lot!
Thomas
--
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@busybox.net
http://lists.busybox.net/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2021-08-01 10:13 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-02-16 20:20 [Buildroot] [PATCH/next v1] package/ifupdown-scripts: send udhcpc to background in case lease is not obtained Peter Seiderer
2021-02-17 16:33 ` Peter Korsgaard
2021-08-01 10:07 ` Thomas Petazzoni
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.