[Bug 749132] Review Request: dpm-dsi - Disk Pool Manager (DPM) plugin to GridFTP

[bugzilla at redhat.com]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=749132

Mattias Ellert <mattias.ellert at fysast.uu.se> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |mattias.ellert at fysast.uu.se

--- Comment #7 from Mattias Ellert <mattias.ellert at fysast.uu.se> 2011-11-03 11:10:38 EDT ---
When looking around I see (slightly different) copies of the gssapi_openssl.h
scattered around various source trees:

- globus-gssapi-gsi (which seems to be the original)
- CGSI-gSOAP
- lcgdm
- lcas
- lcas-lcmaps-gt4-interface
- lcas-plugins-voms
- lcmaps

The globus-gssapi-gsi Makefile.am file lists this file under
libglobus_gssapi_gsi_la_SOURCES and not under include_HEADERS, so the authors
considers this to be an internal header.

Relying on copies of internal headers of your dependencies is not a good thing.

Considering that so many projects seems to use copies of this file it would
make sense to contact globus upstream and ask them what they think about
installing this header as part of the installation.

I don't know if the whole file is used or if only parts of it. Maybe it would
make sense to split it in a public installed part and a part that remains
private.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.