<div dir="ltr">Hi there<div><br></div><div>Thanks a lot :)</div><div><br></div><div>1) You can modify that "Created by line" to mention you :)</div><div>2) You could probably improve that Makefile a bit, by using FOO_CMAKE_BOOL [1]. </div><div>3) I'm personally also not a fan of the Makefile.common used in digikam-kde4, </div><div>    and now in this one too, it makes it more confusing to me [also it contains kde4 bits].</div><div><br></div><div>If you wan't I can import it into plasma5/PORTS :)</div><div><br></div><div><br></div><div>mfg Tobias</div><div><br></div><div><br></div><div>[1] <a href="https://www.freebsd.org/doc/en/books/porters-handbook/makefile-options.html#options-cmake_bool">https://www.freebsd.org/doc/en/books/porters-handbook/makefile-options.html#options-cmake_bool</a> </div></div><div class="gmail_extra"><br><div class="gmail_quote">On 29 August 2016 at 06:46, Thomas Legg <span dir="ltr"><<a href="mailto:tjlegg@gmail.com" target="_blank">tjlegg@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Would love some feedback, especially on the USES, KDE, and QT5 sections of Makefiles.<div><a href="https://github.com/thomaslegg/digikam-kf5" target="_blank">https://github.com/thomaslegg/<wbr>digikam-kf5</a><br></div><div><br></div><div>This builds digikam 5.1.0 built around a mutt version of the area51 kf5 branch. Mutt as I've rolled the kdepim back to 16.04 as 16.08 kdepim calendaring requires qt5-webengine, which seems like it's going to require a lot of work to port.</div><div><br></div><div>If using digikam-kde4, I recommend moving the .db files out of your photo repo directories and letting digikam-kde5 build new ones. I haven't tested all of the digikam features so far, but it launches, indexes, builds thumbnails, displays photos, adds tags. More testing required and if anyone else gets this to build, I'd love feedback on your use.</div><span class="HOEnZb"><font color="#888888"><div><br></div><div>Thomas Legg</div></font></span></div>
</blockquote></div><br></div>