Skip to content

Command position: non-executable files, non-cdable directories, CDPATH false positives #669

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

Conversation

danielshahaf
Copy link
Member

Fix a CDPATH bug and #202. Also handles the case of non-executable directories.

@danielshahaf
Copy link
Member Author

Also adds AUTO_CD support.

danielshahaf added a commit that referenced this pull request Mar 16, 2020
* commit 'b1f36d9c5f45b879fbd2f64195167a60d9f3cb9e':
  'main': Add a comment.
  'main': Fix the $CDPATH from the previous commit.
  'main': Add a test for a $CDPATH bug.
@danielshahaf danielshahaf force-pushed the cdpath-and-nonexecutable-in-command-position-v1 branch from 630b416 to dee05a7 Compare March 16, 2020 21:25
@danielshahaf
Copy link
Member Author

Rebased to current master and rewritten a few bits that weren't ready for prime time.

@danielshahaf danielshahaf force-pushed the cdpath-and-nonexecutable-in-command-position-v1 branch 2 times, most recently from 6323686 to 41974a3 Compare March 17, 2020 17:13
@danielshahaf
Copy link
Member Author

Rebased to solve a conflict with fb69f4c.

The next commits will change this behaviour.
…O_CD is set) and non-executable files.

Fixes zsh-users#202.

Test expectations are updated.  For example, BUFFER='/bin' is now
highlighted as path_prefix because it's a prefix of '/bin/sh' which
would be valid.  However, BUFFER='/bin;' is now properly highlighted
as an error (unless AUTO_CD is set).
@danielshahaf danielshahaf force-pushed the cdpath-and-nonexecutable-in-command-position-v1 branch from 41974a3 to f2726d0 Compare March 19, 2020 06:08
@danielshahaf danielshahaf merged commit e4f24f8 into zsh-users:master Mar 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant