Skip to content

Commit

Permalink
Fix lint
Browse files Browse the repository at this point in the history
  • Loading branch information
@basepi
basepi committed Sep 17, 2015
1 parent 26b6c92 commit 7f391a4
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion salt/modules/localemod.py
View file
Original file line number Diff line number Diff line change
Expand Up @@ -127,7 +127,7 @@ def get_locale():
cmd = 'grep "^LANG=" /etc/sysconfig/i18n'
elif 'Debian' in __grains__['os_family']:
if salt.utils.which('localectl'):
return _localectl_get()
return _locale_get()

cmd = 'grep "^LANG=" /etc/default/locale'
elif 'Gentoo' in __grains__['os_family']:
Expand Down

3 comments on commit 7f391a4

@joejulian
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This whole family based decision tree sucks. RHEL 7 is systemd, Debian Jessie is systemd.

@basepi
Copy link
Contributor Author

@basepi basepi commented on 7f391a4 Mar 1, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Agreed, it could use a rethink with recent changes in distros. I recommend filing an issue, nobody but me will see this comment. :)

@joejulian
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oops, meant to put that on PR #31552 :)

Please sign in to comment.