Unless the 'FIND_REQUIRED' pattern is standard to all FindXXX components (which I don;t think it is) then I think you should follow our convention, not APR's