svn commit: r229032 - head/sys/modules/cfi

[Marius Strobl]

On Sat, Dec 31, 2011 at 04:26:04PM +0000, Bjoern A. Zeeb wrote:
> 
> On 31. Dec 2011, at 15:59 , Marius Strobl wrote:
> 
> > On Sat, Dec 31, 2011 at 02:05:40PM +0000, Bjoern A. Zeeb wrote:
> >> 
> >> On 30. Dec 2011, at 21:22 , Marius Strobl wrote:
> >> 
> >>> Author: marius
> >>> Date: Fri Dec 30 21:22:10 2011
> >>> New Revision: 229032
> >>> URL: http://svn.freebsd.org/changeset/base/229032
> >>> 
> >>> Log:
> >>> Add header required by cfi_bus_fdt.c.
> >>> 
> >>> Modified:
> >>> head/sys/modules/cfi/Makefile
> >>> 
> >> 
> >> This (most likely) broke at least the arm KB920X config:
> >> 
> >> In file included from @/arm/xscale/ixp425/ixp425var.h:47,
> >>                 from /scratch/tmp/bz/head.universe/sys/modules/cfi/../../dev/cfi/cfi_bus_ixp4xx.c:45:
> >> @/dev/pci/pcivar.h:219:20: error: pci_if.h: No such file or directory
> >> mkdep: compile failed
> >> 
> > 
> > It didn't, the original r228981 just reveals a problem with ixp425var.h.
> > The funny thing is that while it's pretty much clear from your above
> > snippet what's going on I can't reproduce this, also not part of an
> > universe build and apparently also not the tinderbox.
> 
> The tinderbox does not build it if you look at the details;-)

I was referring to the absence of any new breakage report by the
tinderbox after r229032, which IMO should have occurred by now.

> 
> The above is from a make universe (or as I want the errors at the end tinderbox,
> which is a horrible name foe that feature still):
> 
> /usr/bin/time make -s -j16 __MAKE_CONF=/dev/null SRCCONF=/dev/null tinderbox
> 
> 
> 
> > I tried to fix
> > this in r229125, however given that I wasn't able to reproduce this
> > problem I'm not sure that will also fix the build for you.
> 
> I'll let you know tomorrow, when the next build has finished.
> 
> Guten Rutsch!
> 

Dir auch!

Gruesse aus Leipzig,
Marius