Bug 12363 - SC downloads firmware and deletes it when .sha file is missing
: SC downloads firmware and deletes it when .sha file is missing
Status: RESOLVED WONTFIX
Product: SB Touch
Classification: Unclassified
Component: Setup
: unspecified
: PC Windows XP
: -- normal (vote)
: ---
Assigned To: Michael Herger
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2009-06-12 14:00 UTC by Jim McAtee
Modified: 2009-09-08 09:21 UTC (History)
3 users (show)

See Also:
Category: ---


Attachments
log (3.95 KB, text/plain)
2009-06-12 14:17 UTC, Jim McAtee
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Jim McAtee 2009-06-12 14:00:22 UTC
This happened with both r6079 and r6082.  SC downloads the new firmware and then instantly deletes it.  Even with server.update at the DEBUG level there's no mention of downloading the firmware .version or .bin file in the server log.

SC should be smart enough not to download a 27 MB .bin file if there's no .sha file.

[09-06-12 14:44:56.6082] Slim::Networking::SimpleAsyncHTTP::onError (230) Failed to connect to http://update.squeezenetwork.com/update/firmware/7.4/fab4_7.4_r6082.bin.sha (404 Not Found)
[09-06-12 14:44:56.6200] Slim::Utils::Firmware::downloadAsyncError (591) Warning: Firmware: Failed to download http://update.squeezenetwork.com/update/firmware/7.4/fab4_7.4_r6082.bin.sha (404 Not Found), will try again in 10 minutes.
Comment 1 Jim McAtee 2009-06-12 14:17:10 UTC
Created attachment 5334 [details]
log

Sorry, I see that you need to use the player.firmare log setting.  Here's a new log snippet from when the bin file was downloaded and immediately deleted.
Comment 2 Andy Grundman 2009-06-12 15:33:30 UTC
It's working as designed.  Not going to add logic to SC just to deal with a situation where the build server breaks.
Comment 3 Jim McAtee 2009-06-12 15:39:01 UTC
All you'd have to do is download the stupid .sha file first.  But suit yourself - whatever requires less effort.
Comment 4 Matt Wise 2009-06-12 15:42:36 UTC
I wouldn't add too much additional logic ... but it might be smart to have SC check first for the .sha files and then fail if they don't exist. In this particular case the user would have been rate-limited on his downloads anyways by the update server ... but still, it seems like it might be checking things in the wrong order.

On a side note, all the files were in place except for the .sha for the .bin file itself ... OOPS. I've fixed the build script and am re-running it now.
Comment 5 Jim McAtee 2009-06-19 13:53:35 UTC
This is broken again with fw r6189.
Comment 6 Matt Wise 2009-06-19 13:56:29 UTC
JJ,
   Fixed... but in the future, we don't need to have bugzilla updated  
with notes like this. Please use the forums to let us know when  
something as simple as a firmware update is temporarily hosed.