Skip to content

Commit

Permalink
Experiment to understand root issue
Browse files Browse the repository at this point in the history
  • Loading branch information
Julian-O committed Sep 11, 2023
1 parent 5ff63f2 commit 98ac244
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 6 deletions.
2 changes: 0 additions & 2 deletions .github/workflows/ios.yml
Original file line number Diff line number Diff line change
Expand Up @@ -33,8 +33,6 @@ jobs:
- name: Initialize buildozer in project folder
run: |
source .ci/osx_ci.sh
# Turn off warning that tests run as root.
echo "BUILDOZER_WARN_ON_ROOT=0" >> "$GITHUB_ENV"
arm64_set_path_and_python_version ${{ matrix.python }}
buildozer init
- name: Install dependencies
Expand Down
2 changes: 1 addition & 1 deletion buildozer/commandline.py
Original file line number Diff line number Diff line change
Expand Up @@ -243,7 +243,7 @@ def _check_if_root_user(config):
print(
"\033[91mThis is \033[1mnot\033[0m \033[91mrecommended, and may lead to problems later.\033[0m"
)

print(os.geteuid())
if not _get_user_permission():
sys.exit()

Expand Down
4 changes: 1 addition & 3 deletions tests/test_commandline.py
Original file line number Diff line number Diff line change
Expand Up @@ -44,9 +44,7 @@ class CommandLineLocalTestCase(unittest.TestCase):
# For those, this tests for the right behaviour.

def setUp(self):
# We run as root, against recommendations, during testing.
# Turn off check.
os.environ["BUILDOZER_WARN_ON_ROOT"] = "0"
print("Running test as ", os.geteuid())

def test_version(self):
output = cli_output("--version", 0)
Expand Down

0 comments on commit 98ac244

Please sign in to comment.