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

DESC coord package doesn't build on macOS Mojave

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: Third Party Software
    • Labels:
      None
    • Story Points:
      0.5
    • Sprint:
      Arch 2018-10-08
    • Team:
      Architecture

      Description

      The DESC coord package does not build in our anaconda system because anaconda by default tries to build macOS binaries targeting OS X 10.7. Mojave refuses to support that:

      clang -Wno-unused-result -Wsign-compare -Wunreachable-code -DNDEBUG -g -fwrapv -O3 -Wall -Wstrict-prototypes -I/Users/timj/work/lsstsw3/miniconda/envs/lsst-scipipe/include -arch x86_64 -I/Users/timj/work/lsstsw3/miniconda/envs/lsst-scipipe/include -arch x86_64 -Iinclude -I/Users/timj/work/lsstsw3/miniconda/envs/lsst-scipipe/include/python3.7m -c src/Angle.cpp -o build/temp.macosx-10.7-x86_64-3.7/src/Angle.o
      warning: include path for stdlibc++ headers not found; pass '-std=libc++' on the command line to use the libc++ standard library instead [-Wstdlibcxx-not-found]
      src/Angle.cpp:23:10: fatal error: 'cmath' file not found
      #include <cmath>
               ^~~~~~~
      1 warning and 1 error generated.
      

      The fix is to force MACOSX_DEPLOYMENT_TARGET to be at least 10.9.

        Attachments

          Issue Links

            Activity

            Hide
            tjenness Tim Jenness added a comment -

            Jonathan Sick can you see if this works for you please? I copied the fix from John Swinbank's fix for mpi4py.

            Show
            tjenness Tim Jenness added a comment - Jonathan Sick can you see if this works for you please? I copied the fix from John Swinbank 's fix for mpi4py.
            Hide
            jsick Jonathan Sick added a comment -

            Thank you this works. Confirmed by others too.

            Show
            jsick Jonathan Sick added a comment - Thank you this works. Confirmed by others too.

              People

              Assignee:
              tjenness Tim Jenness
              Reporter:
              tjenness Tim Jenness
              Reviewers:
              Jonathan Sick
              Watchers:
              Jonathan Sick, Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.