Skip to content

Dockerfile: Fix cdrdao version detection & discid issues with trixie update#689

Open
sjrct wants to merge 3 commits into
whipper-team:dockerfile-trixiefrom
sjrct:dockerfile-trixie-fix
Open

Dockerfile: Fix cdrdao version detection & discid issues with trixie update#689
sjrct wants to merge 3 commits into
whipper-team:dockerfile-trixiefrom
sjrct:dockerfile-trixie-fix

Conversation

@sjrct

@sjrct sjrct commented Jun 14, 2026

Copy link
Copy Markdown
Contributor

When I tested #668, I ran into the issues reported there and an issue with the python discid package not being installed.

This fixes those issues as follows:

  • Change the version detection to break at the first space, rather than include everything after the version number as well
  • Re-add the discid pip package. I don't believe there is a debian package, at least not that I could find.
  • Run pip install . rather than setup.py directly to fix the 0.0.0 version issue
    • Remove unused PyGObject that was causing that to fail

I was able to rip a CD with this branch

sjrct added 2 commits June 14, 2026 15:25
Newer crdo gives "1.2.5 - (C) Andreas Mueller <andreas@daneb.de>" when
we only want the version number, so change the regex to break at the
first space.

Signed-off-by: Chris Harding <sjrct@users.noreply.github.com>
I don't believe there is a debian package for this, at least not that I
could see.

Signed-off-by: Chris Harding <sjrct@users.noreply.github.com>
Using pip install . rather than running setup.py directly correctly sets
up the whipper version. However, it was then failing when installing
PyGObject/pycairo. But nothing appears to use that, so I removed it.

Signed-off-by: Chris Harding <sjrct@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant