Commit 87f0031a authored by Adam Simpkins's avatar Adam Simpkins Committed by Facebook Github Bot

fbcode_builder: fix a minor bug in THRIFT_INCLUDE_DIR argument parsing

Summary:
Fix a minor bug in the argument parsing for `add_fbthrift_cpp_library()`:
only a single value should be supplied for the `THRIFT_INCLUDE_DIR` argument.
This parameter should not accept a list of arguments.

Reviewed By: strager

Differential Revision: D17125810

fbshipit-source-id: 31f354e4d6299d3626a94f3b4b5ff8c58bb0cf7b
parent 50f425bb
...@@ -22,8 +22,8 @@ include(FBCMakeParseArgs) ...@@ -22,8 +22,8 @@ include(FBCMakeParseArgs)
# desired. # desired.
function(add_fbthrift_cpp_library LIB_NAME THRIFT_FILE) function(add_fbthrift_cpp_library LIB_NAME THRIFT_FILE)
# Parse the arguments # Parse the arguments
set(one_value_args INCLUDE_DIR) set(one_value_args INCLUDE_DIR THRIFT_INCLUDE_DIR)
set(multi_value_args SERVICES DEPENDS OPTIONS THRIFT_INCLUDE_DIR) set(multi_value_args SERVICES DEPENDS OPTIONS)
fb_cmake_parse_args( fb_cmake_parse_args(
ARG "" "${one_value_args}" "${multi_value_args}" "${ARGN}" ARG "" "${one_value_args}" "${multi_value_args}" "${ARGN}"
) )
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment