Re: [OpenWrt-Devel] Discussion: Replace BUSYBOX NSLOOKUP by hostip package

2016-01-25 Thread Felix Fietkau
On 2016-01-24 00:06, Christian Schoenebeck wrote:
> 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
>   - use hostip package instead. Tiny and functional
In my test build, the hostip package is ~30k compressed and depends on
libsodium which is ~115k compressed.
I wouldn't exactly call that tiny...

- Felix
___
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel


Re: [OpenWrt-Devel] Discussion: Replace BUSYBOX NSLOOKUP by hostip package

2016-01-24 Thread Christian Schoenebeck
I prepared ddns-scripts to use hostip if installed. 
I already published but closed the pull because of uclient-fetch to be 
implemented.
Christian
___
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel


Re: [OpenWrt-Devel] Discussion: Replace BUSYBOX NSLOOKUP by hostip package

2016-01-24 Thread Arjen de Korte

Citeren Christian Schoenebeck :

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


[OpenWrt-Devel] Discussion: Replace BUSYBOX NSLOOKUP by hostip package

2016-01-23 Thread Christian Schoenebeck
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
- use hostip package instead. Tiny and functional
What do you think?
___
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel