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

Bug#785770: RM: xserver-xorg-video-siliconmotion [arm64] -- RoP; FTBFS; out-of-date; uninstallable; quite possiblly never worked, little if any utility



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


Reply to: