Kian-Tat Lim added a comment - 24/Mar/16 10:39 PM Like anything else, it needs an RFC if you believe that some affected party might seriously object to your decision and implementation.

John Swinbank added a comment - 31/Mar/16 1:19 PM We agreed at the DRP standup of 31 March that this is unlikely to need an RFC and Paul Price will go ahead with it when DM-5409 is resolved.

Paul Price added a comment - 25/Apr/16 11:24 AM The determination that this was "unlikely to need an RFC" was based on the perception that ctrl_pool was already in lsst_distrib, but it turns out this was wrong. Submitted RFC-171 to include pipe_drivers in lsst_distrib.

Paul Price added a comment - 09/May/16 2:10 PM Sorry to bother you again so soon, Tim Jenness , but this is super easy, and I wanted to bother someone with your high-level view to ensure that this is indeed OK to merge. price@price-laptop:~/LSST/lsst_distrib (tickets/DM-5634=) $ git sub-patch commit fa72bcd6f89136c65b004d79c628bb16d738382e Author: Paul Price <price@astro.princeton.edu> Date: Mon May 9 17:08:39 2016 -0400   add pipe_drivers   diff --git a/ups/lsst_distrib.table b/ups/lsst_distrib.table index 9fe0d5a..9e29b60 100644 --- a/ups/lsst_distrib.table +++ b/ups/lsst_distrib.table @@ -6,3 +6,4 @@ setupRequired(ctrl_platform_lsst) setupRequired(datarel) setupOptional(obs_subaru) setupOptional(meas_extensions_shapeHSM) +setupOptional(pipe_drivers)

Tim Jenness added a comment - 09/May/16 2:15 PM Looks fine as optional given the other optional items in there. I assume there was a Jenkins run of this patch to ensure it all works as we expect? Once closed I think that means the RFC can be changed to Implemented.

Paul Price added a comment - 09/May/16 6:25 PM Thanks again, Tim Jenness ! Jenkins passed . Merged to master.