Skip to content

Commit

Permalink
Ensure --version works with non-ascii project path
Browse files Browse the repository at this point in the history
Fixes #66617
  • Loading branch information
mkrizek committed Mar 26, 2020
1 parent c61f141 commit 5811796
Show file tree
Hide file tree
Showing 4 changed files with 11 additions and 1 deletion.
2 changes: 2 additions & 0 deletions changelogs/fragments/66617-version-unicode-fix.yml
@@ -0,0 +1,2 @@
bugfixes:
- Ensure that ``--version`` works with non-ascii ansible project paths (https://github.com/ansible/ansible/issues/66617)
2 changes: 1 addition & 1 deletion lib/ansible/config/manager.py
Expand Up @@ -231,7 +231,7 @@ def find_ini_config_file(warnings=None):
if os.path.exists(cwd_cfg):
warn_cmd_public = True
else:
potential_paths.append(cwd_cfg)
potential_paths.append(to_text(cwd_cfg, errors='surrogate_or_strict'))
except OSError:
# If we can't access cwd, we'll simply skip it as a possible config source
pass
Expand Down
@@ -0,0 +1,2 @@
[defaults]
library=~/.ansible/plugins/modules:/usr/share/ansible/plugins/modules:.
6 changes: 6 additions & 0 deletions test/integration/targets/unicode/runme.sh
Expand Up @@ -5,3 +5,9 @@ set -eux
ansible-playbook unicode.yml -i inventory -v -e 'extra_var=café' "$@"
# Test the start-at-task flag #9571
ANSIBLE_HOST_PATTERN_MISMATCH=warning ansible-playbook unicode.yml -i inventory -v --start-at-task '' -e 'start_at_task=True' "$@"

# Test --version works with non-ascii ansible project paths #66617
# Unset these so values from the project dir are used
unset ANSIBLE_CONFIG
unset ANSIBLE_LIBRARY
pushd křížek-ansible-project && ansible --version; popd

0 comments on commit 5811796

Please sign in to comment.