Source: virt-viewer
Version: 11.0-2
Severity: normal
Tags: patch
Dear Maintainer,
Build fails here:
cc -Itests/test-version-compare.p -Itests -I../tests -I. -I.. -Isrc -I../src
-I/usr/include/glib-2.0 -I/usr/lib/hppa-linux-gnu/glib-2.0/include
-I/usr/include/gtk-3.0 -I/usr/include/at-spi2-atk/2.0 -I/usr/include/at-spi-2.0
-I/usr/include/dbus-1.0 -I/usr/lib/hppa-linux-gnu/dbus-1.0/include
-I/usr/include/gio-unix-2.0 -I/usr/include/cairo -I/usr/include/pango-1.0
-I/usr/include/harfbuzz -I/usr/include/fribidi -I/usr/include/atk-1.0
-I/usr/include/pixman-1 -I/usr/include/uuid -I/usr/include/freetype2
-I/usr/include/gdk-pixbuf-2.0 -I/usr/include/libpng16
-I/usr/include/hppa-linux-gnu -I/usr/include/libmount -I/usr/include/blkid
-fdiagnostics-color=always -D_FILE_OFFSET_BITS=64 -Wall -Winvalid-pch
-std=gnu99 -O0 -Werror -fno-common -W -Wabsolute-value -Waddress
-Waddress-of-packed-member -Waggressive-loop-optimizations -Wall
-Wattribute-warning -Wattributes -Wbool-compare -Wbool-operation
-Wbuiltin-declaration-mismatch -Wbuiltin-macro-redefined -Wcannot-profile
-Wcast-align -Wcast-align=strict -Wcast-function-type -Wchar-subscripts
-Wclobbered -Wcomment -Wcomments -Wcoverage-mismatch -Wcpp -Wdangling-else
-Wdate-time -Wdeprecated-declarations -Wdesignated-init -Wdouble-promotion
-Wdiscarded-array-qualifiers -Wdiscarded-qualifiers -Wdiv-by-zero
-Wduplicated-cond -Wduplicate-decl-specifier -Wempty-body -Wendif-labels
-Wexpansion-to-defined -Wextra -Wformat-contains-nul -Wformat-extra-args
-Wformat-nonliteral -Wformat-security -Wformat-y2k -Wformat-zero-length
-Wframe-address -Wfree-nonheap-object -Whsa -Wif-not-aligned
-Wignored-attributes -Wignored-qualifiers -Wimplicit
-Wimplicit-function-declaration -Wimplicit-int -Wincompatible-pointer-types
-Winit-self -Winline -Wint-conversion -Wint-in-bool-context
-Wint-to-pointer-cast -Winvalid-memory-model -Winvalid-pch
-Wlogical-not-parentheses -Wlogical-op -Wmain -Wmaybe-uninitialized
-Wmemset-elt-size -Wmemset-transposed-args -Wmisleading-indentation
-Wmissing-attributes -Wmissing-braces -Wmissing-declarations
-Wmissing-field-initializers -Wmissing-include-dirs -Wmissing-parameter-type
-Wmissing-profile -Wmissing-prototypes -Wmultichar -Wmultistatement-macros
-Wnarrowing -Wnested-externs -Wnonnull -Wnonnull-compare -Wnull-dereference
-Wodr -Wold-style-declaration -Wold-style-definition -Wopenmp-simd -Woverflow
-Woverride-init -Wpacked-bitfield-compat -Wpacked-not-aligned -Wparentheses
-Wpointer-arith -Wpointer-compare -Wpointer-sign -Wpointer-to-int-cast
-Wpragmas -Wpsabi -Wrestrict -Wreturn-local-addr -Wreturn-type
-Wscalar-storage-order -Wsequence-point -Wshadow -Wshift-count-negative
-Wshift-count-overflow -Wshift-negative-value -Wsizeof-array-argument
-Wsizeof-pointer-div -Wsizeof-pointer-memaccess -Wstrict-aliasing
-Wstrict-prototypes -Wstringop-truncation -Wsuggest-attribute=cold
-Wsuggest-attribute=const -Wsuggest-attribute=format
-Wsuggest-attribute=noreturn -Wsuggest-attribute=pure -Wsuggest-final-methods
-Wsuggest-final-types -Wswitch -Wswitch-bool -Wswitch-unreachable -Wsync-nand
-Wtautological-compare -Wtrampolines -Wtrigraphs -Wtype-limits -Wuninitialized
-Wunknown-pragmas -Wunused -Wunused-but-set-parameter -Wunused-but-set-variable
-Wunused-function -Wunused-label -Wunused-local-typedefs -Wunused-parameter
-Wunused-result -Wunused-value -Wunused-variable -Wvarargs -Wvariadic-macros
-Wvector-operation-performance -Wvla -Wvolatile-register-var -Wwrite-strings
-Walloc-size-larger-than=2147483647 -Warray-bounds=2 -Wattribute-alias=2
-Wformat-overflow=2 -Wformat-truncation=2 -Wimplicit-fallthrough=5
-Wnormalized=nfc -Wshift-overflow=2 -Wstringop-overflow=2
-Wunused-const-variable=2 -Wno-sign-compare -Wno-cast-function-type
-Wjump-misses-init -Wswitch-enum -Wno-format-nonliteral -Wno-format-truncation
-Wframe-larger-than=4096 -fexceptions -fasynchronous-unwind-tables
-fipa-pure-const -Wno-suggest-attribute=pure -Wno-suggest-attribute=const
-fstack-protector-strong -DGLIB_VERSION_MIN_REQUIRED=GLIB_VERSION_2_48
-DGLIB_VERSION_MAX_ALLOWED=GLIB_VERSION_2_48
-DGDK_VERSION_MIN_REQUIRED=GDK_VERSION_3_18
-DGDK_VERSION_MAX_ALLOWED=GDK_VERSION_3_18 '-DLOCALE_DIR="/usr/share/locale"'
-g -O2 -ffile-prefix-map=/<<PKGBUILDDIR>>=. -Wformat -Werror=format-security
-Wdate-time -D_FORTIFY_SOURCE=2 -fPIE -pthread -MD -MQ
tests/test-version-compare.p/test-version-compare.c.o -MF
tests/test-version-compare.p/test-version-compare.c.o.d -o
tests/test-version-compare.p/test-version-compare.c.o -c
../tests/test-version-compare.c
cc1: error: ‘-fstack-protector’ not supported for this target [-Werror]
cc1: all warnings being treated as errors
[49/83] cc -Isrc/libvirt-viewer-common.a.p -Isrc -I../src -I. -I..
-I/usr/include/libxml2 -I/usr/include/glib-2.0
-I/usr/lib/hppa-linux-gnu/glib-2.0/include -I/usr/include/gtk-3.0
-I/usr/include/at-spi2-atk/2.0 -I/usr/include/at-spi-2.0
-I/usr/include/dbus-1.0 -I/usr/lib/hppa-linux-gnu/dbus-1.0/include
-I/usr/include/gio-unix-2.0 -I/usr/include/cairo -I/usr/include/pango-1.0
-I/usr/include/harfbuzz -I/usr/include/fribidi -I/usr/include/atk-1.0
-I/usr/include/pixman-1 -I/usr/include/uuid -I/usr/include/freetype2
-I/usr/include/gdk-pixbuf-2.0 -I/usr/include/libpng16
-I/usr/include/hppa-linux-gnu -I/usr/include/libmount -I/usr/include/blkid
-I/usr/include/gtk-vnc-2.0 -I/usr/include/gvnc-1.0 -I/usr/include/p11-kit-1
-I/usr/include/spice-client-glib-2.0 -I/usr/include/spice-1
-I/usr/include/spice-client-gtk-3.0 -I/usr/include/vte-2.91
-fdiagnostics-color=always -D_FILE_OFFSET_BITS=64 -Wall -Winvalid-pch
-std=gnu99 -O0 -Werror -fno-common -W -Wabsolute-value -Waddress
-Waddress-of-packed-member -Waggressive-loop-optimizations -Wall
-Wattribute-warning -Wattributes -Wbool-compare -Wbool-operation
-Wbuiltin-declaration-mismatch -Wbuiltin-macro-redefined -Wcannot-profile
-Wcast-align -Wcast-align=strict -Wcast-function-type -Wchar-subscripts
-Wclobbered -Wcomment -Wcomments -Wcoverage-mismatch -Wcpp -Wdangling-else
-Wdate-time -Wdeprecated-declarations -Wdesignated-init -Wdouble-promotion
-Wdiscarded-array-qualifiers -Wdiscarded-qualifiers -Wdiv-by-zero
-Wduplicated-cond -Wduplicate-decl-specifier -Wempty-body -Wendif-labels
-Wexpansion-to-defined -Wextra -Wformat-contains-nul -Wformat-extra-args
-Wformat-nonliteral -Wformat-security -Wformat-y2k -Wformat-zero-length
-Wframe-address -Wfree-nonheap-object -Whsa -Wif-not-aligned
-Wignored-attributes -Wignored-qualifiers -Wimplicit
-Wimplicit-function-declaration -Wimplicit-int -Wincompatible-pointer-types
-Winit-self -Winline -Wint-conversion -Wint-in-bool-context
-Wint-to-pointer-cast -Winvalid-memory-model -Winvalid-pch
-Wlogical-not-parentheses -Wlogical-op -Wmain -Wmaybe-uninitialized
-Wmemset-elt-size -Wmemset-transposed-args -Wmisleading-indentation
-Wmissing-attributes -Wmissing-braces -Wmissing-declarations
-Wmissing-field-initializers -Wmissing-include-dirs -Wmissing-parameter-type
-Wmissing-profile -Wmissing-prototypes -Wmultichar -Wmultistatement-macros
-Wnarrowing -Wnested-externs -Wnonnull -Wnonnull-compare -Wnull-dereference
-Wodr -Wold-style-declaration -Wold-style-definition -Wopenmp-simd -Woverflow
-Woverride-init -Wpacked-bitfield-compat -Wpacked-not-aligned -Wparentheses
-Wpointer-arith -Wpointer-compare -Wpointer-sign -Wpointer-to-int-cast
-Wpragmas -Wpsabi -Wrestrict -Wreturn-local-addr -Wreturn-type
-Wscalar-storage-order -Wsequence-point -Wshadow -Wshift-count-negative
-Wshift-count-overflow -Wshift-negative-value -Wsizeof-array-argument
-Wsizeof-pointer-div -Wsizeof-pointer-memaccess -Wstrict-aliasing
-Wstrict-prototypes -Wstringop-truncation -Wsuggest-attribute=cold
-Wsuggest-attribute=const -Wsuggest-attribute=format
-Wsuggest-attribute=noreturn -Wsuggest-attribute=pure -Wsuggest-final-methods
-Wsuggest-final-types -Wswitch -Wswitch-bool -Wswitch-unreachable -Wsync-nand
-Wtautological-compare -Wtrampolines -Wtrigraphs -Wtype-limits -Wuninitialized
-Wunknown-pragmas -Wunused -Wunused-but-set-parameter -Wunused-but-set-variable
-Wunused-function -Wunused-label -Wunused-local-typedefs -Wunused-parameter
-Wunused-result -Wunused-value -Wunused-variable -Wvarargs -Wvariadic-macros
-Wvector-operation-performance -Wvla -Wvolatile-register-var -Wwrite-strings
-Walloc-size-larger-than=2147483647 -Warray-bounds=2 -Wattribute-alias=2
-Wformat-overflow=2 -Wformat-truncation=2 -Wimplicit-fallthrough=5
-Wnormalized=nfc -Wshift-overflow=2 -Wstringop-overflow=2
-Wunused-const-variable=2 -Wno-sign-compare -Wno-cast-function-type
-Wjump-misses-init -Wswitch-enum -Wno-format-nonliteral -Wno-format-truncation
-Wframe-larger-than=4096 -fexceptions -fasynchronous-unwind-tables
-fipa-pure-const -Wno-suggest-attribute=pure -Wno-suggest-attribute=const
-fstack-protector-strong -DGLIB_VERSION_MIN_REQUIRED=GLIB_VERSION_2_48
-DGLIB_VERSION_MAX_ALLOWED=GLIB_VERSION_2_48
-DGDK_VERSION_MIN_REQUIRED=GDK_VERSION_3_18
-DGDK_VERSION_MAX_ALLOWED=GDK_VERSION_3_18 '-DLOCALE_DIR="/usr/share/locale"'
-g -O2 -ffile-prefix-map=/<<PKGBUILDDIR>>=. -Wformat -Werror=format-security
-Wdate-time -D_FORTIFY_SOURCE=2 -fPIC -pthread -MD -MQ
src/libvirt-viewer-common.a.p/glib-compat.c.o -MF
src/libvirt-viewer-common.a.p/glib-compat.c.o.d -o
src/libvirt-viewer-common.a.p/glib-compat.c.o -c ../src/glib-compat.c
The -fstack-protector-strong test passes because the che doesn't add
-Werror:
Compiler for C supports arguments -fstack-protector-strong: YES
First supported argument: -fstack-protector-strong
...
The attached patch fixes build. See:
https://buildd.debian.org/status/fetch.php?pkg=virt-viewer&arch=hppa&ver=11.0-2&stamp=1660418207&raw=0
Regards,
Dave Anglin
-- System Information:
Debian Release: bookworm/sid
APT prefers buildd-unstable
APT policy: (500, 'buildd-unstable'), (500, 'unstable')
Architecture: hppa (parisc64)
Kernel: Linux 5.18.17+ (SMP w/4 CPU threads)
Locale: LANG=C, LC_CTYPE=C.UTF-8 (charmap=UTF-8), LANGUAGE not set
Shell: /bin/sh linked to /bin/dash
Init: systemd (via /run/systemd/system)
Index: virt-viewer-11.0/meson.build
===================================================================
--- virt-viewer-11.0.orig/meson.build
+++ virt-viewer-11.0/meson.build
@@ -278,8 +278,8 @@ if host_machine.cpu_family() != 'aarch64
if host_machine.system() in [ 'linux', 'freebsd', 'windows' ]
# we prefer -fstack-protector-strong but fallback to -fstack-protector-all
fstack_cflags = cc.first_supported_argument([
- '-fstack-protector-strong',
- '-fstack-protector-all',
+ '-fstack-protector-strong -Werror',
+ '-fstack-protector-all -Werror',
])
cc_flags += fstack_cflags
