Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Merge Review: cdrdao https://bugzilla.redhat.com/show_bug.cgi?id=225639 bugzilla@xxxxxxxxxx changed: What |Removed |Added ---------------------------------------------------------------------------- Version|devel |rawhide ------- Additional Comments From denis@xxxxxxxxxxxxx 2007-12-21 05:39 EST ------- Created an attachment (id=290225) --> (https://bugzilla.redhat.com/attachment.cgi?id=290225&action=view) Remove bogus error checking Ok, let's get this over with :-) Looking good. 2 things: first, I'd add '--with-scglib=pkg' to the configure line to make it explicit that we're building with the scsi library shipped with cdrdao. Ok it's a minor point. second, could you add the patch i just attached ? I fixes the scanbus command on F-8 and devel. Patch should be applied at the end of current patch list. Otherwise package looks good. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug, or are watching the QA contact. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review