Skip to content

Commit

Permalink
Fix linking for boost::program_options, fix ubuntu packages
Browse files Browse the repository at this point in the history
Summary:
program_options is not a header only library, and has linking
failures. Also fix resolution of ubuntu packages, which replace "_" with "-".

Reviewed By: bigfootjon

Differential Revision: D66305639

fbshipit-source-id: 2486cc5cf3885b09ad2228911022023840c0f7b7
  • Loading branch information
ckwalsh authored and facebook-github-bot committed Nov 21, 2024
1 parent fe8b492 commit 90eba8f
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 4 deletions.
2 changes: 1 addition & 1 deletion third-party/boost/BUCK
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,6 @@ boost_libs(
"lexical_cast",
"multi_index",
"preprocessor",
"program_options",
"range",
"sort",
"variant",
Expand All @@ -25,6 +24,7 @@ boost_libs(
"container",
"context",
"filesystem",
"program_options",
"random",
"regex",
"thread",
Expand Down
4 changes: 1 addition & 3 deletions third-party/boost/boost.bzl
Original file line number Diff line number Diff line change
Expand Up @@ -7,8 +7,6 @@

load("@//third-party:defs.bzl", "system_library")

HOMEBREW_BREW = "boost"

def boost_libs(libraries, header_only):
system_library(
name = "boost",
Expand All @@ -33,7 +31,7 @@ def boost_library(library: str, header_only: bool):
packages = {
"//os:linux-fedora": ["boost-devel"],
"//os:linux-ubuntu": [
"libboost-dev" if header_only else "libboost-{}-dev".format(library),
"libboost-dev" if header_only else "libboost-{}-dev".format(library.replace("_", "-")),
],
"//os:macos-homebrew": ["boost"],
},
Expand Down

0 comments on commit 90eba8f

Please sign in to comment.