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

Check all directories exist #5

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

Conversation

schneems
Copy link
Contributor

@schneems schneems commented Jun 3, 2020

A possible failure mode is accidentally adding a path that does not exist to the PATH. Another failure mode is adding the file such as bin/yarn instead of the directory bin. This check guarantees that the directory exists and that it is in fact a directory.

schneems added 2 commits June 3, 2020 12:58
A possible failure mode is accidentally adding a path that does not exist to the PATH. Another failure mode is adding the file such as `bin/yarn` instead of the directory `bin`. This check guarantees that the directory exists and that it is in fact a directory.
@schneems schneems force-pushed the schneems/check-dirs-exist branch from 3dd3e4e to fb313ed Compare June 3, 2020 18:22
@schneems
Copy link
Contributor Author

schneems commented Jun 3, 2020

On the Ruby buildpack i'm seeing this on the PATH:

remote:        "/tmp/tmp.aXTZIh1fah/bin/:/usr/local/bin:/usr/bin:/bin:/tmp/codon/vendor/bin"        

I don't know where the /tmp/tmp.aXTZIh1fah/bin is coming from. It looks like it's coming from here: https://github.com/heroku/heroku-buildpack-ruby/blob/6ce99e08d65f380eed0b83c00b64775ede125537/bin/support/bash_functions.sh#L44

I'm tired of looking at this for now, i'm going to put a pin in it. We should perhaps re-visit. For now it can't be applied without breaking the Ruby buildpack tests and its unclear there's benefit from fixing this immediate issue. It would be good to get it out of the PATH, but dealing with the bash bootstrapping process is 😨

@schneems schneems changed the base branch from master to main July 27, 2020 16:18
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