-
Notifications
You must be signed in to change notification settings - Fork 86
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Compilation in multi-module mode (--module-source-path) not working #118
Comments
Another thing is that AJC has the very comfortable See #119. |
I think I'd make it separate unless you found codepaths overlapping and could fix the latter whilst doing the former. I still haven't met a person who uses java modules, but I guess they do exist :) |
If anyone needs something to play with, here is an example project ZIP with two modules, both using AspectJ annotation-based aspects in *.java files (not *.aj). This enables the modules to also be compiled with Javac or ECJ, even though in those cases of course no aspects are woven and the example just runs without AOP. There is a module The example archive also contains two Bash scripts,
Both scripts also run the application after compilation. Commented out are run commands using The scripts run in Git Bash on my Windows 10 system. You might need to adjust tool/library paths and path separators ( |
Using the
--module-source-path
compiler switch always results in AJC core dumps like ajcore.20220121.082827.213.txt:When testing the latest ECJ 3.28.0 stand-alone in comparison with Javac, I also found out that
*
module name joker feature for more complicated directory layouts like--module-source-path "./modules/*/src"
only works in Javac, not in ECJ. I just created JDT Core bug #578306 for that.What I also noticed is that for AJC, I always have to put
aspectjrt.jar
both on the class-path and the module-path, which feels kind of redundant. In Javac, I do not have to do that when compiling @AspectJ aspects. (Of course, that creates unfinished aspects.)References:
*
patterns in multi-module directory names--module-source-path
supportThe text was updated successfully, but these errors were encountered: