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

Document habitat license acceptance #6552

Merged
merged 1 commit into from
May 15, 2019
Merged

Document habitat license acceptance #6552

merged 1 commit into from
May 15, 2019

Conversation

raskchanky
Copy link
Contributor

I'm sorry this got overlooked until now.

Signed-off-by: Josh Black raskchanky@gmail.com

@chef-expeditor
Copy link
Contributor

Hello raskchanky! Thanks for the pull request!

Here is what will happen next:

  1. Your PR will be reviewed by the maintainers.
  2. If everything looks good, one of them will approve it, and your PR will be merged.

Thank you for contributing!

* You can create an empty file at either <code>/hab/accepted-licenses/habitat</code> (if your hab commands run as root) or <code>$HOME/.hab/accepted-licenses/habitat</code> (if they run as a user other than root). Note that if you have Habitat commands running as a mix of users, creating both files is a good idea.
* You can export the <code>HAB_LICENSE</code> environment variable with either of two values:
* <code>accept</code> will accept the license and write the necessary metadata to disk, avoiding any future prompts. This behavior is identical to running <code>hab license accept</code>.
* <code>accept-no-persist</code> will accept the license for 1 command invocation only - no metadata will get written to disk and you will be prompted again in the future. This is most suitable for CI systems or other areas where persisting files to disk is undesireable.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

instead of for 1 command invocation only, perhaps this should be within the scope of the environment variable.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sure

@raskchanky raskchanky force-pushed the jb/document-license branch 3 times, most recently from 54450d3 to 6c8b466 Compare May 15, 2019 00:25
Signed-off-by: Josh Black <raskchanky@gmail.com>
@raskchanky raskchanky merged commit a8ac288 into master May 15, 2019
@raskchanky raskchanky deleted the jb/document-license branch May 15, 2019 03:57
chef-ci added a commit that referenced this pull request May 15, 2019
Obvious fix; these changes are the result of automation not creative thinking.
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.

None yet

2 participants