[Buildroot] [PATCH 1/1] busybox: udhcpc create leases file

Peter Korsgaard peter at korsgaard.com
Fri May 1 21:13:05 UTC 2015


>>>>> "Arnout" == Arnout Vandecappelle <arnout at mind.be> writes:

Hi,

 >> > Sorry, I can add that.  In general, it adds the leases functionality
 >> > that mimics full dhclient behavior.  This file can then be used at a
 >> > system level to know least expiration and other server provided
 >> > information.
 >> 
 >> Ok, good. Is that really something we want to enforce for everyone using
 >> udhcpc with Buildroot?

 >  I don't see why not. It's creating a small file in /tmp, that's all. Except for
 > the appending, of course, otherwise a long-running system could just fill up
 > /tmp with leases. Overwriting has the disadvantage that there's a race condition
 > when the file is truncated at the same time that another process is reading from
 > it, but I don't think the complexity of fixing that race is worth the bother.

True. I would like to see this (and other non-essential things like
E.G. avahi-autoipd) move to hooks though.

I wonder if the dhclient format is a sensible choice. Is that something
any "standard" programs can work with? It doesn't seem very easy to
parse in shell scripts. Maybe just shell compatible NAME="<val>" strings
would be better?

-- 
Venlig hilsen,
Peter Korsgaard 


More information about the buildroot mailing list