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

remove dead code and dependencies from datarel

    Details

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

      Description

      Removing the datarel package entirely has proved to be difficult (DM-2928, DM-2948), so instead I'm simply going to remove non-ingest code (and dead ingest code) from the package, along with its dependencies on ap and testing_endToEnd. Other dependencies will be retained even if they aren't necessary for the code that will remain in datarel, to support lsstDoxygen's use of datarel as a top-level package for documentation generation.

        Attachments

          Issue Links

            Activity

            Hide
            jbosch Jim Bosch added a comment -

            I don't think it makes sense to change the name yet, as we're also still using this package as a metapackage for the cross-linked Doxygen build (see DM-2948). That means we're still carrying more dependencies than the ingest code here needs, and we can't change the name without changing the buildbot scripts (which I can't do without help from SQuaRE, and they're busy).

            Show
            jbosch Jim Bosch added a comment - I don't think it makes sense to change the name yet, as we're also still using this package as a metapackage for the cross-linked Doxygen build (see DM-2948 ). That means we're still carrying more dependencies than the ingest code here needs, and we can't change the name without changing the buildbot scripts (which I can't do without help from SQuaRE, and they're busy).
            Hide
            jbosch Jim Bosch added a comment -

            Merged to master.

            Show
            jbosch Jim Bosch added a comment - Merged to master.
            Hide
            ktl Kian-Tat Lim added a comment -

            Jim Bosch You missed the part about adding a new (nearly-empty) datarel that would serve as the metapackage for the cross-linked Doxygen build and the buildbot scripts.

            Show
            ktl Kian-Tat Lim added a comment - Jim Bosch You missed the part about adding a new (nearly-empty) datarel that would serve as the metapackage for the cross-linked Doxygen build and the buildbot scripts.
            Hide
            jbosch Jim Bosch added a comment -

            Oh, I did miss that part somehow, reading too fast - sorry about that. I'm still not sure it's worthwhile to do right now, as I think a better solution for the Doxygen/buildbot problem is to remove the need for a single top-level package. But we may want to do that first if Jacek Becla's team starts working on updating the ingest code before DM-2948 is addressed.

            Show
            jbosch Jim Bosch added a comment - Oh, I did miss that part somehow, reading too fast - sorry about that. I'm still not sure it's worthwhile to do right now, as I think a better solution for the Doxygen/buildbot problem is to remove the need for a single top-level package. But we may want to do that first if Jacek Becla 's team starts working on updating the ingest code before DM-2948 is addressed.
            Hide
            swinbank John Swinbank added a comment -

            Adding this to DM-1769 since it was necessary to unblock DM-1766.

            Show
            swinbank John Swinbank added a comment - Adding this to DM-1769 since it was necessary to unblock DM-1766 .

              People

              • Assignee:
                jbosch Jim Bosch
                Reporter:
                jbosch Jim Bosch
                Reviewers:
                Kian-Tat Lim
                Watchers:
                Jim Bosch, John Swinbank, Kian-Tat Lim
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel