verilator: Some environment varibles are no longer required (#44655)
* verilator: Some environment varibles are no longer required * Revert depends_on for flex back to the deafult case * Use decorator, when, for setup_run_environment insdtead of if inside function
This commit is contained in:
		| @@ -76,15 +76,17 @@ class Verilator(AutotoolsPackage): | ||||
|     depends_on("help2man", type="build") | ||||
|     depends_on("bison", type="build") | ||||
|     depends_on("flex") | ||||
|     depends_on("ccache", type=("build", "run"), when="@5.018:") | ||||
|     depends_on("perl", type=("build", "run")) | ||||
|     depends_on("ccache", type=("build", "run"), when="@5.018:") | ||||
| 
 | ||||
|     conflicts("%gcc@:6", msg="C++14 support required") | ||||
| 
 | ||||
|     # we need to fix the CXX and LINK paths, as they point to the spack | ||||
|     # wrapper scripts which aren't usable without spack | ||||
|     filter_compiler_wrappers("verilated.mk", relative_root="include") | ||||
|     filter_compiler_wrappers("verilated.mk", relative_root="share/verilator/include") | ||||
| 
 | ||||
|     @when("@:5.022") | ||||
|     def setup_run_environment(self, env): | ||||
|         env.prepend_path("VERILATOR_ROOT", self.prefix) | ||||
| 
 | ||||
|   | ||||
		Reference in New Issue
	
	Block a user
	 Dave Keeshan
					Dave Keeshan