Bug 225567 - mail/qpopper: does not compile with "--with-cyrus-sasl=/usr/local"; set depraceted and expiration_date
Summary: mail/qpopper: does not compile with "--with-cyrus-sasl=/usr/local"; set depra...
Status: Closed Overcome By Events
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Dmitry Marakasov
Depends on:
Reported: 2018-01-30 17:21 UTC by ixew
Modified: 2021-03-09 22:35 UTC (History)
3 users (show)

See Also:
bugzilla: maintainer-feedback? (bc979)

svn-diff-qpopper (978 bytes, patch)
2018-02-11 13:50 UTC, Walter Schwarzenfeld
no flags Details | Diff
svn-diff-expiration (417 bytes, patch)
2019-08-18 12:58 UTC, Walter Schwarzenfeld
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description ixew 2018-01-30 17:21:03 UTC
Dear maintainer of the mail/qpopper port:

Since the handbook recommends security/cyrus-sasl2 as the means for SMTP (sendmail) authentication I tried the same with mail/qpopper by adding "--with-cyrus-sasl=/usr/local" to the Makefile CONFIGURE_ARGS.

Unfortunately the port then does not compile, see below.
Is this an upstream bug or something to do with FreeBSD 11.1?
Is this the reason why this option is not offered via FreeBSD port "make config"?

It's nice to have the same authentication method in all mail operations.

Thanks in advance, Enoch.


In file included from mime.c:68:
../popper/popper.h:693:25: error: field has incomplete type 'struct _pop_sasl'
    struct _pop_sasl    sasl;                   /* General place to put 
../popper/popper.h:282:16: note: forward declaration of 'struct _pop_sasl'
typedef struct _pop_sasl            pop_sasl;           /* defined in pop_sasl.h */
mime.c:536:52: warning: implicit conversion from 'int' to 'char' changes value from 255 to -1 [-Wconstant-conversion]
  MemSet ( (char *) (mType->paramIndex), paramMax, kParamIndexUnused );
  ~~~~~~                                           ^~~~~~~~~~~~~~~~~
./mime.h:101:28: note: expanded from macro 'kParamIndexUnused'
#define kParamIndexUnused (255) /* don't change this */
1 warning and 1 error generated.
*** [mime.o] Error code 1
Comment 1 bc979 2018-02-11 00:50:10 UTC
I had no idea there was a cyrus-sasl option.  However, it is in the code, but not in the configuration parameters.  In config.h is a CYRUS_SASL define that needs to be defined.  It is by default undefined.  That "should" resolve the problem.  You should be able to manually include the #define CYRUS_SASL 1 in the Makefile, but I suspect going through configure might be required.

You do need to consider that qpopper is not being maintained upstream much anymore.  You might want to switch over to dovecot.  Dovecot does not have bulletins which was why I used qpopper.  However, that need went away and I switched to dovecot.
Comment 2 Walter Schwarzenfeld freebsd_triage 2018-02-11 13:50:12 UTC
Created attachment 190511 [details]

Please test this.  With the patch it builds fine. 
(If have not tested funcionallity).
Comment 3 bc979 2018-02-16 07:18:23 UTC
On FreeBSD 11.1 with base openssl it does not build properly.  There is an issue with openssl.  What system are you building it on, and what openssl are you using (base, port etc.)?
Comment 4 Walter Schwarzenfeld freebsd_triage 2018-02-16 11:18:13 UTC
Sorry, was 10.4 (I interrupted tests for some reasons) and after this I overlooked I have not tested on 
Comment 5 Walter Schwarzenfeld freebsd_triage 2019-08-18 12:58:19 UTC
Created attachment 206657 [details]
Comment 6 Walter Schwarzenfeld freebsd_triage 2019-08-18 13:02:11 UTC
Btw, builds with openssl ports and libressl (cannot test openssl base).
Comment 7 Dmitry Marakasov freebsd_committer 2021-02-18 20:20:55 UTC
This needs a resolution. If the port is still used, it should be adopted and tested with the sasl patch. Otherwise, we should mark it as DEPRECATED.
Comment 8 Dmitry Marakasov freebsd_committer 2021-03-09 22:35:07 UTC
Port marked as deprecated
Comment 9 commit-hook freebsd_committer 2021-03-09 22:35:29 UTC
A commit references this bug:

Author: amdmi3
Date: Tue Mar  9 22:34:55 UTC 2021
New revision: 567967
URL: https://svnweb.freebsd.org/changeset/ports/567967

  mail/qpopper: mark deprecated

  The port is unmaintained and upstream is dead since 2011

  PR:		225567