Summary: | TDE/CDE desktop needs cleanup and fixes | ||
---|---|---|---|
Product: | LibreOffice | Reporter: | rezso |
Component: | LibreOffice | Assignee: | Not Assigned <libreoffice-bugs> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | bfo.bugmail, jbfaure, michael.meeks, robinson.libreoffice |
Priority: | medium | Keywords: | difficultyBeginner, easyHack, skillScript, topicCleanup |
Version: | 4.1.0.1 rc | ||
Hardware: | Other | ||
OS: | Linux (All) | ||
Whiteboard: | BSA | ||
Crash report or crash signature: | Regression By: | ||
Attachments: | cleanup for CDE and TDE desktops |
Description
rezso
2013-03-31 15:39:50 UTC
(In reply to comment #0) > It seems to me, that LO not supports CDE desktop. Please elaborate. - no .mk file, and any code for cde library under /vcl - no configure test for cde - no --enable-cde / --disable-cde configure option For this reason I think, that the CDE support is removed. But vcl/inc/unx/desktops.hxx contains it as known desktop, and the cde-open-url script is exists under /shell/source/unix/misc, and always installed. Set Michael in CC. Hi Michael, do you have an opinion about that ? Best regards. JBF No idea; AFAIK there never was specific support for CDE - we just use the X backend for that; but - I imagine some integration points in .desktop files or there equivalent were needed to launch that thing. Then again - since we don't really support solaris, which switched to GNOME long ago, I'd be well up for a patch to remove the CDE support :-) All patches gratefully received :-) Created attachment 81803 [details]
cleanup for CDE and TDE desktops
- remove DESKTOP_CDE from /vcl/inc/unx/desktops.hxx
- remove cde-open-url script and its installation
- add ENABLE_TDE condition to install tde-open-url script.
Nice work; pushed to master in your name; can you send a license statement to the dev list (no subscription required) that we can link into: https://wiki.documentfoundation.org/Development/Developers Thanks for contributing to LibreOffice ! :-) Thank you, I sent my license statement. My second „contribution” is a patch for bug46990 :) Migrating Whiteboard tags to Keywords: (,EasyHack,DifficultyBeginner,SkillScript,TopicCleanup) [NinjaEdit] |