[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#828853: marked as done (libexplain: FTBFS - 'FS_ECOMPR_FL' undeclared)

Your message dated Sun, 14 Aug 2016 18:49:23 +0000
with message-id <E1bZ0TL-0008A6-Kg@franck.debian.org>
and subject line Bug#828853: fixed in libexplain 1.4.D001-3
has caused the Debian Bug report #828853,
regarding libexplain: FTBFS - 'FS_ECOMPR_FL' undeclared
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact owner@bugs.debian.org
immediately.)


-- 
828853: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=828853
Debian Bug Tracking System
Contact owner@bugs.debian.org with problems
--- Begin Message --- 
Package: libexplain
Version: 1.4.D001-2+b1
Severity: serious
Tags: sid + patch
Justification: FTBFS
User: debian-mips@lists.debian.org
Usertags: mips-patch

Hi,

Package libexplain FTBFS on mips and mipsel with following error:
libexplain/buffer/file_inode_flags.c:47:27: error: 'FS_ECOMPR_FL' undeclared (first use in this function)
         { "FS_ECOMPR_FL", FS_ECOMPR_FL },
                           ^
libexplain/buffer/file_inode_flags.c:47:27: note: each undeclared identifier is reported only once for each function it appears in
libexplain/buffer/file_inode_flags.c:47:9: warning: missing initializer for field 'value' of 'explain_parse_bits_table_t {aka const struct explain_parse_bits_table_t}' [-Wmissing-field-initializers]
         { "FS_ECOMPR_FL", FS_ECOMPR_FL },
         ^
In file included from libexplain/buffer/file_inode_flags.c:25:0:
./libexplain/parse_bits.h:31:21: note: 'value' declared here
     int             value;
                     ^
libexplain/buffer/file_inode_flags.c:56:29: error: 'FS_DIRECTIO_FL' undeclared (first use in this function)
         { "FS_DIRECTIO_FL", FS_DIRECTIO_FL },
                             ^
libexplain/buffer/file_inode_flags.c:56:9: warning: missing initializer for field 'value' of 'explain_parse_bits_table_t {aka const struct explain_parse_bits_table_t}' [-Wmissing-field-initializers]
         { "FS_DIRECTIO_FL", FS_DIRECTIO_FL },
         ^
In file included from libexplain/buffer/file_inode_flags.c:25:0:
./libexplain/parse_bits.h:31:21: note: 'value' declared here
     int             value;
                     ^
Makefile:10623: recipe for target 'libexplain/buffer/file_inode_flags.lo' failed
make[1]: *** [libexplain/buffer/file_inode_flags.lo] Error 1

build logs:
https://buildd.debian.org/status/fetch.php?pkg=libexplain&arch=mips&ver=1.4.D001-2%2Bb1&stamp=1465482111
https://buildd.debian.org/status/fetch.php?pkg=libexplain&arch=mipsel&ver=1.4.D001-2%2Bb1&stamp=1465051476

The code as it is doesn't compile against latest linux headers because FS_ECOMPR_FL and FS_DIRECTIO_FL defines are removed:
https://github.com/torvalds/linux/commit/68ce7bfcd995a8a393b1b14fa67dbc16fa3dc784

On openSUSE this issue is resolved by defining the missing defines in libexplain package:
https://build.opensuse.org/package/view_file/openSUSE:Factory/libexplain/libexplain-1.4-missing-defines.patch?expand=1

With attached patch I was able to build libexplain successfully for mips, mipsel, i386 and amd64.

Regards,
Daniel
--- libexplain-1.4.D001.orig/libexplain/buffer/file_inode_flags.c
+++ libexplain-1.4.D001/libexplain/buffer/file_inode_flags.c
@@ -28,6 +28,14 @@

 #if defined(FS_IOC_GETFLAGS) || defined(FS_IOC32_GETFLAGS)

