Page MenuHomePhabricator

[CMAKE] Add an append_flag_to_variable facility
AbandonedPublic

Authored by Fabien on Mar 6 2019, 11:55.

Details

Reviewers
deadalnix
Group Reviewers
Restricted Project
Summary

This diff adds a macro for appending a flag to a given variable. This is
a refactoring only change.

Depends on D2653

Test Plan
mkdir buildcmake && cd buildcmake
cmake -GNinja ..
ninja -v

Check the flags are correctly added to the command lines.

Diff Detail

Repository
rABC Bitcoin ABC
Branch
cmake_add_flag_to_var
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 5158
Build 8379: Bitcoin ABC Buildbot (legacy)
Build 8378: arc lint + arc unit

Event Timeline

Fabien planned changes to this revision.Mar 6 2019, 11:56
deadalnix requested changes to this revision.Mar 7 2019, 16:45

Unless there is something more that append_flag_to_variable is supposed to do, this really sound to me like the kind of contraption you'd find in J2EE. Command line are space agnostics, so it really doesn't matter, and create a level of indirection for nothing.

This revision now requires changes to proceed.Mar 7 2019, 16:45

This is purely aesthetic, no functional change intended here. If you're uncomfortable with it I can just abandon it, it won't hurt.