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

ci_hsc fails to execute tasks from with SCons on OSX 10.11/SIP

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: ci_hsc
    • Labels:
      None

      Description

      The ci_hsc package executes a number of command line tasks directly from SCons based on Command directives in a SConstruct file. On an OSX 10.11 system with SIP enabled, there are two distinct problems which prevent the necessary environment being propagated to the tasks:

      • The scons executable starts with a #!/usr/bin/env python. Running through /usr/bin/env strips DYLD_LIBRARY_PATH from the environment. (duplicates DM-4954)
      • SCons executes command using the sh shell on posix systems. By default, that means /bin/sh on a Mac, which, again, will strip DYLD_LIBRARY_PATH.

      Please make it possible to run ci_hsc on such a system.

        Attachments

          Issue Links

            Activity

            Hide
            swinbank John Swinbank added a comment -

            Hey Nate Lust – could you take a look at this please? All changes on ci_hsc PR#1. Also including some documentation changes (which are probably all we can/should do about DM-4954 for now). Please check it works if you remove your funky shell!

            Show
            swinbank John Swinbank added a comment - Hey Nate Lust – could you take a look at this please? All changes on ci_hsc PR#1 . Also including some documentation changes (which are probably all we can/should do about DM-4954 for now). Please check it works if you remove your funky shell!
            Hide
            nlust Nate Lust added a comment -

            Sorry for taking so long to look at this. Your changes look good, but could you rebase your branch off master so I run this? Alternatively, just verify that it runs on your mac laptop

            Show
            nlust Nate Lust added a comment - Sorry for taking so long to look at this. Your changes look good, but could you rebase your branch off master so I run this? Alternatively, just verify that it runs on your mac laptop
            Hide
            swinbank John Swinbank added a comment -

            Rebased and cleaned up – and you just saw it running. Let me know if you're happy!

            Show
            swinbank John Swinbank added a comment - Rebased and cleaned up – and you just saw it running. Let me know if you're happy!
            Hide
            nlust Nate Lust added a comment -

            Merge away

            Show
            nlust Nate Lust added a comment - Merge away
            Hide
            swinbank John Swinbank added a comment -

            Thanks; merged.

            Show
            swinbank John Swinbank added a comment - Thanks; merged.

              People

              • Assignee:
                swinbank John Swinbank
                Reporter:
                swinbank John Swinbank
                Reviewers:
                Nate Lust
                Watchers:
                John Swinbank, Nate Lust, Tim Jenness
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel