You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@causeway.apache.org by da...@apache.org on 2023/03/01 19:16:10 UTC

[causeway] branch CAUSEWAY-3252 updated: CAUSEWAY-3252: removes core/config's lombok.processor, as not nec there (we have an explicit declaration)

This is an automated email from the ASF dual-hosted git repository.

danhaywood pushed a commit to branch CAUSEWAY-3252
in repository https://gitbox.apache.org/repos/asf/causeway.git


The following commit(s) were added to refs/heads/CAUSEWAY-3252 by this push:
     new 1139980412 CAUSEWAY-3252: removes core/config's lombok.processor, as not nec there (we have an explicit declaration)
1139980412 is described below

commit 1139980412e23f1d4185a5e895f7e95ea3c4b284
Author: danhaywood <da...@haywood-associates.co.uk>
AuthorDate: Wed Mar 1 19:16:05 2023 +0000

    CAUSEWAY-3252: removes core/config's lombok.processor, as not nec there (we have an explicit declaration)
---
 core/config/lombok.processor | 10 ----------
 1 file changed, 10 deletions(-)

diff --git a/core/config/lombok.processor b/core/config/lombok.processor
deleted file mode 100644
index d5ead5b02b..0000000000
--- a/core/config/lombok.processor
+++ /dev/null
@@ -1,10 +0,0 @@
-The presence of this file activates a profile (defined in bom/pom.xml) that configures maven-compiler-plugin to run the lombok annotation processor.
-
-This is a workaround; the compiler ought to pick up all available processors on the classpath anyway; unknown why it does not, but is probably a badly formed pom.xml somewhere causing dependencies to be ignored.
-
-This file has only been added to the modules where it was found to be required.
-
-The one place where this file is _not_ required is with the JDO modules that declare @PersistenceCapable entities.   
-Such modules need both lombok AND the DN annotation processor (datanucleus-jdo-query) that generates the Qxxx query classes.
-For some reason, with these JDO modules, the default behaviour (picking up all available processors) does seem to work.
-