dbmigrator 0.7.0 → 0.7.1

data/VERSION

@@ -1 +1 @@
-0.7.0
+0.7.1

data/dbmigrator.gemspec

@@ -5,11 +5,11 @@
 
 Gem::Specification.new do |s|
   s.name = "dbmigrator"
-  s.version = "0.7.0"
+  s.version = "0.7.1"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Andrey Vasenin"]
-  s.date = "2013-03-20"
+  s.date = "2013-03-21"
   s.email = "vasenin@aboutecho.com"
   s.extra_rdoc_files = [
     "README.markdown"

data/lib/dbmigrator/tasks/databases.rake

@@ -4,6 +4,7 @@ require "task_manager"
 require "migration_generator"
 require "postgres"
 require "migration"
+require "tempfile"
 
 load "active_record/railties/databases.rake"
 
@@ -63,19 +64,25 @@ db_namespace = namespace :db do
 
   namespace :structure do
     override_task :dump => [:environment, :load_config] do
-      config = current_config
-      unless config['adapter'] =~ /postgres/
-        Rake::Task['db:structure:dump:original'].invoke
-        next
-      end
-      filename = ENV['DB_STRUCTURE'] || File.join(Rails.root, "db", "structure.sql")
-      set_psql_env(config)
-      `pg_dump -i -s -O -f #{Shellwords.escape(filename)} #{Shellwords.escape(config['database'])}`
-      raise 'Error dumping database' if $?.exitstatus == 1
-      if ActiveRecord::Base.connection.supports_migrations?
-        File.open(filename, "a") { |f| f << ActiveRecord::Base.connection.dump_schema_information }
+      tempfile = Tempfile.new('migrator-structure').tap(&:close)
+      begin
+        config = current_config
+        unless config['adapter'] =~ /postgres/
+          Rake::Task['db:structure:dump:original'].invoke
+          next
+        end
+        set_psql_env(config)
+        `pg_dump -i -s -O -f #{Shellwords.escape(tempfile.path)} #{Shellwords.escape(config['database'])}`
+        raise 'Error dumping database' if $?.exitstatus == 1
+        if ActiveRecord::Base.connection.supports_migrations?
+          File.open(tempfile.path, "a") { |f| f << ActiveRecord::Base.connection.dump_schema_information }
+        end
+        filename = ENV['DB_STRUCTURE'] || File.join(Rails.root, "db", "structure.sql")
+        FileUtils.cp tempfile.path, filename
+        db_namespace['structure:dump'].reenable
+      ensure
+         tempfile.unlink   # deletes the temp file
       end
-      db_namespace['structure:dump'].reenable
     end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: dbmigrator
 version: !ruby/object:Gem::Version
-  version: 0.7.0
+  version: 0.7.1
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-03-20 00:00:00.000000000 Z
+date: 2013-03-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
@@ -96,7 +96,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 1522855695555397277
+      hash: -2354698246710805387
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements: