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

#420 and #577 - use registry for CMDER_START #1372

Closed
wants to merge 7 commits into from
Closed

#420 and #577 - use registry for CMDER_START #1372

wants to merge 7 commits into from

Conversation

superelitist
Copy link

Environment variables don't seem to work properly when cmder starts
ConEmu using /single. These changes communicate the intended start path
using a registry key.

Sorry, I didn't make changes from development, but that branch looks pretty out of date.

Environment variables don't seem to work properly when cmder starts
ConEmu using /single. These changes communicate the intended start path
using a registry key.
@DRSDavidSoft
Copy link
Contributor

@Stanzilla on the subject of branches, if the development isn't getting much activity and even seems to be obsolete, could you remove it? I was also confused when I first tried to make a PR.

@Stanzilla
Copy link
Member

That would certainly be a an option. @cmderdev/trusted-contributors thoughts?

@Jackbennett Jackbennett mentioned this pull request Mar 1, 2018
3 tasks
@daxgames
Copy link
Member

daxgames commented Mar 1, 2018

No issues.

@superelitist
Copy link
Author

Sorry it took me so long. I'd never rebased anything before. I think I did it correctly.

@Stanzilla
Copy link
Member

Not quite, but almost, you commited merge conflicts here https://github.com/cmderdev/cmder/pull/1372/files#diff-159626d08a0475178d80ae25e9d36f17R91 and in init.bat

@Stanzilla Stanzilla requested review from MartiUK and daxgames April 1, 2018 08:23
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.

@superelitist Thanks for the effort but I don't understand the point of this PR since we have now fixed the env var issues. Also to make it all work users should use a nightly build, at least until cmder 1.3.6+ is released, and remove /new_console args from their default start tasks.

This PR should be closed as rejected unless someone can prove it is still required.

@superelitist
Copy link
Author

I agree. Having tested it (under 1.3.5 actually), it seems to be working (almost) as expected (except if it was launched elevated all tasks then run elevated, but that's a completely separate issue - and may be fixed in 1.3.6, but I don't usually run nightly builds).

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.

5 participants