Skip to content

Conversation

bisqq
Copy link

@bisqq bisqq commented Sep 25, 2025

Description

This script sets up the development environment by:

  • Creating a Conda environment
  • Installing all required dependencies

Enables contributors to quickly replicate a working environment.

Tests

Checklist:

  • I have read the contribution guidelines.
  • I have updated the documentation, if applicable.
  • I have ensured that the change is tested somewhere in the testsuite (adding new test cases if necessary).
  • My code follows the prevailing code style of this project. If I haven't
    already run clang-format v17 before submitting, I definitely will look at
    the CI test that runs clang-format and fix anything that it highlights as
    being nonconforming.

This script sets up the development environment by:
- Creating a Conda environment
- Installing all required dependencies
- Cloning the forked repository for development

Enables contributors to quickly replicate a working environment.

Signed-off-by: Alexandru Biscoveanu <alex.biscoveanu.gfx@gmail.com>
@lgritz
Copy link
Collaborator

lgritz commented Sep 25, 2025

Maybe put this script in the src/build-scripts directory?

cmake \
llvmdev=20.1.8 clangdev clangxx_linux-64 libcxx \
python=3.12 numpy pybind11 \
openimageio=2.5 imath flex bison pugixml zlib
Copy link
Collaborator

Choose a reason for hiding this comment

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

That's pretty old. Can we use at least OpenImageIO 3.0 by default?

Copy link

Choose a reason for hiding this comment

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

Looks like conda-forge has up to v2.5.18.0 sadly :/
https://anaconda.org/conda-forge/openimageio

Maybe mamba could be an alternative?
https://openmamba.org/en/rpms/base/python-OpenImageIO/aarch64/

Copy link
Author

@bisqq bisqq Sep 26, 2025

Choose a reason for hiding this comment

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

What we could do is, download from the OpenImageIO repo binaries and install them directly

Copy link
Collaborator

Choose a reason for hiding this comment

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

The OpenImageIO repo does not currently provide pre-built binaries.

However, this OSL repo already has a src/build-scripts/build_openimageio.bash that can build OIIO from scratch.

@lgritz
Copy link
Collaborator

lgritz commented Sep 25, 2025

I think this could be pretty useful for people. Can you add some documentation to "INSTALL.md" explaining how to use it?

@lgritz lgritz added the build / testing / port / CI Affecting the build system, tests, platform support, porting, or continuous integration. label Oct 3, 2025
@@ -0,0 +1,51 @@
#!/bin/bash

Copy link
Collaborator

Choose a reason for hiding this comment

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

Let's add the customary copyright notice here.

Suggested change
# Copyright Contributors to the Open Shading Language project.
# SPDX-License-Identifier: BSD-3-Clause
# https://github.com/AcademySoftwareFoundation/OpenShadingLanguage

@lgritz
Copy link
Collaborator

lgritz commented Oct 13, 2025

I think that to wrap up this PR, we're primarily waiting for some modification to INSTALL.md instructions to explain briefly how and when to use it.

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

Labels

build / testing / port / CI Affecting the build system, tests, platform support, porting, or continuous integration. devdays25

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants