Re: CVS commit: pkgsrc/devel/ocaml-cmdliner

[Joerg Sonnenberger <joerg%britannica.bec.de@localhost>]

On Wed, Mar 04, 2015 at 08:06:50PM +0100, Alistair Crooks wrote:
> On Tue, Mar 03, 2015 at 11:43:45PM +0000, Sevan Janiyan wrote:
> > Module Name:        pkgsrc
> > Committed By:       sevan
> > Date:               Tue Mar  3 23:43:45 UTC 2015
> > 
> > Modified Files:
> >     pkgsrc/devel/ocaml-cmdliner: Makefile
> > 
> > Log Message:
> > Include mk/bsd.fast.prefs.mk, it's needed for setting MACHINE_ARCH correctly.
> > Resolves packing issue on FreeBSD/AMD64
> > 
> > Reviewed by jaapb@ wiz@
> 
> Thanks!
> 
> Is there any reason to use "bsd.fast.prefs.mk" in preference to
> "bsd.prefs.mk"?  bsd.prefs.mk is what we do almost everywhere else,
> and is mentioned in (at least) section 19.1.2 of the pkgsrc guide.

If there is a chance that either was already included, bsd.fast.prefs.mk
is preferred as it requires less parse time. In this case, bsd.prefs.mk
is likely the better choice given the commit message.

Joerg