From owner-freebsd-ports@FreeBSD.ORG Wed Jul 13 17:09:09 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 6A27B1065670; Wed, 13 Jul 2011 17:09:09 +0000 (UTC) (envelope-from jkim@FreeBSD.org) From: Jung-uk Kim To: freebsd-ports@FreeBSD.org Date: Wed, 13 Jul 2011 13:08:59 -0400 User-Agent: KMail/1.6.2 References: <201107121826.00020.jkim@FreeBSD.org> <4E1DCB00.2030906@missouri.edu> <201107131304.35636.jkim@FreeBSD.org> In-Reply-To: <201107131304.35636.jkim@FreeBSD.org> MIME-Version: 1.0 Content-Disposition: inline Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit Message-Id: <201107131309.02454.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:09:09 -0000 On Wednesday 13 July 2011 01:04 pm, Jung-uk Kim wrote: > 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.di > > >>>ff ?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 ^^^^^^^^ employer I am not there yet. :-( Jung-uk Kim > 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.