Re: CVS commit: src/sys

2015-08-31 Thread Masao Uebayashi
sys/netinet/in.c calls a function in sys/net/if_llatbl.c sys/net/if_llatbl.c calls a function in sys/net/if_arp.c This doesn't look very good from modularity POV. Maybe if_llatbl.c should not directly refer to if_arp.c functions.

Re: CVS commit: src/sys

2015-08-31 Thread Masao Uebayashi
Off course you can do "file if_llatbl.c a|b|c" and leave the burden of fixing modularity for future people. :)

Re: CVS commit: src/sys

2015-08-31 Thread Ryota Ozaki
On Mon, Aug 31, 2015 at 11:59 PM, Masao Uebayashi wrote: > sys/netinet/in.c calls a function in sys/net/if_llatbl.c > sys/net/if_llatbl.c calls a function in sys/net/if_arp.c > > This doesn't look very good from modularity POV. Maybe if_llatbl.c > should not directly refer to

Re: CVS commit: src/external/gpl3

2015-08-31 Thread David Laight
On Tue, Aug 11, 2015 at 10:26:43PM +0400, Alan Barrett wrote: > On Tue, 11 Aug 2015, David Laight wrote: > >The system should probably clean 'turds' from both /tmp and /var/tmp. > > That would be surprising, at least to me. > > I rely on /var/tmp/vi.recover being persistent, and I often > use

Re: CVS commit: src/usr.bin/config

2015-08-31 Thread David Laight
On Sat, Aug 29, 2015 at 12:03:51PM +0900, Masao Uebayashi wrote: > Such a hack is needed because config(1) has to generate rules > explicitly for each *.[cS]. If you try to override a rule (e.g. > compile this pmap_bootstrap.c with ${NOPROF_C}), it will be a > duplicated rule. > > If *.[cS] ->