Merge pull request #154 from ElrondNetwork/fix-env-rust

Hot fix: Rust env for contract reports is missing system PATH
multiversx · Sep 23, 2022 · a6b4819 · a6b4819
2 parents 879ff75 + 25eedbf
commit a6b4819
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 3 deletions.
diff --git a/erdpy/CHANGELOG.md b/erdpy/CHANGELOG.md
@@ -7,6 +7,9 @@ Check [Keep a Changelog](http://keepachangelog.com/) for recommendations on how
 ## [Unreleased]
  - TBD
 
+## [2.0.3]
+ - [Hot fix: Rust env for contract reports is missing system PATH](https://github.com/ElrondNetwork/elrond-sdk-erdpy/pull/154)
+
 ## [2.0.2]
  - [Fix environment variables for Rust](https://github.com/ElrondNetwork/elrond-sdk-erdpy/pull/153)
 

diff --git a/erdpy/_version.py b/erdpy/_version.py
@@ -1 +1 @@
-__version__ = "2.0.2"
+__version__ = "2.0.3"
diff --git a/erdpy/dependencies/modules.py b/erdpy/dependencies/modules.py
@@ -350,7 +350,8 @@ def get_env(self):
         directory = self.get_directory("")
 
         return {
-            "PATH": f"{path.join(directory, 'bin')}",
+            # At this moment, cc (build-essential) is sometimes required by the meta crate (e.g. for reports)
+            "PATH": f"{path.join(directory, 'bin')}:{os.environ['PATH']}",
             "RUSTUP_HOME": directory,
             "CARGO_HOME": directory
         }
@@ -359,6 +360,7 @@ def get_env_for_is_installed(self):
         directory = self.get_directory("")
 
         return {
+            # Here, we do not include the system PATH
             "PATH": f"{path.join(directory, 'bin')}",
             "RUSTUP_HOME": directory,
             "CARGO_HOME": directory

diff --git a/setup.py b/setup.py
@@ -3,7 +3,7 @@
 with open("README.md", "r") as fh:
     long_description = "https://github.com/ElrondNetwork/elrond-sdk-erdpy"
 
-VERSION = "2.0.2"
+VERSION = "2.0.3"
 
 try:
     with open('./erdpy/_version.py', 'wt') as versionfile: