Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 29 Feb 2016 16:26:36 -0800
From:      Bryan Drewery <bdrewery@FreeBSD.org>
To:        Ngie Cooper <yaneurabeya@gmail.com>
Cc:        "src-committers@freebsd.org" <src-committers@freebsd.org>, "svn-src-all@freebsd.org" <svn-src-all@freebsd.org>, "svn-src-head@freebsd.org" <svn-src-head@freebsd.org>
Subject:   Re: svn commit: r296116 - head/share/mk
Message-ID:  <56D4E1BC.1070508@FreeBSD.org>
In-Reply-To: <CAGHfRMB9kAjc0HTQ6Tynchc=kJ013HXyfMzD1i-7efp6Q1am5w@mail.gmail.com>
References:  <201602262213.u1QMDVjX033086@repo.freebsd.org> <CAGHfRMB9kAjc0HTQ6Tynchc=kJ013HXyfMzD1i-7efp6Q1am5w@mail.gmail.com>

next in thread | previous in thread | raw e-mail | index | archive | help
On 2/29/16 4:12 PM, Ngie Cooper wrote:
> On Fri, Feb 26, 2016 at 2:13 PM, Bryan Drewery <bdrewery@freebsd.org> wrote:
>> Author: bdrewery
>> Date: Fri Feb 26 22:13:31 2016
>> New Revision: 296116
>> URL: https://svnweb.freebsd.org/changeset/base/296116
>>
>> Log:
>>   Remove _manpages and _maninstall targets as they are no longer needed.
>>
>>   r96164 added them to avoid recursing twice with _SUBDIR.  That issue was
>>   fixed in bsd.subdir.mk in r291635 for all targets.
>>
>>   Sponsored by: EMC / Isilon Storage Division
> 
> This probably deserves release notes. I know that we've used some of
> these targets internally at least... someone might have implemented
> their own targets based on this (not so) documented info.
> 

"We" being Isilon haven't used these.

These were very much internal only, committed as such and not
documented. I don't think we need to document it anywhere. Either
someone hits an error with it due to abuse and fixes it or they don't
hit an error on it. It would just be noise in release notes, like
removing any other private API (like static functions).

-- 
Regards,
Bryan Drewery



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?56D4E1BC.1070508>