Bug 343 - songs requiring transcoding fail unless lame is installed
: songs requiring transcoding fail unless lame is installed
Status: CLOSED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: Formats
: 5.x or older
: PC All
: P2 major (vote)
: ---
Assigned To: KDF
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2004-06-03 18:07 UTC by Vidur Apparao
Modified: 2009-09-08 09:21 UTC (History)
1 user (show)

See Also:
Category: ---


Attachments
checks formats and allows fallback if lame isn't found. (1.72 KB, patch)
2004-06-07 14:37 UTC, KDF
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Vidur Apparao 2004-06-03 18:07:10 UTC
For a Squeezebox that is rate limited (wireless Squeezeboxes, for example),
songs requiring transcoding (wma, aac, ogg, etc.) fail to play if LAME is not
installed. Rather than failing, we should fall back to streaming uncompressed PCM.

Dean suggests that line 1060 of Source.pm should be:

return 1 if !Slim::Utils::Misc::findbin('lame');

instead of:

return undef if !Slim::Utils::Misc::findbin('lame');

Kevin, what do you think?
Comment 1 Vidur Apparao 2004-06-03 18:07:52 UTC
Marking this (using the Target Milestone field) as a 5.2 requirement.
Comment 2 KDF 2004-06-04 09:30:59 UTC
returning 1 wouldn't be good, because other players (slimp3) would be returned
ok if lame doesn't exist.  What I need to do is check the logic again in
getConvertCommand so that it allows the unlimited playback if underMax returns
undef.
Comment 3 KDF 2004-06-07 14:37:32 UTC
Created attachment 42 [details]
checks formats and allows fallback if lame isn't found.

I've tested with softsqueeze and triplefat and the server uses the correct
fallback.  Slimp3 should skip songs that MUST be translated to mp3 and will
play mp3 at native bitrate.  Squeezebox will play mp3 at native, and fallback
to conversions to WAV/AIFF if found/enabled.
Comment 4 Vidur Apparao 2004-06-08 11:47:54 UTC
The patch works for me on wireless SQ and S3. In the future, we may want to have
a query method for whether a player deals with uncompressed streams, rather than
the formats[0] stuff that's there now. But let's go with what you have now.
Comment 5 Roy Silvernail 2004-06-09 04:02:01 UTC
*** Bug 358 has been marked as a duplicate of this bug. ***
Comment 6 Vidur Apparao 2004-06-10 16:39:45 UTC
Final fix checked in on 6/10/2004.
Comment 7 Chris Owens 2007-05-24 15:54:08 UTC
This Slimserver bug was fixed a very long time ago, and is now being marked as Closed.  If you're still experiencing this issue, please verify you are running a current version of Slimserver and re-open the bug.
Comment 8 Chris Owens 2008-12-18 11:55:18 UTC
Routine bug db maintenance; removing old versions which cause confusion.  I apologize for the inconvenience.