<table><tr><td style="">romangg added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D29028">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
color: #6B748C;
font-style: italic;
margin: 4px 0 12px 0;
padding: 8px 12px;
background-color: #F8F9FC;">
<div style="font-style: normal;
padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D29028#654526" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">D29028#654526</a>, <a href="https://phabricator.kde.org/p/tcanabrava/" style="
border-color: #f1f7ff;
color: #19558d;
background-color: #f1f7ff;
border: 1px solid transparent;
border-radius: 3px;
font-weight: bold;
padding: 0 4px;">@tcanabrava</a> wrote:</div>
<div style="margin: 0;
padding: 0;
border: 0;
color: rgb(107, 116, 140);"><p>I understand David's point: Wrapland project has one developer and we don't know how successful it will be, while the other backends have developers. What would happen if you suddenly quit / disappear and the project dies? Then kscreen will have a folder of dead code.</p>
<p>This is a Plugin, it can live in any folder / project, you already forked KWinFT, create a new project to put this KScreen plugin, we can't scale to N projects adding code as plugins that will need to be maintained for quite a while in the future.</p>
<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>Come on man, do you really want to make this ugly? I thought we would still treat each other with respect David. :(</p></blockquote>
<p>Your words. His words are technical, Please don't distort things and make it about you.</p></div>
</blockquote>
<p>I haven't noticed much technical words in what was said. I did ask about the headers-only proposal that was of technical nature. Also the tone made it seem to me personal, but that sadly might have been just what is often the default tone in Plasma reviews (no finger pointing intended, I have been guilty of that in the past too).</p>
<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>Which "established practices" does this not satisfy?</p></blockquote>
<p>It's an external library that does not solve any problem within the plasma, nor adds value to plasma users. It's an experimental project, with only one developer, not stable, not ready, not core. You can see on this picture that we are not blocking your project here because of $excuse, but this is something that we do if the project is not core, please check the date.<br />
<a href="https://phabricator.kde.org/F8253102" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">F8253102: image.png</a></p>
<p>There's more information on the full phabricator ticket:<br />
<a href="https://phabricator.kde.org/D20265" class="remarkup-link" target="_blank" rel="noreferrer">https://phabricator.kde.org/D20265</a></p></blockquote>
<p>I looked at that and suspected to see a hard dependency there. But it also is only an optional dependency with a check if the package is available. So I'm wondering what are the rules if something can become an <em>optional</em> dependency for a Plasma project? Is this written down somewhere?</p>
<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>How would that look in detail?</p></blockquote>
<p>That will give you the possibility to create plugins for KScreen that are not bound to KScreen code, then you can create your project in gitlab and have different release schedules, in a way that:<br />
1 - The plugin code is independent to KDE Releases<br />
2 - We have a clear separation on project responsabilities<br />
3 - Being independent means that it can also test with experimental libraries.</p>
<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>Why is this even relevant?</p></blockquote>
<p>We need to be sure that the code of the project will always be buildable through the lifespan of Plasma 5.19, and right now we can't guarantee it.</p></blockquote>
<p>Since the dependency is optional you are guaranteed to be able to build libkscreen by removing Wrapland or not installing it in the first place. Also Wrapland will have a 0.519.0 release as it has now a 0.518.0 release. Releases will be synced up with Plasma releases. Otherwise the KWinFT-Plasma integration would not work anyway.</p>
<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>Create this code outside of KSCreen as a plugin. We will gladly accept code that install the Interface Headers to make that possible. :)</p></blockquote>
<p>I think that this is unnecessary complicated because releases will be synced, but I can live with it. Would this mean only the first patch of this patch series would go in and I export the <tt style="background: #ebebeb; font-size: 13px;">wayland_interface.h</tt> and <tt style="background: #ebebeb; font-size: 13px;">waylandoutput.h</tt> headers (they are being used in the plugins)?</p>
<p>Thanks for taking the time Tomaz.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R110 KScreen Library</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D29028">https://phabricator.kde.org/D29028</a></div></div><br /><div><strong>To: </strong>romangg, Plasma, davidedmundson, apol<br /><strong>Cc: </strong>anthonyfieroni, tcanabrava, apol, davidedmundson, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, ahiemstra, mart<br /></div>