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

Cannot build packages against galsim binaries distributed by eups distrib

    XMLWordPrintable

Details

    • Story
    • Status: Done
    • Resolution: Done
    • None
    • galsim
    • None
    • 1
    • SQuaRE

    Description

      I have installed my stack using the eups distrib binaries. Today I went to rebuild the meas_extensions_hsm package which links to the galsim package, and the build failed. It complained about not being able to find libgalsim.dylib in the galsim package. Upon looking in the directory for galsim I see:

      _-rwxr-xr-x  1 nate  staff  3289320 Dec 13 10:12_ _libgalsim.1.5.dylib_
      _lrwxr-xr-x  1 nate  staff      190 Dec 13 10:12_ _libgalsim.dylib_ _-> /Users/square/jenkins/workspace/release/tarball/osx/10.9/clang-800.0.42.1/miniconda3-4.3.21-10a4fa6/build/stack/miniconda3-4.3.21-10a4fa6/DarwinX86/galsim/1.5.1.lsst1/lib/libgalsim.1.5.dylib_
      

      Which shows the simlink for the dylib is still pointing to the a directory on the build system. I propose that if this cant be fixed on the packaging side that we need a program like the shebangtron that will rewrite all the simlinks

      Attachments

        Issue Links

          Activity

            OK, I've fixed things and tested on both CentOS and OS X. I tested by installing the binary distro and building sims_GalSimInterface and meas_extensions_shapeHSM. There is a question about pytest that I've put in #dm, but I think that's unrelated. The changes are small, but took several iterations to reach a conclusion. I will squash when the review is complete. I'm hoping nlust is willing to have a look.

            krughoff Simon Krughoff (Inactive) added a comment - OK, I've fixed things and tested on both CentOS and OS X. I tested by installing the binary distro and building sims_GalSimInterface and meas_extensions_shapeHSM . There is a question about pytest that I've put in #dm, but I think that's unrelated. The changes are small, but took several iterations to reach a conclusion. I will squash when the review is complete. I'm hoping nlust is willing to have a look.
            nlust Nate Lust added a comment -

            Some comments up on github for you to look at if you would please.

            nlust Nate Lust added a comment - Some comments up on github for you to look at if you would please.

            nlust thanks for looking. I've responded on the PR.

            krughoff Simon Krughoff (Inactive) added a comment - nlust thanks for looking. I've responded on the PR.

            I believe I've made the requested changes. nlust let me know what you think of them.

            krughoff Simon Krughoff (Inactive) added a comment - I believe I've made the requested changes. nlust let me know what you think of them.
            krughoff Simon Krughoff (Inactive) added a comment - Jenkins has passed: https://ci.lsst.codes/blue/organizations/jenkins/stack-os-matrix/detail/stack-os-matrix/27685/ .

            People

              krughoff Simon Krughoff (Inactive)
              nlust Nate Lust
              Nate Lust
              Frossie Economou, John Swinbank, Joshua Hoblitt, Nate Lust, Robert Lupton, Simon Krughoff (Inactive), Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              7 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Jenkins

                  No builds found.