Re: CVS commit: src/sys/arch/x86/include

[Andrew Doran <ad%netbsd.org@localhost>]

On Sat, May 10, 2008 at 12:45:37AM +0200, Christoph Egger wrote:

> Christoph Egger wrote:
> >Andrew Doran wrote:
> >>On Sat, May 10, 2008 at 12:31:53AM +0200, Christoph Egger wrote:
> >>
> >>>Andrew Doran wrote:
> >>>>Something is wrong with this picture... Have I made a mistake with the
> >>>>commands below?
> >>>I think, someone did not compile-test :)
> >>
> >>What error do you get? Read the diffs, I think that the CVS server has
> >>corrupted the change.
> >
> >./x86/i82489reg.h:111:1: error: "LAPIC_ID_MASK" redefined
> >./x86/i82489reg.h:39:1: error: this is the location of the previous 
> >definition
> 
> r1.7 fixed this.
> 
> looking at the annotate of r1.5, I see the duplicates are there since 
> r1.1 committed by fvdl.

Ah, I missed the line numbers. I did compile it, but it reported expected
errors and nothing about the cpp constants.

Sorry about the noise.

Thanks,
Andrew