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

Hi,

On Thu, Sep 29, 2011 at 6:20 PM, Craig Rodrigues <rodrigc@crodrigues.org> w=
rote:
> On Wed, Sep 28, 2011 at 1:15 AM, Jaakko Heinonen <jh@freebsd.org> wrote:
>>
>> I think that using the FEATURE() macro and feature_present(3) might be
>> more appropriate for this.
>>
>> Thanks.
>> --
>> Jaakko
>>
>
> Oh, OK. =A0I was unfamiliar with these API's because they are new in Free=
BSD 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 =3D 0, block_type =3D 0, cdopen =3D 0, dvdrw =3D 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@crodrigues.org
>
> _______________________________________________
> freebsd-current@freebsd.org mailing list
> http://lists.freebsd.org/mailman/listinfo/freebsd-current
> To unsubscribe, send any mail to "freebsd-current-unsubscribe@freebsd.org=
"
>
_______________________________________________
freebsd-current@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-current
To unsubscribe, send any mail to "freebsd-current-unsubscribe@freebsd.org"

