Skip to content

fix #191 #192

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Closed
wants to merge 1 commit into from
Closed

fix #191 #192

wants to merge 1 commit into from

Conversation

vipcxj
Copy link
Contributor

@vipcxj vipcxj commented Feb 14, 2022

Add a custom compile argument replaceProcessorPathWithProcessorModulePath to force the plugin replace processorPath with processormodulepath.
Usage:

                    <plugin>
                        <artifactId>maven-compiler-plugin</artifactId>
                        <configuration>
                            <compilerId>eclipse</compilerId>
                            <compilerArguments>
                                <replaceProcessorPathWithProcessorModulePath/>
                            </compilerArguments>
                        </configuration>
                        <dependencies>
                            <dependency>
                                <groupId>org.codehaus.plexus</groupId>
                                <artifactId>plexus-compiler-eclipse</artifactId>
                                <version>${ecj-plugin.version}</version>
                            </dependency>
                        </dependencies>
                    </plugin>

slachiewicz pushed a commit that referenced this pull request May 6, 2022
…Path to force the plugin replace processorPath with processormodulepath

Closes #191 #192
slachiewicz pushed a commit that referenced this pull request May 6, 2022
…Path to force the plugin replace processorPath with processormodulepath

Closes #191 #192
@slachiewicz
Copy link
Member

thx merged 8cc8c9f

@slachiewicz slachiewicz closed this May 6, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants