Bug 395227 - Latest KMM Master Version is aborting at 29% of the build
Summary: Latest KMM Master Version is aborting at 29% of the build
Status: RESOLVED FIXED
Alias: None
Product: kmymoney
Classification: Applications
Component: buildsystem (show other bugs)
Version: git (master)
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: ---
Assignee: KMyMoney Devel Mailing List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-06-10 21:24 UTC by Michael Carpino
Modified: 2018-06-22 14:13 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments
Output regarding abort (89.15 KB, text/plain)
2018-06-10 21:24 UTC, Michael Carpino
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Carpino 2018-06-10 21:24:52 UTC
Created attachment 113206 [details]
Output regarding abort

I'm on Manjaro RELEASE=17.1.10 with Kernel 4.17.0-2 and the build is aborting,  See the attachment for the output produced with the abort.
Comment 1 Jack 2018-06-10 22:17:10 UTC
Please specify the version of qt you have, I see cmake (3.11), gcc (8.1.1), qt, kde appllications (5.46.0), and kde-frameworks (5.8.2) from the output you attached.)  I have been having this problem for some time, but nobody else has been unable to replicate it, so any evidence that will help figure out the actual cause will be helpful.  (I'm on Artix LInux, and I think all my versions are the same or higher, with qt 5.11)  I still suspect some gcc difference, but have no real proof.  I would suspect qt, but I'm pretty sure we differ by minor version only, and they don't break API compatability within major version.
Comment 2 Michael Carpino 2018-06-10 22:37:21 UTC
I'm running qt version 5.11.0-1.  Let me know if there is anything else.  I updated KMM fine last weekend and then I received probably 200 updates for various programs including the latest kernel yesterday and today.  Then I manually tried to update KMM and then the abort happened.
Comment 3 Thomas Baumgart 2018-06-11 07:08:36 UTC
This is the part that makes me wonder:

no known conversion for argument 1 from ‘QHeaderView*’ to ‘const QObject*’
Comment 4 Jack 2018-06-19 17:33:57 UTC
I agree with Thomas in Comment 3 that this particular error just doesn't really make sense, as following that arguement all the way through the templates and macros (assuming I've done it correctly) it does seem the correct type.

I marked the bug as Confirmed, since it's showed up more than once, and changed from Manjaro to compiled sources, as this is not depndent on distribution.

Also - I just got the same error compiling with craft on Windows 10.  It said [727/995] so I assume it was nina and not make, although it shouldn't matter.  What is odd (from my perspective is that gcc is 7.3.0, so gcc 8 is not required for this error.  Frameworks are 5.47 and qt is 5.11 so I'm now thinking it is most likely one of them - although I don't think either is supposed to break API compatability.  I wonder if this is worth posting to some QT mailing list or forum?  Any other ideas?
Comment 5 Jack 2018-06-21 14:56:18 UTC
The patch just committed by Jonathan Riddell seems to have fixed this, at least on Artix Linux.  I will test later with craft on Win 10.  

My guess now is that between QT 5.9 and QT 5.11, "#include <QHeaderView>" was dropped from some other header file where it had indirectly been included.  Now we need to explicitly include it.
Comment 6 Michael Carpino 2018-06-21 17:18:16 UTC
Thanks for the update, Jack.  The patch allowed me to complete the install as well.  I'll leave the Bug open for a few weeks to see if anyone still has any issues.  If nobody comments I'll then close the Bug.
Comment 7 Jack 2018-06-21 23:27:29 UTC
Interestingly, on Artix Linux I actually needed one more patch to complete the build - adding "#include <QAbstractItemView>" to kmymoney/views/newtransactioneditor.cpp.  The craft build is running now.
Comment 8 Thomas Baumgart 2018-06-22 14:10:05 UTC
Git commit 61fff98b33b2127c8a07e5c41b8b1e962f501676 by Thomas Baumgart.
Committed on 22/06/2018 at 14:10.
Pushed by tbaumgart into branch 'master'.

Fix compilation with Qt5.11

M  +1    -0    kmymoney/views/newtransactioneditor.cpp

https://commits.kde.org/kmymoney/61fff98b33b2127c8a07e5c41b8b1e962f501676
Comment 9 Thomas Baumgart 2018-06-22 14:13:10 UTC
The problem you mention in comment 7 is due to the fact, that you have enabled the non-completed features which others have not. So it shows up for you and not for them. I added a fix (see comment 8) to master and close this one.