RFR: JDK-8179453 Add a proper SetupProcessMarkdown
Magnus Ihse Bursie
magnus.ihse.bursie at oracle.com
Wed May 3 07:01:40 UTC 2017
On 2017-05-02 20:32, Erik Joelsson wrote:
> 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.
Good point. I fixed it like that.
/Magnus
>
> /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