<table><tr><td style="">arojas created this revision.<br />arojas added reviewers: KDE Games, asturmlechner, lbeltrame, fvogt.<br />Herald added a subscriber: kde-games-devel.<br />arojas requested review of this revision.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D27330">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>kajongg's build system uses DESTDIR during cmake to construct the command that creates the kajongg and kajonggserver symlinks in bindir. This is a problem for distributions, which usually have separated ckame/make/make install stages with DESTDIR only available in the later one. So distros are currently working around this in different ways, either creating the symlinks manually after make install (Gentoo, Fedora) or defining DESTDIR before running cmake (Arch, Opensuse).<br />
This patch changes the symlink installation process to only make the DESTDIR substitution during make install.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>cmake && make && make DESTDIR=foo install, symlinks are correctly created inside foo.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R387 Kajongg</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D27330">https://phabricator.kde.org/D27330</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>cmake-modules/KDEPython.cmake</div></div></div><br /><div><strong>To: </strong>arojas, KDE Games, asturmlechner, lbeltrame, fvogt<br /><strong>Cc: </strong>kde-games-devel<br /></div>