Re: [Cerowrt-devel] cerowrt-3.10.34-4 dev build released

[Robert Bradley <robert.bradley1@gmail.com>]

[-- Attachment #1.1: Type: text/plain, Size: 1193 bytes --]

On 06/04/2014 01:15, Jim Reisert AD1C wrote:
> On Sat, Apr 5, 2014 at 1:26 PM, Dave Taht <dave.taht@gmail.com> wrote:
>
>> It would have helped to have a backup.
> Yup, hindsight is 50/50 [sic]
>
>> As best I recall we'd had to add 3 lines to the smb.conf.template file, notably
>>
>> wins support = yes
> Yup, did that.  Not working this time around.
>
> Maybe I'll play around some more tomorrow.
>

I noticed last time that Samba tends to bind to localhost only on
CeroWRT, since no interface is set in /etc/config/samba.  For the
moment, you need to add "list interface <interface>" lines to
/etc/config/samba manually:

uci add_list samba.@samba[0].interface=se00
uci add_list samba.@samba[0].interface=sw00
uci add_list samba.@samba[0].interface=sw10
uci commit samba && /etc/init.d/samba restart

This limits Samba to the non-guest interfaces.  I think you should be
able to add interfaces from the GUI too but that seems to be missing
from CeroWRT and OpenWRT.  The attached patch adds it back in (cd /;
patch -p1 to apply it) to  and seems to work fine despite the Wiki at
http://wiki.openwrt.org/doc/uci/samba stating otherwise.

-- 
Robert Bradley


[-- Attachment #1.2: samba.patch --]
[-- Type: text/plain, Size: 810 bytes --]

--- rom/usr/lib/lua/luci/model/cbi/samba.lua
+++ overlay/usr/lib/lua/luci/model/cbi/samba.lua
@@ -28,6 +28,15 @@
         translate("Allow system users to reach their home directories via " ..
                 "network shares"))
 
+net = s:taboption("general", Value, "interface", translate("Served interfaces"))
+net.template = "cbi/network_netlist"
+net.widget   = "checkbox"
+net.nocreate = true
+
+function net.cfgvalue(self, section)
+        return m.uci:get("samba", section, "interfaces")
+                end
+
 tmpl = s:taboption("template", Value, "_tmpl",
 	translate("Edit the template that is used for generating the samba configuration."), 
 	translate("This is the content of the file '/etc/samba/smb.conf.template' from which your samba configuration will be generated. " ..

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 899 bytes --]