Bug 1393076

Summary: [FTBFS] fabtests-1.3.0-1.el7.src.rpm under specific build requires
Product: Red Hat Enterprise Linux 7 Reporter: Pat Riehecky <riehecky>
Component: fabtestsAssignee: Honggang LI <honli>
Status: CLOSED NOTABUG QA Contact: yanfu,wang <yanwang>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 7.3CC: csieh, misterbonnie, rdma-dev-team, riehecky, toracat
Target Milestone: rc   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2016-11-09 15:26:54 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:
Attachments:
Description Flags
mock logfiles none

Description Pat Riehecky 2016-11-08 20:05:19 UTC 
Description of problem:
fabtests-1.3.0-1.el7.src.rpm does not list its dependency on libfabric-devel.

Version-Release number of selected component (if applicable):fabtests-1.3.0-1.el7.src.rpm


How reproducible:100%


Steps to Reproduce:
1.populate minimal build root
2.attempt to rebuild following RPM BuildRequires
3.

Actual results:
checking dependency style of gcc... (cached) none
checking for fi_getinfo in -lfabric... no
configure: error: fi_getinfo() not found.  fabtests requires libfabric.
RPM build errors:
error: Bad exit status from /var/tmp/rpm-tmp.bZa9iJ (%build)

Expected results:
BuildRequires: libfabric-devel

Additional info:
Comment 1 Honggang LI 2016-11-09 01:23:47 UTC 
Hi, Pat

 I checked the SRPM downloaded from our internal RPM building system. "fabtests.spec" has 'BuildRequires: libfabric-devel' in line 9.

brew]$ md5sum fabtests-1.3.0-1.el7.src.rpm
c08ff8a7e38eef58f01a31e133c374ae  fabtests-1.3.0-1.el7.src.rpm

  1 Name: fabtests
  2 Version: 1.3.0
  3 Release: 1%{?dist}
  4 Summary: Test suite for libfabric API
  5 Group: System Environment/Libraries
  6 License: GPLv2 or BSD
  7 Url: http://www.github.com/ofiwg/fabtests
  8 Source: https://github.com/ofiwg/%{name}/releases/download/v%{version}/%{name}-%{version}.tar.bz2
  9 BuildRequires: libfabric-devel
 10 
 11 %description
 12 Fabtests provides a set of examples that uses libfabric - a high-performance
 13 fabric software library.


1) please provide the md5sum of your fabtests-1.3.0-1.el7.src.rpm
2) please build fabtests with mock: 

mock -r epel-7-x86_64 --rebuild --resultdir epel7 fabtests-1.3.0-1.el7.src.rpm

You need replace "epel-7-x86_64" with corresponding platform configuration file available in /etc/mock, if you build fabtests for non x86_64 platform.
3) If mock failed, please feedback *ALL* mock log files.

Thanks
Comment 2 Pat Riehecky 2016-11-09 15:16:15 UTC 
Created attachment 1218976 [details]
mock logfiles

$ sha256sum fabtests.spec 
1997e15259366f2c334f999ce359550d11565166c56255c6e0019f3695fce199  fabtests.spec
$ sha1sum fabtests.spec 
d42487866b3d702b1788ddfeacc74fab1c2fec0f  fabtests.spec

SRPM build from git.centos.org
Comment 3 Honggang LI 2016-11-09 15:26:54 UTC 
(In reply to Pat Riehecky from comment #2)
> 
> SRPM build from git.centos.org

Sorry, you had opened a CENTOS bug against RHEL. We can't work in that way. I'm closing this bug as NOTABUG. Please file a CENTOS bug.
Comment 4 Akemi Yagi 2016-11-11 23:56:09 UTC 
As far as I can see, this is not a problem about the origin of the source file. Red Hat provides all source files for RHEL-7 *through* git.centos.org. So what is downloaded from there is the "official" code from Red Hat.

In fact, the spec file for fabtests does have this line:

BuildRequires: libfabric-devel

However, what is missing is a version requirement. It is supposed to be:

BuildRequires: libfabric-devel >= 1.3.0

In other words, if the version of libfabric-devel is older than the version of fabtests (for whatever reason), the build fails.
Comment 5 Honggang LI 2016-11-14 01:19:59 UTC 
(In reply to Akemi Yagi from comment #4)
> 
> However, what is missing is a version requirement. It is supposed to be:
> 
> BuildRequires: libfabric-devel >= 1.3.0
> 

You are right on this. I had fixed this issue for Fedora.

BuildRequires:  libfabric-devel >= %{version}

https://bugzilla.redhat.com/show_bug.cgi?id=1389568
Comment 6 Akemi Yagi 2016-11-14 16:40:06 UTC 
(In reply to Honggang LI from comment #5)

> You are right on this. I had fixed this issue for Fedora.
> 
> BuildRequires:  libfabric-devel >= %{version}
> 
> https://bugzilla.redhat.com/show_bug.cgi?id=1389568

Good to learn that you had fixed it for Fedora. Now if you could reopen this bug report and get the same fix for RHEL 7, that would be great.
Comment 7 Honggang LI 2016-11-15 00:27:23 UTC 
(In reply to Akemi Yagi from comment #6)
> Good to learn that you had fixed it for Fedora. Now if you could reopen this
> bug report and get the same fix for RHEL 7, that would be great.

It is unnecessary to reopen this bug, because we will update fabtests for RHEL-7.4 and fix this issue. RHEL-7.3 has been released, it is unlikely to open a z-stream bug for this minor issue.

thanks