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

Add webrick as Ruby 3 doesn’t include it #914

Merged
merged 2 commits into from
Jul 18, 2024

Conversation

mrmanc
Copy link
Contributor

@mrmanc mrmanc commented Mar 22, 2024

Fixes #752. webrick wasn’t available it’s not bundled as of Ruby v3.0.0. I’ve resolved this by adding it to the dependencies class.

I’ve specified the BUNDLE_GEMFILE as the one that exists in the /src/site location (the source of the site being built) was preventing Jekyll from finding the installed dependencies.
This may be fixable by upgrading Jekyll as it is now declared as a dependency.
@kwin
Copy link

kwin commented Apr 4, 2024

I would rather expect that the docker image uses the same ruby version by default as GitHub Pages publishing (still 2.x)

@mrmanc
Copy link
Contributor Author

mrmanc commented Apr 5, 2024

Yep, me too (GHP is using Ruby 2.7.4), but it was upgraded to use 3.x in a1e407c in January. There’s a conversation in #898 where this was discussed it seems.

@yoannchaudet yoannchaudet merged commit 010b49a into github:master Jul 18, 2024
@yoannchaudet yoannchaudet mentioned this pull request Jul 18, 2024
@mrmanc mrmanc deleted the add-webrick branch July 18, 2024 13:55
TomHennen referenced this pull request in slsa-framework/slsa Aug 21, 2024
[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [github-pages](https://github.com/github/pages-gem) | `231` -> `232`
|
[![age](https://developer.mend.io/api/mc/badges/age/rubygems/github-pages/232?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/rubygems/github-pages/232?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/rubygems/github-pages/231/232?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/rubygems/github-pages/231/232?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

### Release Notes

<details>
<summary>github/pages-gem (github-pages)</summary>

### [`v232`](https://github.com/github/pages-gem/releases/tag/v232)

[Compare
Source](https://github.com/github/pages-gem/compare/v231...v232)

#### What's Changed

- Bump docker/build-push-action from 5 to 6 by
[@&#8203;dependabot](https://github.com/dependabot) in
[https://github.com/github/pages-gem/pull/916](https://github.com/github/pages-gem/pull/916)
- Fix obscured gemfile issue by
[@&#8203;mrmanc](https://github.com/mrmanc) in
[https://github.com/github/pages-gem/pull/912](https://github.com/github/pages-gem/pull/912)
- Add webrick as Ruby 3 doesn’t include it by
[@&#8203;mrmanc](https://github.com/mrmanc) in
[https://github.com/github/pages-gem/pull/914](https://github.com/github/pages-gem/pull/914)
- Update nokogiri CVE-2024-25062 by
[@&#8203;naxhh](https://github.com/naxhh) in
[https://github.com/github/pages-gem/pull/911](https://github.com/github/pages-gem/pull/911)
- Parkr jekyll3.10 by
[@&#8203;yoannchaudet](https://github.com/yoannchaudet) in
[https://github.com/github/pages-gem/pull/919](https://github.com/github/pages-gem/pull/919)
- Fix improperly bound regex by
[@&#8203;yoannchaudet](https://github.com/yoannchaudet) in
[https://github.com/github/pages-gem/pull/921](https://github.com/github/pages-gem/pull/921)
- Prep 232 by [@&#8203;yoannchaudet](https://github.com/yoannchaudet)
in
[https://github.com/github/pages-gem/pull/923](https://github.com/github/pages-gem/pull/923)

#### New Contributors

- [@&#8203;naxhh](https://github.com/naxhh) made their first
contribution in
[https://github.com/github/pages-gem/pull/911](https://github.com/github/pages-gem/pull/911)

**Full Changelog**:
github/pages-gem@v231...v232

</details>

---

### Configuration

📅 **Schedule**: Branch creation - "every weekend" (UTC), Automerge - At
any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR was generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View the
[repository job
log](https://developer.mend.io/github/slsa-framework/slsa).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzOC4yNi4xIiwidXBkYXRlZEluVmVyIjoiMzguMjYuMSIsInRhcmdldEJyYW5jaCI6Im1haW4iLCJsYWJlbHMiOltdfQ==-->

---------

Signed-off-by: Mend Renovate <bot@renovateapp.com>
Signed-off-by: Tom Hennen <TomHennen@users.noreply.github.com>
Co-authored-by: Tom Hennen <TomHennen@users.noreply.github.com>
andrewpollock referenced this pull request in google/osv-scanner Aug 22, 2024
[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [github-pages](https://github.com/github/pages-gem) | `"~> 231"` ->
`"~> 232"` |
[![age](https://developer.mend.io/api/mc/badges/age/rubygems/github-pages/232?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/rubygems/github-pages/232?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/rubygems/github-pages/231/232?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/rubygems/github-pages/231/232?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

### Release Notes

<details>
<summary>github/pages-gem (github-pages)</summary>

### [`v232`](https://github.com/github/pages-gem/releases/tag/v232)

[Compare
Source](https://github.com/github/pages-gem/compare/v231...v232)

#### What's Changed

- Bump docker/build-push-action from 5 to 6 by
[@&#8203;dependabot](https://github.com/dependabot) in
[https://github.com/github/pages-gem/pull/916](https://github.com/github/pages-gem/pull/916)
- Fix obscured gemfile issue by
[@&#8203;mrmanc](https://github.com/mrmanc) in
[https://github.com/github/pages-gem/pull/912](https://github.com/github/pages-gem/pull/912)
- Add webrick as Ruby 3 doesn’t include it by
[@&#8203;mrmanc](https://github.com/mrmanc) in
[https://github.com/github/pages-gem/pull/914](https://github.com/github/pages-gem/pull/914)
- Update nokogiri CVE-2024-25062 by
[@&#8203;naxhh](https://github.com/naxhh) in
[https://github.com/github/pages-gem/pull/911](https://github.com/github/pages-gem/pull/911)
- Parkr jekyll3.10 by
[@&#8203;yoannchaudet](https://github.com/yoannchaudet) in
[https://github.com/github/pages-gem/pull/919](https://github.com/github/pages-gem/pull/919)
- Fix improperly bound regex by
[@&#8203;yoannchaudet](https://github.com/yoannchaudet) in
[https://github.com/github/pages-gem/pull/921](https://github.com/github/pages-gem/pull/921)
- Prep 232 by [@&#8203;yoannchaudet](https://github.com/yoannchaudet)
in
[https://github.com/github/pages-gem/pull/923](https://github.com/github/pages-gem/pull/923)

#### New Contributors

- [@&#8203;naxhh](https://github.com/naxhh) made their first
contribution in
[https://github.com/github/pages-gem/pull/911](https://github.com/github/pages-gem/pull/911)

**Full Changelog**:
github/pages-gem@v231...v232

</details>

---

### Configuration

📅 **Schedule**: Branch creation - "before 6am on monday" in timezone
Australia/Sydney, Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR was generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View the
[repository job
log](https://developer.mend.io/github/google/osv-scanner).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzOC4yNi4xIiwidXBkYXRlZEluVmVyIjoiMzguMjYuMSIsInRhcmdldEJyYW5jaCI6Im1haW4iLCJsYWJlbHMiOlsiZGVwZW5kZW5jaWVzIl19-->
lehors referenced this pull request in slsa-framework/slsa Aug 27, 2024
[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [github-pages](https://github.com/github/pages-gem) | `231` -> `232`
|
[![age](https://developer.mend.io/api/mc/badges/age/rubygems/github-pages/232?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/rubygems/github-pages/232?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/rubygems/github-pages/231/232?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/rubygems/github-pages/231/232?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

### Release Notes

<details>
<summary>github/pages-gem (github-pages)</summary>

### [`v232`](https://github.com/github/pages-gem/releases/tag/v232)

[Compare
Source](https://github.com/github/pages-gem/compare/v231...v232)

#### What's Changed

- Bump docker/build-push-action from 5 to 6 by
[@&#8203;dependabot](https://github.com/dependabot) in
[https://github.com/github/pages-gem/pull/916](https://github.com/github/pages-gem/pull/916)
- Fix obscured gemfile issue by
[@&#8203;mrmanc](https://github.com/mrmanc) in
[https://github.com/github/pages-gem/pull/912](https://github.com/github/pages-gem/pull/912)
- Add webrick as Ruby 3 doesn’t include it by
[@&#8203;mrmanc](https://github.com/mrmanc) in
[https://github.com/github/pages-gem/pull/914](https://github.com/github/pages-gem/pull/914)
- Update nokogiri CVE-2024-25062 by
[@&#8203;naxhh](https://github.com/naxhh) in
[https://github.com/github/pages-gem/pull/911](https://github.com/github/pages-gem/pull/911)
- Parkr jekyll3.10 by
[@&#8203;yoannchaudet](https://github.com/yoannchaudet) in
[https://github.com/github/pages-gem/pull/919](https://github.com/github/pages-gem/pull/919)
- Fix improperly bound regex by
[@&#8203;yoannchaudet](https://github.com/yoannchaudet) in
[https://github.com/github/pages-gem/pull/921](https://github.com/github/pages-gem/pull/921)
- Prep 232 by [@&#8203;yoannchaudet](https://github.com/yoannchaudet)
in
[https://github.com/github/pages-gem/pull/923](https://github.com/github/pages-gem/pull/923)

#### New Contributors

- [@&#8203;naxhh](https://github.com/naxhh) made their first
contribution in
[https://github.com/github/pages-gem/pull/911](https://github.com/github/pages-gem/pull/911)

**Full Changelog**:
github/pages-gem@v231...v232

</details>

---

### Configuration

📅 **Schedule**: Branch creation - "every weekend" (UTC), Automerge - At
any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR was generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View the
[repository job
log](https://developer.mend.io/github/slsa-framework/slsa).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzOC4yNi4xIiwidXBkYXRlZEluVmVyIjoiMzguMjYuMSIsInRhcmdldEJyYW5jaCI6Im1haW4iLCJsYWJlbHMiOltdfQ==-->

---------

Signed-off-by: Mend Renovate <bot@renovateapp.com>
Signed-off-by: Arnaud J Le Hors <lehors@us.ibm.com>
Co-authored-by: Arnaud J Le Hors <lehors@us.ibm.com>
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.

Jekyll serve fails on Ruby 3.0 (webrick missing)
3 participants