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

Bug#1089334: gcpegg: Supporting rootless builds by default



Source: gcpegg
Version: 5.1-16
Severity: important
Tags: ftbfs
Justification: FTBFS
X-Debbugs-Cc: niels@thykier.net
User: niels@thykier.net
Usertags: rrr-no-as-default-issue

Dear maintainer,

During a test rebuild for building packages with
`Rules-Requires-Root: no` as the default in `dpkg`,
gcpegg failed to rebuild.

Log Summary:
-------------------------------------------------------------------------------
[...]
55a735061e1262640b8cd5117c942659b47357f4bb3e99fd4fb86a564ee038f7 13672 gcpegg_5.1-16.debian.tar.xz
Files:
 9f479408fa34c873db5500eba2a574b0 53670 gcpegg_5.1.orig.tar.gz
 1ccc8a225e004b08b559abad56cd19aa 13672 gcpegg_5.1-16.debian.tar.xz

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

iQIzBAEBCgAdFiEEHguq2FwiMqGzzpLrtwRxBYMLn6EFAmXRENgACgkQtwRxBYML
n6HDIRAArmVLj2U96eSKOG6C7yb68A3v6TJzexQSbilUFfV7EyGeYkbqPAeozxVD
EvAx2P/F/ymEbdjFcwq/52Tq0B6WDh8vzIWsWwLAEm0rbENIrZFPxRDM5p4MtFsy
tOrF/gWTG1smUrjdLZoXf92xFbt1bnRHjpGgXwFKVPjBNqnMnLAMAQr6SyhROLn7
B+9hqy98eGzZLP28rVPmpd+p/9GSni+xpLXOXlYMg1T6cLcNKg2z7XQGRdZCpWO3
Sz89mQE3teqMJ5TTw2N3ZSwk7vjyDoOZkqviDUDJWSUdn+G1o028Tu6bwmcVPitZ
lyLp5bBbBibbRWZuBvTeXZtY1zgHaYzRvFACDwgkyvbjLRb2T68lDrejCZfRxFuj
50fr9XQA4pcoo/FkNtmO1UqcilUp7CjfU1G5L3Oi0fqWsh2NDmzmrfN5VlvWUgRt
w744U5XQAgRWZDLFTdKh9ZPxAzAX+dWGHqOf3HJk5VCs3hxN3Kncl+nL6+wekkY2
K2UwMabJ0fhGHvAZ32yvWjDQlohfiZcfFUlwoYu1z3rmsZvrn763DRLuq5VHrByT
Gx8FyfbL8BkHStvouFrdct+6T6reWlF5JgkKtqGG7FHYFipqO4VNlZ9pUoYsvuHo
gFqmfNVAKJ9qzCGbx4Y6W2QnEL60LF1wsV7kXgm1sTX7UALH01E=
=JFxY
-----END PGP SIGNATURE-----

gpgv: Signature made Sat Feb 17 20:02:32 2024 UTC
gpgv:                using RSA key 1E0BAAD85C2232A1B3CE92EBB7047105830B9FA1
gpgv: Can't check signature: No public key
dpkg-source: warning: cannot verify inline signature for ./gcpegg_5.1-16.dsc: no acceptable signature found
dpkg-source: info: extracting gcpegg in /<<PKGBUILDDIR>>
dpkg-source: info: unpacking gcpegg_5.1.orig.tar.gz
dpkg-source: info: unpacking gcpegg_5.1-16.debian.tar.xz
dpkg-source: info: using patch list from debian/patches/series
dpkg-source: info: applying all_my_changes.diff

Check disk space
----------------

Sufficient free space for build

User Environment
----------------

APT_CONFIG=/var/lib/sbuild/apt.conf
HOME=/sbuild-nonexistent
LANG=en_US.UTF-8
LC_ALL=C.UTF-8
LOGNAME=debusine-worker
PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games
SHELL=/bin/sh
USER=debusine-worker

dpkg-buildpackage
-----------------

Command: dpkg-buildpackage -us -uc -b -rfakeroot
dpkg-buildpackage: info: source package gcpegg
dpkg-buildpackage: info: source version 5.1-16
dpkg-buildpackage: info: source distribution unstable
dpkg-buildpackage: info: source changed by Bdale Garbee <bdale@gag.com>
 dpkg-source --before-build .
dpkg-buildpackage: info: host architecture arm64
 debian/rules clean
