Bug 11467 - Samba for SqueezeOS
: Samba for SqueezeOS
Status: CLOSED FIXED
Product: SB Touch
Classification: Unclassified
Component: OS
: unspecified
: PC Other
: P1 major (vote)
: 7.5.0
Assigned To: Weldon Matt
: TinySC
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2009-03-25 09:18 UTC by Felix Mueller
Modified: 2010-04-08 17:24 UTC (History)
5 users (show)

See Also:
Category: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Felix Mueller 2009-03-25 09:18:44 UTC
Include samba into SqueezeOS platform
Comment 1 Felix Mueller 2009-05-12 13:48:44 UTC
Basic Samba (smbd, nmbd) support added in r5727. It currently uses the same username/pw as ssh.

Open tasks:

- Applet to manage service, i.e. start and stop
- Possibly further reduce size / footprint of binaries (debug info is already stripped)
- Probably use guest access instead of root as user
- Need to have the mounts/volumes correspond to the devices that are plugged into the system. USB and SD, if they exist.
Comment 2 Felix Mueller 2009-05-19 11:19:22 UTC
Reduced binary sizes using a shared library. Next step is to remove remaining printing stuff from binaries.
Comment 3 Blackketter Dean 2009-05-19 22:06:41 UTC
Feliix: Size optimization can probably be put off to later.  Are we
functionally complete?

On Tue, May 19, 2009 at 11:19 AM, <bugs@bugs.slimdevices.com> wrote:

> https://bugs-archive.lyrion.org/show_bug.cgi?id=11467
>
>
> Felix Mueller <felix@slimdevices.com> changed:
>
>           What    |Removed                     |Added
>
> ----------------------------------------------------------------------------
>                  CC|                            |bklaas@slimdevices.com
>
>
>
>
> --- Comment #2 from Felix Mueller <felix@slimdevices.com>  2009-05-19
> 11:19:22 ---
> Reduced binary sizes using a shared library. Next step is to remove
> remaining
> printing stuff from binaries.
>
> --
> Configure bugmail: https://bugs-archive.lyrion.org/userprefs.cgi?tab=email
> ------- You are receiving this mail because: -------
> You are on the CC list for the bug.
>
Comment 4 Felix Mueller 2009-05-20 08:53:14 UTC
I think we need to figure out the correct access rights if we switch from user root to user guest so it can write to sd card and usb stick (reading shouldn't be an issue.)
Comment 5 Blackketter Dean 2009-05-22 17:07:45 UTC
I"m proposing that we have a user managable account (user name and password) changable from an applet.  I've put some of this up on the fab4 wiki and am working on a demo.
Comment 6 Felix Mueller 2009-06-05 06:03:16 UTC
Added simple setup applet that allows enable/disable Samba. Still user: root pw: 1234. No user management yet. r6013
Comment 7 Matt Wise 2009-07-01 18:29:03 UTC
Changes 6315, 6316, 6318, 6319 make the Samba server a bit more useful. It now creates individual shares for each filesystem found in /media (previously it shared all of /media, which broke filesystem size-checking for remote clients). Additionally the hostname of the server changes based on the name setup in SqueezePlay. Lastly, Samba starts and is restarted automatically every time a device is plugged in or removed.
Comment 8 Matt Wise 2009-07-01 18:44:28 UTC
I propose that the "On/Off" setting for Samba create a file in /etc/samba named "enabled". At that point, I'll modify the samba start script to check for that file before starting up. This allows users to disable/enable Samba without needing to modify any udev rules or anything else. 

The option of whether to enable or disable Samba I guess should come up the first time a USB/SD stick is inserted.
Comment 9 Richard Titmuss 2009-07-27 01:11:17 UTC
Reset priority before triage.
Comment 10 Matt Wise 2009-07-27 07:47:36 UTC
Things left to be done now:

- Probably use guest access instead of root as user
- Possibly further reduce size / footprint of binaries (debug info is already
stripped)
Comment 11 Felix Mueller 2009-07-27 08:05:47 UTC
Re: Reducing size. I already reduced size a bit by putting commonly used stuff into a dynamic library (libsmb.so). There is more that could go in there, but when I tried to do that the binaries (nmbd, smbd and smbpasswd) would crash upon executing. But maybe you are more successful.
Comment 12 Felix Mueller 2009-10-08 09:57:59 UTC
Please set default to: off
Comment 13 Matt Wise 2009-10-08 11:24:12 UTC
*** Bug 14666 has been marked as a duplicate of this bug. ***
Comment 14 SVN Bot 2009-10-09 09:55:08 UTC
 == Auto-comment from SVN commit #7838 to the jive repo by mwise ==
 == https://svn.slimdevices.com/jive?view=revision&revision=7838 ==

Bug 11467: Setting Samba to DISABLED by default.
Comment 15 Matt Wise 2009-10-09 09:57:01 UTC
This bug is all-but-closed. We just need updated text for the Samba UI so its more user-friendly. Weldon, let me know when you have this text written up and pass the ticket back to me.
Comment 16 Felix Mueller 2009-11-23 06:45:35 UTC
The standalone samba applet has been integrated with "USB & SD Storage" in the "Advanced" menu. You can find it under "Sharing". The wording has been taken from the wiki page:

http://embargo.wiki.slimdevices.com/index.php/TinySC#Version_1_2

- Sharing can be enabled / disabled
- A sharing account name / password can be defined

Please check it out on a Fab4 with recent firmware.

Please also see bug 11952 - TinySC settings and related management.
Comment 17 Chris Owens 2010-04-08 17:24:48 UTC
This bug has been marked fixed in a released version of Squeezebox Server or the accompanying firmware or mysqueezebox.com release.

If you are still seeing this issue, please let us know!