buck: HaskellGhciRule: getCellUnsafeRelPath -> getRelativePath and provide ProjectFilesystem as a parameter

luciang · facebook-github-bot · commit 613b45483a96 · 2021-09-27T15:41:02.000-07:00
Summary: {D22289392} added `getCellUnsafeRelPath` and recommends replacing it with a call to "getRelativePath and provide ProjectFilesystem as a parameter".

Reviewed By: cjhopman

fbshipit-source-id: f62b67592864bc601f577359279782c4338e206f
diff --git a/src/com/facebook/buck/features/haskell/HaskellGhciRule.java b/src/com/facebook/buck/features/haskell/HaskellGhciRule.java
@@ -295,7 +295,7 @@ public ResolveAndSymlinkStep(
     @Override
     public StepExecutionResult execute(StepExecutionContext context)
         throws IOException, InterruptedException {
-      Path src = resolver.getCellUnsafeRelPath(lib).getPath().toRealPath();
+      Path src = resolver.getRelativePath(getProjectFilesystem(), lib).getPath().toRealPath();
       Path dest = symlinkDir.resolve(name);
       return SymlinkFileStep.of(getProjectFilesystem(), src, dest).execute(context);
     }

Original file line number	Diff line number	Diff line change
`@@ -295,7 +295,7 @@ public ResolveAndSymlinkStep(`
`295`	`295`	`@Override`
`296`	`296`	`public StepExecutionResult execute(StepExecutionContext context)`
`297`	`297`	`throws IOException, InterruptedException {`
`298`		`- Path src = resolver.getCellUnsafeRelPath(lib).getPath().toRealPath();`
	`298`	`+ Path src = resolver.getRelativePath(getProjectFilesystem(), lib).getPath().toRealPath();`
`299`	`299`	`Path dest = symlinkDir.resolve(name);`
`300`	`300`	`return SymlinkFileStep.of(getProjectFilesystem(), src, dest).execute(context);`
`301`	`301`	`}`