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

fastwalk: only check if MSYSTEM is set during MSYS/MSYS2 #28

Merged
merged 1 commit into from
Jul 17, 2024

Conversation

charlievieth
Copy link
Owner

This changes the MSYS/MSYS2 detection to only check if the MSYSTEM environment variable is set instead of asserting on its value. This is to support other MSYS environments like UCRT64.

See:

@charlievieth charlievieth force-pushed the cev/msystem branch 9 times, most recently from f3bb796 to 057acb7 Compare July 16, 2024 19:49
This changes the MSYS/MSYS2 detection to only check if the MSYSTEM
environment variable is set instead of asserting on its value. This
is to support other MSYS environments like UCRT64.

It also removes the check for Windows Subsystem for Linux (WSL) since
it was unreliable and required us to stat and potentially read a file
on initialization (which is bad).

See:
 * 87029d9#r144282929
 * sharkdp/fd@f875ea9a5
@charlievieth charlievieth merged commit 875daa3 into master Jul 17, 2024
8 checks passed
charlievieth added a commit to charlievieth/fzf that referenced this pull request Jul 18, 2024
This commit updates github.com/charlievieth/fastwalk to v1.0.7 which
improves MSYS detection and adds optional sorting of directory entries.
It also updates fzf to use the SortFilesFirst sort mode which improves
the output by making it a bit more sorted and grouped by directory
previously entries were visited in directory order (which is basically
random).

PRs Included:

  * charlievieth/fastwalk#25
  * charlievieth/fastwalk#27
  * charlievieth/fastwalk#28
charlievieth added a commit to charlievieth/fzf that referenced this pull request Jul 18, 2024
This commit updates github.com/charlievieth/fastwalk to v1.0.8 which
improves MSYS detection and adds optional sorting of directory entries.
It also updates fzf to use the SortFilesFirst sort mode which improves
the output by making it a bit more sorted and grouped by directory
previously entries were visited in directory order (which is basically
random).

PRs Included:

  * charlievieth/fastwalk#25
  * charlievieth/fastwalk#27
  * charlievieth/fastwalk#28
charlievieth added a commit to charlievieth/fzf that referenced this pull request Jul 18, 2024
This commit updates github.com/charlievieth/fastwalk to v1.0.8 which
improves MSYS detection and adds optional sorting of directory entries.
It also updates fzf to use the SortFilesFirst sort mode which improves
the output by making it a bit more sorted and grouped by directory
previously entries were visited in directory order (which is basically
random).

PRs Included:

  * charlievieth/fastwalk#25
  * charlievieth/fastwalk#27
  * charlievieth/fastwalk#28
junegunn pushed a commit to junegunn/fzf that referenced this pull request Jul 19, 2024
This commit updates github.com/charlievieth/fastwalk to v1.0.8 which
improves MSYS detection and adds optional sorting of directory entries.
It also updates fzf to use the SortFilesFirst sort mode which improves
the output by making it a bit more sorted and grouped by directory
previously entries were visited in directory order (which is basically
random).

PRs Included:

  * charlievieth/fastwalk#25
  * charlievieth/fastwalk#27
  * charlievieth/fastwalk#28
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.

1 participant