Bug 358 - WMA transcoding selection isn't sticky
: WMA transcoding selection isn't sticky
Status: RESOLVED DUPLICATE of bug 343
Product: Logitech Media Server
Classification: Unclassified
Component: Transcoding
: 5.x or older
: PC Linux (other)
: P2 normal (vote)
: ---
Assigned To: Blackketter Dean
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2004-06-08 14:49 UTC by Roy Silvernail
Modified: 2011-03-16 04:39 UTC (History)
1 user (show)

See Also:
Category: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Roy Silvernail 2004-06-08 14:49:58 UTC
If the server doesn't find the wmadec program on the server machine during
startup, WMA decoding is turned off in File Types, even though it was turned on
when the server was last running.

Linux doesn't have wmadec, and my WMA rules are overridden in
slimserver-convert.conf (using ffmpeg).  Apparently, the decision to turn WMA
transcoding off happens before the alternative *-convert.conf files are read.
Comment 1 Vidur Apparao 2004-06-08 15:17:45 UTC
Kevin, is this not dealt with in your checkin for bug 343? Seems like moving the
checkBin out of the parsing loop will deal with the early disabling of settings.
Comment 2 KDF 2004-06-09 02:26:02 UTC
Unfortunately, its not that easy. I hoped it might, but it doesn't actually fix
anything. Moving the section for checkBin allowed me to have it split off from
other code so that I can hopefully get better code-reuse.  The routine still
checks each profile one at a time, so the first failure causes the profile to
disable even if there is one later that does work.  

One option is to make a hash of arrays so users can choose which conversion to
use.  Another is to add the decoder to the hash key so that each is still
unique.   The option that I'm hoping will work is to simply check for duplicates
and enable if any one of the dupes works. Option 1 or 2 would probably be better
long term, but partly falls into feature enhancement and would require changes
to several modules.
Comment 3 Roy Silvernail 2004-06-09 04:01:59 UTC
Vidur's conversion patch of Tue, 08 Jun 2004 23:01:09 -0700 appears to fix this.
 Since this also appears to dupe bug 343, I'll mark this one as such.

*** This bug has been marked as a duplicate of 343 ***
Comment 4 Chris Owens 2008-12-18 11:50:56 UTC
Routine bug db maintenance; removing old versions which cause confusion.  I apologize for the inconvenience.