Bug 1266712

Summary: trojita: FTBFS in rawhide
Product: [Fedora] Fedora Reporter: Raphael Groner <projects.rg>
Component: trojitaAssignee: Raphael Groner <projects.rg>
Status: CLOSED ERRATA QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: high Docs Contact:
Priority: high    
Version: rawhideCC: kde-sig, kevin, projects.rg
Target Milestone: ---Keywords: Reopened
Target Release: ---   
Hardware: All   
OS: Linux   
URL: https://apps.fedoraproject.org/koschei/package/trojita
Whiteboard:
Fixed In Version: trojita-0.5-8.fc23 trojita-0.5-8.fc22 trojita-0.5a-2.20151216gitefa30f3.fc23 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2015-12-25 00:25:59 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1250005, 1254125, 1256476, 1257267, 1257356    

Description Raphael Groner 2015-09-26 19:27:56 UTC
Description of problem:
Package trojita fails to build from source in rawhide.

Version-Release number of selected component (if applicable):
0.5-7.fc23

Steps to Reproduce:
koji build --scratch f24 trojita-0.5-7.fc23.src.rpm

Additional info:
This package is tracked by Koschei. See:
https://apps.fedoraproject.org/koschei/package/trojita

Comment 1 Kevin Kofler 2015-09-27 23:07:11 UTC
error: definition of implicitly-declared 'Gui::AbookAddressbook::AbookAddressbook()' AbookAddressbook::AbookAddressbook(): m_updateTimer(0)

This appears to be a bug in upstream Trojita.

What the error message means is that if you're going to implement the default constructor with a function (method) definition, e.g.:
AbookAddressbook::AbookAddressbook(): m_updateTimer(0)
{
…
}
you need to explicitly declare it:
AbookAddressbook();
in your class definition. If you don't declare it, a default AbookAddressbook() constructor is inserted, but also with the default definition. I am surprised g++ ever let you get away with such an error.

Comment 2 Raphael Groner 2015-09-29 09:38:05 UTC
Hi Kevin,
thanks for your explanation, I'm already aware of that cause but did not find any spare time to prepare a patch although this FTBFS should be fixed ASAP cause of F23 final freeze is going to happen soon. No idea why g++ did not fail earlier. 
I would be happy if someone can suggest a patch to let me send it to upstream.

Comment 3 Kevin Kofler 2015-10-04 14:40:35 UTC
So I looked at the code, and I'm starting to think it is actually a g++ issue after all. I see this in the header file:

namespace Gui
{

/** @short A generic local adressbook interface*/
class AbookAddressbook : public QObject, public AbstractAddressbook {
    Q_OBJECT
public:
    AbookAddressbook();
…

so this is explicitly declared, why does g++ think it is implicitly declared?

Comment 4 Raphael Groner 2015-10-04 17:11:40 UTC
Kevin, this is no bug of g++. There's a missing inclusion of QObject header, you seem to concentrate at the error of cpp file only, but the parsing of header is failing at first.

Though after have fixed that header inclusion, I stumble upon other errors.

Comment 5 Upstream Release Monitoring 2015-10-05 11:14:49 UTC
raphgro's trojita-0.5-8.fc24 completed http://koji.fedoraproject.org/koji/buildinfo?buildID=689647

Comment 6 Fedora Update System 2015-10-05 12:49:03 UTC
trojita-0.5-8.fc23 has been submitted as an update to Fedora 23. https://bodhi.fedoraproject.org/updates/FEDORA-2015-0c610f1807

Comment 7 Fedora Update System 2015-10-05 13:36:09 UTC
trojita-0.5-8.fc22 has been submitted as an update to Fedora 22. https://bodhi.fedoraproject.org/updates/FEDORA-2015-d40188ca10

Comment 8 Fedora Update System 2015-10-06 01:08:01 UTC
trojita-0.5-8.fc22 has been pushed to the Fedora 22 testing repository. If problems still persist, please make note of it in this bug report.
If you want to test the update, you can install it with
$ su -c 'dnf --enablerepo=updates-testing update trojita'
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2015-d40188ca10

Comment 9 Fedora Update System 2015-10-06 04:22:02 UTC
trojita-0.5-8.fc23 has been pushed to the Fedora 23 testing repository. If problems still persist, please make note of it in this bug report.
If you want to test the update, you can install it with
$ su -c 'dnf --enablerepo=updates-testing update trojita'
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2015-0c610f1807

Comment 10 Fedora Update System 2015-10-12 06:27:48 UTC
trojita-0.5-8.fc23 has been pushed to the Fedora 23 stable repository. If problems still persist, please make note of it in this bug report.

Comment 11 Fedora Update System 2015-10-20 01:55:20 UTC
trojita-0.5-8.fc22 has been pushed to the Fedora 22 stable repository. If problems still persist, please make note of it in this bug report.

Comment 12 Fedora Update System 2015-12-17 10:27:23 UTC
trojita-0.5a-2.20151216gitefa30f3.fc23 has been pushed to the Fedora 23 testing repository. If problems still persist, please make note of it in this bug report.
If you want to test the update, you can install it with
$ su -c 'dnf --enablerepo=updates-testing update trojita'
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2015-afb9844b79

Comment 13 Fedora Update System 2015-12-25 00:25:51 UTC
trojita-0.5a-2.20151216gitefa30f3.fc23 has been pushed to the Fedora 23 stable repository. If problems still persist, please make note of it in this bug report.