[Buildroot] [PATCH v4] Allow a single DHCP configuration via the system configuration submenu

Peter Korsgaard peter at korsgaard.com
Mon Feb 2 16:36:31 UTC 2015

>>>>> "Jérémy" == Jérémy Rosen <jeremy.rosen at openwide.fr> writes:


 > This patch allows the setup of simple a single interface to be
 > automatically brought up and configured via DHCP on system startup.

 > The interface name can be set via a configuration option. This patch
 > does not support systemd-networkd, any complex network configuration should
 > be done via overlay of /etc/network/interfaces or the relevant networkd
 > configuration file

 > Signed-off-by: Jérémy Rosen <jeremy.rosen at openwide.fr>

 > ---
 > v4 : - changed condition to also depend on ifupdown or busybox
 >        suggestion by Gustavo Zacarias

Sorry for the slow response. We have discussed this during the dev days
and agreed to include it. Committed with some changes, see below.


> +config BR2_SIMPLE_DHCP

We're not completely consist with this, but the new system options are
called BR2_SYSTEM_*, so I've renamed to the option.

> +	string "Name of the physical network interface to run DHCP on"

This is quite long, so I reworded it to:

"Network interface to configure through DHCP"

> +	default ""
 > +	help
 > +	  The name of the network interface to configure automatically.
 > +	  A DHCP request will automatically happen on startup on the selected
 > +	  interface.

And here I reworded it a bit and mentioned an example (eth0).

 > +        echo "# interface file auto-generated by buildroot" >  $(TARGET_DIR)/etc/network/interfaces
 > +        echo                                                >> $(TARGET_DIR)/etc/network/interfaces
 > +        echo "auto lo"                                      >> $(TARGET_DIR)/etc/network/interfaces
 > +        echo "iface lo inet loopback"                       >> $(TARGET_DIR)/etc/network/interfaces
 > +        echo                                                >> $(TARGET_DIR)/etc/network/interfaces
 > +endef

Elsewhere in BR we use syntax like:

( \
  echo foo ; \
  echo bar ; \
) > file

Which is a bit less verbose, so I've changed it to that.

Bye, Peter Korsgaard

More information about the buildroot mailing list