Bug 90 - Cannot specify cdparanoia in conf file - results in syntax error
Summary: Cannot specify cdparanoia in conf file - results in syntax error
Status: CONFIRMED
Alias: None
Product: abcde
Classification: Unclassified
Component: Ripping (show other bugs)
Version: unspecified
Hardware: All Mac OS
: Normal normal
Assignee: Steve McIntyre
URL:
Depends on:
Blocks:
 
Reported: 2018-08-31 00:06 BST by chonnete
Modified: 2018-10-11 17:12 BST (History)
1 user (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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.