zeroshade commented on code in PR #41180:
URL: https://github.com/apache/arrow/pull/41180#discussion_r1581116454
##########
ci/docker/linux-apt-docs.dockerfile:
##########
@@ -55,7 +57,10 @@ RUN apt-get update -y && \
r-base=${r}* \
rsync \
ruby-dev \
- wget && \
+ wget \
+ libasound2-dev && \
+ wget -q
https://freeshell.de/phd/chromium/jammy/pool/latest/chromium_124.0.6367.60~linuxmint1+virginia_amd64.deb
&& \
+ apt-get install -y ./chromium_124.0.6367.60~linuxmint1+virginia_amd64.deb
&& \
Review Comment:
Since we were already using ubuntu as the base image, I didn't want to
change the docs base image like that. we can try shifting the whole thing to
debian instead though, which is a pretty big change. The docs image uses
`ubuntu-cpp` and `ubuntu-python` as bases, I'd have to change those to use
debian instead of ubuntu also.
If you feel like its warranted, give it a try! :smile:
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
