Page MenuHomePhabricator

[CMAKE] Install executables
ClosedPublic

Authored by Fabien on Tue, Sep 24, 14:45.

Details

Summary

This diff adds the rules to install the built executables.

Test Plan

For all platforms:

mkdir -p buildcmake/install && cd install
cmake -GNinja .. -DCMAKE_INSTALL_PREFIX="install" # Other options

depending on the platform

ninja
ninja install

Check that the executables are installed under install/bin/.

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

Fabien created this revision.Tue, Sep 24, 14:45
Herald added a reviewer: Restricted Project. · View Herald TranscriptTue, Sep 24, 14:45
deadalnix accepted this revision.Thu, Sep 26, 10:19
deadalnix added inline comments.
cmake/modules/InstallationHelper.cmake
5 ↗(On Diff #13113)

Why is that a macro rather than a function?

This revision is now accepted and ready to land.Thu, Sep 26, 10:19
Fabien added inline comments.Thu, Sep 26, 11:51
cmake/modules/InstallationHelper.cmake
5 ↗(On Diff #13113)

Because I have no need to separate the scope from the caller in this case. A function would work the same with more overhead.

Fabien updated this revision to Diff 13146.Thu, Sep 26, 14:32

Rebase.

This revision was automatically updated to reflect the committed changes.