diff options
author | Jacob Vosmaer <contact@jacobvosmaer.nl> | 2014-05-09 00:49:27 +0400 |
---|---|---|
committer | Jacob Vosmaer <contact@jacobvosmaer.nl> | 2014-05-09 00:50:48 +0400 |
commit | 0f29ccffd2b9805545db63d8266bb8367de1e35c (patch) | |
tree | ef27d25fef52d687c1e299d4ba8508a2a4fd26a3 | |
parent | 23ae3911aa55f075e30da9da4e687e064e59a9fa (diff) |
Backup wiki repo even if the main repo is empty
This fixes a bug where wiki repositories for projects with an empty main
repository would not get backed up.
-rw-r--r-- | CHANGELOG | 1 | ||||
-rw-r--r-- | lib/backup/repository.rb | 9 |
2 files changed, 4 insertions, 6 deletions
diff --git a/CHANGELOG b/CHANGELOG index 61eb24f1c53..e43edc5d7ad 100644 --- a/CHANGELOG +++ b/CHANGELOG @@ -11,6 +11,7 @@ v 6.9.0 - Improve issue and merge request mobile UI (Drew Blessing) - Document how to convert a backup to PostgreSQL - Fix locale bug in backup manager + - Fix wiki backup skip bug v 6.8.0 - Ability to at mention users that are participating in issue and merge req. discussion diff --git a/lib/backup/repository.rb b/lib/backup/repository.rb index 214d9824ee1..6f7c4f7c909 100644 --- a/lib/backup/repository.rb +++ b/lib/backup/repository.rb @@ -10,15 +10,12 @@ module Backup Project.find_each(batch_size: 1000) do |project| print " * #{project.path_with_namespace} ... " - if project.empty_repo? - puts "[SKIPPED]".cyan - next - end - # Create namespace dir if missing FileUtils.mkdir_p(File.join(backup_repos_path, project.namespace.path)) if project.namespace - if system(*%W(git --git-dir=#{path_to_repo(project)} bundle create #{path_to_bundle(project)} --all), silent) + if project.empty_repo? + puts "[SKIPPED]".cyan + elsif system(*%W(git --git-dir=#{path_to_repo(project)} bundle create #{path_to_bundle(project)} --all), silent) puts "[DONE]".green else puts "[FAILED]".red |