Skip to content
This repository has been archived by the owner on Jan 13, 2022. It is now read-only.

Proper Installation #29

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Proper Installation #29

wants to merge 3 commits into from

Conversation

TheComet
Copy link

@TheComet TheComet commented Sep 5, 2014

CMake lets you define CMAKE_INSTALL_PREFIX on the command line, giving the user the ability to choose an installation directory. This defaults to /usr/local on linux and C:\Program Files on Windows.

Usually this is enabled for every platform, but it isn't executed on Windows unless you actually run the "install" target.

png85 added a commit to png85/RakNet that referenced this pull request Sep 19, 2015
@Luke1410
Copy link

Changes to ReplicManager3 integrated in https://github.com/SLikeSoft/SLikeNet (incorporated in SLikeNet 0.1.0).

@Luke1410
Copy link

Due to lack of a different way to reach out to you, I'm adding this one as a comment to your pull request:

Since we incorporated (part of) your pull request in SLikeNet, we'd like to offer you to be added to the acknowledgement section in the accompanying readme file.

By default the entry would look like:
TheComet93: https://github.com/TheComet93 (pull request: RAKPR_29)

Please let me know (preferably by email) if we'd add such an entry in the next release of SLikeNet for you (and in case you'd prefer a different style, how it should look like). If there's no reply from your side, we'll not add such an entry to respect your privacy. Note that for those who decided not to be listed in the acknowledgement section and/or those we failed to contact, we added the following generic statement to cover these contributions:
"We'd also like to thank those contributors who have requested to remain
anonymous and/or those who we could not contact at all (due to lack of contact
information)."

@Luke1410
Copy link

Luke1410 commented Jan 8, 2018

Changes to the cmake files were integrated in https://github.com/SLikeSoft/SLikeNet (partially to be released as part of 0.1.1 (i.e. fix for missing installed headers with the DLL-project); remaining part (change to relative install paths) in 0.2.0).

Please note that we did not include the change setting the install-directive for Windows platforms intentionally, since on Windows installing files into lib/include directories is uncommon practice.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants