Skip to content

Commit

Permalink
[#2645] Fixing the reading of lombok.config on sbt 1.4+
Browse files Browse the repository at this point in the history
  • Loading branch information
rzwitserloot committed Nov 13, 2020
1 parent f51be15 commit e1f82ac
Show file tree
Hide file tree
Showing 2 changed files with 81 additions and 3 deletions.
3 changes: 2 additions & 1 deletion doc/changelog.markdown
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,8 @@ Lombok Changelog

### v1.18.17 "Edgy Guinea Pig"
* BUGFIX: Netbeans would not work with 1.18.16 anymore. [Issue #2612](https://github.com/rzwitserloot/lombok/issues/2612)
* (potential) BUGFIX: Using lombok with Maven Tycho now works. With assistance from [Rabea Gransberger](https://github.com/rgra). [Issue #285](https://github.com/rzwitserloot/lombok/issues/285)
* PLATFORM: using `lombok.config` files when compiling with sbt 1.4 now works again. [Issue #2645](https://github.com/rzwitserloot/lombok/issues/2645)
* (potential) BUGFIX: Using lombok with Maven Tycho now works. With assistance from [Rabea Gransberger](https://github.com/rgra). [Issue #285](https://github.com/rzwitserloot/lombok/issues/285) __UPDATE: This doesn't quite work yet, still investigating.__

### v1.18.16 (October 15th, 2020)
* BUGFIX: Version 1.18.14 could not be installed in Eclipse, it would break Eclipse.
Expand Down
81 changes: 79 additions & 2 deletions src/core/lombok/javac/JavacAST.java
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
/*
* Copyright (C) 2009-2019 The Project Lombok Authors.
* Copyright (C) 2009-2020 The Project Lombok Authors.
*
* Permission is hereby granted, free of charge, to any person obtaining a copy
* of this software and associated documentation files (the "Software"), to deal
Expand All @@ -21,6 +21,7 @@
*/
package lombok.javac;

import java.io.File;
import java.lang.reflect.Field;
import java.lang.reflect.Method;
import java.net.URI;
Expand Down Expand Up @@ -110,14 +111,90 @@ public JavacAST(Messager messager, Context context, JCCompilationUnit top, Clean
return memoizedAbsoluteFileLocation;
}

private static Class<?> wrappedFileObjectClass, sbtJavaFileObjectClass, sbtMappedVirtualFileClass, sbtOptionClass;
private static Field wrappedFileObjectField, sbtJavaFileObjectField, sbtMappedVirtualFilePathField, sbtMappedVirtualFileRootsField, sbtOptionField;
private static Method sbtMapGetMethod;

public static URI getAbsoluteFileLocation(JCCompilationUnit cu) {
try {
return cu.sourcefile.toUri();
URI uri = cu.sourcefile.toUri();
String fn = uri.toString();
if (fn.startsWith("file:")) return uri;
URI sbtUri = tryGetSbtFile(cu.sourcefile);
if (sbtUri != null) return sbtUri;
return uri;
} catch (Exception e) {
return null;
}
}

private static URI tryGetSbtFile(JavaFileObject sourcefile) {
try {
return tryGetSbtFile_(sourcefile);
} catch (Exception e) {
return null;
}
}

private static URI tryGetSbtFile_(JavaFileObject sourcefile) throws Exception {
Class<?> c = sourcefile.getClass();
String cn;

if (wrappedFileObjectClass == null) {
if (!c.getName().equals("com.sun.tools.javac.api.ClientCodeWrapper$WrappedJavaFileObject")) return null;
wrappedFileObjectClass = c;
}
if (c != wrappedFileObjectClass) return null;

if (wrappedFileObjectField == null) wrappedFileObjectField = Permit.permissiveGetField(wrappedFileObjectClass.getSuperclass(), "clientFileObject");
if (wrappedFileObjectField == null) return null;
Object fileObject = wrappedFileObjectField.get(sourcefile);
c = fileObject.getClass();

if (sbtJavaFileObjectClass == null) {
cn = c.getName();
if (!cn.startsWith("sbt.") || !cn.endsWith("JavaFileObject")) return null;
sbtJavaFileObjectClass = c;
}
if (sbtJavaFileObjectClass != c) return null;
if (sbtJavaFileObjectField == null) sbtJavaFileObjectField = Permit.permissiveGetField(sbtJavaFileObjectClass, "underlying");
if (sbtJavaFileObjectField == null) return null;

Object mappedVirtualFile = sbtJavaFileObjectField.get(fileObject);
c = mappedVirtualFile.getClass();

if (sbtMappedVirtualFileClass == null) {
cn = c.getName();
if (!cn.startsWith("sbt.") || !cn.endsWith("MappedVirtualFile")) return null;
sbtMappedVirtualFileClass = c;
}
if (sbtMappedVirtualFilePathField == null) sbtMappedVirtualFilePathField = Permit.permissiveGetField(sbtMappedVirtualFileClass, "encodedPath");
if (sbtMappedVirtualFilePathField == null) return null;
if (sbtMappedVirtualFileRootsField == null) sbtMappedVirtualFileRootsField = Permit.permissiveGetField(sbtMappedVirtualFileClass, "rootPathsMap");
if (sbtMappedVirtualFileRootsField == null) return null;

String encodedPath = (String) sbtMappedVirtualFilePathField.get(mappedVirtualFile);
if (!encodedPath.startsWith("${")) return null;
int idx = encodedPath.indexOf('}');
if (idx == -1) return null;
String base = encodedPath.substring(2, idx);
Object roots = sbtMappedVirtualFileRootsField.get(mappedVirtualFile);
if (sbtMapGetMethod == null) sbtMapGetMethod = Permit.getMethod(roots.getClass(), "get", Object.class);
if (sbtMapGetMethod == null) return null;

Object option = sbtMapGetMethod.invoke(roots, base);
c = option.getClass();
if (sbtOptionClass == null) {
if (c.getName().equals("scala.Some")) sbtOptionClass = c;
}
if (c != sbtOptionClass) return null;
if (sbtOptionField == null) sbtOptionField = Permit.permissiveGetField(sbtOptionClass, "value");
if (sbtOptionField == null) return null;

Object path = sbtOptionField.get(option);
return new File(path.toString() + encodedPath.substring(idx + 1)).toURI();
}

private static String sourceName(JCCompilationUnit cu) {
return cu.sourcefile == null ? null : cu.sourcefile.toString();
}
Expand Down

0 comments on commit e1f82ac

Please sign in to comment.