Page MenuHomeFreeBSD

inet/ceph-devel: bimonthly ports update v2017-07-27
ClosedPublic

Authored by wjw_digiware.nl on Jul 28 2017, 8:00 AM.

Details

Summary

The "bimonthly" port upgrade to the current state of affairs.
This version is more or less equal to

12.1.2 Luminous RC2

Major diffrence is that it also incluse the code for rbd-ggate, which allows to mount ceph-pools as raw device. (/dev/ggate)

ceph-fuse is working, albeit slow and typical fuse style.

Test Plan

Ceph is extensively tested by Jenkins at:

http://cephdev.digiware.nl:8180/jenkins

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 10725
Build 11122: arc lint + arc unit

Event Timeline

net/ceph-devel/Makefile
20

Maybe you can add this to LIB_DEPENDS? without ${LOCALBASE} part

net/ceph-devel/Makefile
20

This was already in the original patch, and I think we had some discussion on it there too.
What I remember is that LIB_DEPENDS only likes .so files.
But I'll give it a try.

net/ceph-devel/Makefile
20

Portlint warns:
WARN: Makefile: LIB_DEPENDS the new format is libFOO.so (e.g., liblibatomic_ops.a.so).

20

make package:
Error: pattern libatomic_ops.a in LIB_DEPENDS is not valid

So nope, that does not work.

net/ceph-devel/Makefile
20

is it because of .a extension or because you have ${LOCALBASE} before it?

net/ceph-devel/Makefile
20

Because of the .a extension.

net/ceph-devel/Makefile
20

LIB_DEPENDS is for shared libraries, not static libraries. (Static libraries should not be used in the ports tree, but even after 20 something years of them being outdated, they persist.)

net/ceph-devel/Makefile
20

@mat
I understand and appreciate your sentiment.

The project uses it, so I need it for building, and it is in ports.
And it does not generate an .so

It would not not an easy fix for me, because I would have to dive into libatomic_ops
And only just by its name, I would not like to touch it with a 10 meter pole.

Any other suggestion for getting the port loaded that pleases 'the court' is welcome

@mat @mmokhi
Any particular reason why this cannot move forward.
I'm having the first official release of net/ceph in the pipe.
Just waiting for it to be release by the Ceph-developers.
Would be nice if that one have not the same discussion we have here.

@mat @mmokhi
Any particular reason why this cannot move forward.
I'm having the first official release of net/ceph in the pipe.
Just waiting for it to be release by the Ceph-developers.
Would be nice if that one have not the same discussion we have here.

I think I already said it, but, this, here, is a code review tool, not a bug report tool.
If you want a patch committed in the tree, the fastest way is to open a bug report on our bugzilla.

@mat
I understand, and will ask on the ports-list if somebody wants to do the honnurs.

pi added a subscriber: pi.

testbuilds are fine on 11a, 12a.

This revision is now accepted and ready to land.Aug 11 2017, 3:47 PM
This revision was automatically updated to reflect the committed changes.