Re: RFR: JEP 367: Remove the Pack200 Tools and API

2019-12-06 Thread Erik Joelsson
Sounds good and looks good. /Erik On 2019-12-05 20:18, Henry Jen wrote: OK, so I created an issue[1] for follow up for Windows build and reverted the change in flags-cflags.m4, if nothing else, I’ll push without another webrev pinging that I get an +1 from someone in build-de, Erik? [1] http

Re: RFR: JEP 367: Remove the Pack200 Tools and API

2019-12-05 Thread Henry Jen
OK, so I created an issue[1] for follow up for Windows build and reverted the change in flags-cflags.m4, if nothing else, I’ll push without another webrev pinging that I get an +1 from someone in build-de, Erik? [1] https://bugs.openjdk.java.net/browse/JDK-8235461 Cheers, Henry > On Dec 5, 201

Re: RFR: JEP 367: Remove the Pack200 Tools and API

2019-12-05 Thread Mandy Chung
On 12/5/19 12:41 AM, Alan Bateman wrote: On 05/12/2019 08:16, Henry Jen wrote: Hi, Updated webrev[1] reflect comments since last webrev. Vicente had done all the heavy-lifting and hand over to me to finish up. Changes to symbols is reverted, as we expect that only need to be updated in n

Re: RFR: JEP 367: Remove the Pack200 Tools and API

2019-12-05 Thread Alan Bateman
On 05/12/2019 08:16, Henry Jen wrote: Hi, Updated webrev[1] reflect comments since last webrev. Vicente had done all the heavy-lifting and hand over to me to finish up. Changes to symbols is reverted, as we expect that only need to be updated in next release by running make/scripts/generate-s

Re: RFR: JEP 367: Remove the Pack200 Tools and API

2019-12-05 Thread Henry Jen
Hi, Updated webrev[1] reflect comments since last webrev. Vicente had done all the heavy-lifting and hand over to me to finish up. Changes to symbols is reverted, as we expect that only need to be updated in next release by running make/scripts/generate-symbol-data.sh. The jar files are confu

Re: RFR: JEP 367: Remove the Pack200 Tools and API

2019-12-02 Thread Joe Darcy
Hi Vicente, It looks like the update to make/data/symbols/symbols removes the jdk.pack module from the history JDKs 9, 10, and 11 when --release is used. If that is the case, it would be incorrect since historically the jdk.pack module was present in those releases. Thanks, -Joe On 11/22/

Re: RFR: JEP 367: Remove the Pack200 Tools and API

2019-11-25 Thread Mandy Chung
On 11/22/19 1:30 PM, Vicente Romero wrote: [1] http://cr.openjdk.java.net/~vromero/8234542/webrev.03/ make/lib/Lib-jdk.pack.gmk should be removed. make/scripts/compare_exceptions.sh.incl     Should ./lib/libunpack.so also be removed? make/scripts/compare.sh     line 535-543 invokes unpack200.

Re: RFR: JEP 367: Remove the Pack200 Tools and API

2019-11-22 Thread Erik Joelsson
Hello, On 2019-11-22 13:30, Vicente Romero wrote: Hi all, On 11/22/19 3:21 AM, Alan Bateman wrote: On 21/11/2019 19:53, Vicente Romero wrote: Hi, I think I have covered all the proposed fixes so far. This is the last iteration of the webrev [1], all the current changes are in this one, t

Re: RFR: JEP 367: Remove the Pack200 Tools and API

2019-11-22 Thread Vicente Romero
Hi all, On 11/22/19 3:21 AM, Alan Bateman wrote: On 21/11/2019 19:53, Vicente Romero wrote: Hi, I think I have covered all the proposed fixes so far. This is the last iteration of the webrev [1], all the current changes are in this one, the code hasn't been split into different webrevs. I'

Re: RFR: JEP 367: Remove the Pack200 Tools and API

2019-11-22 Thread Alan Bateman
On 21/11/2019 19:53, Vicente Romero wrote: Hi, I think I have covered all the proposed fixes so far. This is the last iteration of the webrev [1], all the current changes are in this one, the code hasn't been split into different webrevs. I'm also forwarding to build-dev as there are some

Re: RFR: JEP 367: Remove the Pack200 Tools and API

2019-11-21 Thread Vicente Romero
Hi Mandy, On 11/21/19 5:45 PM, Mandy Chung wrote: CSR needs to mention that jar -n option is removed.  I made minor edit to the CSR to state that jdk.pack module is removed. thanks for the changes, I added a mention to the jar -n option, Mandy Thanks, Vicente On 11/21/19 2:22 PM, Vicen

Re: RFR: JEP 367: Remove the Pack200 Tools and API

2019-11-21 Thread Mandy Chung
CSR needs to mention that jar -n option is removed.  I made minor edit to the CSR to state that jdk.pack module is removed. Mandy On 11/21/19 2:22 PM, Vicente Romero wrote: please wait, I found some additional dependencies on module jdk.pack, will submit another webrev, sorry Vicente On 11/

Re: RFR: JEP 367: Remove the Pack200 Tools and API

2019-11-21 Thread Vicente Romero
please wait, I found some additional dependencies on module jdk.pack, will submit another webrev, sorry Vicente On 11/21/19 2:53 PM, Vicente Romero wrote: Hi, I think I have covered all the proposed fixes so far. This is the last iteration of the webrev [1], all the current changes are in th

Re: RFR: JEP 367: Remove the Pack200 Tools and API

2019-11-21 Thread Vicente Romero
Hi, I think I have covered all the proposed fixes so far. This is the last iteration of the webrev [1], all the current changes are in this one, the code hasn't been split into different webrevs. I'm also forwarding to build-dev as there are some build related changes too. The CSR for this ch