Skip to content
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

F186 Updated pom file to be able to build and test for jakarta #188

Closed
wants to merge 6 commits into from

Conversation

mann-ed
Copy link
Contributor

@mann-ed mann-ed commented May 25, 2024

New single project javax and jakarta build.
See #186

To run for javax I had to use Java 8. 17 and 21 did not work. I can post error if needed.
mvn clean package install integration-test test -Pjavax
For jakarta i used Java 21
mvn clean package install integration-test test -Pjakarta
This will update main and test source. So you will need to revert changes back if want to run javax again.

If you want to use jakarta just specify a classifier on the dependency.

<dependency>
            <groupId>io.xlate</groupId>
            <artifactId>property-inject</artifactId>
            <version>1.1.1-SNAPSHOT</version>
            <classifier>jakarta</classifier>
        </dependency>

@mann-ed
Copy link
Contributor Author

mann-ed commented May 26, 2024

I was able to figure out the argLines so now Java 21 can compile for javax. So no need for two different java versions to build now.

@MikeEdgar
Copy link
Member

Closing - replaced by #194

@MikeEdgar MikeEdgar closed this Jun 20, 2024
@mann-ed mann-ed deleted the jakarta-single branch July 6, 2024 15:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants