Ceph » rgw

Thomas/Marcus: Please open a PR at ceph/ceph.git against the "jewel" branch. The end of the PR description could look like this:

Fixes: http://tracker.ceph.com/issues/19098
Signed-off-by: Marcus Watts . . . 

Note: this autotools-specific fix is not cherry-picked from master because master uses cmake.

Thanks!

I'll make a PR for master that fixes cmake. Is it worth trying to fix the cmake for this in jewel also or do we just ignore it?

Marcus Watts wrote:

Is it worth trying to fix the cmake for this in jewel also or do we just ignore it?

cmake in jewel is not production-ready so we are not backporting cmake fixes to jewel.

@Marcus Sorensen: This issue (#19096) is for your autotools fix - are you going to open a PR against jewel for that, at or around the same time that you open the master PR with the cmake version?

Well, I think I have good news. Turns out the cmake logic already gets the right soname in xenial. So, nothing to fix there.

So this is only a problem if using autoconf/configure to do builds. I believe we no longer support that in master. Do we in kraken?

I believe we no longer support that in master. Do we in kraken?

Nope. Jewel is the last release that uses autotools, and Kraken is the first that uses cmake. (There is some cmake stuff in Jewel, but it is not supported.)

Since Nathan suggested making another PR to address this I've created https://github.com/ceph/ceph/pull/14215 with the required change. Looks like that's going to incorporate and supersede PR # 12917.