Date: Wed, 13 Jul 2011 13:04:23 -0400 From: Jung-uk Kim <jkim@FreeBSD.org> To: freebsd-ports@FreeBSD.org Cc: Stephen Montgomery-Smith <stephen@missouri.edu>, Pav Lucistnik <pav@freebsd.org>, Stephen Montgomery-Smith <stephen@freebsd.org> Subject: Re: [RFC] A trivial change for DESKTOP_ENTRIES Message-ID: <201107131304.35636.jkim@FreeBSD.org> In-Reply-To: <4E1DCB00.2030906@missouri.edu> References: <201107121826.00020.jkim@FreeBSD.org> <201107131141.43840.jkim@FreeBSD.org> <4E1DCB00.2030906@missouri.edu>
next in thread | previous in thread | raw e-mail | index | archive | help
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
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201107131304.35636.jkim>