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

Bug#749378: marked as done (citadel: Excess arguments in call to smtp_try_one_queue_entry)

Your message dated Mon, 29 Nov 2021 18:31:58 +0000
with message-id <E1mrlRW-00060b-7n@fasolo.debian.org>
and subject line Bug#996650: Removed package(s) from unstable
has caused the Debian Bug report #749378,
regarding citadel: Excess arguments in call to smtp_try_one_queue_entry
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.)


-- 
749378: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=749378
Debian Bug Tracking System
Contact owner@bugs.debian.org with problems
--- Begin Message --- 
Package: citadel
Version: 8.24-1
Severity: minor
Usertags: goto-cc

During a rebuild of all Debian packages in a clean sid chroot (using cowbuilder
and pbuilder) the build failed with the following error. Please note that we
use our research compiler tool-chain (using tools from the cbmc package), which
permits extended reporting on type inconsistencies at link time.

[...]
CC md5.c

error: conflicting function declarations "smtp_try_one_queue_entry"
old definition in module serv_smtpeventclient file modules/smtp/serv_smtpeventclient.c line 619
void (struct queueitem *MyQItem, struct _mailq_entry *MyQEntry, struct StrBuf *MsgText, signed int KeepMsgText, signed int MsgCount)
new definition in module serv_smtpqueue file modules/smtp/serv_smtpqueue.c line 115
void (struct queueitem *, struct _mailq_entry *, struct StrBuf *, signed int, signed int, struct ParsedURL *)
Makefile:142: recipe for target 'citserver' failed
make[1]: *** [citserver] Error 64

The misleading declaration may be reviewed here:

http://sources.debian.net/src/citadel/8.24-1/modules/smtp/serv_smtpqueue.c?hl=115#L115

Observe the extra argument RelayUrls (which is also passed along in calls),
compared to the actual definition:

http://sources.debian.net/src/citadel/8.24-1/modules/smtp/serv_smtpeventclient.c?hl=619#L619

This may hint at an incomplete refactoring and possibly other errors.

Best,
Michael

Attachment: pgpFybKMss81J.pgp
Description: PGP signature


--- End Message ---
--- Begin Message --- 
Version: 917-5+rm

Dear submitter,

as the package citadel has just been removed from the Debian archive
unstable we hereby close the associated bug reports.  We are sorry
that we couldn't deal with your issue properly.

For details on the removal, please see https://bugs.debian.org/996650

The version of this package that was in Debian prior to this removal
can still be found using http://snapshot.debian.org/.

Please note that the changes have been done on the master archive and
will not propagate to any mirrors until the next dinstall run at the
earliest.

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

Debian distribution maintenance software
pp.
Scott Kitterman (the ftpmaster behind the curtain)

--- End Message ---
Reply to: