Bug 14880 - UI for SD/USB management
: UI for SD/USB management
Status: CLOSED FIXED
Product: SqueezePlay
Classification: Unclassified
Component: UI
: unspecified
: PC Other
: P1 normal (vote)
: 7.5.0
Assigned To: Ben Klaas
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2009-10-21 06:57 UTC by Ben Klaas
Modified: 2010-04-08 17:26 UTC (History)
7 users (show)

See Also:
Category: Feature


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Ben Klaas 2009-10-21 06:57:55 UTC
this is the engineering side for this feature flagged as a 7.5/P1 feature:
- tinySC: flows for attaching/scanning/navigating/ejecting SD and USB devices

Felix, your bug on TinySC UI may or may not encapsulate this. If your TinySC UI bug covers this, this one can probably close.
Comment 1 SVN Bot 2009-11-20 14:27:35 UTC
 == Auto-comment from SVN commit #8139 to the jive repo by bklaas ==
 == https://svn.slimdevices.com/jive?view=revision&revision=8139 ==

Bug: 14880
Description: poach scanning progress strings from server-side strings file
Comment 2 Ben Klaas 2009-11-20 14:32:17 UTC
burn 6 hours on this bug...the remaining hours are still mostly a guess at this point.
Comment 3 Ben Klaas 2009-11-23 08:34:47 UTC
note to self: bug 15027 and bug 15037 are subsets of this one. Make sure that what they request is done as part of this bug
Comment 4 Ben Klaas 2009-11-23 08:35:57 UTC
*** Bug 15027 has been marked as a duplicate of this bug. ***
Comment 5 Ben Klaas 2009-11-23 08:36:17 UTC
*** Bug 15037 has been marked as a duplicate of this bug. ***
Comment 6 SVN Bot 2009-11-23 13:09:52 UTC
 == Auto-comment from SVN commit #29428 to the slim repo by bklaas ==
 == https://svn.slimdevices.com/slim?view=revision&revision=29428 ==

Bug: 14880
Description: add --pidfile command argument
Comment 7 SVN Bot 2009-11-23 13:11:09 UTC
 == Auto-comment from SVN commit #8148 to the jive repo by bklaas ==
 == https://svn.slimdevices.com/jive?view=revision&revision=8148 ==

Bug: 14880
Description: add stopscan argument to init script
add --pidfile argument to scanner.pl command

to stop both a running scanner and the subsequently spawned squeezecenter.pl, run these commands
/etc/init.d/squeezecenter stopscan && sleep 1 && /etc/init.d/squeezecenter stop
Comment 8 SVN Bot 2009-11-24 12:53:47 UTC
 == Auto-comment from SVN commit #8152 to the jive repo by bklaas ==
 == https://svn.slimdevices.com/jive?view=revision&revision=8152 ==

Bug: 14880
Description: First crack at reporting scan progress as scanner is running
work in progress
Comment 9 SVN Bot 2009-11-24 14:48:27 UTC
 == Auto-comment from SVN commit #8153 to the jive repo by bklaas ==
 == https://svn.slimdevices.com/jive?view=revision&revision=8153 ==

Bug: 14880
Description: necessary strings as described in wiki specification for TinySC UI
Comment 10 SVN Bot 2009-11-25 15:12:59 UTC
 == Auto-comment from SVN commit #8155 to the jive repo by bklaas ==
 == https://svn.slimdevices.com/jive?view=revision&revision=8155 ==

Bug: 14880
Description: work in progress-- udevEventListener methods for pushing user through TinySC flow after adding a drive
Comment 11 SVN Bot 2009-11-27 09:43:48 UTC
 == Auto-comment from SVN commit #8161 to the jive repo by felix ==
 == https://svn.slimdevices.com/jive?view=revision&revision=8161 ==

Bug: 14880 
Description: UI for SD/USB management - work in progress
- Added timeout to spinny and check if new device got mounted
-
Comment 12 SVN Bot 2009-11-27 09:46:21 UTC
 == Auto-comment from SVN commit #8162 to the jive repo by felix ==
 == https://svn.slimdevices.com/jive?view=revision&revision=8162 ==

Bug: 14880 
Description: UI for SD/USB management - work in progress - oops, forgot to disable for the moment
-
Comment 13 Felix Mueller 2009-11-27 09:48:37 UTC
Update hours worked
Comment 14 SVN Bot 2009-11-30 05:22:50 UTC
 == Auto-comment from SVN commit #8164 to the jive repo by felix ==
 == https://svn.slimdevices.com/jive?view=revision&revision=8164 ==

Bug: 14880 
Description: UI for SD/USB management - work in progress
- Added 'Unsupported file fomrat error' message window
- Added 'Important notice' message window about ejecting SD/USB drives 
-
Comment 15 SVN Bot 2009-11-30 07:11:01 UTC
 == Auto-comment from SVN commit #8165 to the jive repo by felix ==
 == https://svn.slimdevices.com/jive?view=revision&revision=8165 ==

