Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 662944 - >=net-mail/cyrus-imapd-3.0.4 [+sphinx]: imap/cyr_sphinxmgr.c:808:27: error: storage size of ‘parts’ isn’t known
Summary: >=net-mail/cyrus-imapd-3.0.4 [+sphinx]: imap/cyr_sphinxmgr.c:808:27: error: s...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: PPC Linux
: Normal normal
Assignee: No maintainer - Look at https://wiki.gentoo.org/wiki/Project:Proxy_Maintainers if you want to take care of it
URL: https://github.com/cyrusimap/cyrus-im...
Whiteboard:
Keywords:
Depends on:
Blocks: 671576
  Show dependency tree
 
Reported: 2018-08-06 13:33 UTC by ernsteiswuerfel
Modified: 2019-06-09 11:40 UTC (History)
3 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
build.log (cyrus-imapd-3.0.5:20180806-130506.log,134.09 KB, text/plain)
2018-08-06 13:33 UTC, ernsteiswuerfel
Details
emerge --info (file_662944.txt,5.39 KB, text/plain)
2018-08-06 13:34 UTC, ernsteiswuerfel
Details
build.log (3.0.9, ppc) (cyrus-imapd-3.0.9:20190402-150539.log,166.31 KB, text/plain)
2019-04-02 15:11 UTC, ernsteiswuerfel
Details
emerge --info (file_662944.txt,5.92 KB, text/plain)
2019-04-02 15:12 UTC, ernsteiswuerfel
Details

Note You need to log in before you can comment on or make changes to this bug.
Description ernsteiswuerfel archtester 2018-08-06 13:33:38 UTC
Created attachment 542540 [details]
build.log

imap/cyr_sphinxmgr.c: In function ‘indexd_start’:
imap/cyr_sphinxmgr.c:752:5: warning: implicit declaration of function ‘assert’; did you mean ‘qsort’? [-Wimplicit-function-declaration]
     assert(id->state == STOPPED);
     ^~~~~~
     qsort
imap/cyr_sphinxmgr.c: In function ‘sphinx_paths_from_mboxname’:
imap/cyr_sphinxmgr.c:808:27: error: storage size of ‘parts’ isn’t known
     struct mboxname_parts parts;
                           ^~~~~
imap/cyr_sphinxmgr.c:813:5: warning: implicit declaration of function ‘mboxname_init_parts’; did you mean ‘mboxname_metapath’? [-Wimplicit-function-declaration]
     mboxname_init_parts(&parts);
     ^~~~~~~~~~~~~~~~~~~
     mboxname_metapath
imap/cyr_sphinxmgr.c:829:9: warning: implicit declaration of function ‘mboxname_to_parts’; did you mean ‘mboxname_lockpath’? [-Wimplicit-function-declaration]
     r = mboxname_to_parts(mboxname, &parts);
         ^~~~~~~~~~~~~~~~~
         mboxname_lockpath
imap/cyr_sphinxmgr.c:876:5: warning: implicit declaration of function ‘mboxname_free_parts’; did you mean ‘mboxname_make_parent’? [-Wimplicit-function-declaration]
     mboxname_free_parts(&parts);
     ^~~~~~~~~~~~~~~~~~~
     mboxname_make_parent
imap/cyr_sphinxmgr.c: In function ‘create_server_socket’:
imap/cyr_sphinxmgr.c:1078:45: error: ‘IMAPOPT_SPHINXMGR_SOCKET’ undeclared (first use in this function); did you mean ‘IMAPOPT_SPHINX_SOCKET’?
     const char *sockname = config_getstring(IMAPOPT_SPHINXMGR_SOCKET);
                                             ^~~~~~~~~~~~~~~~~~~~~~~~
                                             IMAPOPT_SPHINX_SOCKET
imap/cyr_sphinxmgr.c:1078:45: note: each undeclared identifier is reported only once for each function it appears in
imap/cyr_sphinxmgr.c: In function ‘main’:
imap/cyr_sphinxmgr.c:1306:36: error: ‘IMAPOPT_SPHINXMGR_TIMEOUT’ undeclared (first use in this function); did you mean ‘IMAPOPT_SYNC_TIMEOUT’?
     sphinx_timeout = config_getint(IMAPOPT_SPHINXMGR_TIMEOUT);
                                    ^~~~~~~~~~~~~~~~~~~~~~~~~
                                    IMAPOPT_SYNC_TIMEOUT