dh_testdir
dh_testroot
rm -f build-stamp install-stamp
[ ! -f Makefile ] || /usr/bin/make clean
make[1]: Entering directory '/<<PKGBUILDDIR>>'
rm -f eggsh basket regtest *.o *.bak .*.bak core dumpreg.dat egg.status eggsample.pid
make[1]: Leaving directory '/<<PKGBUILDDIR>>'
dh_clean
	rm -f debian/debhelper-build-stamp
	rm -rf debian/.debhelper/
	rm -f -- debian/gcpegg.substvars debian/files
	rm -fr -- debian/gcpegg/ debian/tmp/
	find .  \( \( \
\( -path .\*/.git -o -path .\*/.svn -o -path .\*/.bzr -o -path .\*/.hg -o -path .\*/CVS -o -path .\*/.pc -o -path .\*/_darcs \) -prune -o -type f -a \
	        \( -name '#*#' -o -name '.*~' -o -name '*~' -o -name DEADJOE \
		 -o -name '*.orig' -o -name '*.rej' -o -name '*.bak' \
		 -o -name '.*.orig' -o -name .*.rej -o -name '.SUMS' \
		 -o -name TAGS -o \( -path '*/.deps/*' -a -name '*.P' \) \
		\) -exec rm -f {} + \) -o \
\( -type d -a \( -name autom4te.cache -o -name __pycache__ \) -prune -exec rm -rf {} + \) \)
 debian/rules binary
dh_testdir
/usr/bin/make make[1]: Entering directory '/<<PKGBUILDDIR>>'
gcc -g -ansi -Wall -DLinux -DUSLEEP -D_GNU_SOURCE -DREPORT=1 -DEGG_DYNAMIC -c -o egg.o egg.c
egg.c: In function ‘main’:
egg.c:184:26: warning: variable ‘remname’ set but not used [-Wunused-but-set-variable]
  184 |   char                  *remname;
      |                          ^~~~~~~
gcc -g -ansi -Wall -DLinux -DUSLEEP -D_GNU_SOURCE -DREPORT=1 -DEGG_DYNAMIC -c -o eggui.o eggui.c gcc -g -ansi -Wall -DLinux -DUSLEEP -D_GNU_SOURCE -DREPORT=1 -DEGG_DYNAMIC -c -o lecuyer.o lecuyer.c gcc -g -ansi -Wall -DLinux -DUSLEEP -D_GNU_SOURCE -DREPORT=1 -DEGG_DYNAMIC -c -o usleep.o usleep.c gcc -g -ansi -Wall -DLinux -DUSLEEP -D_GNU_SOURCE -DREPORT=1 -DEGG_DYNAMIC -c -o reg_orion.o reg_orion.c
reg_orion.c: In function ‘OpenDev’:
reg_orion.c:100:19: warning: too many arguments for format [-Wformat-extra-args]
  100 |   sprintf(ttydev, "/dev/REG", opts->port);
      |                   ^~~~~~~~~~
reg_orion.c:81:17: warning: variable ‘res’ set but not used [-Wunused-but-set-variable]
   81 |   int32 TTY_fd, res;
      |                 ^~~
reg_orion.c: In function ‘EvalSpeed’:
reg_orion.c:251:16: warning: variable ‘samp’ set but not used [-Wunused-but-set-variable]
  251 |   int32 bitct, samp;
      |                ^~~~
gcc -g -ansi -Wall -DLinux -DUSLEEP -D_GNU_SOURCE -DREPORT=1 -DEGG_DYNAMIC -c -o reg_pear.o reg_pear.c
reg_pear.c: In function ‘OpenDev’:
reg_pear.c:80:19: warning: too many arguments for format [-Wformat-extra-args]
   80 |   sprintf(ttydev, "/dev/REG", opts->port);
      |                   ^~~~~~~~~~
reg_pear.c:61:17: warning: variable ‘res’ set but not used [-Wunused-but-set-variable]
   61 |   int32 TTY_fd, res;
      |                 ^~~
reg_pear.c: In function ‘EvalSpeed’:
reg_pear.c:177:16: warning: variable ‘samp’ set but not used [-Wunused-but-set-variable]
  177 |   int32 bitct, samp;
      |                ^~~~
gcc -g -ansi -Wall -DLinux -DUSLEEP -D_GNU_SOURCE -DREPORT=1 -DEGG_DYNAMIC -c -o reg_pseudo.o reg_pseudo.c
reg_pseudo.c: In function ‘EvalSpeed’:
reg_pseudo.c:99:16: warning: variable ‘samp’ set but not used [-Wunused-but-set-variable]
   99 |   int32 bitct, samp;
      |                ^~~~
gcc -g -ansi -Wall -DLinux -DUSLEEP -D_GNU_SOURCE -DREPORT=1 -DEGG_DYNAMIC -c -o storage.o storage.c
storage.c: In function ‘SavePacket’:
storage.c:220:25: warning: variable ‘res’ set but not used [-Wunused-but-set-variable]
  220 |   uint32        pktime, res;
      |                         ^~~
storage.c: In function ‘next_filename’:
storage.c:501:17: warning: variable ‘res’ set but not used [-Wunused-but-set-variable]
  501 |   int32         res;
      |                 ^~~
gcc -g -ansi -Wall -DLinux -DUSLEEP -D_GNU_SOURCE -DREPORT=1 -DEGG_DYNAMIC -c -o network.o network.c
network.c: In function ‘NetListen’:
network.c:205:25: warning: variable ‘pkttype’ set but not used [-Wunused-but-set-variable]
  205 |   uint16                pkttype, cksumt, cksumc;
      |                         ^~~~~~~
