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

fix(elvish): before-chdir should assign to oldpwd upvalue #818

Merged
merged 1 commit into from
May 29, 2024

Commits on May 17, 2024

  1. fix(elvish): before-chdir should assign to oldpwd upvalue

    elvish uses lexical scoping for closure capture, as such all `oldpwd`
    references within the script refers to the `oldpwd` defined at the top.
    However, before-chdir hook is trying to assign to `oldpwd` within the
    editor scope, which is not used by this script.
    
    This manifested as a bug in which:
    
    ```
    ~
    > mkdir -p /tmp/another
    
    ~
    > z /tmp
    
    /tmp
    > z another
    
    /tmp/another
    > z -
    
    ~
    > # The previous dir should be /tmp not ~!
    ```
    
    Because the hook was updating a variable that was not used.
    
    Fix the hook so that before-chdir assign to the proper upvalue.
    alaviss committed May 17, 2024
    Configuration menu
    Copy the full SHA
    6943799 View commit details
    Browse the repository at this point in the history