Bug 764732 (GLUSTER-3000)

Summary: Add -devel package for headers
Product: [Community] GlusterFS Reporter: Jeff Darcy <jdarcy>
Component: buildAssignee: Amar Tumballi <amarts>
Status: CLOSED CURRENTRELEASE QA Contact: Sachidananda Urs <sac>
Severity: low Docs Contact:
Priority: medium    
Version: mainlineCC: gluster-bugs, vraman
Target Milestone: ---   
Target Release: ---   
Hardware: x86_64   
OS: Linux   
Whiteboard:
Fixed In Version: glusterfs-3.4.0 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2013-07-24 17:43:31 UTC Type: ---
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: 817967    

Description Jeff Darcy 2011-06-07 21:20:55 UTC
OK, I admit it, I did this partly to get bug 764732.  Seriously, though, since extensibility is one of the most significant benefits of GlusterFS relative to most alternatives, I think it's imperative to facilitate actual extension without needing to have a full GlusterFS source tree available (which violates many projects' packaging guidelines).  For CloudFS I've managed to build translators "out of tree" using simple makefiles and only headers installed as part of the Fedora glusterfs-devel package.  Ideally this same work would also apply to Gluster's own packaging, to Debian, etc.  Also, the particular files that are part of the "external API" represented by the -devel package - especially those necessary to build any translator - should be carefully chosen and clearly identified (e.g. by placing them in a single directory).  Right now that's not the case, with some essential header files buried e.g. in contrib/uuid and every include directory mingling essential with non-essential files.

Comment 1 Amar Tumballi 2011-10-25 03:56:13 UTC
Jeff,

Let me know what is your thought on this now.

Regards,
Amar

Comment 2 Amar Tumballi 2012-03-06 06:23:05 UTC
After merging Joe's Patch to make it FHS compatible, this should be fixed.

Comment 3 Sachidananda Urs 2012-05-31 07:35:43 UTC
Amar can you give me the commit id/patch number/url to verify. I think it is e8b6dad4dea4535a179a03824aa44628f9adaa17, need your clarification.

Comment 4 Sachidananda Urs 2012-05-31 07:37:31 UTC
Not part of release, will verify again.

Comment 5 Sachidananda Urs 2012-05-31 07:44:14 UTC
My bad, I see the spec changes in 3.3.0 tar ball, and the new rpms are built on the new spec.