RFR: JDK-8215635: Pandoc check in Docs.gmk does not work on Windows
Andrew Luo
andrewluotechnologies at outlook.com
Thu Dec 20 18:18:13 UTC 2018
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).
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
>
More information about the build-dev
mailing list