
<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 id="divtagdefaultwrapper" style="font-size:12pt;color:#000000;font-family:Calibri,Helvetica,sans-serif;" dir="ltr">

<p><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></p>

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

<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> 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="BodyFragment"><font size="2"><span style="font-size:11pt;">

<div class="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" previewremoved="true">

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" previewremoved="true">

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>

</body>

</html>