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

Installer expects 'Users' group. Language specific. #150

Closed
warcode opened this issue Aug 21, 2014 · 10 comments
Closed

Installer expects 'Users' group. Language specific. #150

warcode opened this issue Aug 21, 2014 · 10 comments
Labels
Type: Bug Type: Bug
Milestone

Comments

@warcode
Copy link

warcode commented Aug 21, 2014

In a non-English install of Windows 7 x64 I get the error "An error occurred while applying security settings. Users is not a valid user or group.", which is correct as it doesn't exist. All local groups are language specific. Manually creating the group skips the error but causes a nondescript error later. Install can't finish.

@gbiellem
Copy link
Contributor

Looks like this code to localize the name in this custom action isn't working correctly when called from the installer

@johnsimons
Copy link
Member

Could it be something to do with our msi being set to english ?
And maybe that causes the Culture in the .net code be set to english?

@johnsimons
Copy link
Member

Or maybe we just use the sid ?

@gbiellem
Copy link
Contributor

@johnsimons yep I had the same thoughts

gbiellem added a commit that referenced this issue Aug 22, 2014
@gbiellem gbiellem added this to the 1.1.1 milestone Aug 22, 2014
@gbiellem
Copy link
Contributor

SID not supported by netsh.exe - resorted to SDDL

@gbiellem
Copy link
Contributor

The Custom Action was only one part of the problem, Advanced Installer stores reg and file permissions sets for wellknown groups in english rather that SIDs so the setting here are also killing the installer on foreign language OSes.
See https://github.com/Particular/ServicePulse/blob/master/installer/ServicePulse.aip#L396

@johnsimons
Copy link
Member

That is going to be harder to fix!

@gbiellem
Copy link
Contributor

This FAQ article seems to indicate that Advanced Installer should do the magic mapping of "Users" to the correct localized name.

I'll try a evaluation version of the latest version of Adv Installer to see if this makes difference, otherwise I'll add a custom action.

@gbiellem
Copy link
Contributor

Verified that latest Adv Installer doesn't fix the issue - looks like custom actions are the way to go.

@gbiellem
Copy link
Contributor

Fixed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Bug Type: Bug
Projects
None yet
Development

No branches or pull requests

3 participants