Hello jonie,

jonie [2010-08-08 16:37 -0000]:
> - include DVD-RW (in Restricted Overwrite mode it's basically same as
> DVD+RW, I finally found one that I could blank)
> 
> -       if ((cd_media_dvd_plus_rw || cd_media_dvd_plus_rw_dl || 
> cd_media_dvd_ram) && (header[2] & 3) > 1) {
> +       if ((cd_media_dvd_rw || cd_media_dvd_plus_rw || 
> cd_media_dvd_plus_rw_dl || cd_media_dvd_ram) && (header[2] & 3) > 1) {

Hm, that wasn't in your original patch (I took your PPA as reference),
but if you tested it, it looks good. Committed as
a6b03c09617354d729cedada9e63dc2a595c0c62.
 
> - also change this line
> 
> -        if ((header[2] & 3) != 2)
> +        if ((header[2] & 3) < 2)
> 
> DVD-RAM has "other" (=3) status and here a session can't be appended
> either

It was unrelated to the others, so I committed it separately as
45b1d780b960f3bcad84ae8a2702cdcf43b3e5cb.

Thank you!

Martin
-- 
Martin Pitt                        | http://www.piware.de
Ubuntu Developer (www.ubuntu.com)  | Debian Developer  (www.debian.org)

-- 
cdrom_id: Does not recognize reblanked DVD+RW and DVD-RAM
https://bugs.launchpad.net/bugs/581925
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.

-- 
ubuntu-bugs mailing list
[email protected]
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to