Re: [aspectj-users] Java 9 sun.boot.class.path gone

2016-07-22 Thread Andy Clement
I thought the fix for this was already in (a variant of it) - I was doing some 
work on 1.9.0 BETA4 recently. This included picking up all the latest Eclipse 
JDT support for Java9 and coping with the changes to paths. But I will admit I 
was working with a 121 build version of Java9, perhaps things have changed 
again since then. I’ll have a look when I get a moment.

cheers,
Andy
 
> On Jul 22, 2016, at 11:31 AM, Mario Ivankovits  wrote:
> 
> Hi!
> 
> In latest Java 9 builds the „sun.boot.class.path“ is gone. Due to this, the 
> core java classes can no longer be found by aspectj.
> On AspectJs earlyJava9 branch I’ve patched it like this to make that work 
> again.
> 
> 
> Index: weaver/src/org/aspectj/weaver/bcel/ClassPathManager.java
> ===
> --- weaver/src/org/aspectj/weaver/bcel/ClassPathManager.java(revision 
> f8b86ff2c03a77e47e87573b59bc43c57cfdee38)
> +++ weaver/src/org/aspectj/weaver/bcel/ClassPathManager.java(revision )
> @@ -38,6 +38,7 @@
> 
>  import org.aspectj.bridge.IMessageHandler;
>  import org.aspectj.bridge.MessageUtil;
> +import org.aspectj.util.LangUtil;
>  import org.aspectj.weaver.BCException;
>  import org.aspectj.weaver.UnresolvedType;
>  import org.aspectj.weaver.WeaverMessages;
> @@ -70,6 +71,11 @@
> if (trace.isTraceEnabled())
> trace.enter("", this, new Object[] { classpath, 
> handler });
> entries = new ArrayList();
> +
> +   if (LangUtil.is19VMOrGreater()) {
> +   entries.add(new JImageEntry(null));
> +   }
> +
> for (Iterator i = classpath.iterator(); i.hasNext();) 
> {
> String name = i.next();
> addPath(name, handler);
> 
> Regards,
> Mario
> ___
> aspectj-users mailing list
> aspectj-users@eclipse.org
> To change your delivery options, retrieve your password, or unsubscribe from 
> this list, visit
> https://dev.eclipse.org/mailman/listinfo/aspectj-users

___
aspectj-users mailing list
aspectj-users@eclipse.org
To change your delivery options, retrieve your password, or unsubscribe from 
this list, visit
https://dev.eclipse.org/mailman/listinfo/aspectj-users

[aspectj-users] Java 9 sun.boot.class.path gone

2016-07-22 Thread Mario Ivankovits
Hi!

In latest Java 9 builds the „sun.boot.class.path“ is gone. Due to this, the 
core java classes can no longer be found by aspectj.
On AspectJs earlyJava9 branch I’ve patched it like this to make that work again.


Index: weaver/src/org/aspectj/weaver/bcel/ClassPathManager.java
===
--- weaver/src/org/aspectj/weaver/bcel/ClassPathManager.java(revision 
f8b86ff2c03a77e47e87573b59bc43c57cfdee38)
+++ weaver/src/org/aspectj/weaver/bcel/ClassPathManager.java(revision )
@@ -38,6 +38,7 @@

 import org.aspectj.bridge.IMessageHandler;
 import org.aspectj.bridge.MessageUtil;
+import org.aspectj.util.LangUtil;
 import org.aspectj.weaver.BCException;
 import org.aspectj.weaver.UnresolvedType;
 import org.aspectj.weaver.WeaverMessages;
@@ -70,6 +71,11 @@
if (trace.isTraceEnabled())
trace.enter("", this, new Object[] { classpath, 
handler });
entries = new ArrayList();
+
+   if (LangUtil.is19VMOrGreater()) {
+   entries.add(new JImageEntry(null));
+   }
+
for (Iterator i = classpath.iterator(); i.hasNext();) {
String name = i.next();
addPath(name, handler);

Regards,
Mario

smime.p7s
Description: S/MIME cryptographic signature
___
aspectj-users mailing list
aspectj-users@eclipse.org
To change your delivery options, retrieve your password, or unsubscribe from 
this list, visit
https://dev.eclipse.org/mailman/listinfo/aspectj-users