svn commit: r281055 - head/share/mk

[Garrett Cooper]

> On Apr 4, 2015, at 03:43, Baptiste Daroussin <bapt at FreeBSD.org> wrote:
> 
>> On Fri, Apr 03, 2015 at 05:36:21PM -0700, Garrett Cooper wrote:
>>> On Apr 3, 2015, at 17:31, Baptiste Daroussin <bapt at FreeBSD.org> wrote:
>>> 
>>> Author: bapt
>>> Date: Sat Apr  4 00:31:40 2015
>>> New Revision: 281055
>>> URL: https://svnweb.freebsd.org/changeset/base/281055
>>> 
>>> Log:
>>> Do not install scripts multiple time when using bsd.progs.mk
>>> 
>>> Modified:
>>> head/share/mk/bsd.progs.mk
>> 
>> I’m sorry I’ve been lax with this, but unfortunately this doesn’t solve the underlying issue 100%.
>> 
>> 1. You forgot FILES/FILESGROUPS
>> 2. This doesn’t help cases were targets are manually overridden.
>> 3. Etc.
>> 
>> The project I was working on has reached steady state. We should be done with the big hurdle soon-ish (another month or two). I’ll try to resolve it sometime between now and then by putting all of the PROGS logic down into bsd.prog.mk (where it should have been in the first place…), because bsd.progs.mk being included recursively is a layering violation bug that parallel installs expose.
>> 
>> Thanks!
> 
> Well I have hit this one while packaging base because pkg(8) yell when things
> are being packaged twice, so I'm fixing when I see them I plan to fix others
> when I'll catch them, if you prefer I can keep that on my own branch to not make
> collision with your work?

That'd be good. I'll check in the short term minimal hack to fix this today and work on moving bsd.test.mk away from bsd.progs.mk on another branch.

Thanks for all the packaging stuff in base BTW -- really looking forward to it :)!!!