Skip to content

Commit a6a75aa

Browse files
Chris Dinhfacebook-github-bot
authored andcommitted
Fix for missing paths in simple suffix evaluation
Summary: # Bug Description Watchman would return with missing path components S468253 # This Diff Adds the relative root into the path evaluation # Technical Details Previously without this change, the contents of the EdenFileResult would be missing the relative root component, causing following path evaluators to remove the same number of paths from the head of the desired path Reviewed By: ezzak Differential Revision: D65901018 fbshipit-source-id: b47700a3802a23f59af2b7da7f172f8e0caf631b
1 parent 37830b2 commit a6a75aa

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

watchman/watcher/eden.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -903,7 +903,7 @@ class EdenView final : public QueryableView {
903903
auto file = make_unique<EdenFileResult>(
904904
rootPath_,
905905
thriftChannel_,
906-
w_string::pathCat({mountPoint_, item.name}),
906+
w_string::pathCat({mountPoint_, relative_root, item.name}),
907907
/*ticks=*/nullptr,
908908
/*isNew=*/false,
909909
item.dtype);

0 commit comments

Comments
 (0)