On 4/29/20 5:12 AM, Mikael Vidstedt wrote:
> Please review this small change which disables JVMCI, Graal, and AOT when
> building linux-aarch64 at Oracle, for now.
>
> JBS: https://bugs.openjdk.java.net/browse/JDK-8244061
> webrev:
> http://cr.openjdk.java.net/~mikael/webrevs/8244061/webrev.00/o
On 2020-04-29 06:12, Mikael Vidstedt wrote:
Please review this small change which disables JVMCI, Graal, and AOT when
building linux-aarch64 at Oracle, for now.
JBS: https://bugs.openjdk.java.net/browse/JDK-8244061
webrev:
http://cr.openjdk.java.net/~mikael/webrevs/8244061/webrev.00/open/we
If I understand correctly, this disables building jvmci/graal/aot in *any*
linux-aarch64 jib based build, not just those done at Oracle. Wouldn’t this be
better done on the jib command line?
-Doug
> On 29 Apr 2020, at 06:12, Mikael Vidstedt wrote:
>
>
> Please review this small change which
JIB is an Oracle (internal) tool, meaning the jib-profiles.js configuration
file is only relevant at Oracle, so this really only affects builds done at
Oracle. Does that address your concern?
Cheers,
Mikael
> On Apr 29, 2020, at 1:45 AM, Doug Simon wrote:
>
> If I understand correctly, this
> On Apr 29, 2020, at 12:46 AM, Andrew Haley wrote:
>
> On 4/29/20 5:12 AM, Mikael Vidstedt wrote:
>> Please review this small change which disables JVMCI, Graal, and AOT when
>> building linux-aarch64 at Oracle, for now.
>>
>> JBS: https://bugs.openjdk.java.net/browse/JDK-8244061
>> webrev:
On 2020-04-29 10:45, Doug Simon wrote:
If I understand correctly, this disables building jvmci/graal/aot in *any*
linux-aarch64 jib based build, not just those done at Oracle. Wouldn’t this be
better done on the jib command line?
While this is technically correct, the jib tool is only used at O
Ok, thanks for the clarification. I wasn’t sure if external parties had managed
to set up a jib server but it sounds like that’s not so easy in practice.
-Doug
> On 29 Apr 2020, at 11:02, Magnus Ihse Bursie
> wrote:
>
> On 2020-04-29 10:45, Doug Simon wrote:
>> If I understand correctly, this
Due to the current design of the AddPackagesAttribute build tool, all
module-info.class files will be written to, when the tool is run. This
happens whenever at least one module-info.class file has been updated.
The end result of this is that if you recompile a single module, *all*
modules will
On 29/04/2020 10:27, Magnus Ihse Bursie wrote:
Due to the current design of the AddPackagesAttribute build tool, all
module-info.class files will be written to, when the tool is run. This
happens whenever at least one module-info.class file has been updated.
The end result of this is that if
The IDE support in OpenJDK unfortunately leaves a lot to be desired.
There have been a garden variety of attempt to support a specific IDE
for a specific part of the code base, cluttered all over the code base.
This patch is a first attempt go get one ring, eh..., structure, to rule
them all.
On 2020-04-29 12:31, Alan Bateman wrote:
On 29/04/2020 10:27, Magnus Ihse Bursie wrote:
Due to the current design of the AddPackagesAttribute build tool, all
module-info.class files will be written to, when the tool is run.
This happens whenever at least one module-info.class file has been
u
On 4/29/20 10:02 AM, Mikael Vidstedt wrote:
>
>> On Apr 29, 2020, at 12:46 AM, Andrew Haley wrote:
>>
>> On 4/29/20 5:12 AM, Mikael Vidstedt wrote:
>>> Please review this small change which disables JVMCI, Graal, and AOT when
>>> building linux-aarch64 at Oracle, for now.
>>>
>>> JBS: https://bu
On 29/04/2020 11:40, Magnus Ihse Bursie wrote:
:
http://cr.openjdk.java.net/~ihse/JDK-8243665-fix-AddPackagesAttribute/webrev.02
Looks good. It could be optimized to only read the module-info.class
once but will hardly be noticeable I suspect.
-Alan
Didn't review in detail, but some pieces seems suspicious - e.g.
moving "/make/langtools/build.properties" to
"///make/ide/idea/langtools/build.properties//
Note that langtools's build.properties is used by the langtools ANT
build which is independent from make - and also mostly independent f
On 2020-04-29 12:47, Alan Bateman wrote:
On 29/04/2020 11:40, Magnus Ihse Bursie wrote:
:
http://cr.openjdk.java.net/~ihse/JDK-8243665-fix-AddPackagesAttribute/webrev.02
Looks good.
Thanks.
It could be optimized to only read the module-info.class once but will
hardly be noticeable I s
On 2020-04-29 12:47, Maurizio Cimadamore wrote:
Didn't review in detail, but some pieces seems suspicious - e.g.
moving "/make/langtools/build.properties" to
"///make/ide/idea/langtools/build.properties//
Note that langtools's build.properties is used by the langtools ANT
build which is ind
Hi all,
May I get reviews for this fix?
JBS: https://bugs.openjdk.java.net/browse/JDK-8244097
Webrev: http://cr.openjdk.java.net/~jiefu/8244097/webrev.00/
Thanks a lot.
Best regards,
Jie
I am not sure if anyone is still using make/jdk/netbeans. Apache
NetBeans does not (should not) need these config files, it supports
OpenJDK modules out of the box (with some tweaks/dependencies on
make/langtools/** to speed up langtools build).
As Maurizio, there may be some need to move the
On 2020-04-29 13:02, jiefu(傅杰) wrote:
Hi all,
May I get reviews for this fix?
JBS: https://bugs.openjdk.java.net/browse/JDK-8244097
Dang it! I thought I had tested bootcycle-images, but maybe that wasn't
the last iteration of my fix.
Webrev: http://cr.openjdk.java.net/~jiefu/8244097/webrev.00
On 29/04/2020 11:57, Magnus Ihse Bursie wrote:
So who/what is using the ant build? I thought it was just the IDEA
project that generated ant targets. Is it run explicitly using ant by
langtool developers? I know for sure it's not used when we build the
product in our CI.
As Jan mentioned, t
On 2020-04-29 13:36, Maurizio Cimadamore wrote:
On 29/04/2020 11:57, Magnus Ihse Bursie wrote:
So who/what is using the ant build? I thought it was just the IDEA
project that generated ant targets. Is it run explicitly using ant by
langtool developers? I know for sure it's not used when we bui
On 2020-04-29 13:23, Magnus Ihse Bursie wrote:
On 2020-04-29 13:02, jiefu(傅杰) wrote:
Hi all,
May I get reviews for this fix?
JBS: https://bugs.openjdk.java.net/browse/JDK-8244097
Dang it! I thought I had tested bootcycle-images, but maybe that
wasn't the last iteration of my fix.
Webrev: htt
On 2020-04-29 13:06, Jan Lahoda wrote:
I am not sure if anyone is still using make/jdk/netbeans. Apache
NetBeans does not (should not) need these config files, it supports
OpenJDK modules out of the box (with some tweaks/dependencies on
make/langtools/** to speed up langtools build).
It's by fa
Thanks Magnus for your review and nice help.
It seems that your patch didn't fix the build failure when configure
--with-boot-jdk=jdk15.
I've made : http://cr.openjdk.java.net/~jiefu/8244097/webrev.01/ based on your
work.
Please review it and give me some advice.
Thanks.
Best regards,
Jie
O
Looks good.
/Erik
On 2020-04-28 21:12, Mikael Vidstedt wrote:
Please review this small change which disables JVMCI, Graal, and AOT when
building linux-aarch64 at Oracle, for now.
JBS: https://bugs.openjdk.java.net/browse/JDK-8244061
webrev:
http://cr.openjdk.java.net/~mikael/webrevs/8244061/
Looks good to me.
/Erik
On 2020-04-29 03:47, Alan Bateman wrote:
On 29/04/2020 11:40, Magnus Ihse Bursie wrote:
:
http://cr.openjdk.java.net/~ihse/JDK-8243665-fix-AddPackagesAttribute/webrev.02
Looks good. It could be optimized to only read the module-info.class
once but will hardly be n
> On 29 Apr 2020, at 11:36, Magnus Ihse Bursie
> wrote:
>
> ...
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8244093
> WebRev:
> http://cr.openjdk.java.net/~ihse/JDK-8244093-move-ide-support/webrev.01
The make/idea changes look ok to me.
-Chris.
On 2020-04-29 04:48, Magnus Ihse Bursie wrote:
On 2020-04-29 13:23, Magnus Ihse Bursie wrote:
On 2020-04-29 13:02, jiefu(傅杰) wrote:
Hi all,
May I get reviews for this fix?
JBS: https://bugs.openjdk.java.net/browse/JDK-8244097
Dang it! I thought I had tested bootcycle-images, but maybe that
w
On 2020-04-29 05:25, jiefu(傅杰) wrote:
Thanks Magnus for your review and nice help.
It seems that your patch didn't fix the build failure when configure
--with-boot-jdk=jdk15.
I've made : http://cr.openjdk.java.net/~jiefu/8244097/webrev.01/ based on your
work.
Ah, you already discovered thi
If this works both when building normally and when building bootcycle-images,
I’m okay with this.
Are you a committer, or do you need someone to sponsor the patch?
/Magnus
> 29 apr. 2020 kl. 14:25 skrev jiefu(傅杰) :
>
> Thanks Magnus for your review and nice help.
> It seems that your patch di
Thanks Erik and Magnus for your review.
I'll push it soon.
On 2020/4/29, 9:33 PM, "Magnus Ihse Bursie"
wrote:
If this works both when building normally and when building
bootcycle-images, I’m okay with this.
Are you a committer, or do you need someone to sponsor the patch?
Hello,
My fix for JDK-8243510 was not enough to catch all potential exceptions
needed. Here is a new attempt that casts a wider net.
Webrev: http://cr.openjdk.java.net/~erikj/8244051/webrev.01/
Bug: https://bugs.openjdk.java.net/browse/JDK-8244051
/Erik
LGTM.
/Magnus
> 29 apr. 2020 kl. 17:23 skrev Erik Joelsson :
>
> Hello,
>
> My fix for JDK-8243510 was not enough to catch all potential exceptions
> needed. Here is a new attempt that casts a wider net.
>
> Webrev: http://cr.openjdk.java.net/~erikj/8244051/webrev.01/
>
> Bug: https://bugs.
On 29/04/2020 12:45, Magnus Ihse Bursie wrote:
Let's say we keep it in langtools (unless we can find a better home
for it). My question is: Are we talking only about build.properties
and build.xml? Or are there more files that these depend on, that I
had assumed belong to the IDE generation?
Jan,
What is your current recommended technique to use NetBeans to
build/edit/test OpenJDK for normal OpenJDK library developers?
After many versions of Netbeans, my current setup is to:
1. Do an external "exploded" build
2. Run Netbeans, open the src/java.base module project and any other
> On Apr 29, 2020, at 3:44 AM, Andrew Haley wrote:
>
> On 4/29/20 10:02 AM, Mikael Vidstedt wrote:
>>
>>> On Apr 29, 2020, at 12:46 AM, Andrew Haley wrote:
>>>
>>> On 4/29/20 5:12 AM, Mikael Vidstedt wrote:
Please review this small change which disables JVMCI, Graal, and AOT when
Vladimir/Magnus/Erik,
Thanks for the reviews, change pushed.
Cheers,
Mikael
> On Apr 29, 2020, at 5:45 AM, Erik Joelsson wrote:
>
> Looks good.
>
> /Erik
>
> On 2020-04-28 21:12, Mikael Vidstedt wrote:
>> Please review this small change which disables JVMCI, Graal, and AOT when
>> buildin
37 matches
Mail list logo