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

Use default files for default user profiles #1987

Merged
merged 2 commits into from
Dec 16, 2018

Conversation

CodyRay
Copy link
Contributor

@CodyRay CodyRay commented Dec 13, 2018

I am hoping to make a custom version of Cmder with some defaults configured for our organization - as part of that it would be great if the default content of the user-profile files came from default files like user-aliases.example does. That way it should be easier able to keep our fork in sync and makes it easier to compare with the default config.

This PR pulls the default contents of the file out into their own files. I also took the liberty of calling these ".default" files and renamed the "user-aliases.cmd.example" for consistency with "ConEmu.xml.default" - are there any upgrade considerations for that change?

@daxgames
Copy link
Member

@haroldhues Thanks and I don't know why we didn't do all of this a long time ago.

Have you looked at the cmder.exe /c [path] argument. It allows running Cmder from a central location using config from the central location then applying user unique config from [path] on top of that. Any change to the central location automatically applies to any user the next time they run it with no need to copy config to the users workstation. See: PR #1949 for a detailed explanation.

Copy link
Member

@daxgames daxgames left a comment

Choose a reason for hiding this comment

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

See inline comments for requested changes.

@daxgames daxgames merged commit 8101923 into cmderdev:master Dec 16, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants