Merge #1 - Make rvmn exit with a meaningful exit status

jruby · Oct 7, 2020 · 3d2ce35 · 3d2ce35
2 parents 4b51401 + a932609
commit 3d2ce35
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/bin/rmvn b/bin/rmvn
@@ -5,3 +5,5 @@ args = ARGV.dup
 ARGV.clear # clean up in case another script gets executed it gets clear ARGV
 
 RubyMaven.exec(*args)
+
+exit $?.exitstatus
diff --git a/spec/ruby_maven_spec.rb b/spec/ruby_maven_spec.rb
@@ -16,21 +16,21 @@
     end
   end
 
-  let :gem do
+  let :gem_name do
     v = Maven::Ruby::VERSION
     v += '-SNAPSHOT' if v =~ /[a-zA-Z]/
     "pkg/ruby-maven-#{v}.gem"
   end
 
   it 'pack the gem' do
-    FileUtils.rm_f gem
+    FileUtils.rm_f gem_name
     CatchStdout.exec do
       # need newer jruby version
       RubyMaven.exec( '-Dverbose', 'package', '-Djruby.version=1.7.24' )
     end
     #puts CatchStdout.result
     CatchStdout.result.must_match /mvn -Dverbose package/
-    File.exists?( gem ).must_equal true
+    File.exists?( gem_name ).must_equal true
     File.exists?( '.mvn/extensions.xml' ).must_equal true
     File.exists?( '.mvn/extensions.xml.orig' ).wont_equal true
   end
Original file line number	Diff line number	Diff line change
Expand Up		@@ -5,3 +5,5 @@ args = ARGV.dup
		ARGV.clear # clean up in case another script gets executed it gets clear ARGV

		RubyMaven.exec(*args)

		exit $?.exitstatus