After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 642984 - Drop --enable-ssl configure option (nss/nspr always required)
Drop --enable-ssl configure option (nss/nspr always required)
Status: RESOLVED FIXED
Product: evolution-data-server
Classification: Platform
Component: Mailer
2.32.x (obsolete)
Other Linux
: Normal normal
: ---
Assigned To: evolution-mail-maintainers
Evolution QA team
: 656413 665416 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2011-02-22 19:09 UTC by Pacho Ramos
Modified: 2011-12-02 17:37 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
build.log (27.77 KB, text/plain)
2011-02-22 19:09 UTC, Pacho Ramos
Details

Description Pacho Ramos 2011-02-22 19:09:53 UTC
Created attachment 181619 [details]
build.log

As reported downstream at:
http://bugs.gentoo.org/show_bug.cgi?id=356107

e-d-s fails to build when ssl support is disabled in a similar way of old bug 625039


  CC     libcamel_1_2_la-camel-utf8.lo
  CC     libcamel_1_2_la-camel.lo
  CC     libcamel_1_2_la-camel-lock-client.lo
In file included from camel.h:124,
                 from camel.c:40:
../camel/camel-tcp-stream.h:47:18: error: prio.h: No such file or directory
In file included from camel.h:124,
                 from camel.c:40:
../camel/camel-tcp-stream.h:141: error: expected specifier-qualifier-list before 'PRFileDesc'
../camel/camel-tcp-stream.h:152: error: expected '=', ',', ';', 'asm' or '__attribute__' before '*' token
In file included from camel.h:125,
                 from camel.c:40:
../camel/camel-tcp-stream-raw.h:90: error: expected declaration specifiers or '...' before 'PRFileDesc'
make[4]: *** [libcamel_1_2_la-camel.lo] Error 1
make[4]: *** Waiting for unfinished jobs....
camel-lock-client.c: In function 'lock_helper_init':
camel-lock-client.c:135: warning: ignoring return value of 'dup', declared with attribute warn_unused_result
camel-lock-client.c:137: warning: ignoring return value of 'dup', declared with attribute warn_unused_result
make[4]: Leaving directory `/var/tmp/portage/gnome-extra/evolution-data-server-2.32.2/work/evolution-data-server-2.32.2/camel'
make[3]: *** [all-recursive] Error 1
make[3]: Leaving directory `/var/tmp/portage/gnome-extra/evolution-data-server-2.32.2/work/evolution-data-server-2.32.2/camel'
make[2]: *** [all] Error 2
make[2]: Leaving directory `/var/tmp/portage/gnome-extra/evolution-data-server-2.32.2/work/evolution-data-server-2.32.2/camel'
make[1]: *** [all-recursive] Error 1
make[1]: Leaving directory `/var/tmp/portage/gnome-extra/evolution-data-server-2.32.2/work/evolution-data-server-2.32.2'
make: *** [all] Error 2
emake failed

Thanks for your help
Comment 1 Milan Crha 2011-03-02 16:03:00 UTC
Do you really want to disable ssl support? Quite many things depend on that, together with certificate manipulation parts and similar using nss/nspr.
Comment 2 Pacho Ramos 2011-03-02 18:15:12 UTC
I compile it under Gentoo with "ssl" support... but as configure allows to disable it, we allow our users to disable it. Then, either fix building without ssl support or make it not choosable

Thanks :-)
Comment 3 Milan Crha 2011-03-03 07:38:49 UTC
I would prefer to drop --disable-ssl then, as ssl is a good thing to have. OK?
Comment 4 Pacho Ramos 2011-03-03 08:21:53 UTC
From my point of view I don't have any problems with forcing ssl
Comment 5 Matthew Barnes 2011-03-03 12:09:49 UTC
We've had to do some strange backflips to get E-D-S and Evolution to build correctly when SSL is disabled, so I agree with Milan about dropping the option.  Especially since I'd like to make using secure network connections the default in 3.x.  But there's no urgency to this, so I would prefer if we wait and do it for 3.1.  We're a bit too close to 3.0 code freeze right now.
Comment 6 Milan Crha 2011-03-03 15:23:57 UTC
OK, sounds good, just please remind me of this when the time comes, as I'll forget of it for sure :)
Comment 7 Milan Crha 2011-08-15 13:51:35 UTC
*** Bug 656413 has been marked as a duplicate of this bug. ***
Comment 8 Milan Crha 2011-09-26 19:45:32 UTC
I made this finally in, the nss/nspr are required now:

Created commit b50ab07 in eds master (3.3.1+)
Comment 9 Matthew Barnes 2011-12-02 17:37:20 UTC
*** Bug 665416 has been marked as a duplicate of this bug. ***