
<html>

<head>

<meta http-equiv="Content-Type" content="text/html; charset=gb2312">

<style type="text/css" style="display:none;"> P {margin-top:0;margin-bottom:0;} </style>

</head>

<body dir="ltr">

<div style="font-family: Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0); background-color: rgba(0, 0, 0, 0);">

Hi all, </div>

<div style="font-family: Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0); background-color: rgba(0, 0, 0, 0);">

<br>

</div>

<div style="font-family: Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0); background-color: rgba(0, 0, 0, 0);">

Plasma-Mycroft has been in KDE review over a month, there have been a few additions and bug fixes to the plasmoid but nothing major, all fixes have also been made to the build system which were mentioned by <span style="color: rgb(0, 0, 0); font-family: Calibri, sans-serif; font-size: 14.6667px; font-style: normal; font-variant-ligatures: normal; font-variant-caps: normal; font-weight: 400;">Christophe

 Giboudeaux, I am hoping plasma-mycroft can complete its review soon as I am looking forward to making a stable release. As there haven't also been any other objections currently can this review process move ahead / moved to extra gears. </span><span style="color: rgb(0, 0, 0); font-family: Calibri, sans-serif; font-size: 14.6667px; font-style: normal; font-variant-ligatures: normal; font-variant-caps: normal; font-weight: 400; background-color: rgba(0, 0, 0, 0);"> </span></div>

<div style="font-family: Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">

<br>

</div>

<div style="font-family: Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">

Regards,</div>

<div style="font-family: Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">

Aditya Mehra</div>

<hr style="display:inline-block;width:98%" tabindex="-1">

<div id="divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" style="font-size:11pt" color="#000000"><b>From:</b> Aditya Mehra<br>

<b>Sent:</b> Monday, December 18, 2017 4:56 PM<br>

<b>To:</b> kde-core-devel@kde.org<br>

<b>Subject:</b> Re: Plasma-Mycroft is in kdereview</font>

<div> </div>

</div>

<div dir="ltr">

<div id="x_divtagdefaultwrapper" dir="ltr" style="font-size:12pt; color:#000000; font-family:Calibri,Helvetica,sans-serif">

<p style="margin-top: 0px; margin-bottom: 0px;"><span style="font-family:Calibri,Helvetica,sans-serif,EmojiFont,"Apple Color Emoji","Segoe UI Emoji",NotoColorEmoji,"Segoe UI Symbol","Android Emoji",EmojiSymbols; font-size:16px">Hi, </span></p>

<div style="font-family:Calibri,Helvetica,sans-serif,EmojiFont,"Apple Color Emoji","Segoe UI Emoji",NotoColorEmoji,"Segoe UI Symbol","Android Emoji",EmojiSymbols; font-size:16px">

<br>

</div>

<div style="font-family:Calibri,Helvetica,sans-serif,EmojiFont,"Apple Color Emoji","Segoe UI Emoji",NotoColorEmoji,"Segoe UI Symbol","Android Emoji",EmojiSymbols; font-size:16px">

I have fixed the following:</div>

<div style="font-family:Calibri,Helvetica,sans-serif,EmojiFont,"Apple Color Emoji","Segoe UI Emoji",NotoColorEmoji,"Segoe UI Symbol","Android Emoji",EmojiSymbols; font-size:16px">

- removed Author, Email, Bug_Address from top CMakeLists.txt</div>

<div style="font-family:Calibri,Helvetica,sans-serif,EmojiFont,"Apple Color Emoji","Segoe UI Emoji",NotoColorEmoji,"Segoe UI Symbol","Android Emoji",EmojiSymbols; font-size:16px">

- set destination to use <span style="font-size:14.6667px">${SYSCONF_INSTALL_DIR} </span></div>

<div style="font-family:Calibri,Helvetica,sans-serif,EmojiFont,"Apple Color Emoji","Segoe UI Emoji",NotoColorEmoji,"Segoe UI Symbol","Android Emoji",EmojiSymbols; font-size:16px">

<span style="font-size:14.6667px">- use ecm_install_icons from ecm to install the icons</span></div>

<div style="font-family:Calibri,Helvetica,sans-serif,EmojiFont,"Apple Color Emoji","Segoe UI Emoji",NotoColorEmoji,"Segoe UI Symbol","Android Emoji",EmojiSymbols; font-size:16px">

<span style="font-size:14.6667px">- in plugin/CMakeLists.txt removed unused pkgconfig</span></div>

<div style="font-family:Calibri,Helvetica,sans-serif,EmojiFont,"Apple Color Emoji","Segoe UI Emoji",NotoColorEmoji,"Segoe UI Symbol","Android Emoji",EmojiSymbols; font-size:16px">

- added a ATTRIBUTION file to top directory to mark usage of images (wikipedia icon)</div>

<div style="font-family:Calibri,Helvetica,sans-serif,EmojiFont,"Apple Color Emoji","Segoe UI Emoji",NotoColorEmoji,"Segoe UI Symbol","Android Emoji",EmojiSymbols; font-size:16px">

- added a Copying.apache2 license file as MSM.sh, Mycroft are now Apache 2.0 licensed </div>

<div style="font-family:Calibri,Helvetica,sans-serif,EmojiFont,"Apple Color Emoji","Segoe UI Emoji",NotoColorEmoji,"Segoe UI Symbol","Android Emoji",EmojiSymbols; font-size:16px">

