Page MenuHomePhabricator

[CMAKE] Install DLL to bin/ by default
ClosedPublic

Authored by Fabien on Nov 6 2019, 20:15.

Details

Summary

The shared libraries are installed by CMake to lib/ by default, with
the exception of windows DLLs which are installed under bin/.
This behavior was changed in D4229 to reflect the location of the
delivered files from our release.

However it appears that the windows Gitian build is searching for the
DLL files in the bin/ directory, before copying it to lib/. This
workaround is then unnecessary as it is handled by Gitian, and can be
simply removed.

Test Plan

Cross-build for Windows:

ninja install

Diff Detail

Repository
rABC Bitcoin ABC
Branch
cmake_dll_install_default
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 8019
Build 14036: Bitcoin ABC Buildbot (legacy)
Build 14035: arc lint + arc unit

Event Timeline

This revision is now accepted and ready to land.Nov 10 2019, 12:55
This revision was automatically updated to reflect the committed changes.