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

Add Windows Service and Setup #1

Open
wants to merge 24 commits into
base: master
Choose a base branch
from
Open

Add Windows Service and Setup #1

wants to merge 24 commits into from

Conversation

rgl
Copy link

@rgl rgl commented Jan 29, 2011

No description provided.

@dmajkic
Copy link
Owner

dmajkic commented Jan 29, 2011

Rui,

I would like to avoid binary (exe, dll) file in this repository.

Please change to this:

[Dirs]
Name: {app}; Permissions: {#ServiceAccountName}-full
Name: {app}\data; Permissions: {#ServiceAccountName}-full
Name: {app}\logs; Permissions: {#ServiceAccountName}-full

So that SetACL.exe can be removed.

Also, try if it works for 64bit. (It should, but please try).

BTW InnoSetup also allows one fat setup file with both 32bin and 64bit targets.

@rgl
Copy link
Author

rgl commented Jan 29, 2011

I also wished that would be possible, but unfortunately Inno Setup (out-of-box at least) only supports these known Identifiers:

admins Built-in Administrators group
authusers Authenticated Users group
everyone Everyone group
powerusers Built-in Power Users group
system Local SYSTEM user
users Built-in Users group

Sure, I will try to build a 64-bit version.

@rgl
Copy link
Author

rgl commented Feb 16, 2011

I've added these changes into the 2.2 branch too.

@johnnyshields
Copy link

+1

@dmajkic
Copy link
Owner

dmajkic commented Aug 12, 2014

Miscrosoft OpenTech divison took this project, and further development is done by MS team on
http://github.com/msopentech/redis and more on their site http://msopentech.com/opentech-projects/redis

@justinclift
Copy link

Hmmm, seems like that MS team version is now outdated too. 😦

@milomiros
Copy link

milomiros commented Jun 23, 2019 via email

@gyhappy
Copy link

gyhappy commented Jun 11, 2022

ok

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.

6 participants