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

Close #84 Allow apps to enable RUBYOPT=--jit #848

Merged
merged 1 commit into from
Jan 10, 2019

Conversation

schneems
Copy link
Contributor

If you set:

$ heroku config:set RUBYOT=--jit

Then this will be a global value so when the buildpack attempts to detect ruby version by running:

$ bundle platform --ruby

It is executed with the RUBYOPT set. Since the version of Ruby the buildpack executes does not have --jit it fails:

-----> Ruby app detected
-----> Compiling Ruby/Rails
 !
 !     There was an error parsing your Gemfile, we cannot continue
 !     ruby: invalid option --jit  (-h will show valid options) (RuntimeError)
 !
 !     Push rejected, failed to compile Ruby app.
 !     Push failed

If you set:

```
$ heroku config:set RUBYOT=--jit
```

Then this will be a global value so when the buildpack attempts to detect ruby version by running:

```
$ bundle platform --ruby
```

It is executed with the RUBYOPT set. Since the version of Ruby the buildpack executes does not have `--jit` it fails:

```
-----> Ruby app detected
-----> Compiling Ruby/Rails
 !
 !     There was an error parsing your Gemfile, we cannot continue
 !     ruby: invalid option --jit  (-h will show valid options) (RuntimeError)
 !
 !     Push rejected, failed to compile Ruby app.
 !     Push failed
```
@schneems schneems changed the title Close #84 Allows apps to enable RUBYOPT=--jit Close #84 Allow apps to enable RUBYOPT=--jit Jan 10, 2019
@schneems schneems merged commit d329287 into master Jan 10, 2019
@edmorley edmorley deleted the schneems/no-more-RUBYOPT branch August 27, 2020 06:54
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

Successfully merging this pull request may close these issues.

2 participants