Jim Bosch added a comment - 06/Oct/14 10:54 AM Frossie, entirely trivial code review for you: lsst_apps:u/jbosch/DM-1283 % git show commit d9cac129b6ea7ae4faaa5a4c99b2e19c305916b0 Author: Jim Bosch <jbosch@astro.princeton.edu> Date: Mon Oct 6 13:52:10 2014 -0400   Remove meas_extensions_multiShapelet from dependencies   diff --git a/ups/lsst_apps.table b/ups/lsst_apps.table index d5f8895..3790d36 100644 --- a/ups/lsst_apps.table +++ b/ups/lsst_apps.table @@ -1,6 +1,5 @@ setupRequired(lsst_libs) setupRequired(meas_deblender) -setupRequired(meas_extensions_multiShapelet) setupRequired(pipe_tasks) setupRequired(obs_lsstSim) setupRequired(obs_sdss) I'm mostly just looking for your sign-off that: this would be a non-disruptive time to merge this change to master there's nothing else I need to do to remove this package from buildbot and eups distrib releases.

Frossie Economou added a comment - 06/Oct/14 11:02 AM Looks good - and I believe that's all you need to do. Do you want me to backport this change to S14? Or is it going forward only?

Jim Bosch added a comment - 06/Oct/14 11:13 AM Do you want me to backport this change to S14? Or is it going forward only? Going forward only, though I plan to request an early W15 release that includes this in the next week or so, once a few more issues are in as well.

Jim Bosch added a comment - 09/Oct/14 3:10 PM Looks like this is still part of buildbot, so there must be something else I need to do.

Jim Bosch added a comment - 09/Oct/14 6:37 PM Found another (optional) dependency in obs_sdss, which I've now removed as well. Will close once I've confirmed that it's not part of buildbot.

Jim Bosch added a comment - 15/Oct/14 11:53 AM Russell Owen helped out by removing this from the dependencies of datarel and testing_endToEnd, and I can now confirm that meas_extensions_multiShapelet is no longer being built by buildbot.