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

Improve build system for sphgeom

    Details

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

      Attachments

        Issue Links

          Activity

          Hide
          smonkewitz Serge Monkewitz added a comment -

          Yes: DM-2646 and DM-2178, which appear to be duplicates (see the issue links section).

          Show
          smonkewitz Serge Monkewitz added a comment - Yes: DM-2646 and DM-2178 , which appear to be duplicates (see the issue links section).
          Hide
          fritzm Fritz Mueller added a comment -

          Thanks, Serge. Is there a follow-up ticket to cut qserv over to this and cut loose the clone in the qserv source tree?

          Show
          fritzm Fritz Mueller added a comment - Thanks, Serge. Is there a follow-up ticket to cut qserv over to this and cut loose the clone in the qserv source tree?
          Hide
          jbecla Jacek Becla added a comment -

          Looks fine to me!

          Show
          jbecla Jacek Becla added a comment - Looks fine to me!
          Hide
          smonkewitz Serge Monkewitz added a comment -

          The pull request is https://github.com/lsst/sphgeom/pull/2

          Should be straightforward - changes consist of removing the previous build system, moving code around, and some minor adjustment of includes.

          Show
          smonkewitz Serge Monkewitz added a comment - The pull request is https://github.com/lsst/sphgeom/pull/2 Should be straightforward - changes consist of removing the previous build system, moving code around, and some minor adjustment of includes.
          Hide
          jbecla Jacek Becla added a comment -

          Given there is potential MariaDB's new GIS will come handy, we are putting this story on the back burner for a little while to see how things are progressing with MariaDB GIS.

          Show
          jbecla Jacek Becla added a comment - Given there is potential MariaDB's new GIS will come handy, we are putting this story on the back burner for a little while to see how things are progressing with MariaDB GIS.
          Hide
          ktl Kian-Tat Lim added a comment -

          Note that if, contra the last comment, sphgeom is to be considered a fully non-LSST external package, then it should be packaged like other external packages (as a tar-and-patch package).

          Show
          ktl Kian-Tat Lim added a comment - Note that if, contra the last comment, sphgeom is to be considered a fully non-LSST external package, then it should be packaged like other external packages (as a tar-and-patch package).
          Hide
          ktl Kian-Tat Lim added a comment -

          sphgeom needs at a minimum to be installable with "eups distrib install" (i.e. be eupspkg-compatible). It should also be sconsUtils-compatible, which means that it should contain a .cfg file saying what it exports. Finally, it would be nice if it is eups-compatible (meaning that developers can do "setup -r" and build in the expected way for the LSST Stack, as opposed to making it more of an external package).

          Show
          ktl Kian-Tat Lim added a comment - sphgeom needs at a minimum to be installable with "eups distrib install" (i.e. be eupspkg-compatible). It should also be sconsUtils-compatible, which means that it should contain a .cfg file saying what it exports. Finally, it would be nice if it is eups-compatible (meaning that developers can do "setup -r" and build in the expected way for the LSST Stack, as opposed to making it more of an external package).

            People

            • Assignee:
              smonkewitz Serge Monkewitz
              Reporter:
              fritzm Fritz Mueller
              Reviewers:
              Fritz Mueller, Jacek Becla
              Watchers:
              Fritz Mueller, Jacek Becla, Kian-Tat Lim, Serge Monkewitz
            • Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Summary Panel