imap/cyr_sphinxmgr.c:1307:34: error: ‘IMAPOPT_SPHINXMGR_MAX_CHILDREN’ undeclared (first use in this function); did you mean ‘IMAPOPT_SPHINXMGR_TIMEOUT’?
     max_children = config_getint(IMAPOPT_SPHINXMGR_MAX_CHILDREN);
                                  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
                                  IMAPOPT_SPHINXMGR_TIMEOUT
make[2]: *** [Makefile:4557: imap/cyr_sphinxmgr.o] Error 1
Comment 1 ernsteiswuerfel archtester 2018-08-06 13:34:30 UTC
Created attachment 542542 [details]
emerge --info
Comment 2 ernsteiswuerfel archtester 2018-12-30 17:54:31 UTC
Happens on cyrus-imapd-3.0.4, -3.0.5, -3.0.6, -3.0.8.
Comment 3 Pacho Ramos gentoo-dev 2019-04-02 08:03:16 UTC
Is that fixed in 3.0.9?
Comment 4 ernsteiswuerfel archtester 2019-04-02 15:11:32 UTC
Created attachment 571580 [details]
build.log (3.0.9, ppc)

No, still around in 3.0.9.

[...]
imap/search_sphinx.c: In function ‘generate_query’:
imap/search_sphinx.c:432:9: warning: implicit declaration of function ‘assert’ [-Wimplicit-function-declaration]
         assert(on->children == NULL);
         ^~~~~~
imap/search_sphinx.c:432:9: note: ‘assert’ is defined in header ‘<assert.h>’; did you forget to ‘#include <assert.h>’?
imap/search_sphinx.c:75:1:
+#include <assert.h>
 
imap/search_sphinx.c:432:9:
         assert(on->children == NULL);
         ^~~~~~
imap/search_sphinx.c: In function ‘sphinx_basedir’:
imap/search_sphinx.c:783:27: error: storage size of ‘parts’ isn’t known
     struct mboxname_parts parts;
[...]
Comment 5 ernsteiswuerfel archtester 2019-04-02 15:12:26 UTC
Created attachment 571582 [details]
emerge --info
Comment 6 Pacho Ramos gentoo-dev 2019-04-19 16:59:46 UTC
Was this reported to upstream? I don't think they will fix it otherwise
https://github.com/cyrusimap/cyrus-imapd/issues

Thanks
Comment 7 ernsteiswuerfel archtester 2019-04-26 16:49:42 UTC
Took it upstream for ppc64, which is also affcted:
https://github.com/cyrusimap/cyrus-imapd/issues/2748
Comment 8 ernsteiswuerfel archtester 2019-05-13 11:47:42 UTC
Upstream says [+sphinx] is depreciated:

"Cyrus's support for the Sphinx search engine is unmaintained (and has already been removed from the master branch), and I don't think any of the current maintainers know much about it...

If you're able to reproduce this in a bisect and identify the commit where it stopped building, I might be able to produce a patch that will make it build. But, I have no idea if it even works when it does build...

I'd recommend using the Xapian search engine instead, if for no other reason than it's being actively maintained!"
Comment 9 Thomas Beutin 2019-05-13 11:54:35 UTC
(In reply to ernsteiswuerfel from comment #8)
> Upstream says [+sphinx] is depreciated:

> I'd recommend using the Xapian search engine instead, if for no other reason
> than it's being actively maintained!"

cyrusd+xapian has cjk issues[1] - but i don't know if this affects gentoo (the is no cjk USE flag for net-mail/cyrus-imapd)


[1] https://cyrusimap.org/imap/developer/install-xapian.html
Comment 10 Larry the Git Cow gentoo-dev 2019-06-09 11:40:05 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=f6d75bdfc33d380556d045c5610e402b8799065e

commit f6d75bdfc33d380556d045c5610e402b8799065e
Author:     Pacho Ramos <pacho@gentoo.org>
AuthorDate: 2019-06-09 11:38:22 +0000
Commit:     Pacho Ramos <pacho@gentoo.org>
CommitDate: 2019-06-09 11:39:57 +0000

    net-mail/cyrus-imapd: sphinx support is unmaintained and broken
    
    Closes: https://bugs.gentoo.org/662944
    Closes: https://bugs.gentoo.org/426044
    Package-Manager: Portage-2.3.67, Repoman-2.3.13
    Signed-off-by: Pacho Ramos <pacho@gentoo.org>

 net-mail/cyrus-imapd/cyrus-imapd-3.0.10-r1.ebuild | 224 ++++++++++++++++++++++
 1 file changed, 224 insertions(+)