Bug 335057 - CMakeLists.txt is outdated
Summary: CMakeLists.txt is outdated
Status: RESOLVED FIXED
Alias: None
Product: kajongg
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: Gentoo Packages Linux
: NOR normal
Target Milestone: ---
Assignee: Wolfgang Rohdewald
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-05-20 00:50 UTC by Martin Walch
Modified: 2014-05-21 06:29 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In: 4.13.2


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Martin Walch 2014-05-20 00:50:01 UTC
Maybe I am wrong, but I think the CMakeLists.txt from kajongg is outdated. At least, I can currently not create a working version without adding several Python source files to CMakeLists.txt (src/configdialog.py, src/qt.py, …)
Comment 1 Wolfgang Rohdewald 2014-05-20 04:49:21 UTC
You are right, but only for the master branch. Which surprises me, I was assuming that cmake is only used by packagers.

The release branches KDE/4.X should not have such a problem, and at least configdialog.py is indeed not used in any of them.

configdialog.py and others are only used when there is no usable PyKDE binding installed, they contain stuff that emulates PyKDE (not yet as good as I want to). This lets me run kajongg on Windows as a pure Qt program, and it might perhaps help with the migration to Qt5/KF5

qt*.py hides differences between qt4 and qt5.
Comment 2 Martin Walch 2014-05-20 07:44:13 UTC
(In reply to comment #1)
> You are right, but only for the master branch. Which surprises me, I was
> assuming that cmake is only used by packagers.
> 
> The release branches KDE/4.X should not have such a problem, and at least
> configdialog.py is indeed not used in any of them.

Gentoo builds KDE packages using the cmake infrastructure. I actually came across the problem, because I can not get kajongg to work on Gentoo with the latest ebuild which uses the 4.13.1 source tarball.

Are you sure 4.13.1 should work and this should be fixed downstream?
Comment 3 Wolfgang Rohdewald 2014-05-21 06:27:09 UTC
Git commit 985bf653088e53483722009f6277dd439423fbfa by Wolfgang Rohdewald.
Committed on 21/05/2014 at 06:06.
Pushed by wrohdewald into branch 'KDE/4.13'.

CMakeLists.txt: add missing .py files
FIXED-IN: 4.13.2

M  +5    -0    CMakeLists.txt

http://commits.kde.org/kajongg/985bf653088e53483722009f6277dd439423fbfa
Comment 4 Wolfgang Rohdewald 2014-05-21 06:29:52 UTC
Git commit f6c50a41d8fccfb212abf1f3e1caa2460b73c18f by Wolfgang Rohdewald.
Committed on 21/05/2014 at 06:06.
Pushed by wrohdewald into branch 'master'.

CMakeLists.txt: add missing .py files
FIXED-IN: 4.13.2

M  +5    -0    CMakeLists.txt

http://commits.kde.org/kajongg/f6c50a41d8fccfb212abf1f3e1caa2460b73c18f