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

RavenDb as a windows service support #49

Closed
wants to merge 2 commits into from

Conversation

danielmarbach
Copy link
Contributor

Added support to install ravendb server as a windows service. I hope this is compliant to the chuck norris style.

It is now possible to uninstall RavenDb as windows service
@danielmarbach
Copy link
Contributor Author

Added additional support to uninstall ravendb server as windows service

@drusellers
Copy link
Member

I am hesitant to add this as it is very RavenDB specific - is there a way this can just be a service install type task??? I am ultimately open to adding this, just need to be convinced. :)

@danielmarbach
Copy link
Contributor Author

Hy
There is also stuff in there which is Nservicebus specific. This commit doesn't take any dependency to RavenDB so why not adding it?

@drusellers
Copy link
Member

bah! I forgot about that NSB stuff (not a fan of that either for the same reason) ok, i will review it again.

@danielmarbach
Copy link
Contributor Author

Maybe it would be wise to create a contrib project and then move all the crazy stuff such as NSB and RavenDB into that project. This would allow more aggressiv release cycles

@drusellers
Copy link
Member

Nah. Just more overhead for now. I will just add it in. Stuff is easy to move out if need be later.

-d

On Jul 10, 2012, at 12:32, [email protected] wrote:

Maybe it would be wise to create a contrib project and then move all the crazy stuff such as NSB and RavenDB into that project. This would allow more aggressiv release cycles


Reply to this email directly or view it on GitHub:
#49 (comment)

@danielmarbach
Copy link
Contributor Author

I will not complain ;)

@drusellers
Copy link
Member

done 9ba2450

@drusellers drusellers closed this Jul 23, 2012
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