Perhaps something like this would be acceptable?
diff --git a/config b/config
index 4543d90..83d931f 100644
--- a/config
+++ b/config
@@ -446,9 +446,9 @@ trackoffset=0
# to the track counter ($tracknum) in case a multi disc release has
# been detected.
# Possible values: 0 - off, 1 - on
-# Default: off
+# Default: on
-addtrackoffset=0
+#addtrackoffset=0
# infolog: Log certain operations to file
# (e.g. system calls, creation of dirs/files)
diff --git a/ripit.pl b/ripit.pl
index 9515f6d..018db99 100755
--- a/ripit.pl
+++ b/ripit.pl
@@ -758,7 +758,6 @@ $verbose = $pverbose if defined $pverbose;
$vatag = $pvatag if defined $pvatag;
#
# And for the negatable options.
-$addtrackoffset = $paddtrackoffset if defined $paddtrackoffset;
$accuracy = $paccuracy if defined $paccuracy;
$archive = $parchive if defined $parchive;
$config = $pconfig if defined $pconfig;
@@ -779,6 +778,10 @@ $submission = $psubmission if defined $psubmission;
$underscore = $punderscore if defined $punderscore;
$utftag = $putftag if defined $putftag;
$wav = $pwav if defined $pwav;
+if($mp) {
+ $addtrackoffset = 1;
+}
+$addtrackoffset = $paddtrackoffset if defined $paddtrackoffset;
#
########################################################################
#
The if() code did not have any effect when testing it because the
/etc/ripit/config file had addtrackoffset=0. Why is the default to
disable this? But I am unable to get the if() test to take effect. No
idea why.
Or more fundamentally, why is this optional at all, given that ripit
select the wrong titles without it when using musicbrainz?
--
Vennlig hilsen
Petter Reinholdtsen
