Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 548558

Summary: dev-python/cffi-0.9.2 - src_test(): testing/test_verify.py injects -Werror
Product: Gentoo Linux Reporter: Jeroen Roovers (RETIRED) <jer>
Component: [OLD] DevelopmentAssignee: Python Gentoo Team <python>
Status: RESOLVED OBSOLETE    
Severity: normal CC: jlec
Priority: Normal Keywords: TESTFAILURE
Version: unspecified   
Hardware: All   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---

Description Jeroen Roovers (RETIRED) gentoo-dev 2015-05-04 05:56:14 UTC
testing/__pycache__/_cffi__xbdcdd635xbdd8af4e.c: In function ‘myprintf’:
testing/__pycache__/_cffi__xbdcdd635xbdd8af4e.c:211:13: error: format not a string literal, argument types not checked [-Werror=format-nonliteral]
             return fprintf(myfile, out, value);
             ^
cc1: all warnings being treated as errors
Comment 1 Mike Gilbert gentoo-dev 2015-05-04 13:18:07 UTC
From reading the code a bit, it seems like this is intentional; the upstream devs want to catch code that generates warnings.

We will probably need to patch/sed this out perpetually.
Comment 2 Patrick Lauer gentoo-dev 2015-05-20 05:53:34 UTC
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ 

tmpdir = '/var/tmp/portage/dev-python/cffi-1.0.0/work/cffi-1.0.0-python3_3/testing/cffi0/__pycache__', ext = <distutils.extension.Extension object at 0x7fc9894f3390>

    def _build(tmpdir, ext):
        # XXX compact but horrible :-(
        _hack_at_distutils()
        from distutils.core import Distribution
        import distutils.errors
        #
        dist = Distribution({'ext_modules': [ext]})
        dist.parse_config_files()
        options = dist.get_option_dict('build_ext')
        options['force'] = ('ffiplatform', True)
        options['build_lib'] = ('ffiplatform', tmpdir)
        options['build_temp'] = ('ffiplatform', tmpdir)
        #
        try:
            dist.run_command('build_ext')
        except (distutils.errors.CompileError,
                distutils.errors.LinkError) as e:
>           raise VerificationError('%s: %s' % (e.__class__.__name__, e))
E           cffi.ffiplatform.VerificationError: CompileError: command 'x86_64-pc-linux-gnu-gcc' failed with exit status 1