Bug: 14880 
Description: UI for SD/USB management - work in progress - nicer supported formats list
Comment 16 SVN Bot 2009-11-30 11:36:49 UTC
 == Auto-comment from SVN commit #8167 to the jive repo by bklaas ==
 == https://svn.slimdevices.com/jive?view=revision&revision=8167 ==

Bug: 14880
Description: USB and SD specific strings, as it appears we should be able to discern between the two in most circumstances
Comment 17 SVN Bot 2009-11-30 12:26:56 UTC
 == Auto-comment from SVN commit #8168 to the jive repo by bklaas ==
 == https://svn.slimdevices.com/jive?view=revision&revision=8168 ==

Bug: 14880
Description: more strings for a rainy day
Comment 18 SVN Bot 2009-11-30 15:04:05 UTC
 == Auto-comment from SVN commit #8169 to the jive repo by bklaas ==
 == https://svn.slimdevices.com/jive?view=revision&revision=8169 ==

Bug: 14880
Description: work in progress, USB/SD + TinySC UI
-successfully mounting device starts scanner
-use devName to discern between USB and SD devices
-save mounted device in self.mountedDevices table
-add eject menu item for drive after successful mount
Comment 19 SVN Bot 2009-11-30 20:47:29 UTC
 == Auto-comment from SVN commit #8170 to the jive repo by bklaas ==
 == https://svn.slimdevices.com/jive?view=revision&revision=8170 ==

Bug: 14880
Description: icon style for hm_eject, used only in SB Touch for ejecting removable media
Comment 20 Ben Klaas 2009-11-30 21:03:01 UTC
estimating work half done
Comment 21 SVN Bot 2009-12-01 09:10:34 UTC
 == Auto-comment from SVN commit #8174 to the jive repo by bklaas ==
 == https://svn.slimdevices.com/jive?view=revision&revision=8174 ==

Bug: 14880
Description: more device-specific strings for a snowy day...
Comment 22 SVN Bot 2009-12-01 09:14:51 UTC
 == Auto-comment from SVN commit #8175 to the jive repo by bklaas ==
 == https://svn.slimdevices.com/jive?view=revision&revision=8175 ==

Bug: 14880
Description: Eject doesn't need to be EJECT
Comment 23 SVN Bot 2009-12-02 14:46:41 UTC
 == Auto-comment from SVN commit #8176 to the jive repo by bklaas ==
 == https://svn.slimdevices.com/jive?view=revision&revision=8176 ==

Bug: 14880
Description: work in progress. "Happy Path" device eject...

-- This is still all disabled in firmwares being pushed out. See udevEventHandler() comments for details

-- add eject item to My Music after successful mount
-- add eject confirmation window 
-- currently scanner and server are stopped for all drives before umount is issued on eject (end goal is to have one drive be designated as the tinySC drive and scanner/server will only stop when ejecting that one)
-- a hack was added to wait 3 secs after successful umount to kill the server and remove stale /media/<devName> dir to deal with a race condition (squeezecenter init script spawns server after scanner is stopped)
-- fullscreen popup while drive is being ejected
-- remove menu item after device is successfully ejected
-- abstraction of some reusable methods: _stopScanner() _stopServer() _getItemFromDevName(devName) _mediaDirExists(devName) _checkDriveUnmounted(devName) processRunning(process)

-- Add more necessary strings
Comment 24 SVN Bot 2009-12-03 15:39:20 UTC
 == Auto-comment from SVN commit #8182 to the jive repo by bklaas ==
 == https://svn.slimdevices.com/jive?view=revision&revision=8182 ==

Bug: 14880
Description: the framework for doing "the right thing" with tinySC at squeezeplay startup time
-- add an init method to init some variables
-- add squeezecenterStartupCheck method, which is run (by the meta file via callService), to do the right thing at start time
-- addMountedDevice() for adding elements to self.mountedDevices table
-- writeSCPrefsFile() writes audiodir path to /etc/squeezecenter/prefs.json for tinySC to read
-- readSCPrefsFile() to read the contents of the prefs.json file at startup time
Comment 25 SVN Bot 2009-12-04 14:55:22 UTC
 == Auto-comment from SVN commit #8188 to the jive repo by bklaas ==
 == https://svn.slimdevices.com/jive?view=revision&revision=8188 ==

Bug: 14880
Description: hand over ownership of squeezecenter start/stop to squeezeplay (accompanying tinySC checkin forthcoming)
add some verbose debug messages to startup SC check method
Comment 26 SVN Bot 2009-12-04 14:56:19 UTC
 == Auto-comment from SVN commit #8189 to the jive repo by bklaas ==
 == https://svn.slimdevices.com/jive?view=revision&revision=8189 ==

Bug: 14880
Description: remove execution of media-watcher.pl script from rcS
bump rev to keep poky happy
Comment 27 SVN Bot 2009-12-04 14:59:09 UTC
 == Auto-comment from SVN commit #8190 to the jive repo by bklaas ==
 == https://svn.slimdevices.com/jive?view=revision&revision=8190 ==

Bug: 14880
Description: remove media-watcher script
Comment 28 SVN Bot 2009-12-04 22:24:15 UTC
 == Auto-comment from SVN commit #8196 to the jive repo by bklaas ==
 == https://svn.slimdevices.com/jive?view=revision&revision=8196 ==

