[Abcde-users] Failback for musicbrainz failure?

[Geoff Sweet]

Ok I was running on a slightly older version. I manually updated to the
latest version on the site and the failback method works correctly. However
I may have discovered a bug.  It appears that the new version of
abcde-musicbrainz-tool has this code, specifically line 101:

  if ($release->release_event_list()->count() > 0) {
   my @events = @{$release->release_event_list()->events()};
            my $rel_year = "";
   $rel_year =  substr($events[0]->date(),0,4);
}

And it generates this error:

Global symbol "$rel_year" requires explicit package name (did you forget to
declare "my $rel_year"?) at Downloads/abcde-2.8/abcde-musicbrainz-tool line
126.
Execution of Downloads/abcde-2.8/abcde-musicbrainz-tool aborted due to
compilation errors.

So I altered the tool to like so:

  my $rel_year = "";
  if ($release->release_event_list()->count() > 0) {
   my @events = @{$release->release_event_list()->events()};
   $rel_year =  substr($events[0]->date(),0,4);
}

and now the tool runs tidy and the errors go away, the failback works, and
all my missed CD's are happily ripping right now!

-Geoff



On Tue, Nov 7, 2017 at 4:03 PM, Geoff Sweet <geoffrsweet+abcde at gmail.com>
wrote:

> Interesting! I must be working with an older version from my repo (Mint
> 18).  The documentation doesn't reflect that.  Now, is that a failure like
> it receives back a timeout or http 500? Or does it include an "Unknown"
> disk situation?
>
> I will also try to update manually to make sure I am using a version that
> supports the comma delimited list.
>
> Thank you!
>
> -Geoff
>
> On Mon, Nov 6, 2017 at 11:32 PM, Sam Hawkens <sam.hawkens at freenet.de>
> wrote:
>
>> You can specifiy several cddb methods as a comma separated list.
>>
>> # CDDB options
>> # Choose whether you want to use "cddb" and/or "musicbrainz". Default
>> # is "musicbrainz", but both can be specified in a comma delimited list
>> # to be tried sequentially in the event of failure of the first
>> # search.
>> #CDDBMETHOD=musicbrainz
>>
>> Sam
>>
>>
>>
>> Am 05.11.2017 um 05:29 schrieb Geoff Sweet:
>>
>>> Howdy all.
>>>    I'm progressing through a long process of ripping a couple thousand
>>> CD's into a digital collection. I've setup several old laptops to keep the
>>> pace going and abcde is KILLING it for me. Its such a great tool.
>>>
>>>    However some of my CD's are old and I've found several that aren't
>>> listed in musicbrainz but are listed in CDDB.  Currently I just wrap abcde
>>> around some bash scripting so that if it produces a "Unknown Artist"
>>> directory, it kills the autorip process.  I've noticed that in nearly all
>>> these cases the CD is discovered in CDDB.  Instead of creating a "one off"
>>> stack for a laptop that is configured to use CDDB, is there a way for abcde
>>> to fail-back onto CDDB in the event of a "not found" situation with
>>> musicbrainz. I was hoping it was as easy as overloading the config like so:
>>>
>>> CDDBMETHOD=musicbrainz,cddb
>>>
>>> But that doesn't seem to work.
>>>
>>> Any ideas?
>>>
>>> Thanks,
>>> Geoff
>>>
>>>
>>> _______________________________________________
>>> Abcde-users mailing list
>>> Abcde-users at lists.einval.com
>>> https://lists.einval.com/cgi-bin/mailman/listinfo/abcde-users
>>>
>>>
>> _______________________________________________
>> Abcde-users mailing list
>> Abcde-users at lists.einval.com
>> https://lists.einval.com/cgi-bin/mailman/listinfo/abcde-users
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.einval.com/pipermail/abcde-users/attachments/20171107/5539661e/attachment.html>