Commitless repos should be bare

release/v1.15
Ethan Koenig 2017-05-28 22:25:23 -04:00
parent e67ece26f0
commit c85ff532e9
2 changed files with 5 additions and 1 deletions

View File

@ -517,7 +517,7 @@ func CommitRepoAction(opts CommitRepoActionOptions) error {
} }
// Change repository bare status and update last updated time. // Change repository bare status and update last updated time.
repo.IsBare = false repo.IsBare = repo.IsBare && opts.Commits.Len <= 0
if err = UpdateRepository(repo, false); err != nil { if err = UpdateRepository(repo, false); err != nil {
return fmt.Errorf("UpdateRepository: %v", err) return fmt.Errorf("UpdateRepository: %v", err)
} }

View File

@ -392,6 +392,10 @@ func RepoRef() macaron.Handler {
if err != nil { if err != nil {
ctx.Handle(500, "GetBranches", err) ctx.Handle(500, "GetBranches", err)
return return
} else if len(brs) == 0 {
err = fmt.Errorf("No branches in non-bare repository %s",
ctx.Repo.GitRepo.Path)
ctx.Handle(500, "GetBranches", err)
} }
refName = brs[0] refName = brs[0]
} }