Page MenuHomePhabricator

[CMAKE] Generate the split-debug.sh script
ClosedPublic

Authored by Fabien on Nov 1 2019, 21:06.

Details

Summary

The split-debug.sh script is generated from a template at build time,
and used by the linux gitian build to separate the debug symbols from
the stripped binaries.

Test Plan

For all the linux platforms:

cmake ..

Check the <build_dir>/contrib/devtools/split-debug.sh script is
generated as expected.

Diff Detail

Repository
rABC Bitcoin ABC
Branch
cmake_split_debug
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 7951
Build 13905: Bitcoin ABC Buildbot (legacy)
Build 13904: arc lint + arc unit

Event Timeline

deadalnix requested changes to this revision.Nov 2 2019, 17:02
deadalnix added inline comments.
contrib/devtools/CMakeLists.txt
19 ↗(On Diff #13872)

Can you explain why this dance is necessary?

24 ↗(On Diff #13872)

Does this break deps tracking?

This revision now requires changes to proceed.Nov 2 2019, 17:02
contrib/devtools/CMakeLists.txt
19 ↗(On Diff #13872)

file(COPY/INSTALL) is the only cmake directive to change permissions on a file, and the script needs to be executable while the template should not.

24 ↗(On Diff #13872)

The copy will update the file and cause cmake to re-configure every time, which is not ideal. I'll fix that.

Avoid re-configuring at each call.

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