Re: PERFORCE change 136345 for review (fwd)

On 2008-02-27 11:54, Robert Watson wrote:
> 
> Tomas,
> 
> Following a CVS update from the Arla anoncvs, I needed the attached patch 
> to build on FreeBSD.  I still have the following further local changes in 
> my tree still:
> 
> - Remove nnpfs_deb.h include from fs_local.h.
> 
> - Add /usr/include to various paths so that machine/whatever.h includes
>   properly.
> 
> - #ifdef NNPFS_DEBUG around local variables in nnpfs_lk_info to avoid unused
>   variables warning/error.

I've been playing with this some tonight.  My current thought about
dealing with the "machine/whatever.h" includes is to copy the
pertinent headers out of the kernel sources (via $SYS) at the same
time configure generates vnode_if.h.  Any thoughts about that
approach?  It's kinda unfortunate that these headers aren't inside
a machine/ directory in /sys;  then we could -I these directly from 
/sys.  Does anyone have any idea how this used to work?

I haven't loked at the NNPFS_DEBUG related issues yet, but I will.  

-- 
Alec Kloss  [email protected]   IM: [email protected]
PGP key at http://pgp.mit.edu:11371/pks/lookup?op=get&search=0xA241980E
"No Bunny!" -- Simon, from Frisky Dingo

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.5 (FreeBSD)

iD8DBQFHx4dC2s33paJBmA4RAtqiAJ4zubzpHxbr4fitPQlkDFAuDtsXpACeLwrf
7r5RDCByn7/WG4DQBYlQkrs=
=RD4B
-----END PGP SIGNATURE-----