[cmake-developers] [PATCH] Discussion needed: FindBISON: refine VERBOSE feature of BISON_TARGET

[Brad King]

On 07/15/2016 04:51 AM, Eon Jeong wrote:
> review past discussion on this issue.

Thanks for summarizing.

> 3.5. [VERBOSE [<file>]] and [REPORT_FILE <file>]. In this case, <file>
> argument of [VERBOSE <file>] can be either ignored or used (copying
> the report file.) This does not break compatibility at all

I think this is the best solution.  One could also reject use of
both <file> options at the same time.

> - Behavioral & syntactic compatibility of [VERBOSE <file>] between
> before and after the patch.

We need to remain compatible with users of [VERBOSE <file>].

> - If we add [REPORT_FILE <file>], it does not work on systems using
> bison before 2007. We may care it with custom command using `move`, or
> not.

If a project chooses to use REPORT_FILE then it is implying a dependency
on a sufficiently new version of bison, not CMake.  This is fine.

Thanks,
-Brad