Bug 90

Summary: Cannot specify cdparanoia in conf file - results in syntax error
Product: abcde Reporter: chonnete
Component: RippingAssignee: Steve McIntyre <steve>
Status: CONFIRMED ---    
Severity: normal CC: andrew.david.strong
Priority: Normal    
Version: unspecified   
Hardware: All   
OS: Mac OS   

Description chonnete 2018-08-31 00:06:07 BST
I am running abcde v2.9.2 on OSX (El Capitan).

I know that abcde properly reads my .abcde.conf file because it respects my custom naming format specified by OUTPUTFORMAT.

abcde works and I can rip cds, cddb lookup, etc.

However, if I uncomment this line in .abcde.conf:

CDROMREADERSYNTAX=cdparanoia

I receive this error:

/usr/local/bin/abcde: line 521: ((168338 + 150 + ) / 75) - ((150 + ) / 75): syntax error: operand expected (error token is ") / 75) - ((150 + ) / 75)")

... and cddb/musicbrainz lookups fail completely - unknown CD, all entires blank.  Also, the program freezes after the "Is the CD multi-artist" question ...ctrl-c to get out.

I can repeat this behavior with different CDs, etc.

Thanks.
Comment 1 chonnete 2018-09-04 17:14:49 BST
The issue, as it appears to me, is that the expression in line 521 has no value for PREGAP, which is why there is a missing number in this sum:

(168338 + 150 + )

PREGAP is computed in line 520, just prior:

PREGAP=$(echo $OFFSETS | cut -f1 -d' ')

... and a debug echo I put in shows that OFFSETS is empty, which is why PREGAP ends up empty ...
Comment 2 chonnete 2018-10-11 17:12:08 BST
How else may I be of assistance with this issue ?

Please let me know if there is additional information, or tests I could run, that would help with this bug.

Thank you.