dia 1.1.pre → 1.1

data/NEWS.md

@@ -1,7 +1,11 @@
 ## NEWS
 
-### 1.1.pre
+### 1.1 (final)
+* Dia::SandBox#run\_with\_block will exit the child process spawned by itself incase the user forgets to ..
 
+* Added some tests for Dia::SandBox.new#run\_with\_block ..  
+  We ain't got full coverage but we're getting there.
+  
 * A person reported that ffi 0.6.0 does not work with dia ..  
   Supplied an explicit dependency on ffi 0.5.4 until I figure it out.
     
@@ -10,3 +14,4 @@
   If you're running a block of ruby, you can forget about the second.
   
 * I documented my methods!
+

data/lib/dia/sandbox.rb

@@ -20,16 +20,17 @@ module Dia
     #  
     # @raise  [ArgumentError]         Will raise an ArgumentError if an application has not been supplied to 
     #                                 the constructer.
-    # @raise  [Dia::SandBoxException] Will raise Dia::SandBoxException if the sandbox could not be initiated.
+    # @raise  [Dia::SandBoxException] Will raise Dia::SandBoxException in a child process and exit if the sandbox could not be initiated.
     # @return [Fixnum] The Process ID(PID) that the sandboxed application is being run under.
     def run
       raise ArgumentError, "No application path supplied"  if @app_path.nil?
       
       @pid = fork do
-        unless ( ret = sandbox_init(@profile, 0x0001, error = FFI::MemoryPointer.new(:pointer)) ) == 0
+        if ( ret = sandbox_init(@profile, 0x0001, error = FFI::MemoryPointer.new(:pointer)) ) != 0
           raise Dia::SandBoxException, "Couldn't sandbox #{@app_path}, sandbox_init returned #{ret} with error message: '#{error.get_pointer(0).read_string}'"
+        else
+          exec(@app_path)
         end
-        exec(@app_path)
       end
     end
   
@@ -40,17 +41,19 @@ module Dia
     # the sandbox .. In any case, the parent process will not be affected and if you want to catch an exception you
     # should do so in your block.
     #
-    # @raise  [Dia::SandBoxException] Will raise Dia::SandBoxException if the sandbox could not be initiated. 
+    # @raise  [Dia::SandBoxException] Will raise Dia::SandBoxException in a child process and exit if the sandbox could not be initiated. 
     # @return [Fixnum] The Process ID(PID) that the sandboxed block of code is being run under.
     def run_with_block &blk
       @pid = fork do
-         unless ( ret = sandbox_init(@profile, 0x0001, error = FFI::MemoryPointer.new(:pointer)) ) == 0
+         if ( ret = sandbox_init(@profile, 0x0001, error = FFI::MemoryPointer.new(:pointer)) ) != 0
            raise Dia::SandBoxException, "Couldn't sandbox #{@app_path}, sandbox_init returned #{ret} with error message: '#{error.get_pointer(0).read_string}'"
+         else
+           yield
          end
-         yield
+         exit
       end
     end
     
   end
   
-end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: dia
 version: !ruby/object:Gem::Version 
-  version: 1.1.pre
+  version: "1.1"
 platform: ruby
 authors: 
 - Robert Gleeson
@@ -9,7 +9,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2010-02-12 00:00:00 +00:00
+date: 2010-02-16 00:00:00 +00:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
@@ -22,7 +22,17 @@ dependencies:
       - !ruby/object:Gem::Version 
         version: 0.5.4
     version: 
-description: Dia allows you to sandbox applications on the OSX platform
+- !ruby/object:Gem::Dependency 
+  name: baretest
+  type: :development
+  version_requirement: 
+  version_requirements: !ruby/object:Gem::Requirement 
+    requirements: 
+    - - ">="
+      - !ruby/object:Gem::Version 
+        version: 0.2.4
+    version: 
+description: Dia allows you to sandbox applications and/or a block of ruby on the OSX platform
 email: rob@flowof.info
 executables: []
 
@@ -41,19 +51,25 @@ has_rdoc: true
 homepage: 
 licenses: []
 
-post_install_message: " Dia\n\
-  -----\n\
-  Thanks for taking the time to try out the prereleae of Dia 1.1\n\n\
-  For people who had problems with Dia and FFI 0.6.0, I have added an explicit \n\
-  dependency on 0.5.4 ..\n\n\
-  Slight API changes alter the way Dia behaves .. \n\
-  The change is minor, but worth noting because it breaks code written for 1.0\n\n\
-  For more information: \n\
-  http://github.com/robgleeson/Dia/blob/experimental/NEWS.md\n\
-  http://github.com/robgleeson/Dia/blob/experimental/README.md\n\n\
-  Please report bugs if you find any! \n\
-  http://github.com/robgleeson/dia/issues\n\n\
-  Rob\n"
+post_install_message: |+
+  
+  ----------------------------------------------------------
+  Thanks for taking the time to try out Dia 1.1 (Final)
+  
+  API docs:
+  http://yardoc.org/docs/robgleeson-Dia
+  
+  NEWS & README:
+  http://github.com/robgleeson/Dia/blob/master/NEWS.md
+  http://github.com/robgleeson/Dia/blob/master/README.md
+  
+  IRC:
+  irc.freenode.net / #flowof.info
+  
+  Bug tracker: 
+  http://github.com/robgleeson/dia/issues
+  ----------------------------------------------------------
+  
 rdoc_options: []
 
 require_paths: 
@@ -66,9 +82,9 @@ required_ruby_version: !ruby/object:Gem::Requirement
   version: 
 required_rubygems_version: !ruby/object:Gem::Requirement 
   requirements: 
-  - - ">"
+  - - ">="
     - !ruby/object:Gem::Version 
-      version: 1.3.1
+      version: "0"
   version: 
 requirements: []
 
@@ -76,6 +92,6 @@ rubyforge_project:
 rubygems_version: 1.3.5
 signing_key: 
 specification_version: 3
-summary: Dia allows you to sandbox applications on the OSX platform
+summary: Dia allows you to sandbox applications and/or a block of ruby on the OSX platform
 test_files: []