Hey Seth, While you are here are you interested with something like this? It changes files to use 4-space indents and no hard tab characters. Also trim excess spaces and tabs from ends of lines, and remove empty lines at the end of files and ensure the last line ends with a newline. [caglar@zanpaktou:~/Downloads/fedoraproject/func@master] git diff | diffstat contrib/find_resources.py | 8 contrib/packagekit.py | 6 examples/drive_checker.py | 3 examples/find_recalled_parts.py | 4 examples/service_checker.py | 2 func/CommonErrors.py | 1 func/commonconfig.py | 3 func/forkbomb.py | 16 func/index_db.py | 13 func/jobthing.py | 48 func/logger.py | 15 func/minion/acls.py | 28 func/minion/codes.py | 2 func/minion/facts/minion_query.py | 94 func/minion/facts/modules/fact_module.py | 11 func/minion/facts/modules/hardware.py | 17 func/minion/facts/overlord_query.py | 19 func/minion/facts/query.py | 48 func/minion/facts/query_utils.py | 2 func/minion/facts/tree.py | 2 func/minion/func_arg.py | 42 func/minion/modules/bridge.py | 11 func/minion/modules/certmastermod.py | 8 func/minion/modules/command.py | 2 func/minion/modules/confmgt_augeas.py | 23 func/minion/modules/copyfile.py | 2 func/minion/modules/delegation.py | 22 func/minion/modules/echo.py | 20 func/minion/modules/fact.py | 21 func/minion/modules/filetracker.py | 24 func/minion/modules/func_getargs.py | 15 func/minion/modules/func_module.py | 23 func/minion/modules/hardware.py | 11 func/minion/modules/httpd.py | 2 func/minion/modules/iptables/common.py | 2 func/minion/modules/iptables/port.py | 18 func/minion/modules/jboss.py | 44 func/minion/modules/jobs.py | 23 func/minion/modules/mount.py | 14 func/minion/modules/nagios-check.py | 2 func/minion/modules/netapp/common.py | 3 func/minion/modules/netapp/options.py | 3 func/minion/modules/netapp/snap.py | 5 func/minion/modules/netapp/vol/__init__.py | 10 func/minion/modules/netapp/vol/clone.py | 10 func/minion/modules/networktest.py | 12 func/minion/modules/overlord.py | 2 func/minion/modules/process.py | 30 func/minion/modules/pullfile.py | 1 func/minion/modules/reboot.py | 1 func/minion/modules/rpms.py | 10 func/minion/modules/service.py | 12 func/minion/modules/smart.py | 2 func/minion/modules/snmp.py | 4 func/minion/modules/sysctl.py | 8 func/minion/modules/test.py | 2 func/minion/modules/users.py | 3292 ++++++++++++------------ func/minion/modules/virt.py | 2 func/minion/modules/vlan.py | 8 func/minion/modules/yumcmd.py | 26 func/minion/server.py | 40 func/overlord/base_command.py | 4 func/overlord/client.py | 210 - func/overlord/cmd_modules/call.py | 88 func/overlord/cmd_modules/check.py | 86 func/overlord/cmd_modules/copyfile.py | 2 func/overlord/cmd_modules/grep.py | 46 func/overlord/cmd_modules/group.py | 38 func/overlord/cmd_modules/listminions.py | 3 func/overlord/cmd_modules/show.py | 2 func/overlord/cmd_modules/show_hardware.py | 14 func/overlord/command.py | 3 func/overlord/delegation_tools.py | 60 func/overlord/func_command.py | 9 func/overlord/group/base.py | 6 func/overlord/group/conf_backend.py | 35 func/overlord/group/sqlite_backend.py | 43 func/overlord/groups.py | 58 func/overlord/inventory.py | 24 func/overlord/mapper.py | 35 func/overlord/modules/copyfile.py | 4 func/overlord/modules/netapp.py | 4 func/overlord/modules/utils.py | 4 func/overlord/scripts.py | 5 func/utils.py | 53 func/yaml/dump.py | 42 func/yaml/load.py | 2 func/yaml/ordered_dict.py | 4 func/yaml/redump.py | 1 func/yaml/stream.py | 8 func/yaml/timestamp.py | 52 func/yaml/ypath.py | 6 funcweb/funcweb/async_tools.py | 25 funcweb/funcweb/commands.py | 8 funcweb/funcweb/controllers.py | 102 funcweb/funcweb/identity/pam.py | 2 funcweb/funcweb/result_handler.py | 36 funcweb/funcweb/tests/bork.py | 2 funcweb/funcweb/tests/test_async_tools.py | 56 funcweb/funcweb/tests/test_client_rendering.py | 5 funcweb/funcweb/tests/test_model.py | 1 funcweb/funcweb/tests/test_result_handler.py | 9 funcweb/funcweb/tests/test_widget_automation.py | 57 funcweb/funcweb/tests/test_widget_validation.py | 79 funcweb/funcweb/widget_automation.py | 52 funcweb/funcweb/widget_validation.py | 74 pusher.py | 41 setup.py | 131 test/async_test.py | 95 test/test-confmgt_augeas.py | 106 test/test_func_db.py | 36 test/unittest/cover_to_html.py | 68 test/unittest/plugins/funccover.py | 17 test/unittest/test_client.py | 56 test/unittest/test_config.py | 38 test/unittest/test_facts_minion.py | 94 test/unittest/test_facts_overlord.py | 32 test/unittest/test_facts_tree.py | 28 test/unittest/test_func_arg.py | 27 test/unittest/test_func_transmit.py | 36 test/unittest/test_groups_api.py | 112 test/unittest/test_minion.py | 40 test/unittest/test_users.py | 722 ++--- 123 files changed, 3600 insertions(+), 3692 deletions(-) On Oct 14, 2010, at 12:09 PM, Seth Vidal wrote: > func/minion/modules/bridge.py | 4 > func/minion/modules/jboss.py | 30 +++--- > func/minion/modules/service.py | 18 +-- > func/minion/modules/test.py | 14 +-- > func/minion/modules/virt.py | 185 ++++++++++++++++++++--------------------- > 5 files changed, 126 insertions(+), 125 deletions(-) > > New commits: > commit 2441779de383f87a441ae09abd1b8871a6479bba > Author: Seth Vidal <skvidal@xxxxxxxxxxxxxxxxx> > Date: Thu Oct 14 12:08:06 2010 -0400 > > - fix tab/space issues all over the place (do people still have editors that use tabs?) > - pylint the code and look for indentation errors and/or other odd language-isms. > -- S.ÃaÄlar Onur <caglar@xxxxxxxxxxxxxxxx> _______________________________________________ Func-list mailing list Func-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/func-list