Skip to content

Commit

Permalink
add xcodebuild env var name
Browse files Browse the repository at this point in the history
  • Loading branch information
xiazhvera committed Dec 31, 2024
1 parent 42d15ba commit 70e328a
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion builder/actions/setup_cross_ci_crt_environment.py
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,8 @@
class SetupCrossCICrtEnvironment(Action):

def _setenv(self, env, env_name, env_data, is_secret=False):
if self.is_xcodebuild:
env_name = "TEST_RUNNER_"+env_name
# Kinda silly to have a function for this, but makes the API calls consistent and looks better
# beside the other functions...
env.shell.setenv(env_name, str(env_data), is_secret=is_secret)
Expand Down Expand Up @@ -433,7 +435,7 @@ def _common_setup(self, env):

pass

def run(self, env):
def run(self, is_xcodebuild=False, env):
# A special environment variable indicating that we want to dump test environment variables to a specified file.
env_dump_file = env.shell.getenv("AWS_SETUP_CRT_TEST_ENVIRONMENT_DUMP_FILE")

Expand Down Expand Up @@ -461,6 +463,8 @@ def run(self, env):
self.is_arm = False
# Will be True if on Codebuild
self.is_codebuild = False
# Will be True if is using xcodebuild
self.is_xcodebuild = is_xcodebuild

# Any easier way to use in docker without having to always modify the builder action
if (env.shell.getenv("SETUP_CROSSS_CRT_TEST_ENVIRONMENT_LOCAL", "0") == "1"):
Expand Down

0 comments on commit 70e328a

Please sign in to comment.