OpenUSD icon indicating copy to clipboard operation
OpenUSD copied to clipboard

Remove ZLib as mandatory dependency

Open dgovil opened this issue 11 months ago • 6 comments

Description of Change(s)

build_usd.py has zlib as a mandatory dependency for USD. However it seems that nothing in the core of USD seems to use it.

Instead, I've moved this to only specific dependencies that say they need it in their READMEs

This further reduces the library dependency count for USD core to just TBB (and Boost if you want Python bindings)

  • [X] I have verified that all unit tests pass with the proposed changes
  • [X] I have submitted a signed Contributor License Agreement

dgovil avatar Mar 05 '24 03:03 dgovil

Filed as internal issue #USD-9389

jesschimein avatar Mar 05 '24 17:03 jesschimein

/AzurePipelines run

jesschimein avatar Mar 05 '24 17:03 jesschimein

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines[bot] avatar Mar 05 '24 17:03 azure-pipelines[bot]

+1 to this - I came to a similar conclusion recently, but never got around to making a PR for it. Thanks @dgovil!

pmolodo avatar Mar 05 '24 20:03 pmolodo

I think you need to add ZLIB when building PTEX as well.

davidgyu avatar Mar 05 '24 21:03 davidgyu

@davidgyu Updated to include it in Ptex.

dgovil avatar Mar 05 '24 22:03 dgovil