svn commit: r290220 - head/share/mk

[NGie Cooper]

> On Oct 30, 2015, at 20:30, Bryan Drewery <bdrewery at FreeBSD.org> wrote:
> 
>> On Oct 30, 2015, at 19:18, NGie Cooper <yaneurabeya at gmail.com> wrote:
>> 
>>> On Fri, Oct 30, 2015 at 7:07 PM, Bryan Drewery <bdrewery at freebsd.org> wrote:
>>> Author: bdrewery
>>> Date: Sat Oct 31 02:07:30 2015
>>> New Revision: 290220
>>> URL: https://svnweb.freebsd.org/changeset/base/290220
>>> 
>>> Log:
>>> Don't hide stderr when checking ${CC} --version.
>>> 
>>> This can have important debugging information such as 'cc: not found' or
>>> 'ccache: error: Could not find compiler "cc" in PATH'.
>> 
>> Won't this spam installworld/distribution a lot?
> 
> Not for me.
> 
> What do you expect from stderr that is non-fatal? The only cases I've
> had so far resulted in the .error triggering later due to not being able
> to determine compiler type.
> 
> Also, install depending on the compiler type is wrong. We need to
> eliminate the cases where it determines SUBDIR or FILES to fix this. I'd
> like to effectively set CC=false during install but can't while those
> checks exist.

I’ll have to run the build again. I’ve seen a lot of noise without it redirected to /dev/null in the past, but it’s likely ok.

I’d like CC=false to work too with installworld/distribution. I think it’s really bad that it’s not the default variable right now because it could cause things to get recompiled at install, which inevitably leads to confusing emails later from end-users.