Bug 1223385

Summary: packaging: .pc files included in -api-devel should be in -devel
Product: [Community] GlusterFS Reporter: Kaleb KEITHLEY <kkeithle>
Component: packagingAssignee: Kaleb KEITHLEY <kkeithle>
Status: CLOSED CURRENTRELEASE QA Contact:
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: mainlineCC: ndevos
Target Milestone: ---Keywords: Reopened, Triaged
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: glusterfs-3.8rc2 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
: 1223390 (view as bug list) Environment:
Last Closed: 2016-06-16 13:03:20 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:    
Bug Blocks: 1223390    

Description Kaleb KEITHLEY 2015-05-20 12:36:19 UTC
1.>> On 05/20/2015 03:42 PM, David Robinson wrote:
>>>
>>> When I try to upgrade from 3.6.3, it shows a number of missing
>>> dependencies.  I am running SL6.6.  Can you help point me in the right
>>> direction for these new dependencies?  pkgconfig is installed.  I don't
>>> see python-argparse or the others in the SL repo.
>>>
>>> glusterfs-3.7.0-1.el6.x86_64.rpm
>>> glusterfs-debuginfo-3.7.0-1.el6.x86_64.rpm
>>> glusterfs-libs-3.7.0-1.el6.x86_64.rpm
>>> glusterfs-api-3.7.0-1.el6.x86_64.rpm
>>> glusterfs-devel-3.7.0-1.el6.x86_64.rpm
>>> glusterfs-rdma-3.7.0-1.el6.x86_64.rpm
>>> glusterfs-api-devel-3.7.0-1.el6.x86_64.rpm
>>> glusterfs-extra-xlators-3.7.0-1.el6.x86_64.rpm
>>> glusterfs-server-3.7.0-1.el6.x86_64.rpm
>>> glusterfs-cli-3.7.0-1.el6.x86_64.rpm
>>> glusterfs-fuse-3.7.0-1.el6.x86_64.rpm
>>> python-gluster-3.7.0-1.el6.noarch.rpm
>>> glusterfs-client-xlators-3.7.0-1.el6.x86_64.rpm
>>> glusterfs-geo-replication-3.7.0-1.el6.x86_64.rpm
>>>
>>> [root@shelf4 gluster_3.7.0]# rpm -Uvh *.rpm
>>> error: Failed dependencies:
>>>         pkgconfig(sqlite3) is needed by
>>> glusterfs-api-devel-3.7.0-1.el6.x86_64
>
> -api-devel needs sqlite3?
>
> auto dependency decided that?

It seems that some of the .pc files are included in glusterfs-api-devel
that should have been in glusterfs-devel:

 968 %files api-devel
 969 %{_libdir}/pkgconfig/glusterfs-api.pc
 970 %{_libdir}/pkgconfig/libgfchangelog.pc
 971 %{_libdir}/pkgconfig/libgfdb.pc
 972 %{_libdir}/libgfapi.so
 973 %{_includedir}/glusterfs/api/*
 974 

The auto dependencies check for .pc files and their dependencies.
glusterfs-api-devel should only contain libgfapi things, nothing more.

Comment 1 Anand Avati 2015-05-20 13:12:37 UTC
REVIEW: http://review.gluster.org/10843 (glusterfs.spec.in: move libgf{db,changelog}.pc from -api-devel to -devel) posted (#1) for review on master by Kaleb KEITHLEY (kkeithle)

Comment 2 Anand Avati 2015-05-20 13:22:24 UTC
REVIEW: http://review.gluster.org/10844 (glusterfs.spec.in: move libgf{db,changelog}.pc from -api-devel to -devel) posted (#1) for review on release-3.7 by Kaleb KEITHLEY (kkeithle)

Comment 3 Anand Avati 2015-05-26 13:09:17 UTC
REVIEW: http://review.gluster.org/10843 (glusterfs.spec.in: move libgf{db,changelog}.pc from -api-devel to -devel) posted (#2) for review on master by Kaleb KEITHLEY (kkeithle)

Comment 4 Anand Avati 2015-05-26 18:16:27 UTC
REVIEW: http://review.gluster.org/10843 (glusterfs.spec.in: move libgf{db,changelog}.pc from -api-devel to -devel) posted (#3) for review on master by Kaleb KEITHLEY (kkeithle)

Comment 5 Anand Avati 2015-05-28 09:27:43 UTC
COMMIT: http://review.gluster.org/10843 committed in master by Niels de Vos (ndevos) 
------
commit 38a5cd8c136640d23288aca0afe19a82e1b320f8
Author: Kaleb S. KEITHLEY <kkeithle>
Date:   Wed May 20 09:10:38 2015 -0400

    glusterfs.spec.in: move libgf{db,changelog}.pc from -api-devel to -devel
    
    pkgconfig files in the wrong package results in unexpected package
    dependencies
    
    Change-Id: I0551114d09bcc3a7a73e152275175ba6c1bc0da3
    BUG: 1223385
    Signed-off-by: Kaleb S. KEITHLEY <kkeithle>
    Reviewed-on: http://review.gluster.org/10843
    Tested-by: NetBSD Build System
    Tested-by: Gluster Build System <jenkins.com>
    Reviewed-by: Niels de Vos <ndevos>

Comment 6 Nagaprasad Sathyanarayana 2015-10-25 15:09:14 UTC
Fix for this BZ is already present in a GlusterFS release. You can find clone of this BZ, fixed in a GlusterFS release and closed. Hence closing this mainline BZ as well.

Comment 7 Niels de Vos 2016-06-16 13:03:20 UTC
This bug is getting closed because a release has been made available that should address the reported issue. In case the problem is still not fixed with glusterfs-3.8.0, please open a new bug report.

glusterfs-3.8.0 has been announced on the Gluster mailinglists [1], packages for several distributions should become available in the near future. Keep an eye on the Gluster Users mailinglist [2] and the update infrastructure for your distribution.

[1] http://blog.gluster.org/2016/06/glusterfs-3-8-released/
[2] http://thread.gmane.org/gmane.comp.file-systems.gluster.user