No -fno-common for Darwin
authorRichard Levitte <levitte@openssl.org>
Fri, 4 Mar 2016 12:48:59 +0000 (13:48 +0100)
committerRichard Levitte <levitte@openssl.org>
Sat, 5 Mar 2016 11:10:13 +0000 (12:10 +0100)
When object files with common block symbols are added to static
libraries on Darwin, those symbols are invisible to the linker that
tries to use them.  Our solution was to use -fno-common when compiling
C source.

Unfortunately, there is assembler code that defines OPENSSL_ia32cap_P
as a common block symbol, unconditionally, and in some cases, there is
no other definition.  -fno-common doesn't help in this case.

However, 'ranlib -c' adds common block symbols to the index of the
static library, which makes them visible to the linker using it, and
that solves the problem we've seen.

The common conclusion is, either use -fno-common or ranlib -c on
Darwin.  Since we have common block symbols unconditionally, choosing
the method for our source is easy.

Reviewed-by: Kurt Roeckx <kurt@openssl.org>
Configurations/10-main.conf

index 6fade9811c168d3de1d5150f69984ae12eb66dad..576ac45ce7ea35558820f8426f6ddfbacac6f59c 100644 (file)
@@ -1422,8 +1422,9 @@ sub combine {
         thread_scheme    => "pthreads",
         perlasm_scheme   => "osx32",
         dso_scheme       => "dlfcn",
+        ranlib           => "ranlib -c",
         shared_target    => "darwin-shared",
-        shared_cflag     => "-fPIC -fno-common",
+        shared_cflag     => "-fPIC",
         shared_ldflag    => "-dynamiclib",
         shared_extension => ".\$(SHLIB_MAJOR).\$(SHLIB_MINOR).dylib",
     },