cffi/ffiplatform.py:65: VerificationError
----------------------------------------------------------------------------------------------------------- Captured stderr call -----------------------------------------------------------------------------------------------------------
testing/cffi0/__pycache__/_cffi__x412c976ex31f83f0a.c: In function ‘_cffi_f_foo_uint16_t’:
testing/cffi0/__pycache__/_cffi__x412c976ex31f83f0a.c:92:5: error: conversion to ‘uint16_t {aka short unsigned int}’ from ‘int’ may alter its value [-Werror=conversion]
     (sizeof(type) == 1 ? (((type)-1) > 0 ? (type)_cffi_to_c_u8(o)        \
     ^
testing/cffi0/__pycache__/_cffi__x412c976ex31f83f0a.c:737:8: note: in expansion of macro ‘_cffi_to_c_int’
   x0 = _cffi_to_c_int(arg0, uint16_t);
        ^
testing/cffi0/__pycache__/_cffi__x412c976ex31f83f0a.c: In function ‘_cffi_f_foo_uint8_t’:
testing/cffi0/__pycache__/_cffi__x412c976ex31f83f0a.c:92:5: error: conversion to ‘uint8_t {aka unsigned char}’ from ‘int’ may alter its value [-Werror=conversion]
     (sizeof(type) == 1 ? (((type)-1) > 0 ? (type)_cffi_to_c_u8(o)        \
     ^
testing/cffi0/__pycache__/_cffi__x412c976ex31f83f0a.c:797:8: note: in expansion of macro ‘_cffi_to_c_int’
   x0 = _cffi_to_c_int(arg0, uint8_t);
        ^
testing/cffi0/__pycache__/_cffi__x412c976ex31f83f0a.c: In function ‘_cffi_f_foo_uint_fast8_t’:
testing/cffi0/__pycache__/_cffi__x412c976ex31f83f0a.c:92:5: error: conversion to ‘uint_fast8_t {aka unsigned char}’ from ‘int’ may alter its value [-Werror=conversion]
     (sizeof(type) == 1 ? (((type)-1) > 0 ? (type)_cffi_to_c_u8(o)        \
     ^
testing/cffi0/__pycache__/_cffi__x412c976ex31f83f0a.c:877:8: note: in expansion of macro ‘_cffi_to_c_int’
   x0 = _cffi_to_c_int(arg0, uint_fast8_t);
        ^
testing/cffi0/__pycache__/_cffi__x412c976ex31f83f0a.c: In function ‘_cffi_f_foo_uint_least16_t’:
testing/cffi0/__pycache__/_cffi__x412c976ex31f83f0a.c:92:5: error: conversion to ‘uint_least16_t {aka short unsigned int}’ from ‘int’ may alter its value [-Werror=conversion]
     (sizeof(type) == 1 ? (((type)-1) > 0 ? (type)_cffi_to_c_u8(o)        \
     ^
testing/cffi0/__pycache__/_cffi__x412c976ex31f83f0a.c:897:8: note: in expansion of macro ‘_cffi_to_c_int’
   x0 = _cffi_to_c_int(arg0, uint_least16_t);
        ^
testing/cffi0/__pycache__/_cffi__x412c976ex31f83f0a.c: In function ‘_cffi_f_foo_uint_least8_t’:
testing/cffi0/__pycache__/_cffi__x412c976ex31f83f0a.c:92:5: error: conversion to ‘uint_least8_t {aka unsigned char}’ from ‘int’ may alter its value [-Werror=conversion]
     (sizeof(type) == 1 ? (((type)-1) > 0 ? (type)_cffi_to_c_u8(o)        \
     ^
testing/cffi0/__pycache__/_cffi__x412c976ex31f83f0a.c:957:8: note: in expansion of macro ‘_cffi_to_c_int’
   x0 = _cffi_to_c_int(arg0, uint_least8_t);
        ^
testing/cffi0/__pycache__/_cffi__x412c976ex31f83f0a.c: In function ‘_cffi_f_foo_unsigned_char’:
testing/cffi0/__pycache__/_cffi__x412c976ex31f83f0a.c:92:5: error: conversion to ‘unsigned char’ from ‘int’ may alter its value [-Werror=conversion]
     (sizeof(type) == 1 ? (((type)-1) > 0 ? (type)_cffi_to_c_u8(o)        \
     ^
testing/cffi0/__pycache__/_cffi__x412c976ex31f83f0a.c:1017:8: note: in expansion of macro ‘_cffi_to_c_int’
   x0 = _cffi_to_c_int(arg0, unsigned char);
        ^
testing/cffi0/__pycache__/_cffi__x412c976ex31f83f0a.c: In function ‘_cffi_f_foo_unsigned_short’:
testing/cffi0/__pycache__/_cffi__x412c976ex31f83f0a.c:92:5: error: conversion to ‘short unsigned int’ from ‘int’ may alter its value [-Werror=conversion]
     (sizeof(type) == 1 ? (((type)-1) > 0 ? (type)_cffi_to_c_u8(o)        \
     ^
testing/cffi0/__pycache__/_cffi__x412c976ex31f83f0a.c:1097:8: note: in expansion of macro ‘_cffi_to_c_int’
   x0 = _cffi_to_c_int(arg0, unsigned short);
        ^
cc1: all warnings being treated as errors
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! Interrupted: stopping after 1 failures !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
============================================================================================ 1 failed, 540 passed, 40 skipped in 22.97 seconds =============================================================================================
 * ERROR: dev-python/cffi-1.0.0::gentoo failed (test phase):
 *   Testing failed with python3.3

Yes, should be patched out ...
Comment 3 Justin Lecher (RETIRED) gentoo-dev 2015-09-24 11:51:27 UTC
commit 1da5d928cb4d92af2a8433476014f752fc72ebb1
Author: Justin Lecher <jlec@gentoo.org>
Date:   Thu Sep 24 13:50:28 2015 +0200

    dev-python/cffi: Drop old

    obsoletes
    Gentoo-Bug: https://bugs.gentoo.org/show_bug.cgi?id=510114
    Gentoo-Bug: https://bugs.gentoo.org/show_bug.cgi?id=510176
    Gentoo-Bug: https://bugs.gentoo.org/show_bug.cgi?id=548558

    Package-Manager: portage-2.2.21
    Signed-off-by: Justin Lecher <jlec@gentoo.org>

    https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=1da5d928cb4d92af2a8433476014f752fc72ebb1