Bug: 14880
Description: write both an audiodir and mountpath key/val pair to the prefs.json file
Comment 29 SVN Bot 2009-12-04 22:44:31 UTC
 == Auto-comment from SVN commit #8197 to the jive repo by bklaas ==
 == https://svn.slimdevices.com/jive?view=revision&revision=8197 ==

Bug: 14880
Description: the check of prefs.json by SCApplet should really be mountpath now, not audiodir
Comment 30 Felix Mueller 2009-12-07 01:11:25 UTC
*** Bug 13006 has been marked as a duplicate of this bug. ***
Comment 31 SVN Bot 2009-12-07 22:07:31 UTC
 == Auto-comment from SVN commit #8200 to the jive repo by bklaas ==
 == https://svn.slimdevices.com/jive?view=revision&revision=8200 ==

Bug: 14880
Description: use a settings file to keep track of self.mountedDevices table when applet is taken out of memory
push a warning window when a device is unsafely ejected.
also, more strings!
Comment 32 SVN Bot 2009-12-08 08:46:57 UTC
 == Auto-comment from SVN commit #8201 to the jive repo by bklaas ==
 == https://svn.slimdevices.com/jive?view=revision&revision=8201 ==

Bug: 14880
Description: do not push to tinySC settings window after OK for eject warning on non-tinySC drive attachment
remove ugly hack that kills server after 3 seconds. Not necessary now that scanner is part of the same process
Comment 33 SVN Bot 2009-12-08 22:17:20 UTC
 == Auto-comment from SVN commit #8202 to the jive repo by bklaas ==
 == https://svn.slimdevices.com/jive?view=revision&revision=8202 ==

Bug: 14880
Description: Settings menu items dynamically chosen to be more appropriate based on SBS current state (i.e., don't put a "start server" item if the server is already running)
add confirmation windows before stopping or restarting the server
cleaned up code to deal with lack of separate scanner process
put Eject items on home menu instead of My Music (currently weighted to the top of the menu, which will likely change)
lots more strings! :)
Comment 34 Ben Klaas 2009-12-08 22:17:56 UTC
I've put a ton of work into this and feel it's getting close.
Comment 35 SVN Bot 2009-12-09 07:57:52 UTC
 == Auto-comment from SVN commit #8203 to the jive repo by felix ==
 == https://svn.slimdevices.com/jive?view=revision&revision=8203 ==

Bug: 14880 
Description: Fix typo.
Comment 36 SVN Bot 2009-12-09 08:14:36 UTC
 == Auto-comment from SVN commit #8204 to the jive repo by bklaas ==
 == https://svn.slimdevices.com/jive?view=revision&revision=8204 ==

Bug: 14880
Description: add iconStyles for hm_sdcard and hm_usbdrive. These are used when starting tinySC with more than one already attached drive
Comment 37 SVN Bot 2009-12-10 15:34:04 UTC
 == Auto-comment from SVN commit #8214 to the jive repo by bklaas ==
 == https://svn.slimdevices.com/jive?view=revision&revision=8214 ==

Bug: 14880
Description: Display details of scan progress better on tinySC settings window
more strings (hopefully this is it)
Comment 38 SVN Bot 2009-12-10 15:42:22 UTC
 == Auto-comment from SVN commit #8215 to the jive repo by bklaas ==
 == https://svn.slimdevices.com/jive?view=revision&revision=8215 ==

Fixed Bug: 14880
Description: I'm considering this bug complete for 7.5.0 release.
Please open new bugs for things not working properly in regards to the expected behavior
Comment 39 SVN Bot 2009-12-11 12:51:22 UTC
 == Auto-comment from SVN commit #8216 to the jive repo by bklaas ==
 == https://svn.slimdevices.com/jive?view=revision&revision=8216 ==

Bug: 14880
Description: push unmount failure window to screen after 10 failed seconds of trying to umount a drive
try again item on this window will try "lazy" umount with the -l flag. 
more strings!
Comment 40 SVN Bot 2009-12-11 15:21:23 UTC
 == Auto-comment from SVN commit #8217 to the jive repo by bklaas ==
 == https://svn.slimdevices.com/jive?view=revision&revision=8217 ==

Bug: 14880
Description: Restarting Squeezebox Server was too long to fit across the screen (was cutoff by one char)
shortening to "Starting Squeezebox Server"
Comment 41 SVN Bot 2009-12-14 13:51:22 UTC
 == Auto-comment from SVN commit #8220 to the jive repo by bklaas ==
 == https://svn.slimdevices.com/jive?view=revision&revision=8220 ==

Bug: 14880
Description: read the discovering_files step correctly
Comment 42 SVN Bot 2009-12-14 14:07:50 UTC
 == Auto-comment from SVN commit #8221 to the jive repo by bklaas ==
 == https://svn.slimdevices.com/jive?view=revision&revision=8221 ==

Bug: 14880
Description: move eject icons weight from 1 to 5 to put it below My Music
Comment 43 Chris Owens 2010-04-08 17:26:14 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!