- added license headers to all files in plasmoid/content/ui missing the header</div>

<div style="font-family:Calibri,Helvetica,sans-serif,EmojiFont,"Apple Color Emoji","Segoe UI Emoji",NotoColorEmoji,"Segoe UI Symbol","Android Emoji",EmojiSymbols; font-size:16px">

- removed .directory file in plasmoid/contents/images</div>

<div style="font-family:Calibri,Helvetica,sans-serif,EmojiFont,"Apple Color Emoji","Segoe UI Emoji",NotoColorEmoji,"Segoe UI Symbol","Android Emoji",EmojiSymbols; font-size:16px">

<span style="font-size:12pt"><br>

</span></div>

<div style="font-family:Calibri,Helvetica,sans-serif,EmojiFont,"Apple Color Emoji","Segoe UI Emoji",NotoColorEmoji,"Segoe UI Symbol","Android Emoji",EmojiSymbols; font-size:16px">

<span style="font-size:12pt">I am not sure about the COPYING file as I think this should be under GPL-2.1 as most other plasmoid/applets also seemed to be released under this license does this file need to be updated to reflect 2.1 </span>specifically<span style="font-size:12pt"> ?</span></div>

<div style="font-family:Calibri,Helvetica,sans-serif,EmojiFont,"Apple Color Emoji","Segoe UI Emoji",NotoColorEmoji,"Segoe UI Symbol","Android Emoji",EmojiSymbols; font-size:16px">

<span style="font-size:12pt"><br>

</span></div>

<div style="font-family:Calibri,Helvetica,sans-serif,EmojiFont,"Apple Color Emoji","Segoe UI Emoji",NotoColorEmoji,"Segoe UI Symbol","Android Emoji",EmojiSymbols; font-size:16px">

<span style="font-size:12pt">Regards,</span></div>

<div style="font-family:Calibri,Helvetica,sans-serif,EmojiFont,"Apple Color Emoji","Segoe UI Emoji",NotoColorEmoji,"Segoe UI Symbol","Android Emoji",EmojiSymbols; font-size:16px">

<span style="font-size:12pt"></span></div>

<div style="font-family:Calibri,Helvetica,sans-serif,EmojiFont,"Apple Color Emoji","Segoe UI Emoji",NotoColorEmoji,"Segoe UI Symbol","Android Emoji",EmojiSymbols; font-size:16px">

<span style="font-size:12pt">Aditya Mehra</span></div>

<div style="font-family:Calibri,Helvetica,sans-serif,EmojiFont,"Apple Color Emoji","Segoe UI Emoji",NotoColorEmoji,"Segoe UI Symbol","Android Emoji",EmojiSymbols; font-size:16px">

<span style="font-size:12pt"><br>

</span></div>

<p style="margin-top: 0px; margin-bottom: 0px;"></p>

<div style="color:rgb(0,0,0)">

<hr tabindex="-1" style="display:inline-block; width:98%">

<div id="x_divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" color="#000000" style="font-size:11pt"><b>From:</b> kde-core-devel <kde-core-devel-bounces@kde.org> on behalf of Christophe Giboudeaux <christophe@krop.fr><br>

<b>Sent:</b> Friday, December 15, 2017 1:40 AM<br>

<b>To:</b> kde-core-devel@kde.org<br>

<b>Subject:</b> Re: Plasma-Mycroft is in kdereview</font>

<div> </div>

</div>

<div class="x_BodyFragment"><font size="2"><span style="font-size:11pt">

<div class="x_PlainText">Hi,<br>

<br>

On mardi 5 décembre 2017 08:52:18 CET Aditya Mehra wrote:<br>

> Hi all,<br>

> <br>

> This is a request email for the review process of the mycroft plasmoid, the<br>

> plasma-mycroft project has been moved to kdereview<br>

> <br>

> The repository url: <a href="https://cgit.kde.org/plasma-mycroft.git/" id="LPlnk522896">

https://cgit.kde.org/plasma-mycroft.git/</a><br>

> <br>

<br>

Just commenting on the build system part.<br>

In the top CMakeLists.txt :<br>

-  AUTHOR, EMAIL, BUG_ADDRESS are set but not used anywhere, just remove them.<br>

- install( DIRECTORY mycroft DESTINATION /etc ) → use ${SYSCONF_INSTALL_DIR} <br>

instead of /etc<br>

- Instead of using install( DIRECTORY ...), please use ecm_install_icons from <br>

ECM.<br>

- in plugin/CMakeLists.txt, you include pkgconfig but don't use it for <br>

anything<br>

<br>

other notes :<br>

- COPYING contains the GPL-2.0 license but nothing needs it<br>

- msm.sh is GPL-3.0+ but there's no license file in the repository<br>

- There's a wikipedia icon in the repo, don't forget the terms (the <br>

attribution part) : <a href="https://creativecommons.org/licenses/by-sa/3.0/" id="LPlnk182597">

https://creativecommons.org/licenses/by-sa/3.0/</a><br>

<br>

- some files in plasmoid/content/ui don't have a license header<br>

- there's a .directory file in plasmoid/contents/images, this should probably <br>

not be in the repo :)<br>

<br>

That's all.<br>

<br>

<br>

<br>

<br>

<br>

<br>

<br>

</div>

</span></font></div>

</div>

</div>

</div>

</body>

</html>