Package: ftp.debian.org
x-debbugs-cc: debian-arm@lists.debian.org,debian-x@lists.debian.org
A recent xorg update caused xserver-xorg-siliconmotion to fail to build
with the following error.
/bin/bash ../libtool --tag=CC --mode=compile gcc -std=gnu99
-DHAVE_CONFIG_H -I. -I../../src -I.. -D_FORTIFY_SOURCE=2
-fvisibility=hidden -I/usr/include/pixman-1 -I/usr/include/libdrm
-I/usr/include/xorg -I/usr/include/X11/dri -Wall -Wpointer-arith
-Wmissing-declarations -Wformat=2 -Wstrict-prototypes
-Wmissing-prototypes -Wnested-externs -Wbad-function-cast
-Wold-style-definition -Wdeclaration-after-statement -Wunused
-Wuninitialized -Wshadow -Wmissing-noreturn -Wmissing-format-attribute
-Wredundant-decls -Wlogical-op -Werror=implicit -Werror=nonnull
-Werror=init-self -Werror=main -Werror=missing-braces
-Werror=sequence-point -Werror=return-type -Werror=trigraphs
-Werror=array-bounds -Werror=write-strings -Werror=address
-Werror=int-to-pointer-cast -Werror=pointer-to-int-cast
-fno-strict-aliasing -g -O2 -fstack-protector-strong -Wformat
-Werror=format-security -c -o smi_501.lo ../../src/smi_501.c
libtool: compile: gcc -std=gnu99 -DHAVE_CONFIG_H -I. -I../../src -I..
-D_FORTIFY_SOURCE=2 -fvisibility=hidden -I/usr/include/pixman-1
-I/usr/include/libdrm -I/usr/include/xorg -I/usr/include/X11/dri -Wall
-Wpointer-arith -Wmissing-declarations -Wformat=2 -Wstrict-prototypes
-Wmissing-prototypes -Wnested-externs -Wbad-function-cast
-Wold-style-definition -Wdeclaration-after-statement -Wunused
-Wuninitialized -Wshadow -Wmissing-noreturn -Wmissing-format-attribute
-Wredundant-decls -Wlogical-op -Werror=implicit -Werror=nonnull
-Werror=init-self -Werror=main -Werror=missing-braces
-Werror=sequence-point -Werror=return-type -Werror=trigraphs
-Werror=array-bounds -Werror=write-strings -Werror=address
-Werror=int-to-pointer-cast -Werror=pointer-to-int-cast
-fno-strict-aliasing -g -O2 -fstack-protector-strong -Wformat
-Werror=format-security -c ../../src/smi_501.c -fPIC -DPIC -o
.libs/smi_501.o
In file included from ../../src/smi.h:317:0,
from ../../src/smi_501.c:33:
../../src/regsmi.h: In function 'VGAIN8_INDEX':
../../src/regsmi.h:67:2: error: implicit declaration of function
'outb' [-Werror=implicit-function-declaration]
outb(pSmi->PIOBase + indexPort, index);
^
../../src/regsmi.h:67:2: warning: nested extern declaration of 'outb'
[-Wnested-externs]
../../src/regsmi.h:68:2: error: implicit declaration of function 'inb'
[-Werror=implicit-function-declaration]
return(inb(pSmi->PIOBase + dataPort));
^
../../src/regsmi.h:68:2: warning: nested extern declaration of 'inb'
[-Wnested-externs]
cc1: some warnings being treated as errors
Further investigation (see
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=785491) showed that
the reason for the FTBFS was the removal of non-functional stub
implementations of those functions from xorg.
I belive the out of date binary should be removed from sid for the
following reasons.
1: it's uninstallable with the xorg currently in sid/stretch
2: while I have not investigated throughly it seems very likely that the
use of stub versions of those functions would have rendered the driver
unusable.
3: the siliconmotion chips in question seem to be obsolete parts mostly
used for onboard graphics in old systems (though I did find one
industrial minipcie card that claimed to use a siliconmotion chip, dunno
if it's one supported by this driver). It seems unlikely that one would
end up in an arm64 system.
If this removal request is actioned I will downgrade bug 785491