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

ruby: add pipeline to install the gemspec defined runtime deps #794

Closed

Conversation

hectorj2f
Copy link
Member

Melange Pull Request Template

Fixes #785. This pipeline installs the gem runtime dependencies that are required by a specific ruby project.

Functional Changes

  • This change can build all of Wolfi without errors (describe results in notes)

Notes:

SCA Changes

  • Examining several representative APKs show no regression / the desired effect (details in notes)

Notes:

Linter

  • The new check is clean across Wolfi
  • The new check is opt-in or a warning

Notes:

@hectorj2f hectorj2f requested a review from rawlingsj October 26, 2023 08:20
@hectorj2f hectorj2f self-assigned this Oct 26, 2023
@hectorj2f hectorj2f force-pushed the ruby_install_runtime_deps branch from 93b2801 to aeabaaa Compare October 26, 2023 11:34
Signed-off-by: Hector Fernandez <[email protected]>
@hectorj2f hectorj2f force-pushed the ruby_install_runtime_deps branch 2 times, most recently from 35b6662 to 9325060 Compare October 26, 2023 11:56
Signed-off-by: Hector Fernandez <[email protected]>
@hectorj2f hectorj2f force-pushed the ruby_install_runtime_deps branch from 9325060 to 2021562 Compare October 26, 2023 12:32
@hectorj2f hectorj2f requested a review from imjasonh October 26, 2023 14:11
@kaniini
Copy link
Contributor

kaniini commented Oct 26, 2023

I am pretty sure we don't want to do this, as this will result in Gems being vendored. Instead, melange convert should be used to generate a forest of dependencies.

@hectorj2f
Copy link
Member Author

@kaniini Okay. I'll have a look at 'melange convert' to understand how it could help us here.

@hectorj2f
Copy link
Member Author

Closing this PR! I'll use melange instead to vendor all the runtime deps.

@hectorj2f hectorj2f closed this Oct 30, 2023
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.

pipelines/ruby: add support to install all the required runtime deps from a gemspec
2 participants