bin/160979: 9.0 burncd error caused by change to cd0 from acd0

Arnaud Lacombe lacombar at gmail.com
Fri Sep 30 00:01:13 UTC 2011


Hi,

On Thu, Sep 29, 2011 at 6:20 PM, Craig Rodrigues <rodrigc at crodrigues.org> wrote:
> On Wed, Sep 28, 2011 at 1:15 AM, Jaakko Heinonen <jh at freebsd.org> wrote:
>>
>> I think that using the FEATURE() macro and feature_present(3) might be
>> more appropriate for this.
>>
>> Thanks.
>> --
>> Jaakko
>>
>
> Oh, OK.  I was unfamiliar with these API's because they are new in FreeBSD 8. :)
> How about the attached patch?

--- usr.sbin/burncd/burncd.c	(revision 225368)
+++ usr.sbin/burncd/burncd.c	(working copy)
@@ -82,6 +82,15 @@
 	int block_size = 0, block_type = 0, cdopen = 0, dvdrw = 0;
 	const char *dev, *env_speed;

+	if (feature_present("ata_cam")) {
+		printf("\nATA_CAM option is enabled in kernel.\n"
+		    "Install the sysutils/cdrtools port and use cdrecord "
+		    "instead.\n\n"
maybe could you avoid the line break here, this makes the sentence
un-grepable. You are breaking the 80-char limit anyway with the URL.

 - Arnaud

+		    "Please refer to:\n"
+		    "http://www.freebsd.org/doc/handbook/creating-cds.html#CDRECORD\n");
+		exit(1);

>
> --
> Craig Rodrigues
> rodrigc at crodrigues.org
>
> _______________________________________________
> freebsd-current at freebsd.org mailing list
> http://lists.freebsd.org/mailman/listinfo/freebsd-current
> To unsubscribe, send any mail to "freebsd-current-unsubscribe at freebsd.org"
>


More information about the freebsd-current mailing list