<div dir="ltr">Erm... I would really expect that the version without Vc would go into a separate branch, not doing any reverts in master. Now you made all the people who already built Vc 1.2 rebuild Krita again and waste about an hour of their time.<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Apr 9, 2016 at 1:39 PM, Boudewijn Rempt <span dir="ltr"><<a href="mailto:boud@valdyas.org" target="_blank">boud@valdyas.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Making the build work without Vc was too tricky for me, which blocks building<br>
Krita on Windows with MSVC. While I'm working to setup a mingw-build system on<br>
Windows, I've moved the Vc code to a new branch:<br>
<br>
rempt-port-vc<br>
<br>
Where we should work to fix building without Vc. In the meantime, I've reverted<br>
everything in master, so CI and Windows users can at least build again.<br>
<br>
Also note this bug report:<br>
<br>
<a href="https://bugs.kde.org/show_bug.cgi?id=361546" rel="noreferrer" target="_blank">https://bugs.kde.org/show_bug.cgi?id=361546</a><br>
<br>
"The brushes have become really slow in recent 3.0 alpha builds compared to their 2.9 counterparts"<div class="HOEnZb"><div class="h5"><br>
<br>
On Fri, 8 Apr 2016, Friedrich W. H. Kossebau wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi Thorsten,<br>
<br>
seems this breaks non-Vc builds of Krita (to be seen on CI, which still has Vc<br>
0.7.4 and thus tries to build without), e.g.<br>
<br>
<a href="https://build.kde.org/job/krita%20master%20kf5-qt5/" rel="noreferrer" target="_blank">https://build.kde.org/job/krita%20master%20kf5-qt5/</a><br>
PLATFORM=Linux,compiler=gcc/63/<br>
<br>
Making Vc a hard dep would long-term be perhaps the best, but for now Vc 1.0<br>
is not that spread yet, so would put extra burdens to new contributors (&<br>
distributors).<br>
<br>
See below for one thing I found already.<br>
<br>
Am Donnerstag, 7. April 2016, 12:56:24 CEST schrieb Thorsten Zachmann:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Git commit b68c1c7666510e268babc4cd87cd9d5ad34ff2d5 by Thorsten Zachmann.<br>
Committed on 07/04/2016 at 12:54.<br>
Pushed by zachmann into branch 'master'.<br>
<br>
Update krita to compile against Vc >= 1.0.0<br>
<br>
</blockquote>
<br>
<snip><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
diff --git a/libs/pigment/compositeops/KoVcMultiArchBuildSupport.h<br>
b/libs/pigment/compositeops/KoVcMultiArchBuildSupport.h index<br>
09677a6..2b10257 100644<br>
--- a/libs/pigment/compositeops/KoVcMultiArchBuildSupport.h<br>
+++ b/libs/pigment/compositeops/KoVcMultiArchBuildSupport.h<br>
@@ -35,6 +35,7 @@<br>
 #pragma warning ( disable : 4244 )<br>
 #pragma warning ( disable : 4800 )<br>
 #endif<br>
+#include <Vc/global.h><br>
 #include <Vc/Vc><br>
 #include <Vc/support.h><br>
 #if defined _MSC_VER<br>
@@ -44,11 +45,9 @@<br>
 #else /* HAVE_VC */<br>
<br>
 namespace Vc {<br>
-    typedef enum {ScalarImpl} Implementation;<br>
+    typedef enum {ScalarImpl} CurrentImplementation;<br>
 }<br>
</blockquote>
<br>
There still is Vc::Implementation in the code, so should this here be not a<br>
change Implementation->CurrentImplementation, but rather an addition of<br>
CurrentImplementation?<br>
<br>
There is also Vc::CurrentImplementation::current() in the general code, so the<br>
typedef might not be the working solution.<br>
<br>
Cheers<br>
Friedrich<br>
<br>
</blockquote>
<br></div></div><span class="HOEnZb"><font color="#888888">
-- <br>
Boudewijn Rempt | <a href="http://www.krita.org" rel="noreferrer" target="_blank">http://www.krita.org</a>, <a href="http://www.valdyas.org" rel="noreferrer" target="_blank">http://www.valdyas.org</a></font></span><div class="HOEnZb"><div class="h5"><br>
_______________________________________________<br>
Krita mailing list<br>
<a href="mailto:kimageshop@kde.org" target="_blank">kimageshop@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/kimageshop" rel="noreferrer" target="_blank">https://mail.kde.org/mailman/listinfo/kimageshop</a><br>
</div></div></blockquote></div><br><br clear="all"><br>-- <br><div class="gmail_signature">Dmitry Kazakov</div>
</div>