Skip to content

allow home by env as $HOME in paths#469

Closed
madogiwa0124 wants to merge 1 commit intocapistrano:masterfrom
madogiwa0124:allow-tilde-home-env
Closed

allow home by env as $HOME in paths#469
madogiwa0124 wants to merge 1 commit intocapistrano:masterfrom
madogiwa0124:allow-tilde-home-env

Conversation

@madogiwa0124
Copy link
Copy Markdown

Added support for $HOME in relation to #464, #465

@madogiwa0124 madogiwa0124 changed the title allow tilde as $HOME in paths allow home by env as $HOME in paths Jul 20, 2019
@madogiwa0124 madogiwa0124 force-pushed the allow-tilde-home-env branch from 24372c0 to f235939 Compare July 20, 2019 11:15
@madogiwa0124 madogiwa0124 force-pushed the allow-tilde-home-env branch from f235939 to 63dc802 Compare July 20, 2019 14:29
@leehambley
Copy link
Copy Markdown
Member

Is there a very good reason why you cannot use ~?

Expanding ~ is a Bash (and other shells) feature https://www.gnu.org/software/bash/manual/html_node/Tilde-Expansion.html which is distinct from the expansion of environment variables.

Whilst you may find it convenient to permit $HOME in paths, many people may be distressed and confused when finding out that it does not in fact do environment variable expansion and is simply a string replacement alias.

I prefer not to support confusing APIs, and would prefer not to merge this contribution, but I am open to discussion about the relative merits.

Thanks, and sorry it took me some time to get back to you, I've been travelling for work.

@madogiwa0124
Copy link
Copy Markdown
Author

@leehambley

I see 😄

Whilst you may find it convenient to permit $HOME in paths, many people may be distressed and confused when finding out that it does not in fact do environment variable expansion and is simply a string replacement alias.

Since I used $ HOME for my service, I created a PR, I agree with your opinion.
I fix my service to use a tilde. 👍

Thanks! ✨

@leehambley
Copy link
Copy Markdown
Member

Thank you @madogiwa0124 - I look forward to future contributions from you :) Thank you for using and trusting SSHKit 🙇‍♂️

@madogiwa0124 madogiwa0124 deleted the allow-tilde-home-env branch July 26, 2019 08: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.

2 participants