GNOME Bugzilla – Bug 271045
HAVE_NSS defined even when no development files installed, only libs
Last modified: 2013-09-14 16:48:40 UTC
configure.in finds the mozilla libs, but not the development files/headers. Still it defines HAVE_NSS Attached patch fixes that, and also fixes an autogen-time warning. As a suggestion, it's better to test "x$have_whatever" = "xyes" than "x$have_whatever" != "xno" to catch uninitialized variables. The nss checks could do with some cleanups.
Created attachment 44536 [details] [review] fix build
Created attachment 44537 [details] [review] newer patch for build
added a newer patch, which fixes the previous things, as well as: - make configure output what part it's looking CFLAGS and LIBS for, instead of EVOLUTION, which was confusing - fix some more tests, so that no CFLAGS get defined as "-I" (with missing dir), which caused the build to break further on also, it seems some autogenerated files can be removed from cvs, like INSTALL files and one generated flex file.
with this latest patch e-d-s builds all the way through without mozilla devel stuff. Now to rebuild with mozilla-devel :) I also added mozilla as a dep to jhbuild's setup.
Thomas: please send your patch to evolution-patches mailing list. You may need to subscribe. See http://lists.ximian.com
retargetting from 1.1 to 1.5 to get rid of that busted milestones (as discussed with harish on irc). patch (hint, hint)!
Andre, the patch has been right there ? Not sure what I need to do to get this in :/
retargetting to force a decision. also note that fejj has changed something in the codebase in the meantime, see http://cvs.gnome.org/viewcvs/evolution-data-server/configure.in?r1=1.167&r2=1.169 thomas, does this perhaps fix your issue?
thomas: *ping* does fejj's commit (see my last comment) fix this issue for you?
thomas: can we close this bug if you are not facing this issue any more? Thanks in Advance.
Thanks for taking the time to report this bug; however, closing due to lack of response of the reporter, sorry. if you still see this issue with a current release of evolution (2.24.x or later), please reopen. thanks in advance.