Your message dated Sun, 08 Dec 2024 22:59:12 +0000
with message-id <e1tkqf6-00fg2m...@fasolo.debian.org>
and subject line Bug#1089417: fixed in pptp-linux 1.10.0-2
has caused the Debian Bug report #1089417,
regarding pptp-linux: Supporting rootless builds by default
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact ow...@bugs.debian.org
immediately.)


-- 
1089417: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1089417
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems
--- Begin Message ---
Source: pptp-linux
Version: 1.10.0-1
Severity: important
Tags: ftbfs
Justification: FTBFS
X-Debbugs-Cc: ni...@thykier.net
User: ni...@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`,
pptp-linux failed to rebuild.

Log Summary:
-------------------------------------------------------------------------------
[...]
gcc -g -O2 -Werror=implicit-function-declaration -ffile-prefix-map=/<<PKGBUILDDIR>>=. -fstack-protector-strong -fstack-clash-protection -Wformat -Werror=format-security -mbranch-protection=standard -Wdate-time -D_FORTIFY_SOURCE=2 -c -o orckit_quirks.o orckit_quirks.c
util.c: In function ‘_log’:
util.c:29:34: warning: ‘%s’ directive output may be truncated writing up to 255 bytes into a region of size 245 [-Wformat-truncation=]
   29 | snprintf(string, sizeof(string), "%s %s[%s:%s:%d]: %s", \
      |                                  ^~~~~~~~~~~~~~~~~~~~~
   30 |          log_string, label, func, file, line, buf);     \
      |                                               ~~~
util.c:47:5: note: in expansion of macro ‘MAKE_STRING’
   47 |     MAKE_STRING("log");
      |     ^~~~~~~~~~~
util.c:29:52: note: format string is defined here
   29 | snprintf(string, sizeof(string), "%s %s[%s:%s:%d]: %s", \
      |                                                    ^~
In file included from /usr/include/stdio.h:970,
                 from util.c:7:
In function ‘snprintf’,
    inlined from ‘_log’ at util.c:47:5:
/usr/include/aarch64-linux-gnu/bits/stdio2.h:68:10: note: ‘__builtin___snprintf_chk’ output 12 or more bytes (assuming 267) into a destination of size 256 68 | return __builtin___snprintf_chk (__s, __n, __USE_FORTIFY_LEVEL - 1, | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   69 |                                    __glibc_objsize (__s), __fmt,
      |                                    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   70 |                                    __va_arg_pack ());
      |                                    ~~~~~~~~~~~~~~~~~
util.c: In function ‘_warn’:
util.c:29:34: warning: ‘%s’ directive output may be truncated writing up to 255 bytes into a region of size 244 [-Wformat-truncation=]
   29 | snprintf(string, sizeof(string), "%s %s[%s:%s:%d]: %s", \
      |                                  ^~~~~~~~~~~~~~~~~~~~~
   30 |          log_string, label, func, file, line, buf);     \
      |                                               ~~~
util.c:54:5: note: in expansion of macro ‘MAKE_STRING’
   54 |     MAKE_STRING("warn");
      |     ^~~~~~~~~~~
util.c:29:52: note: format string is defined here
   29 | snprintf(string, sizeof(string), "%s %s[%s:%s:%d]: %s", \
      |                                                    ^~
In function ‘snprintf’,
    inlined from ‘_warn’ at util.c:54:5:
/usr/include/aarch64-linux-gnu/bits/stdio2.h:68:10: note: ‘__builtin___snprintf_chk’ output 13 or more bytes (assuming 268) into a destination of size 256 68 | return __builtin___snprintf_chk (__s, __n, __USE_FORTIFY_LEVEL - 1, | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   69 |                                    __glibc_objsize (__s), __fmt,
      |                                    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   70 |                                    __va_arg_pack ());
      |                                    ~~~~~~~~~~~~~~~~~
pptp_ctrl.c: In function ‘randci’:
pptp_ctrl.c:336:17: warning: ignoring return value of ‘read’ declared with attribute ‘warn_unused_result’ [-Wunused-result]
  336 |                 read(fd, &i, 2);
      |                 ^~~~~~~~~~~~~~~
gcc -g -O2 -Werror=implicit-function-declaration -ffile-prefix-map=/<<PKGBUILDDIR>>=. -fstack-protector-strong -fstack-clash-protection -Wformat -Werror=format-security -mbranch-protection=standard -Wdate-time -D_FORTIFY_SOURCE=2 -c -o pqueue.o pqueue.c
util.c: In function ‘_fatal’:
util.c:29:34: warning: ‘%s’ directive output may be truncated writing up to 255 bytes into a region of size 243 [-Wformat-truncation=]
   29 | snprintf(string, sizeof(string), "%s %s[%s:%s:%d]: %s", \
      |                                  ^~~~~~~~~~~~~~~~~~~~~
   30 |          log_string, label, func, file, line, buf);     \
      |                                               ~~~
util.c:62:5: note: in expansion of macro ‘MAKE_STRING’
   62 |     MAKE_STRING("fatal");
      |     ^~~~~~~~~~~
util.c:29:52: note: format string is defined here
   29 | snprintf(string, sizeof(string), "%s %s[%s:%s:%d]: %s", \
      |                                                    ^~
In function ‘snprintf’,
    inlined from ‘_fatal’ at util.c:62:5:
/usr/include/aarch64-linux-gnu/bits/stdio2.h:68:10: note: ‘__builtin___snprintf_chk’ output 14 or more bytes (assuming 269) into a destination of size 256 68 | return __builtin___snprintf_chk (__s, __n, __USE_FORTIFY_LEVEL - 1, | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   69 |                                    __glibc_objsize (__s), __fmt,
      |                                    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   70 |                                    __va_arg_pack ());
      |                                    ~~~~~~~~~~~~~~~~~
gcc -g -O2 -Werror=implicit-function-declaration -ffile-prefix-map=/<<PKGBUILDDIR>>=. -fstack-protector-strong -fstack-clash-protection -Wformat -Werror=format-security -mbranch-protection=standard -Wdate-time -D_FORTIFY_SOURCE=2 -c -o pptp_callmgr.o pptp_callmgr.c gcc -g -O2 -Werror=implicit-function-declaration -ffile-prefix-map=/<<PKGBUILDDIR>>=. -fstack-protector-strong -fstack-clash-protection -Wformat -Werror=format-security -mbranch-protection=standard -Wdate-time -D_FORTIFY_SOURCE=2 -c -o routing.o routing.c gcc -g -O2 -Werror=implicit-function-declaration -ffile-prefix-map=/<<PKGBUILDDIR>>=. -fstack-protector-strong -fstack-clash-protection -Wformat -Werror=format-security -mbranch-protection=standard -Wdate-time -D_FORTIFY_SOURCE=2 -c -o pptp_compat.o pptp_compat.c
routing.c: In function ‘routing_init’:
routing.c:121:3: warning: ignoring return value of ‘fgets’ declared with attribute ‘warn_unused_result’ [-Wunused-result]
  121 |   fgets(buf, 255, p);
      |   ^~~~~~~~~~~~~~~~~~
pod2man --section=8 pptpsetup > pptpsetup.8
gcc -g -O2 -Werror=implicit-function-declaration -ffile-prefix-map=/<<PKGBUILDDIR>>=. -fstack-protector-strong -fstack-clash-protection -Wformat -Werror=format-security -mbranch-protection=standard -Wdate-time -D_FORTIFY_SOURCE=2 -c -o pptp.o pptp.c
pptp_callmgr.c: In function ‘call_callback’:
pptp_callmgr.c:76:13: warning: ignoring return value of ‘write’ declared with attribute ‘warn_unused_result’ [-Wunused-result]
   76 |             write(lci->unix_sock, &call_id, sizeof(call_id));
      |             ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
pptp_callmgr.c: In function ‘callmgr_main’:
pptp_callmgr.c:225:13: warning: ignoring return value of ‘read’ declared with attribute ‘warn_unused_result’ [-Wunused-result]
  225 |             read(s, &lci->pid[0], sizeof(lci->pid[0]));
      |             ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
pptp_callmgr.c:226:13: warning: ignoring return value of ‘read’ declared with attribute ‘warn_unused_result’ [-Wunused-result]
  226 |             read(s, &lci->pid[1], sizeof(lci->pid[1]));
      |             ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
gcc -o pptp pptp.o pptp_gre.o ppp_fcs.o pptp_ctrl.o dirutil.o vector.o util.o version.o test-redirections.o pptp_quirks.o orckit_quirks.o pqueue.o pptp_callmgr.o routing.o pptp_compat.o -Wl,-z,relro -Wl,-z,now -lutil
make[1]: Leaving directory '/<<PKGBUILDDIR>>'
   dh_auto_test
        make -j8 test
make[1]: Entering directory '/<<PKGBUILDDIR>>'
gcc -g -O2 -Werror=implicit-function-declaration -ffile-prefix-map=/<<PKGBUILDDIR>>=. -fstack-protector-strong -fstack-clash-protection -Wformat -Werror=format-security -mbranch-protection=standard -Wdate-time -D_FORTIFY_SOURCE=2 -c -o vector_test.o vector_test.c
gcc -o vector_test vector_test.o vector.o
./vector_test
make[1]: Leaving directory '/<<PKGBUILDDIR>>'
   create-stamp debian/debhelper-build-stamp
   dh_prep
   dh_auto_install --destdir=debian/pptp-linux/
make -j8 install DESTDIR=/<<PKGBUILDDIR>>/debian/pptp-linux AM_UPDATE_INFO_DIR=no
make[1]: Entering directory '/<<PKGBUILDDIR>>'
mkdir -p /<<PKGBUILDDIR>>/debian/pptp-linux/usr/sbin
install -o root -m 555 pptp /<<PKGBUILDDIR>>/debian/pptp-linux/usr/sbin
install: cannot change ownership of '/<<PKGBUILDDIR>>/debian/pptp-linux/usr/sbin/pptp': Operation not permitted
make[1]: *** [Makefile:75: install] Error 1
make[1]: Leaving directory '/<<PKGBUILDDIR>>'
dh_auto_install: error: make -j8 install DESTDIR=/<<PKGBUILDDIR>>/debian/pptp-linux AM_UPDATE_INFO_DIR=no returned exit code 2
make: *** [debian/rules:6: binary] Error 25
dpkg-buildpackage: error: debian/rules binary subprocess returned exit status 2
--------------------------------------------------------------------------------
Build finished at 2024-11-17T21:29:45Z

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


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/1029897.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


--- End Message ---
--- Begin Message ---
Source: pptp-linux
Source-Version: 1.10.0-2
Done: Christoph Biedl <debian.a...@manchmal.in-ulm.de>

We believe that the bug you reported is fixed in the latest version of
pptp-linux, which is due to be installed in the Debian FTP archive.

A summary of the changes between this version and the previous one is
attached.

Thank you for reporting the bug, which will now be closed.  If you
have further comments please address them to 1089...@bugs.debian.org,
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Christoph Biedl <debian.a...@manchmal.in-ulm.de> (supplier of updated 
pptp-linux package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing ftpmas...@ftp-master.debian.org)


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA512

Format: 1.8
Date: Sun, 08 Dec 2024 20:25:52 +0100
Source: pptp-linux
Architecture: source
Version: 1.10.0-2
Distribution: unstable
Urgency: medium
Maintainer: Christoph Biedl <debian.a...@manchmal.in-ulm.de>
Changed-By: Christoph Biedl <debian.a...@manchmal.in-ulm.de>
Closes: 1082224 1089417
Changes:
 pptp-linux (1.10.0-2) unstable; urgency=medium
 .
   * Support rootless build. Closes: #1089417
   * Cherry-pick some changes from upstream development.
     Closes: #1082224
Checksums-Sha1:
 2e1dab82c3e21e6fa5e30bb0d075af2b38097d25 2017 pptp-linux_1.10.0-2.dsc
 663c411ab811f2c93a7a9d3e84c4c78634908486 9432 pptp-linux_1.10.0-2.debian.tar.xz
 eb8a5a3d30cbc50a91c86c9c95515206b7510f89 5911 
pptp-linux_1.10.0-2_armel.buildinfo
Checksums-Sha256:
 09c25239f6b11a56710945267c7df239dc0c9e3b7239f4a0ca04f7b022469f79 2017 
pptp-linux_1.10.0-2.dsc
 17e8253a5779912ceda2be1177ea5243a5a57076b37a2305fa737e16a935c2a3 9432 
pptp-linux_1.10.0-2.debian.tar.xz
 4e1f36f9eb70ac418a75955d6cb82fd679404ab370ae723e7ab7be3d416afd9a 5911 
pptp-linux_1.10.0-2_armel.buildinfo
Files:
 c740eab54a8f26524c4305728ccf524f 2017 net optional pptp-linux_1.10.0-2.dsc
 766411a26515f1688cf6e8b6a9ff2a22 9432 net optional 
pptp-linux_1.10.0-2.debian.tar.xz
 41f6a97d2f6030a43637aa8e3d8698a3 5911 net optional 
pptp-linux_1.10.0-2_armel.buildinfo

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

iQIzBAEBCgAdFiEEWXMI+726A12MfJXdxCxY61kUkv0FAmdV91EACgkQxCxY61kU
kv2sYBAAs+aYe/AU3ue6l+7684tcuxfMO91nM2120DZm0eez5wtcfyK6TwKjdDph
LXlGXDaF/JXMhkI9RbVwNHYstNQVbAtLIdRS7LZFg5RM6TA0ovK+0z42V7VoC/Np
VlmBQG53M+0dhUee1/Ppr7jEFhyj+hqMp9f/jYtjHCQwZWEdj2Z9YlUka+gtTpdC
vI0CT0hfBvCi7omndQC+TeqlrX9HY1IxW/kPgsFHR6MZV8s+Z4KcRYQH0HkyDrkE
2Lb3UCL6saL6HeSgEu66NUFO2urKsYH4PZy5wGqmjoiwEAyYXFXQ7xZuApzjNU+3
cdQWdG4o+T6VGQg09ToxWfzaCGW0Qt/Jnrh66hLT6vw9g9k29eaNq4V1xrHwuf+P
vEIQz9G7PUcwZtsbam6n+kPHLF22udPnqSd1Yb2C4eZrvVT4A1xNiXdv3ye8g4GI
9oqe0EjyUmcLef3MIqWb14h0pBf/hXjFpJTQ+EmJolGHVRrnbUdfvQkYi8WQSd3D
kYjJMVOgsbeXGA9UmEtDTUReV5IVLKKQzbFv5ImfAGHG5lbOFpTRbtBbDaoIMHNE
ye1bYLUwe/r/rXLxNuBtTvuLBrElfhGp9ftTVYmZpG3nP6susrteB12Y6uTkcwVs
CGzFpNv+4EqfTJEsrMC87inQUqDnpXqpSG4g70O/ro/d91k6wW4=
=6llR
-----END PGP SIGNATURE-----

Attachment: pgpPmelzlVGyN.pgp
Description: PGP signature


--- End Message ---

Reply via email to