From owner-cvs-all@FreeBSD.ORG Wed Sep 21 20:05:16 2011 Return-Path: Delivered-To: cvs-all@FreeBSD.org Received: from mx2.freebsd.org (mx2.freebsd.org [IPv6:2001:4f8:fff6::35]) by hub.freebsd.org (Postfix) with ESMTP id 3A25D106566C; Wed, 21 Sep 2011 20:05:16 +0000 (UTC) (envelope-from dougb@FreeBSD.org) Received: from 172-17-198-245.globalsuite.net (hub.freebsd.org [IPv6:2001:4f8:fff6::36]) by mx2.freebsd.org (Postfix) with ESMTP id 2A93314E74F; Wed, 21 Sep 2011 20:05:15 +0000 (UTC) Message-ID: <4E7A437A.5030401@FreeBSD.org> Date: Wed, 21 Sep 2011 13:05:14 -0700 From: Doug Barton Organization: http://SupersetSolutions.com/ User-Agent: Mozilla/5.0 (X11; FreeBSD amd64; rv:6.0.2) Gecko/20110912 Thunderbird/6.0.2 MIME-Version: 1.0 To: Emanuel Haupt References: <201103100949.p2A9nOO8057627@repoman.freebsd.org> <20110919234126.GA82010@server.vk2pj.dyndns.org> <20110921115818.2878356b.ehaupt@FreeBSD.org> <4E7A23D3.5070509@FreeBSD.org> <20110921210838.893ad8f0.ehaupt@FreeBSD.org> In-Reply-To: <20110921210838.893ad8f0.ehaupt@FreeBSD.org> X-Enigmail-Version: undefined OpenPGP: id=1A1ABC84 Content-Type: multipart/mixed; boundary="------------020702050009040500090504" Cc: cvs-ports@FreeBSD.org, cvs-all@FreeBSD.org, ports-committers@FreeBSD.org Subject: Re: cvs commit: ports/x11/xterm Makefile pkg-plist X-BeenThere: cvs-all@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: **OBSOLETE** CVS commit messages for the entire tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 21 Sep 2011 20:05:16 -0000 This is a multi-part message in MIME format. --------------020702050009040500090504 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit On 09/21/2011 12:08, Emanuel Haupt wrote: > Doug Barton wrote: >> On 09/21/2011 02:58, Emanuel Haupt wrote: >>> Peter Jeremy wrote: >> >>>> This is a bit belated but why does xterm need a hard dependency >>>> on gnome? Could you please look at making this optional so that >>>> those of us who don't run bloatware can turn it off. >>> >>> Good point. I've just committed a patch that makes the installation >>> of desktop entries conditional. >> >> What isn't clear to me is why the installation of the desktop entry >> depends on gnome at all. Why not just install the file? I ask because >> (at least) xfce also uses the desktop entries, so having them is >> useful for non-gnome cases as well. > > From what I understand "desktopfileutils" needs to be set in (USE| > WANT)_GNOME if there are *.desktop files installed. This is where some > gnome dependencies are pulled in. Yes, I understand that this is your understanding, otherwise you wouldn't have done it that way. :) I am attempting to challenge that understanding. IOW, can the installation of the desktop file be split out from the gnome desktopfileutils part? The attached works for the non-gnome case, and the xterm.desktop file appears as expected for xfce. Doug -- Nothin' ever doesn't change, but nothin' changes much. -- OK Go Breadth of IT experience, and depth of knowledge in the DNS. Yours for the right price. :) http://SupersetSolutions.com/ --------------020702050009040500090504 Content-Type: text/plain; name="xterm-desktop.diff" Content-Transfer-Encoding: 7bit Content-Disposition: attachment; filename="xterm-desktop.diff" Index: Makefile =================================================================== RCS file: /home/pcvs/ports/x11/xterm/Makefile,v retrieving revision 1.89 diff -u -r1.89 Makefile --- Makefile 21 Sep 2011 09:57:30 -0000 1.89 +++ Makefile 21 Sep 2011 20:02:44 -0000 @@ -7,7 +7,7 @@ PORTNAME= xterm PORTVERSION= 275 -PORTREVISION= 1 +PORTREVISION= 2 CATEGORIES= x11 MASTER_SITES= ftp://invisible-island.net/xterm/ \ CRITICAL @@ -27,7 +27,7 @@ 256_COLOR "enable 256-color support" off \ DEC_LOCATOR "enable DECterm Locator support" off \ PCRE "use PCRE for regular-expressions" off \ - DESKTOP "install desktop entries" off + GNOME "include gnome support for .desktop file" off CPPFLAGS+= -I${LOCALBASE}/include @@ -55,11 +55,11 @@ LIB_DEPENDS= pcre.0:${PORTSDIR}/devel/pcre .endif -.if defined(WITH_DESKTOP) +.if defined(WITH_GNOME) USE_GNOME= desktopfileutils -PLIST_SUB+= DESKTOP="" +PLIST_SUB+= GNOME="" .else -PLIST_SUB+= DESKTOP="@comment " +PLIST_SUB+= GNOME="@comment " .endif MAN1= resize.1 koi8rxterm.1 uxterm.1 xterm.1 @@ -74,9 +74,9 @@ .for f in koi8rxterm resize uxterm xterm @${CHMOD} ${BINMODE} ${PREFIX}/bin/${f} .endfor -.if defined(WITH_DESKTOP) @${MKDIR} ${PREFIX}/share/applications/ @${INSTALL_DATA} ${WRKSRC}/xterm.desktop ${PREFIX}/share/applications/ +.if defined(WITH_GNOME) @-update-desktop-database .endif Index: pkg-plist =================================================================== RCS file: /home/pcvs/ports/x11/xterm/pkg-plist,v retrieving revision 1.9 diff -u -r1.9 pkg-plist --- pkg-plist 21 Sep 2011 09:57:30 -0000 1.9 +++ pkg-plist 21 Sep 2011 20:02:44 -0000 @@ -8,11 +8,11 @@ lib/X11/app-defaults/UXTerm-color lib/X11/app-defaults/XTerm lib/X11/app-defaults/XTerm-color -%%DESKTOP%%share/applications/xterm.desktop +share/applications/xterm.desktop share/pixmaps/xterm-color_32x32.xpm share/pixmaps/xterm-color_48x48.xpm share/pixmaps/xterm_32x32.xpm share/pixmaps/xterm_48x48.xpm -%%DESKTOP%%@dirrmtry share/applications -%%DESKTOP%%@exec %%LOCALBASE%%/bin/update-desktop-database > /dev/null || /usr/bin/true -%%DESKTOP%%@unexec %%LOCALBASE%%/bin/update-desktop-database > /dev/null || /usr/bin/true +@dirrmtry share/applications +%%GNOME%%@exec %%LOCALBASE%%/bin/update-desktop-database > /dev/null || /usr/bin/true +%%GNOME%%@unexec %%LOCALBASE%%/bin/update-desktop-database > /dev/null || /usr/bin/true --------------020702050009040500090504--