Remove KDEGAMES_EXPORT
ClosedPublic

Authored by twighk on Sun, Oct 6, 3:08 PM.

Details

Summary

KDEGAMES_EXPORT Macro causes many linking errors when compiling with craft on windows using mingw.

for example:
CMakeFiles/kreversi.dir/highscores.cpp.obj: In function `KExtHighscore::ExtManager::ExtManager()':
C:/CraftRoot/build/_/27965d37/kreversi-19.08.1/highscores.cpp:33: undefined reference to `impZN13KExtHighscore7Ma
nagerC2Ejj'
C:/CraftRoot/build/_/27965d37/kreversi-19.08.1/highscores.cpp:35: undefined reference to `impZN13KExtHighscore7Ma
nager11setShowModeENS0_8ShowModeE'
C:/CraftRoot/build/_/27965d37/kreversi-19.08.1/highscores.cpp:36: undefined reference to `impZN13KExtHighscore7Ma
nager17setShowStatisticsEb'
C:/CraftRoot/build/_/27965d37/kreversi-19.08.1/highscores.cpp:37: undefined reference to `impZN13KExtHighscore7Ma
nager25setShowDrawGamesStatisticEb'
C:/CraftRoot/build/_/27965d37/kreversi-19.08.1/highscores.cpp:43: undefined reference to `impZN13KExtHighscore7Ma
nager17setScoreHistogramERK7QVectorIjENS0_14ScoreTypeBoundE'

Diff Detail

Repository
R411 KReversi
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
twighk created this revision.Sun, Oct 6, 3:08 PM
Restricted Application added a reviewer: KDE Games. · View Herald TranscriptSun, Oct 6, 3:08 PM
Restricted Application added a subscriber: kde-games-devel. · View Herald Transcript
twighk requested review of this revision.Sun, Oct 6, 3:08 PM
aacid accepted this revision.Mon, Oct 7, 9:16 PM
aacid added a subscriber: aacid.

Nice patch.

I see you've been working on getting things compiling on Windows, nice work :)

Do you know of the initiative we have for getting some applications to the Windows Store? Want to join?

This revision is now accepted and ready to land.Mon, Oct 7, 9:16 PM
This revision was automatically updated to reflect the committed changes.
twighk added a comment.Wed, Oct 9, 7:49 PM

Nice patch.

I see you've been working on getting things compiling on Windows, nice work :)

Do you know of the initiative we have for getting some applications to the Windows Store? Want to join?

Thanks ! I didn't know there were any initatives, do you know where i could find more information about it?