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

Easy-RSA: New Windows Start menu item for Easy-RSA Non-Admin mode #560

Open
TinCanTech opened this issue Feb 1, 2024 · 1 comment · May be fixed by #561
Open

Easy-RSA: New Windows Start menu item for Easy-RSA Non-Admin mode #560

TinCanTech opened this issue Feb 1, 2024 · 1 comment · May be fixed by #561

Comments

@TinCanTech
Copy link
Contributor

TinCanTech commented Feb 1, 2024

Following from #559, I would like to request a new Windows Start Menu Item:

  • Name="Start EasyRSA shell (Non-admin)"

This would be a second version of this:

  • <Component Id="shortcut.easyrsa.easyrsa_start.bat" Guid="{FE9645C1-6EF1-4046-895A-4ED5C98E2570}">
    <Shortcut
    Id="shortcut.easyrsa.easyrsa_start.bat"
    Name="Start EasyRSA shell"
    Target="[EASYRSADIR]EasyRSA-Start.bat"
    WorkingDirectory="EASYRSADIR"/>
    <RemoveFolder Id="shortcut.easyrsa.easyrsa_start.bat" On="uninstall"/>
    <RegistryValue Root="HKCU" Key="Software\$(var.PRODUCT_NAME)\Shortcuts" Name="easyrsa.easyrsa_start.bat" Type="integer" Value="1"/>
    </Component>

The command to be executed is:

  • Target="[EASYRSADIR]EasyRSA-Start.bat /NA"

The /NA label is for Non-Admin mode. I am happy to go with any suitable label.

I am not familiar with msi.wxs, which is why this is a feature request and not a PR.

lstipakov added a commit to lstipakov/openvpn-build that referenced this issue Feb 2, 2024
@lstipakov lstipakov linked a pull request Feb 2, 2024 that will close this issue
@TinCanTech
Copy link
Contributor Author

TinCanTech commented Feb 3, 2024

This needs further testing: It may be preferable to use the icon property field "Start in", to set %USERHOMEDRIVE%%USERHOMEPATH%.

Linking: OpenVPN/easy-rsa#1075

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 a pull request may close this issue.

1 participant