gcc -g -ansi -Wall -DLinux -DUSLEEP -D_GNU_SOURCE -DREPORT=1 -DEGG_DYNAMIC -c -o crc16.o crc16.c gcc -g -ansi -Wall -DLinux -DUSLEEP -D_GNU_SOURCE -DREPORT=1 -DEGG_DYNAMIC -c -o genlib.o genlib.c gcc -g -ansi -Wall -DLinux -DUSLEEP -D_GNU_SOURCE -DREPORT=1 -DEGG_DYNAMIC -c -o xdsub.o xdsub.c gcc -g -ansi -Wall -DLinux -DUSLEEP -D_GNU_SOURCE -DREPORT=1 -DEGG_DYNAMIC -o eggsh egg.o eggui.o lecuyer.o usleep.o reg_orion.o reg_pear.o reg_pseudo.o storage.o network.o crc16.o genlib.o xdsub.o -lncurses gcc -g -ansi -Wall -DLinux -DUSLEEP -D_GNU_SOURCE -DREPORT=1 -DEGG_DYNAMIC -c -o basket.o basket.c
basket.c: In function ‘main’:
basket.c:245:26: warning: variable ‘remname’ set but not used [-Wunused-but-set-variable]
  245 |   char                  *remname;
      |                          ^~~~~~~
basket.c: In function ‘BasketReceiveDataPacket’:
basket.c:614:25: warning: variable ‘offset’ set but not used [-Wunused-but-set-variable]
  614 |   uint16                offset, rec;
      |                         ^~~~~~
gcc -g -ansi -Wall -DLinux -DUSLEEP -D_GNU_SOURCE -DREPORT=1 -DEGG_DYNAMIC -o basket basket.o storage.o network.o crc16.o genlib.o xdsub.o -lncurses gcc -g -ansi -Wall -DLinux -DUSLEEP -D_GNU_SOURCE -DREPORT=1 -DEGG_DYNAMIC regtest.c -o regtest
regtest.c: In function ‘main’:
regtest.c:33:10: warning: variable ‘res’ set but not used [-Wunused-but-set-variable]
   33 |   int i, res, n1, mean;
      |          ^~~
make[1]: Leaving directory '/<<PKGBUILDDIR>>'
touch build-stamp
dh_testdir
dh_testroot
dh_prep
	rm -f -- debian/gcpegg.substvars
	rm -fr -- debian/.debhelper/generated/gcpegg/ debian/gcpegg/ debian/tmp/
dh_installdirs
	install -m0755 -d debian/gcpegg
install -m0755 -d debian/gcpegg/etc debian/gcpegg/usr/share/man/man1 debian/gcpegg/usr/sbin debian/gcpegg/usr/share/lintian/overrides debian/gcpegg/var/spool/gcpegg
/usr/bin/install -o root -g root -m 0755 eggsh debian/gcpegg/usr/sbin
/usr/bin/install: cannot change ownership of 'debian/gcpegg/usr/sbin/eggsh': Operation not permitted
make: *** [debian/rules:26: install-stamp] Error 1
dpkg-buildpackage: error: debian/rules binary subprocess returned exit status 2
--------------------------------------------------------------------------------
Build finished at 2024-11-16T06:53:58Z

-------------------------------------------------------------------------------


The above is just how the build ends and not necessarily the most
relevant part. If required, the full build log is available here:

https://people.debian.org/~nthykier/rrr-no-as-default/logs/987671.gz

You can find common solutions at
https://people.debian.org/~nthykier/rrr-no-as-default/docs/solutions.md

If this is really a bug in one of the build-depends, please use
reassign and affects, so that this is still visible in the BTS web
page for this package.

If this package is listed in
https://people.debian.org/~nthykier/rrr-no-as-default/docs/static-ownership.list,
then please just set `Rules-Requires-Root: binary-targets` to the source
stanza of `debian/control` as a fix to this bug.

If this package is listed in
https://people.debian.org/~nthykier/rrr-no-as-default/docs/maybe-misbuilds.list,
then the package was deemed at risk for misbuilding (having wrong
ownership) but had a FTBFS problem we tested it. Please test whether the
package works with `Rules-Requires-Root: no` validating that the
resulting deb has the correct ownership for all paths in the deb.

The goal is to have the default changed in `dpkg` either in `Trixie` or
`Forky`, depending on progress and feasibility with the release schedule
for Trixie.

For more information on this bug filing, please see:
https://lists.debian.org/debian-dpkg/2024/11/msg00016.html

Thanks,


PS: The builds were performed in mid-November. If you fixed the problem
between between then and this bug being filed, then please just close
the bug with the version it was fixed in.

Attachment: OpenPGP_signature.asc
Description: OpenPGP digital signature


Reply to: