Re: CVS commit: src/sbin/ifconfig

[Arnaud Degroote <arnaud.degroote%laas.fr@localhost>]

On 15/Sep - 11:26, Havard Eidnes wrote:
> > On 15/Sep - 00:33, Izumi Tsutsui wrote:
> > > > Modified Files:
> > > >         src/sbin/ifconfig: Makefile.inc
> > > >
> > > > Log Message:
> > > > Change from using ${.CURDIR} to ${NETBSDSRCDIR} because this file
> > > > may be included by different parts of the source tree, in particular
> > > > deep down in the distrib/ tree, where ../.. isn't sufficient to "climb"
> > > > up to the top of the tree.  Fixes the build at least for our arc port.
> > >
> > > Hmm, is it really worth to put pfsync support into install media?
> > > (It might cause floods of ramdisks)
> > >
> > > If not, pfsync.c should be moved into sbin/ifconfig/Makefile.
> >
> > You are right, we doesn't need to put pfsync into install media so we 
> > probably
> > can move it to Makefile. Sorry for breaking the build. Feel free to move
> > pfsync.c from Makefile.inc to Makefile now, I can't do it before a few 
> > hours.
> 
> I took the liberty of committing this suggested change, after
> verifying that a good two handfulls of ports completed their
> builds with the change in place.  They did complete earlier as
> well, so no actual overflow was detected, but why tempt fate?
> 
Thanks for doing it. Regards,

--
Arnaud Degroote
degroote%NetBSD.org@localhost