ports/125984: [PATCH] x11/xscreensaver-gnome: GnomeKeyring integration
Marcin Wisnicki
mwisnicki+freebsd at gmail.com
Sat Jul 26 15:20:01 UTC 2008
>Number: 125984
>Category: ports
>Synopsis: [PATCH] x11/xscreensaver-gnome: GnomeKeyring integration
>Confidential: no
>Severity: non-critical
>Priority: low
>Responsible: freebsd-ports-bugs
>State: open
>Quarter:
>Keywords:
>Date-Required:
>Class: change-request
>Submitter-Id: current-users
>Arrival-Date: Sat Jul 26 15:20:01 UTC 2008
>Closed-Date:
>Last-Modified:
>Originator: Marcin Wisnicki
>Release: FreeBSD 7.0-STABLE i386
>Organization:
>Environment:
System: FreeBSD ghost.pnet.one.pl 7.0-STABLE FreeBSD 7.0-STABLE #15: Fri Jul 25 18:16:53 CEST 2008
>Description:
- Optional (enabled by default) GnomeKeyring integration via PAM.
- Enable PAM by default
This patch assumes that ports/125967 is commited.
I will also submit similar patch for x11/gdm.
Added file(s):
- files/xscreensaver.pam.in
Removed file(s):
- files/xscreensaver.pam
Port maintainer (gnome at FreeBSD.org) is cc'd.
Generated with FreeBSD Port Tools 0.77
>How-To-Repeat:
>Fix:
--- xscreensaver-gnome-5.06_1.patch begins here ---
diff -ruN --exclude=CVS /usr/ports/x11/xscreensaver-gnome.orig/Makefile /usr/ports/x11/xscreensaver-gnome/Makefile
--- /usr/ports/x11/xscreensaver-gnome.orig/Makefile 2008-07-19 15:52:49.000000000 +0200
+++ /usr/ports/x11/xscreensaver-gnome/Makefile 2008-07-26 01:36:46.000000000 +0200
@@ -8,7 +8,7 @@
PORTNAME?= xscreensaver-gnome
PORTVERSION= 5.06
-PORTREVISION?= 0
+PORTREVISION?= 1
CATEGORIES= x11 gnome
MASTER_SITES= http://www.jwz.org/xscreensaver/
DISTNAME= xscreensaver-${PORTVERSION}
@@ -36,10 +36,13 @@
CONFIGURE_ENV= CPPFLAGS="-I${LOCALBASE}/include" \
LDFLAGS="-L${LOCALBASE}/lib"
-OPTIONS?= PAM "Pluggable Authentication Module support" off \
+OPTIONS?= PAM "Pluggable Authentication Module support" on \
+ KEYRING "Enable GnomeKeyring/PAM integration" on \
ALL_FORTUNES "Enable support for all fortunes" off \
SETUID_HACKS "Install sonar hack suid so it can ping" off
+SUB_FILES+= xscreensaver.pam
+
MAN1?= xscreensaver-command.1 xscreensaver-demo.1 \
xscreensaver-getimage-file.1 xscreensaver-getimage-video.1 \
xscreensaver-getimage.1 xscreensaver-text.1 xscreensaver.1
@@ -90,9 +93,18 @@
.if defined(WITH_PAM)
CONFIGURE_ARGS+= --with-pam
PLIST_SUB+= PAM=""
+.if defined(WITH_KEYRING)
+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+= --without-pam
PLIST_SUB+= PAM="@comment "
+.if defined(WITH_KEYRING)
+.warning Option KEYRING needs PAM, but PAM is disabled.
+.endif
.endif
.if defined(WITH_SETUID_HACKS)
@@ -114,12 +126,16 @@
@${REINPLACE_CMD} -e 's|/etc/pam.d|${PREFIX}/etc/pam.d|g' \
${WRKSRC}/${f}
.endfor
- @${CP} ${FILESDIR}/xscreensaver.pam ${WRKSRC}/driver/xscreensaver.pam
.else
@${REINPLACE_CMD} -e 's|install-xml install-pam|install-xml|' \
${WRKSRC}/driver/Makefile.in
.endif
+pre-install:
+.if defined(WITH_PAM)
+ @${CP} ${WRKDIR}/xscreensaver.pam ${WRKSRC}/driver/xscreensaver.pam
+.endif
+
.if !target(post-install-slave)
post-install-slave:
@${DO_NADA}
diff -ruN --exclude=CVS /usr/ports/x11/xscreensaver-gnome.orig/files/xscreensaver.pam /usr/ports/x11/xscreensaver-gnome/files/xscreensaver.pam
--- /usr/ports/x11/xscreensaver-gnome.orig/files/xscreensaver.pam 2008-07-06 23:04:51.000000000 +0200
+++ /usr/ports/x11/xscreensaver-gnome/files/xscreensaver.pam 1970-01-01 01:00:00.000000000 +0100
@@ -1,2 +0,0 @@
-auth include system
-account include system
diff -ruN --exclude=CVS /usr/ports/x11/xscreensaver-gnome.orig/files/xscreensaver.pam.in /usr/ports/x11/xscreensaver-gnome/files/xscreensaver.pam.in
--- /usr/ports/x11/xscreensaver-gnome.orig/files/xscreensaver.pam.in 1970-01-01 01:00:00.000000000 +0100
+++ /usr/ports/x11/xscreensaver-gnome/files/xscreensaver.pam.in 2008-07-26 01:25:24.000000000 +0200
@@ -0,0 +1,3 @@
+auth include system
+%%PAM_KEYRING%%auth optional %%LOCALBASE%%/lib/pam_gnome_keyring.so
+account include system
--- xscreensaver-gnome-5.06_1.patch ends here ---
>Release-Note:
>Audit-Trail:
>Unformatted:
More information about the freebsd-ports-bugs
mailing list