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

Compile pybind11 with hidden symbol visibility on linux

    Details

    • Story Points:
      0.25
    • Sprint:
      AP F18-4
    • Team:
      Alert Production

      Description

      DM-15151 added compilation of pybind11 with hidden symbol visibility, but unfortunately only on macOS. Also add it to linux.

      If we are lucky this is a trivial change to sconsUtils. If we are unlucky then that trivial change will break the build on linux.

        Attachments

          Issue Links

            Activity

            Hide
            rowen Russell Owen added a comment - - edited

            If you are willing, please build a package with this and make sure the warnings have gone away.

            It has passed Jenkins for linux (including ci_hsc) and the change should not be meaningful on macOS, so I think it is ready to review.

            Show
            rowen Russell Owen added a comment - - edited If you are willing, please build a package with this and make sure the warnings have gone away. It has passed Jenkins for linux (including ci_hsc) and the change should not be meaningful on macOS, so I think it is ready to review.
            Hide
            jbosch Jim Bosch added a comment -

            Looks good!  I have not done a local test, but I'd be happy to poke at the Jenkins build logs if you can point me at the Jenkins run; I bet we can verify that they're gone that way.

            Show
            jbosch Jim Bosch added a comment - Looks good!  I have not done a local test, but I'd be happy to poke at the Jenkins build logs if you can point me at the Jenkins run; I bet we can verify that they're gone that way.
            Hide
            rowen Russell Owen added a comment - - edited

            The Jenkins link in my previous comment points to the appropriate Jenkins run. And I think you are right – it was easy to find the afw build log in Artifacts and I just see one warning, and I'm pretty sure it's been with us for awhile. No symbol visibility warnings.

            Show
            rowen Russell Owen added a comment - - edited The Jenkins link in my previous comment points to the appropriate Jenkins run. And I think you are right – it was easy to find the afw build log in Artifacts and I just see one warning, and I'm pretty sure it's been with us for awhile. No symbol visibility warnings.
            Hide
            rowen Russell Owen added a comment -

            Thank you for the speedy review and the helpful suggestion for looking at the linux build logs.

            I could not find artifacts from the macOS build, but the change should not have affected macOS.

            Show
            rowen Russell Owen added a comment - Thank you for the speedy review and the helpful suggestion for looking at the linux build logs. I could not find artifacts from the macOS build, but the change should not have affected macOS.
            Hide
            jbosch Jim Bosch added a comment -

            Yup, don't know how I missed that link before, but I agree it's all clear.

            Show
            jbosch Jim Bosch added a comment - Yup, don't know how I missed that link before, but I agree it's all clear.

              People

              • Assignee:
                rowen Russell Owen
                Reporter:
                rowen Russell Owen
                Reviewers:
                Jim Bosch
                Watchers:
                Jim Bosch, Russell Owen
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel