Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

unreadable FLAC file: open failed: > 1 Vorbis comment block found #2701

Open
claudioruiz opened this Issue Sep 25, 2017 · 9 comments

Comments

Projects
None yet
5 participants
@claudioruiz
Copy link

claudioruiz commented Sep 25, 2017

Problem

I've been trying to add up a FLAC album which files are correct after checking with metaflac

Running this command in verbose (-vv) mode:

c@c ~> beet -vv import seeding/album/
user configuration: /home/blah/
data directory: /home/user/.config/beets
plugin paths:
Sending event: pluginload
library database: /home/user/Music/_beets/musiclibrary.blb
library directory: /home/user/Music
Sending event: library_opened
Sending event: import_begin
open failed: > 1 Vorbis comment block found
unreadable file: /home/c/seeding/Album/01. Song.flac
open failed: > 1 Vorbis comment block found

Setup

  • OS: Debian
  • Python version: 2.7.9
  • beets version: 1.4.5
  • Turning off plugins made problem go away (yes/no): no

My configuration (output of beet config) is:

paths:
    default: $albumartist/$album%aunique{}/$track - $title
    singleton: Non-Album/$artist - $title
    comp: Compilations/$album%aunique{}/$track - $title
threaded: yes
art_filename: albumart
fetchart:
    auto: yes
    cover_names: cover front art album folder
    sources: filesystem coverart amazon albumart
    cautious: yes
    minwidth: 0
    google_engine:
    enforce_ratio: no
    maxwidth: 0
    store_source: no
    google_key: REDACTED
    fanarttv_key: REDACTED
color: yes
library: ~/Music/_beets/musiclibrary.blb
ignore: .AppleDouble ._* *~ .DS_Store
original_date: yes

plugins: discogs inline convert fetchart missing duplicates export
directory: ~/Music

import:
    copy: yes
    write: yes
    move: no
    resume: ask
    incremental: yes
    log: ~/Music/_beets/beetslog.log

match:
    strong_rec_thresh: 0.08
per_disc_numbering: yes
pathfields: {}
album_fields: {}
item_fields: {}
convert:
    never_convert_lossy_files: no
    album_art_maxwidth: 0
    format: mp3
    dest:
    auto: no
    threads: 8
    copy_album_art: no
    tmpdir:
    max_bitrate: 500

    paths: {}
    pretend: no
    quiet: no
    formats:
        alac:
            command: ffmpeg -i $source -y -vn -acodec alac $dest
            extension: m4a
        aac:
            command: ffmpeg -i $source -y -vn -acodec aac -aq 1 $dest
            extension: m4a
        opus: ffmpeg -i $source -y -vn -acodec libopus -ab 96k $dest
        mp3: ffmpeg -i $source -y -vn -aq 2 $dest
        flac: ffmpeg -i $source -y -vn -acodec flac $dest
        ogg: ffmpeg -i $source -y -vn -acodec libvorbis -aq 3 $dest
        wma: ffmpeg -i $source -y -vn -acodec wmav2 -vn $dest
    embed: yes
export:
    default_format: json
    json:
        formatting:
            ensure_ascii: no
            indent: 4
            sort_keys: yes
            separators: [',', ': ']
discogs:
    tokenfile: discogs_token.json
    user_token: REDACTED
    apikey: REDACTED
    apisecret: REDACTED
    source_weight: 0.5
duplicates:
    count: no
    full: no
    format: ''
    keys: []
    move: ''
    tag: ''
    path: no
    copy: ''
    tiebreak: {}
    album: no
    strict: no
    checksum: ''
    merge: no
    delete: no
missing:
    count: no
    album: no
    total: no

@sampsyo sampsyo added the needinfo label Sep 30, 2017

@sampsyo

This comment has been minimized.

Copy link
Member

sampsyo commented Sep 30, 2017

Thanks for the report! For this kind of problem, we need sample files to help reproduce the issue (which may exist it Mutagen rather than beets proper). Do you have a freely-distributable file you can share that triggers this?

@dinojr

This comment has been minimized.

Copy link

dinojr commented Nov 2, 2017

I'm experiencing the same problem since a recent upgrade of Ubuntu 17.10. I was formerly using sound-juicer to rip cds to flac, but the flac files it now creates give this error with beet import, though metaflac doesn't seem to see anything wrong with them and mplayer can play them just fine. The flacs created by asunder are still correctly imported by beet. I can provide the metaflac --list output for both files if that helps.

@sampsyo

This comment has been minimized.

Copy link
Member

sampsyo commented Nov 2, 2017

Huh! That's interesting. I'm not sure how useful the metaflac output would be, but maybe you can convert a short silence track and provide that as a sample?

@lazka

This comment has been minimized.

Copy link
Contributor

lazka commented Dec 10, 2017

If someone wants to send me an affected file per mail: reiter.christoph@gmail.com

@janpeeters70

This comment has been minimized.

Copy link

janpeeters70 commented Mar 3, 2019

Hi @lazka I've send a few files, not freely distributable, to your email. Although it's a long time since your post I hope you or someone else can have a look. Thanks, Jan

@lazka

This comment has been minimized.

Copy link
Contributor

lazka commented Mar 13, 2019

@janpeeters70 thanks!

Not sure what to do here. The files have multiple VORBIS_COMMENT blocks with different tags which isn't allowed according to the flac spec. (https://xiph.org/flac/format.html "There may be only one VORBIS_COMMENT block in a stream")

metaflac/vlc/gstreamer seem to use the first when displaying things. metaflac ignores the second one, vlc deletes the second one on changes.

So if you want to fix the files, open in vlc, ctrl+i, change some value, change it back, press save.

@lazka

This comment has been minimized.

Copy link
Contributor

lazka commented Mar 13, 2019

I've opened quodlibet/mutagen#377

@janpeeters70

This comment has been minimized.

Copy link

janpeeters70 commented Mar 13, 2019

Great that you’ve found the problem and opened a issue for this @lazka.

@lazka

This comment has been minimized.

Copy link
Contributor

lazka commented Mar 16, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.