[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-core-devel
Subject: Re: Review Request 120287: [OS X] make kde-workspace build
From: René J.V. Bertin <rjvbertin () gmail ! com>
Date: 2014-10-14 18:46:39
Message-ID: 20141014184639.21224.80502 () probe ! kde ! org
[Download RAW message or body]
--===============3048692362758362316==
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit
> On Oct. 14, 2014, 6:38 p.m., Martin Gräßlin wrote:
> > I'm not understanding the changes in Plasma Netbook. Why do you want the Netbook \
> > shell on OSX while on the other side you disabled the desktop shell? AFAIK you \
> > cannot replace the shell of OSX, so having Netbook sounds pretty useless to me.
>
> René J.V. Bertin wrote:
> The point is not to have the shell, but to have access to plasmoids via \
> plasma-windowed (or plasmoid-viewer if that application is still around and \
> functional). I've tried to explain that I have made a rather coarse selection \
> (everything also included in MS Windows builds and that actually builds) rather \
> than hand-picking only those components that would have a potential use. Doing so I \
> indeed noticed that the desktop shell was excluded on Win32 but not the netbook \
> shell. It didn't take me long to realise that the netbook shell is different enough \
> from the usual desktop paradigm to be of interest to some users, to the extent that \
> it can be made to function in a rooted, normal window of course. I am of course \
> open to feedback concerning the components that can be removed from the build \
> without effect on components that do have a use.
> If I may think aloud a little bit:
> Netbook or Desktop shells ... they open (fullscreen) windows in practice, right? If \
> so, there could be an (academic?) interest in supporting them but with a regular \
> window, allowing the user to set up a sort of MDI version of a desktop shell with \
> "goodies" that would give a more coherent experience than running those same \
> goodies individually on the OS X desktop. One could also think of a "shell" that \
> only serves to host panels and widgets, not unlike Yahoo! Widgets \
> (http://en.wikipedia.org/wiki/Yahoo!_Widgets)
> Martin Gräßlin wrote:
> > Netbook or Desktop shells ... they open (fullscreen) windows in practice, right?
>
> no, they open "desktop" windows.
>
> > If so, there could be an (academic?) interest in supporting them but with a \
> > regular window
>
> no, that's what plasma-windowed is for. And I'm certainly not giving a +1 for the \
> rather big changes to netbook shell if the only need is on an academic scale ;-)
I wasn't about to start hacking to pursue any of these ideas, and certainly not in \
KDE4. We'll see if and when we get to KF5 on OS X but in the meantime the thought is \
out there. In case there are KDE developers outside of the kde-mac "kernel" who'd \
appreciate to see part of the power of a KDE desktop available beyond Linux and other \
Unix/X11 systems.
Now that you mention plasma-windowed: I *would* like to get that utility to be able \
to run in multiple instances! Whatever good reasons there might be on *n*x for it to \
be a KUniqueApplication, on OS X there's a good reason for it not to be ...
- René J.V.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/120287/#review68404
-----------------------------------------------------------
On Oct. 14, 2014, 6:06 p.m., René J.V. Bertin wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/120287/
> -----------------------------------------------------------
>
> (Updated Oct. 14, 2014, 6:06 p.m.)
>
>
> Review request for KDE Software on Mac OS X and kde-workspace.
>
>
> Repository: kde-workspace
>
>
> Description
> -------
>
> A few rather straightforward patches to make the relevant bits of KDE4's \
> kde-workspace build and function on OS X. The main interest is having the \
> systemsettings control panel to control the various relevant KDE settings among \
> which desktop search, fonts, colours and even style. The oxygen style builds and \
> looks good but shows some updating glitches due to compositing.
> I'm submitting this patch partly in hope it may be useful in bringing kf5-workspace \
> to OS X, one day.
>
> Diffs
> -----
>
> CMakeLists.txt df8a1f7
> kcontrol/CMakeLists.txt fc666b1
> kcontrol/krdb/krdb.cpp 36fc99c
> kcontrol/style/CMakeLists.txt d832b20
> libs/CMakeLists.txt c0576fe
> plasma/CMakeLists.txt 199dbb0
> plasma/generic/shells/plasma-windowed/plasmaapp.cpp dbdff47
> plasma/netbook/CMakeLists.txt 1eff685
> plasma/netbook/containments/CMakeLists.txt c96a688
> plasma/desktop/CMakeLists.txt 2de78dd
> plasma/desktop/applets/CMakeLists.txt 6f80cec
> plasma/generic/CMakeLists.txt cfaf14f
> plasma/generic/applets/CMakeLists.txt 2b888ee
> plasma/generic/dataengines/CMakeLists.txt d240683
> plasma/generic/runners/CMakeLists.txt 6831ac0
> plasma/generic/shells/plasma-windowed/CMakeLists.txt 86b7770
> plasma/generic/shells/plasma-windowed/Info.plist.template PRE-CREATION
>
> Diff: https://git.reviewboard.kde.org/r/120287/diff/
>
>
> Testing
> -------
>
> On OS X 10.6.8 and 10.9.4 with KDE/MacPorts (4.12.5 and more recently kdelibs \
> git/master, 4.14.1).
>
> File Attachments
> ----------------
>
> copy of the diff file saved locally, which had no tabs when I uploaded it. \
> Checksum: 3989cdd46af3c891e570974d66c330403dcd41c4ee5e17a372fa385080cbabd1 \
> https://git.reviewboard.kde.org/media/uploaded/files/2014/09/20/b212730f-6258-4277-851c-226bc0673aa1__patchreview-20140920.patch
>
>
> Thanks,
>
> René J.V. Bertin
>
>
--===============3048692362758362316==
MIME-Version: 1.0
Content-Type: text/html; charset="utf-8"
Content-Transfer-Encoding: 8bit
<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="12" style="border: 1px #c9c399 \
solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;"> \
<tr> <td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://git.reviewboard.kde.org/r/120287/">https://git.reviewboard.kde.org/r/120287/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: \
10px;"> <p style="margin-top: 0;">On October 14th, 2014, 6:38 p.m. CEST, <b>Martin \
Gräßlin</b> wrote:</p> <blockquote style="margin-left: 1em; border-left: 2px solid \
#d0d0d0; padding-left: 10px;"> <pre style="white-space: pre-wrap; white-space: \
-moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: \
break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: \
inherit;white-space: inherit;">I'm not understanding the changes in Plasma Netbook. \
Why do you want the Netbook shell on OSX while on the other side you disabled the \
desktop shell? AFAIK you cannot replace the shell of OSX, so having Netbook sounds \
pretty useless to me.</p></pre> </blockquote>
<p>On October 14th, 2014, 6:57 p.m. CEST, <b>René J.V. Bertin</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: \
10px;"> <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: \
-pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: \
0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">The \
point is not to have the shell, but to have access to plasmoids via plasma-windowed \
(or plasmoid-viewer if that application is still around and functional). I've tried \
to explain that I have made a rather coarse selection (everything also included in MS \
Windows builds and that actually builds) rather than hand-picking only those \
components that would have a potential use. Doing so I indeed noticed that the \
desktop shell was excluded on Win32 but not the netbook shell. It didn't take me long \
to realise that the netbook shell is different enough from the usual desktop paradigm \
to be of interest to some users, to the extent that it can be made to function in a \
rooted, normal window of course. I am of course open to feedback concerning the \
components that can be removed from the build without effect on components that do \
have a use.</p> <p style="padding: 0;text-rendering: inherit;margin: 0;line-height: \
inherit;white-space: inherit;">If I may think aloud a little bit: Netbook or Desktop \
shells ... they open (fullscreen) windows in practice, right? If so, there could be \
an (academic?) interest in supporting them but with a regular window, allowing the \
user to set up a sort of MDI version of a desktop shell with "goodies" that would \
give a more coherent experience than running those same goodies individually on the \
OS X desktop. One could also think of a "shell" that only serves to host panels and \
widgets, not unlike Yahoo! Widgets \
(http://en.wikipedia.org/wiki/Yahoo!_Widgets)</p></pre> </blockquote>
<p>On October 14th, 2014, 8:15 p.m. CEST, <b>Martin Gräßlin</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: \
10px;"> <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: \
-pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><blockquote \
style="text-rendering: inherit;padding: 0 0 0 1em;border-left: 1px solid \
#bbb;white-space: normal;margin: 0 0 0 0.5em;line-height: inherit;"> <p \
style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: \
inherit;">Netbook or Desktop shells ... they open (fullscreen) windows in practice, \
right?</p> </blockquote>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: \
inherit;white-space: inherit;">no, they open "desktop" windows.</p> <blockquote \
style="text-rendering: inherit;padding: 0 0 0 1em;border-left: 1px solid \
#bbb;white-space: normal;margin: 0 0 0 0.5em;line-height: inherit;"> <p \
style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: \
inherit;">If so, there could be an (academic?) interest in supporting them but with a \
regular window</p> </blockquote>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: \
inherit;white-space: inherit;">no, that's what plasma-windowed is for. And I'm \
certainly not giving a +1 for the rather big changes to netbook shell if the only \
need is on an academic scale ;-)</p></pre> </blockquote>
</blockquote>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: \
-pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: \
0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">I \
wasn't about to start hacking to pursue any of these ideas, and certainly not in \
KDE4. We'll see if and when we get to KF5 on OS X but in the meantime the thought is \
out there. In case there are KDE developers outside of the kde-mac "kernel" who'd \
appreciate to see part of the power of a KDE desktop available beyond Linux and other \
Unix/X11 systems.</p> <p style="padding: 0;text-rendering: inherit;margin: \
0;line-height: inherit;white-space: inherit;">Now that you mention plasma-windowed: I \
<em style="padding: 0;text-rendering: inherit;margin: 0;line-height: \
inherit;white-space: normal;">would</em> like to get that utility to be able to run \
in multiple instances! Whatever good reasons there might be on <em style="padding: \
0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: \
normal;">n</em>x for it to be a KUniqueApplication, on OS X there's a good reason for \
it not to be ...</p></pre> <br />
<p>- René J.V.</p>
<br />
<p>On October 14th, 2014, 6:06 p.m. CEST, René J.V. Bertin wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="12" style="border: \
1px #888a85 solid; border-radius: 6px; -moz-border-radius: 6px; \
-webkit-border-radius: 6px;"> <tr>
<td>
<div>Review request for KDE Software on Mac OS X and kde-workspace.</div>
<div>By René J.V. Bertin.</div>
<p style="color: grey;"><i>Updated Oct. 14, 2014, 6:06 p.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
kde-workspace
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" \
style="border: 1px solid #b8b5a0"> <tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: \
-moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: \
break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: \
inherit;white-space: inherit;">A few rather straightforward patches to make the \
relevant bits of KDE4's kde-workspace build and function on OS X. The main interest \
is having the systemsettings control panel to control the various relevant KDE \
settings among which desktop search, fonts, colours and even style. The oxygen style \
builds and looks good but shows some updating glitches due to compositing.</p> <p \
style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: \
inherit;">I'm submitting this patch partly in hope it may be useful in bringing \
kf5-workspace to OS X, one day.</p></pre> </td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: \
1px solid #b8b5a0"> <tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: \
-moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: \
break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: \
inherit;white-space: inherit;">On OS X 10.6.8 and 10.9.4 with KDE/MacPorts (4.12.5 \
and more recently kdelibs git/master, 4.14.1).</p></pre> </td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>CMakeLists.txt <span style="color: grey">(df8a1f7)</span></li>
<li>kcontrol/CMakeLists.txt <span style="color: grey">(fc666b1)</span></li>
<li>kcontrol/krdb/krdb.cpp <span style="color: grey">(36fc99c)</span></li>
<li>kcontrol/style/CMakeLists.txt <span style="color: grey">(d832b20)</span></li>
<li>libs/CMakeLists.txt <span style="color: grey">(c0576fe)</span></li>
<li>plasma/CMakeLists.txt <span style="color: grey">(199dbb0)</span></li>
<li>plasma/generic/shells/plasma-windowed/plasmaapp.cpp <span style="color: \
grey">(dbdff47)</span></li>
<li>plasma/netbook/CMakeLists.txt <span style="color: grey">(1eff685)</span></li>
<li>plasma/netbook/containments/CMakeLists.txt <span style="color: \
grey">(c96a688)</span></li>
<li>plasma/desktop/CMakeLists.txt <span style="color: grey">(2de78dd)</span></li>
<li>plasma/desktop/applets/CMakeLists.txt <span style="color: \
grey">(6f80cec)</span></li>
<li>plasma/generic/CMakeLists.txt <span style="color: grey">(cfaf14f)</span></li>
<li>plasma/generic/applets/CMakeLists.txt <span style="color: \
grey">(2b888ee)</span></li>
<li>plasma/generic/dataengines/CMakeLists.txt <span style="color: \
grey">(d240683)</span></li>
<li>plasma/generic/runners/CMakeLists.txt <span style="color: \
grey">(6831ac0)</span></li>
<li>plasma/generic/shells/plasma-windowed/CMakeLists.txt <span style="color: \
grey">(86b7770)</span></li>
<li>plasma/generic/shells/plasma-windowed/Info.plist.template <span style="color: \
grey">(PRE-CREATION)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/120287/diff/" style="margin-left: \
3em;">View Diff</a></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">File Attachments \
</h1>
<li><a href="https://git.reviewboard.kde.org/media/uploaded/files/2014/09/20/b212730f-6258-4277-851c-226bc0673aa1__patchreview-20140920.patch">copy \
of the diff file saved locally, which had no tabs when I uploaded it. Checksum: \
3989cdd46af3c891e570974d66c330403dcd41c4ee5e17a372fa385080cbabd1 </a></li>
</ul>
</td>
</tr>
</table>
</div>
</body>
</html>
--===============3048692362758362316==--
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic