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

[PowerShell + CMD] fzf (Windows) does not work on Hyper #2445

Closed
yoav-lavi opened this issue Nov 8, 2017 · 2 comments
Closed

[PowerShell + CMD] fzf (Windows) does not work on Hyper #2445

yoav-lavi opened this issue Nov 8, 2017 · 2 comments
Labels
🎨 Type: Enhancement Issue or PR is an enhancement request/proposal for Hyper 📜 Type: Guide Issues has evolved into a helpful document where users help other users

Comments

@yoav-lavi
Copy link

yoav-lavi commented Nov 8, 2017

  • OS version and name: Microsoft Windows 10 Enterprise Build 15063
  • Hyper.app version: 2.1.1 (canary)
  • The issue is reproducible in vanilla Hyper.app: Using a theme plugin

Issue

When using the fzf command on both PowerShell and CMD, Hyper returns character set not supported.
This does not reproduce in a normal PowerShell or CMD window.
fzf repo

@peterdanis
Copy link

Check junegunn/fzf#963 (comment). There is a Hyper specific solution.

@yoav-lavi
Copy link
Author

yoav-lavi commented Nov 15, 2017

The solution specified works, although it may be beneficial for the Windows Hyper version to not set TERM so it (and other utilities that would run into this problem) will work out of the box.

See janlazo's comment:

xterm-256color is hardcoded in app/session.js

    const baseEnv = Object.assign({}, process.env, {
           LANG: app.getLocale().replace('-', '_') + '.UTF-8',
           TERM: 'xterm-256color',
           TERM_PROGRAM: productName,
           TERM_PROGRAM_VERSION: version
     }, envFromConfig);

Hyper should not set TERM in Windows and let the shells handle this, such as sh.exe or bash.exe for TERM=cygwin.

At the very least, maybe Hyper should suggest adding the configuration below on Windows in a comment in the preferences \ Upon outputting Character Set Not Supported.

Per janlazo's suggestion:

env: {
   TERM: '',     
},

@yoav-lavi yoav-lavi reopened this Nov 15, 2017
@Stanzilla Stanzilla added 📜 Type: Guide Issues has evolved into a helpful document where users help other users 🎨 Type: Enhancement Issue or PR is an enhancement request/proposal for Hyper labels Apr 24, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🎨 Type: Enhancement Issue or PR is an enhancement request/proposal for Hyper 📜 Type: Guide Issues has evolved into a helpful document where users help other users
Projects
None yet
Development

No branches or pull requests

3 participants