Added SConscript file bin.src in drp_stella (tickets/INFRA-42) and obs_pfs (tickets/INFRA-42). While the bin executables ARE getting created now for drp_stella, somehow they are still NOT being created for obs_pfs... [scratches head]

Work on obs_pfs is now ready for review: pull request here.

I am confused by what's happened on drp_stella: the tickets/INFRA-42 branch contains this commit which looks like it should really be part of PIPE2D-33. aritter, I would like to rebase tickets/INFRA-42 to remove that commit before this is merged. Can you confirm that that's ok? I don't want to lose any of your work.

Please go ahead John, sorry for that. Guess I had the wrong branch while working on it. Will double-check from now on...

Thanks!

Now ready for review on both:

• drp_stella branch tickets/INFRA-42
• obs_pfs branch tickets/INFRA-42

aritter, you did all the work on drp_stella, so I am happy to sign off on that. Can you please check that you are happy with the changes to obs_pfs?

Works for me. What was the problem?

The master SConstruct file can be set to only build certain "targets" (effectively, directories). If we don't specify, it uses a default list (which includes bin.src). However, in obs_pfs that default list was being over-ridden to something which did not include bin.src, so the SConscript you added wasn't being included.

I edited the SConstruct to use the defaults (plus an appended pfs).

Merged.