Skip to content

Commit 3dc56a7

Browse files
charles-typfacebook-github-bot
authored andcommitted
Fix OSS build cpp_include path handling
Summary: The OSS build install script was missing sed handling for `cpp_include` directives in Thrift files, which caused build failures when the thrift compiler processed `UcacheBenchService.thrift` with hardcoded internal fbcode paths. Reviewed By: excelle08 Differential Revision: D92609819
1 parent d6e9533 commit 3dc56a7

1 file changed

Lines changed: 2 additions & 0 deletions

File tree

packages/ucache_bench/install_ucache_bench.sh

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -561,6 +561,8 @@ for f in "$PROTOCOL_GEN_DIR"/*.cpp "$PROTOCOL_GEN_DIR"/*.h "$PROTOCOL_GEN_DIR"/*
561561
sed -i 's|#include "cea/chips/benchpress/packages/ucache_bench/server/|#include "../server/|g' "$f"
562562
# Fix thrift include paths (for .thrift files)
563563
sed -i 's|include "cea/chips/benchpress/packages/ucache_bench/protocol/gen/|include "|g' "$f"
564+
# Fix cpp_include paths (for .thrift files)
565+
sed -i 's|cpp_include "cea/chips/benchpress/packages/ucache_bench/protocol/gen/|cpp_include "|g' "$f"
564566
fi
565567
done
566568

0 commit comments

Comments
 (0)