rubyforge 0.4.0 → 0.4.1

data/History.txt

@@ -1,3 +1,10 @@
+Version History:
+
+* 0.4.1 / 2007-03-08:
+  * Verify that login succeeded and warn against if not (prolly should raise).
+  * Print a friendly error if you have the wrong package id.
+  * Handle upload error in add_release a bit better.
+
 * 0.4.0 / 2007-01-09:
   * config.yml split and moved to user-config.yml (up to the user to do).
   * auto-config.yml now generated via config command.

data/lib/rubyforge.rb

@@ -26,7 +26,7 @@ end
 class RubyForge
 
   # :stopdoc:
-  VERSION     = "0.4.0"
+  VERSION     = "0.4.1"
   HOME        = ENV["HOME"] || ENV["HOMEPATH"] || File::expand_path("~")
   RUBYFORGE_D = File::join HOME, ".rubyforge"
   CONFIG_F    = File::join RUBYFORGE_D, "user-config.yml"
@@ -141,7 +141,15 @@ class RubyForge
       "login"          => "Login"
     }
 
-    run page, form
+    response = run page, form
+
+    re = %r/personal\s+page\s+for:\s+#{ Regexp.escape username }/iom
+    unless response =~ re
+      warn("%s:%d: warning: potentially failed login using %s:%s" %
+        [__FILE__,__LINE__,username,password]) unless $TESTING
+    end
+
+    response
   end
 
   def create_package(group_id, package_name)
@@ -251,8 +259,17 @@ class RubyForge
     boundary = Array::new(8){ "%2.2d" % rand(42) }.join('__')
     boundary = "multipart/form-data; boundary=___#{ boundary }___"
 
+
     html = run(page, form, 'content-type' => boundary)
-    release_id = html[/release_id=\d+/][/\d+/].to_i
+    raise "Invalid package_id #{package_id}" if html[/Invalid package_id/]
+
+    release_id = html[/release_id=\d+/][/\d+/].to_i rescue nil
+
+    unless release_id then
+      puts html if $DEBUG
+      raise "Couldn't get release_id, upload failed\?"
+    end
+
     puts "RELEASE ID = #{release_id}" if $DEBUG
 
     files.each do |file|

metadata

@@ -3,8 +3,8 @@ rubygems_version: 0.9.0.9
 specification_version: 1
 name: rubyforge
 version: !ruby/object:Gem::Version 
-  version: 0.4.0
-date: 2007-01-10 00:00:00 -05:00
+  version: 0.4.1
+date: 2007-03-08 00:00:00 -08:00
 summary: A simplistic script which automates a limited set of rubyforge operations
 require_paths: 
 - lib