[pkgsrc/trunk]: pkgsrc/devel/gnustep-make gnustep-make: Add comments about co...

[gdt <gdt%pkgsrc.org@localhost>]

details:   https://anonhg.NetBSD.org/pkgsrc/rev/1343e39251e8
branches:  trunk
changeset: 373376:1343e39251e8
user:      gdt <gdt%pkgsrc.org@localhost>
date:      Sun Dec 31 16:39:41 2017 +0000

description:
gnustep-make: Add comments about compiler selection

This package overrides the user-settable variable PKGSRC_COMPILER,
which is irregular.  Add comments/\todos explaining this.

(This is a comment-only change.)

diffstat:

 devel/gnustep-make/gnustep.mk |  13 ++++++++++++-
 1 files changed, 12 insertions(+), 1 deletions(-)

diffs (29 lines):

diff -r 136a960673ce -r 1343e39251e8 devel/gnustep-make/gnustep.mk
--- a/devel/gnustep-make/gnustep.mk     Sun Dec 31 15:59:24 2017 +0000
+++ b/devel/gnustep-make/gnustep.mk     Sun Dec 31 16:39:41 2017 +0000
@@ -1,4 +1,4 @@
-#      $NetBSD: gnustep.mk,v 1.25 2015/03/04 16:51:57 tnn Exp $
+#      $NetBSD: gnustep.mk,v 1.26 2017/12/31 16:39:41 gdt Exp $
 
 .if !defined(GNUSTEP_MK)
 GNUSTEP_MK=            #defined
@@ -9,8 +9,19 @@
 .include "../../mk/bsd.options.mk"
 
 .if empty(PKG_OPTIONS:Mfragile)
+# It is necessary to use clang to build gnustep packages.
+# PKGSRC_COMPILER is a user-settable variable and may not be set by
+# packages, but there is not a package-settable way to force a
+# different compiler.  For now, abuse it, realizing that this will
+# also drop any ccache or distcc set by the user.
+# \todo Stop abusing PKGSRC_COMPILER.
 PKGSRC_COMPILER=       clang
+
+# \todo Explain.
 ONLY_FOR_COMPILER=     clang
+
+# \todo Explain.  Is this about forcing clang as the compiler, or
+# about using libraries in the clang package, or ?
 BUILDLINK_API_DEPENDS.clang+=   clang>=3.1
 DEPENDS+=              clang-[0-9]*:../../lang/clang
 .endif