aboutsummaryrefslogtreecommitdiff
path: root/x11/gnome-screensaver
diff options
context:
space:
mode:
authorJoe Marcus Clarke <marcus@FreeBSD.org>2008-08-03 06:57:47 +0000
committerJoe Marcus Clarke <marcus@FreeBSD.org>2008-08-03 06:57:47 +0000
commit8ac57f52fe0d944566907fc8d6a0c2f1adf8fe6b (patch)
tree6cf8327ba9bd986c41e312cdd26c081aaea48efb /x11/gnome-screensaver
parentbc23de35a1ec09012b82efec75c485c2f5c5f59a (diff)
downloadports-8ac57f52fe0d944566907fc8d6a0c2f1adf8fe6b.tar.gz
ports-8ac57f52fe0d944566907fc8d6a0c2f1adf8fe6b.zip
Notes
Diffstat (limited to 'x11/gnome-screensaver')
-rw-r--r--x11/gnome-screensaver/Makefile27
1 files changed, 18 insertions, 9 deletions
diff --git a/x11/gnome-screensaver/Makefile b/x11/gnome-screensaver/Makefile
index 1663a4258d2b..a8b29b5c58a7 100644
--- a/x11/gnome-screensaver/Makefile
+++ b/x11/gnome-screensaver/Makefile
@@ -42,26 +42,35 @@ SCREENSAVER_DIR= share/applications/screensavers
XSCREENSAVER_EXCLUDE= "(popsquares.xml)"
-OPTIONS= PAM "Pluggable Authentication Module support" off \
- KEYRING "Enable GnomeKeyring/PAM integration" off
+OPTIONS= PAM "Pluggable Authentication Module support (broken!)" off \
+ KEYRING "Enable GnomeKeyring integration (needs PAM)" off
.include <bsd.port.pre.mk>
.if defined(WITH_PAM)
+# Currently semi-broken: http://bugzilla.gnome.org/show_bug.cgi?id=370847
+# Our default pam_unix will not be able to authenticate
+# Define GNOME_SCREENSAVER_WITH_BROKEN_PAM only if you know what you are doing.
+.if !defined(GNOME_SCREENSAVER_WITH_BROKEN_PAM)
+IGNORE= PAM support is partially broken. Please read Makefile
+.endif
CONFIGURE_ARGS+= --enable-pam
PLIST_SUB+= PAM=""
-.if defined(WITH_KEYRING)
+.else
+CONFIGURE_ARGS+= --disable-pam
+PLIST_SUB+= PAM="@comment "
+.endif
+
+.if defined(WITH_KEYRING) && defined(WITH_PAM)
SUB_LIST+= PAM_KEYRING=
RUN_DEPENDS+= ${LOCALBASE}/lib/pam_gnome_keyring.so:${PORTSDIR}/security/gnome-keyring
.else
SUB_LIST+= PAM_KEYRING=\#
.endif
-.else
-CONFIGURE_ARGS+= --disable-pam
-PLIST_SUB+= PAM="@comment "
-.if defined(WITH_KEYRING)
-.warning Option KEYRING needs PAM, but PAM is disabled.
-.endif
+
+pre-everything::
+.if defined(WITHOUT_PAM) && defined(WITH_KEYRING)
+ @${ECHO_MSG} "Keyring integration is disabled because it needs PAM."
.endif
post-patch: