Bug 787440

Summary: icedax cannot parse -L command line param
Product: [Fedora] Fedora Reporter: Joseph Shraibman <jks>
Component: cdrkitAssignee: Honza Horak <hhorak>
Status: CLOSED NOTABUG QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: unspecified    
Version: 15CC: hhorak
Target Milestone: ---   
Target Release: ---   
Hardware: x86_64   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-02-06 16:42:23 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

Description Joseph Shraibman 2012-02-05 04:41:37 UTC 
Description of problem:
The man page of icedax shows a -L command line paramater.  The built in help shows it.   It does not work.

Version-Release number of selected component (if applicable):
icedax-1.1.11-8.fc15.x86_64

How reproducible:
always

Steps to Reproduce:
1. icedax -L
  
Actual results:
icedax: Bad Option: -L.
use 'icedax -help' to get more information.


Additional info:
Filing under crdkit component because icedax doesn't have its own
Comment 1 Honza Horak 2012-02-06 16:42:23 UTC 
This is because a mode (0 or 1) is needed after -L option:

$ man icedax
...
       -L  cddb mode --cddb
              does  a cddbp album- and track title lookup based on the cddb id.  The parameter
              cddb mode defines how multiple entries shall be handled.

              ┌──────────┬──────────────────────────────────────────┐
              │Parameter │               Description                │
              ├──────────┼──────────────────────────────────────────┤
              │        0 │ interactive mode. The user selects the   │
              │          │ entry to use.                            │
              ├──────────┼──────────────────────────────────────────┤
              │        1 │ first fit mode. The first entry is taken │
              │          │ unconditionally.                         │
              └──────────┴──────────────────────────────────────────┘
...
Comment 2 Joseph Shraibman 2012-02-06 17:29:17 UTC 
I was trying "icedax -L cddb=0" based on the online help which says:

 (-L) cddb=cddbpmode           do cddbp title lookups.
        resolve multiple entries according to cddbpmode: 0=interactive, 1=first entry

but that didn't work.  The online help and the error message could be clearer.
Comment 3 Honza Horak 2012-02-07 08:49:27 UTC 
Help isn't very clear, you're right. You can report suggested changes to upstream: http://alioth.debian.org/projects/debburn

Just to sum it up, it's able to use these variants:
icedax -L 0
icedax -L 1
icedax -cddb 0
icedax -cddb 1
icedax --cddb 0
icedax --cddb 1