Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 1091671 - code-editor bundles qtsingleapplication (and qtlockedfile)
Summary: code-editor bundles qtsingleapplication (and qtlockedfile)
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: code-editor
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Ilyes Gouta
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: DuplicSysLibsTracker
TreeView+ depends on / blocked
 
Reported: 2014-04-27 07:55 UTC by Ville Skyttä
Modified: 2014-05-12 13:02 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2014-05-12 13:02:41 UTC
Type: Bug
Embargoed:

Attachments (Terms of Use)

Description Ville Skyttä 2014-04-27 07:55:23 UTC 
https://fedoraproject.org/wiki/Packaging:No_Bundled_Libraries
https://fedoraproject.org/wiki/Packaging:Treatment_Of_Bundled_Libraries

code-editor is using a bundled version of qtsingleapplication (and its dependency, qtlockedfile). Looks like the bundled version is newer than the system one and isn't switchable without code changes, but should be handled using one of the ways documented in the links above.
Comment 1 Ilyes Gouta 2014-04-27 09:53:58 UTC 
Hi Ville,

$ rpm -qlp code-editor-2.8.1-5.fc20.x86_64.rpm | grep qtsingleapplication

doesn't showup anything.

or Am I missing something?

Ilyes
Comment 2 Ville Skyttä 2014-04-27 10:29:12 UTC 
Yes, you cannot check for code bundling like that.

Just do a "ls src/shared/qt*" in the expanded source tarball dir, and the existence of those sources in the debuginfo package is strong evindence that they got also used:

$ repoquery --repoid=rawhide-debuginfo -l code-editor-debuginfo | grep src/shared/qt
/usr/src/debug/code-editor/src/shared/qtlockedfile
/usr/src/debug/code-editor/src/shared/qtlockedfile/qtlockedfile.cpp
/usr/src/debug/code-editor/src/shared/qtlockedfile/qtlockedfile.h
/usr/src/debug/code-editor/src/shared/qtlockedfile/qtlockedfile_unix.cpp
/usr/src/debug/code-editor/src/shared/qtsingleapplication
/usr/src/debug/code-editor/src/shared/qtsingleapplication/qtlocalpeer.cpp
/usr/src/debug/code-editor/src/shared/qtsingleapplication/qtlocalpeer.h
/usr/src/debug/code-editor/src/shared/qtsingleapplication/qtsingleapplication.cpp
/usr/src/debug/code-editor/src/shared/qtsingleapplication/qtsingleapplication.h

(qtlockedfile is qtsingleapplication's dependency so solving the latter will solve both issues)
Comment 3 Ilyes Gouta 2014-05-04 14:10:24 UTC 
Hi Ville,

I had a look at the source code and found that indeed qtsingleapplication and qtlockedfile have been integrated in QtCreator's source code (which code-editor is based on) however those respective classes were placed (by the original upstream) in the dedicated C++ SharedTools namespace so that they won't clash with what the qtsingleapplication package is providing for Fedora (including the debug symbols).

The change is documented here:

https://qt.gitorious.org/qt-creator/code-editor/source/46cf4248615ab6437421a7e6cfa4bd8bbd1cbb7f:src/shared/qtsingleapplication/README.txt

Would this alleviate/solve the issue?

Regards,
Comment 4 Ville Skyttä 2014-05-05 06:44:20 UTC 
No, please read the docs linked to in the initial comment. Either the bundling needs to be removed by using the system library, or you'll need to get an exception for the bundling from FPC and mark it as appropriate in the specfile. Here's that link again:

https://fedoraproject.org/wiki/Packaging:No_Bundled_Libraries#Exceptions
Comment 5 Ilyes Gouta 2014-05-11 16:02:38 UTC 
(In reply to Ville Skyttä from comment #4)
> No, please read the docs linked to in the initial comment. Either the
> bundling needs to be removed by using the system library,

This is likely feasible and I'm checking it right now. I'll be updating the ticket with my findings.

Thanks,
Comment 6 Ilyes Gouta 2014-05-11 23:15:08 UTC 
(In reply to Ilyes Gouta from comment #5)
> This is likely feasible and I'm checking it right now. I'll be updating the
> ticket with my findings.

I just pushed code-editor-2.8.1-6 to rawhide which has the following patch applied:

https://qt.gitorious.org/qt-creator/code-editor/commit/6fbc52a82bb58c0f5ba1aba43a3c13edce0e0118

(scratch-build: http://koji.fedoraproject.org/koji/taskinfo?taskID=6839762)
Comment 7 Ville Skyttä 2014-05-12 13:02:41 UTC 
Looks good to me on surface. While checking, I took the liberty to do some sources and .gitignore cleanups in devel git.
Note You need to log in before you can comment on or make changes to this bug.