Project

General

Profile

Actions
Copy link

Feature #760

closed

librbd: Include headers in the development package

Added by Wido den Hollander about 13 years ago. Updated about 13 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Yehuda Sadeh
Category:
librbd
Target version:
-
% Done:

70%

Spent time:
2:00 h
Source:
Tags:
Backport:
Reviewed:
Affected Versions:
Pull request ID:

Description

Right now librbd.h and librbd.hpp are not added to librbd1-dev

Attached is a patch which includes them.

Files

Download all files
0001-librbd-Install-the-headers-in-the-development-packag.patch (1.18 KB) 0001-librbd-Install-the-headers-in-the-development-packag.patch Wido den Hollander, 02/03/2011 05:07 AM
0002-librbd-Use-the-correct-namespace-for-string-and-buff.patch (1.59 KB) 0002-librbd-Use-the-correct-namespace-for-string-and-buff.patch Wido den Hollander, 02/03/2011 05:52 AM
Actions
Copy link
 #1

Updated by Wido den Hollander about 13 years ago

Not worth opening a new issue, see the second patch.

Use the right namespace for string and bufferlist.

Actions
Copy link
 #2

Updated by Sage Weil about 13 years ago

  • Assignee set to Yehuda Sadeh
Actions
Copy link
 #3

Updated by Yehuda Sadeh about 13 years ago

Applying the first patch. Shouldn't the namespace in the second one be ceph and not librados?

Actions
Copy link
 #4

Updated by Wido den Hollander about 13 years ago

It's declared in rados/buffer.h, so it should be librados?

Btw, you just added AIO write with a bufferlist without a namespace.

Actions
Copy link
 #5

Updated by Yehuda Sadeh about 13 years ago

  • Status changed from New to Resolved

Actually rados/buffer.h is just a symlink to include/buffer.h and is actually under the ceph namespace and shared between the server and client code. I modified the librbd.hpp to use the ceph namespace.

Actions
Copy link

Also available in: Atom PDF