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

Change INSTALL_DIR default for aws2 #4660

Closed
lousyd opened this issue Nov 14, 2019 · 2 comments
Closed

Change INSTALL_DIR default for aws2 #4660

lousyd opened this issue Nov 14, 2019 · 2 comments
Labels
closed-for-staleness feature-request A feature should be added or improved. installation needs-review This issue or pull request needs review from a core team member. v2

Comments

@lousyd
Copy link

lousyd commented Nov 14, 2019

In the installer for aws2, the default INSTALL_DIR is /usr/local/aws-cli. I suggest changing it to /opt/aws-cli. The /usr/local hierarchy is not set up for installing whole packages in one subdirectory. It is split into different directories for bin files, source files, libraries, etc. Creating a new directory in /usr/local for aws2 breaks this convention.

Also, the Filesystem Hierarchy Standard, for Linux systems, says that no other directories should exist in /usr/local. It does however, specify a place for vendor package trees: /opt.

I suggest changing the default INSTALL_DIR for the aws2 installer to /opt/aws-cli. Nothing else needs to be changed to support changing that default. The /opt directory works exactly like the installer is expecting.

@kyleknap
Copy link
Contributor

kyleknap commented Dec 3, 2019

We would need to think about changing it to /opt/aws-cli. In the meantime though, you can control where the install dir lives using the -i parameter. So something like:

$ ./install -i /opt/aws-cli

Also, you can get a full list of the install options by running: ./install --help

@kdaily kdaily added feature-request A feature should be added or improved. installation labels Mar 4, 2021
@kdaily kdaily added the needs-review This issue or pull request needs review from a core team member. label Aug 27, 2021
@github-actions
Copy link

Greetings! It looks like this issue hasn’t been active in longer than one year. We encourage you to check if this is still an issue in the latest release. In the absence of more information, we will be closing this issue soon. If you find that this is still a problem, please feel free to provide a comment or upvote with a reaction on the initial post to prevent automatic closure. If the issue is already closed, please feel free to open a new one.

@github-actions github-actions bot added closing-soon This issue will automatically close in 4 days unless further comments are made. closed-for-staleness and removed closing-soon This issue will automatically close in 4 days unless further comments are made. labels Aug 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
closed-for-staleness feature-request A feature should be added or improved. installation needs-review This issue or pull request needs review from a core team member. v2
Projects
None yet
Development

No branches or pull requests

4 participants