-
Notifications
You must be signed in to change notification settings - Fork 15
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
fix: intel compute runtime packages #102
Conversation
6432c91
to
d0a0e36
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is removing other dependencies we also get from sid. It should only remove intel-opencl-icd
from the command.
1559415
to
8204fb7
Compare
libwebp shouldn't be installed from stable.
df12579
to
bca13d8
Compare
server/Dockerfile
Outdated
RUN if [ $(arch) = "x86_64" ]; then \ | ||
wget https://github.com/intel/intel-graphics-compiler/releases/download/igc-1.0.17193.4/intel-igc-core_1.0.17193.4_amd64.deb && \ | ||
wget https://github.com/intel/intel-graphics-compiler/releases/download/igc-1.0.17193.4/intel-igc-opencl_1.0.17193.4_amd64.deb && \ | ||
wget https://github.com/intel/compute-runtime/releases/download/24.26.30049.6/intel-opencl-icd-dbgsym_24.26.30049.6_amd64.ddeb && \ | ||
wget https://github.com/intel/compute-runtime/releases/download/24.26.30049.6/intel-opencl-icd_24.26.30049.6_amd64.deb && \ | ||
wget https://github.com/intel/compute-runtime/releases/download/24.26.30049.6/libigdgmm12_22.3.20_amd64.deb && \ | ||
dpkg -i *.deb && \ | ||
apt-get install -t testing --no-install-recommends -yqq intel-media-va-driver-non-free && \ | ||
rm *.deb; \ | ||
fi && \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't really like it, it's hard to tell if there are any missing dependencies.
I have something like this for the AIO image, which comes from linuxserver:
if [ $(arch) = "x86_64" ]; then \
echo "**** install intel dependencies ****" && \
apt-get install --no-install-recommends -y \
intel-media-va-driver-non-free \
ocl-icd-libopencl1 && \
INTEL_DEPENDENCIES=$(curl -sX GET "https://api.github.com/repos/intel/compute-runtime/releases/latest" | jq -r '.body' | grep wget | grep -v .sum | grep -v .ddeb | sed 's|wget ||g') && \
mkdir -p /tmp/intel && \
for i in $INTEL_DEPENDENCIES; do \
curl -fS --retry 3 --retry-connrefused -o \
/tmp/intel/$(basename "${i%$'\r'}") -L \
"${i%$'\r'}"; \
done && \
dpkg -i /tmp/intel/*.deb; \
fi && \
Not sure if we need wget https://github.com/intel/compute-runtime/releases/download/24.26.30049.6/intel-opencl-icd-dbgsym_24.26.30049.6_amd64.ddeb
, it's not installed by dpkg -i *.deb
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Interesting! I guess an advantage of this is that you can just point it at a release (or latest) without changing the specific file name of each dependency.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the issue is we don't want all of the dependencies from the release, maybe we could filter which ones we take though
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removed the unused ddeb package, leaving the rest as-is for now
65d1f86
to
ad90e52
Compare
ad90e52
to
a3010f3
Compare
To mirror immich-app/immich#11818