[Bug 1317592] Review Request: rubygem-websocket-extensions - Generic extension manager for WebSocket connections

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



https://bugzilla.redhat.com/show_bug.cgi?id=1317592



--- Comment #3 from Jun Aruga <jaruga@xxxxxxxxxx> ---
Thanks, Vit for the review.
I fixed the issued that you pointed out. [1]

Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=13353927
I ran Koji again after that.

Do we need to update below page, because I referred the page?
https://fedoraproject.org/wiki/Packaging:Ruby#Test_suites_not_included_in_the_package

And thanks for your sponsor!! and will check next steps[2].


[1]
$ git diff 97f720687b588892ceb9e3d810711589c9468b58
rubygem-websocket-extensions.spec 
diff --git a/rubygem-websocket-extensions.spec
b/rubygem-websocket-extensions.spec
index fabfade..f9c419c 100644
--- a/rubygem-websocket-extensions.spec
+++ b/rubygem-websocket-extensions.spec
@@ -40,8 +40,6 @@ gem unpack %{SOURCE0}

 gem spec %{SOURCE0} -l --ruby > %{gem_name}.gemspec

-tar -xzf %{SOURCE1}
-
 %build
 # Create the gem as gem install only works on a gem file
 gem build %{gem_name}.gemspec
@@ -60,10 +58,10 @@ cp -a .%{gem_dir}/* \

 # Run the test suite
 %check
+tar -xzf %{SOURCE1}
 cp -pr spec/ ./%{gem_instdir}
 pushd .%{gem_instdir}
 rspec spec
-rm -rf spec
 popd

 %files

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review




[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]