GafferHQ / dependencies

All 3rd party dependencies necessary for building Gaffer
33 stars 26 forks source link

Cycles : Update to version 3.4.0 #223

Closed boberfly closed 1 year ago

boberfly commented 1 year ago
johnhaddon commented 1 year ago

Thanks Alex, LGTM. What Gaffer release were you hoping to target this to? We're so close to getting the Windows build into a preview release (of 1.2.0.0) that I'm a little reluctant to add another dependencies update hurdle before we get that done...

boberfly commented 1 year ago

Oh no urgency, I just wanted to test out v3.4.0 on my end and thought I'd put this up for whenever you feel comfortable with merging it.

johnhaddon commented 1 year ago

Coolcool. From what I understand, there's not much in the way of API changes for Cycles 3.4, so there shouldn't be too much to do on the Gaffer side. Is that right?

boberfly commented 1 year ago

Yeah it seemed to compile fine as-is. The Epoxy stuff is Hydra-only so if the USD path config is removed, Epoxy isn't needed.

johnhaddon commented 1 year ago

Closing, since we've updated to 3.4 via https://github.com/GafferHQ/dependencies/pull/226 (without Hydra/Epoxy).