Skip to content

fix IS_CYGWIN (was not working anyway), replaced by test on env variable ConEmuANSI=ON#146

Closed
Arnaud-Nauwynck wants to merge 1 commit intofusesource:masterfrom
Arnaud-Nauwynck:patch-cygwin
Closed

fix IS_CYGWIN (was not working anyway), replaced by test on env variable ConEmuANSI=ON#146
Arnaud-Nauwynck wants to merge 1 commit intofusesource:masterfrom
Arnaud-Nauwynck:patch-cygwin

Conversation

@Arnaud-Nauwynck
Copy link
Copy Markdown
Contributor

…was typo "!"
IS_CYGWIN = ... !"cygwin".equals(System.getenv("TERM")
instead of
IS_CYGWIN = ... "cygwin".equals(System.getenv("TERM")

notice that in a windows cmd.exe console, the jansi mode "WINDOWS" works fine ... this is independent of the fact that the process "bash.exe" from cygwin is running

notice also that on windows, a very frequent usage is to use cmder console emulator (https://cmder.net/) + cygwin
The cmder console define several environment variables, including
ConEmuANSI="ON"
which could be tested to use the standard ANSI mode of jansi

hboutemy added a commit that referenced this pull request Jun 28, 2019
@hboutemy
Copy link
Copy Markdown
Collaborator

kept ConEmu support part of the PR
didn't merge the Cygwin removal: absolutely independent, and you have no proof that there is any issue with the detection as it was written

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.

2 participants