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

remove "BUNDLED WITH" from Gemfile.lock #63

Open
flokli opened this issue Sep 1, 2019 · 9 comments
Open

remove "BUNDLED WITH" from Gemfile.lock #63

flokli opened this issue Sep 1, 2019 · 9 comments

Comments

@flokli
Copy link

flokli commented Sep 1, 2019

When using bundix to package new applications, we should probably remove the "BUNDLED WITH" information from the generated Gemfile.lock.

For example, when packaging gitlab, on gitaly startup (done via bundle exec), it complains about the running version of bundler being older than the version that created the lockfile.

As we assemble the ruby environment via nix and not via bundler anyways, we can probably just remove the two lines from the Gemfile.lock

@manveru
Copy link
Collaborator

manveru commented Sep 1, 2019

Sounds reasonable, do you want to make an attempt?

@flokli
Copy link
Author

flokli commented Sep 1, 2019

Huh, my ruby is pretty bad. Happy to try a patch, though ;-)

@alyssais
Copy link
Member

alyssais commented Sep 1, 2019

Hmmm. Not sure this is a good idea — BUNDLED WITH is recorded so that Bundler can introduce breaking changes in its lockfiles. If we remove it, that’s not going to work.

I think we should be better at keeping Bundler up to date in Nixpkgs, but that probably requires a bit of Bundix work to make it happy with 1.x and 2.x.

@flokli
Copy link
Author

flokli commented Sep 10, 2019

If you call version numbers MAJOR.MINOR.PATCH, in my case it was really just the "PATCH" part being being incremented by one in the Gemfile.lock - so having bundler complain about this makes me worry about version guarantees they try to make.

IIRC, this happened while backporting the gitlab rubyenv to 19.03.

I didn't really look into how we assemble bundler environments inside nixpkgs, but how much does bundler still do dependency resolution and environment setup if we use it inside such a bundix-generated expression for it to become a problem? Won't we notice too, as it'll simply fail to start later on?

@alyssais
Copy link
Member

alyssais commented Sep 10, 2019 via email

@flokli
Copy link
Author

flokli commented Sep 10, 2019

So what do we do here? Remove the BUNDLED WITH or keep it?

@manveru
Copy link
Collaborator

manveru commented Sep 11, 2019

Let's keep this unless it causes any actual issues.

@flokli
Copy link
Author

flokli commented Sep 11, 2019

Ok.

@flokli
Copy link
Author

flokli commented Apr 20, 2020

This seems to cause actual issues - see NixOS/nixpkgs#85623 (comment).

@flokli flokli reopened this Apr 20, 2020
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

No branches or pull requests

3 participants