git_init_db1(): remove default behaviour, require a choice

Signed-off-by: Tobias Ulmer <tu@emlix.com>

diff --git a/generic/generic_git.lua b/generic/generic_git.lua
index 28a0ebdc7d910d47d12c3f610d94c3ed998ca88d..2ecc2b6eb2493e90fc2b3ce06557b9c2290c2c52 100644 (file)
--- a/generic/generic_git.lua
+++ b/generic/generic_git.lua
@@ -344,18 +344,12 @@ end
 --- Initialize a git repository.
 -- @param rurl URL string where the repository should be created.
 -- @param shared Should the repository be shared with other users or not.
---               Defaults to true.
 -- @return True on success, false on error.
 -- @return Error object on failure.
 function generic_git.git_init_db1(rurl, shared)
     local rc, re, e, u, gitdir, gitargv, argv
 
-    -- XXX: Remove the default behaviour and require a choice
-    if shared == nil then
-        shared = true
-    end
-
-    if not rurl then
+    if not rurl or type(shared) ~= "boolean" then
         return false, err.new("git_init_db1(): missing parameter")
     end
 
@@ -560,7 +554,7 @@ function generic_git.git_init_db(c, server, location)
     if not rurl then
         return false, e:cat(re)
     end
-    local rc, re = generic_git.git_init_db1(rurl)
+    local rc, re = generic_git.git_init_db1(rurl, true)
     if not rc then
         return false, re
     end