Wireshark · Wireshark-bugs: [Wireshark-bugs] [Bug 2398] New: openssl dependency and --with-ssl configure switch are redundant

[bugzilla-daemon@xxxxxxxxxxxxx]

http://bugs.wireshark.org/bugzilla/show_bug.cgi?id=2398

           Summary: openssl dependency and --with-ssl configure switch are
                    redundant
           Product: Wireshark
           Version: 1.0.0
          Platform: PC
        OS/Version: Linux
            Status: NEW
          Severity: Major
          Priority: Low
         Component: Wireshark
        AssignedTo: wireshark-bugs@xxxxxxxxxxxxx
        ReportedBy: pva@xxxxxxxxxx


Build Information:
This is build problem.
--
openssl check and --with-ssl configure switch are completely redundant. The
only place it is used is inside AC_WIRESHARK_KRB5_CHECK (acinclude.m4): if
kerberos library was built with -lcrypto then we require --with-ssl switch. But
of course if working kerberos installation exist on the system and depend on
-lcrypto then libcrypto is installed in system too, so no need to double check
that. Just drop --with-ssl configure switch and the following code in
(acinclude.m4):

            case "$KRB5_LIBS" in
            *-lcrypto*)
                if test ! -z "$SSL_LIBS"
                then
                    KRB5_LIBS=`echo $KRB5_LIBS | sed 's/-lcrypto//'`
                    KRB5_LIBS="$KRB5_LIBS $SSL_LIBS"
                else
                    AC_MSG_ERROR([Kerberos library requires -lcrypto but
--with-ssl not specified])
                fi
                ;;
            esac

Or... what do I miss here? What's the point in
                    KRB5_LIBS=`echo $KRB5_LIBS | sed 's/-lcrypto//'`
                    KRB5_LIBS="$KRB5_LIBS $SSL_LIBS"
?

Even if this code is required --with-ssl could be dropped and enabled instead
of AC_MSG_ERROR in the code sniplet above...


-- 
Configure bugmail: http://bugs.wireshark.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.