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

ndarray sometimes fails to build using lsstsw

    Details

      Description

      Today I asked lsstsw to rebuild a product (astshim) and it failed while bulding ndarray, with this error:

      /Users/rowen/UW/LSST/lsstsw3/build/ndarray/include/ndarray/detail/Core.h:22:10: fatal error: 'ndarray/Vector.h' file not found
      #include "ndarray/Vector.h"
      

      That file is built by m4 so I suspect race condition or other sequencing error in the build instructions. When I resubmitted the rebuild command it worked.

        Attachments

          Issue Links

            Activity

            Hide
            jbosch Jim Bosch added a comment -

            Looks like this wasn't fixed by DM-14728 after all; Paul Price rediscovered it, put together a fix, and I've merged that and cut a release upstream.  I'll use this ticket to push out an LSST uprev of ndarray.

            Show
            jbosch Jim Bosch added a comment - Looks like this wasn't fixed by DM-14728 after all; Paul Price rediscovered it, put together a fix, and I've merged that and cut a release upstream.  I'll use this ticket to push out an LSST uprev of ndarray.
            Hide
            jbosch Jim Bosch added a comment -

            Paul Price, mind reviewing?  Not much to see in LSST repos, as the only change is the (binary) update to the tarball.  But it got through Jenkins, and ndarray's own travis-ci, and I'm happy to trust your testing that the build race is fixed.

            Show
            jbosch Jim Bosch added a comment - Paul Price , mind reviewing?  Not much to see in LSST repos, as the only change is the (binary) update to the tarball.  But it got through Jenkins, and ndarray's own travis-ci, and I'm happy to trust your testing that the build race is fixed.
            Hide
            price Paul Price added a comment -

            Dunno what's really to review, but I approve.

            Show
            price Paul Price added a comment - Dunno what's really to review, but I approve.
            Hide
            rowen Russell Owen added a comment -

            Jim Bosch do you want me to merge this? Have you done it?

            Show
            rowen Russell Owen added a comment - Jim Bosch do you want me to merge this? Have you done it?
            Hide
            jbosch Jim Bosch added a comment -

            Merged.  Thanks for the reminder, and sorry for the delay.

            Show
            jbosch Jim Bosch added a comment - Merged.  Thanks for the reminder, and sorry for the delay.

              People

              • Assignee:
                rowen Russell Owen
                Reporter:
                rowen Russell Owen
                Reviewers:
                Paul Price
                Watchers:
                Jim Bosch, Paul Price, Pim Schellart [X] (Inactive), Russell Owen
              • Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel