opentechinstitute / commotion-feed

Feed of Commotion components to be pulled in by OpenWRT
GNU Affero General Public License v3.0
5 stars 11 forks source link

Role our own wpad-mini package with IBSS-RSN and test to make sure it works. #27

Closed jheretic closed 10 years ago

jheretic commented 10 years ago

We can save some space by using wpad-mini (as is the default usually in OpenWRT) rather than wpad. The only reason we don't is because wpad-mini is not compiled with support for IBSS-RSN. However if it works to just add a compile-time option, we could save a good 500K or so, as we don't need all of the WPA-Enterprise stuff that is included in wpad.

hawkinswnaf commented 10 years ago

I'm happy to take this.

On 11/04/2013 04:05 PM, Josh King wrote:

We can save some space by using wpad-mini (as is the default usually in OpenWRT) rather than wpad. The only reason we don't is because wpad-mini is not compiled with support for IBSS-RSN. However if it works to just add a compile-time option, we could save a good 500K or so, as we don't need all of the WPA-Enterprise stuff that is included in wpad.

— Reply to this email directly or view it on GitHub https://github.com/opentechinstitute/commotion-feed/issues/27.

jheretic commented 10 years ago

You got it!

westbywest commented 10 years ago

In lieu of some discussion about Commotion-OpenWRT possibly supporting BATMAN-ADV, or some other routing schema similar to Libremesh, do note that would require the modified wpad-mini package retain support for batctl.

hawkinswnaf commented 10 years ago

@technosopher and I built an image with wpad-mini (+IBSS RSN). Everything works as expected and the node meshes just fine. Here is a comparison of the sizes for the resulting wpad binary:

Mini: -rwxr-xr-x 1 root root 395109 Nov 4 23:14 /usr/sbin/wpad

Full: -rwxr-xr-x 1 root root 715717 Sep 25 14:28 /usr/sbin/wpad

Tomorrow @technosopher and I are going to package this up in a pull request and we'll submit.

On 11/04/2013 05:05 PM, Ben West wrote:

In lieu of some discussion about Commotion-OpenWRT possibly supporting BATMAN-ADV, or some other routing schema similar to Libremesh, do note that would require the modified wpad-mini package retain support for batctl.

— Reply to this email directly or view it on GitHub https://github.com/opentechinstitute/commotion-feed/issues/27#issuecomment-27727466.

hawkinswnaf commented 10 years ago

Just FYI:

Here is an output from df -h:

Mini: Filesystem Size Used Available Use% Mounted on rootfs 1.7M 324.0K 1.4M 19% /

Full: Filesystem Size Used Available Use% Mounted on rootfs 1.6M 376.0K 1.3M 23% /

On 11/04/2013 06:56 PM, Will Hawkins wrote:

@technosopher and I built an image with wpad-mini (+IBSS RSN). Everything works as expected and the node meshes just fine. Here is a comparison of the sizes for the resulting wpad binary:

Mini: -rwxr-xr-x 1 root root 395109 Nov 4 23:14 /usr/sbin/wpad

Full: -rwxr-xr-x 1 root root 715717 Sep 25 14:28 /usr/sbin/wpad

Tomorrow @technosopher and I are going to package this up in a pull request and we'll submit.

On 11/04/2013 05:05 PM, Ben West wrote:

In lieu of some discussion about Commotion-OpenWRT possibly supporting BATMAN-ADV, or some other routing schema similar to Libremesh, do note that would require the modified wpad-mini package retain support for batctl.

— Reply to this email directly or view it on GitHub https://github.com/opentechinstitute/commotion-feed/issues/27#issuecomment-27727466.

jheretic commented 10 years ago

@westbywest I don't think batctl is dependent on wpa_supplicant, is it? Also, isn't batctl just a nicety? Last I checked, it was just a utility for more easily doing stuff you could do through sysfs anyway.

hawkinswnaf commented 10 years ago

Pull requests are in https://github.com/opentechinstitute/commotion-openwrt/pull/64

areynold commented 10 years ago

opentechinstitute/commotion-openwrt#64 was merged