Bug #2390
closeddencoder: depends on expat
0%
Description
dencoder uses rgw_dencoder.cc which requires the expat library.
Updated by Sage Weil almost 12 years ago
Hmm, ideally ceph-dencoder would still understand the types but not need expat to compile. Which is shouldn't, in principle.. maybe #ifdef out the expat bits if rgw itself isn't enabled? or HAVE_EXPAT isn't defined or something like that?
Updated by Sage Weil about 11 years ago
- Status changed from New to Won't Fix
not sure this is worth fixing or worrying about.
Updated by David Disseldorp over 8 years ago
I plan on submitting a fix for the (currently broken) Expat dependency checking:
- Check for Expat regardless of rgw build settings, as it's needed for dencoder
- Check for the expat.h header file, rather than just the runtime dependency
Could someone please reopen this bug? It looks like I don't have permission to do so.
Updated by Nathan Cutler over 8 years ago
- Status changed from Won't Fix to In Progress
Updated by David Disseldorp over 8 years ago
David Disseldorp wrote:
I plan on submitting a fix for the (currently broken) Expat dependency checking:
- Check for Expat regardless of rgw build settings, as it's needed for dencoder
Fix submitted via:
https://github.com/ceph/ceph/pull/6772
Updated by Nathan Cutler over 8 years ago
- Status changed from In Progress to Fix Under Review
Updated by David Disseldorp about 8 years ago
This was fixed via:
https://github.com/ceph/ceph/pull/7476
This bug can be closed.
Updated by Nathan Cutler about 8 years ago
- Status changed from Fix Under Review to Resolved