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

          jbecla Jacek Becla created issue -
          jbecla Jacek Becla made changes -
          Field Original Value New Value
          Rank Ranked higher
          jbecla Jacek Becla made changes -
          Component/s sphgeom [ 12231 ]
          danielw Daniel Wang [X] (Inactive) made changes -
          Link This issue relates to DM-2178 [ DM-2178 ]
          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).
          jbecla Jacek Becla made changes -
          Status To Do [ 10001 ] In Progress [ 3 ]
          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).
          smonkewitz Serge Monkewitz made changes -
          Sprint DB_S15_03 [ 136 ] DB_S15_04 [ 137 ]
          jbecla Jacek Becla made changes -
          Assignee Serge Monkewitz [ smonkewitz ] Jacek Becla [ jbecla ]
          jbecla Jacek Becla made changes -
          Link This issue blocks DM-2646 [ DM-2646 ]
          jbecla Jacek Becla made changes -
          Sprint DB_S15_04 [ 137 ] DB_S15_04, DB_S15_05 [ 137, 138 ]
          jbecla Jacek Becla made changes -
          Rank Ranked higher
          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.
          jbecla Jacek Becla made changes -
          Sprint DB_S15_04, DB_S15_05 [ 137, 138 ] DB_S15_04 [ 137 ]
          jbecla Jacek Becla made changes -
          Status In Progress [ 3 ] To Do [ 10001 ]
          jbecla Jacek Becla made changes -
          Assignee Jacek Becla [ jbecla ]
          smonkewitz Serge Monkewitz made changes -
          Assignee Serge Monkewitz [ smonkewitz ]
          Status To Do [ 10001 ] In Progress [ 3 ]
          smonkewitz Serge Monkewitz made changes -
          Sprint DB_S15_04 [ 137 ] DB_S15_04, DB_W16_12 [ 137, 172 ]
          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.
          smonkewitz Serge Monkewitz made changes -
          Reviewers Fritz Mueller, Jacek Becla [ fritzm, jbecla ]
          Status In Progress [ 3 ] In Review [ 10004 ]
          Hide
          jbecla Jacek Becla added a comment -

          Looks fine to me!

          Show
          jbecla Jacek Becla added a comment - Looks fine to me!
          tjenness Tim Jenness made changes -
          Link This issue relates to DM-4609 [ DM-4609 ]
          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?
          fritzm Fritz Mueller made changes -
          Status In Review [ 10004 ] Reviewed [ 10101 ]
          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).
          jbecla Jacek Becla made changes -
          Link This issue blocks DM-2178 [ DM-2178 ]
          jbecla Jacek Becla made changes -
          Link This issue relates to DM-2178 [ DM-2178 ]
          smonkewitz Serge Monkewitz made changes -
          Resolution Done [ 10000 ]
          Status Reviewed [ 10101 ] Done [ 10002 ]
          jbecla Jacek Becla made changes -
          Epic Link DM-4401 [ 21533 ]
          fritzm Fritz Mueller made changes -
          Reporter Jacek Becla [X] [ jbecla ] Fritz Mueller [ fritzm ]

            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