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

Support --debug flag without port argument #629

Closed
wants to merge 1 commit into from

Conversation

briehman
Copy link

Add a dummy positional parameter in the instance where --debug is
specified but no port is provided. This dummy parameter will be
swallowed instead of the next true parameter.

Add a dummy positional parameter in the instance where --debug is
specified but no port is provided. This dummy parameter will be
swallowed instead of the next true parameter.
@wildfly-ci
Copy link

Can one of the admins verify this patch?

@bstansberry
Copy link
Contributor

this is ok to test

@wildfly-ci
Copy link

Windows Build 1201 is now running using a merge of 40941ba

@wildfly-ci
Copy link

Linux Build 1547 is now running using a merge of 40941ba

@wildfly-ci
Copy link

Core - Full Integration Build 802 is now running using a merge of 40941ba

@wildfly-ci
Copy link

Linux Build 1547 outcome was SUCCESS using a merge of 40941ba
Summary: Tests passed: 3092, ignored: 66 Build time: 0:18:51

@wildfly-ci
Copy link

Windows Build 1201 outcome was SUCCESS using a merge of 40941ba
Summary: Tests passed: 3092, ignored: 66 Build time: 0:19:57

@wildfly-ci
Copy link

Core - Full Integration Build 802 outcome was SUCCESS using a merge of 40941ba
Summary: Tests passed: 2847, ignored: 412, muted: 1 Build time: 0:50:52

@jamezp
Copy link
Member

jamezp commented Apr 30, 2015

Which OS seems to have this issue?

@jamezp
Copy link
Member

jamezp commented Apr 30, 2015

FWIW this looks like it would fix the issue as well #638.

@briehman
Copy link
Author

briehman commented May 1, 2015

Yes, this is fixed by #638. Closing this PR.

@briehman briehman closed this May 1, 2015
@jamezp
Copy link
Member

jamezp commented May 1, 2015

Excellent, thank you!

dcihak pushed a commit to dcihak/wildfly-core that referenced this pull request Dec 12, 2018
[JBEAP-15852] Upgrade WildFly Common to 1.2.1.Final
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