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

Inject user-emacs-directory' in async-start' #3391

Closed
wants to merge 1 commit into from

Conversation

brabalan
Copy link
Contributor

This should address issue #3390 .

@syl20bnr
Copy link
Owner

@brabalan @justbur is this PR still required or did you manage to resolve the issue differently ?

@justbur
Copy link
Contributor

justbur commented Oct 19, 2015

I think @brabalan's reason for needing this is somewhat unique (running two separate instances of emacs side-by-side, one of which is spacemacs), and this is not necessarily a robust solution. However, I don't think it hurts anyone to do this.

We talked about other possible solutions, but haven't found any yet that satisfies @brabalan's use case.

@brabalan
Copy link
Contributor Author

@syl20bnr I still need this to be able to run spacemacs alongside another instance of emacs. This works for me, but I would be happy with any other solutions that lets me do that.

@AdrieanKhisbe
Copy link
Contributor

I'm facing the same use case than @brabalan,
The patch is very clean and explicit and would not cause trouble to people having just one .emacs.d

Shall we merge it? :)

@TheBB
Copy link
Collaborator

TheBB commented Nov 16, 2015

Indeed, let's.

Thanks! Cherry-picked in develop. You can safely delete your branch.

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.

5 participants