Skip to content
This repository has been archived by the owner on Apr 29, 2021. It is now read-only.

Fix abs_dirname #224

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

blueyed
Copy link

@blueyed blueyed commented Jun 23, 2017

There are no tests for this unfortunately and it is copied into two
places.

Ref: rbenv/rbenv#868

There are no tests for this unfortunately and it is copied into two
places.

Ref: rbenv/rbenv#868
blueyed added a commit to blueyed/rbenv that referenced this pull request Jun 24, 2017
Without this patch you would see the following error for a symlink like
"/usr/bin/asciidoc -> asciidoc.py":

> …/libexec/{py,rb}env: line 43: cd: asciidoc.py: Not a directory

Ref: sstephenson/bats#224
Ref: pyenv/pyenv#689
Closes: rbenv#868
@blueyed
Copy link
Author

blueyed commented Sep 19, 2018

Pushed a fixup, and a test.
Should be squash-merged.
Not sure about the fixup, but had this in a local stash/checkout still.

blueyed added a commit to blueyed/pyenv that referenced this pull request Sep 19, 2018
blueyed added a commit to blueyed/pyenv that referenced this pull request Sep 19, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant