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

[Jeremy Rosen <jeremy.rosen@openwide.fr>]

> Dear J?r?my Rosen,
> 
> On Fri, 16 Jan 2015 14:08:39 +0100, J?r?my Rosen wrote:
> 
> > +config BR2_SIMPLE_DHCP
> > +	string "Name of the physical network interface to run DHCP on"
> > +	default ""
> > +	depends on !BR2_PACKAGE_SYSTEMD_NETWORKD && (BR2_PACKAGE_BUSYBOX
> > || BR2_PACKAGE_IFUPDOWN)
> > +	help
> > +	  The name of the network interface to configure automatically.
> > +	  A DHCP request will automatically happen on startup on the
> > selected
> > +	  interface.
> > +
> > +	  If left empty, no automatic DHCP requests will take place.
> > +
> > +	  For more complicated network setups use an overlay to overwrite
> > +	  /etc/network/interfaces or add a networkd configuration file.
> > +
> > +comment "automatic network configuration via DHCP is not
> > compatible with networkd"
> > +	depends on BR2_PACKAGE_SYSTEMD_NETWORKD
> > +
> > +comment "automatic network configuration via DHCP needs ifupdown
> > or busybox"
> > +	depends on !(BR2_PACKAGE_BUSYBOX || BR2_PACKAGE_IFUPDOWN)
> 
> I'm still not really happy with this. It's
> really /etc/network/interfaces as a whole that doesn't make sense in
> a
> context where you don't have Busybox ifupdown, or the full blown
> ifupdown, no? Not just the DHCP part of it.
> 


i'm not sure how you want me to correct that...

* not generate network/interfaces at all if ifupdown is not available
* correct the comment sections "generation of /etc/network/interfaces needs  ifupdown"
  (but that does not reflect what the option is when it is available)
* something else

please tell me and i'll send a new patch


> Thomas
> --
> Thomas Petazzoni, CTO, Free Electrons
> Embedded Linux, Kernel and Android engineering
> http://free-electrons.com
>