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

Gaze is adding all folders in HOME to watch list #1776

Closed
2 tasks done
paulbouwer opened this issue Apr 29, 2017 · 4 comments
Closed
2 tasks done

Gaze is adding all folders in HOME to watch list #1776

paulbouwer opened this issue Apr 29, 2017 · 4 comments

Comments

@paulbouwer
Copy link
Contributor

paulbouwer commented Apr 29, 2017

  • I am on the latest Hyper.app version
  • I have searched the issues of this repo and believe that this is not a duplicate

  • OS version and name: Windows 10 (16179)
  • Hyper.app version: 1.3.1
  • Link of a Gist with the contents of your .hyper.js: Default
  • Relevant information from devtools (CMD+ALT+I on Mac OS, CTRL+SHIFT+I elsewhere): N/A
  • The issue is reproducible in vanilla Hyper.app: Yes

Issue

Relevant to the following hyper issues:

Relevant to the following gaze issues:

About 2 weeks ago I started seeing the following when starting hyper. I wasn't sure what had changed. It was not a new hyper version, but I had just upgraded to the 16176 Windows Insider build - although I'm also sure that didn't cause it - confused ... I started investigating.

image

The only item being added to glaze from hyper for watching seems to be the .hyper.js file in the user's HOME folder. But I noticed (after putting tons of logging into glaze) that it added the .hyper.js file to its watch list, but then also added all the sub-folders in my HOME folder too. This is caused by the following logic within glaze:

Once glaze starts watching additional folders in the HOME folder, there are a number of permission and other types of errors that start happening for various users. In my case above, the OneDrive folders have a specific REPARSE POINT which causes the fs.lstatSync call made by glaze to throw a EINVAL error. I'm adding issues to the glaze and node repos for this.

If hyper could ensure that only the .hyper.js file was watched, then I believe that all of the referenced issues would be resolved.

@MartyGentillon
Copy link
Contributor

With the new configuration setup, this shouldn't cause a problem after #1509. (Config files are moving into a directory.)

This was referenced Apr 30, 2017
@albinekb
Copy link
Contributor

We don't use gaze anymore 👍

@darsain
Copy link

darsain commented Jan 23, 2018

@albinekb any ETA on when this might ship? been annoyed by error popups this issue is triggering for half a year now :)

@albinekb
Copy link
Contributor

@darsain can you please try hyper 2.0, read how to change your updateChannel at https://zeit.co/blog/canary 👍 🙏 If that doesn't resolve it, let's reopen 👍

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

No branches or pull requests

4 participants