Review Request 123653: New Minimize Windows Plasmoid

Sebastian Kügler sebas at kde.org
Thu Apr 14 10:28:31 UTC 2016


[I put plasma-devel back in CC:]

On Thursday, April 14, 2016 01:14:26 PM you wrote:
>   Yeah, i know it can't be downloaded because it's need to be compiled. 
> It's not a problem, from my side, to add in kde' repo,  i have dev access,
> but i don't know where to push it etc.

Original review puts it into kdeplasma-addons, so it would go under the 
applets/ subdirectory there.

I think all the change requests in my original review have been addressed, so 
I could push it there, and you take over?

Cheers,
-- sebas


>  >-------- Оригинално писмо --------
>  >
>  >От: Sebastian Kügler sebas at kde.org
>  >
>  >Относно: Re: Review Request 123653: New Minimize Windows Plasmoid
>  >
>  >До: Anthony Fieroni
>  >
>  >Изпратено на: 14.04.2016 12:22
> 
> Hi Anthony,
> 
> On Wednesday, April 13, 2016 06:00:24 PM Anthony Fieroni wrote:
> > This applet is must have for many users, let's make'm happy :) =>
> > 
> >  http://kde-look.org/content/show.php/Minimize+All?content=175817
> 
> I had discarded this review request since I didn't have time to maintain the
> 
> applet and my primary use-case went away.
> 
> 
> 
> From GHNS you can't really install binaries (which this applet needs), so
> 
> linking to github is just going to generate unhappy users (they'll have to
> 
> hand-compile the applet, so install compiler, dev environment and all that),
> I
> 
> think in that light, it's actually less than useful -- you give the
> impression
> 
> that the user can easily install it, and then the user ends up on github
> (or,
> 
> who-knows-where if they used the "Download new Plasmoids" dialog), so it
> 
> simply won't work.
> 
> 
> 
> But, there's a solution: You could maintain the code in KDE's git repos, and
> 
> we put the applet in kdeplasma-addons.
> 
> 
> 
> Is this something you would consider?
> 
> 
> 
> Cheers,

-- 
sebas

http://www.kde.org | http://vizZzion.org


More information about the Plasma-devel mailing list