Merge pull request #5152 from ItsNickBarry/fix-foundry-path

alcuadrado · web-flow · commit e54f4f728220 · 2024-04-23T18:17:10.000-03:00
use absolute path for foundry sources
diff --git a/.changeset/cyan-cheetahs-add.md b/.changeset/cyan-cheetahs-add.md
@@ -0,0 +1,5 @@
+---
+"@nomicfoundation/hardhat-foundry": patch
+---
+
+Use absolute rather than relative path when overriding sources configuration
diff --git a/packages/hardhat-foundry/src/index.ts b/packages/hardhat-foundry/src/index.ts
@@ -59,7 +59,7 @@ extendConfig((config, userConfig) => {
   }
 
   // Set sources path
-  config.paths.sources = foundrySourcesPath;
+  config.paths.sources = path.resolve(config.paths.root, foundrySourcesPath);
 
   // Change hardhat's cache path if it clashes with foundry's
   const foundryCachePath = path.resolve(

Original file line number	Diff line number	Diff line change
`@@ -59,7 +59,7 @@ extendConfig((config, userConfig) => {`
`59`	`59`	`}`
`60`	`60`
`61`	`61`	`// Set sources path`
`62`		`- config.paths.sources = foundrySourcesPath;`
	`62`	`+ config.paths.sources = path.resolve(config.paths.root, foundrySourcesPath);`
`63`	`63`
`64`	`64`	`// Change hardhat's cache path if it clashes with foundry's`
`65`	`65`	`const foundryCachePath = path.resolve(`