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

Ignore value of EMACS if INSIDE_EMACS is set #156

Closed
wants to merge 1 commit into from

Conversation

svend
Copy link
Contributor

@svend svend commented Dec 11, 2013

Ignore value of EMACS, regardless of its value, if INSIDE_EMACS is set.

According to the Emacs manual, EMACS should be set to "t" if
INSIDE_EMACS is set:

36.2 Interactive Subshell

Emacs sets the environment variable INSIDE_EMACS' in the subshell to VERSION,comint', where VERSION is the Emacs version (e.g., 24.1'). Programs can check this variable to determine whether they are running inside an Emacs subshell. (It also sets theEMACS' environment
variable to t', if that environment variable is not already defined. However, this environment variable is deprecated; programs that use it should switch to usingINSIDE_EMACS' instead.)

However, EMACS is set to a string containing the Emacs version in
24.3.50.1:

$ env | grep EMACS
EMACS=24.3.1 (term:0.96)
INSIDE_EMACS=24.3.1,term:0.96

Instead of relying on a known value of EMACS when INSIDE_EMACS is set,
just ignore the value of EMACS.

Ignore value of EMACS, regardless of its value, if INSIDE_EMACS is set.

According to the Emacs manual, EMACS should be set to "t" if
INSIDE_EMACS is set:

  36.2 Interactive Subshell

  Emacs sets the environment variable `INSIDE_EMACS' in the subshell to
  `VERSION,comint', where VERSION is the Emacs version (e.g., `24.1').
  Programs can check this variable to determine whether they are running
  inside an Emacs subshell. (It also sets the `EMACS' environment
  variable to `t', if that environment variable is not already defined.
  However, this environment variable is deprecated; programs that use it
  should switch to using `INSIDE_EMACS' instead.)

However, EMACS is set to a string containing the Emacs version in
24.3.50.1:

$ env | grep EMACS
EMACS=24.3.1 (term:0.96)
INSIDE_EMACS=24.3.1,term:0.96

Instead of relying on a known value of EMACS when INSIDE_EMACS is set,
just ignore the value of EMACS.
@svend
Copy link
Contributor Author

svend commented Dec 11, 2013

Sorry, need to fix a syntax error. Will resubmit.

@svend svend closed this Dec 11, 2013
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.

None yet

1 participant