RE: RFR: 8237192: Generate stripped/public pdbs on Windows for jdk images

2020-02-19 Thread Langer, Christoph
: 'hotspot-...@openjdk.java.net' ; Baesken, Matthias Subject: Re: RFR: 8237192: Generate stripped/public pdbs on Windows for jdk images Hello Christoph, Thanks for hanging in there, this is now looking good to me. Nice to see a more general solution to the java.pdb problem. /Erik On 2020-02-18 06:47

Re: RFR: 8237192: Generate stripped/public pdbs on Windows for jdk images

2020-02-18 Thread Erik Joelsson
-...@openjdk.java.net>>; Baesken, Matthias mailto:matthias.baes...@sap.com>> *Subject:* Re: RFR: 8237192: Generate stripped/public pdbs on Windows for jdk images Hello Christoph, This patch certainly looks better to me, though I agree it's a bit hackish to have to filter and rename t

RE: RFR: 8237192: Generate stripped/public pdbs on Windows for jdk images

2020-02-18 Thread Langer, Christoph
' ; Baesken, Matthias Subject: RE: RFR: 8237192: Generate stripped/public pdbs on Windows for jdk images Hi Erik, thanks for your review. While I’ve addressed all your points (thanks for the great hints regarding usage of SetupCopyFiles), I also enhanced the configure option a little bit. What you

RE: RFR: 8237192: Generate stripped/public pdbs on Windows for jdk images

2020-02-17 Thread Langer, Christoph
-...@openjdk.java.net' ; Baesken, Matthias Subject: Re: RFR: 8237192: Generate stripped/public pdbs on Windows for jdk images Hello Christoph, This patch certainly looks better to me, though I agree it's a bit hackish to have to filter and rename the stripped.pdb files twice, once for jmods and again

RE: RFR: 8237192: Generate stripped/public pdbs on Windows for jdk images

2020-02-14 Thread Langer, Christoph
Hi Alan, > On 12/02/2020 22:16, Erik Joelsson wrote: > > Hello Christoph, > > > > This patch certainly looks better to me, though I agree it's a bit > > hackish to have to filter and rename the stripped.pdb files twice, > > once for jmods and again for bundles. I think I'm ok with it for now > >

Re: RFR: 8237192: Generate stripped/public pdbs on Windows for jdk images

2020-02-13 Thread Alan Bateman
On 12/02/2020 22:16, Erik Joelsson wrote: Hello Christoph, This patch certainly looks better to me, though I agree it's a bit hackish to have to filter and rename the stripped.pdb files twice, once for jmods and again for bundles. I think I'm ok with it for now though. The future improvement

Re: RFR: 8237192: Generate stripped/public pdbs on Windows for jdk images

2020-02-12 Thread Erik Joelsson
-dev@openjdk.java.net' ; 'hotspot-...@openjdk.java.net' *Subject:* Re: RFR: 8237192: Generate stripped/public pdbs on Windows for jdk images On 2020-02-07 09:50, Baesken, Matthias wrote: Hello, here is  a slightly changed  new  webrev : http://cr.openjdk.java.net/~mbaesken/web

RE: RFR: 8237192: Generate stripped/public pdbs on Windows for jdk images

2020-02-12 Thread Langer, Christoph
java.net>; 'hotspot- d...@openjdk.java.net<mailto:d...@openjdk.java.net>'<mailto:hotspot-...@openjdk.java.net> Subject: RE: RFR: 8237192: Generate stripped/public pdbs on Windows for jdk images Hi David , yes I think it makes sense to have a configure option for this . Not everyone would like to have a larger JDK (even it is only a bit larger).

Re: RFR: 8237192: Generate stripped/public pdbs on Windows for jdk images

2020-02-07 Thread Magnus Ihse Bursie
istoph -Original Message- From: build-dev On Behalf Of Erik Joelsson Sent: Donnerstag, 23. Januar 2020 18:49 To: Baesken, Matthias; David Holmes ; 'build-dev@openjdk.java.net' ; 'hotspot-...@openjdk.java.net' Subject: Re: RFR: 8237192: Generate stripped/public pdbs on Windows for j

RE: RFR: 8237192: Generate stripped/public pdbs on Windows for jdk images

2020-02-07 Thread Baesken, Matthias
ken to juggle the > two > >>>>> sets of pdb files around, making sure each build and test use case has > >>>>> the correct one in place where and when it's needed. Quite possibly, > we > >>>>> cannot cover all us

Re: RFR: 8237192: Generate stripped/public pdbs on Windows for jdk images

2020-02-06 Thread Erik Joelsson
20 18:49 To: Baesken, Matthias; David Holmes ; 'build-dev@openjdk.java.net' ; 'hotspot-...@openjdk.java.net' Subject: Re: RFR: 8237192: Generate stripped/public pdbs on Windows for jdk images On 2020-01-23 00:03, Baesken, Matthias wrote: Hi Erik, yes true sorry for answering your comments a bit

Re: RFR: 8237192: Generate stripped/public pdbs on Windows for jdk images

2020-02-06 Thread Magnus Ihse Bursie
From: build-dev On Behalf Of Erik Joelsson Sent: Donnerstag, 23. Januar 2020 18:49 To: Baesken, Matthias ; David Holmes ; 'build-dev@openjdk.java.net' ; 'hotspot-...@openjdk.java.net' Subject: Re: RFR: 8237192: Generate stripped/public pdbs on Windows for jdk images On 2020-01-23 00:03, Baesken,

