[PATCH] Move resolv.conf updating from DHCP Client to net_server

[vegardw]

I'm currently working on a patch that will move the updating of resolv.conf from the DHCP client to the net_server.

The way I've invisioned it is to define a separate kMsgConfigureResolver message that is sent to the net_server from the DHCP client. The rationale for creating a new message and not adding the info to the existing kMsgConfigureInterface message is that the resolver settings is system wide, not really Interface-specific.

This could also be used by when the network preflet need to update resolv.conf instead of the preflet writing the file by itself as it does now.

Does this seem like a good way to handle the resolv.conf updating?

[vegardw]

Initial attempt attached. Please review.

[axeld]

Looks good, thanks! I think I would make the third argument of _ParseOptions() a reference, though, so that you don't have to check for its validity - and if one want to choose between different offers, you will have to take its data into account, anyway.

There are a few style issues, though, like '{' on the next line, missing space between "for" and '(', extra spaces after '(', etc. Also, I think I would rename fResolverConfig to fResolverConfiguration in order to make it fit better to the existing fConfiguration.

[vegardw]

Updated patch

[vegardw]

Attached updated patch with issued noted by axeld addressed

[axeld]

Thanks for your work! I've applied it now in hrev36135.

There were some minor coding style issues (missing spaces around operators, 80 character line limit, extra curly braces), and I've somewhat simplified NetServer::_ConfigureResolver().