Don't install with setuid root by default. Some people claims

it casuses problem.
If you wish to enable LM78/79 feature support, please specify
WITH_SENSOR=yes at building.

4 files changed, 28 insertions, 20 deletions

diff --git a/sysutils/gkrellm/Makefile b/sysutils/gkrellm/Makefile
index 6915db2cd86e..863f909a0fa8 100644
--- a/sysutils/gkrellm/Makefile
+++ b/sysutils/gkrellm/Makefile
@@ -22,10 +22,12 @@ INCLUDES=	gkrellm.h gkrellm_public_proto.h
 MAN1=		gkrellm.1
 DOCS=		COPYRIGHT README Themes.html
 
-# You should aware that GKrellM must be setuid root to obtain
-# temperatures.  If your motherboard doesn't support LM78/79 feature,
-# you may not want this.
-# WITHOUT_SENSOR=yes
+# If you want to support LM78/79 feature, set this to `yes'.  You
+# should aware that GKrellM must be setuid root to obtain
+# temperatures.  Furthermore, this causes problem on some
+# motherboards.  If your motherboard doesn't support LM78/79 feature,
+# you should not enable this.
+# WITH_SENSOR=yes
 
 .include <bsd.port.pre.mk>
 
@@ -36,7 +38,7 @@ ALL_TARGET=	freebsd
 .endif
 
 BINGRP=		kmem
-.if !defined(WITHOUT_SENSOR)
+.if defined(WITH_SENSOR)
 BINOWN=		root
 BINMODE=	4111
 .else
@@ -67,7 +69,7 @@ do-install:
 	${INSTALL_DATA} ${WRKSRC}/${file} ${PREFIX}/share/doc/gkrellm
 .endfor
 .endif
-.if !defined(WITHOUT_SENSOR)
+.if !defined(WITH_SENSOR)
 	@${CAT} ${PKGMESSAGE}
 .endif
 
diff --git a/sysutils/gkrellm/pkg-message b/sysutils/gkrellm/pkg-message
index 70c1f568cdd0..039d8d9dacf8 100644
--- a/sysutils/gkrellm/pkg-message
+++ b/sysutils/gkrellm/pkg-message
@@ -1,9 +1,11 @@
 ****************************************************************************
 
-  You should aware that GKrellM must be setuid root to obtain
-  temperatures.  If your motherboard doesn't support LM78/79 feature,
-  you may not want this.  You can disable it by typing
-  `make install WITHOUT_SENSOR=yes' at the "${PORTSDIR}/sysutils/gkrellm"
+  If you want to support LM78/79 feature,
+  `make install WITH_SENSOR=yes' at the "${PORTSDIR}/sysutils/gkrellm"
   directory to re-make this package.
+  You should aware that GKrellM must be setuid root to obtain
+  temperatures.  Furthermore, this causes problem on some
+  motherboards.  If your motherboard doesn't support LM78/79 feature,
+  you should not enable this.
 
 ****************************************************************************
diff --git a/sysutils/gkrellm2/Makefile b/sysutils/gkrellm2/Makefile
index 6915db2cd86e..863f909a0fa8 100644
--- a/sysutils/gkrellm2/Makefile
+++ b/sysutils/gkrellm2/Makefile
@@ -22,10 +22,12 @@ INCLUDES=	gkrellm.h gkrellm_public_proto.h
 MAN1=		gkrellm.1
 DOCS=		COPYRIGHT README Themes.html
 
-# You should aware that GKrellM must be setuid root to obtain
-# temperatures.  If your motherboard doesn't support LM78/79 feature,
-# you may not want this.
-# WITHOUT_SENSOR=yes
+# If you want to support LM78/79 feature, set this to `yes'.  You
+# should aware that GKrellM must be setuid root to obtain
+# temperatures.  Furthermore, this causes problem on some
+# motherboards.  If your motherboard doesn't support LM78/79 feature,
+# you should not enable this.
+# WITH_SENSOR=yes
 
 .include <bsd.port.pre.mk>
 
@@ -36,7 +38,7 @@ ALL_TARGET=	freebsd
 .endif
 
 BINGRP=		kmem
-.if !defined(WITHOUT_SENSOR)
+.if defined(WITH_SENSOR)
 BINOWN=		root
 BINMODE=	4111
 .else
@@ -67,7 +69,7 @@ do-install:
 	${INSTALL_DATA} ${WRKSRC}/${file} ${PREFIX}/share/doc/gkrellm
 .endfor
 .endif
-.if !defined(WITHOUT_SENSOR)
+.if !defined(WITH_SENSOR)
 	@${CAT} ${PKGMESSAGE}
 .endif
 
diff --git a/sysutils/gkrellm2/pkg-message b/sysutils/gkrellm2/pkg-message
index 70c1f568cdd0..039d8d9dacf8 100644
--- a/sysutils/gkrellm2/pkg-message
+++ b/sysutils/gkrellm2/pkg-message
@@ -1,9 +1,11 @@
 ****************************************************************************
 
-  You should aware that GKrellM must be setuid root to obtain
-  temperatures.  If your motherboard doesn't support LM78/79 feature,
-  you may not want this.  You can disable it by typing
-  `make install WITHOUT_SENSOR=yes' at the "${PORTSDIR}/sysutils/gkrellm"
+  If you want to support LM78/79 feature,
+  `make install WITH_SENSOR=yes' at the "${PORTSDIR}/sysutils/gkrellm"
   directory to re-make this package.
+  You should aware that GKrellM must be setuid root to obtain
+  temperatures.  Furthermore, this causes problem on some
+  motherboards.  If your motherboard doesn't support LM78/79 feature,
+  you should not enable this.
 
 ****************************************************************************