volname does not work for dvd's / udf images
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
eject (Ubuntu) |
Incomplete
|
Undecided
|
Dan Kegel |
Bug Description
Binary package hint: eject
The package 'eject' includes the handy utility 'volname'.
eject works with both cd-roms and dvds, but volname doesn't.
Let's fix that. I had a quick look at what it would take.
First, udf seems complex, so rather than extend the simple
hardcoded iso parser in volname.c, it might make sense to
use libdvdread's handy UDFGetVolumeIde
and fall back to the existing iso parser if that fails (e.g. if the disc
is not udf format).
Second, volname currently right-justifies the volume name,
which probably works nicely with isos because their names
are space-padded, but looks funny with dvds, because
UDFGetVolumeIde
to left-justify the output of volname, or remove the padding.
Third, libdvdread suffers from a bug
( http://
which keeps it from reading the volume name properly.
This is fixed upstream, but they haven't don't a release
yet, so we'd want to include a patch.
Fourth, libdvd's DVDOpen() function prints an annoying
warning about encrypted dvds not being supported, even
when you're just getting a volume name. That probably should
be suppressed or printed later, so it's not seen when just
retrieving the volume name.
I'll attach my quickie patches to the eject and libdvdread packages
showing what I have so far. Ideally, someone will reply
and say "Oh, we already fixed all that" :-)
ProblemType: Bug
DistroRelease: Ubuntu 10.04
Package: eject 2.1.5+deb1+
ProcVersionSign
Uname: Linux 2.6.32-25-generic i686
NonfreeKernelMo
Architecture: i386
Date: Sun Nov 21 23:59:49 2010
InstallationMedia: Ubuntu 9.10 "Karmic Koala" - Release i386 (20091028.5)
ProcEnviron:
LANG=en_US.utf8
SHELL=/bin/bash
SourcePackage: eject
tags: | added: patch |
Changed in eject (Ubuntu): | |
status: | Confirmed → Incomplete |
tags: |
added: patch-needswork removed: patch |
I can confirm the patch fixes the problem, although I'll note that this will pull libdvdread into the default install if done.