| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|\
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
7259: Relax bundler binstub r=indirect a=deivid-rodriguez
### What was the end-user problem that led to this PR?
The problem was that the logic to select bundler versions was different when bundler is run from a rubygems binstub that when bundler is run from a bundler binstub.
### What was your diagnosis of the problem?
My diagnosis was that we should unify the logic.
### What is your fix for the problem, implemented in this PR?
My fix is to use the same logic implemented in the rubygems version finder, namely, only fail if the major version of bundler does not match.
### Why did you choose this fix out of the possible options?
I chose this fix because it makes things consistent.
Fixes #7243.
Co-authored-by: David Rodríguez <deivid.rodriguez@riseup.net>
|
| |
| |
| |
| |
| | |
In these versions, the `approximate_recommendation` method doesn't
include the prerelease logic.
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
7226: Improve quality specs r=hsbt a=deivid-rodriguez
### What was the end-user problem that led to this PR?
I'm not really sure of the exact context when I made these changes, but I recall I was getting some invalid encoding errors somehow, and they turned out to be an estraneous `’` quote in one of the documentation files.
### What was your diagnosis of the problem?
My diagnosis was that we should not include this character anywhere, but instead use regular quotes.
### What is your fix for the problem, implemented in this PR?
My fix is to add a quality spec for this, and while I was at it, I added some improvements to quality specs in general.
Co-authored-by: David Rodríguez <deivid.rodriguez@riseup.net>
|
| |/
| |
| |
| | |
And use single quotes consistenly.
|
|/ |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
|
|
| |
- See https://blog.travis-ci.com/2018-11-19-required-linux-infrastructure-migration for historical detail
|
| |
|
| |
|
| |
|
| |
|
| |
|
|\
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Use `__dir__` instead of `__FILE__` in newgem.gemspec template
### What was the end-user problem that led to this PR?
Since Ruby 2.0 we've had `__dir__` as well as `__FILE__`.
The initial gem codes written with `bundle gem` using Ruby 2.0 or higher is an old description using `__FILE__`.
### What was your diagnosis of the problem?
Ruby 1.9 is EOL, so I think that there is not much Gem to start developed using it.
### What is your fix for the problem, implemented in this PR?
This PR uses `__dir__` when starting Gem development (i.e. `bundle gem`) using Ruby 2.0 or higher version.
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
### What was the end-user problem that led to this PR?
Since Ruby 2.0 we've had `__dir__` as well as `__FILE__`.
The initial gem codes written with `bundle gem` using Ruby 2.0 or
higher is an old description using `__FILE__`.
### What was your diagnosis of the problem?
Ruby 1.9 is EOL, so I think that there is not much Gem to start
developed using it.
### What is your fix for the problem, implemented in this PR?
This PR uses `__dir__` when starting Gem development (i.e. `bundle gem`)
using Ruby 2.0 or higher version.
|
|/
|
|
|
|
|
|
|
|
|
|
|
| |
Some packaging systems require modifications to the hash-bang
interpreter path, making them longer than normal. One such example of
this is the habitat-sh/habitat project. The result is that the 150
character limit does not find the regex it is looking for, and prevents
an otherwise valid binary from starting.
This change doubles the length of the check from 150 characters to 300
characters. This change has been validated in an impacted environment.
Signed-off-by: Tom Duffield <tom@chef.io>
|
| |
|
| |
|
| |
|
|
|
|
| |
Introduced in https://github.com/bundler/bundler/pull/6267.
|
|
|
|
|
|
|
|
|
| |
This file is not executable at all, since the shebang line contains just
placeholder for template expansion.
Please not that the RuboCop check is disabled on purpose, because the
executable bit of generated executables is set by other code, not by the
permissions on this template file.
|
| |
|
|
|
|
| |
Closes #6260.
|
|\
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Show warning message about binstub outside generation.
/cc @indirect
Fixes #6149.
Originated from https://github.com/bundler/bundler/issues/6149#issuecomment-347783823
|
| | |
|
| |
| |
| |
| | |
Fixes #6149.
|
|/ |
|
|\
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Setup custom Gemfile path before loading bundler for binstubs
### What was the end-user problem that led to this PR?
While you have several gemfiles: `Gemfile` and `Gemfile.tools`.
and generates binstubs for gems from second gemfile: `BUNDLE_GEMFILE=Gemfile.tools bundle binstubs rubocop`
when you invoke those bin `bin/rubocop`
then you see error like:
```bash
/usr/local/opt/rbenv/versions/2.4.2/lib/ruby/gems/2.4.0/gems/bundler-1.16.0/lib/bundler/rubygems_integration.rb:458:in `block in replace_bin_path': can't find executable rubocop for gem rubocop. rubocop is not currently included in the bundle, perhaps you meant to add it to your Gemfile? (Gem::Exception)
from /usr/local/opt/rbenv/versions/2.4.2/lib/ruby/gems/2.4.0/gems/bundler-1.16.0/lib/bundler/rubygems_integration.rb:489:in `block in replace_bin_path'
from bin/rubocop:21:in `<main>'
```
### What was your diagnosis of the problem?
When you have generated `bin/bundler` by rails or by `bundler` it has setup of `BUNDLE_GEMFILE` by default as `Gemfile` or by gemfile which has been setup on `bundle binstub bundler`.
So your binstub for rubocop could not change it.
### What is your fix for the problem, implemented in this PR?
I propose to use`BUNDLE_GEMFILE` from gem's binstub over bundler's binstub version
### Why did you choose this fix out of the possible options?
This was default behavior before #5878 introduced. Just added some fix to related PR.
|
| | |
|
|/
|
|
|
|
|
| |
When a Gemfile was pointing to a local gem using git to list its files the git command
was executed from within the wrong dir. This fixes the issue without:
1. introducing new dependencies (e.g. `shellwords`)
2. any risks with non-trivial paths (e.g. paths containing spaces)
|
| |
|
|
|
|
|
|
|
| |
dotfiles. and it is inconsistency with gitignore file.
Picked r60122 from ruby/ruby:
https://github.com/ruby/ruby/commit/5b3d137b1fd9f9f56fa6f17888ab8038906ed760
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|