https://bugs.freedesktop.org/show_bug.cgi?id=46990

Michael Meeks <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED

--- Comment #6 from Michael Meeks <[email protected]> ---
> I reworked the is_xfce_desktop() and is_mate_desktop() with return

Ah - I was expecting a simple { return "mate" == getenv(); } method :-)

> unset DESKTOP_SESSION: why? If any user log in to any desktop environment,
> this environment variable automatically gets a value.

True today, but certainly not true for older machines, DESKTOP_SESSION is a new
innovation it seems ;-) and crashing for a NULL env. var is really lame - so we
should avoid that.

> I'm not an expert in LO code and programming, my patch just a quick hack, 
> which works for me (I use ~2 months ago).

I pushed another patch which made it somewhat simpler and hopefully easier to
read. Anyhow - thanks for your work here ! good to get this fixed :-) any other
bugs / easy hacks on your radar ?

-- 
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Libreoffice-bugs mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

Reply via email to