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

Appbundler inspec-bin not inspec #2043

Merged
merged 1 commit into from
May 1, 2019
Merged

Appbundler inspec-bin not inspec #2043

merged 1 commit into from
May 1, 2019

Conversation

tas50
Copy link
Contributor

@tas50 tas50 commented May 1, 2019

This gets us a working inspec binary with InSpec 4.0.

Signed-off-by: Tim Smith tsmith@chef.io

@@ -75,7 +75,7 @@
appbundle "chef", lockdir: project_dir, gem: "chef", without: %w{integration docgen maintenance ci travis}, env: env
Copy link
Contributor

Choose a reason for hiding this comment

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

you need to appbundle gem: "chef-bin" instead of "chef" here.

Copy link
Contributor

Choose a reason for hiding this comment

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

(actually it should be in addition to, we still need gem: "chef" for knife)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

We're not pulling in chef 15 yet

@@ -75,7 +75,7 @@
appbundle "chef", lockdir: project_dir, gem: "chef", without: %w{integration docgen maintenance ci travis}, env: env
appbundle "foodcritic", lockdir: project_dir, gem: "foodcritic", without: %w{development}, env: env
appbundle "test-kitchen", lockdir: project_dir, gem: "test-kitchen", without: %w{changelog debug docs}, env: env
appbundle "inspec", lockdir: project_dir, gem: "inspec", without: %w{deploy tools maintenance integration}, env: env
appbundle "inspec-bin", lockdir: project_dir, gem: "inspec", without: %w{deploy tools maintenance integration}, env: env
Copy link
Contributor

Choose a reason for hiding this comment

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

this needs to be gem: "inspec-bin", i am not clear any more on what the first arg even does here....

Signed-off-by: Tim Smith <tsmith@chef.io>
@tas50 tas50 merged commit e1d0d73 into master May 1, 2019
@tas50
Copy link
Contributor Author

tas50 commented May 1, 2019

merging based on slack approval

@chef-ci chef-ci deleted the inspec-bin branch May 1, 2019 20:29
@lock
Copy link

lock bot commented Jun 30, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Jun 30, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants