Bug 1308099

Summary: rubygem-xpath: FTBFS in rawhide
Product: [Fedora] Fedora Reporter: Fedora Release Engineering <releng>
Component: rubygem-xpathAssignee: Jun Aruga <jaruga>
Status: CLOSED ERRATA QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 24CC: clalancette, jaruga, mmorsi, strzibny, vondruch
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: rubygem-xpath-2.0.0-4.fc25 rubygem-xpath-2.0.0-4.fc24 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2016-06-18 18:41:54 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Bug Depends On:    
Bug Blocks: 1305208    
Attachments:
Description Flags
build.log
none
root.log
none
state.log
none
Fix test suite for RSpec 3 compatibility
none
Fix test suite for RSpec 3 compatibility
none
Fix test suite for RSpec 3 compatibility none

Description Fedora Release Engineering 2016-02-13 22:26:06 UTC
Your package rubygem-xpath failed to build from source in current rawhide.

http://koji.fedoraproject.org/koji/taskinfo?taskID=12867858

For details on mass rebuild see https://fedoraproject.org/wiki/Fedora_24_Mass_Rebuild

Comment 1 Fedora Release Engineering 2016-02-13 22:26:09 UTC
Created attachment 1126358 [details]
build.log

Comment 2 Fedora Release Engineering 2016-02-13 22:26:11 UTC
Created attachment 1126359 [details]
root.log

Comment 3 Fedora Release Engineering 2016-02-13 22:26:12 UTC
Created attachment 1126360 [details]
state.log

Comment 4 Jan Kurik 2016-02-24 15:12:00 UTC
This bug appears to have been reported against 'rawhide' during the Fedora 24 development cycle.
Changing version to '24'.

More information and reason for this action is here:
https://fedoraproject.org/wiki/Fedora_Program_Management/HouseKeeping/Fedora24#Rawhide_Rebase

Comment 5 Jun Aruga 2016-05-25 06:40:32 UTC
I will fix this!

Comment 6 Jun Aruga 2016-05-25 07:00:32 UTC
Created attachment 1161320 [details]
Fix test suite for RSpec 3 compatibility

I fixed the test suite for RSpec 3.

# Highlight

- I did not update the rubygem-xpath version because it was latest version.
- Removed below line for rpmlint.
  Provides: rubygem(%{gem_name}) = %{version}

# Others
- Dependency: ok
- Install & Irb check: ok

# Detail for the dependency

$ dnf repoquery --enablerepo=rawhide --whatrequires 'rubygem(xpath)' 2> /dev/null
rubygem-capybara-0:2.4.1-2.fc23.noarch
rubygem-capybara-0:2.4.1-3.fc24.noarch
  => gem spec ok
  => rpm spec ok

$ dnf repoquery --enablerepo=rawhide --whatrequires 'rubygem-xpath' 2> /dev/null
rubygem-xpath-doc-0:2.0.0-1.fc22.noarch

$ dnf repoquery --enablerepo=rawhide-source --arch src --whatrequires 'rubygem(xpath)'
  => no result

$ dnf repoquery --enablerepo=rawhide-source --arch src --whatrequires 'rubygem-xpath'
  => no result

Could you check this?
Thanks.

Comment 7 Vít Ondruch 2016-05-25 13:16:47 UTC
I'd say that this would be more concise version of your patch:

```
diff --git a/spec/html_spec.rb b/spec/html_spec.rb
index 142cac1..a6e4eeb 100644
--- a/spec/html_spec.rb
+++ b/spec/html_spec.rb
@@ -5,13 +5,13 @@ describe XPath::HTML do
   let(:template) { 'form' }
   let(:template_path) { File.read(File.expand_path("fixtures/#{template}.html", File.dirname(__FILE__))) }
   let(:doc) { Nokogiri::HTML(template_path) }
+  let(:type) {|example| example.metadata[:type]}
 
   def get(*args)
     all(*args).first
   end
 
   def all(*args)
-    type = example.metadata[:type]
     doc.xpath(XPath::HTML.send(subject, *args).to_xpath(type)).map { |node| node[:data] }
   end
 
```

Nevertheless your upstream PR does not pass the test suite, so not sure about it ...

Comment 8 Jun Aruga 2016-05-26 13:11:04 UTC
Created attachment 1161997 [details]
Fix test suite for RSpec 3 compatibility

> I'd say that this would be more concise version of your patc

Thanks for your review. I updated my patch, and uploaded it again.
Could you check this?
Thanks.

Comment 9 Vít Ondruch 2016-05-26 14:16:58 UTC
LGTM. But since I am not maintainer, lets wait for a few days if you get approved, otherwise I'll build it myself in the near future.

Comment 10 Jun Aruga 2016-05-26 14:36:40 UTC
Created attachment 1162027 [details]
Fix test suite for RSpec 3 compatibility

I modified the patch a little.
I changed the position of Patch0, under the Source0.
Thanks.

Comment 11 Jun Aruga 2016-06-03 08:52:37 UTC
Hi, Vit.

I wait for 1 week from last time.
After that, I have not been able to see the maintainer's response, and have not been able to get approval.
Could you build it yourself?

Thanks.

Comment 12 Chris Lalancette 2016-06-06 00:02:57 UTC
Ug, am I really the maintainer?  I shouldn't be.  Vit, I've added you to the package administrators so you can push and build as you need.  Should I just orphan it so you can pick it up?

Comment 13 Vít Ondruch 2016-06-06 10:14:56 UTC
(In reply to Chris Lalancette from comment #12)
> Ug, am I really the maintainer?  I shouldn't be.  Vit, I've added you to the
> package administrators so you can push and build as you need.

Thx for that. I could build it anyway, since I am proven packager ;)

> Should I just orphan it so you can pick it up?

Yes, sure. Thx. I guess you should be able to use use "pkgdb-cli give" command to transfer the rights to me without the orphaning step

Comment 14 Upstream Release Monitoring 2016-06-06 11:02:33 UTC
jaruga's rubygem-xpath-2.0.0-4.fc25 completed http://koji.fedoraproject.org/koji/buildinfo?buildID=770743

Comment 15 Fedora Update System 2016-06-06 11:24:49 UTC
rubygem-xpath-2.0.0-4.fc24 has been submitted as an update to Fedora 24. https://bodhi.fedoraproject.org/updates/FEDORA-2016-13342931ae

Comment 16 Chris Lalancette 2016-06-06 14:34:17 UTC
All right, I think I've made you the maintainer now.  If not, let me know and I'll look again.  Thanks for taking it over!

Comment 17 Fedora Update System 2016-06-06 17:57:21 UTC
rubygem-xpath-2.0.0-4.fc24 has been pushed to the Fedora 24 testing repository. If problems still persist, please make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2016-13342931ae

Comment 18 Vít Ondruch 2016-06-07 06:52:41 UTC
(In reply to Chris Lalancette from comment #16)
> All right, I think I've made you the maintainer now.

Yep! Thx.

Comment 19 Fedora Update System 2016-06-18 18:41:48 UTC
rubygem-xpath-2.0.0-4.fc24 has been pushed to the Fedora 24 stable repository. If problems still persist, please make note of it in this bug report.