From owner-freebsd-ports@FreeBSD.ORG Wed Jul 13 17:04:43 2011 Return-Path: Delivered-To: freebsd-ports@FreeBSD.org Received: from [127.0.0.1] (freefall.freebsd.org [IPv6:2001:4f8:fff6::28]) by hub.freebsd.org (Postfix) with ESMTP id 3838C1065673; Wed, 13 Jul 2011 17:04:43 +0000 (UTC) (envelope-from jkim@FreeBSD.org) From: Jung-uk Kim To: freebsd-ports@FreeBSD.org Date: Wed, 13 Jul 2011 13:04:23 -0400 User-Agent: KMail/1.6.2 References: <201107121826.00020.jkim@FreeBSD.org> <201107131141.43840.jkim@FreeBSD.org> <4E1DCB00.2030906@missouri.edu> In-Reply-To: <4E1DCB00.2030906@missouri.edu> MIME-Version: 1.0 Content-Disposition: inline Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit Message-Id: <201107131304.35636.jkim@FreeBSD.org> Cc: Stephen Montgomery-Smith , Pav Lucistnik , Stephen Montgomery-Smith Subject: Re: [RFC] A trivial change for DESKTOP_ENTRIES X-BeenThere: freebsd-ports@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Porting software to FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 13 Jul 2011 17:04:43 -0000 On Wednesday 13 July 2011 12:42 pm, Stephen Montgomery-Smith wrote: > On 07/13/2011 10:41 AM, Jung-uk Kim wrote: > > On Wednesday 13 July 2011 07:39 am, Pav Lucistnik wrote: > >> On 2011/07/13 00:25, Jung-uk Kim wrote: > >>> After I updated x11-wm/compiz, GNOME was not able to start the > >>> window manager. Basically, it complained that compiz-manager > >>> was not found. Then, I realized compiz-manager.desktop was > >>> automagically replaced by compizmanager.desktop. Now I tracked > >>> it down to this commit: > >>> > >>> Sat Nov 27 17:42:46 2010 UTC (7 months, 2 weeks ago) by pav > >>> > >>> - DESKTOP_ENTRIES: commandline is used to name installed > >>> .desktop file, this can lead to files containing whitespace and > >>> funny characters; thus strip all non-alphanumeric characters > >>> > >>> http://www.freebsd.org/cgi/cvsweb.cgi/ports/Mk/bsd.port.mk.diff > >>>?r 1=1.656;r2=1.657 > >>> > >>> To me, it looks far too restrictive. At least, I'd like to > >>> allow '-' and '_'. Please see the attached patch. > >>> > >>> Any objections? > >> > >> Shouldn't you fix whatever is trying to call compizmanager not > >> to use .desktop file instead? > > > > GNOME session manager calls the compiz-manager, i.e., the user > > has to change it manually. Actually, x11-wm/compiz/pkg-message > > recommended this: > > > > "If you are using gnome, you can use the configuration editor to > > set the value of: > > > > desktop->gnome->session->required_components->windowmanager = > > compiz-manager > > ^^^^^^^^^^^^^^ > > > > This will enable compiz as your default window manager." > > > > I am quite sure there are similar instructions on the net. > > > > Jung-uk Kim > > Also, bsd.ports.mk shouldn't change what the port tells it to do, > without informing anybody. Exactly. > I'm sure it took Jung-uk Kim many hours to figure out why it wasn't > working. Other users are going to be in a similar spot. Many > users will never figure it out. Sssssh... Please don't tell it to my employee. ;-P It only took me about 1/2 hour. But you're right; average users may misinterpret unrelated commits (such as yours) and think it broke their desktop environment after updating ports. Jung-uk Kim