Support building OpenSubdiv libraries under Debian (Ubuntu) with our build_deps.sh script.
ClosedPublic

Authored by Thomas Beck (plasmasolutions) on Aug 6 2015, 3:03 PM.

Details

Summary

Use --skip-osd to skip building opensubdiv libs,
use --force-osd to force a rebuild of the libs.

I added all needed user information (where to find the include dir, which -D options to set) to the print_info method that is called at the end too, so it should be pretty intuitive to use.

Diff Detail

Repository
rB Blender
Thomas Beck (plasmasolutions) retitled this revision from to Support building OpenSubdiv libraries under Debian (Ubuntu) with our build_deps.sh script..Aug 6 2015, 3:03 PM
Thomas Beck (plasmasolutions) updated this object.
  • Missed this wrong force option in the last commit.
  • Accidentially changed osl option... reverted it to previous state

Some minor feedback, but @Bastien Montagne (mont29) is the one who is the most familiar with the script.

To me it seems rather fine.

build_files/build_environment/install_deps.sh
537

v3_0_2 is to be replaced with OSD_VERSION?

Hi Sergey,

great,... will wait for mont then and fix the minor flaw in the meantime :)

Greetings and wish you an awesome trip to LA!

build_files/build_environment/install_deps.sh
537

100% ack .. will change that!

Changed the fixed version string (from the archive) to a variable one.

Thomas Beck (plasmasolutions) marked an inline comment as done.Aug 6 2015, 4:22 PM

Solved the fixed version string...weird that I can't commit a "done" item without posting a comment...

Ah, wanted to add this, nice to have a start point, thanks a bunch.

Will check, and add other distro code as well (should not be really hard).

Aside from minor note below, patch lgtm.

build_files/build_environment/install_deps.sh
3156–3158

Replace those three by single:

_2="-D OPENSUBDIV_ROOT_DIR=$INST/osd"

Also, below we lack the scons options, something like:

if [ "$OSD_SKIP" = false ]; then
  PRINT "WITH_BF_OPENSUBDIV = True"
  if [ -d $INST/osd ]; then
    PRINT "BF_OPENSUBDIV = '$INST/osd'"
  fi
fi
This revision is now accepted and ready to land.Aug 7 2015, 9:55 PM
This revision was automatically updated to reflect the committed changes.