On 07/14/2010 10:06 AM, Rex Dieter wrote:
On 07/14/2010 10:58 AM, Orion Poplawski wrote:
> On 06/28/2010 03:27 PM, Kalev Lember wrote:
>> Perhaps it'd be better to completely remove -DBUILD_SHARED_LIBS:BOOL=ON
>> from the default %cmake macro? I would imagine that most (all?)
>> libraries which are meant to be actually used as shared libraries
>> contain explicit add_library(... SHARED) and set_target_properties(...
>> PROPERTIES VERSION ... SOVERSION ...) calls, but maybe that's not how it
>> is. Any thoughts whether removing BUILD_SHARED_LIBS=ON would break
>> existing packages?
>
> Sorry for not replying earlier.
>
> Looking at my existing cmake built packages, I'm a bit leery of this change.
> Most of them don't seem to specify SHARED in add_library. Has anyone tested
> this out on a large number of packages? Perhaps it makes sense to request a
> cmake koji build tag for this and rebuild all cmake built packages there.
Either way, I'm sure we'd be up to the task of clearly documenting the
behavior change, and helping fix/cleanup any resulting messes.
Discounting how much actually breakage ensues, on principal how do you
feel about this change?
I'm of two minds:
- It's good to have the %cmake macro has as little hidded vodoo in it as possible.
- Fedora strongly encourages shared libraries as shared libraries are good.
If a package really wants a static library it should explicitly state it.
I'm fine for trying it, but we really should have a mass cmake rebuild
afterwards to shake out the issues.
I'm moving this to fedora-devel to get more comment.
--
Orion Poplawski
Technical Manager 303-415-9701 x222
NWRA/CoRA Division FAX: 303-415-9702
3380 Mitchell Lane orion(a)cora.nwra.com
Boulder, CO 80301
http://www.cora.nwra.com