+#ifndef FS_ECOMPR_FL
+#define FS_ECOMPR_FL                    0x00000800 /* Compression error */
+#endif
+
+#ifndef FS_DIRECTIO_FL
+#define FS_DIRECTIO_FL                  0x00100000 /* Use direct i/o */
+#endif
+
 void
 explain_buffer_file_inode_flags(explain_string_buffer_t *sb, int value)
 {

--- End Message ---
--- Begin Message --- 
Source: libexplain
Source-Version: 1.4.D001-3

We believe that the bug you reported is fixed in the latest version of
libexplain, which is due to be installed in the Debian FTP archive.

A summary of the changes between this version and the previous one is
attached.

Thank you for reporting the bug, which will now be closed.  If you
have further comments please address them to 828853@bugs.debian.org,
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
James Cowgill <jcowgill@debian.org> (supplier of updated libexplain package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing ftpmaster@ftp-master.debian.org)


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA512

Format: 1.8
Date: Sun, 14 Aug 2016 19:21:21 +0100
Source: libexplain
Binary: explain libexplain-doc libexplain51 libexplain-dev
Architecture: source
Version: 1.4.D001-3
Distribution: unstable
Urgency: medium
Maintainer: Debian QA Group <packages@qa.debian.org>
Changed-By: James Cowgill <jcowgill@debian.org>
Description:
 explain    - utility to explain system call errors
 libexplain-dev - library of system-call-specific strerror repl - development files
 libexplain-doc - library of system-call-specific strerror repl - documentation
 libexplain51 - library of system-call-specific strerror repl
Closes: 770819 775357 791347 828853
Changes:
 libexplain (1.4.D001-3) unstable; urgency=medium
 .
   * QA upload.
 .
   * debian/control:
     - Replace -dbg package with automatic package.
     - Use secure Vcs-Git URL.
     - Bump standards to 3.9.8.
     - Drop useless direct dependency from explain to libexplain51.
     - Run wrap-and-sort.
   * debian/patches:
     - Number the patches.
     - 03: Fix FTBFS with linux headers 4.5. (Closes: #828853)
     - 04: Fix test t0274a.sh on recent kernels.
     - 05: Fix handling of O_LARGEFILE on various 64-bit arches.
       (Closes: #775357, #791347)
     - 06: Only compile sysctl code if sysctl exists. (Closes: #770819)
     - 07: Do not require fully defined struct ustat.
   * debian/rules:
     - Enable parallel build.
Checksums-Sha1:
 c6b20c96d0676d0431b9142a85d87bcf1d70360b 2180 libexplain_1.4.D001-3.dsc
 1ac884b0a92faecc6ddb4c6cf8fb779ad75b7036 41416 libexplain_1.4.D001-3.debian.tar.xz
Checksums-Sha256:
 915e486e019c97ed0f10b7a0ee3a9104b78264acf7f25980efd88992e2d824f6 2180 libexplain_1.4.D001-3.dsc
 ddbd95ec082674b7b2892e77fc37c9d3f94c861d6c5f7da9061c8598c7b8bb3e 41416 libexplain_1.4.D001-3.debian.tar.xz
Files:
 a57bcdcc53a0f624ec23d606570c6949 2180 devel optional libexplain_1.4.D001-3.dsc
 bc13cafc6f642e82bff9ff68838a66d4 41416 devel optional libexplain_1.4.D001-3.debian.tar.xz

-----BEGIN PGP SIGNATURE-----

iQIcBAEBCgAGBQJXsLseAAoJEMfxZ23qLQHvUsMP/iOxPKGhZ3vD9ViXIsKFmFCZ
khGAIdbU94sCq6ONofbzflmLtTjCi35ijpUrSSU9HKkmadNlTtjq6z/c7bUk9C17
G9EUXi39SRyELv6f115FcYYuEWuAiRTamhwTIpRXdYcj/gFrPaJv1g9xKxKr0cnA
iY8BH8022rDiDSyAHFsRHYqzevarHb05RdeZDBzow+ek09Y20bri2XHa+anrxKTC
XzMHcwFbnzH5zXukK1WfPHn47pTrajJ4WVcKctEzUutUZhcnsc/Al06yLHkUBnJt
H42IpwqFpW/AJHt6nrK3uKztbVM1APHDfyhrPZ8I6useqpSP34r+sDg78B8kkI1p
p/aUOohYpmhgrBPeIiScwHDb8L2UR3b99fW4ELetH2ja61cxesHAU4x6LiJDhWOb
7G4k9d/0SjS75nMjNiPxZ2q8lXE+aO8SF70kgynpswlEDyOMOow1UU32eH+FPmuf
3Bkzo3Qdi4my07lCDfTu3nTAq9yutOkf7rzZjdLDI7KuIvvzPBhiYYVQu7qfVpWD
+qnJR2fLSXGwKkLvEGtJSreMTCC4Q7kbo6imZSFQV4OiQnUiaOhWcCEz8MyhZSS5
vdB3ntiCXppcx23H7W9o7iscTZEbWccWSu9qksoS49F4PxVFvJxKBlJvUhga18ip
SSoglTDgD3sjObq7Fn0h
=Rne9
-----END PGP SIGNATURE-----

--- End Message ---
Reply to: