Add APPLELINK_NO_CURRENT_VERSION and APPLELINK_NO_COMPATIBILITY_VERSION to allow user to disable generating either or both -current_version and/or -compatibility_version to the applelink linker. Updates to docs. Add docstrings to generator functions

2 files changed, 68 insertions, 8 deletions

diff --git a/src/engine/SCons/Tool/applelink.py b/src/engine/SCons/Tool/applelink.py
index af854b7..efa7ee1 100644
--- a/src/engine/SCons/Tool/applelink.py
+++ b/src/engine/SCons/Tool/applelink.py
@@ -110,10 +110,20 @@ def _applelib_check_valid_version(version_string):
 
 def _applelib_currentVersionFromSoVersion(source, target, env, for_signature):
     """
-    -Wl,current_version=2.3
+    A generator function to create the -Wl,-current_version flag if needed.
+    If env['APPLELINK_NO_CURRENT_VERSION'] contains a true value no flag will be generated
+    Otherwise if APPLELINK_CURRENT_VERSION is not specified, env['SHLIBVERSION']
+    will be used.
+
+    :param source:
+    :param target:
+    :param env:
+    :param for_signature:
+    :return: A string providing the flag to specify the current_version of the shared library
     """
-
-    if env.get('APPLELINK_CURRENT_VERSION', False):
+    if env.get('APPLELINK_NO_CURRENT_VERSION', False):
+        return ""
+    elif env.get('APPLELINK_CURRENT_VERSION', False):
         version_string = env['APPLELINK_CURRENT_VERSION']
     elif env.get('SHLIBVERSION', False):
         version_string = env['SHLIBVERSION']
@@ -131,16 +141,29 @@ def _applelib_currentVersionFromSoVersion(source, target, env, for_signature):
 
 def _applelib_compatVersionFromSoVersion(source, target, env, for_signature):
     """
-    -Wl,compat_version=2.0
+    A generator function to create the -Wl,-compatibility_version flag if needed.
+    If env['APPLELINK_NO_COMPATIBILITY_VERSION'] contains a true value no flag will be generated
+    Otherwise if APPLELINK_COMPATIBILITY_VERSION is not specified
+    the first two parts of env['SHLIBVERSION'] will be used with a .0 appended.
+
+    :param source:
+    :param target:
+    :param env:
+    :param for_signature:
+    :return: A string providing the flag to specify the compatibility_version of the shared library
     """
-
-    if env.get('APPLELINK_COMPATIBILITY_VERSION', False):
+    if env.get('APPLELINK_NO_COMPATIBILITY_VERSION', False):
+        return ""
+    elif env.get('APPLELINK_COMPATIBILITY_VERSION', False):
         version_string = env['APPLELINK_COMPATIBILITY_VERSION']
     elif env.get('SHLIBVERSION', False):
         version_string = ".".join(env['SHLIBVERSION'].split('.')[:2] + ['0'])
     else:
         return ""
 
+    if version_string is None:
+        return ""
+
     valid, reason = _applelib_check_valid_version(version_string)
     if not valid:
         raise AppleLinkInvalidCompatibilityVersionException(reason)
@@ -181,7 +204,6 @@ def generate(env):
     env['LDMODULEFLAGS'] = SCons.Util.CLVar('$LINKFLAGS -bundle')
     env['LDMODULECOM'] = '$LDMODULE -o ${TARGET} $LDMODULEFLAGS $SOURCES $_LIBDIRFLAGS $_LIBFLAGS $_FRAMEWORKPATH $_FRAMEWORKS $FRAMEWORKSFLAGS'
 
-    #
     env['__SHLIBVERSIONFLAGS'] = '${__libversionflags(__env__,"SHLIBVERSION","_SHLIBVERSIONFLAGS")}'
 
 
diff --git a/src/engine/SCons/Tool/applelink.xml b/src/engine/SCons/Tool/applelink.xml
index c3346db..ca76aee 100644
--- a/src/engine/SCons/Tool/applelink.xml
+++ b/src/engine/SCons/Tool/applelink.xml
@@ -43,6 +43,9 @@ See its __doc__ string for a discussion of the format.
             <item>LDMODULECOM</item>
             <item>APPLELINK_CURRENT_VERSION</item>
             <item>APPLELINK_COMPATIBILITY_VERSION</item>
+            <item>APPLELINK_NO_CURRENT_VERSION</item>
+            <item>APPLELINK_NO_COMPATIBILITY_VERSION</item>
+
         </sets>
         <uses>
             <item>FRAMEWORKSFLAGS</item>
@@ -59,13 +62,30 @@ See its __doc__ string for a discussion of the format.
             </para>
             <para>
                 The value is specified as X[.Y[.Z]] where X is between 1 and 65535, Y can be omitted or between 1 and
-                255, Z can be omitted or between 1 and 255. This value will be derived from &cv-link-SHLIBVERSION; if not
+                255, Z can be omitted or between 1 and 255. This value will be derived from &cv-link-SHLIBVERSION; if
+                not
                 specified. The lowest digit will be dropped and replaced by a 0.
             </para>
+            <para>
+                If the &cv-link-APPLELINK_NO_COMPATIBILITY_VERSION; is set then no -compatibility_version will be
+                output.
+            </para>
             <para>See MacOS's ld manpage for more details</para>
         </summary>
     </cvar>
 
+    <cvar name="APPLELINK_NO_COMPATIBILITY_VERSION">
+        <summary>
+            <para>
+                Set this to any True (1|True|non-empty string) value to disable adding -compatibility_version flag when
+                generating versioned shared libraries.
+            </para>
+            <para>
+                This overrides &cv-link-APPLELINK_COMPATIBILITY_VERSION;.
+            </para>
+        </summary>
+    </cvar>
+
 
     <cvar name="APPLELINK_CURRENT_VERSION">
         <summary>
@@ -79,11 +99,29 @@ See its __doc__ string for a discussion of the format.
                 255, Z can be omitted or between 1 and 255. This value will be set to &cv-link-SHLIBVERSION; if not
                 specified.
             </para>
+            <para>
+                If the &cv-link-APPLELINK_NO_CURRENT_VERSION; is set then no -current_version will be
+                output.
+            </para>
             <para>See MacOS's ld manpage for more details</para>
 
         </summary>
     </cvar>
 
+
+    <cvar name="APPLELINK_CURRENT_VERSION">
+        <summary>
+            <para>
+                Set this to any True (1|True|non-empty string) value to disable adding -current_version flag when
+                generating versioned shared libraries.
+            </para>
+            <para>
+                This overrides &cv-link-APPLELINK_CURRENT_VERSION;.
+            </para>
+        </summary>
+    </cvar>
+
+
     <cvar name="FRAMEWORKSFLAGS">">
         <summary>
             <para>