Skip to content

Conversation

stanhu
Copy link
Contributor

@stanhu stanhu commented Jan 21, 2025

This resolves the OpenSSL 1.1 dependency issue: oxidize-rb/rb-sys#497

@gjtorikian
Copy link
Collaborator

LGTM but

This might resolve

Are you able to point your CI action to this ref and see if it resolves the issue? I'm wary about that "might," although in general I agree this dependency should probably be bumped regardless.

@stanhu
Copy link
Contributor Author

stanhu commented Jan 21, 2025

@gjtorikian I attempted to point it to this ref in https://github.com/oxidize-rb/rb-sys/actions/runs/12877578630/job/35902248995?pr=496, but unfortunately that doesn't work because this repo calls cargo-binstall with. the version here:

Run oxidize-rb/actions/cargo-binstall@v1
  with:
    crate: sccache
    version: 0.3.3
    strategies: quick-install
    install-path: /home/runner/work/_actions/oxidize-rb/actions/v1/setup-ruby-and-rust/bin

I'd need to make another change here to change cargo-binstall to point to this ref, and then point the other repo to that change. 😄

Never mind, updated in oxidize-rb/rb-sys#498.

@stanhu stanhu force-pushed the sh-update-sccache branch from 6c7133f to 039a655 Compare January 21, 2025 19:00
@stanhu
Copy link
Contributor Author

stanhu commented Jan 21, 2025

Yeah, https://github.com/oxidize-rb/rb-sys/actions/runs/12894027138/job/35951762057?pr=498 seems to have passed now with this change:


   INFO resolve: Verified signature for package 'sccache-0.9.1-x86_64-unknown-linux-gnu': timestamp:1736256999	file:sccache-0.9.1-x86_64-unknown-linux-gnu.tar.gz	hashed
   WARN resolve: When resolving sccache bin sccache-dist is not found. But since it requies features dist-server, this bin is ignored.
   WARN The package sccache v0.9.1 (x86_64-unknown-linux-gnu) has been downloaded from third-party source QuickInstall
   INFO This will install the following binaries:
   INFO   - sccache (sccache -> /home/runner/work/_actions/oxidize-rb/actions/50c22e098940b3e773ee8bbc1be80beba580a3c3/setup-ruby-and-rust/bin/sccache)
   INFO Installing binaries...

This resolves the OpenSSL 1.1 dependency issue:
oxidize-rb/rb-sys#497
@stanhu stanhu force-pushed the sh-update-sccache branch from 50c22e0 to 3a41198 Compare January 21, 2025 19:11
@gjtorikian gjtorikian merged commit 841a470 into oxidize-rb:main Jan 21, 2025
@gjtorikian
Copy link
Collaborator

tysm!

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