Skip to content

[Devops] Install ISPC into Blender container - #21938

Merged
sarnex merged 1 commit into
intel:syclfrom
sarnex:ispc
May 6, 2026
Merged

[Devops] Install ISPC into Blender container#21938
sarnex merged 1 commit into
intel:syclfrom
sarnex:ispc

Conversation

@sarnex

@sarnex sarnex commented May 5, 2026

Copy link
Copy Markdown
Contributor

Required to build Blender and its deps. We might be able to get rid of the dependency but I'm not sure of the impact and it's a small self contained package so IMO not worth, just include it.

It's already installed on the Windows machines which don't use Docker.

Signed-off-by: Nick Sarnie <nick.sarnie@intel.com>
@sarnex
sarnex marked this pull request as ready for review May 5, 2026 18:36
@sarnex
sarnex requested a review from a team as a code owner May 5, 2026 18:36
@sarnex
sarnex requested a review from uditagarwal97 May 5, 2026 18:36
rm rt.deb

# Install ISPC which is a dep for Blender's CPU device.
RUN wget https://github.com/ispc/ispc/releases/download/v1.30.0/ispc-v1.30.0-linux.tar.gz -O ispc.tar.gz && \

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does Windows machines also have ISPC v1.30.0, installed?
Since we didn't install ISPC ourselves on Windows, should we somewhere check the ISPC version? To make sure that ISPC on Windows machines doesn't change abruptly.

@sarnex sarnex May 6, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry I did install ISPC myself on the Windows machine and it won't change abruptly because we don't use the bundled Blender one. I think its 1.29.0, I will update it before merging this PR, thanks

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sounds good. Thanks!

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just checked, Window is already using 1.30.0, lucky :P

@sarnex
sarnex requested a review from uditagarwal97 May 6, 2026 15:49
@sarnex
sarnex merged commit 6b424de into intel:sycl May 6, 2026
8 of 9 checks passed
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