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

Cmder exinit #740

Merged
merged 10 commits into from
Feb 29, 2016
Merged

Cmder exinit #740

merged 10 commits into from
Feb 29, 2016

Conversation

daxgames
Copy link
Member

From the vendor/cmder_exinit file:

Copy this file to your externally installed meaning not in the cmder/vendor
folder Cygwin/Msys2/Msys installs '/etc/profile.d/' folder to integrate
the externally installed Unix like environment into Cmder so it has
access to settings stored in Cmder/config folder when launched.

The destination file extension depends on the shell you use. For example:

bash - Copy to /etc/profile.d/cmder_exinit.sh
zsh - Copy to /etc/profile.d/cmder_exinit.zsh

@@ -13,3 +13,4 @@ build/
Version v*
*.bak
config/user-*
config/aliases
Copy link
Contributor

Choose a reason for hiding this comment

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

IMO this could be changed to config/*, only two files are missing: #694

@daxgames
Copy link
Member Author

I made a suggestion in #694 and I agree if we can deal with those last two files.

@daxgames
Copy link
Member Author

Wait on this one I need to check some stuff before merging.

@daxgames
Copy link
Member Author

This is good to go now. I was seeing some init funkyness and determined it was unrelated to this change. This has been tested with Babun(Cygwin), msys, msys2, and the Git for Windows SDK(msys2)

@daxgames
Copy link
Member Author

@MartiUK - Should I close this? Is there any chance it will make it in? If it might did you see my profile.d PR #855. If this might make it in I would want to add the profile.d bits to it as well.

@MartiUK
Copy link
Member

MartiUK commented Feb 26, 2016

@daxgames go for it

@daxgames
Copy link
Member Author

Updating with profile.d like support in #855 - Do not merge until I release here.

@daxgames
Copy link
Member Author

This is ready whenever you are.

MartiUK added a commit that referenced this pull request Feb 29, 2016
@MartiUK MartiUK merged commit 8fa151d into cmderdev:development Feb 29, 2016
@daxgames daxgames deleted the cmder_exinit branch February 29, 2016 13:44
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.

3 participants