According to man sudo, -i should be placed at the end as it can take a command as param

Author: root

Diff for: lib/redmine_gitolite/gitolite_wrapper.rb

@@ -197,7 +197,7 @@ def self.gitolite_banner
     # * (-n) non-interactive
     # * (-u `gitolite_user`) target user
     def self.sudo_shell_params
-      ['-i', '-n', '-u', gitolite_user]
+      ['-n', '-u', gitolite_user, '-i']
     end
 
 

Diff for: lib/redmine_gitolite/scripts.rb

@@ -198,7 +198,7 @@ def self.can_gitolite_sudo_to_redmine_user?
       end
 
       begin
-        test = GitoliteWrapper.sudo_capture('sudo', '-i', '-n', '-u', redmine_user, 'whoami')
+        test = GitoliteWrapper.sudo_capture('sudo', '-n', '-u', redmine_user, '-i', 'whoami')
         if test.match(/#{redmine_user}/)
           logger.info { "OK!" }
           @@sudo_gitolite_to_redmine_user_cached = true