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

Update Windows-Service.md #304

Open
wants to merge 5 commits into
base: 2.2.x
Choose a base branch
from

Conversation

likemusic
Copy link

Add to "installService.bat" 3rd parameter with password for root-user to be able shutdown server by windows service (#6552)

Add to "installService.bat" 3rd parameter with password for root-user to be able shutdown server by windows service (#6552)
Copy link
Member

@andrii0lomakin andrii0lomakin left a comment

Choose a reason for hiding this comment

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

Please fix the very small mistake and I will merge it.

:printUsage
echo usage:
echo installService JVM_DLL_location OrientDB_Home
echo installService JVM_DLL_location OrientDB_Home root_password
EXIT /B
```

The script requires two input parameters:
Copy link
Member

Choose a reason for hiding this comment

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

I suppose you missed small mistake, script requires three input parameters ))

Copy link
Author

Choose a reason for hiding this comment

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

Fixed :)

Set password value the same as in Tutorial-Run-the-server.html
@andrii0lomakin
Copy link
Member

@likemusic I am very sorry but you also have unresolved conflicts, could you resolve them too?

@likemusic
Copy link
Author

Conflict resolved. But I have one commit (#e5aed3d264d7cdf1392817d93776a25c661e7939) pushed without really merged section. May be I should rebase/marge that and next commit to one, if it's possible without breaking git/github hooks?

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.

2 participants