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

[WFLY-4554] cd into JBOSS_HOME instead of fancy escaping. #7386

Merged
merged 1 commit into from Apr 24, 2015

Conversation

darranl
Copy link
Contributor

@darranl darranl commented Apr 24, 2015

The problem with the jconsole script is the arguments get passed through the jconsole binary included with the JVM, there is not sufficient flexibility to escape the classpath so that it reaches the underlying Java process in the correct form.

On the other hand if we ensure we are in the troubled directory then the classpath can be relative.

…mand line ensure jconsole is executed from JBOSS_HOME, this way our custom classpath can be relative.
@wildfly-ci
Copy link

Windows Build 1437 is now running using a merge of b9dd151

@wildfly-ci
Copy link

Linux Build 6357 is now running using a merge of b9dd151

@wildfly-ci
Copy link

Linux with security manager Build 1285 is now running using a merge of b9dd151

@wildfly-ci
Copy link

Linux with security manager Build 1285 outcome was SUCCESS using a merge of b9dd151
Summary: Tests passed: 824, ignored: 356 Build time: 0:05:07

@wildfly-ci
Copy link

Windows Build 1437 outcome was SUCCESS using a merge of b9dd151
Summary: Tests passed: 2856, ignored: 412, muted: 1 Build time: 0:47:25

@wildfly-ci
Copy link

Linux Build 6357 outcome was SUCCESS using a merge of b9dd151
Summary: Tests passed: 2857, ignored: 412 Build time: 0:47:26

n1hility added a commit that referenced this pull request Apr 24, 2015
[WFLY-4554] cd into JBOSS_HOME instead of fancy escaping.
@n1hility n1hility merged commit 3a88b8b into wildfly:master Apr 24, 2015
@darranl darranl deleted the WFLY-4554 branch April 26, 2015 16:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants