<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Sep 30, 2014 at 7:44 AM, Gilles Caulier <span dir="ltr"><<a href="mailto:caulier.gilles@gmail.com" target="_blank">caulier.gilles@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class=""><a href="tel:2014-09-30%203" value="+420201409303">2014-09-30 3</a>:06 GMT+02:00 Vishesh Handa <<a href="mailto:me@vhanda.in">me@vhanda.in</a>>:<br>
> Hey Tobias<br>
><br>
> Some comments about the code -<br>
><br>
> 1. The code seems to be licensed under GPL. In order to make it into a<br>
> framework, it will need to be re-licensed. This library seems like an ideal<br>
> candidate for becoming a framework.<br>
<br>
</span>libkface have been writted in same way than libkipi, libkexiv2, and<br>
libkdcraw, already in KDEGraphics.<br></blockquote><div><br></div><div>The thing is - if libkface is set to become a framework and be part of the KDE Frameworks effort, it has to follow KDE Frameworks policies and rules. One of those is that the code is licensed under LGPLv2.1+ (I think, someone correct me if I'm wrong). So libkface would have to be relicensed. Unfortunately same for the other listed libraries if they should become frameworks. And we would most certainly welcome that ;)</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">
><br>
> 2. The copyright header seems to say "Part of the Digikam Project". You may<br>
> want to change that.<br>
<br>
</span>Idem here. libkface follow exactly the same way than libkipi,<br>
libkexiv2, libkdcraw.<br></blockquote><div><br></div><div>Personally I see nothing wrong with the "Part of the Digikam Project"; if that framework is being developed as part of Digikam project, then why not (we still have many files with "this is part of kdelibs project" btw).</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">
><br>
> 4. The coding style uses seems a little unorthdox. Could you perhaps add a<br>
> link to where one can know what style is being followed? Maybe this could go<br>
> in the README file.<br>
<br>
</span>coding style follow instructions from digiKam project :<br></blockquote><div><br></div><div>That's another policy of the Frameworks - we try to have a consistent coding style all over Frameworks, that means that libkface would have to start following the same style in order to be included in Frameworks.</div><div><br></div></div><div>Cheers</div>-- <br><div><span style="color:rgb(102,102,102)">Martin Klapetek | KDE Developer</span></div>
</div></div>