Skip to content

Commit c7a79f9

Browse files
authored
Merge pull request #40 from radomirbosak/allow-home-install
Do not install bash completion as non-root
2 parents 4db000d + 69c87c8 commit c7a79f9

File tree

1 file changed

+6
-3
lines changed

1 file changed

+6
-3
lines changed

setup.py

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -25,9 +25,12 @@
2525
completion_dirs = ['/usr/share/bash-completion/completions',
2626
'/usr/local/opt/bash-completion/etc/bash_completion.d']
2727

28-
for d in completion_dirs:
29-
if os.path.isdir(d):
30-
data_files.append((d, ['contrib/bash-completion/jenkins']))
28+
if os.geteuid() == 0:
29+
for d in completion_dirs:
30+
if os.path.isdir(d):
31+
data_files.append((d, ['contrib/bash-completion/jenkins']))
32+
else:
33+
print("Non-root user detected. Bash completion won't be installed.")
3134

3235
setup(
3336
name='jenkins-cli',

0 commit comments

Comments
 (0)