Bug 1111948 (kf5-kio)

Summary: Review Request: kf5-kio - KDE Frameworks 5 Tier 3 solution for filesystem abstraction
Product: [Fedora] Fedora Reporter: Daniel Vrátil <dvratil>
Component: Package ReviewAssignee: Jan Grulich <jgrulich>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: jeischma, jgrulich, ltinkl, package-review
Target Milestone: ---Flags: jgrulich: fedora-review+
gwync: fedora-cvs+
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2014-07-02 14:02:03 UTC Type: ---
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: 1108808, 1108811, 1108812, 1108824, 1108827, 1108830    
Bug Blocks: 656997, 1086148, 1111951, 1111952, 1111954, 1111955, 1111957, 1111963, 1111964, 1111965, 1111968, 1111970, 1111975, 1111981, 1111982    

Description Daniel Vrátil 2014-06-22 11:06:03 UTC
Spec URL: http://dvratil.fedorapeople.org/kf5/review/kf5-kio.spec
SRPM URL: http://dvratil.fedorapeople.org/kf5/review/kf5-kio-4.100.0-1.fc20.src.rpm
Description: KDE Frameworks 5 Tier 3 solution for resource and network access abstraction.
Fedora Account System Username: dvratil

Comment 1 Lukáš Tinkl 2014-06-24 17:50:39 UTC
Build fails with:

/builddir/build/BUILD/kio-4.100.0/src/core/dataprotocol.cpp: In member function 'virtual void KIO::DataProtocol::get(const QUrl&)':
/builddir/build/BUILD/kio-4.100.0/src/core/dataprotocol.cpp:280:89: error: no matching function for call to 'QString::toLatin1() const'
         QTextCodec *codec = QTextCodec::codecForName(hdr.attributes["charset"].toLatin1());
                                                                                         ^
/builddir/build/BUILD/kio-4.100.0/src/core/dataprotocol.cpp:280:89: note: candidates are:
In file included from /usr/include/qt5/QtCore/QString:1:0,
                 from /builddir/build/BUILD/kio-4.100.0/src/core/global.h:23,
                 from /builddir/build/BUILD/kio-4.100.0/src/core/dataslave_p.h:25,
                 from /builddir/build/BUILD/kio-4.100.0/src/core/dataprotocol_p.h:49,
                 from /builddir/build/BUILD/kio-4.100.0/src/core/dataprotocol.cpp:19:
/usr/include/qt5/QtCore/qstring.h:465:16: note: QByteArray QString::toLatin1() const &
     QByteArray toLatin1() const & Q_REQUIRED_RESULT
                ^
/usr/include/qt5/QtCore/qstring.h:465:16: note:   no known conversion for implicit 'this' parameter from 'const QString' to 'const QString&'
/usr/include/qt5/QtCore/qstring.h:467:16: note: QByteArray QString::toLatin1() &&
     QByteArray toLatin1() && Q_REQUIRED_RESULT
                ^
/usr/include/qt5/QtCore/qstring.h:467:16: note:   no known conversion for implicit 'this' parameter from 'const QString' to 'QString&&'
src/core/CMakeFiles/KF5KIOCore.dir/build.make:725: recipe for target 'src/core/CMakeFiles/KF5KIOCore.dir/dataprotocol.cpp.o' failed
make[2]: *** [src/core/CMakeFiles/KF5KIOCore.dir/dataprotocol.cpp.o] Error 1
make[2]: *** Waiting for unfinished jobs....
make[2]: Leaving directory '/builddir/build/BUILD/kio-4.100.0/i686-redhat-linux-gnu'
CMakeFiles/Makefile2:3743: recipe for target 'src/core/CMakeFiles/KF5KIOCore.dir/all' failed
make[1]: *** [src/core/CMakeFiles/KF5KIOCore.dir/all] Error 2
make[1]: Leaving directory '/builddir/build/BUILD/kio-4.100.0/i686-redhat-linux-gnu'
Makefile:120: recipe for target 'all' failed
make: Leaving directory '/builddir/build/BUILD/kio-4.100.0/i686-redhat-linux-gnu'
make: *** [all] Error 2

Comment 2 Daniel Vrátil 2014-06-25 09:16:27 UTC
Builds just fine here and in Copr and the error itself does not make much sense to me. Maybe there's something wrong with your Qt 5 setup or environment?

Comment 3 Jan Grulich 2014-06-27 07:40:00 UTC
I hit the same problem when I was building qt5-qtbase in rawhide, it was an issue in gcc and it worked later with a newer build. I'll finish the review since Lukáš is on PTO.

Comment 4 Jan Grulich 2014-06-27 08:15:27 UTC
1) License - some files are also under GPLv2+, BSD and MIT licenses.
2) Missing call of update-desktop-database, because of a desktop file with a Mimetype entry.
3) Missing %config before files installed to /etc/xdg

Comment 5 Daniel Vrátil 2014-06-28 11:41:05 UTC
Spec URL: http://dvratil.fedorapeople.org/kf5/review/kf5-kio.spec
SRPM URL: http://dvratil.fedorapeople.org/kf5/review/kf5-kio-4.100.0-2.fc20.src.rpm

- fixed licenses
- added %%config
- added update-desktop-database

Comment 6 Jan Grulich 2014-07-01 07:54:11 UTC
It's not necessary to have Requires: kf5-kwallet-libs, just remove it during import. The rest looks good, approved.

Comment 7 Daniel Vrátil 2014-07-01 10:26:45 UTC
New Package SCM Request
=======================
Package Name: kf5-kio
Short Description: KDE Frameworks 5 Tier 3 solution for filesystem abstraction
Upstream URL: http://www.kde.org
Owners: dvratil jgrulich ltinkl kkofler rdieter than
Branches: 
InitialCC:

Comment 8 Gwyn Ciesla 2014-07-01 12:06:24 UTC
Git done (by process-git-requests).

Comment 9 Daniel Vrátil 2014-09-25 09:03:17 UTC
Package Change Request
=======================
Package Name: kf5-kio
New Branches: f20
Owners: dvratil ltinkl jgrulich kkofler rdieter than

Comment 10 Gwyn Ciesla 2014-09-25 13:02:14 UTC
Git done (by process-git-requests).