[PATCH 1/2] s3/modules: New vfs module: cephfs

Sam Lang slang at inktank.com
Fri Apr 19 13:55:40 MDT 2013


On Fri, Apr 19, 2013 at 1:15 PM, Jeremy Allison <jra at samba.org> wrote:
> On Tue, Apr 16, 2013 at 05:25:30PM -0500, Sam Lang wrote:
>> Implements a vfs module for cephfs, a distributed file system
>> with posix semantics, built on the ceph distributed object
>> storage layer.  The ceph vfs module interfaces to the libcephfs
>> userspace API, and is primarily a lightweight wrapper around
>> libcephfs, translating error codes and parameters as necessary.
>
> Ok, I installed the ceph development environment on
> my Ubuntu 12.10 system and build this. I got the
> following errors. Can you fix these (mostly casts
> I think,

The warnings with printf are platform specific (unsigned long must not
be 64 bits on your system).  The standard way to resolve this is samba
is to cast?  Down to a 32-bit int?  Is using PRIu64 a viable option in
samba code?

 plus you'll need to use smb_strdup()
> not strdup()) then resubmit ?

Yep.  Not sure why doesn't throw an error on my build...
-sam

>
> Thanks,
>
> Jeremy.


More information about the samba-technical mailing list