RFR: JDK-8179453 Add a proper SetupProcessMarkdown
Erik Joelsson
erik.joelsson at oracle.com
Tue May 2 18:32:12 UTC 2017
Hello,
Looks good. For efficiency reasons, I would recommend the following change:
Assign $1_$2_OPTIONS using just = for lazy evaluation so that we only do
the shell call if executing the recipe. Also remove it from VARDEPS
because the contents of OPTIONS only depends on the source file anyway
and the rule already depends on the source file.
/Erik
On 2017-04-28 21:52, Magnus Ihse Bursie wrote:
> To cover the coming needs of the added markdown specifications, a more
> proper function for setting up markdown processing is needed.
>
> The CSS file is intented as a first approach/placeholder. It needs to
> be updated later on.
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8179453
> WebRev:
> http://cr.openjdk.java.net/~ihse/JDK-8179453-add-process-markdown/webrev.01
>
> /Magnus
More information about the build-dev
mailing list