On Tue, 22 Mar 2022 19:07:12 GMT, Maurizio Cimadamore
wrote:
>> This PR contains the API and implementation changes for JEP-424 [1]. A more
>> detailed description of such changes, to avoid repetitions during the review
>> process, is included as a separate comment.
>>
>> [1] - https://openjd
On Tue, 22 Mar 2022 14:04:07 GMT, Maurizio Cimadamore
wrote:
>> This PR contains the API and implementation changes for JEP-424 [1]. A more
>> detailed description of such changes, to avoid repetitions during the review
>> process, is included as a separate comment.
>>
>> [1] - https://openjd
On Mon, 21 Mar 2022 10:45:27 GMT, Maurizio Cimadamore
wrote:
> This PR contains the API and implementation changes for JEP-424 [1]. A more
> detailed description of such changes, to avoid repetitions during the review
> process, is included as a separate comment.
>
> [1] - https://openjdk.jav
On Mon, 21 Mar 2022 16:29:25 GMT, Magnus Ihse Bursie wrote:
>> A lot (but not all) of the data in make/data is tied to a specific module.
>> For instance, the publicsuffixlist is used by java.base, and fontconfig by
>> java.desktop. (A few directories, like mainmanifest, is *actually* used by
On Thu, 17 Mar 2022 00:12:38 GMT, Magnus Ihse Bursie wrote:
>> A lot (but not all) of the data in make/data is tied to a specific module.
>> For instance, the publicsuffixlist is used by java.base, and fontconfig by
>> java.desktop. (A few directories, like mainmanifest, is *actually* used by
On Sat, 5 Mar 2022 06:49:16 GMT, Julian Waters wrote:
> Should I change the JBS issue title to match the PR title, or is it preferred
> for the PR title to change?
They need to match. You can either do it manually, or change the title to just
the bug number and the bot will change it for you.
On Wed, 22 Dec 2021 01:18:58 GMT, Stuart Marks wrote:
>> Enable the security manager in rmiregistry's launcher arguments.
>
> Stuart Marks has updated the pull request incrementally with one additional
> commit since the last revision:
>
> Change java.security.manager to "allow"; filter warni
On Thu, 2 Dec 2021 18:03:50 GMT, Andrew Leonard wrote:
>> my assumption was the recipe gets resolved later
>
> this was my understanding:
> https://www.gnu.org/software/make/manual/html_node/Variables-in-Recipes.html
>
> This occurs after make has finished reading all the makefiles and the targ
On Thu, 2 Dec 2021 12:13:03 GMT, Andrew Leonard wrote:
>> Addition of a configure option --with-cacerts-src='user cacerts folder' to
>> allow developers to specify their own cacerts PEM folder for generation of
>> the cacerts store using the deterministic openjdk GenerateCacerts tool.
>>
>> Si
On Thu, 14 Oct 2021 13:36:19 GMT, Weijun Wang wrote:
> The cacerts file is now a password-less PKCS12 file. This make sure old code
> that uses a JKS KeyStore object can continuously load it using a null
> password (in fact, any password) and see all certificates inside.
Build change looks goo
On Tue, 12 Oct 2021 11:16:51 GMT, Maurizio Cimadamore
wrote:
> This PR contains the API and implementation changes for JEP-419 [1]. A more
> detailed description of such changes, to avoid repetitions during the review
> process, is included as a separate comment.
>
> [1] - https://openjdk.jav
On Mon, 27 Sep 2021 01:00:18 GMT, Joe Darcy wrote:
> This is an initial PR for expanded lint warnings done under two bugs:
>
> 8202056: Expand serial warning to check for bad overloads of serial-related
> methods and ineffectual fields
> 8160675: Issue lint warning for non-serializable non-tran
On Sun, 22 Aug 2021 23:02:06 GMT, Sergey Bylokhov wrote:
>> This is the continuation of JDK-8233884, JDK-8271456, and JDK-8272120.
>>
>> In many places standard charsets are looked up via their names, for example:
>> absolutePath.getBytes("UTF-8");
>>
>> This could be done more efficiently(up t
On Thu, 3 Jun 2021 16:43:51 GMT, Maurizio Cimadamore
wrote:
>> This patch overhauls the library loading mechanism used by the Foreign
>> Linker API. We realized that, while handy, the *default* lookup abstraction
>> (`LibraryLookup::ofDefault`) was behaving inconsistentlt across platforms.
>>
On Tue, 25 May 2021 18:04:45 GMT, Stuart Marks wrote:
> This is the implementation of [JEP 407](https://openjdk.java.net/jeps/407).
>
> This is a fairly straightforward removal of this component.
> - Activation implementation classes removed
> - Activation tests removed
> - adjustments to var
On Mon, 17 May 2021 18:23:41 GMT, Weijun Wang wrote:
> Please review this implementation of [JEP
> 411](https://openjdk.java.net/jeps/411).
>
> The code change is divided into 3 commits. Please review them one by one.
>
> 1.
> https://github.com/openjdk/jdk/commit/576161d15423f58281e384174d28
On Mon, 22 Mar 2021 12:50:14 GMT, Anton Kozlov wrote:
>> Please review the implementation of JEP 391: macOS/AArch64 Port.
>>
>> It's heavily based on existing ports to linux/aarch64, macos/x86_64, and
>> windows/aarch64.
>>
>> Major changes are in:
>> * src/hotspot/cpu/aarch64: support of the
On 2021-02-26 06:37, daniel.daughe...@oracle.com wrote:
On 2/26/21 7:55 AM, Vladimir Kempik wrote:
On Tue, 2 Feb 2021 23:07:08 GMT, Daniel D. Daugherty
wrote:
Anton Kozlov has updated the pull request incrementally with one
additional commit since the last revision:
support macos_aarc
On Mon, 1 Feb 2021 18:49:04 GMT, djelinski
wrote:
>> Under certain load, MemoryCache operations take a substantial fraction of
>> the time needed to complete SSL handshakes. This series of patches improves
>> performance characteristics of MemoryCache, at the cost of a functional
>> change: e
On Mon, 1 Feb 2021 16:51:12 GMT, Weijun Wang wrote:
>> This fix covers both
>>
>> - [[macOS]: Remove JNF dependency from
>> libosxsecurity/KeystoreImpl.m](https://bugs.openjdk.java.net/browse/JDK-8257858)
>> - [[macOS]: Remove JNF dependency from
>> libosxkrb5/SCDynamicStoreConfig.m](https://b
On Mon, 1 Feb 2021 18:24:46 GMT, djelinski
wrote:
>> make/test/BuildMicrobenchmark.gmk line 97:
>>
>>> 95: SRC := $(MICROBENCHMARK_SRC), \
>>> 96: BIN := $(MICROBENCHMARK_CLASSES), \
>>> 97: JAVAC_FLAGS := $(JAVAC_FLAGS) --add-exports
>>> java.base/sun.security.util=ALL-UNNAMED, \
On Mon, 1 Feb 2021 18:30:14 GMT, Erik Joelsson wrote:
>> I'm trying to benchmark a class that is in a non-exported package
>> `sun.security.util`. Without this line the benchmark doesn't compile. I
>> couldn't find any other benchmarks that access non-exported
On Wed, 27 Jan 2021 11:32:08 GMT, djelinski
wrote:
> Under certain load, MemoryCache operations take a substantial fraction of the
> time needed to complete SSL handshakes. This series of patches improves
> performance characteristics of MemoryCache, at the cost of a functional
> change: expi
On Fri, 18 Dec 2020 19:20:47 GMT, Weijun Wang wrote:
> This fix covers both
>
> - [[macOS]: Remove JNF dependency from
> libosxsecurity/KeystoreImpl.m](https://bugs.openjdk.java.net/browse/JDK-8257858)
> - [[macOS]: Remove JNF dependency from
> libosxkrb5/SCDynamicStoreConfig.m](https://bugs.o
On 2021-01-26 04:44, Magnus Ihse Bursie wrote:
On 2021-01-26 13:09, Vladimir Kempik wrote:
On Tue, 26 Jan 2021 12:02:02 GMT, Alan Hayward
wrote:
AIUI, the configure line needs passing a prebuilt
JavaNativeFoundation framework
ie:
`--with-extra-ldflags='-F
/Applications/Xcode.app/Contents
On Fri, 22 Jan 2021 18:49:42 GMT, Anton Kozlov wrote:
> Please review the implementation of JEP 391: macOS/AArch64 Port.
>
> It's heavily based on existing ports to linux/aarch64, macos/x86_64, and
> windows/aarch64.
>
> Major changes are in:
> * src/hotspot/cpu/aarch64: support of the new ca
On Thu, 14 Jan 2021 06:32:37 GMT, Jamil Nimeh wrote:
> This is the security libs portion of the effort to replace
> archaic/non-inclusive words with more neutral terms (see JDK-8253315 for
> details).
>
> Here are the changes covering core libraries code and tests. Terms were
> changed as fol
On Tue, 15 Dec 2020 22:56:15 GMT, Magnus Ihse Bursie wrote:
>> A lot (but not all) of the data in make/data is tied to a specific module.
>> For instance, the publicsuffixlist is used by java.base, and fontconfig by
>> java.desktop. (A few directories, like mainmanifest, is *actually* used by
On 2020-12-08 00:30, Magnus Ihse Bursie wrote:
On Tue, 8 Dec 2020 02:40:43 GMT, Mandy Chung wrote:
I have reviewed all lines in the patch file with or near instances of
`jdk.compiler`
Hi Magnus,
I see the motivation of moving these build files for better identification of
ownership. Pl
On Fri, 4 Dec 2020 12:30:02 GMT, Alan Bateman wrote:
>> And I can certainly move jdwp.spec to java.base instead. That's the reason I
>> need input on this: All I know is that is definitely not the responsibility
>> of the Build Group to maintain that document, and I made my best guess at
>> wh
On Fri, 4 Dec 2020 14:03:08 GMT, Erik Joelsson wrote:
>>> And I can certainly move jdwp.spec to java.base instead.
>>
>> If jdwp.spec has to move to the src tree then src/java.se is probably the
>> most suitable home because Java SE specifies JDWP as an optional i
On Tue, 17 Nov 2020 19:58:47 GMT, Jim Laskey wrote:
> This PR is to introduce a new random number API for the JDK. The primary API
> is found in RandomGenerator and RandomGeneratorFactory. Further description
> can be found in the JEP https://openjdk.java.net/jeps/356 .
It looks like you have
On Thu, 8 Oct 2020 13:59:20 GMT, Maurizio Cimadamore
wrote:
>> This patch contains the changes associated with the third incubation round
>> of the foreign memory access API incubation
>> (see JEP 393 [1]). This iteration focus on improving the usability of the
>> API in 3 main ways:
>> * firs
On Thu, 8 Oct 2020 10:29:24 GMT, Maurizio Cimadamore
wrote:
>> This patch contains the changes associated with the third incubation round
>> of the foreign memory access API incubation
>> (see JEP 393 [1]). This iteration focus on improving the usability of the
>> API in 3 main ways:
>> * firs
On Wed, 7 Oct 2020 17:13:22 GMT, Maurizio Cimadamore
wrote:
> This patch contains the changes associated with the third incubation round of
> the foreign memory access API incubation
> (see JEP 393 [1]). This iteration focus on improving the usability of the API
> in 3 main ways:
> * first, by
On Tue, 22 Sep 2020 00:18:07 GMT, Anthony Scarpino
wrote:
>> This change removes the native elliptic curves library code; as well as, and
>> calls to that code, tests, and files
>> associated with those libraries. The makefiles have been changed to remove
>> from all source builds of the ec c
On Mon, 21 Sep 2020 21:10:58 GMT, Anthony Scarpino
wrote:
> This change removes the native elliptic curves library code; as well as, and
> calls to that code, tests, and files
> associated with those libraries. The makefiles have been changed to remove
> from all source builds of the ec code.
On Mon, 21 Sep 2020 18:17:55 GMT, Yumin Qi wrote:
>> With more CDS related code added to VM, it is time to move CDS code to a
>> separate class. CDS is the new class which is
>> specific to CDS.
>> Tests: tier1-4
>
> Yumin Qi has updated the pull request incrementally with one additional
> comm
Build changes look ok.
/Erik
On 2020-02-29 05:02, Weijun Wang wrote:
Please take a review at
https://cr.openjdk.java.net/~weijun/8240261/webrev.00
Thanks,
Max
On 2019-11-26 16:39, Weijun Wang wrote:
On Nov 27, 2019, at 12:14 AM, Erik Joelsson wrote:
On 2019-11-25 16:42, Weijun Wang wrote:
On Nov 26, 2019, at 12:36 AM, Erik Joelsson wrote:
Build change looks good.
Thanks.
One question: I see the output to stdout from FieldGen.java shown in
On 2019-11-25 16:42, Weijun Wang wrote:
On Nov 26, 2019, at 12:36 AM, Erik Joelsson wrote:
Build change looks good.
Thanks.
One question: I see the output to stdout from FieldGen.java shown in build. Is
it possible to hide it but still store the output in the log file?
No, we are not
Build change looks good.
/Erik
On 2019-11-22 18:59, Weijun Wang wrote:
Please review the change at
http://cr.openjdk.java.net/~weijun/8234697/webrev.00/
The new lines in Gensrc-java.base.gmk mimics the one for CLDR_GEN_DONE at the
beginning of the same file.
I changed the BigInteger par
patch.
Hope you wouldn't mind it. Thanks.
Could you please sponsor it?
Thanks a lot.
Best regards,
Jia
在 2019年09月23日 23:18, Erik Joelsson 写道:
I think this type of comment fits well in the top level test doc. It
just provides basic instructions for setting up these tests so that
they pass wi
I think this type of comment fits well in the top level test doc. It
just provides basic instructions for setting up these tests so that they
pass without going into too much detail. Perhaps a reference to the
pkcs11 README for more details would be a good idea.
Looks good to me.
/Erik
On 20
I'm happy with this.
/Erik
On 2019-06-14 08:33, Weijun Wang wrote:
Here is the updated webrev
http://cr.openjdk.java.net/~weijun/8225392/webrev.01/
The only change is ordering in 'keytool -list' and its test.
Thanks,
Max
On Jun 14, 2019, at 7:55 PM, Sean Mullan wrote:
On 6/14/19 1:49
e not integrated
into the JDK until after they were created by the CA. Can we somehow
extract the last revision time of each PEM file instead? That is more
aligned with the previous creation date that we used.
--Sean
On 6/12/19 12:38 PM, Erik Joelsson wrote:
Hello Max,
Much appreciated! I
timestamp.
On Mon, Jun 10, 2019 at 12:40 PM Erik Joelsson wrote:
Since JDK-8193255, when we started generating the cacerts file in the
build, the build compare baseline builds have started failing. It seems
the cacerts binary file has some non determinism built in so it doesn't
get genera
Hello Max,
I believe $(call FindSrcDirsForLib, $(MODULE), sspi_bridge) for SRC is
the default so you shouldn't need to specify it explicitly, or did you
experience some problems with the default that I'm missing?
/Erik
On 2019-06-09 19:30, Weijun Wang wrote:
Updated webrev at
http://cr
, BUILD_TOOLS_JDK, MakeTargetDir, files instead of dir
Thanks,
Max
On May 31, 2019, at 1:34 AM, Erik Joelsson wrote:
On 2019-05-30 08:32, Weijun Wang wrote:
On May 30, 2019, at 10:01 PM, Erik Joelsson wrote:
In my experience, using directories for dependencies in make does not work
well. Since all
On 2019-05-30 08:32, Weijun Wang wrote:
On May 30, 2019, at 10:01 PM, Erik Joelsson wrote:
In my experience, using directories for dependencies in make does not work
well. Since all the files in make/data/cacerts are in a flat structure, I would
recommend expressing the prerequisites as
Hello Max,
Looking in ToolsJdk.gmk, I realize that the BUILD_TOOLS variable was
renamed back when we unified the repositories and is now called
BUILD_TOOLS_JDK. It seems like I missed updating the references to this
variable in the gendata dir. If you use the new variable name in the
prerequi
Build change looks good.
/Erik
On 2018-06-21 04:12, Weijun Wang wrote:
Please take a review on this change
http://cr.openjdk.java.net/~weijun/8205445/webrev.00/
and the release note at
https://bugs.openjdk.java.net/browse/JDK-8205471
The code change adds RSASSA-PSS signature sup
Looks good.
/Erik
On 2018-06-08 01:50, Magnus Ihse Bursie wrote:
On 2018-06-07 23:20, Erik Joelsson wrote:
Hello Magnus,
Very nice refactoring!
Thanks!
JdkNativeCompilation.gmk
line 126-127 looks a bit long. There is an extra space on 126. Also,
why not addprefix for adding -I instead
Hello Magnus,
Very nice refactoring!
JdkNativeCompilation.gmk
line 126-127 looks a bit long. There is an extra space on 126. Also, why
not addprefix for adding -I instead of clunky foreach? Not that I care
greatly, but I usually prefer that construct.
Otherwise looks good.
/Erik
On 2018-0
ata/Gendata-java.base.gmk
59-$(MV) $@.tmp $@
60:$(CHMOD) 444 $@
61-
common/JavaCompilation.gmk
80-$(CP) $$< $$@
81:$(CHMOD) -f ug+w $$@
Thanks
Max
/Magnus
22 maj 2018 kl. 17:44 skrev Erik Joelsson :
Build changes look ok.
/Erik
On 2018-05-22 08:25, Weijun Wang wr
Build changes look ok.
/Erik
On 2018-05-22 08:25, Weijun Wang wrote:
Please take a review at
http://cr.openjdk.java.net/~weijun/8201815/webrev.00/
With this change, We switch from a home-grown public suffix list (implemented
in sun/net/RegisteredDomain.java) to Mozilla's PSL. The PSL da
Build changes still look good to me.
/Erik
On 2018-03-28 03:31, Magnus Ihse Bursie wrote:
On 2018-03-28 01:52, Weijun Wang wrote:
On Mar 24, 2018, at 6:03 AM, Magnus Ihse Bursie
wrote:
https://bugs.openjdk.java.net/browse/JDK-8200193 -- for
jdk.security.auth
There is only one function t
I have looked at the build changes and they look good.
Will you file followups for each component team to look over their
exported symbols, at least for the libraries with $(EXPORT_ALL_SYMBOLS)?
It sure looks like there is some technical debt laying around here.
/Erik
On 2018-03-23 06:56, M
Looks good.
/Erik
On 2018-03-14 16:18, Magnus Ihse Bursie wrote:
BUILD_LIBKRB5 is currently a strange chimera between compiling
w2k_lsa_auth on windows, and osxkrb5 on macos. They do not share
source code, name or compilation options.
This patch separates them into two separate compilations
Looks good.
/Erik
On 2018-03-14 14:50, Magnus Ihse Bursie wrote:
For some odd reason, the native library compiled for jdk.security.auth
is called jaas_unix on unix and jaas_nt on windows. There's no good
reason for this, and it breaks with the common practice in OpenJDK.
This patch renames
From a build point of view this looks good.
/Erik
On 2017-09-06 06:17, Weijun Wang wrote:
Hi All
Please review the change, which spans to root, jdk and langtools repos.
http://cr.openjdk.java.net/~weijun/8148371/
I've searched for the "policytool" word in the whole jdk10/jdk10 forests,
Nah, it's fine.
/Erik
On 2017-05-09 04:37, Magnus Ihse Bursie wrote:
On 2017-05-05 17:52, Erik Joelsson wrote:
What's the reason for adding the conditional around SetupCopyFiles in
Javadoc.gmk? The SetupCopyFiles macro should be safe to call with an
empty FILES list. It should e
What's the reason for adding the conditional around SetupCopyFiles in
Javadoc.gmk? The SetupCopyFiles macro should be safe to call with an
empty FILES list. It should evaluate to basically nothing in that case.
/Erik
On 2017-05-05 06:17, Magnus Ihse Bursie wrote:
The Security Standard Names d
Looks good.
/Erik
On 2016-12-03 00:50, Bradford Wetmore wrote:
Hi,
I need reviewers for these related bugs:
https://bugs.openjdk.java.net/browse/JDK-8170157
Enable unlimited cryptographic policy by default in OracleJDK
https://bugs.openjdk.java.net/browse/JDK-8169335
Add a crypto policy fa
Build changes look ok to me.
/Erik
On 2016-11-04 14:42, Seán Coffey wrote:
Looking to push this enhancement to jdk8u. The change introduces the
new Security property which was brought into JDK 9 via JDK-8061842.
The code differs in that jar files continue to be used and backwards
compatibil
Looks ok to me.
/Erik
On 2016-08-25 23:20, Anthony Scarpino wrote:
Hi,
Can I get a review of this change to remove the warning suppression
and fix the minor compiler issues that it was hiding in the pkcs11
wrapper library.
http://cr.openjdk.java.net/~ascarpino/8074838/webrev/
thanks
Tony
On 2016-08-17 18:43, Sean Mullan wrote:
On 8/17/2016 12:33 PM, Erik Joelsson wrote:
Hello Sean,
The change looks ok, but it could also be expressed like this to
avoid duplication:
ifneq ($(filter $(OPENJDK_TARGET_OS), windows solaris), )
DEF_POLICY_SRC_LIST +=
$(JDK_TOPDIR)/src
Hello Sean,
The change looks ok, but it could also be expressed like this to avoid
duplication:
ifneq ($(filter $(OPENJDK_TARGET_OS), windows solaris), )
DEF_POLICY_SRC_LIST +=
$(JDK_TOPDIR)/src/java.base/$(OPENJDK_TARGET_OS)/lib/security/default.policy
endif
/Erik
On 2016-08-17 18:18, S
Build change looks fine.
/Erik
On 2016-08-08 15:46, Weijun Wang wrote:
Please review the code changes at
http://cr.openjdk.java.net/~weijun/8162739/webrev.00/
A new -cacerts option is added to keytool so there is no need to
provide the full cacerts path on the command line.
lib/security
Much better, and thank you for fixing the existing mkdir/echo lines too.
Just one nit, for this continuation:
$(TOOL_CACERTSHASHER) -i $(GENDATA_CACERTSHASHER_IN) \
-o $(GENDATA_CACERTSHASHER)
please use tab+4spaces for the second line. No need to resend webrev for
that. See [
Makefile change looks ok.
/Erik
On 2015-10-02 19:08, Anthony Scarpino wrote:
Hi all,
I'm need a review of the last developement piece to JEP 246, the
configuration changes.
I've copied the build-dev in case there were any comments on the minor
changes in the make directory related to the j
One is enough.
/Erik
On 2015-08-18 02:21, Valerie Peng wrote:
Thanks for the review.
Is one more reviewer from build team needed?
Valerie
On 8/14/2015 4:58 PM, Mandy Chung wrote:
Looks good.
Mandy
On Aug 14, 2015, at 4:30 PM, Valerie Peng
wrote:
Updated the webrev in place to use "osxse
Build changes still look good.
/Erik
On 2015-08-15 01:30, Valerie Peng wrote:
Updated the webrev in place to use "osxsecurity" given peer feedbacks.
Thanks,
Valerie
On 8/13/2015 7:31 PM, Valerie Peng wrote:
Can someone please help reviewing this change?
This is to move Apple provider from j
Build changes look good.
/Erik
On 2015-08-14 04:31, Valerie Peng wrote:
Can someone please help reviewing this change?
This is to move Apple provider from jdk.deploy.osx module to java.base
module.
The native library for Apple provider is separated out from the "osx"
one generated in jdk.dep
Hello Valerie,
To reduce duplication, I think something like this would be preferable.
Remove the windows and macosx checks and just always run this line:
POLICY_SRC_LIST += \
$(wildcard
$(JDK_TOPDIR)/src/java.base/$(OPENJDK_TARGET_OS)/conf/security/java.policy)
That way, I platform spe
Hello Valerie,
The merging seems ok, but I thought there was non determinism in the
image builder regarding which provider would get picked up. Is that
resolved or do you really need to override all of those providers with
your generated file in gensrc? I can assist in writing that makefile
l
On 2015-05-22 18:53, Mandy Chung wrote:
On 05/22/2015 08:09 AM, Alan Bateman wrote:
On 22/05/2015 13:55, Chris Hegarty wrote:
:
I think it could be done either way.
Valerie - have you considered not pushing the services configuration
files with this change? With the change then the java.
On 2015-05-22 02:46, Mandy Chung wrote:
I’m including build-dev and we need to ask for Erik and Magnus advice what’s
the best way to work around this.
Erik, Magnus,
Security providers now become service providers. They are provided from 11
different modules, 3 of them are os-specific. T
Looks good to me.
/Erik
On 2015-04-17 14:52, Magnus Ihse Bursie wrote:
With JDK-8074096, the number of warnings in the product was reduced to
a minimum. This enables the next step, which is turning on the
respective compiler flags that turns warnings into errors. In the long
run, this is the
Looks good to me. Thanks for fixing warnings!
/Erik
On 2015-03-16 04:12, Wang Weijun wrote:
Hi All
Please review the change at
http://cr.openjdk.java.net/~weijun/8074836/webrev.00/
Thanks
Max
Thanks, looks good!
/Erik
On 2015-03-06 17:14, Magnus Ihse Bursie wrote:
On 2015-03-04 14:31, Erik Joelsson wrote:
Hello,
Really nice to finally see this patch getting done!
Only one comment:
flags.m4:
In the grep expression, could you move the extra [] outside of the
actual command line
Hello Mandy,
The build changes look ok to me.
/Erik
On 2015-03-05 02:13, Mandy Chung wrote:
As listed in an open issue in JEP 200:
The jdk.dev and jdk.runtime modules contain miscellaneous tools that do
not obviously belong to any other module; these modules will eventually
be either renamed
Hello,
Really nice to finally see this patch getting done!
Only one comment:
flags.m4:
In the grep expression, could you move the extra [] outside of the
actual command line options to grep so that the command line could be
copied to the shell for debugging in the future? Also, how hard would
Hello,
Please review the open part of this patch, which changes the building of
policy jars to happen even if BUILD_CRYPTO is false. Previously these
weren't built as there were signed versions of these jars, but since we
no longer sign them, there is no need to not build them.
Bug: https://
Change looks good to me.
/Erik
On 2014-09-10 18:28, Valerie Peng wrote:
Could someone please review this build related change for moving
sunpkcs11-solaris.cfg file to the pkcs11 module?
Webrev: http://cr.openjdk.java.net/~valeriep/8039898/webrev.00/
Thanks,
Valerie
On 2014-08-06 11:14, Weijun Wang wrote:
On 8/6/2014 17:04, Erik Joelsson wrote:
Speaking of indentation, please also change the ifdef bodies to 2
spaces.
Sure. Whenever I edit a Makefile, I dare not use spaces and always use
TABs. Obviously an ifdef does not need it.
The tab character
this
provider if testing an OpenJDK build.
--Sean
Thanks
Max
On Jul 28, 2014, at 19:43, Erik Joelsson
wrote:
Hello Max,
Shouldn't the rule for $(GENDATA_JAVA_SECURITY) depend on
$(RESTRICTED_PKGS_SRC) so that updates to the pkgs file triggers a
rebuild? For that to work, th
Build change looks good to me now.
/Erik
On 2014-07-28 15:53, Wang Weijun wrote:
Yes, you are right.
Webrev updated at http://cr.openjdk.java.net/~weijun/6997010/webrev.02.
GendataJavaSecurity.gmk and MakeJavaSecurity.java updated.
Thanks
Max
On Jul 28, 2014, at 19:43, Erik Joelsson wrote
Hello Max,
Shouldn't the rule for $(GENDATA_JAVA_SECURITY) depend on
$(RESTRICTED_PKGS_SRC) so that updates to the pkgs file triggers a
rebuild? For that to work, the variable $(RESTRICTED_PKGS_SRC) needs to
be empty for the OPENJDK case rather than have a dummy name and
MakeJavaSecurity.java
Hello Mandy,
The logic looks fine. Just some style issues. I would like indentation
for the conditionals to be 2 spaces as is currently the standard in the
makefiles. I would also like to have POLICY_SRC_LIST to be declared
empty with := instead of just =. We only use = assignment when
explic
Changeset: 7238a870ddb7
Author:erikj
Date: 2014-01-24 10:39 +0100
URL: http://hg.openjdk.java.net/jdk8/tl/rev/7238a870ddb7
8032632: Wrong version for the first jdk8 fcs build
Reviewed-by: katleman
! common/autoconf/spec.gmk.in
Changeset: ff56039c4870
Author:erikj
Date: 2014-01-22 12:13 +0100
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/ff56039c4870
8032217: failure in man page processing
Reviewed-by: dholmes, tbell
! make/Images.gmk
Changeset: 6f3a3bd78c57
Author:erikj
Date: 2014-01-10 10:25 +0100
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/6f3a3bd78c57
8031300: No jdeps.1 and jjs.1 man pages in jdk8 b122 build and jvisualvm.1 and
jcmd.1 missing on macosx
8030946: No jmc.1 for man page of JMC
Reviewed-by:
Changeset: 2437ccbf3504
Author:erikj
Date: 2014-01-08 14:04 +0100
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/2437ccbf3504
8030781: System.setProperties(null) drops all system properties (RELEASE not
set)
Reviewed-by: alanb
+ test/java/lang/System/SetPropertiesNull.java
Changeset: 53d74b77ee53
Author:erikj
Date: 2014-01-08 14:02 +0100
URL: http://hg.openjdk.java.net/jdk8/tl/rev/53d74b77ee53
8030781: System.setProperties(null) drops all system properties (RELEASE not
set)
Reviewed-by: alanb, ihse, tbell
! common/autoconf/generated-configure.sh
! c
Changeset: 427c78c88229
Author:erikj
Date: 2013-12-05 09:25 +0100
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/427c78c88229
8027963: Create unlimited policy jars.
Reviewed-by: wetmore, ihse
! make/CreateSecurityJars.gmk
! make/SignJars.gmk
- make/data/cryptopolicy/limited/LIMITE
Changeset: c009462c1e92
Author:erikj
Date: 2013-12-04 12:45 +0100
URL: http://hg.openjdk.java.net/jdk8/tl/rev/c009462c1e92
8027963: Create unlimited policy jars.
Reviewed-by: wetmore, ihse
! common/autoconf/spec.gmk.in
Changeset: 76a7c0bc74fd
Author:erikj
Date: 2013-10-16 13:50 +0200
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/76a7c0bc74fd
6604021: RMIC is defaulting to BOOT jdk version, needs to be rmic.jar
Reviewed-by: dholmes, chegar
! makefiles/GendataBreakIterator.gmk
! makefiles/Generat
Changeset: 438c54c148a6
Author:erikj
Date: 2013-10-16 13:49 +0200
URL: http://hg.openjdk.java.net/jdk8/tl/corba/rev/438c54c148a6
6604021: RMIC is defaulting to BOOT jdk version, needs to be rmic.jar
Reviewed-by: dholmes, chegar
! makefiles/BuildCorba.gmk
Changeset: af81988013b5
Author:erikj
Date: 2013-10-16 13:50 +0200
URL: http://hg.openjdk.java.net/jdk8/tl/rev/af81988013b5
6604021: RMIC is defaulting to BOOT jdk version, needs to be rmic.jar
Reviewed-by: dholmes, chegar
! common/makefiles/JavaCompilation.gmk
! common/makefiles/RM
1 - 100 of 125 matches
Mail list logo