RFR: JDK-8215635: Pandoc check in Docs.gmk does not work on Windows

[Erik Joelsson]

Hello,

On 2018-12-20 19:18, Andrew Luo wrote:
> Not a reviewer, but also looks good to me.  I encountered this issue myself while working on the WSL patch (perhaps that's also how you found this issue).  I do wonder if this is the only utility that has this issue - if not, another possibility could be where we prefix PANDOC with FIXPATH, we could check if PANDOC is empty before prefixing FIXPATH (if it's empty we can just not prefix FIXPATH).

Yes, that was indeed how I encountered it because I couldn't configure 
using our internal wrapper framework (Jib) which provides all 
dependencies like pandoc.

I think in several cases, the prepending of fixpath should be handled in 
configure rather than in spec.gmk.in. With WSL, we have the possibility 
of having either WSL tools or Windows tools for certain things, like the 
bootjdk and pandoc. In that case, configure needs to detect which kind 
it is and only prefix fixpath when needed. I think that would be a good 
future improvement.

/Erik

> Thanks,
>
> -Andrew
>
> -----Original Message-----
> From: build-dev <build-dev-bounces at openjdk.java.net> On Behalf Of Magnus Ihse Bursie
> Sent: Thursday, December 20, 2018 2:49 AM
> To: Erik Joelsson <erik.joelsson at oracle.com>
> Cc: build-dev <build-dev at openjdk.java.net>
> Subject: Re: RFR: JDK-8215635: Pandoc check in Docs.gmk does not work on Windows
>
> Lgtm.
>
> /Magnus
>
>> 19 dec. 2018 kl. 12:40 skrev Erik Joelsson <erik.joelsson at oracle.com>:
>>
>> Hello,
>>
>> In JDK-8214910 a check for if the PANDOC variable had a value was added to Docs.gmk. This check does not work correctly on Windows where PANDOC will always have the fixpath prefix. This patch adds a separate ENABLE_PANDOC variable that make can check.
>>
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8215635
>>
>> Webrev: http://cr.openjdk.java.net/~erikj/8215635/webrev.01/
>>
>> /Erik
>>