RFR: JDK-8217916: Build compare script is not comparing jmods
Magnus Ihse Bursie
magnus.ihse.bursie at oracle.com
Wed Jan 30 14:34:29 UTC 2019
On 2019-01-29 01:34, Erik Joelsson wrote:
> This patch enhances the build compare script to also look at the jmod
> files in the JDK image. This was triggered by JDK-8217826 (linked in
> issue) where we see a rare intermittent comparison failure on Windows
> when looking at file types. That seems to be caused by the file
> command not knowing about jmod files (at least not the versions of the
> command we have).
>
> With this patch, I'm filtering out .jmod files from the file type
> comparison since it seems pointless. Instead I use the jmod tool to
> list the contents of each jmod file and verify that they contain the
> same file names. I don't think there is a point to comparing the
> contents of those files since all the same files will be in the JDK
> image and will be compared anyway. Doing it again would be a waste of
> time.
>
> Tested manually introducing differences and running the script, as
> well as running the build with COMPARE_BUILD=PATCH= on all relevant
> platforms.
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8217916
>
> Webrev: http://cr.openjdk.java.net/~erikj/8217916/webrev.01/index.html
Looks good to me.
/Magnus
>
> /Erik
>
More information about the build-dev
mailing list