avoid duplicate nameserver entries in resolv.conf

[thechile]

currently the deemed best nameserver denoted by "use_server" will occur twice in the resolv.conf output which isn't ideal and in some circumstances could incur delayed lookups if it were to fail before the next cronjob to test for availability.

Proposed change ensures each nameserver exists only once.

[kvz]

Thanks for the patch but resolvconf+="nameserver $ns\n" doesn't look like bash?

[thechile]

http://www.tldp.org/LDP/abs/html/ops.html gives example of the += as a numeric variable operator with a footnote http://www.tldp.org/LDP/abs/html/ops.html#FTN.AEN3900 saying it can also be used as string concatenation.. as used in the patch.

[kvz]

Looks like it. But only as of bash 3.1. For the sake for portability I think I'd prefer full="first${surname}". Also would like the code style to match. It also is a little bit more complex, risking bugs. I'll think about it some more and maybe rewrite it to address my concerns. I'll do it on top of your work so you'll still be credited then though

[thechile]

ok, sounds good. Look forward to using the script soon. thanks.