• Bug#1066479: opendnssec: FTBFS: ../../common/scheduler/task.c:137:25: e

    From Lucas Nussbaum@21:1/5 to All on Wed Mar 13 13:20:07 2024
    Source: opendnssec
    Version: 1:2.1.13-1
    Severity: serious
    Justification: FTBFS
    Tags: trixie sid ftbfs
    User: lucas@debian.org
    Usertags: ftbfs-20240313 ftbfs-trixie ftbfs-impfuncdef

    Hi,

    During a rebuild of all packages in sid, your package failed to build
    on amd64.

    This is most likely caused by a change in dpkg 1.22.6, that enabled -Werror=implicit-function-declaration. For more information, see https://wiki.debian.org/qa.debian.org/FTBFS#A2024-03-13_-Werror.3Dimplicit-function-declaration

    Relevant part (hopefully):
    gcc -DHAVE_CONFIG_H -I. -I../../common -I/usr/include -Wdate-time -D_FORTIFY_SOURCE=2 -g -O2 -Werror=implicit-function-declaration -ffile-prefix-map=/<<PKGBUILDDIR>>=. -fstack-protector-strong -fstack-clash-protection -Wformat -Werror=format-security
    -fcf-protection -Wall -pedantic -DSQLITE3_SONAME=\"libsqlite3.so.0\" -pedantic -rdynamic -c -o scheduler/worker.o ../../common/scheduler/worker.c
    ../../common/cmdhandler.c: In function ‘cmdhandler_accept_client’: ../../common/cmdhandler.c:386:9: warning: unused variable ‘err’ [-Wunused-variable]
    386 | int err;
    | ^~~
    ../../common/cmdhandler.c: In function ‘cmdhandler_start’: ../../common/cmdhandler.c:509:13: warning: unused variable ‘i’ [-Wunused-variable]
    509 | ssize_t i;
    | ^
    ../../common/cmdhandler.c: In function ‘self_pipe_trick’: ../../common/cmdhandler.c:596:35: warning: zero-length gnu_printf format string [-Wformat-zero-length]
    596 | client_printf(sockfd, "");
    | ^~
    ../../common/cmdhandler.c: In function ‘cmdhandler_perform_command’: ../../common/cmdhandler.c:173:13: warning: ignoring return value of ‘asprintf’ declared with attribute ‘warn_unused_result’ [-Wunused-result]
    173 | asprintf(&statusstr, "unknown generic arguments");
    | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ../../common/cmdhandler.c:202:17: warning: ignoring return value of ‘asprintf’ declared with attribute ‘warn_unused_result’ [-Wunused-result]
    202 | asprintf(&statusstr, "memory error");
    | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ../../common/cmdhandler.c:209:21: warning: ignoring return value of ‘asprintf’ declared with attribute ‘warn_unused_result’ [-Wunused-result]
    209 | asprintf(&statusstr, "Error parsing arguments %s command line %s", command->name, arg);
    | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    ../../common/cmdhandler.c:232:25: warning: ignoring return value of ‘asprintf’ declared with attribute ‘warn_unused_result’ [-Wunused-result]
    232 | asprintf(&statusstr, "no help for command %s.", arg);
    | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    ../../common/cmdhandler.c:246:13: warning: ignoring return value of ‘asprintf’ declared with attribute ‘warn_unused_result’ [-Wunused-result]
    246 | asprintf(&statusstr, "Unknown command %s.", arg);
    | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ../../common/scheduler/schedule.c: In function ‘schedule_get_first_task’: ../../common/scheduler/schedule.c:81:20: warning: unused variable ‘node’ [-Wunused-variable]
    81 | ldns_rbnode_t* node = LDNS_RBTREE_NULL;
    | ^~~~
    ../../common/scheduler/task.c: In function ‘task_perform’: ../../common/scheduler/task.c:137:25: error: implicit declaration of function ‘clamp’ [-Werror=implicit-function-declaration]
    137 | task->backoff = clamp(task->backoff * 2, 60, ODS_SE_MAX_BACKOFF);
    | ^~~~~
    cc1: some warnings being treated as errors
    make[4]: *** [Makefile:601: scheduler/task.o] Error 1


    The full build log is available from: http://qa-logs.debian.net/2024/03/13/opendnssec_2.1.13-1_unstable.log

    All bugs filed during this archive rebuild are listed at: https://bugs.debian.org/cgi-bin/pkgreport.cgi?tag=ftbfs-20240313;users=lucas@debian.org
    or: https://udd.debian.org/bugs/?release=na&merged=ign&fnewerval=7&flastmodval=7&fusertag=only&fusertagtag=ftbfs-20240313&fusertaguser=lucas@debian.org&allbugs=1&cseverity=1&ctags=1&caffected=1#results

    A list of current common problems and possible solutions is available at http://wiki.debian.org/qa.debian.org/FTBFS . You're welcome to contribute!

    If you reassign this bug to another package, please mark it as 'affects'-ing this package. See https://www.debian.org/Bugs/server-control#affects

    If you fail to reproduce this, please provide a build log and diff it with mine so that we can identify if something relevant changed in the meantime.

    --- SoupGate-Win32 v1.05
    * Origin: fsxNet Usenet Gateway (21:1/5)
  • From Cyril Brulebois@21:1/5 to All on Tue Mar 26 14:50:01 2024
    --Qq62ARYUWrxEMDyg
    Content-Type: text/plain; charset=utf-8
    Content-Disposition: inline
    Content-Transfer-Encoding: quoted-printable

    Control: tag -1 patch pending

    Lucas Nussbaum <lucas@debian.org> (2024-03-13):
    This is most likely caused by a change in dpkg 1.22.6, that enabled -Werror=implicit-function-declaration. For more information, see https://wiki.debian.org/qa.debian.org/FTBFS#A2024-03-13_-Werror.3Dimplicit-function-declaration

    Relevant part (hopefully):
    ../../common/scheduler/task.c: In function ‘task_perform’: ../../common/scheduler/task.c:137:25: error: implicit declaration of function ‘clamp’ [-Werror=implicit-function-declaration]
    137 | task->backoff = clamp(task->backoff * 2, 60, ODS_SE_MAX_BACKOFF);
    | ^~~~~
    cc1: some warnings being treated as errors
    make[4]: *** [Makefile:601: scheduler/task.o] Error 1

    I thought there would be several things but apparently that's just the
    one. A quick look upstream shows there are more PRs and more fixups
    needed for even newer compilers, but I'm limiting my patch to the bare
    minimum.

    Since that's been open for 10+ days, and since reverse dependencies
    could get kicked out of testing, I'm uploading an NMU right now so that
    I don't forget, but to DELAYED/2 so there's some room to do things
    differently if desired. I'm happy to reschedule/cancel if needed.


    Cheers,
    --
    Cyril Brulebois -- Debian Consultant @ DEBAMAX -- https://debamax.com/

    --Qq62ARYUWrxEMDyg
    Content-Type: text/x-diff; charset=us-ascii
    Content-Disposition: attachment; filename="opendnssec+ftbfs.diff" Content-Transfer-Encoding: quoted-printable

    diff -Nru opendnssec-2.1.13/debian/changelog opendnssec-2.1.13/debian/changelog --- opendnssec-2.1.13/debian/changelog 2023-09-22 17:22:55.000000000 +0200
    +++ opendnssec-2.1.13/debian/changelog 2024-03-26 14:27:44.000000000 +0100
    @@ -1,3 +1,11 @@
    +opendnssec (1:2.1.13-1.1) unstable; urgency=medium
    +
    + * Non-maintainer upload.
    + * Fix FTBFS due to missing utilities.h include for the clamp declaration
    + (Closes: #1066479): 0018-fix-missing-include.patch
    +
    + -- Cyril Brulebois <cyril@debamax.com> Tue, 26 Mar 2024 14:27:44 +0100
    +
    opendnssec (1:2.1.13-1) unstable; urgency=medium

    * New upstream version 2.1.13
    diff -Nru opendnssec-2.1.13/debian/patches/0018-fix-missing-include.patch opendnssec-2.1.13/debian/patches/0018-fix-missing-inc
  • From Debian Bug Tracking System@21:1/5 to All on Tue Mar 26 14:50:02 2024
    Processing control commands:

    tag -1 patch pending
    Bug #1066479 [src:opendnssec] opendnssec: FTBFS: ../../common/scheduler/task.c:137:25: error: implicit declaration of function ‘clamp’ [-Werror=implicit-function-declaration]
    Added tag(s) patch and pending.

    --
    1066479: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1066479
    Debian Bug Tracking System
    Contact owner@bugs.debian.org with problems

    --- SoupGate-Win32 v1.05
    * Origin: fsxNet Usenet Gateway (21:1/5)
  • From Debian Bug Tracking System@21:1/5 to All on Thu Mar 28 15:50:02 2024
    This is a multi-part message in MIME format...

    Your message dated Thu, 28 Mar 2024 14:43:58 +0000
    with message-id <E1rpqz0-00Bh54-Ll@fasolo.debian.org>
    and subject line Bug#1066479: fixed in opendnssec 1:2.1.13-1.1
    has caused the Debian Bug report #1066479,
    regarding opendnssec: FTBFS: ../../common/scheduler/task.c:137:25: error: implicit declaration of function ‘clamp’ [-Werror=implicit-function-declaration]
    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 owner@bugs.debian.org
    immediately.)


    --
    1066479: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1066479
    Debian Bug Tracking System
    Contact owner@bugs.debian.org with problems

    Received: (at submit) by bugs.debian.org; 13 Mar 2024 12:16:38 +0000 X-Spam-Checker-Version: SpamAssassin 3.4.6-bugs.debian.org_2005_01_02
    (2021-04-09) on buxtehude.debian.org
    X-Spam-Level:
    X-Spam-Status: No, score=-106.2 required=4.0 tests=BAYES_00,DKIMWL_WL_HIGH,
    DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,FROMDEVELOPER,
    SPF_HELO_NONE,SPF_NONE,T_SCC_BODY_TEXT_LINE,UNPARSEABLE_RELAY,
    USER_IN_DKIM_WELCOMELIST,USER_IN_DKIM_WHITELIST autolearn=ham
    autolearn_force=no version=3.4.6-bugs.debian.org_2005_01_02 X-Spam-Bayes: score:0.0000 Tokens: new, 27; hammy, 150; neutral, 202; spammy,
    0. spammytokens:
    hammytokens:0.000-+--Hx-spam-relays-external:sk:stravin,
    0.000-+--H*RT:sk:stravin, 0.000-+--Hx-spam-relays-external:311,
    0.000-+--H*RT:108, 0.000-+--H*RT:311
    Return-path: <lucas@debian.org>
    Received: from stravinsky.debian.org ([2001:41b8:2
  • From Mathieu Mirmont@21:1/5 to Cyril Brulebois on Mon Apr 15 23:30:01 2024
    On Tue, Mar 26, 2024 at 02:36:06PM +0100, Cyril Brulebois wrote:
    Control: tag -1 patch pending

    Lucas Nussbaum <lucas@debian.org> (2024-03-13):
    This is most likely caused by a change in dpkg 1.22.6, that enabled -Werror=implicit-function-declaration. For more information, see https://wiki.debian.org/qa.debian.org/FTBFS#A2024-03-13_-Werror.3Dimplicit-function-declaration

    Relevant part (hopefully):
    ../../common/scheduler/task.c: In function ‘task_perform’: ../../common/scheduler/task.c:137:25: error: implicit declaration of function ‘clamp’ [-Werror=implicit-function-declaration]
    137 | task->backoff = clamp(task->backoff * 2, 60, ODS_SE_MAX_BACKOFF);
    | ^~~~~
    cc1: some warnings being treated as errors
    make[4]: *** [Makefile:601: scheduler/task.o] Error 1

    I thought there would be several things but apparently that's just the
    one. A quick look upstream shows there are more PRs and more fixups
    needed for even newer compilers, but I'm limiting my patch to the bare minimum.

    Since that's been open for 10+ days, and since reverse dependencies
    could get kicked out of testing, I'm uploading an NMU right now so that
    I don't forget, but to DELAYED/2 so there's some room to do things differently if desired. I'm happy to reschedule/cancel if needed.

    Thanks for the NMU.

    I had a look at the new warnings and they all seem to be false
    positives. I'd rather not introduce unnecessary changes in the
    packaged version compared to upstream just to silence them. This
    missing header will do.

    --
    Mathieu Mirmont <mat@parad0x.org>

    -----BEGIN PGP SIGNATURE-----
    Version: GnuPG v1.4.12 (GNU/Linux)

    iQIcBAEBAgAGBQJmHZiTAAoJENaa1EWGzzIiD08P/003JBiCFPTRb+J0uOCtb/S5 FSsdd9c1JIz9SMCs+RdfAs5Ipvx0uTc0Z4kBzgYI8tz5gJXnIoOgljvhsSbqgE+v hV+Iy6GNOyRi3QKwih9xkw/nFnr6gZSd6+wq9GnRIyV/zwwB1BLLtrMLewduz1vn WFKaBMSKZOgC8A0jzszBawYgN5v1yXa+JJuTgHNzkKqdmm8k0AJ4LaTamEsR5Xhj IHTqLAdi/H1HCYYtp55wL0KgWon/Gs4FJSxIKGE1TiLB7yaJSf8VmEcHrvsbr3Bj w6ifGQ2okVo18rW4eMdfSlRh6kEr6dBzO7+RyFVAQAepRT0Tf7yOQznzHibRbgPS X6h4E1aX3utdEfHfG+kg55XrUscXf3TrRQWy8753HPxvfVAsJSkfVVVCiuSVsJ6a 371CcNb+6jt9/k78gJEP8v35MieYf6gLNQo6O07BBnoAR/BhwAISthTTDjWCKOmI Uq62Z+gc6lKplKXcM/UqXj0EgoveXqlmn6WHui5GmT1IQEkcGaCpyCfnz4Eqiim/ TOLbqtys+3eiQ1NJmmFV5kAdw2rIFH8wEpKhJkhoERA4hEjxIYpBSGsXbJEGpinm /jCfj34tGIVQwBTph342MgHlilfR7HEzNhAkgGjYIM/5ajhuqS6m/wd6JWdjn85W NKZI6h2cKwTpCsecAu2k
    =s/kT
    -----END PGP SIGNATURE-----

    --- SoupGate-Win32 v1.05
    * Origin: fsxNet Usenet Gateway (21:1/5)