pkgsrc-Changes-HG archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

[pkgsrc/trunk]: pkgsrc/mk Remove "CC" from CWRAPPERS_ALIASES.cxx.



details:   https://anonhg.NetBSD.org/pkgsrc/rev/a79d04aa7b12
branches:  trunk
changeset: 643060:a79d04aa7b12
user:      schmonz <schmonz%pkgsrc.org@localhost>
date:      Wed Dec 10 14:29:21 2014 +0000

description:
Remove "CC" from CWRAPPERS_ALIASES.cxx.

Motivation: with "USE_CWRAPPERS=yes" on the default case-insensitive
OS X filesystem, linking the wrappers was failing, stopping the
build. For example:

    ===> Creating toolchain wrappers for mess822-0.58nb3
    ln: .../pkgsrc-current/mail/mess822/work/.cwrapper/bin/cc: File exists
    *** Error code 1

    Stop.

I never noticed this with the old wrappers, but they've also been
failing to link "CC" as an alias to "c++"; it's just that the failure
has been silent and I've never noticed a build failure for lack of
a "CC" wrapper, despite there probably never having been such a
wrapper.

Given that jperkin's SmartOS bulk build with this change produced
results similar to last week's non-cwrappers trunk build, we can
conclude that across platforms, the need for a "CC" wrapper is very
limited and we needn't try (and sometimes fail) to create one by
default.

diffstat:

 mk/cwrappers.mk |  4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diffs (18 lines):

diff -r ef04ef7410fe -r a79d04aa7b12 mk/cwrappers.mk
--- a/mk/cwrappers.mk   Wed Dec 10 14:05:48 2014 +0000
+++ b/mk/cwrappers.mk   Wed Dec 10 14:29:21 2014 +0000
@@ -1,4 +1,4 @@
-# $NetBSD: cwrappers.mk,v 1.16 2014/12/09 12:50:24 joerg Exp $
+# $NetBSD: cwrappers.mk,v 1.17 2014/12/10 14:29:21 schmonz Exp $
 #
 # This Makefile fragment implements integration of pkgtools/cwrappers.
 
@@ -26,7 +26,7 @@
 
 CWRAPPERS_ALIASES.as=          as
 CWRAPPERS_ALIASES.cc=          cc gcc clang
-CWRAPPERS_ALIASES.cxx=         c++ g++ CC cxx clang++
+CWRAPPERS_ALIASES.cxx=         c++ g++ cxx clang++
 CWRAPPERS_ALIASES.cpp=         cpp clang-cpp
 CWRAPPERS_ALIASES.f77=         f77 g77
 CWRAPPERS_ALIASES.imake=       imake



Home | Main Index | Thread Index | Old Index