RE: RFR: 8237192: Generate stripped/public pdbs on Windows for jdk images

2020-02-06 Thread Langer, Christoph
net' d...@openjdk.java.net>; 'hotspot-...@openjdk.java.net' d...@openjdk.java.net> > Subject: Re: RFR: 8237192: Generate stripped/public pdbs on Windows for > jdk images > > > On 2020-01-23 00:03, Baesken, Matthias wrote: > > Hi Erik, yes true sorry for answering your

RE: RFR: 8237192: Generate stripped/public pdbs on Windows for jdk images

2020-02-04 Thread Baesken, Matthias
> > Hi Erik, maybe we can just rename the configure option to > > --enable-stripped-pdbs-for-bundle > > AND make the default = no/false . > Then without setting the configure flag, everything stays as it is for JDK > vendors/distributors who do not want the stripped pdbs in the bundle.

RE: RFR: 8237192: Generate stripped/public pdbs on Windows for jdk images

2020-01-24 Thread Baesken, Matthias
gt; On 2020-01-22 07:46, Baesken, Matthias wrote: > >>> Hello, here is an updated version : > >>> > >>> http://cr.openjdk.java.net/~mbaesken/webrevs/8237192.3/ > >>> > >>> this one supports a configure switch "

Re: RFR: 8237192: Generate stripped/public pdbs on Windows for jdk images

2020-01-23 Thread Erik Joelsson
...@openjdk.java.net' ; 'hotspot- d...@openjdk.java.net' Subject: RE: RFR: 8237192: Generate stripped/public pdbs on Windows for jdk images Hi David , yes I think it makes sense to have a configure option for this . Not everyone would like to have a larger JDK (even it is only a bit larger).

RE: RFR: 8237192: Generate stripped/public pdbs on Windows for jdk images

2020-01-23 Thread Baesken, Matthias
feature . > > > > Best regards, Matthias > > > > > >> -Original Message- > >> From: Baesken, Matthias > >> Sent: Dienstag, 21. Januar 2020 11:03 > >> To: 'David Holmes' ; 'build- > >> d...@openjdk.java.net' ; 'hotspot- >

Re: RFR: 8237192: Generate stripped/public pdbs on Windows for jdk images

2020-01-22 Thread Erik Joelsson
e feature . Best regards, Matthias -Original Message- From: Baesken, Matthias Sent: Dienstag, 21. Januar 2020 11:03 To: 'David Holmes' ; 'build- d...@openjdk.java.net' ; 'hotspot- d...@openjdk.java.net' Subject: RE: RFR: 8237192: Generate stripped/public pdbs on Windows for jdk images

RE: RFR: 8237192: Generate stripped/public pdbs on Windows for jdk images

2020-01-22 Thread Baesken, Matthias
ienstag, 21. Januar 2020 11:03 > To: 'David Holmes' ; 'build- > d...@openjdk.java.net' ; 'hotspot- > d...@openjdk.java.net' > Subject: RE: RFR: 8237192: Generate stripped/public pdbs on Windows for > jdk images > > > Hi David , yes I think it makes sense to have a confi

RE: RFR: 8237192: Generate stripped/public pdbs on Windows for jdk images

2020-01-21 Thread Baesken, Matthias
Hi David , yes I think it makes sense to have a configure option for this . Not everyone would like to have a larger JDK (even it is only a bit larger). Best regards, Matthias > > Hi Matthias, > > This also needs to be a configurable option not one done by default as > there can be

Re: RFR: 8237192: Generate stripped/public pdbs on Windows for jdk images

2020-01-17 Thread David Holmes
Hi Matthias, This also needs to be a configurable option not one done by default as there can be non-technical issues relating to shipping symbol files in a product. Thanks, David On 17/01/2020 6:44 pm, Baesken, Matthias wrote: Hello, please review this change related to stripped/"public"

Re: RFR: 8237192: Generate stripped/public pdbs on Windows for jdk images

2020-01-17 Thread Erik Joelsson
Hello Matthias, Providing these stripped pdb files in the distribution is a good idea, but finding a good solution in the build is unfortunately more complicated than this. The JDK image we ship should (with very few exceptions) be the result of running jlink on all the jmods. If a user runs

RE: RFR: 8237192: Generate stripped/public pdbs on Windows for jdk images

2020-01-17 Thread Baesken, Matthias
Hello, my example product build (64 bit Windows / VS2017) shows the following sizes for the uncompressed pdb files : sum of size of all full pdbs : 117 MB (jvm.pdb is 73,1 MB ) sum of size of all stripped pdbs: 18,2 MB (jvm.pdb is 9,46 MB = ~ 50 % of all) Best regards, Matthias On 2020-01-17

Re: RFR: 8237192: Generate stripped/public pdbs on Windows for jdk images

2020-01-17 Thread Magnus Ihse Bursie
On 2020-01-17 09:44, Baesken, Matthias wrote: Hello, please review this change related to stripped/"public" pdb file generation on Windows . Currently the JDK bundle on Windows does not contain pdb files (full pdb files are in a separate symbols bundle). This leads currently to bad native