Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
Don't use ::set-env in python-setup
Rasmus Wriedt Larsen committed Oct 8, 2020

Unverified

No user is associated with the committer email.
1 parent b1e2c9b commit 64ebf10
Showing 2 changed files with 10 additions and 3 deletions.
5 changes: 4 additions & 1 deletion python-setup/auto_install_packages.py
@@ -142,5 +142,8 @@ def install_packages(codeql_base_dir) -> Optional[str]:
python_executable_path = install_packages(codeql_base_dir)

if python_executable_path is not None:
# see https://docs.github.com/en/free-pro-team@latest/actions/reference/workflow-commands-for-github-actions#setting-an-environment-variable
env_file = open(os.environ["GITHUB_ENV"], mode="at")

print("Setting CODEQL_PYTHON={}".format(python_executable_path))
print("::set-env name=CODEQL_PYTHON::{}".format(python_executable_path))
print("CODEQL_PYTHON={}".format(python_executable_path), file=env_file)
8 changes: 6 additions & 2 deletions python-setup/tests/from_python_exe.py
@@ -1,6 +1,7 @@
#!/usr/bin/env python3

import sys
import os
import subprocess
from typing import Tuple

@@ -24,8 +25,11 @@ def get_details(path_to_python_exe: str) -> Tuple[str, str]:
if __name__ == "__main__":
version, import_path = get_details(sys.argv[1])

# see https://docs.github.com/en/free-pro-team@latest/actions/reference/workflow-commands-for-github-actions#setting-an-environment-variable
env_file = open(os.environ["GITHUB_ENV"], mode="at")

print("Setting LGTM_PYTHON_SETUP_VERSION={}".format(version))
print("::set-env name=LGTM_PYTHON_SETUP_VERSION::{}".format(version))
print("LGTM_PYTHON_SETUP_VERSION={}".format(version), file=env_file)

print("Setting LGTM_INDEX_IMPORT_PATH={}".format(import_path))
print("::set-env name=LGTM_INDEX_IMPORT_PATH::{}".format(import_path))
print("LGTM_INDEX_IMPORT_PATH={}".format(import_path), file=env_file)

0 comments on commit 64ebf10

Please sign in to comment.