Project

General

Profile

Actions

Feature #23476

open

Add more cmake options to support building against system libraries

Added by Shengjing Zhu about 6 years ago. Updated about 6 years ago.

Status:
New
Priority:
Normal
Assignee:
-
Category:
build
Target version:
-
% Done:

0%

Source:
Tags:
Backport:
Reviewed:
Affected Versions:
Pull request ID:

Description

Most git submodules can be replaced with system libraries. And most linux distribution developers want to use the system libraries instead of the bundled.

So it would be great to add more cmake options like WITH_SYSTEM_BOOST.

I currently began to test building/qa on the unbundled version for Debian, patches for cmake are at https://salsa.debian.org/ceph-team/ceph/tree/debian/experimental/debian/patches

I intend to use this issue to track sending my patches to upstream.

Actions #1

Updated by Nathan Cutler about 6 years ago

I'm not sure most distribution developers prefer using system libraries. The system libraries are often at different versions than the libraries embedded as submodules, and since only the versions embedded as submodules are tested, using the system libraries can cause buggy behavior.

Actions #2

Updated by Shengjing Zhu about 6 years ago

Nathan Cutler wrote:

I'm not sure most distribution developers prefer using system libraries.

Debian has its policy not using bundled libraries1. Fedora also prefer not using bundled2.
robbat2 also told me Gentoo doesn't like bundled on cephalocon.

[1] https://www.debian.org/doc/debian-policy/#document-ch-source
[2] https://fedoraproject.org/wiki/Bundled_Libraries?rd=Packaging:Bundled_Libraries

The system libraries are often at different versions than the libraries embedded as submodules, and since only the versions embedded as submodules are tested, using the system libraries can cause buggy behavior.

Yes, most people concerns about the tests for unbundled version.

So after I adding these options, I'd like to run qa suites on it too(might taking some time, but it's on my plan).

Actions

Also available in: Atom PDF