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

toPath not working in windows #732

Closed
wants to merge 3 commits into from
Closed

toPath not working in windows #732

wants to merge 3 commits into from

Conversation

@nealstar
Copy link

@nealstar nealstar commented Oct 2, 2014

graylog2 server won't start under windows with the following error.

java.nio.file.InvalidPathException: Illegal char <:> at index 4: file:\C:\Program Files\graylog2-server\graylog2-server.jar!\blacklist.drl.

Have changed the .toPath(); -> Paths.get(builtinRuleUrl.toURI());

@nealstar

This comment has been minimized.

Copy link
Owner Author

@nealstar nealstar commented on 92819d7 Oct 2, 2014

toPath not working under windows

@joschi
Copy link
Contributor

@joschi joschi commented Oct 14, 2014

Thank you very much for providing this PR!

In order to continue we need you to sign our Contributor Agreement (outlined in CONTRIBUTING.md). Ideally you send the signed CLA via email to hello@torch.sh. Our fax has been shown to be a little flakey at times…

@nealstar nealstar closed this Oct 20, 2014
@abhee
Copy link

@abhee abhee commented Dec 3, 2014

I am still having this issue with graylog2-server version 0.92 on Windows 7. It seems that the fix is not present in this version.

@ckasek
Copy link

@ckasek ckasek commented Jan 28, 2015

Any chance this fix can be merged and released? It's pretty much a show-stopper for running on Windows.

@kroepke
Copy link
Member

@kroepke kroepke commented Jan 28, 2015

I'll look at it today.

Thank you for bringing this up again!
On Jan 28, 2015 4:48 AM, "Chuck Kasek" notifications@github.com wrote:

Any chance this fix can be merged and released? It's pretty much a
show-stopper for running on Windows.


Reply to this email directly or view it on GitHub
#732 (comment)
.

@kroepke kroepke reopened this Jan 28, 2015
@kroepke kroepke self-assigned this Jan 28, 2015
@kroepke kroepke closed this in 625af19 Jan 28, 2015
@joschi joschi removed the needs-input label Jan 28, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

5 participants
You can’t perform that action at this time.