Citeren Christian Schoenebeck <christian.schoeneb...@gmail.com>:

For some (good) reason you decided to remove wget from BusyBox and replace it by uclient-fetch.
Why not also remove nslookup from BusyBox.
        - If compiled with musl it malfunction with given dns-server

Correct. It will completely ignore the [SERVER] argument from nslookup [HOST] [SERVER] (see [1] & [2]). Either nslookup.c from BusyBox should be changed to allow overriding the list of nameservers from /etc/resolv.conf (which are the only ones used with the musl library) or patched to allow this argument only when compiled against a library that actually supports this. I have little hope that musl can be convinced to make such functionality available. Note that in many cases one can workaround this issue by creating a new /etc/resolv.conf with the nameserver(s) to use. By default dnsmasq will not use this file, so this will only influence resolving locally (not on clients connected).

        - use hostip package instead. Tiny and functional

It's not as big as bind-host, but I wouldn't call it tiny either (on ar71xx at least). The hostip package (~30k) requires libsodium (~120k), which is much higher than the additional size of nslookup in BusyBox (which is essentially just a bunch of calls to stuff which is already in the libc). Adding the functionality in BusyBox nslookup will increase the size of BusyBox somewhat, but not nearly by that much. If needed, I can write this.

What do you think?

I think adding this functionality to BusyBox will be far cheaper in terms of size used. Since for many people the [SERVER] argument is not going to be used anyway, so switching to hostip by default seems overkill. For the time being, people that need this functionality should either use a modified /etc/resolv.conf (if possible) or use bind-host instead.

Regards, Arjen

[1] https://dev.openwrt.org/ticket/18333
[2] https://dev.openwrt.org/ticket/20893
_______________________________________________
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel

Reply via email to