-
-
Notifications
You must be signed in to change notification settings - Fork 2k
Gemfile.lock is the wrong file extension for the file's purpose. #694
Comments
Gemfile.manifest is a decent choice. You might also consider .bundler, .bmf (bundler manifest file), .gmf (gem manifest file), etc. I agree that this naming is unfortunate. |
+1 |
Naming it after the Rails environment would be interesting, especially if Rails would key it off RAILS_ENV. So Gemfile.production would be the "locked" version of the gems that you want to deploy when in the production environment. Seems like that's the typical use case anyway. |
It would be a pretty serious backward compatibility problem to rename it at this point. Does this naming cause any real-world problems, or is it just a naming convention clash that could cause confusion? |
Yehuda, I'm sensitive to that argument, and I wish I'd made a suggestion a while ago when things were pre 1.0. Daggumit. In order to work at this point, it would have to be some sort of a phased thing. Pick up Gemfile.lock if it's there, suggest changing it, etc. I'd be happy to pitch in the work to make it happen if you do decide that you'd like to do it. Favorite story told to me about Makefiles: Apparently fixing tab/space sensitivity in makefiles was rejected early on by the author of make by citing the installed base. It'd been a few months since it was released and there were already many dozens of users using it. Of course, that was how many decades ago and how many people have had to deal with it since. In reality, it's only going to be a real-world problem for a small number of people and then only if they haven't seen what the purpose of the file is and kill it without thinking because they assume it's a temp file. Because .lock does mean .tmp. Even then, since it can be recreated, it's not fatal. Mostly, it's a nasty metallic taste to see it there. Kind of like typing "public static void main(String[] args)" is nasty to some :) |
I'm currently leaning towards automatically transitioning to "Gemlock" in 1.1. It will (hopefully) be obvious and apparent to current Bundler users, but lack the .lock extension that makes it seem deletable. It should be pretty easy to print a "add the new file to source control" message when this rename happens. Anybody else have thoughts? |
To add another data point, I recently witnessed in #git on freenode a Ruby/Git amateur be successfully convinced to remove Gemfile.lock from version control by a couple of Git experts who had no context for what that file was but assumed it to fall under the old UNIX convention. So yes, this is definitely confusing people in practice. Gemlock sounds reasonable enough to me. |
Speaking of alternative names, I think Anyway I believe everyone knows now that |
"Anyway I believe everyone knows now that Gemfile.lock has nothing to do with UNIX conventions" Yes, but everyone also knows that .lock indicates a temporary file and also that it may be in use. As such, could reasonably be ignored by file sync apps. The issue is closed, but you might re-consider. Insync google drive app ignores it - a reasonable thing for them to do but Gemfile.lock makes it inconvenient for me to keep working directories synced among my workstations. |
@lnigra the default file name cannot be changed until Bundler 2.0, but it is likely to happen then. |
@indirect A reminder that non-Rails project do not check-in the lock file. As such a developer I often wish the lock file could be a hidden dot file and out of the way. But I understand that is less than ideal for Rails developers and/or if alternate Gemfiles are needed. Which is also why Personally I don't see the problem with Another idea, let a configuration option in the Gemfile itself specify the name of its lock file. For example, in a generic Ruby project I could imagine adding:
Not 100% sure if that's possible (or even desirable) but it would offer the most flexibility. |
We've been expanding in the direction of allowing the lockfile name to be configured, and it may even be possible already. We've also shipped support for files named Ultimately, I expect that in Bundler 2.0 the Gemfile will default to |
Sorry if this looks dumb, but is there any difference in syntax between |
Same file, different name |
My expectation is that it's a temporary file and will be released when the changes to the file without lock have been completed. |
|
(and once the Gemfile is instead named |
I might be late, but why |
@godfat it is literally a ruby file, though, and we "parse" it using |
@indirect Oh, you're right. Sorry I was confused with the lock file. Not sure what I was thinking. |
For better or worse, Gemfile.lock is a poor name choice. The .lock extension is a long used UNIX indication that a resource is locked for changing or update by a process. It's as entrenched a convention as .profile or the like. When I was a sysadmin, back in the stone ages, it wasn't unusual to see scripts that would remove any hinky .lock files on system startup anywhere they existed on the system. Not that those were very smart scripts, but... the expectation is that a .lock file is a transitory thing that can be vaporized.
In other words, Bundler's use of this extension is running headlong into a multiple-decade long UNIX convention. When I first saw it, it was like watching one culture clash right into another.
Not wanting to poke a hole in a naming convention without a counter proposal, here's a suggestion: use Gemfile.manifest as a formal name for this file. After all, that's what Bunder is doing when it creates it--itemizing everything that needs to go into the box and writing it down.
(btw, Bundler is good sauce. I like it.)
-Duncan
The text was updated successfully, but these errors were encountered: