[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-panel-devel
Subject:    Re: Plasma 5.10 Frameworks version problem
From:       David Edmundson <david () davidedmundson ! co ! uk>
Date:       2017-04-29 15:49:42
Message-ID: CAGeFrHD1+0JruG-rbnB4fV2OeSic9GC3NdAT5f=SioTffya0Tw () mail ! gmail ! com
[Download RAW message or body]

On Fri, Apr 28, 2017 at 7:22 PM, Martin Gr=C3=A4=C3=9Flin <mgraesslin@kde.o=
rg> wrote:

> Am 2017-04-28 17:22, schrieb David Edmundson:
>
>> What's the minimum version of frameworks we can use in Plasma 5.10?
>>
>> The schedule currently is:
>>
>> 2017-05-06: Frameworks 5.33 gets tagged
>> 2017-05-11: Plasma 5.10 beta is tagged && released
>> 2017-05-13: Frameworks 5.33 comes out
>> 2017-05-25: Plasma 5.10 is tagged
>>
>> With the current schedule we can't use frameworks 5.33 in Plasma 5.10
>> beta as it will have an unreleased dependency.
>>
>> With frameworks 5.32:
>>
>> 1) Kwin won't compile (the popupDone code)
>>
>> 2) Half of plasma won't load.  New QML api was added in units:
>> "iconSizeHints"
>>
>> This is used in icon applet, trash, folderview compact, kicker,
>> showdesktop, minimizeall
>>
>> The worst part about the latter is that you can't see the problem when
>> you compile :/
>>
>> We can either:
>>
>>  revert/ifdef those changes
>>
>>  shuffle something above
>>
>
> I vote for shuffling around as it sounds like lots of efforts for little
> gain. The KWin case is rather trivial, but nevertheless probably half an
> hour of work to ensure it works. The iconSizeHints is even more work.
>
> So maybe just move the release by two days and ship together with
> frameworks?
>
> Me too, we can still tag and branch on the original day, and the final
doesn't need to move.
It's just the public release of the beta that would need to slip.

David


> Cheers
> Martin
>

[Attachment #3 (text/html)]

<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Apr \
28, 2017 at 7:22 PM, Martin Gräßlin <span dir="ltr">&lt;<a \
href="mailto:mgraesslin@kde.org" target="_blank">mgraesslin@kde.org</a>&gt;</span> \
wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px \
#ccc solid;padding-left:1ex"><span>Am 2017-04-28 17:22, schrieb David Edmundson:<br> \
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc \
solid;padding-left:1ex"> What&#39;s the minimum version of frameworks we can use in \
Plasma 5.10?<br> <br>
The schedule currently is:<br>
<br>
2017-05-06: Frameworks 5.33 gets tagged<br>
2017-05-11: Plasma 5.10 beta is tagged &amp;&amp; released<br>
2017-05-13: Frameworks 5.33 comes out<br>
2017-05-25: Plasma 5.10 is tagged<br>
<br>
With the current schedule we can&#39;t use frameworks 5.33 in Plasma 5.10<br>
beta as it will have an unreleased dependency.<br>
<br>
With frameworks 5.32:<br>
<br>
1) Kwin won&#39;t compile (the popupDone code)<br>
<br>
2) Half of plasma won&#39;t load.   New QML api was added in units:<br>
&quot;iconSizeHints&quot;<br>
<br>
This is used in icon applet, trash, folderview compact, kicker,<br>
showdesktop, minimizeall<br>
<br>
The worst part about the latter is that you can&#39;t see the problem when<br>
you compile :/<br>
<br>
We can either:<br>
<br>
  revert/ifdef those changes<br>
<br>
  shuffle something above<br>
</blockquote>
<br></span>
I vote for shuffling around as it sounds like lots of efforts for little gain. The \
KWin case is rather trivial, but nevertheless probably half an hour of work to ensure \
it works. The iconSizeHints is even more work.<br> <br>
So maybe just move the release by two days and ship together with frameworks?<br>
<br></blockquote><div>Me too, we can still tag and branch on the original day, and \
the final doesn&#39;t need to move. <br>It&#39;s just the public release of the beta \
that would need to slip.<br><br></div><div>David<br></div><div>  </div><blockquote \
class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc \
solid;padding-left:1ex"> Cheers<span class="m_-9195543931014588491HOEnZb"><font \
color="#888888"><br> Martin<br>
</font></span></blockquote></div><br></div></div>



[prev in list] [next in list] [prev in thread] [next in thread]