Uploaded image for project: 'Data Management'
  1. Data Management
  2. DM-7620

Update the conda versions before next release

    Details

      Description

      In lsstsw we currently force miniconda2 version to 3.19.0 and miniconda3 to 4.0.5.

      After deploying miniconda2 we get:

      $ otool -L miniconda/lib/libpython2.7.dylib 
      miniconda/lib/libpython2.7.dylib:
      	libpython2.7.dylib (compatibility version 2.7.0, current version 2.7.0)
      	/usr/lib/libSystem.B.dylib (compatibility version 1.0.0, current version 111.0.0)
      	/System/Library/Frameworks/CoreFoundation.framework/Versions/A/CoreFoundation (compatibility version 150.0.0, current version 476.0.0)
      	/usr/lib/libgcc_s.1.dylib (compatibility version 1.0.0, current version 1.0.0)
      

      Doing conda update python results in:

      python:      2.7.11-0     --> 2.7.12-1
      

      and:

      $ otool -L miniconda/lib/libpython2.7.dylib 
      miniconda/lib/libpython2.7.dylib:
      	@rpath/libpython2.7.dylib (compatibility version 2.7.0, current version 2.7.0)
      	/usr/lib/libSystem.B.dylib (compatibility version 1.0.0, current version 159.1.0)
      	/System/Library/Frameworks/CoreFoundation.framework/Versions/A/CoreFoundation (compatibility version 150.0.0, current version 635.21.0)
      

      which gives the correct use of @rpath without us having to manually modify it (which we stopped doing a while back). This is important because Mac OS X builds break with galsim on XCode 8 if the @rpath is missing.

      Updating our miniconda versions to v4.1.11 fixes the problem, although we will need to update our pinned package versions after making this change as currently without the -b option the 2.7.12 fixed python is installed and then immediately downgraded.

      4.1.11 is needed for both miniconda2 and miniconda3.

      The workaround for mac users is to run conda update python before building galsim.

        Attachments

          Issue Links

            Activity

            Hide
            jhoblitt Joshua Hoblitt added a comment -

            symlinks are out because github doesn't give the raw text of the link target.

            Show
            jhoblitt Joshua Hoblitt added a comment - symlinks are out because github doesn't give the raw text of the link target.
            Hide
            tjenness Tim Jenness added a comment -

            symlinks not reading through is a bit annoying.

            Show
            tjenness Tim Jenness added a comment - symlinks not reading through is a bit annoying.
            Hide
            jhoblitt Joshua Hoblitt added a comment -

            I've merged the lsstsw update, pinned the miniconda2/3 env urls to use the sha of current master, and cut new git tags for miniconda2/3. A build/publish cycle is in a progress, which is required in order to test the changes to newinstall.sh.

            Show
            jhoblitt Joshua Hoblitt added a comment - I've merged the lsstsw update, pinned the miniconda2/3 env urls to use the sha of current master, and cut new git tags for miniconda2/3. A build/publish cycle is in a progress, which is required in order to test the changes to newinstall.sh .
            Hide
            jhoblitt Joshua Hoblitt added a comment - - edited

            The changes to newinstall.sh have been tested as working on centos7 / py2. Waiting for OSX to finish up before merging.

            Show
            jhoblitt Joshua Hoblitt added a comment - - edited The changes to newinstall.sh have been tested as working on centos7 / py2. Waiting for OSX to finish up before merging.
            Hide
            jhoblitt Joshua Hoblitt added a comment -

            newinstall.sh was tested as working on both el7 / osx for py2. Changes have been merged to master and "published" as https://sw.lsstcorp.org/eupspkg/newinstall.sh .

            Show
            jhoblitt Joshua Hoblitt added a comment - newinstall.sh was tested as working on both el7 / osx for py2. Changes have been merged to master and "published" as https://sw.lsstcorp.org/eupspkg/newinstall.sh .

              People

              • Assignee:
                jhoblitt Joshua Hoblitt
                Reporter:
                tjenness Tim Jenness
                Reviewers:
                Kian-Tat Lim, Tim Jenness
                Watchers:
                Chris Walter, Frossie Economou, Joshua Hoblitt, Kian-Tat Lim, Pim Schellart [X] (Inactive), Scott Daniel, Tim Jenness
              • Votes:
                0 Vote for this issue
                Watchers:
                7 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: