Control: retitle -1 libbtrfs missing headers
Control: found -1 3.17-1
Control: tags -1 + upstream fixed-upstream patch

* Andrey Rahmatullin <w...@debian.org> [2014-11-15 20:49:10 +0500]:

> Control: reassign -1 btrfs-tools
> Control: forcemerge 768694 -1
> 
> On Sun, Nov 09, 2014 at 08:15:16AM +0100, Lucas Nussbaum wrote:
> > > /bin/bash ../libtool  --tag=CXX   --mode=compile g++ -DHAVE_CONFIG_H -I. 
> > > -I.. -I/usr/include/libxml2  -D_FORTIFY_SOURCE=2 
> > > -DCONFDIR='"/etc/default"' -D_FILE_OFFSET_BITS=64 -g -O2 
> > > -fstack-protector-strong -Wformat -Werror=format-security -std=c++0x 
> > > -Wall -Wextra -Wformat=2 -Wnon-virtual-dtor -Wno-unused-parameter -c -o 
> > > BtrfsUtils.lo BtrfsUtils.cc
> > > libtool: compile:  g++ -DHAVE_CONFIG_H -I. -I.. -I/usr/include/libxml2 
> > > -D_FORTIFY_SOURCE=2 -DCONFDIR=\"/etc/default\" -D_FILE_OFFSET_BITS=64 -g 
> > > -O2 -fstack-protector-strong -Wformat -Werror=format-security -std=c++0x 
> > > -Wall -Wextra -Wformat=2 -Wnon-virtual-dtor -Wno-unused-parameter -c 
> > > BtrfsUtils.cc  -fPIC -DPIC -o .libs/BtrfsUtils.o
> > > BtrfsUtils.cc: In function 'void snapper::create_snapshot(int, int, const 
> > > string&, bool, snapper::qgroup_t)':
> > > BtrfsUtils.cc:127:27: error: 'btrfs_qgroup_inherit' was not declared in 
> > > this scope
> The relevant header is included only with HAVE_LIBBTRFS (which is
> apparently wrong because the source can't be compiled without the header)
> which is false because libbtrfs.so can't be linked against.

Well, yes, that's a bug in btrfs-tools: libbtrfs stopped exporting some headers
after an upstream refactoring, and can't be linked against anymore.

Cherry-picking the two following patches from the 3.17 branch of btrfsprogs
restores the headers and lets snapper compile properly.

- 
https://github.com/kdave/btrfs-progs/commit/dcf11c371cbcdca78f297fe042095912634a8323.patch
- 
https://github.com/kdave/btrfs-progs/commit/cafacda441120976105d01c07286e843cb7cbb94.patch

Unless the maintainer disagrees, I will upload a NMU (to DELAYED/5) adding
those two patches.

Cheers,
-- 
Nicolas Dandrimont

BOFH excuse #129:
The ring needs another token

Attachment: signature.asc
Description: Digital signature

Reply via email to