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

Check in .bundle/config --without omnibus_package --frozen #784

Merged
merged 1 commit into from
Apr 8, 2016

Conversation

jkeiser
Copy link
Contributor

@jkeiser jkeiser commented Apr 8, 2016

Add "rake bundle[]" task to unfreeze and work with file

@@ -11,7 +11,7 @@ pkg/*
# Do not check in the .bundle directory, or any of the files inside it. Those
# files are specific to each particular machine, and are used to persist
# installation options between runs of the bundle install command.
Copy link
Contributor

Choose a reason for hiding this comment

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

This comment is no longer correct.

@@ -0,0 +1,3 @@
---
BUNDLE_WITHOUT: omnibus_package
BUNDLE_FROZEN: '1'
Copy link
Contributor

Choose a reason for hiding this comment

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

Do we need to explicitly set BUNDLE_FROZEN or can we use a --deployment flag when needed?
Per the docs: BUNDLE_FROZEN defaults to true when --deployment is used.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@chefsalim --deployment does a bunch of other stuff (and in fact BUNDLE_FROZEN is one of the flags it sets). Run bundle install --deployment on your machine and then look in .bundle/config and you will see:

BUNDLE_FROZEN: '1'
BUNDLE_PATH: vendor/bundle
BUNDLE_DISABLE_SHARED_GEMS: '1'

This flag is the equivalent of bundle install --frozen, which is all we want to do.

Copy link
Contributor

Choose a reason for hiding this comment

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

OK


To perform a full update of all dependencies everywhere, run `rake dependencies`. This will update the `Gemfile.lock`, `Gemfile.windows.lock`, `omnibus/Gemfile.lock`, `acceptance/Gemfile.lock`, `omnibus/Berksfile.lock`, and `omnibus_overrides.rb`. It will also show you any outdated dependencies due to conflicting constraints. Some outdated dependencies are to be expected; it will inform you if any new ones appear that we don't know about, and tell you how to proceed.

To add or remove a package from the chef-dk, edit `Gemfile`. To change binary versions, edit `version_policy.rb`.

Copy link
Contributor

Choose a reason for hiding this comment

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

Presumably, 'rake bundle:update' or 'rake bundle:install' is required after updating the Gemfile.
Also, will Gemfile.local continue to work (not sure if that worked for chef-dk).

Copy link
Contributor

Choose a reason for hiding this comment

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

Also, does the README in chef/acceptance need to be updated as well? Will bundle install/update work from there as usual?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I don't see any Gemfile.local, but there is no reason that won't work.

You can bundle install and update from omnibus and acceptance as usual, if you like.

@chefsalim
Copy link
Contributor

👍 w/ minor comments, and pending build/tests in Jenkins stay green

@danielsdeleo
Copy link
Contributor

👍 I wonder if it'll be a pain to do normal bundling things when working with the actual chef-dk codebase, but if it is, we can address it when we find out.

Add "rake bundle[<args>]" task to unfreeze and work with file
@jkeiser jkeiser force-pushed the jk/bundle-config branch from 8081b72 to 7a06a12 Compare April 8, 2016 19:54
@jkeiser jkeiser merged commit 9a297dd into master Apr 8, 2016
@jkeiser jkeiser deleted the jk/bundle-config branch April 8, 2016 20:03
@thommay thommay added Type: Enhancement Adds new functionality. and removed Enhancement labels Feb 1, 2017
@chef-boneyard chef-boneyard locked and limited conversation to collaborators Feb 14, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Type: Enhancement Adds new functionality.
Development

Successfully merging this pull request may close these issues.

6 participants