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

ARTEMIS-2366 artemis-service start fails #2733

Closed
wants to merge 1 commit into from

Conversation

jbertram
Copy link
Contributor

No description provided.

@@ -33,7 +33,7 @@
<logmode>roll</logmode>

<executable>%JAVA_HOME%\bin\java.exe</executable>
<startargument>-Xbootclasspath/a:%ARTEMIS_HOME%\lib\${logmanager}</startargument>
<startargument>-Xbootclasspath/a:%ARTEMIS_HOME%\lib\${logmanager};%ARTEMIS_HOME%\lib\${wildfly-common}</startargument>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Have we an updated version yet of jar dep yet? Before we make further changes to support it. Incase it doesnt come and we have to revert the other pr that added it.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't understand what you're asking.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Have we updated wildfly common yet

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@michaelandrepearce even thought there's no real issue in there, the PR is being merged and a new release is being made soon, out of caution.

wildfly/wildfly-common#47

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

FWIW, I just pushed a commit to move Wildfly Common to 1.5.2.Final. That should address the license question.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks

@asfgit asfgit closed this in a2fc928 Jun 26, 2019
@jbertram jbertram deleted the ARTEMIS-2366 branch June 10, 2021 19:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants