[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