Bug 1183210

Summary: vtk-devel is missing Requires: jsoncpp-devel
Product: [Fedora] Fedora Reporter: Scott K Logan <logans>
Component: vtkAssignee: François Cami <contribs>
Status: CLOSED ERRATA QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: unspecified    
Version: 21CC: Axel.Thimm, contribs, jonathan.underwood, lnie, mrceresa, orion
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: vtk-6.1.0-23.fc21 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2015-02-15 03:12:18 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On: 1183530    
Bug Blocks:    
Attachments:
Description Flags
Example CMakeLists.txt none

Description Scott K Logan 2015-01-17 07:45:03 UTC
Created attachment 980965 [details]
Example CMakeLists.txt

Description of problem:
The CMake include dirs for VTK list /usr/include/jsoncpp, which is not required to install vtk-devel as it should be.

Version-Release number of selected component (if applicable): 6.1.0-18.fc21

How reproducible: 100%

Steps to Reproduce:
1. Build the attached CMakeLists.txt
2. Observe presence of /usr/include/jsoncpp
3. Observe lack of jsoncpp-devel as Requires for vtk-devel

Actual results:
vtk-devel is missing Requires: jsoncpp-devel

Expected results:
vtk-devel has correct Requires for building against it

Additional info:
I may be wrong, but I believe python2-devel should also be listed, as /usr/include/python2.7 is also present in the include dirs. It may currently be an indirect requirement for vtk-devel...I'm not sure.

Thanks,

--scott

Comment 1 François Cami 2015-01-17 17:55:41 UTC
Building for rawhide:
http://koji.fedoraproject.org/koji/taskinfo?taskID=8652448

If there are no objections I'll merge into F21 as soon as the armv7hl build is done.

Comment 2 Scott K Logan 2015-01-17 20:51:50 UTC
Thanks much. I don't see this issue on f20, but a merge back to f21 would be appreciated!

Comment 3 François Cami 2015-01-18 12:22:27 UTC
Building: http://koji.fedoraproject.org/koji/taskinfo?taskID=8655968

Comment 4 Scott K Logan 2015-01-19 02:06:20 UTC
This didn't work right, don't submit an update yet.

The added dep for python2-devel was:

Requires: python2-devel%{?_isa}

but python2-devel is a virtual package, and so while python2-devel can be resolved, python2-devel(x86-64) cannot be:

Error: Package: vtk-devel-6.1.0-20.fc21.x86_64
            Requires: python2-devel(x86-64)

This may constitute a bug against python, I'm really not sure.

--scott

Comment 5 François Cami 2015-01-19 09:16:33 UTC
Thanks!

I'm thinking that python and python-devel are not noarch yet python.spec has:
Provides: python2-devel = %{version}-%{release}
so this is probably a bug in python.spec.

Comment 6 Scott K Logan 2015-01-20 22:37:55 UTC
Looks like the python fix is in rawhide, but it doesn't look like they're going to change it in f21.

Rawhide should therefore be good-to-go. Maybe Requires: python2-devel only for f21? It would be nice to get a fix for the jsoncpp issue on QA soon...

Comment 7 François Cami 2015-01-20 23:26:12 UTC
Agreed.

I built http://koji.fedoraproject.org/koji/buildinfo?buildID=605302 earlier today, could you test and report?

Thank you.

Comment 8 Scott K Logan 2015-01-21 00:03:18 UTC
That build checks out. Thanks much!

--scott

Comment 9 Fedora Update System 2015-01-22 00:07:06 UTC
vtk-6.1.0-21.fc21 has been submitted as an update for Fedora 21.
https://admin.fedoraproject.org/updates/vtk-6.1.0-21.fc21

Comment 10 Fedora Update System 2015-01-26 02:39:31 UTC
Package vtk-6.1.0-21.fc21:
* should fix your issue,
* was pushed to the Fedora 21 testing repository,
* should be available at your local mirror within two days.
Update it with:
# su -c 'yum update --enablerepo=updates-testing vtk-6.1.0-21.fc21'
as soon as you are able to.
Please go to the following url:
https://admin.fedoraproject.org/updates/FEDORA-2015-1156/vtk-6.1.0-21.fc21
then log in and leave karma (feedback).

Comment 11 Fedora Update System 2015-02-04 04:54:11 UTC
vtk-6.1.0-23.fc21 has been submitted as an update for Fedora 21.
https://admin.fedoraproject.org/updates/vtk-6.1.0-23.fc21

Comment 12 lnie 2015-02-05 07:10:00 UTC
vtk-6.1.0-23.fc21 works

Comment 13 Fedora Update System 2015-02-15 03:12:18 UTC
vtk-6.1.0-23.fc21 has been pushed to the Fedora 21 stable repository.  If problems still persist, please make note of it in this bug report.