RFR: JDK-8188814 Simplify IncludeCustomExtension
erik.joelsson at oracle.com
Thu Oct 5 11:07:57 UTC 2017
On 2017-10-05 12:02, Magnus Ihse Bursie wrote:
> Now that we only have a single repo, the first argument to
> IncludeCustomExtension is no longer needed.
> I looked at some more ambitious ways to simplify
> IncludeCustomExtension. Getting make to automatically retrieve the
> name of the file is just sooooo tantalizingly close to possible, but
> ultimately not doable. :-(
> One way to achieve this (and also get automatic hooks for custom
> extensions prior and after including a file) would be to replace all
> places now sporting "include $FILE" with a "$(eval $(call include,
> $FILE))". This would also help somewhat with debuggability, by
> providing means from the code, e.g. in error messages, to know the
> name of the current file, and showing the include tree in e.g. LOG=debug.
> However, in the end, I decided it was not really worth it. We have
> 200+ includes that needed changing, and then we would need to remember
> to use this pattern for all new future changes.
> Bug: https://bugs.openjdk.java.net/browse/JDK-8188814
More information about the build-dev