Update conda & scons recipes for macOS

Author: vincefn

SConstruct

@@ -20,21 +20,24 @@ import os
 from os.path import join as pjoin
 import platform
 
+
 def subdictionary(d, keyset):
     return dict(kv for kv in d.items() if kv[0] in keyset)
 
+
 def getsyspaths(*names):
     pall = sum((os.environ.get(n, '').split(os.pathsep) for n in names), [])
     rv = [p for p in pall if os.path.exists(p)]
     return rv
 
+
 # copy system environment variables related to compilation
 DefaultEnvironment(ENV=subdictionary(os.environ, '''
     PATH CPATH CPLUS_INCLUDE_PATH LIBRARY_PATH LD_RUN_PATH
     LD_LIBRARY_PATH DYLD_LIBRARY_PATH DYLD_FALLBACK_LIBRARY_PATH
     MACOSX_DEPLOYMENT_TARGET
     '''.split())
-)
+                   )
 
 # Create construction environment
 env = DefaultEnvironment().Clone()
@@ -45,6 +48,8 @@ env.EnsureSConsVersion(0, 98, 1)
 # Customizable compile variables
 vars = Variables('sconsvars.py')
 
+# TODO: also amend paths when VIRTUAL_ENV variable exists,
+#  if CONDA_PREFIX does not exist ?
 if 'CONDA_PREFIX' in os.environ:
     # building for a conda environment
     vars.Add(PathVariable(
@@ -56,18 +61,18 @@ if 'CONDA_PREFIX' in os.environ:
         vars.Add(PathVariable(
             'libdir',
             'installation directory for compiled library [prefix/Library/lib]',
-            pjoin(env['prefix'],'Library', 'Lib'),
+            pjoin(env['prefix'], 'Library', 'Lib'),
             PathVariable.PathAccept))
         vars.Add(PathVariable(
             'includedir',
             'installation directory for C++ header files [prefix/Library/include]',
-            pjoin(env['prefix'],'Library', 'include'),
+            pjoin(env['prefix'], 'Library', 'include'),
             PathVariable.PathAccept))
     else:
         vars.Add(PathVariable(
             'libdir',
             'installation directory for compiled library [prefix/lib]',
-            pjoin(env['prefix'], 'Lib'),
+            pjoin(env['prefix'], 'lib'),
             PathVariable.PathAccept))
         vars.Add(PathVariable(
             'includedir',
@@ -111,22 +116,27 @@ env.Help(MY_SCONS_HELP % vars.GenerateHelpText(env))
 if platform.system().lower() == "windows":
     # See https://scons.org/faq.html#Linking_on_Windows_gives_me_an_error
     env['ENV']['TMP'] = os.environ['TMP']
-    # the CPPPATH directories are checked by scons dependency scanner
-    cpppath = getsyspaths('CPLUS_INCLUDE_PATH', 'CPATH')
-    env.AppendUnique(CPPPATH=cpppath)
-    # Insert LIBRARY_PATH explicitly because some compilers
-    # ignore it in the system environment.
-    env.PrependUnique(LIBPATH=getsyspaths('LIBRARY_PATH'))
-    if 'CONDA_PREFIX' in os.environ:
-        env.Append(CPPPATH=pjoin(os.environ['CONDA_PREFIX'],'include'))
-        env.Append(CPPPATH=pjoin(os.environ['CONDA_PREFIX'],'Library','include'))
-        env.Append(LIBPATH=pjoin(os.environ['CONDA_PREFIX'],'Library','lib'))
-    # This disable automated versioned named e.g. libboost_date_time-vc142-mt-s-x64-1_73.lib
-    # so we can use conda-installed libraries
-    env.AppendUnique(CPPDEFINES='BOOST_ALL_NO_LIB')
     # Prevent the generation of an import lib (.lib) in addition to the dll
     # Unused as we are using as static library for windows
     # env.AppendUnique(no_import_lib=1)
+    if 'CONDA_PREFIX' in os.environ:
+        env.Append(CPPPATH=pjoin(os.environ['CONDA_PREFIX'], 'include'))
+        env.Append(CPPPATH=pjoin(os.environ['CONDA_PREFIX'], 'Library', 'include'))
+        env.Append(LIBPATH=pjoin(os.environ['CONDA_PREFIX'], 'Library', 'lib'))
+else:
+    if 'CONDA_PREFIX' in os.environ:
+        env.Append(CPPPATH=pjoin(os.environ['CONDA_PREFIX'], 'include'))
+        env.Append(LIBPATH=pjoin(os.environ['CONDA_PREFIX'], 'lib'))
+
+# the CPPPATH directories are checked by scons dependency scanner
+cpppath = getsyspaths('CPLUS_INCLUDE_PATH', 'CPATH')
+env.AppendUnique(CPPPATH=cpppath)
+# Insert LIBRARY_PATH explicitly because some compilers
+# ignore it in the system environment.
+env.PrependUnique(LIBPATH=getsyspaths('LIBRARY_PATH'))
+# This disable automated versioned named e.g. libboost_date_time-vc142-mt-s-x64-1_73.lib
+# so we can use conda-installed libraries
+env.AppendUnique(CPPDEFINES='BOOST_ALL_NO_LIB')
 
 builddir = env.Dir('build/%s-%s' % (env['build'], platform.machine()))
 

conda-recipe/conda_build_config.yaml

@@ -1,2 +1,6 @@
 boost:
   - 1.67
+
+# only for local build ? see https://github.com/conda/conda-build/issues/4064#issuecomment-702983257
+#CONDA_BUILD_SYSROOT:
+#  - /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk

conda-recipe/meta.yaml

@@ -7,6 +7,7 @@ package:
 source:
   # git_url: https://github.com/diffpy/libobjcryst.git
   git_url: ..
+  # path: ..  # To build from modified local tree
 
 build:
   number: 0