Skip to content
This repository was archived by the owner on Mar 28, 2019. It is now read-only.

Conversation

@danieldreier
Copy link
Contributor

I had trouble using 'rake build' with the default 3.4 we'd been using
@actown I believe this resolves the issue you were having building the package, because I got the same error you showed me and it resolved when I used a newer puppet version. The gemfile was installing 3.4.

I had trouble using 'rake build' with the default 3.4 we'd been using
Gemfile.lock Outdated
Copy link
Contributor

Choose a reason for hiding this comment

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

hrm…
maybe we should get rid of Gemfile.lock?

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'd be open to, I've just always thought that the best practice was to share gemfile.lock so that contributors would be using the exact same versions of gems.

The travis-ci job deletes Gemfile.lock prior to installing, so it's testing the newest versions of gems. My thinking is that the travis tests get us knowledge of whether newer gems break stuff, and the Gemfile.lock gets the convenience of working on similar versions most of the time.

I don't have strong feelings about it, though. The gems are just for developers so nothing breaks in production just because of gem versions.

Copy link
Contributor

Choose a reason for hiding this comment

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

+1 for deleting it.

See discussion on PR 148
Add gemfile.lock to gitignore
@danieldreier danieldreier force-pushed the update_default_puppet_version branch from 4a950c3 to a2f40a7 Compare November 29, 2014 19:45
@danieldreier
Copy link
Contributor Author

@igalic I've pushed a second commit to this that removes gemfile.lock and adds it to .gitignore to avoid it being re-added by accident.

igalic added a commit that referenced this pull request Dec 1, 2014
@igalic igalic merged commit 4b6df81 into puppetlabs-operations:master Dec 1, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants