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 missing grep and awk dependencies for pacman #333

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

alyssais
Copy link

The default Arch Docker image doesn't have these installed.

Reproduction:

FROM archlinux/base

RUN pacman -Syu --noconfirm sudo make && \
    curl -Lo ruby-install-master.tar.gz https://github.com/postmodern/ruby-install/archive/master.tar.gz && \
    tar -xzvf ruby-install-master.tar.gz && \
    cd ruby-install-master && \
    make install && \
    pacman -Syu --noconfirm grep awk && \
    yes | ruby-install ruby

The default Arch Docker image doesn't have these installed.

Reproduction:

FROM archlinux/base

RUN pacman -Syu --noconfirm sudo make && \
    curl -Lo ruby-install-master.tar.gz https://github.com/postmodern/ruby-install/archive/master.tar.gz && \
    tar -xzvf ruby-install-master.tar.gz && \
    cd ruby-install-master && \
    make install && \
    pacman -Syu --noconfirm grep awk && \
    yes | ruby-install ruby
@postmodern
Copy link
Owner

@alyssais should these dependencies be added to any other rubies that compile?

@postmodern
Copy link
Owner

@alyssais also, is this an issue that should be fixed in the Arch Docker image, or is it intentionally bare bones?

@alyssais
Copy link
Author

alyssais commented Dec 23, 2020 via email

@postmodern
Copy link
Owner

postmodern commented Jul 4, 2021

We could add awk, but ruby-install uses grep internally before it even gets to installing dependencies, so that wouldn't work.

Also, ruby-install is not necessarily needed to install a ruby into a docker image. It's probably easier to add the commands to install the dependencies, download the .tar.xz archive, extract, ./configure --prefix=... && make && make install.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants