<div dir="auto"><br></div><div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, 17 Aug 2023 at 18:29 Carl Schwan <<a href="mailto:carl.schwan@gnupg.com">carl.schwan@gnupg.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;padding-left:1ex;border-left-color:rgb(204,204,204)">On Thursday, August 17, 2023 11:18:24 AM CEST Tomaz Canabrava wrote:<br>
> Hello Fellow KDE Devs,<br>
> <br>
> I'm here, formally asking for a review of the Codevis project, to move<br>
> forward and make it a part of kdesdk.<br>
<br>
Very cool project, I was amazed by the presentation of it from tarcisio at <br>
Akademy.<br>
<br>
> Currently we are using parts of KWdigetsAddons as a submodule<br>
> Most things that are related to buildsystems will be moved to craft /<br>
> kdesrc-build as soon as possible, right now we rely in conan for windows<br>
> and mac, plus a hand-written build script that downloads and builds llvm<br>
> for those platforms.<br>
> <br>
> Things that I know that are out of KDE Accordance:<br>
> - Translation System (uses Qt's tr() system)<br>
<br>
This isn't an issue and we have other KDE projects using the tr() system. But <br>
if you want to port to ki18n, it's best to do it now since you don't seems to <br>
have any translations yet.<br>
<br>
> - Settings System (it uses my own configuration parser that resembles QML)<br>
<br>
Yeah probably best to use kconfigxt or make your configuration parser part of <br>
kconfigxt next gen ;)<br>
<br>
> - Folder naming specification (follows the lakosian naming specification)<br>
<br>
I don't think we have any folder (and file) naming specification in kde, or at <br>
least if we have one, it varies a lot between projects.<br>
<br>
> - CI used is based on Gitlab, but fails on KDE<br>
<br>
When trying to build it on my laptop it failed, due to the requirement of <br>
clang 16. This might also be an issue with the kde ci on tumbleweed.</blockquote><div dir="auto"><br></div><div dir="auto">Carl,</div><div dir="auto"><br></div><div dir="auto">There’s no requirement for clang16 (I build with 15, tarcisio builds with 14, the previous ci had 13, I believe)</div><div dir="auto"><br></div><div dir="auto">Mind if you share the build logs?</div><div dir="auto"><br></div><div dir="auto">Best</div><div dir="auto"><br></div><div dir="auto"><br></div><div dir="auto"><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;padding-left:1ex;border-left-color:rgb(204,204,204)" dir="auto"><br>
<br>
> The current repository of Codevis is:<br>
> <a href="https://invent.kde.org/tcanabrava/codevis" rel="noreferrer" target="_blank">https://invent.kde.org/tcanabrava/codevis</a><br>
> <br>
> The KDE developers on this project are me, tarcisio fischer (that presented<br>
> Codevis on Akademy), and Richard Dale.<br>
> <br>
> Best regards,<br>
> Tomaz<br>
<br>
</blockquote></div></div>